Expose identity variable for string-based Query calls

Right now there are 2 ways to set up a scope that uses a string query and
Student.scope :in_order, -> (student) { order("#{student}.level_num DESC") }
Student.query_as(:student).in_order(:student)

or
Student.scope :in_order, -> {<EMAIL_ADDRESS>DESC") }
Student.in_order

The second method is much nicer, but you need some magic knowledge of the Neo4j::ActiveNode::Scope::ScopeEvalContext internals.  It would be nice to wrap the @query_proxy.identity as just identity, so this would work:
Student.scope :in_order, -> { order("#{identity}.level_num DESC") }
Student.in_order

...which I saw @subvertallchris mention in chat but it didn't seem to work, so I added it in.
Does this make sense to expose here?  Or did I miss some other idiomatic way to do this somewhere?

Firstly, I'm a bit confused by:
Student.query_as(:student).in_order(:student)

The in_order shouldn't be available to a Query object.  Did you mean as instead of query_as?
So I think this is a very good idea, though in your particular case I think you should be able to do:
Student.scope :in_order, -> { order(level_num: :desc) }

This sort of goes to the Query syntax for order:
http://neo4jrb.readthedocs.org/en/6.1.x/QueryClauseMethods.html#order
When you pass in a hash to order for a QueryProxy it will basically just pass it on down into the Query with identity => argument

hey, thanks for the comments:

yes that should be as instead of query_as

Yeah, my actual case I need this for looks like:

scope :alpha_sorted, -> {<EMAIL_ADDRESS>#{@query_proxy.identity}.imported_name)") }

...but I was trying to stay with the existing doc conventions.  Let me try & make a better example (basically anything that needs a CYPHER function)

Ah, yeah, that would definitely need the identity ;)
Great, I'm happy then with the doc changes

as a side comment I haven't actually taken the time to parse out when as vs query_as will/won't work, I was looking at this which seems hella confusing

Very good point, we should provide better docs for that.  For you and anybody else reading:
query_as is just a way to turn a QueryProxy (or AssociationProxy, but really AssociationProxy is just a wrapper for QueryProxy) into a Query object.  The Query object chaining API is a way to build raw Cypher queries using Ruby like so:
Neo4j::Session.current.query.match('(n:Foo)-[r]-(m:Bar)').where(m: {name: 'Baz'}).return(:n)

Query objects can be simply have exec called on them, but they're also Enumerable and so they'll enumerate over objects that contain whatever you've specified in your one or more return calls.  Of course you can also do pluck instead of return which will immediately return a 2D array.
Query chains can sometimes be overkill for certain queries, but they have a number of advantages:

Uses parameters wherever it can
Allows you to pass along Query objects part-way through and continue building them in different contexts
Sometimes allows you to build your queries with Ruby data without you needing to generate your own Cypher queries

And, of course, the entire ActiveNode association chaining / QueryProxy functionality uses them under the covers.
as, on the other hand, is just a way to turn a model class into a QueryProxy chain so that you can set the identity.  Like:
Model.as(:var)

You only need to use that if you need to refer to the nodes for that model later.
I would actually like to replace the association(:node_var, :rel_var) syntax with association.as(:node_var, :rel_var).  There are some issues where we discuss how to go about it, but it would simplify our code a lot and I don't think it would cause too much of a transition for users of the gem.

Thanks for that explanation.  I'm going to ask the naive questions here, and I'll leave it to you to determine whether I am actually naive or just playing the part ;-)
So as a beginning user, what I would ideally want is a single syntax (as/query_as/etc) that I can use anywhere in the chain to get a handle to the identity, regardless of whether I am calling it on the Model, Query, Query Proxy, etc.
I think this is what your last paragraph is saying?  So we can just use as everywhere in the chain (Model or no)?
On a related topic, I have gotten this far without really understanding the difference at the neo4jrb dev user level between having an AssociationProxy/QueryProxy and having an actual Query object.
I say this because from my standpoint, I just want to build a neo4j query and get some results.  I start with a model (or instance), and call a set of methods against it.  Which layers those methods come from would ideally be opaque to me, so they would be smart enough to know when to convert into Query objects (or not), to convert back if/when needed, etc.
You can argue that I should know what's going on, but as someone who just wants to Get Things Done, I'd ideally want to make it unnecessary unless I'm doing something weirdly out of the ordinary.
Of course, I am discounting all the work required to make that happen (and thank you again for all your work and support on this project so far as well - you guys are super helpful, friendly & responsive, which I really appreciate).
Anyways, since I am coming at it with Beginner's Mind, I thought maybe some questions from here might be helpful to you or others reading along at home.  Thanks again.

Yes, I absolutely think that as much as possible you shouldn't have to care about the underlying implementation and just care about the syntax / API. This is something that I've seen confuse people and I would like to make as smooth as possible.  I definitely welcome your Beginner's Mind to the process ;)
When people start using the gem they can largely be in the "association-chaining" world.  This is the world of AssociationProxy / QueryProxy, but I try to avoid calling it that as much as I can.    I would like to offer more higher-level features where it makes sense, though.  For example until 7.0.0 it was hard to create anything other than a linear association chain using just association chaining.
If, for example, I wanted to find all of the comments on posts for a user, but limit by certain tags for the post I would need to do something like:
# `special_tags` is a hypothetical scope in this case
user.posts(:post).tags.special_tags.query.proxy_as(Post, :post).comments

In 7.0.0 you can now use the branch method:
user.posts.branch { tags.special_tags }.comments

This is a great addition which lets users to accomplish more complex things at a higher level without needing to understand Cypher too much.
So part of our goal with the gem has been to make it so that people can have this easy path into Neo4j and then start using Cypher where it makes sense.  Cypher is a very powerful language and I think there are things that are probably never going to be replicatable at a higher level.  Because of that I think that there's always going to need to be this separate API that users can drop down to when they need more control.
I think a good documentation page on the subject would be a big step forward in making this learning path easier for beginners.  We definitely welcome other suggestions (and confusions).
Separately, as to as:
What I was saying is that I wished you would use as to label your variables throughout your association chain.  Currently, though, it's just for labeling the variables for a model to start a query based on all nodes for that model:
User.posts # user nodes aren't given a variable
User.as(:user).posts # users are given a variable

I would like to see the variable made available throughout like so:
# The syntax
user.posts.as(:post, :rel_var).comments.author.as(:comment_author)
# `rel_var` is the variable between users and posts

# The current syntax
user.posts(:post).comments.author(:comment_author)

I think I actually was working on a branch for that feature but it fell by the wayside


fix: FluentProfileMenu child content not displaying when clicked

🐛 Bug Report
The FluentProfileMenu component in the Fluent UI Blazor library is not displaying the child content when clicked.
💻 Repro or Code Sample
The code can be found in the following GitHub repository: https://github.com/martingagne/fluentblazor-profilemenu
🤔 Expected Behavior
When the FluentProfileMenu is clicked, the child content, which includes the user's name and email, should be displayed.
😯 Current Behavior
The FluentProfileMenu is visible in the UI, but when clicked, the child content does not show.

🌍 Your Environment
OS & Device: Windows 11 on PC
Browser: Firefox, Edge
.NET and Fluent UI Blazor library Version: .NET 8 and Fluent UI Blazor 4.6.1

I don't have time to look at this now.
In the mean time you can compare with the code in de demo site (which is working) which is here in the repo in the examples folder.

Thank you, Vincent (@vnbaaij)! I'm quite new to Blazor and even newer to Fluent UI. I've dedicated a couple hours to this issue and I'm still not making progress. I've double-checked the Fluent UI setup, so maybe it's something else I’m not seeing. I'll take a little break and revisit it with fresh eyes -- perhaps I'll spot something I missed. There’s no rush on this. Whenever you're able to take a look and share any advice, I would greatly appreciate it. Enjoy your weekend!

Found it. You created the site with all the default settings. So with .NET 8 you then get SSR as the standard render mode.... For the ProfileMenu to work, you need interactivity.
(on purpose not spelling out what you need to do to fix it so you can keep learning stuff 😉. Enjoy your weekend too!)

"On purpose not spelling out what you need to do to fix it so you can keep learning stuff"
I'm a beginer in Balzor and Fluent UI so sorry for my questions.
I spent some hours to understand that and It seams that Fluent UI controls can only work in full serverside or clientside rendering mode. Isn't it?
I have also generated app using Your Template but with per Page/Component rendering mode and indeed when FluentProfileMenu is placed in Layout it doesn't work (on Page it works). Unfortunately, it is not enough to add @renderingmode to component itself (beginner's solution).
I have switched to full serverside rendering and now it works but I have strange behavior.
Each time I'm opening menu, page's content is resizing because scrollbars are showing for a moment.
On attached video it blinks only once but it's not true. It blinks every time.
https://github.com/microsoft/fluentui-blazor/assets/77838774/0f8ac5a7-c2d8-4bf8-9efe-4ad7d3eb80e2
By the way. Your Example page is great but too hard to understand at the moment :-).
I will try to use some techniques that You use there in my project.

I spent some hours to understand that and It seams that Fluent UI controls can only work in full serverside or clientside rendering mode. Isn't it?
Most of them render fine in SSR mode but need indeed interactivity to be fully functional. Take for example the DataGrid in our templates on the Weather page. It displays the weather data fine, but you can't sort by clicking on the headers as that code cannot be run without interactivity.


chatroom for contributers to help each other.

Fixes https://github.com/nhsx/COVID-19-app-Android-BETA/issues/20 but for iOS

Thanks. We can't promote an unofficial chat in the README.

LOL :)


Nav bar final

I have made the required changes please merge it under hacktoberfest label

@mahirabidi12 Is there any issue with this PR?

I was working upon making the nav bar responsive as well . I have completed the html and css.

Please open the pull request and add the hacktoberfest tag so that i can upload it .

@mahirabidi12 I have reopened the pull request, you can now do the necessary changes.

Please add the hacktoberfest label as well


Make sbt-jmh an AutoPlugin

Make it an autoplugin :-) http://www.scala-sbt.org/release/docs/Plugins.html

+1

I'm not going to create a PR, because you removed "help wanted" ;-)

Probably by accident :-) please do :-)
On Apr 29, 2015 7:03 PM, "Heiko Seeberger"<EMAIL_ADDRESS>wrote:

I'm not going to create a PR, because you removed "help wanted" ;-)
—
Reply to this email directly or view it on GitHub
https://github.com/ktoso/sbt-jmh/issues/14#issuecomment-97505301.

Did it. See #35.


Same email registration over different tenants

I have a situation where I need an user to be able to register separately with two different tenants. I'm being able to restrict login one user of a tenant to other, but not to register same user to two different tenants.
Thanks!

You could allow for duplicate users in the database linked to different tenant_ids and when you recognize that a user linked to multiple tenants has entered their email/username (ajax check username on change) and present a dropdown of the different tenants and then use that to "route" to the right user id.
Makes sense?

How do I allow duplicate users linked to different tenant_ids? Do I need to customize laravel auth for that? Or, is there anything useful in Landlord?

You need to customize the default laravel auth.

Any reference for that? Or, a brief idea?

Add a route that does this exact thing:

Fetch username from request
Search user table for that username
If returns more than 1 entry return the tenants that match.

In the logincontroller do this:
Override the login method:

    protected function getMatchingUser($tenantId, $username, $password)
    {
        // Either username or email, whatever you're using
        return User::where("username", $username)
            ->where("tenant_id", $tenantId)
            ->where("password", bcrypt($password))
            ->get()->first();
    }

    public function login(Request $request)
    {
        // Get the user that matches the tenant, the tenant_id
        // should be passed in your login form when you detect that
        // the user trying to login has multiple accounts,
        // i suggest performing an ajax check before submitting the
        // login form that checks this and returns a list of tenants
        // and show these in a dropdown (<select name="tenant_id">),
        // the user then has to which tenant to login to.
        $user = $this->getMatchingUser(
            $request->get("tenant_id"),
            $request->get("username"),
            $request->get("password")
        );

        if ($user) {

            // We simply login the user with the login method, so we can specify
            // the user model to login, we've already "authenticated" the user in the previous call
            $this->guard()->login($user);
            return $this->sendLoginResponse($request);
        }

        // If the login attempt was unsuccessful we will increment the number of attempts
        // to login and redirect the user back to the login form. Of course, when this
        // user surpasses their maximum number of attempts they will get locked out.
        $this->incrementLoginAttempts($request);

        return $this->sendFailedLoginResponse($request);
    }

In the registerController, simply remove the validation that forces the email to be unique.
If you've got any question feel free to ask.

The idea is that when a user that has multiple accounts linked to different tenants with the same username/email, he will be notified on login (ajax call to the endpoint i mentioned that returns matching tenants), it populates a dropdown which the tenants he can login to, and the user is then logged into that tenant in the LoginController.

Hey thanks so much for helping! :) Though, I want to point out one thing that, Landlord pretty much helps in logging in tenant specific users. Only problem is with registering duplicate ones. And I think I got a solution, and am trying to implement it.
As you said to remove unique validation, I would rather prefer to use custom validation rule which laravel provides as below (yet not implemented, ongoing):
'email' => Rule::unique('users')->where(function ($query) {
    $query->where('account_id', 1);
})


Problem with guid being repeated

Not urgent, it is solved with recomputing or reloading GH-file

Not sure of the context here, but if you the Loads were modified from the FdModel you need to set Overwrite to True.

I'm not changing anything, just adding supports and loads. I put the files here:
C:\Users\JohannaRiad\StruSoft AB\StruParam - Projects\femdesign-api\Files for debugging\Guid problem
It works when you open the file, but if you try removing or adding the cables to loads for example you should be able to replicate the error

On which build are you experiencing this?

Right now I have the master build installed and get it on that. I think I got it on 20.2.0 as well yesterday

Did not clone model causing elements to be added several times on re-run.

Fixed on 20.2.0


Incorrect return type when slicing a dataframe with a mask (Series[bool])

Describe the bug
mypy seems to think the return type of filtering a dataframe using a series of booleans is a series, not a dataframe. I only get this error with mpy, not pyright.
To Reproduce
A small example:
def filter_dataframe(dataframe: pd.DataFrame) -> pd.DataFrame:
    filtered_dataframe = dataframe[~dataframe["Column"].isna()]
    return filtered_dataframe

This should be fine, but MyPy provides the following:
error: Incompatible return value type (got "Series[Any]", expected "DataFrame")  [return-value]

Please complete the following information:

OS: [e.g. Windows, Linux, MacOS]
Windows 11, running Ubuntu using WSL2
OS Version [e.g. 22]
20.04.2 LTS
python version
3.9.14
version of type checker
mypy==0.991
version of installed pandas-stubs
pandas-stubs==<IP_ADDRESS>124

I can't replicate this.  I used this example:
import pandas as pd

def filter_dataframe(dataframe: pd.DataFrame) -> pd.DataFrame:
    filtered_dataframe = dataframe[~dataframe["Column"].isna()]
    reveal_type(dataframe["Column"].isna())
    reveal_type(~dataframe["Column"].isna())
    reveal_type(filtered_dataframe)
    return filtered_dataframe

Got this result:
filter.py:5: note: Revealed type is "pandas.core.series.Series[builtins.bool]"
filter.py:6: note: Revealed type is "pandas.core.series.Series[builtins.bool]"
filter.py:7: note: Revealed type is "pandas.core.frame.DataFrame"
Success: no issues found in 1 source file

I did use mypy 0.991 as well.  Double check that you don't have any settings in pyproject.toml or mypy.ini file - I didn't use any.

@Dr-Irv thanks for this - it turns out this issue popped up because I was using the mypy extension for pandera. I wouldn't have expected this extension to cause issues with the existing pandas typing - I would have expected it to only touch the pandera-specific constructs.
I'll close this issue now as it is clearly not a pandas-stubs issue.


[Orielly] Log in with cookies

Checklist

[ ] I'm reporting a bug unrelated to a specific site
[ ] I've verified that I'm running yt-dlp version 2021.09.02
[ ] I've checked that all provided URLs are alive and playable in a browser
[ ] The provided URLs do not contain any DRM to the best of my knowledge
[ ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar bug reports including closed ones
[ ] I've read bugs section in FAQ

Verbose log

[debug] Command-line config: ['--config-location', 'D:\\Utils/YoutubeDL/Configs/ytdlp_oreilly.conf', '--playlist-items', '2']
[debug] Batch file urls: ['https://learning.oreilly.com/videos/the-principles-of/9781491935811/']
WARNING: You have asked for unplayable formats to be listed/downloaded. This is a developer option intended for debugging. If you experience any issues while using this option, DO NOT open a bug report
WARNING: --add-metadata is ignored since --allow-unplayable-formats was given
WARNING: --embed-subs is ignored since --allow-unplayable-formats was given
WARNING: --embed-thumbnail is ignored since --allow-unplayable-formats was given
[Cookies] Extracting cookies from firefox
[debug] Extracting cookies from: "C:\Users\mh-sh\AppData\Roaming\Mozilla\Firefox\Profiles\8tceuger.default-release\cookies.sqlite"
[Cookies] Extracted 1589 cookies from firefox
[debug] Loading archive file 'archive.txt'

[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] yt-dlp version 2021.09.02 (source)
[debug] Plugin Extractors: ['SamplePlugin']
[debug] Git HEAD: 57aa7b851
[debug] Python version 3.9.1 (CPython 64bit) - Windows-10-10.0.19041-SP0
[debug] exe versions: ffmpeg n4.4-4-gacb339bb88, ffprobe n4.4-4-gacb339bb88
[debug] Optional libraries: sqlite
[debug] Proxy map: {}
[debug] [safari:course] Extracting URL: https://learning.oreilly.com/videos/the-principles-of/9781491935811/
[safari:course]<PHONE_NUMBER>811: Downloading course JSON
[download] Downloading playlist: The Principles of Microservices
[info] Writing playlist metadata as JSON to: The Principles of Microservices\0. The Principles of Microservices.info.json
WARNING: There's no playlist description to write.
[safari:course] playlist The Principles of Microservices: Collected 15 videos; downloading 1 of them
[download] Downloading video 1 of 1
[debug] [safari:api] Extracting URL: https://learning.oreilly.com/api/v1/book/9781491935811/chapter/video221406.html
[safari:api]<PHONE_NUMBER>811/video221406: Downloading part JSON
[debug] [safari] Extracting URL: https://learning.oreilly.com/videos/the-principles-of/9781491935811/9781491935811-video221406
[debug] [Kaltura] Extracting URL: https://cdnapisec.kaltura.com/html5/html5lib/v2.37.1/mwEmbedFrame.php?wid=_1926081&uiconf_id=29375172&flashvars%5BreferenceId%5D=9781491935811-video221406
[Kaltura]<PHONE_NUMBER>811-video221406: Downloading webpage
[Kaltura] 0_1i6jb4o4: Downloading video info JSON
[Kaltura] 0_1i6jb4o4: Checking mp4-4468 URL
[Kaltura] 0_1i6jb4o4: Downloading m3u8 information
WARNING: [Kaltura] Ignoring subtitle tracks found in the HLS manifest; if any subtitle tracks are missing, please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; see  https://github.com/yt-dlp/yt-dlp  on how to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[debug] Downloading subtitles: en
[info] Available formats for 0_1i6jb4o4:
ID       EXT RESOLUTION FPS |  FILESIZE    TBR PROTO  | VCODEC    VBR ACODEC     ABR MORE INFO
-------- --- ---------- --- - ---------- ----- ------ - ------- ----- --------- ---- ---------
hls-58   mp4 audio only     |              58k m3u8_n |               mp4a.40.2  58k
mp4-56   mp4 audio only 0   | ~2.16MiB     56k http   |               unknown    56k isom
hls-425  mp4 640x360        |             425k m3u8_n | unknown  425k unknown     0k
hls-571  mp4 640x360        |             571k m3u8_n | unknown  571k unknown     0k
hls-782  mp4 640x360        |             782k m3u8_n | unknown  782k unknown     0k
mp4-385  mp4 640x360    29  | ~14.80MiB   385k http   | avc1     385k unknown     0k isom
mp4-525  mp4 640x360    29  | ~20.20MiB   525k http   | avc1     525k unknown     0k isom
mp4-726  mp4 640x360    29  | ~27.90MiB   726k http   | avc1     726k unknown     0k isom
hls-1319 mp4 1280x720       |            1319k m3u8_n | unknown 1319k unknown     0k
mp4-1240 mp4 1280x720   29  | ~47.60MiB  1240k http   | avc1    1240k unknown     0k isom
mp4-1966 mp4 1280x720   29  | ~75.40MiB  1966k http   | avc1    1966k unknown     0k isom
mp4-3082 mp4 1920x1080  29  | ~118.00MiB 3082k http   | avc1    3082k unknown     0k isom
mp4-4468 mp4 1920x1080  29  | ~171.00MiB 4468k http   | avc1    4468k unknown     0k mp42
[download] Finished downloading playlist: The Principles of Microservices

Description

I tried download oreilly videos using cookies but the downloader is only invoked on the trial duration meaning that the cookies weren't included in the requests and we are treated as not logged in.
The download using user/pass works tho
Relevent info: here

I faced the same issue


[DOCS] add instructions on how to add partitions for slurm in example walkthrough

In this PR, I added a bit intructions in example walkthrough regarding how to add request for partitions on Slurm clusters. See rendered docs of PR here.


[otbn] Regression failure: Offending '$stable(lsu_addr_i)'

The regression that ran at Saturday April 24 2021 02:36:14 UTC shows the following error:
Offending '$stable(lsu_addr_i)':

    47.otbn_single.644070143

      Line 51, in log /usr/local/google/home/chencindy/nightly_openTitan/master/otbn-sim-vcs/47.otbn_single/out/run.log
          Offending '$stable(lsu_addr_i)'
      UVM_ERROR @   4706788 ps: (otbn_lsu.sv:98) [ASSERT FAILED] LsuLoadAddrStable
      UVM_INFO @   4706788 ps: (uvm_report_server.svh:901) [UVM/REPORT/SERVER]
      --- UVM Report Summary ---

This seems to be because we're using x1 as the base address for a load. The "value" of x1 changes after the first cycle, leading to the assertion failure. Here's the relevant part of the instruction trace (from running the ISS on the binary):
00000000 | (stall)                                       | []
00000000 | (stall)                                       | []
00000000 | and            x9, x0, x0                     | [x09 = 0x00000000]
00000004 | sll            x17, x0, x9                    | [x17 = 0x00000000]
00000008 | and            x1, x17, x9                    | [x01 = 0x00000000]
0000000c | jalr           x1, x0, 32                     | [x01 = 0x00000010, pc = 0x20]
00000020 | andi           x0, x17, -174                  | []
00000024 | (stall)                                       | []
00000024 | lw             x18, 1388(x1)                  | [x18 = 0xd5964dc5]

And here are the LSU signals, together with the instruction address:

To reproduce: util/dvsim/dvsim.py hw/ip/otbn/dv/uvm/otbn_sim_cfg.hjson -i otbn_single --fixed-seed 644070143 -w.
I suspect that this is caused by the assertion being too strong. Presumably, we don't care about the LSU address changing after we've fired off the load? @GregAC

Presumably, we don't care about the LSU address changing after we've fired off the load?

We do, the address is being supplied by the controller and on the first cycle is used to generate the request but on the second cycle (when we have data) is used to mux out the correct word from the 256-bit result. So things break when you use it with the call stack (assumption being that there will be no register file writes that would alter it).
So this is a bug that needs fixing. I think I'll go with flopping the required address bits to do the muxing. Minimal area costs and reduces complexity vs the alternative (holding the pop until the load data is returned).

Oh! I just didn't understand the comment in the code. That makes sense, and sounds like a good fix to me too.


Namespace or Func Prefix?

Will we use Namespaces or simple prefixes for differentiation between modules?
Also we should discuss what 'global' Namespace/Prefix name we want to use?
I propose something like DUSK or DSKfor Namespace names or dsk for prefix.

We dont know what perdime we will be using so we can assume OOP therefore there might not even be a Name space
I realy like how rust deals with name-spaces where the file it self is the name space.


GitHub Actions fails to push hyp3-isce2 image to GHCR with 403 Forbidden (again)

Jira: https://asfdaac.atlassian.net/browse/TOOL-2191
Note: The above link is accessible only to members of ASF.

The dockerize workflow is failing for https://github.com/ASFHyP3/hyp3-isce2/pull/134 with the following error:
Error: buildx failed with: ERROR: failed to solve: failed to push ghcr.io/asfhyp3/hyp3-isce2:0.7.2.dev2_gb21307f: unexpected status from POST request to https://ghcr.io/v2/asfhyp3/hyp3-isce2/blobs/uploads/: 403 Forbidden

This is very similar to https://github.com/ASFHyP3/hyp3-isce2/issues/120, where the error was as follows:
Error: buildx failed with: ERROR: failed to solve: failed to push ghcr.io/asfhyp3/hyp3-isce2:0.6.2.dev5_g3db5604: unexpected status: 403 Forbidden

Since the previous issue seemed to resolve itself, I figured this one would, too. But the workflow has now been failing for about a week over several different re-runs, so we should probably try to debug it. Might be worth revisiting the steps that I took in my attempts to debug the previous issue.

Inspecting the logs for https://github.com/ASFHyP3/hyp3-isce2/actions/runs/6004893749/job/16558054441?pr=134 I noticed that under the Set up Docker Buildx step (defined here) there are errors under Booting builder:
  #1 21.39 error: failed to list workers: Unavailable: connection error: desc = "transport: error while dialing: dial unix /run/buildkit/buildkitd.sock: connect: no such file or directory"

Error:    Get "http://%2Fvar%2Frun%2Fdocker.sock/v1.24/containers/buildx_buildkit_builder-bc3fa86a-28c5-46bd-87ed-eb905bf2f0db0/json": context deadline exceeded

I also re-opened https://github.com/ASFHyP3/hyp3-isce2/pull/121 (a dummy PR that I previously used to debug this issue) and the dockerize workflow succeeded here. I don't see any errors under Booting builder in the logs. So I suspect that the 403 error is related to the Booting builder errors.
However, before attempting to debug those errors, I want to see if I can just get the failing dockerize workflow to succeed for the dependabot version bump, either by closing and re-opening the PR, or manually opening a PR to make the same changes. I suspect that there is something wrong with the GitHub Actions instance being used to run the dockerize workflow for that PR, so if we can get it to run on a different instance, it should succeed.

Yep, closing and re-opening the PR allowed the dockerize workflow to succeed.

Interestingly, https://github.com/ASFHyP3/hyp3-isce2/pull/135 is also failing for dockerize with a 403 error (see here) but I see no errors under Booting builder in the logs. I will first try updating the branch (merging develop back in) and if that doesn't allow the dockerize job to succeed, I'll try closing and re-opening the PR.

Updating the branch did the trick.

To conclude, the 403 error has resolved itself as it did last time, but in the future, if re-running the failed job doesn't work, then we should try closing the PR / deleting the branch, then restoring the branch / re-opening the PR. If that doesn't work, my next step would be to create a new branch with identical changes and open a new PR. The goal is to get the workflow to run in a brand-new environment, as I suspect there is something wrong with the underlying GitHub Actions runner (virtual machine) in the cases where the 403 error persists across multiple job re-runs.

@jtherrmann is it worth investigating other docker setup/build actions? Could it just be a Buildx problem?

@jhkennedy Maybe, though I believe I was noticing some other actions failing with HTTP errors around the same time, so it could just be a GitHub Actions problem. Let's wait and see if it continues to be a nuisance.


entering a comment redirects to wp-comments-posts.php

I tried entering a comment and got redirected to wp-comments-posts.php, and comment never actually went through. I've tried disabling plugins. I also fresh-installed the theme on wordpress and it still does this. I also tried commenting with a a default theme (twenty-fifteen) and that did work. So Its go to be a theme issue as far as I can tell.

Unable to recreate issue on latest version of FP. Closing issue.


Fix #17

Fixes #17.

Took a look at this. I get how it works, but is it necessary to pass nulls this deeply into the code just to end up retrieving default values that produce a no-op analysis? Wouldn't it be easier to just set an enabled flag somewhere when passed the required configuration, and then check that flag at the start of each Scanner.matchX method and do nothing if it's false?

@lazaroclapp as we discussed, I mistakenly assumed that the Checker constructor is called when the checker is off. The actual problem was that a different name space was used when sending error prone flags. The better way is to still throw an Exception if ConfigPath is null. Will reopen #17  now and update the code.


Implement workload identity and application default credentials for GCP object store.

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

Currently the only authentication implemented for GCP is using service account tokens. Implement the workload identity and application default credentials.
Describe the solution you'd like

An application using the object_store crate should be able to use the additional credential mechanisms.
Describe alternatives you've considered

Additional context

@tustvold this is the more specific issue we were talking about.

label_issue.py automatically added labels {'object-store'} from #3541


[ZEPPELIN-696] Add notification system for AngularJS z functions

What is this PR for?
Add notification system for AngularJS z functions. Now that we expose the z Angular object, we should also catch error/exceptions and display error messages properly to the user.
I changed a little bit the default settings for ngToast. Instead of having notifications on bottom right, I let the default value undefined.
For Interpreter page notification, I force the position to bottom right as before when using default config.
For z functions error messages, I choose to display notifications on top right corner because it's clearer. See screen capture
This is a sub-task of epic ZEPPELIN-635
What type of PR is it?
[Improvement]
Todos

[ ] - Code Review
[ ] - Simple Test

Is there a relevant Jira issue?
ZEPPELIN-696
How should this be tested?

git fetch origin pull/744/head:AngularJSNotification
git checkout AngularJSNotification
mvn clean package -DskipTests
bin/zeppelin-daemon.sh restart
Create a new note
In the first paragraph, put the following code

%angular

<form class="form-inline">
  <button type="submit" class="btn btn-primary" ng-click="z.runParagraph()"> Run Paragraph Without Id</button>
  <button type="submit" class="btn btn-primary" ng-click="z.runParagraph('20160223-141919_305734436')"> Run Paragraph Without Wrong Id</button>
  <button type="submit" class="btn btn-primary" ng-click="z.angularBind('superhero', superhero)"> Angular Bind Without Paragraph Id</button>
  <button type="submit" class="btn btn-primary" ng-click="z.angularUnbind('superhero')"> Angular UnBind Without Paragraph Id</button>
</form>

Click on each button to see the appropriate error message

Screenshots (if appropriate)

Questions:

Does the licenses files need update? --> No
Is there breaking changes for older versions?  --> No
Does this needs documentation?  --> Yes

We already have ngtoast for that

Right, didn't know that we had ngToast. I never see it in action. Will remove angular-growl-2

@corneadoug @Leemoonsoo
Rebased from master, only JS change, can be reviewed and merged quickly.
Do you want me to add also a test in ZeppelinIT to test for the presence at the notification in case of error ,

I have tested and working well.
Looks good to me.
@corneadoug Can you take a look, too?

@corneadoug ping

Can we merge this PR so I can move to the next sub-task ?

ping @Leemoonsoo @corneadoug

@doanduyhai LGTM


Bug installing .Net 3 from source using windows_feature_dism

Cookbook version
4.0.1
Chef-client version
12.22.1
Platform Details
Windows 2012R2
Scenario:
I'm trying to install .Net Framework versions 3.5 from source
Steps to Reproduce:
windows_feature 'install .Net Framework versions: 3.5, 3.0 & 2.0' do
  feature_name 'NetFx3'
  source 'C:\temp\sxs'
  action :install
  all true
end

Expected Result:
NetFx3 is installed
Actual Result:
It says that its already installed.
windows_feature[install .Net Framework versions: 3.5, 3.0 & 2.0] action install[2018-03-14T13:55:18+11:00] INFO: Processing windows_feature[install .Net Framework versions: 3.5, 3.0 & 2.0] action install (test_windows::default line 34)

           * windows_feature_dism[install .Net Framework versions: 3.5, 3.0 & 2.0] action install[2018-03-14T13:55:18+11:00] INFO: Processing windows_feature_dism[install .Net Framework versions: 3.5, 3.0 & 2.0] action install (C:/Users/ADMINI~1/AppData/Local/Temp/kitchen/cache/cookbooks/windows/resources/feature.rb line 47)
        (up to date)
            (up to date)

However, dism /Get-Features /Online /Format:Table /English shows
...
NetFx3  | Disabled with Payload Removed
...

Other observations:
features_to_install does the intersection of what is asked for and node['dism_features_cache']['disabled'] (See https://github.com/chef-cookbooks/windows/blob/master/resources/feature_dism.rb#L83)

Since NetFx3 is in node['dism_features_cache']['removed'], its not installed.
I think the intersection should be between what is asked for, and the combination of node['dism_features_cache']['disabled'] and node['dism_features_cache']['removed']

Also, I noticed that all_available is defined differently in 2 places:

https://github.com/chef-cookbooks/windows/blob/master/resources/feature_dism.rb#L96
https://github.com/chef-cookbooks/windows/blob/master/resources/feature_dism.rb#L109

Having the same problem on Server 2016 after upgrading to the latest version. this was working fine on version 3

Looks like I handled the source scenario correctly in the method that fails if unavailable, but then I missed it entirely when I calculated the list of features to install. I don't actually have a windows system in a state where I can fully test this so I'm hoping that someone can give this code a try to see if it correctly installs removed features when specifying the source.
https://github.com/chef-cookbooks/windows/pull/541

@tas50 Tested this on Windows server 2016 and .net 3.5sp1 installs fine with DISM now.

Perfect. Thanks for checking. I'll get this released right now and I already opened a PR to make sure the version in Chef 14 has this fix.


Logger

Issue description
Describe your issue in detail here
Implement a proper logger to replace console.log & console.error
Definition Of Ready (DoR)
This issue can be worked on if

[x] a developer is assigned
[x] the issue is labeled
[x] the issue is assigned to the current sprint
[x] the issue is estimated
[x] a meaningful title has been set
[x] the issue is described in detail

Having something like this would also be quite useful to trace issues like the tesseract workers crashing

We could build a simple logger ourself: https://adrianhall.github.io/cloud/2019/06/30/building-an-efficient-logger-in-typescript/
Or we maybe could use a library which will make things also easier: https://tslog.js.org/#/

I'd go with a dedicaed library

When creating a composable for it stick to the current conventions
files: use....Name
export default function use...Name
just look into the other composables

Maybe documenting somewhere where you can find the log file ?


Update to bazel 5.0.0

Bazel 5 used to generate build errors on our code:
https://github.com/3rdparty/eventuals-grpc/issues/65
but that was fixed with a newer gRPC release:
https://github.com/3rdparty/eventuals/pull/247

I plan to pull in https://github.com/3rdparty/dev-tools/pull/33 once it's submitted by updating the dev-tools module in this PR before merging.


pugixml: always include pkgconfig file

In master, this is always enabled: https://github.com/zeux/pugixml/pull/193
Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build pugixml


httperror

HI I am getting HTTPerror when I try to execute this block. What is wrong with this?
  - name: Update Image Version in the yaml
    uses: fjogeleit/yaml-update-action@main
    with:
      valueFile: 'dev/deployment-manifests/book-info-service-deployment.yaml'
      propertyPath: 'spec.template.spec.containers[1].image'
      value: 'abvijaykumar/bozo-book-info-service:${{github.event.client_payload.buildnumber}}'
      message: 'Update Image Version to abvijaykumar/bozo-book-info-service:${{github.event.client_payload.buildnumber}} '

Here is what I see on the github actions
Run fjogeleit/yaml-update-action@main
with:
valueFile: dev/deployment-manifests/book-info-service-deployment.yaml
propertyPath: spec.template.spec.containers[1].image
value: abvijaykumar/bozo-book-info-service:46
message: Update Image Version to abvijaykumar/bozo-book-info-service:46
branch: master
masterBranchName: master
targetBranch: master
githubAPI: https://api.github.com/
createPR: false
commitChange: true
updateFile: false
labels: yaml-updates
token: ***
repository: abvijaykumar/book-library-kube-infra
workDir: .
commitUserName: GitHub Actions
commitUserEmail<EMAIL_ADDRESS>Error: HttpError: Not Found

You pushing in the same repository as your source repository? Is it protected some how?
Depending on your configuration its maybe needed to create a GitHub Token for the action because the default one does not have the permissions to push.
So, I would recommend to create a custom GitHub token and use it instead of the default one.

Thanks for the response. Yes I am updating a file in the same repo. Let me try the token.

Hi - I update my code to pass toke, still I am getting the same error.

  - name: Update Image Version in the yaml
    uses: fjogeleit/yaml-update-action@main
    with:
      valueFile: 'dev/deployment-manifests/book-info-service-deployment.yaml'
      propertyPath: 'spec.template.spec.containers[1].image'
      value: 'abvijaykumar/bozo-book-info-service:${{github.event.client_payload.buildnumber}}'
      token: ${{secrets.API_TOKEN_GITHUB}}
      message: 'Update Image Version to abvijaykumar/bozo-book-info-service:${{github.event.client_payload.buildnumber}} '

Your default branch is main, right? Because of compatibility reasons the default branch of the action is master. You can configure it by using the masterBranchName property.

yes I just tried that, still I am getting the same error :-D I am sorry to bother u

Can you please also set the branch to main? This is the actual branch where it tries to commit.
No worries, this action can be a bit hard to configure, depending on the individual needs. Glad to help with it.

Hi. I already tried branch and targetBranch. Gave both the values to main. Still it doesn't seem to work. Just to make sure, can you give me an example. I am using single quotes. Is that right ?

      - name: Update Chart Values
        uses: fjogeleit/yaml-update-action@main
        with:
          valueFile: 'fints-service/values.yaml'
          propertyPath: 'image.tag'
          value: ${{ steps.tags.outputs.version }}
          repository: 'betreu-platform/charts'
          token: ${{ secrets.CHART_TOKEN }}
          masterBranchName: main
          branch: main
          createPR: 'false'
          message: 'Update Image Version to ${{ steps.tags.outputs.version }}'
          workDir: infrastructure

thats an example from one of my projects. Quotes should not matter

I created a fork and check something

So my fork is working as expected.
      - name: Update Image Version in the yaml
        uses: fjogeleit/yaml-update-action@main
        with:
          valueFile: 'dev/deployment-manifests/book-info-service-deployment.yaml'
          propertyPath: 'spec.template.spec.containers[0].image'
          value: 'abvijaykumar/bozo-book-info-service:${{ inputs.number }}'
          masterBranchName: 'main'
          branch: 'main'
          message: 'Update Image Version to abvijaykumar/bozo-book-info-service:${{ inputs.number }}'

https://github.com/fjogeleit/book-library-kube-infra
Have you tried to remove your custom token after changing the branch and masterBranchName? Maybe there is an issue with the token. When the default token also fail, the last thing which comes to my mind is to try a new token with all available permissions, to check if its an permission issue at all.
And a small hint, the index of your propertyPath is wrong. It starts with 0 instead of 1

awaesome it worked for me, Thank you very much!!!

Great, no Problem. Let me know if you encounter any other issue


fix: gosec issues

Description
Fix the new gosec issues in the CI
Using nosec for false positive.
Adding timeout for websocket server.
Need confirmation for the following values:

read timeout - 15sec
write timeout - 15sec
idle timeout - 1 minute

Pinging @zeta-chain/devops in case
Closes: https://github.com/zeta-chain/node/issues/1288

I'm not 100% sure if the idleTimeout will impact log filter users. We can give it a try and see.

Maybe we can increase it to 5min?


ERROR (nni.runtime.msg_dispatcher_base/Thread-1)

dispacher log:
ERROR (nni.runtime.msg_dispatcher_base/Thread-1) 68
Traceback (most recent call last):
File "/home/mm/anaconda3/envs/CLNode_env/lib/python3.8/site-packages/nni/runtime/msg_dispatcher_base.py", line 108, in command_queue_worker
self.process_command(command, data)
File "/home/mm/anaconda3/envs/CLNode_env/lib/python3.8/site-packages/nni/runtime/msg_dispatcher_base.py", line 154, in process_command
command_handlerscommand
File "/home/mm/anaconda3/envs/CLNode_env/lib/python3.8/site-packages/nni/runtime/msg_dispatcher.py", line 148, in handle_report_metric_data
self._handle_final_metric_data(data)
File "/home/mm/anaconda3/envs/CLNode_env/lib/python3.8/site-packages/nni/runtime/msg_dispatcher.py", line 201, in handle_final_metric_data
self.tuner.receive_trial_result(id, trial_params[id], value, customized=customized,
File "/home/mm/anaconda3/envs/CLNode_env/lib/python3.8/site-packages/nni/algorithms/hpo/tpe_tuner.py", line 197, in receive_trial_result
params = self._running_params.pop(parameter_id)
KeyError: 68
NNImanager log:
ERROR (WsChannel.default) Channel closed. Ignored command { type: 'GE', content: '1' }
[2024-01-25 11:08:42] WARNING (WsConnection.default) Missing pong
[2024-01-25 11:08:47] WARNING (WsConnection.default) Missing pong
[2024-01-25 11:08:47] ERROR (WsConnection.default) Failed sending command. Drop connection: Error: WebSocket is not open: readyState 3 (CLOSED)
at sendAfterClose (/home/mm/anaconda3/envs/CLNode_env/lib/python3.8/site-packages/nni_node/node_modules/express-ws/node_modules/ws/lib/websocket.js:988:17)
at WebSocket.send (/home/mm/anaconda3/envs/CLNode_env/lib/python3.8/site-packages/nni_node/node_modules/express-ws/node_modules/ws/lib/websocket.js:405:7)
at node:internal/util:375:7
at new Promise ()
at bound send (node:internal/util:361:12)
at WsConnection.sendAsync (/home/mm/anaconda3/envs/CLNode_env/lib/python3.8/site-packages/nni_node/common/command_channel/websocket/connection.js:92:16)
at WsConnection.heartbeat (/home/mm/anaconda3/envs/CLNode_env/lib/python3.8/site-packages/nni_node/common/command_channel/websocket/connection.js:144:18)
at listOnTimeout (node:internal/timers:569:17)
at process.processTimers (node:internal/timers:512:7)

I ran into the same problem, ran several trials and got this error. I found out that it was due to running NNI in the screen command.

Yes, you can't run NNI in the tmux either.


[Bug] Wrong padding of Message element when widget is loading or empty

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[X] Not reproducible in the monaco editor playground

Monaco Editor Playground Code
No response
Actual Behavior
The vertical padding of the message element hides the border of the outside container.

Expected Behavior
The message element is rendered totally inside of the outside container.

Additional Context
This bug happened when I upgrade monaco-editor from version 0.21.2 to version 0.22.0. Currently, I need to customize the padding of message element by myself but I think it would be better if monaco-editor fix it inside their code.

Another problem is that I tried to upgrade monaco-editor to the latest version, which is 0.33.0, the message element is even rendered outside of the container (as I can see in the picture below).

The workaround by customizing the vertical padding of the message element (padding-top: 0; padding-bottom: 0) that I mentioned above did not work as expected anymore.

As you can see, the content of the message element doesn't align center in a vertical direction anymore.

In version 0.33.0, I finally can align correct the message element by adding this workaround style:
position: absolute;
top: 0;


Update github action versions.

🎉 New feature
Part of https://github.com/maliput/maliput_infrastructure/issues/319
Summary
Update github action versions.
Test it
N/A
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if it affects the public API)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

I triggered again the cmake build job. It is failing to install with rosdep foxy dependencies.


adapter.getMessage is deprecated and will be removed in js-controller 2.3

Warning in Log for upcoming JS-Controller version 2.3 with hint: "Please report this warning to the developer of this adapter!"

Should be already fixed in 1.0.6 (latest repo). Can you please verify?

Correct.
But I had to update manually to 1.0.6, even though I'm on latest channel. No hint for automatically update from 1.0.5.


Placeholder argument for power-select-multiple control

Passing the placeholder argument from the power-select-multiple control to the ember power select.

Released as v4.1.0


Client Allocation Request for: Filecoin Discover

Client Allocation Questions
Core Information

Name: Filecoin Discover
Website / Social Media: https://store.filecoin-discover.com/
DataCap Requested: 2TiB
Addresses to be Notarized: Multiple ( see github comments )

Hey @ribasushi - thanks for your request. Can you kindly add more information / context here on:

yourself , e.g. include links to social media / news mentions etc
your history working on IPFS / Filecoin, e.g. notable contributions to the community, type of engagement shown over time etc
type and source of the data you (and by extension other Discover participants) are looking to store on Filecoin, e.g. public data vs private data etc
how you intend to store the data on Filecoin, e.g. redundancy, location / miner preferences, etc

Thanks

Hi @philippbanhardt, please see the answers below!

yourself , e.g. include links to social media / news mentions etc

I am a contributor at Protocol Labs, mainly working with IPLD. My Filecoin-slack handle ( like my github ) is ribasushi. I have been involved with and contributing to the IPFS/IPLD/Filecoin ecosystems since 2016.
I am applying on behalf of the Filecoin Discover (https://store.filecoin-discover.com/) project. It is a Protocol Labs sponsored initiative to seed the Filecoin network with datasets important for humanity. This project was first publicly announced in May of 2020 (https://filecoin.io/blog/intro-filecoin-discover/), with a community currently consisting of 534 participants in the #fil-discover-support slack room in the Filecoin-slack.

your history working on IPFS / Filecoin, e.g. notable contributions to the community, type of engagement shown over time etc

Filecoin Discover is a Protocol Labs sponsored project to seed the Filecoin network with important datasets for humanity. This project was first publicly announced in May of 2020 (https://filecoin.io/blog/intro-filecoin-discover/). The aim of this project is to store petabytes of valuable data on the Filecoin network. As a result of the preparatory work, a number of useful features have been tested within the Filecoin tooling such as offline deals. A number of other pieces of our tooling have been open sourced as a result of this project, e.g.: https://github.com/chafey/dumbo-drop#readme

type and source of the data you (and by extension other Discover participants) are looking to store on Filecoin, e.g. public data vs private data etc

Filecoin Discover aims to store data from a variety of categories - ranging from important cultural datasets like Project Gutenberg to mapping data to AI/ML datasets. All of these datasets are open source and public. In the future, we aim to launch a website for exploration of these datasets, enabling complete and/or partial retrieval and inspection of everything stored within the course of this program. For a full list of datasets please see: https://store.filecoin-discover.com/pages/full-dataset-list

how you intend to store the data on Filecoin, e.g. redundancy, location / miner preferences, etc

We are storing all our datasets with high redundancy: >= 6 copies per dataset. We do this by first sourcing a set of orders per category (by enabling orders on our store), and then allocating datasets based on customer demand. Where applicable, we bias towards sending datasets to regions where there might be cultural sensitivity. During our beta-test, in order to promote decentralization, we will use this allocation of data cap to only onboard a single sector’s worth of data from each miner - as our intent is to load test starting many deals in parallel.
On request, we can privately share more information about the geographic spread of the miners who applied to the program. As an overview we have representation of miners from over a dozen countries attempting to store Discover data.
All proposed deals under this data-cap are intended to have fast retrieval enabled, in order to facilitate the future retrieval website.

Thank you @ribasushi for sharing the context. You had mentioned that the requested datacap will be spent across a number of different addresses. Can you kindly share those addresses and map with an individual datacap?

@philippbanhardt apologies for the late response here. Please split evenly 512GiB each among these 4:

f1lq35h7sk5pdyzymnhh27ywubr7qsnu7cgcwgjla
f1ozspv62aays4owiwzen2xvi22vbymkplaplxrli
f3qxw65gv4fdlptz6n45aw4s6mvanpegjf3zfnsbiy5ghvuvtortxjmoc2ydrm33t2m2lmpzklkovddw6j5imq
f3vmoie5eprp7hemixg6onrrpmlpsr4frmghlfnxx7fi3rxbdq4c5enpj7r2qjipxenacaijncq35oldgb7irq

@ribasushi thanks, I have awarded 512 GiB for each of the above addresses.


add support for Azure OpenAI API - continue MR 167

Since I didn't see any changes being made in MR https://github.com/di-sukharev/opencommit/pull/167, I decided to synchronize the main branch with @takuya-o changes.
In addition, I ran prettier on the code

@di-sukharev

gonna get to it soon <3

pls fix tests

pls fix tests

done


Prepare for 0.17.0

New changelog, bump version.
Signed-off-by: Kaizhe Huang<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

/kind flaky-test

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area engine

/area rules

/area deployment

/area integrations

/area examples

What this PR does / why we need it:
Which issue(s) this PR fixes:

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

Update CHANGELOG for upcoming 0.17.0 release

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: mstemm
If they are not already assigned, you can assign the PR to them by writing /assign @mstemm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 30d17ebd84a082400b0dae706ae20b288405a2fe

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mstemm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mstemm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Closing for now, will discuss the best way to handle merged to master offline.


Create new config resolver for Airflow env

Codecov Report
Base: 99.09% // Head: 99.02% // Decreases project coverage by -0.06% :warning:

Coverage data is based on head (34f6151) compared to base (19af98a).
Patch coverage: 33.33% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #319      +/-   ##
==========================================
- Coverage   99.09%   99.02%   -0.07%
==========================================
  Files         113      113
  Lines        5724     5730       +6
==========================================
+ Hits         5672     5674       +2
- Misses         52       56       +4

Impacted Files
Coverage Δ

ted_sws/core/adapters/config_resolver.py
65.95% <33.33%> (-4.78%)
:arrow_down:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


invite-modal: Fix invalid custom input to not show "NaN".

Updates the invite modal so that if the custom input is 0, the submit button is deactivated, since an email invitation or invite link that immediately expired would be unusable.
Also, updates the custom input to show 0 if a user navigates away from and back to the custom input option afer putting in an invalid value, e.g. "abc" or "-20". Previously, the custom input showed "NaN" in that case.

Screenshots and screen captures:

Before - custom input set to zero
Submit button is active

Before - custom input invalid
Showing NaN because user set something invalid (e.g., "abc"), switched to preset option (e.g., "1 day") and switched back to custom time option

After - for both of the above cases
Shows 0 for invalid input if navigating back to custom time option and submit button is not active

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @laurynmm!


Upgrade to Swift 1.2

I have updated the project to build on Xcode 6.4 / Swift 1.2, would you like me to commit?

@austin9350 ^^^

Hi @joelshaffer - sorry I just saw this. We updated the project during one of the hackathons, but the updates have been going to dev instead of master.


Common metadata updates

Related Issue(s): #
N/A
Proposed Changes:

Corrects item jsonschema to evaluate all common metadata schemas
Item-spec and common-metadata documentation updated with more information about common metadata fields.

PR Checklist:

[x] This PR has no breaking changes.
[x] I have added my changes to the CHANGELOG or a CHANGELOG entry is not required. CHANGELOG entry not required
[ ] API only: I have run npm run generate-all to update the generated OpenAPI files. N/A

Wondering whether we should move title to the Common metadata?

Wondering whether we should move title to the Common metadata?

Hmm I think that's a good idea.


Question about genome data installation because of invalid download link of mirbase

Hi there,
I'm trying to install bcbio mm10 genome data for RNAseq and ATACseq analysis, but I noticed that several previous issues mentioned there might be some errors when downloading the mirbase. So I downloaded the most recent 'mirbase.yaml' file from github: https://github.com/chapmanb/cloudbiolinux/blob/master/ggd-recipes/mm10/mirbase.yaml, and copy this file up to the folder: /home/xz202/tmpbcbio-install/cloudbiolinux/ggd-recipes/mm10/ to overwrite with the older one.
I also tried using the parameter 'datatarget' to tell bcbio_nextgen not to download any mirna related databases, but my two strategies both failed.
Here is the command I use:
bcbio_nextgen.py upgrade -u skip --genomes mm10 --aligners bowtie2 --datatarget rnaseq
I found that downloading mirbase failed because the link for hairpin.fa and mature.fa is invalid even for the most updated version of 'mirbase.yaml'. So I have the following questions:
How can I just skip installing any small RNA seq related annotation files (i.e. skip installing mirbase and installing all the remaining)? Since I just want to perform RNAseq and ATACseq analysis for mice genome, I don't think I need the small RNA seq reference or index files.
Thanks a lot for your help!
Version info

bcbio version (bcbio_nextgen.py --version): 1.2.9
OS name and version (lsb_release -ds): CentOS Linux release 7.9.2009

To Reproduce
Exact bcbio command you have used:
bcbio_nextgen.py upgrade -u skip --genomes mm10 --aligners bowtie2 --datatarget rnaseq

Log files (could be found in work/log)
--2024-08-14 16:47:08--  https://github.com/chapmanb/cloudbiolinux/archive/master.tar.gz
Resolving github.com (github.com)... <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://codeload.github.com/chapmanb/cloudbiolinux/tar.gz/refs/heads/master [following]
--2024-08-14 16:47:08--  https://codeload.github.com/chapmanb/cloudbiolinux/tar.gz/refs/heads/master
Resolving codeload.github.com (codeload.github.com)... <IP_ADDRESS>
Connecting to codeload.github.com (codeload.github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified [application/x-gzip]
Saving to: ‘STDOUT’
 0K ........ ........ ........ ........ ........ ........  562K

3072K ........ ........ ........ .......                     530K=9.2s
2024-08-14 16:47:18 (549 KB/s) - written to stdout [5195515]
Upgrading bcbio
Upgrading bcbio-nextgen data files
List of genomes to get (from the config file at '{'genomes': [{'dbkey': 'mm10', 'name': 'Mouse (mm10)', 'indexes': ['seq', 'twobit'], 'annotations': ['transcripts', 'rmsk', 'problem_regions', 'prioritize', 'dbsnp', 'vcfanno', 'mirbase']}], 'genome_indexes': ['bowtie2', 'rtg'], 'install_liftover': False, 'install_uniref': False}'): Mouse (mm10)
Running GGD recipe: mm10 srnaseq 20211203
Traceback (most recent call last):
File "/home/xz202/bcbio/anaconda/bin/bcbio_nextgen.py", line 228, in
install.upgrade_bcbio(kwargs["args"])
File "/home/xz202/bcbio/anaconda/lib/python3.7/site-packages/bcbio/install.py", line 109, in upgrade_bcbio
upgrade_bcbio_data(args, REMOTES)
File "/home/xz202/bcbio/anaconda/lib/python3.7/site-packages/bcbio/install.py", line 361, in upgrade_bcbio_data
args.cores, ["ggd", "s3", "raw"])
File "/home/xz202/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 354, in install_data_local
_prep_genomes(env, genomes, genome_indexes, ready_approaches, data_filedir)
File "/home/xz202/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 480, in _prep_genomes
retrieve_fn(env, manager, gid, idx)
File "/home/xz202/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/genomes.py", line 875, in _install_with_ggd
ggd.install_recipe(os.getcwd(), env.system_install, recipe_file, gid)
File "/home/xz202/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/ggd.py", line 30, in install_recipe
recipe["recipe"]["full"]["recipe_type"], system_install)
File "/home/xz202/tmpbcbio-install/cloudbiolinux/cloudbio/biodata/ggd.py", line 62, in _run_recipe
subprocess.check_output(["bash", run_file])
File "/home/xz202/bcbio/anaconda/lib/python3.7/subprocess.py", line 411, in check_output
**kwargs).stdout
File "/home/xz202/bcbio/anaconda/lib/python3.7/subprocess.py", line 512, in run
output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['bash', '/home/xz202/bcbio/genomes/Mmusculus/mm10/txtmp/ggd-run.sh']' returned non-zero exit status 8.

Hi, sorry about the issue. I think sometimes miRBase is down and it leads for this issues. I would try again.
Another options is to modify that yaml file and remove the lines with mirbase to skip that downloading. You can remove things under command and output files to avoid downloading them and checking for them.

Hi, thanks for the reply. I deleted the line about 'miRBase' in the yaml file and it seems to work well. Thank you!


non Admin user on Welcome/Feedback channels.

On prod, the channel does not include the ours admin.


highlighter icon doesn't work ??

[ ] Review the documentation: https://github.com/oblador/react-native-vector-icons
[ ] Search for existing issues (including closed issues): https://github.com/oblador/react-native-vector-icons/issues

Environment

Description
Describe your issue in detail. Include screenshots if needed.

Reproducible Demo
Let us know how to reproduce the issue. Include a code sample, share a project, or share an app that reproduces the issue using https://snack.expo.io/. Please follow the guidelines for providing a MCVE: https://stackoverflow.com/help/mcve

@shadynoah could you please provide an example where we can reproduce your issue? Thanks!


Low access improvements

make a lower access that just lets you go in hallways and stuff like that for every department, like what security has with the "brig" access. this also wouldn't just apply to doors but also like machines, like the anomaly generator for example shouldn't be under science access

I don't agree completely, I think access should be only for low pop times.
Players stealing/breaking things could be a major issue with this and it'd also remove the point of stealing IDs.


Add VF2PostLayout pass

Summary
This commit adds a new transpiler VF2PostLayout and adds a new
phase/model to the preset transpiler pipeline post-layout/qubit
selection. The idea is based on the mapomatic project [1] which
took the code from the existing VF2Layout pass to find an isomorphic
subgraph in the coupling graph after transpilation which had better
noise characteristics than those initial selected as part of the initial
layout phase. Doing post transpile qubit selection provides the pass
more information because we can assume that the circuit's operations are
in the target basis and that there is at least 1 subgraph already in the
coupling graph because we've gone through routing. This enables us to
look at the specific error rates for each instruction and weigh the sum
of error rates for the mapped circuit on each potential qubit mapping to
find the best performing set of qubits for a given circuit. Initial
layout doesn't have access to this information because at the beginning
of the circuit we aren't necessarily going to find a perfect mapping and
we're not guaranteed to be in the target basis. So running post layout
may yield quality improvements even if we found an initial perfect
layout using VF2Layout.
While this new pass is very similar to VF2Layout pass as it builds an
interaction graph representing the 2q interactions in the circuit and
uses retworkx's vf2_mapping() function to find all isomorphic subgraphs
in the coupling graph it behaves differently. This is a separate pass
because it performs the search a bit differently. First the interaction
graphs are annotated with the gate counts on each qubit and edge which
is used to completely apply a heuristic score to the circuit and secondly
in the case of a target we verify the nodes and edges are feasible in the
subgraph isomorphism check since a target can have operations defined on
a subset of bits. Additionally the scoring heursitic checks the
sum of the error rates for each gate on the mapped qubits.
The preset pass managers are updated to use this new pass at the end of
the transpile and apply a layout if a solution is found.
Details and comments
[1] https://github.com/Qiskit-Partners/mapomatic
TODO:

[x]  Fix test failures

[x] ~Target layout score checking key error (looks like mapping or layout generation is wrong, or the node match check is broken)~
[x] Layout not fully allocated error

[x] Add tests for new pass
[x] Add release note

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

208 of 213 (97.65%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 83.765%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/transpiler/passes/layout/vf2_post_layout.py
156
161
96.89%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
54195

Relevant Lines:
64699

💛  - Coveralls

It would be nice if we can mirror the design decisions here in Mapomatic.  What I think would be nice is to use the ability to select the best sub-graph on multiple backends to target a system if an user does not specify the system name in the call to the primitive (which is possible on the cloud but not IQX at present)


Encoding LDAP Server manager-password

Is there any way to hashing the password in authentication-ldap.xml?
<security:ldap-server id="ldapServer" url="ldap://ldapservername:10389" manager-dn="uid=admin, ou=system"  manager-password="Qwerty@123" />
My password is visible here.

Geoportal Server uses the spring framework. Please see https://docs.spring.io/spring-security/reference/features/authentication/password-storage.html#authentication-password-storage for information on the options for storing an encrypted password.


Fix method ambiguity with in

The issue was detected by Aqua.jl running on a client package: https://github.com/JuliaAI/MLJScikitLearnInterface.jl/pull/64#issuecomment-1875712302
I believe, but am not absolutely sure, this overloading of in is not needed in the latest versions of MLJBase.jl. It was needed by now-deprecated learning network apparatus. However, as it is likely needed by  older versions of MLJBase.jl, it cannot be removed until the next breaking release of MLJModelInterface.jl.

Closed by #187


attempt to call field 'GUIShowViewer

🔵 Describe the bug:
When you click on the GSE icon on ChocolateBar, nothing shows up but the two errors listed below.
🔵 To reproduce: (Steps to reproduce the behavior)
Click on the GSE icon on ChocolateBar
🔵 The error:
Paste the error message in this blockquote.
Message: Interface\AddOns\GSE_LDB\LDBProvider.lua:132: attempt to call field 'GUIShowViewer' (a nil value)
Time: Sat May 14 16:04:34 2022
Count: 3
Stack: Interface\AddOns\GSE_LDB\LDBProvider.lua:132: attempt to call field 'GUIShowViewer' (a nil value)
(and)
Message: Interface\AddOns\GSE_LDB\LDBProvider.lua:136: attempt to call field 'GUIShowDebugWindow' (a nil value)
Time: Sat May 14 16:06:18 2022
Count: 2
Stack: Interface\AddOns\GSE_LDB\LDBProvider.lua:136: attempt to call field 'GUIShowDebugWindow' (a nil value)

🔵 Screenshots:

🔵 Expected behavior:
GSE interface should open
🔵 GSE.lua file:
GSE.txt
🔵 Desktop (please complete the following information):

OS: Windows 11
Game Version:" retail

🔵 GSE Version:

Version: 3.0.64
Downloaded From: curse

🔵 Additional context:
Add any other context about the problem here.

It’s not chocolate bar - it’s the LDB plugin for GSE not checking that the GSE GUi is loaded.  If you disable GSE_LDB in your addons until I can get a fix out later today it will allow you to operate.

Fix for this is in 3.0.65


Optimize the MPM kernels on CUDA

Specifically, the following changes are applied:

Mark global constants as __constant__ constexpr;
Add the f postfix to floating-point literals to avoid unexpected fp64 calculations;
Use .square() instead of .pow(2) for Eigen arrays;
Enable the -use_fast_math option for NVCC;
Move the check_cuda_error() (which synchronizes the device, introducing overwhelmingly large overhead to small computation workloads) call out of the step loop inside MPM::advance().

With these modifications, the kernels run significantly faster on CUDA, especially MPM2, which I see a roughly 2x speed-up and is now faster than Taichi (I am on an RTX2080TI):

Also, it might be better to have a separate folder per optimized version. To better tracking the performance improvement. @turbo0628 WDYT?

Hi, thanks for the optimizations. These are really useful. BTW, have you checked the MPM3D performance? how much improvement can we get from there?

Hi, this is the new plot for the 3D case (BTW, getting the results is really time-consuming...):

Hi, thanks for the optimizations. These are really useful. BTW, have you checked the MPM3D performance? how much improvement can we get from there?

Hi, this is the new plot for the 3D case (BTW, getting the results is really time-consuming...):

The number of particles of the 3D version is much larger, so it is very time consuming ;/. Can you please provide your Nvidia gpu driver / CUDA version. We will document this intermediate result. Many thanks!

@Mike-Leo-Smith Thanks for the PR! Really good to have a stronger CUDA baseline here!

BTW, getting the results is really time-consuming...

Agree. We can deprecate some extremely large test cases.

Also, it might be better to have a separate folder per optimized version. To better tracking the performance improvement. @turbo0628 WDYT?

I don't think we need extra folder for these optimizations. Let's work on the Taichi compiler to catch up!

Hi, thanks for the optimizations. These are really useful. BTW, have you checked the MPM3D performance? how much improvement can we get from there?

Hi, this is the new plot for the 3D case (BTW, getting the results is really time-consuming...):

The number of particles of the 3D version is much larger, so it is very time consuming ;/. Can you please provide your Nvidia gpu driver / CUDA version. We will document this intermediate result. Many thanks!

OK. Driver Version: 510.60.02, CUDA Version: 11.6.

Thanks for approving this PR. One minor issue though: the figures are not updated.


how to check number has a whatsapp

hi
how to check specific phone number has a whatsapp ?
best regards.

There is a method for it. check_number_status


wgpu: Update to 0.10

blocked by: https://github.com/hecrj/wgpu_glyph/pull/73 (no longer blocking)
Opengl backedn of wgpu now works as expected in iced (I never managed to run it in prev versions, but now it works)

EDIT: nope, never mind, on my second device it does not work at all, so it's still unstable (at least on linux)

Images in OpenGL are for some reason black, I will investigate this a little bit more soon.
Wgpu SwapChain not longer exists, so surface was added as an argument to draw() in compositor trait, wgpu implementation of compositors uses () as a SwapChain type now.

Please file issues about OpenGL backend!

https://github.com/gfx-rs/naga/pull/1265 Is now blocking this PR

After update of deps (maybe new naga?), text is no longer rendered in OpenGL backend... I'm kinda tired of debugging OpenGL backend, if I'll find some free time I will try to find a reason for this and report it (probably upstream)


:bug: Correct the cleanup command

Summary
Related issue(s)
Fixes #
Tests

[ ] Unit/function tests have been added and incorporated into make unit-tests.
[ ] Integration tests have been added and incorporated into make integration-test.
[ ] E2E tests have been added and incorporated into make e2e-test-all.
[ ] List other manual tests you have done.

/lgtm


Update populate metadata behavior

Adds the ability to populate metadata from any external metadata file.
Fixes a bug where the work can not be updated once an external metadata file is used for population.

Closes #166

Works well for me. 👍


Is not working with HTTPClient and JSON libraries

Bug Report
Is not working with HTTPClient and JSON libraries
Expected Behaviour
If HTTPClient library and JSON libraries are used shold work,
Actual Behaviour
If a POST action is send via HTTPClient or a JSON request is send is not responding.
How to reproduce the behaviour
HTTPClient:
 HttpClient client = HttpClientBuilder.create().build();
 HttpPost post = new HttpPost("https://www.test.com");

 // Create some NameValuePair for HttpPost parameters
 List < NameValuePair > arguments = new ArrayList < > (2);
 arguments.add(new BasicNameValuePair("record1", "test string1"); arguments.add(new BasicNameValuePair("record2", "test string2");

 try {
     post.setEntity(new UrlEncodedFormEntity(arguments));
     HttpResponse response = client.execute(post);
     String jsonResponse = EntityUtils.toString(response.getEntity());

     JSONArray jsonArray = new JSONArray(jsonResponse);

     for (int i = 0; i < jsonArray.length(); i++) {
         JSONObject jsonobject = jsonArray.getJSONObject(i);
         if (jsonobject.getString("valResponse").equals("1")) {
             system.out.print.ln("In");
         }
     }

 } catch (IOException e) {
     e.printStackTrace();

 } catch (JSONException e) {
     // TODO Auto-generated catch block
     e.printStackTrace();

 }

I'll try to reproduce this issue... Btw, I edited your comment (Just changed the code formatting)

Uhm, can you please provide your configuration?

Same happening for me on 1.9.3 using default config. What feature could be causing this?

I think I fixed it by disabling Bad Concat.

@AuligPromotions Interesting...

"I think I fixed it by disabling Bad Concat"
I removed all options and just leaves without anything and still was not working, something with http and json functionality is not working well

@marcoVera java version?

Java 8

@marcoVera I will introduce some new unit test to tests if the obfuscator has any bugs.

23.02 19:53:09 [Server] WARN java.security.InvalidKeyException: Illegal key size or default parameters 23.02 19:53:09 [Server] WARN at javax.crypto.Cipher.checkCryptoPerm(Cipher.java:1026) 23.02 19:53:09 [Server] WARN at javax.crypto.Cipher.implInit(Cipher.java:801) 23.02 19:53:09 [Server] WARN at javax.crypto.Cipher.chooseProvider(Cipher.java:864) 23.02 19:53:09 [Server] WARN at javax.crypto.Cipher.init(Cipher.java:1249) 23.02 19:53:09 [Server] WARN at javax.crypto.Cipher.init(Cipher.java:1186)

@JH3Y50N Did you enable AES?

@JH3Y50N Did you enable AES?

yes

@JH3Y50N AES is marked as Proceed with caution since it can easily break your code. Just turn it of and it will work ;D

I have the same problem, its not working with http/json.

I have the same problem, its not working with http/json.


kconfig: modules: Avoid including Kconfig.tls-generic twice

The glob in modules/Kconfig accidentally picked up Kconfig.tls-generic,
which is only supposed to be included from modules/Kconfig.mbedtls.
Replace the globbing with explicit 'source's to fix it. Best to avoid
globbing unless absolutely necessary, because it tends to pick up random
non-checked-in files as well.
Use 'source' instead of 'osource' since the files are known to exists.

The commit message has a mistake in it:

Also use 'source' instead of 'rsource', to make the paths greppable.
'rsource' is mostly meant for rare out-of-tree cases.

In fact, osource is being used.

Fixed. Confused it with a different spot.

This change itself look good to me.
However, this PR made me wonder if the Kconfig. and so should be moved to the module itself.
Zephyr modules supports loading of Kconfig files when used as a zephyr module.
As example, when using NCS and not having cloned modules/hal/stm32, then symbols such as: HAS_STM32CUBE still shows up in searches, and when enabling Show-all in menuconfig.

@tejlmand we have tried that in the past and it doesn't work. The moment you start removing projects from the manifest (like Intel and  Nordic do in their downstream) all sorts of problems pop up in Kconfig.

Yeah, I'm not sure we need more module stuff. Don't think it's proven to be more helpful than a source of problems at this point.


Why login is required in offline mode?

Weights and Biases version: 0.8.31
Python version: 3.6.6
Operating System: Linux

Description
I need to run wandb in offline mode on Kaggle kernels with no internet access. But whenever I run wandb.init() it asks for login key. Why would I need to login in offline mode? I just want the log files. Even if I put a key, it requires the internet to authenticate which is not possible with no internet access.
What I Did

You can call wandb.login(key=“X”*40) or set the WANDB_API_KEY env variable to a 40 character string.  We'll implement a fix to not assert auth when in jupyter / dryrun mode.  You should also specify your username with the entity keyword in wandb.init, i.e. wandb.init(entity="my_username", project="my_project")

Thanks!! Working now.


when at the first screen resizeToAvoidBottomInset is not work

only appearance at the first screen of the app, other page works well.
[✓] Flutter (Channel stable, v1.5.4-hotfix.2, on Mac OS X 10.14.3 18D109, locale zh-Hans-CN)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[!] iOS toolchain - develop for iOS devices (Xcode 10.2.1)
    ✗ Verify that all connected devices have been paired with this computer in Xcode.
      If all devices have been paired, libimobiledevice and ideviceinstaller may require updating.
      To update with Brew, run:
        brew update
        brew uninstall --ignore-dependencies libimobiledevice
        brew uninstall --ignore-dependencies usbmuxd
        brew install --HEAD usbmuxd
        brew unlink usbmuxd
        brew link usbmuxd
        brew install --HEAD libimobiledevice
        brew install ideviceinstaller
[✓] Android Studio (version 3.3)
[✓] Connected device (3 available)

Can you provide a small app that demonstrates this problem?

@HansMuller  I got the problem，if  a scaffold inset another scaffold it will appearance . it should be my fault.


project_client.telemetry.enable() should  configure otel logging and events to record OpenAI events

OpenAI instrumentation follows otel semcomv and emits prompts/completions as events (aka logs).
We need to enable OTel logging and events pipeline if we want people to get awesome openai  observability.
Here's the sample:
from opentelemetry._logs import get_logger_provider
from opentelemetry._events import set_event_logger_provider
from opentelemetry.sdk._events import EventLoggerProvider
from opentelemetry.sdk._logs.export import SimpleLogRecordProcessor, ConsoleLogExporter
from opentelemetry.exporter.otlp.proto.grpc._log_exporter import OTLPLogExporter

def configure_logging():
    provider = get_logger_provider()
    provider.add_log_record_processor(SimpleLogRecordProcessor(OTLPLogExporter()))
    provider.add_log_record_processor(SimpleLogRecordProcessor(ConsoleLogExporter()))
    set_event_logger_provider(EventLoggerProvider(provider))

OTel Python logging APIs are in beta, so are in the _logs and _events module so if we do it, we should catch and ignore all exceptions in the above code

fixed in #38434


error Couldn't find package "<EMAIL_ADDRESS>

Description
Hi, tried bumping up the tinacms version and noticed a dependency error with<EMAIL_ADDRESS>I wasn't sure this was something to do with my dependencies so I cloned the main repo and tried installing, the same  error happened:
error Couldn't find package "<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>on the "npm" registry.
Steps to reproduce
Clone and install this package https://github.com/tinacms/tinacms/tree/master/packages/gatsby-tinacms-git
Expected result
clean install without fatal error
Actual result
Instalation stopped
error Couldn't find package "<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>on the "npm" registry.
Environment
MacOS nvm 10.13 or Docker nodejs 10.13 base image
Tinacms Package Versions
packages from the repo

Thanks for the heads up @hugominas, missed that one. I just made it public can you try again?

It works as expected, thanks @ncphillips


fix(system-settings): update cache instead of deleting

About this PR
There is not so much code to review, but quite a lot of context to explain:

I found this issue in d2 when working on DHIS2-6860 in the settings-app.
What happens is the following:

When the app is initialised a call to d2.system.settings.all() is done
Now we have access to d2.system.settings.settings. (Quite likely apps are not meant to access that property directly, but since .get returns a promise, I can imagine it is convenient to do so.)
Now, if any setting-field in the settings-app changes, a call to d2.system.settings.set(key, value) is done.
Before this PR, what would happen, is that the entire cached object is removed, which makes typeof d2.system.settings.settings === 'undefined'
After this PR, the cached settings are updated instead of removed.

Alternative approach
After studying the code in SystemSettings a bit more, I realised that you could argue that this problem was purely in the app, and that the behavior in SystemSettings was correct. This argument would be as follows:

The app shouldn't have tried to access d2.system.settings.settings directly.
Instead it should have used the proper API: calling d2.system.settings.set(key) returns a promise and will fetch a value from the server if the cached value is unavailable.

Looking forward to hearing what you think

NB: I am going to update the settings-app too, so that it uses the proper d2 API to interact with the system settings. But IMO this would be a good improvement to the SystemSetting module regardless of what happens to the settings-app.

See https://github.com/dhis2/settings-app/pull/149 for related settings-app PR


Master exclude pr

Docs Build status updates of commit 51530f8:
:warning: Validation status: warnings

File
Status
Preview URL
Details

MarkdownValidationTest/validation.md
:warning:Warning

Details

MarkdownValidationTest/validation_include.md
:warning:Warning

Details

MarkdownValidationTest/includes/token2.md
:white_check_mark:Succeeded

MarkdownValidationTest/validation.md

Line 12: [Warning] Markdown content is not allowed before H1.
Line 12: [Warning] Link https://docs.microsoft.com/en-us contains locale code en-us. For localizability, remove en-us from links to Microsoft sites.
[Suggestion] Invalid value for ms.author: 'jasoz__' is not a valid Microsoft alias.

MarkdownValidationTest/validation_include.md

Line 12: [Warning] Markdown content is not allowed before H1.
Line 12: [Warning] Link https://docs.microsoft.com/en-us contains locale code en-us. For localizability, remove en-us from links to Microsoft sites.
[Suggestion] Invalid value for ms.author: 'jasoz__' is not a valid Microsoft alias.

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

Docs Build status updates of commit 8df1993:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

MarkdownValidationTest/validation_include.md
:x:Error

Details

MarkdownValidationTest/validation.md
:warning:Warning

Details

MarkdownValidationTest/validation_include.md

[Error] Cannot resolve '/includes/token2.md' relative to 'validation_include.md'.
Line 11: [Warning] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.
[Suggestion] Invalid value for ms.author: 'jasoz__' is not a valid Microsoft alias.

MarkdownValidationTest/validation.md

Line 12: [Warning] Markdown content is not allowed before H1.
Line 12: [Warning] Link https://docs.microsoft.com/en-us contains locale code en-us. For localizability, remove en-us from links to Microsoft sites.
[Suggestion] Invalid value for ms.author: 'jasoz__' is not a valid Microsoft alias.

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

Docs Build status updates of commit f9c2436:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

MarkdownValidationTest/validation_include.md
:x:Error

Details

MarkdownValidationTest/validation.md
:warning:Warning

Details

MarkdownValidationTest/validation_include.md

[Error] Cannot resolve '/includes/token2.md' relative to 'validation_include.md'.
Line 11: [Warning] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.
[Suggestion] Invalid value for ms.author: 'jasoz__' is not a valid Microsoft alias.

MarkdownValidationTest/validation.md

Line 12: [Warning] Markdown content is not allowed before H1.
Line 12: [Warning] Link https://docs.microsoft.com/en-us contains locale code en-us. For localizability, remove en-us from links to Microsoft sites.
[Suggestion] Invalid value for ms.author: 'jasoz__' is not a valid Microsoft alias.

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

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

Docs Build status updates of commit 38d7234:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

MarkdownValidationTest/validation_include.md
:x:Error

Details

MarkdownValidationTest/validation.md
:warning:Warning

Details

MarkdownValidationTest/validation_include.md

[Error] Cannot resolve '../includes/token2.md' relative to 'validation_include.md'.
Line 11: [Warning] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.
[Suggestion] Invalid value for ms.author: 'jasoz__' is not a valid Microsoft alias.

MarkdownValidationTest/validation.md

Line 12: [Warning] Markdown content is not allowed before H1.
Line 12: [Warning] Link https://docs.microsoft.com/en-us contains locale code en-us. For localizability, remove en-us from links to Microsoft sites.
[Suggestion] Invalid value for ms.author: 'jasoz__' is not a valid Microsoft alias.

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

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

Docs Build status updates of commit 1299c44:
:warning: Validation status: warnings

File
Status
Preview URL
Details

MarkdownValidationTest/validation.md
:warning:Warning

Details

MarkdownValidationTest/validation_include.md
:warning:Warning

Details

MarkdownValidationTest/includes/token1.md
:white_check_mark:Succeeded

MarkdownValidationTest/includes/token2.md
:white_check_mark:Succeeded

MarkdownValidationTest/validation.md

Line 12: [Warning] Markdown content is not allowed before H1.
Line 12: [Warning] Link https://docs.microsoft.com/en-us contains locale code en-us. For localizability, remove en-us from links to Microsoft sites.
[Suggestion] Invalid value for ms.author: 'jasoz__' is not a valid Microsoft alias.

MarkdownValidationTest/validation_include.md

Line 11: [Warning] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.
[Suggestion] Invalid value for ms.author: 'jasoz__' is not a valid Microsoft alias.

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


Add support for GCP extensions

The support is modeled on azure.go under tlvparse. The GCP spec is similar but
encodes uint64. There's no subtype. Since zero value of uint64 is technically
a valid header value, I've opted for an extra boolean to indicate if it was
found (same as with Azure).

Given that we already have AWS, and Azure in tlvparse I thought it will be a good idea to add the GCP support as well.
We use it with GCP in our pre-prod env and all seems to be working well.
Let me know if you think it makes sense to have this with the upstream lib.

Coverage remained the same at 94.353% when pulling 8a2ef0b29aa5e49804fede83ae691d9d9635031c on igor-kupczynski:gcp-tlv-parser into 3aa7ea95a821cf534fae576651e2ca11f222a4e3 on pires:main.

I've noticed a naming inconsistency and force-pushed a fixed version. Hope that OK -- my thinking was that a simple rename doesn't add a lot to the history.

Nice, thanks for merging :)  and thanks for maintaining the library.

func names tied to PP2 type, eg PP2GCP.ExtractPSCConnectionID, or maybe have tlvparse/{aws,azure,gcp}

Agree, it would be nice to improve the consistency here. I'll think about it for a bit.

Released in https://github.com/pires/go-proxyproto/releases/tag/v0.6.1


Google Analytics cannot verify site ownership

Relates to #1654 and #1505
Currently the GA template is added at the bottom of the body section instead of at the bottom of just before the </head> tag as required by Google
This causes Google to be unable to verify ownership of the property in Webmaster Central.
It should probably also use the asynchronous version from https://developers.google.com/analytics/devguides/collection/analyticsjs/#alternative_async_tracking_snippet

My bad, the template being in the wrong section was the fault of the theme I'm using.
PR for asynchronous GA imminent.


⚠ Bofrost.de has degraded performance

In 78f4738, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2316 ms

Resolved: Bofrost.de performance has improved in a102cd6 after 2 hours, 20 minutes.


docs: Updated README regarding installation steps

updates readme regarding install of

macOS
ubuntu

Thanks for the update!


Cactbot TTS not calling out

Hi
I am having issues with the cactbot not calling out the raid alert, a few other members of the static having the same issue. I have sound and TSS enabled.

Sorry for the belated response.  If you are still having issues, please talk to the FFXIV ACT Discord for more direct support.


🚀 Feature:  Allow adding MR reviewers via publish:gitlab:merge-request action

🔖 Feature description
Similar to the GitHub implementation: https://github.com/backstage/backstage/issues/9571
I'd like to be able to add reviewers to MRs created by Backstage via the publish:gitlab:merge-request action by setting them as parameters of the action.
🎤 Context
Allowing reviewer assignment as parameters would streamline the process and align with GitHub’s functionality, improving workflow automation.
    - id: createMergeRequest
      name: Create GitLab Merge Request
      action: publish:gitlab:merge-request
      input:
        sourcePath: "./template"
        title: "new project ${{ parameters.clusterName }}-${{ parameters.membership }}"
        branchName: new-project/${{ parameters.clusterName }}-${{ parameters.membership }}
        description: New project ${{ parameters.clusterName }}-${{ parameters.membership }}.
        repoUrl: gitlab.com?owner=gitops/argocd&repo=resources
        removeSourceBranch: true
        assignee: max-mustermann
+       reviewer: miri-musterwoman

✌ Possible Implementation
Maybe similar to this PR: https://github.com/backstage/backstage/pull/13008
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

it would be also great if we can just default assign CODEOWNERS for both assignee and reviews just like the github plugin

I wanted to check on some other input parameters in the Gitlab MR action. I'll keep it in mind and see the options.👍

@hghtwr do you still plan to have a look at it?

@dfl-aeb Woah, sorry, I totally forgot about this issue. Let me know if its still relevant, I'll check it, then.

Yes, I totally agree. It is still relevant for us :)