Fix incorrect error messages when certain requests fail

This PR aims to fix the symptoms described in #1530, where error messages contradict themselves. It does not fix the underlying problem (“Could not find "window._sharedData" in html response.“), though. Still, I think this is worth it, since profile loading might break in a similar fashion some time in the future.
I'm new to this codebase, so I do lack the “bigger picture”.

a) handle case where Profile.from_id works, but Profile.from_username doesn't
https://github.com/instaloader/instaloader/blob/385c6c8a353458b8e1f5d1eacd98a714a41e9fc0/instaloader/instaloader.py#L1326-L1337
These are the local variables of interest:
profile_id == 1234
profile == None
profile_name == 'somename'
newname == 'somename'

And here's what happens:

Profile.from_username raises ProfileNotExistsException
It's caught afterwards
profile_id is read from the file
profile is None since it wasn't found earlier

Trying to find profile somename using its unique ID 1234.

Profile.from_id succeeds
Instaloader falsely concludes that the profile has changed its name, since it could not be found using the old name, but using the ID. In reality, Profile.from_username failed for some other reason, and the username remained unchanged.

b) Handle case where QueryReturnedNotFoundException lies and similar_profiles contains the exact username
https://github.com/instaloader/instaloader/blob/385c6c8a353458b8e1f5d1eacd98a714a41e9fc0/instaloader/structures.py#L764-L772

Profile somename does not exist.
The most similar profile is: somename.

Here, Instaloader tries to help by suggesting similar usernames, but doesn't notice that it's actually the one we wanted. It just wasn't able to load it, but since we catch a QueryReturnedNotFoundException, it's plausible to assume that the profile really does not exist.
The exception (in my case) is thrown here:
https://github.com/instaloader/instaloader/blob/385c6c8a353458b8e1f5d1eacd98a714a41e9fc0/instaloader/instaloadercontext.py#L356
An alternative approach would be to raise another kind of exception there. 🤷🏼♂

Thanks for your review!

Oops, seems like I didn't lint myself. I used autopep8 to conform to the maximum line length of 120.
Just let me know if you'd like to have this rebased before merging. :)

Thanks!


create view relation has extra quotes (") and schema name in 10.1

I noticed once I updated to 10.1, our view materializations error out due to an extra wrapping of "" around the table name and the schema is repeated
I haven't noticed this in any of our ephemeral or table models.

Ben

Hey @huisben - thanks for the report. Couple of questions:
What kind of warehouse are you using?
Did you configure quoting in your dbt_project.yml file?
This definitely doesn't look right, I'll take a look.

hey @drewbanin thanks for the tip on quoting. Makes a lot of sense.
I've added the explicit quoting in and getting similar errors.  I tried for all permutations. With both off, there are no quotes but database complains it cannot find the table. With one or the other on it will throw double quotes around whichever one I toggle.
With both on it looks now like this create view "dbt_ben".""dbt_ben"."basic_accounts__dbt_tmp""
we're on redshift.

it actually seems like it's just writing the schema twice, rather than a quoting problem

thanks to Drew, this "issue" (it actually was not an issue with dbt, at all) is RESOLVED.
for future reference of anyone who may see this, we had created a macro in our dbt/macros folder to do late binding views on our Redshift cluster.
Since dbt has natively supported this, we did not update it. It began manifesting itself as an issue with 10.1 because of the new Relation model (https://github.com/fishtown-analytics/dbt/blob/master/dbt/adapters/default/relation.py) which itself contains both a schema and identifier component. Because our legacy code appended schema and identifier is now Relation, it was causing this error.
Thanks Drew and the dbt community!!

thanks @huisben :)


Bad Format in Kafka Topic getOffers

Events from the topic getOffers are missing a value for product_id,
{"product_id": , "http_code": 200, "timestamp": "2018-07-06T15:33:23.425Z"}

This results in an warning in benchmark.py: "Failed to dump kafka topic getOffers"
Do we even need this topic?

Cannot access the code right now. What do we store in getOffer? Is this the storage of market situations when somebody requests an offer (i.e., a market situation?!?!)?


Just wanted to say thanks for putting this together!

I recently had to switch from VSCode to IntelliJ and dearly missed the Synthwave 84 theme. Your adaptation is great, thank you for working on it!

YES! Thanks! I've been looking for a Synthwave 84 theme for Jetbrains! 💯

Yes thanks a looot . I just tweak and put #1D1D1D for backgroudn and gutter to fit my material darker appearance.
Thanks you rocks :)


[SPARK-29324][SQL] Fix overwrite behaviour for saveAsTable

What changes were proposed in this pull request?
When saveAstable is used in overwrite mode, the metadata of the table gets overwritten.
In this PR, adding changes to retain the metadata after overwrite to an existing table
Why are the changes needed?
The tables metadata gets overwritten without this change
Does this PR introduce any user-facing change?
No
How was this patch tested?
Added Unit tests

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can someone help review this change

Sorry, but I miss you point; this is not an expected behaivour? metadata also should be overwritten in the overwrite mode?

Without the change,  an external table become a managed table, the file format could change as well(Say table was created using Orc file format, after overwrite it will change to parquet which is the default FF)
Few issues

Any other user who is consuming the table will be querying the data at an older location, instead of the new data.
In case using overwriting a partitioned table with partitionOverwriteMode set to dynamic,
I think the expectation would be to overwrite the data of the only affected partitions with the same file format(Please correct me if I am wrong). In this case, we will end up writing in a completely different location and maybe with different file format.

Can someone review this change?
Tagging couple of users from the history of the file @cloud-fan @gatorsmile @maropu

saveAsTable with overwrite mode is kind of REPLACE OR CREATE TABLE AS SELECT. This is the expected behavior. I think what you need is insertInto, but you would need to deal with non-existing table manually.

When I write to a partitioned table(with fileformat other than parquet) using saveAstable and partitionOverwriteMode set to dynamic, the affected partitions may have data with Parquet format(default) while other partitions will have data in a different format. User  needs to specify the fileformat when writing using this API.
I think this operation should be disallowed when fileformat is not specifie  or use the format from the dropped table.

Also during overwrite + saveAsTable operation, we drop the table and recreate it

In the case of Hive table, we might transparently change a Hive(provider = HIve) table to datasource table. Can this have any implication when the table is read with Spark version where there are issues with Orc's data source flow and user wanting to use the Hive read flow?
In the case of   partitionOverwriteMode=dynamic, we write to particular partitions only. Due to drop, we lose the partition information. ALso, all the partition level stats is lost(If already analyzed). These are expensive operations in case of larger tables.

@cloud-fan Any thoughts on these?

User needs to specify the fileformat when writing using this API.

This is a bug that we should fix. But I'm not sure this would happen as we explicitly drop and re-create the table.

we might transparently change a Hive(provider = Hive) table to datasource table

This behavior makes sense if you think about it as REPLACE OR CREATE TABLE AS SELECT. I know saveAsTable is not a good name, but that's what it is today. You can try the new DataFrameWriterV2 which has clear semantic after 3.0 preview is released. also cc @rdblue

Closing this per discussed above.


Helper rspec test issue.

I am test my helper.
helper method use account object created from controller by expose.
when i am test.
assign(:account, account)
That is no assign to account object which is we have assigned.
that will assign @account.

hey @Rajniktc this assigns(:account, account) does not work because decent_exposure uses an instance variable like "exposed_#{name}", in this case: :exposed_account would work.
Perhaps, the main point of decent_exposure is to do not use instance variables in views/helpers, so I would avoid that on your tests too. So I would change assigns(:account, account) to expect(controller.account).to eq(account)
For example, in our integration tests we are using the method call directly controller.bird to assert the assigned value.
  it "builds bird if id is not provided" do
    bird = double("Bird")
    expect(Bird).to receive(:new).with({}).and_return(bird)
    get :show
    expect(controller.bird).to eq(bird)
  end

Closing this issue for now.


eccentrical, garishness

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[e,g]

/trunk merge


Improve how people see spec in the website

Reason/Context
Even though AsyncAPI specification markdown file is located in the spec repo, for a better UX we make it available in the AsyncAPI website too. We have proper automation in place that makes sure they are in perfect sync, so no worries 🍺
But we just render markdown as it is and that is it.
It is looooong markdown.
When you jump from 2.0 to 2.1 majority of the content is the same. How one can spot what changed?
Go to release notes? yeah, sure, why not. But you need to know where to look for, and anyway we can do much better here!
Description

Provide reference link to release notes. Somewhere on the top of every spec document we should also render link to GitHub release notes, where user also find link to more detailed article. So basically link from here -> here
Have some kind of toogle (supported as query param), that once enabled, provides a kind of overlay, a smarter view, where in the rendered document, parts that are new/changed are highlighted. I don't know yet how nicely present removed parts

@derberg wanna assign this to me?

@AceTheCreator only if you want it 😄

@derberg guess it's all mine now 😀

@AceTheCreator enjoy! 😄 first research, explore possible options, share with community progress and some screen shots on how you see it and then implement 😉

@derberg How about using an accordion for summarized release notes of every version like so:
https://react-bootstrap.netlify.app/components/accordion/#accordion
and a link to detailed release notes at the footer of this accordion?

@sambhavsaxena the problem is we do not have such "summarized" release notes, only full release notes

Just putting this here, not sure if it can help or not, but Codesandbox open-sourced a nice component that can help with docs.
https://codesandbox.io/post/sandpack-announcement

Heyyyy @derberg , as discussed on slack, I am interested to take this task.
Do you have any ideas or suggestions to start with??
Thank you 😄.

I do not have too many ideas, or tools pointers for this one other than:

have much easier integration between spec doc and the release notes. In other words. When readers go to https://www.asyncapi.com/docs/reference/specification/v2.5.0 then should no be forced to figure out how to find release notes. Readers should see some "panel" or "note" that indicates the link to release notes. You know like "tl;dr if you want to quickly learn what is new, just grab release notes". This would have to be automated, not that someone manually adds it to specification document every time. Docs and release notes need to be "wired" in some way, maybe front-matter. Then during website build, if such "wire" is noticed then pannel is added to the website

this is how release notes look like https://github.com/asyncapi/website/blob/master/pages/blog/release-notes-2.5.0.md
this is how related spec documentation looks like https://github.com/asyncapi/website/blob/master/pages/docs/reference/specification/v2.5.0.md

Would be nice to have some kind of switch. Like dark/light mode one. A toggle that one can switch to say "show diff". So it highlights the UI parts of the spec document that were added to the document, modified, or deleted when compared to the previous version. You need to research tools that are good at figuring out text diff. I do not know of any good recommendations.

Both of the above ideas are good for people that follow spec from version to version. They use 2.4 and want to know what is in 2.5, and then they use 2.5 and in future they want to know what we added in 2.6, etc.
The thing is that there are also people that use 2.0 and want to figure if it makes sense to switch to 2.5. So we also need to figure:

how to also give them option to say "do not show default diff between latest and the last before, but lemme select what version I want to check against, maybe 2.1"
in the panned where we display release notes for 2.5 maybe we should also provide link like https://www.asyncapi.com/blog?tags=Release+Notes for all the other release notes? just for starter

Heyy @derberg,
Have a look at this design

Note: Since the release note and specification document have same title, I used it to link the specification document with release note.
For example, if an user visits 2.5.0  (post.title) documentation, we will be providing a link to release note like, https://github.com/asyncapi/website/blob/master/pages/blog/release-notes-${post.title}.md
I have also sent a message on slack regarding what I have done.
Thank you :smile:

in the panned where we display release notes for 2.5 maybe we should also provide link like https://www.asyncapi.com/blog?tags=Release+Notes for all the other release notes? just for starter

what about this?

Heyy @derberg ,
Yeah, will consider these points. Will let you know about the progress, once I complete this stage.
Thank you

Hey @derberg, I'd like to work on this. Since the main website is built with Next.js. We have some amazing tools like Contentlayer. This will help us populate the routes and the pages at build time. I've worked with this tool before and I'm using it for blogs on my website built with Next.js 13. I'd like to know your thoughts on this :)

@adarsh500 this tool looks interesting, but can you specify what it has to do with this issue?
more challenging is how to present in the UI the diff information.
@Harish-b-03 not sure if you work on it. Can you share all you learned so far?

Heyy @derberg
As I mentioned on slack, I am not working on this issue.
Thank you 😃

oh, thanks, I must have missed that info, sorry
@adarsh500 I think Harish was giving a try to jsdiff

Oops sorry, misunderstood the issue. I thought there issue was with rendering markdown files

@adarsh500 what's the current status??
@derberg is this issue still relevant?

that would be nice but I'm not able to provide a proper guidance and ownership. This requires research I don't have time to do. Closing and if anyone still wants to pick it up, I suggest opening another issue.


help: Document "Inbox view" mobile feature.

Adds a new page under the "Reading messages" section documenting
the Inbox view on mobile.
Adds Inbox, expand more, and expand less SVG mobile icons.
Cross-links with "Recent conversations" and "All messages".

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability

Communicate decisions, questions, and potential concerns.

[x] Automated tests.

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

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

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Links.

Updated PR and screenshot to address feedback.
@zulipbot add "release goal"

@zulipbot add "release goal"

Thanks! The only outstanding comment I'm aware of is:

Let's add ", excluding inactive streams." to the end of the second sentence.

Looks good to me! @timabbott please take a look.

Merged, thanks @drrosa!


load dataset

how to load and train this dataset https://huggingface.co/datasets/tep_en_fa_para on mistral-7b model

@nichellehouston We have a Discord server https://discord.gg/u54VK8m8tk with a lot of super helpful people who can solve dataset issues :)


Change UserOperator data model id to UUID

###Description of the Tech Debt###
In the route specified below, the user_operator_id created is returned to the front-end for use as a query parameter. However, exposing sensitive data in query parameters is not considered a secure practice. Therefore, it is advisable to enhance security by altering the ID of the UserOperator data model to a UUID format. This modification helps prevent potential security vulnerabilities or attacks.
@router.post("/user-operator/operator", response={200: RequestAccessOut, codes_4xx: Message})
###Tech Debt Triage#
The purpose of our technical debt triage process is to analyze technical debt to determine risk level of the technical debt and the value in tackling that technical debt.
Risk Value Scoring:

Level
Value

High
3

Medium
2

Low
1

Technical Debt - Risk Types
Level
Value

Business Area Risk - Risk of business area visibility / damage to user experience

1

Developer Fault Risk - How likely will this tech debt cause a future error related to coding on top of it

1

System Fault Risk - Risk of system errors or application downtime

1

Time Scale Risk - Compound risk effect if left alone. How much more difficult to fix or dangerous will this become over time?

1

Time Sink Risk - How much will this tech debt slow the development process down

1

TOTAL SCORE:

5

Addressed in: https://github.com/bcgov/cas-registration/pull/939


[Network] az network express-route port delete: Add confirmation while deleting

Related command

Description

Resolve: https://github.com/Azure/azure-cli/issues/26678
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Network


Where is your code?

The code seems to fork from SpectralNet. Can you provide your code about paper #Deep-Spectral-Clustering-using-Dual-Autoencoder-Network#? We'd like to compare your method. Thanks

Your paper placed SpectralNet on YTF. And you got a not bad results. Can you put it on there?


Refactoring of the Data Filters

I refactored all the Data Filters (DF) previously in DataPointsFiltersImpl.h and DataPointsFiltersImpl.cpp as follow:

Create a new directory named DataPointsFilters/ containing now the DFs
Separate each DF in a specific file (.h and .cpp):  for instance the IdentityDataPointsFilter is now declared in DataPointsFilters/Identity.h and DataPointsFilters/Identity.cpp
Each DF is now templated on T (float or double) and inherit directly of PointMatcher<T>::DataFilter
Still API compatible due to a trick in DataPointsFiltersImpl.h where each DF is typedef-ed as follow:

template<typename T>
struct DataPointsFiltersImpl {
    typedef ::IdentityDataPointsFilter<T> IdentityDataPointsFilter;
};

so, registration of DF in the registrar works without any change and the headers are included at only one place (DataPointsFiltersImpl.h). However, it's possible to directly change the Registry.cpp file not to be dependent of DataPointsFiltersImpl class anymore
ADD_TO_REGISTRAR_NO_PARAM(DataPointsFilter, IdentityDataPointsFilter, typename DataPointsFiltersImpl<T>::IdentityDataPointsFilter)

Add an utils.h header containing often used functions by the DFs (in a namespace utils, such as computeNormals(...), etc.)
Add minor improvements in Elipsoids, SamplingSurfaceNormal and Gestalt DFs by introducing move semantic where needed.

I'll update the tutorial to adapt it to these changes too! Hope it'll be helpful!

Alright, thanks for the thorough cleanup!
Since it's a large PR, I'll let it open for a day just in case someone has something to say.


_transpile_query should not be a @classmethod

_transpile_query is a classmethod that uses the current connection's dialect and transpiles it into a target dialect. However, in some cases, we might not want to use the currently active connection, but a different one, and transpile the query.
Here's an example. In the _historgam function, we can take an optional conn argument (a sqlalchemy engine); however, the transpilation step uses the dialect from the currently active connection. If the passed conn argument and the active connection do not have the same dialect, this will produce problems
Instead, we should use the information from the conn argument, if any.
Example:
def some_function(conn=None):
    if conn is None:
        conn = # get current active connection
    else:
       # Connection objects can be initialized with sqlalchemy engines
       conn = Connection(engine=conn)

    # transpile should be an instance method that uses the dialect from the `conn` engine
     query = conn.transpile()

To apply this logic, I think we need to change the _get_curr_sqlalchemy_connection_info, _get_curr_sqlglot_dialect, get_curr_identifiers, is_use_backtick_template to instance methods too.

Since currently in _transpile_query(), we use _get_curr_sqlglot_dialect classmethod to get the current sqlglot dialect info
Probably it's better we can fix those together?
        write_dialect = cls._get_curr_sqlglot_dialect()
        try:
            query = sqlglot.parse_one(query).sql(dialect=write_dialect)
        finally:
            return query```

Yeah, for me is clear to understand the purpose of this ticket.
Just need to confirm if we need to apply the same logic to other functions. I think so

yes, let's apply the changes to the other methods. I'm unsure if there are ones that make sense to keep as classmethods but I'll leave it up to you @tonykploomber

@tonykploomber: I updated the top comment in this issue. we'll tackle https://github.com/ploomber/jupysql/issues/383 here too, since it's closely related


Using Bluetooth on RPi3

Stratux version:  0.8r2

Stratux config:
SDR
[ ] single
[ x] dual
GPS
[ x] yes
[ ] no
type: MTK3339 on serial, sirf on usb
AHRS
[ x] yes
[ ] no
power source: all wart
usb cable:

EFB app and version: (e.g., WingX Pro7 8.6.2)  iFlyGPS 9.4
EFB platform: (e.g., iOS 9.2): Android 5.x
EFB hardware: (e.g., iPad Mini 2): Samsung Galaxy Tab S 10.1

Description of your issue:
The RPi-3 brings up an interesting possibility for an addition or change: using Bluteooth (LE) instead of wifi.  Does all EFB hardware have BT?  The main advantage is it consumes much less power than wifi.  Another advantage is for clubs/groups that fly with stratux because BT already has the capability for unique connections along with lower power radio so that fewer planes will see the same signal during close-in flying.  Wifi can be used in that same config by turning on WPA or something but the signal collision overhead is still incurred.
Thoughts?  A BT dongle is just as cheap as a wifi dongle.

But isn't Apple for example, very tight about the BT standards used to connect to their devices ?

I don't know.
/rant on/ I am really sick of these companies locking in their products trying to create "vertical markets."  Garmin, Apple (in many areas), and so on just make me angry and I vote with my wallet.  Let them "sit on the shelf and rot."
/rant off/
I do know that my chosen EFB, iFly, does not have BT capability at this time.  However, lot's of them do.

Skypuppy I'm sure you're talking about iFly's dedicated hardware (720, 740) which are wifi only, but the iFly app on Android and iDevices can use BT recievers as well as wifi receivers.  Just wanted to clarify that for others.

BLE is not avalible on all devices. Bluetooth is but the low energy is not.
Wifi is though

On iFly: I was wondering whether iFly could use BT. So I tried transferring a Flight Plan from one tablet to the other paired via BT. No joy.

I remember iFly via tablets, etc., as not being bt capable.
On 03/22/2016 07:20 PM, Nokomis449 wrote:

Skypuppy I'm sure you're talking about iFly's dedicated hardware (720,
740) which are wifi only, but the iFly app on Android and iDevices can
use BT recievers as well as wifi receivers. Just wanted to clarify
that for others.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/cyoung/stratux/issues/345#issuecomment-200091679

The BT-only Dual XGPS is an option under iFly's "Connected Devices".  Someone on Amazon gave this review:
http://www.amazon.com/review/R1Q45F8B1TL55A

Re: On iFly: I was wondering whether iFly could use BT. So I tried transferring a Flight Plan from one tablet to the other paired via BT. No joy.
Yeah I didn't mean to imply all networking functions will work on iFly via BT; only that Android and iDevices could receive ADS-B via BT devices.  I was specifically referring to skypuppy's BT/vertical market rant (with which I agree).  I wasn't thinking about flight plan sharing via BT.

I just bought the Pi3 and GPS receiver for use with Avare and CloudAhoy.  Unfortunately, I then found that CloudAhoy will only work via BT, so I am motivated to add a BT interface to stratux.  I write S/W for Linux systems for a living but I have not done any projects via Git so I am hesitant to commit taking on this task.  Also, I hope this is not an improper way to jump into an open source project.

Just create another branch and have at it!  :)

Unable to add Bluetooth support. Not enough EFB-side support to be worth the time.

Any chance we could try this for Avare w/ external I/O plugin and bluetooth on the pi3 as a first step?  I can work on this, but don't want to put in the effort if the pull request is unlikely to be merged.

Yes, if you want to add this it will be merged so long as everything works well.

#developers on Slack (see link on website) if you need help getting started.


Video Indexer - Missing Blocks form Documenation

In the output of v2 in the videos insights, there are 'shots' and 'blocks', but there is only 'shots' listed in the documentation.
NOTE:  I am really trying to find out the difference between the two?

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

ID: 4f8134c1-2f88-2aca-b59f-12ea4a444c04
Version Independent ID: 3324997f-8f2d-0914-9e9b-7982264d41f4
Content: Examine the Azure Video Indexer output produced by v2 API
Content Source: articles/cognitive-services/video-indexer/video-indexer-output-json-v2.md
Service: cognitive-services
GitHub Login: @Juliako
Microsoft Alias: juliako

@RGBKnights Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@Juliako Hi, this is a document enhancement issue, can you please check it and update the document as necessary? Thanks!

@RGBKnights, thank you for bringing this to my attention. I will update the doc ASAP. For now, Blocks is the same as https://docs.microsoft.com/en-us/azure/cognitive-services/video-indexer/video-indexer-output-json#transcriptblocks.

#please-close

@RGBKnights We will now close this issue. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


RIS parser includes BOM in first keyword

BibJSON from RIS (UUC publications) translate first:
TT   - JOUR
to
"\ufeffTY": "JOUR"

This is caused by the BOM checking in the parser base class not to work for urllib objects, as these file objects do not have a .seek() method. Thus BOM checking currently only works for files uploaded from yourlocal disk and not via a URL.
Will fix.

On Thu, Jan 26, 2012 at 7:47 PM, Etienne Posthumus <
<EMAIL_ADDRESS>
wrote:

This is caused by the BOM checking in the parser base class not to work
for urllib objects, as these file objects do not have a .seek() method.
Thus BOM checking currently only works for files uploaded from yourlocal
disk and not via a URL.
Will fix.

Thanks
Hope I have used the right mechanism.

Reply to this email directly or view it on GitHub:
https://github.com/okfn/bibserver/issues/169#issuecomment-3675227

--
Peter Murray-Rust
Reader in Molecular Informatics
Unilever Centre, Dep. Of Chemistry
University of Cambridge
CB2 1EW, UK
+44-1223-763069


Drag and drop audio files into 2D viewport to make AudioStreamPlayer2D nodes

This adds the ability to drag AudioStream resources into 2D viewport to add AudioStreamPlayer2D nodes.
This improves usability by making adding spatial 2D audio simpler by simply dragging and dropping an audio, rather than having to create the AudioStreamPlayer2D node, drag it's position, and then finally adding the audio_stream. This makes dragging and dropping files from the filesystem support more than just textures and scenes.
https://user-images.githubusercontent.com/12120644/129424485-75a95660-2875-4093-bdee-b1ac3f92c77c.mp4
A preview for the AudioStreamPlayer2D is created while dragging, even though it doesn't do anything. It would be nice if in a future PR, AudioStreamPlayer2D nodes could have a gizmo that shows it's location similar to 3D.

Needs rebase.

Note that GitHub doesn't appreciate force-pushes on branches whose name contain emoji, so you may have to open a new PR with only ASCII characters in the branch naml.

Note that GitHub seems to ignore force-pushes on branches whose name contain emoji

Does it? I didn't have such problems. Maybe once, but I don't know if it was related.

Note that GitHub seems to ignore force-pushes on branches whose name contain emoji, so you may have to open a new PR with only ASCII characters in the branch name.

Yeah, this seems to be the case


Allowing multiple redirect_uri for one clientId

Description
Please explain the changes you made here.
Checklist

[ ] Created tests which fail without the change (if possible)
[ ] All tests passing
[ ] Extended the documentation, if necessary

⚠ GitGuardian has uncovered 4 secrets following the scan of your pull request.
Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.
🔎 Detected hardcoded secrets in your pull request
🛠 Guidelines to remediate hardcoded secrets
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.Our GitHub checks need improvements? Share your feedbacks!

False positive (related to .env.test)


Add support for modal backdrop flag and container object

Addresses issue #38
The containerStyle is fine. The issue was with the new material-ui modal. Exposed the hideBackdrop and container props so it can be customized easier and allow embedding of the carousel in any component.

I added a ModalProps property instead, that can be used to customize the modal completely, e.g. ModalProps={{ hideBackdrop: true }}


Add Missing Python Versions to Tests for py-postgresql

Overview

Add tests for all supported Python versions to py-postgresql tox list.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 81.16%. Comparing base (67c62da) to head (75dba36).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1262      +/-   ##
==========================================
- Coverage   81.17%   81.16%   -0.02%
==========================================
  Files         200      200
  Lines       21971    21971
  Branches     3482     3482
==========================================
- Hits        17836    17833       -3
- Misses       2984     2987       +3
  Partials     1151     1151

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


tweak Ctrl-k behavior (request)

I'm used to zsh so i might be biased, but i almost always want Ctrl-k to only delete the current line, instead of deleting everything after the cursor
say i were to hit Ctrl-k in the following scenario, the | represents the cursor
{
|import com.duplicate.thing
import com.duplicate.thing
val lots = """
and
lots
of
lines
after
that
you
want
to
keep
"""
}

i'd prefer the result to simply remove the duplicate line, instead of the current behavior which would remove everything after the cursor and leave me with just
{

Go ahead and send a PR; many of these commands were ported from Bash, which does not have multiple lines at all, so it makes sense to tweak them to be more useful in a multi-line context

haven't dug into the code base at all yet, any pointers on where to start?

Probably here https://github.com/lihaoyi/Ammonite/blob/3ccb6b18cd5a18ce63070b2179884995125aaa57/terminal/src/main/scala/ammonite/terminal/filters/ReadlineFilters.scala#L151

Fixed by https://github.com/lihaoyi/Ammonite/pull/529, should be out in the next unstable build


added support for generating prediction output files compatible to official evaluation scripts

Hey @tscholak ,
#17
I have added code corresponding to generating output SQL files.
seq2seq/prediction_output.py this is a copy of the codalab_seq2seq.py file. Changed the names and fixed the above-mentioned bug.
configs/prediction_output.json this is a config file according to arguments required to run.
Config file discription:
inputs_path  : Path to dev.json file
output_path : prediction_sql.txt file location.
db_path : sqlite file path corresponding to dev.json file.
conversational : true incase of cosql_data or false for spider.
And changed to Makefile to make it invoke through prediction_output string.
https://github.com/kotalaraghava/picard/blob/95f3c2af0907409416f933bfd927b59ed89b9732/Makefile
I have tested it with previous docker images by mounting this code. and it's working fine.
I hope this helps.
Thanks,
Raghav

Hey @tscholak ,
I agree with all changes you suggested and I have made other commits containing all changes. kindly check once.
Let me know if you feel like any other changes are required.
Thanks,
Raghav

Thanks again for this contribution!


Update to swift 403

Update to swift403

Codecov Report

Merging #35 into master will decrease coverage by 2.31%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
- Coverage   79.03%   76.71%   -2.32%
==========================================
  Files           1        3       +2
  Lines          62      219     +157
  Branches        0       14      +14
==========================================
+ Hits           49      168     +119
- Misses         13       47      +34
- Partials        0        4       +4

Flag
Coverage Δ

#KituraCompression
76.71% <ø> (-2.32%)
:arrow_down:

Impacted Files
Coverage Δ

Tests/KituraCompressionTests/TestCompression.swift
65.65% <0%> (ø)

Tests/KituraCompressionTests/KituraTest.swift
93.61% <0%> (ø)

Sources/KituraCompression/Compression.swift
80.82% <0%> (+1.78%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ce9d9b8...d4f57a9. Read the comment docs.


Preview

https://user-images.githubusercontent.com/61240530/120516886-7b67d580-c3ed-11eb-9b01-d8f86763b290.mp4

That seems like a preview of different app.


Add data validation to the entire column of Excel.

🚀 Feature Proposal
Hello, Is it possible to add data validation to the entire column of Excel。
Motivation
I want to add data validation to the entire Excel column
Example
to use

Resolved in: https://github.com/exceljs/exceljs/issues/1109#issuecomment-582592549


Feature/custom help text

@Rob--W I have added an option to pass a different help text file to the server. Could you please review and merge if you see that it makes sense to you?
Cheers

Looks OK in general. Could you address the above review comments and squash all commits into one?
And my helpfile is plain text to be simple, but if the help page is configurable, it might be a good idea to also support HTML? E.g. if the extension is .html, then it's HTML, otherwise it's treated as plain text.

@Rob--W all reviews were addressed. Good idea about the html. Please review. cheers

@Rob--W I have made the changes. please review. cheers

There is a (small) rebase conflict. Could you rebase your commit on the current master, fix the linting errors and address one more nit? That'll take only a few minutes.
And I have just added CI to this repo, so you can immediately see test results after a push :)

@Rob--W do you know why github is saying there is an issue but when I go to travis everything is green? cheers

Because test coverage decreased. Don't worry about that.
Could you rebase instead of merge?
 I just don't know how you will be able to merge if github is showing the coverage error. cheers

I can still merge despite the error. After merging your patch I'll add some more tests (e.g. a test with a non-existent helpPath), which will improve the test coverage.

@Rob--W makes sense. cheers

Thanks for your patch.

@Rob--W thank you very much for the help.


Error management when editing profile

When something wrong happens in the ManageController/Index action (e.g. the user changes the email address but it's already in use by another user) an exception is thrown, resulting in a 500 response. Moreover, the details of the error are not clear enough, even locally: in the 'email address already in use' example above, all I get is:

ApplicationException: Unexpected error occurred setting email for user with ID ...

And nothing goes to the logs, so there's no way of understanding what's the problem apart from debugging it in VS.
For me it's quite urgent to solve this issue, so if you are busy I can fix it myself and open a PR, just let me know.

Please, open a PR if you can, because I am currently busy.
Do you try to setup email which is used for another user? I'll be happy for any fix/details on this. 😊
Thanks!

I think this line is your issue: https://github.com/skoruba/IdentityServer4.Admin/blob/master/src/Skoruba.IdentityServer4.STS.Identity/Controllers/ManageController.cs#L80
Please, check the result in this line.
Thanks.

Yes, exactly, and as you see when that call fails an exception is thrown. Instead I think it will be better to return a user-friendly message. I'll do it!

You are right, in this case will be better to have some nice message. :)
Thanks @aiscrim

Working on the fix I discovered a few other related flaws in the ManageController:

Here, the way StatusMessage is classified as success or danger is wrong: not all the errors start with the word "Error" (actually, most of the errors start with "Unexpected error..." and they would be classified as success), and anyway in languages other than English that would never work. I suggest replacing the string StatusMessage with an object (bool IsSuccess + string Message).
TwoFactorAuthenticationViewModel is missing the StatusMessage field, so even if the message is set in the calling action, it never gets displayed.
Some messages contain the user id, so they seem to be intended more for the log than for the UI, but they are displayed as StatusMessage. The pattern used for the PasswordChanged/PasswordChangedLog messages should be used everywhere imho.

If you agree I can fix the first two issues in the same PR. The last one is a bit trickier for me to solve because it involves creating new localized strings in all the supported languages, so I'd just ignore it atm.


Console warning "Could not copy inherited property "_super"."

I get console warning "Could not copy inherited property "_super"." after instantiating OpenSeadragon or after adding a tiledImage.
I think this warning added after https://github.com/openseadragon/openseadragon/pull/2212 for undefined descriptors.
But I'm not sure if should log warning for  "_super" property.

Interesting! I don't get that in the console when going to https://openseadragon.github.io/ (which includes #2212). Any idea why you're getting it? As far as I know, _super isn't a standard part of JavaScript... Is there something in your system adding it? Any idea what it's being added to?
I agree we probably don't need to be warning about it, but it would be good to know what it is. :)

@iangilman I'm using EmberJs frame work. You are right, I think Ember adds proxy on tracked properties to monitor changes. That "_super" prop most likely injected by Ember auto-tracking system.

Cool. So I guess the question is whether it's going to be a problem that we don't copy that over. Is the missing _super going to cause problems for Ember?


Kafka 0.10 support

Hi,
Does this library support Kafka 0.10 ?

Yes, its being tested with <IP_ADDRESS> with Travis: https://github.com/oleksiyk/kafka/blob/master/.travis.yml
But it doesn't support all 0.10 features.


cita-tool Switch signature algorithm

Using blake2b_simd and ed25519-dalek for signature algorithm.
#101

I am more worried about whether it is consistent before and after replacement.
Features version missing:
https://github.com/cryptape/cita-cli/blob/775cb9619cd31028e801371225c58f6543c01472/cita-cli/src/main.rs#L81

For blake2b, the key and personal values are consistent and I have a test for that.
For ed25519, all tests in source codes are passed.
So, in theory, it is consistent before and after replacement.
But no one can promise about that.

I believe that the author of the encryption algorithm and the hash algorithm should not have problems in the implementation.
Emmm, even if there was some problem, I seem don't have a good solution.😱
If it is confirmed the same before and after, it will be combined.


Patch hole and install doorstop in living room

French door has caused a hole. We need to patch the hole and install a doorstop.

Done.


Update external dependencies

Update PTHash to latest commit, so that cmd_line_parser does not take any positional arguments.

Done in 227091a84e53c2f4ec38c19e45940ae2442d620f.


Demonstration task for embedding charts in a PDF report

Ready for draft review.
@isaacbowen do you have a definitive stance on using multi-line assignments for public tasks? I notice very few tasks (if any) in the library use them. Personally, I find them useful when scanning a large task like this with a lot of similar assignments.
@mattsodomsky could use some help writing the task docs for this one.

@isaacbowen do you have a definitive stance on using multi-line assignments for public tasks? I notice very few tasks (if any) in the library use them. Personally, I find them useful when scanning a large task like this with a lot of similar assignments.

No stance. :) Your usage here looks good to me, and I appreciate you evolving the way things are done in this library.


Resource Cml runner failproof dependencies

To prevent not installing the stack by any meaning, the new strategy checks if a file at /var/logs/cml_stack.log exists to decide if the stack needs to be installed or not. So CML AMIs should have that file after setup.

closes #167
Dryer setup introducing go-bindata
Better base64 logic and test
Faster setup removing docker nvidia-smi test
Dryer template
Adds restyle config to avoid golden

fixes https://github.com/iterative/cml/issues/670

^fixes^mitigates^

@0x2b3bfa0 reviewed?

⏳


Kubeflow logout stuck in infinite loop

Describe the bug
I installed Kubeflow 1.4 using Cognito. I can access the Kubeflow UI and signin properly. When I click on logout button, it get stuck in infinite loop. As a result, user is not logged out.
Steps To Reproduce
1- Install Kubeflow from this guide: https://github.com/awslabs/kubeflow-manifests/tree/v1.4-branch/distributions/aws/examples/cognito
2- Sign in via Kubeflow UI
3- Logout via Kubeflow UI
Expected behavior
I expect user to be logged out and redirected to sign-in page
Additional context
This bug is related to the feature: https://github.com/kubeflow/kubeflow/pull/5174

Thanks for reporting the issue.
A workaround until this issue is resolved is to use an incognito window(Chrome)/private window(Firefox) to login to Kubeflow dashboard and close the window when user wants to logout. This works because the browser clears the cookies when the window is closed.

Hey Harmeet, I took a look into tackling this issue. It is not a bug since the kubeflow dashboards logout functionality is tied to "logout will initiate /logout which was implemented in oidc-authservice" which we are not utilizing/installing for our cognito example. To properly logout a user when using Cognito + AWS ALB is for the ALB cookies to be expired and then for the Cognito logout endpoint url to be hit.  These ALB cookies are Http-only and secure so they can only be delete server-side.

Currently have a istio VirtualService which allows for logout only once however subsequent logouts will not clear the cookies and only hit the redirect and not clear the cookies. However if the user manually deletes the cookies using their browsers console they can click on the logout button and be properly logged out.

I am currently working on writing a micro-service to handle the logout logic here, but am open to any suggestions or guidance of anyone who has more experience/knowledge of Istio/Envoy here which may make this easier.


[python] Experiment-level upgrader/resizer

Issue and/or context: As tracked on issue #2407 / [sc-51048].
Note that the intended Python and R API changes are all agreed on and finalized as described in #2407.
Changes:
Tests the experiment-level upgrader and resizer.
Notes for Reviewer:
There is more to do in tiledbsoma.io for append-mode ingest on #3148.
Example verbose output:
uri = "/var/ns/pbmc3k"
tiledbsoma.io.show_experiment_shapes(uri)

[DataFrame] obs
  URI file:///var/ns/pbmc3k/obs
  count                2638
  domain               ((0,<PHONE_NUMBER>854773758),)
  maxdomain            ((0,<PHONE_NUMBER>854773758),)
  upgraded             False

[DataFrame] ms/RNA/var
  URI file:///var/ns/pbmc3k/ms/RNA/var
  count                1838
  domain               ((0,<PHONE_NUMBER>854773758),)
  maxdomain            ((0,<PHONE_NUMBER>854773758),)
  upgraded             False

[SparseNDArray] ms/RNA/X/data
  URI file:///var/ns/pbmc3k/ms/RNA/X/data
  used_shape           ((0, 2637), (0, 1837))
  shape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             False

[SparseNDArray] ms/RNA/obsm/X_draw_graph_fr
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_draw_graph_fr
  used_shape           ((0, 2637), (0, 1))
  shape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             False

[SparseNDArray] ms/RNA/obsm/X_pca
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_pca
  used_shape           ((0, 2637), (0, 49))
  shape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             False

[SparseNDArray] ms/RNA/obsm/X_tsne
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_tsne
  used_shape           ((0, 2637), (0, 1))
  shape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             False

[SparseNDArray] ms/RNA/obsm/X_umap
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_umap
  used_shape           ((0, 2637), (0, 1))
  shape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             False

[SparseNDArray] ms/RNA/obsp/connectivities
  URI file:///var/ns/pbmc3k/ms/RNA/obsp/connectivities
  used_shape           ((0, 2637), (0, 2637))
  shape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             False

[SparseNDArray] ms/RNA/obsp/distances
  URI file:///var/ns/pbmc3k/ms/RNA/obsp/distances
  used_shape           ((0, 2637), (0, 2637))
  shape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             False

[SparseNDArray] ms/RNA/varm/PCs
  URI file:///var/ns/pbmc3k/ms/RNA/varm/PCs
  used_shape           ((0, 1837), (0, 49))
  shape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             False

[DataFrame] ms/raw/var
  URI file:///var/ns/pbmc3k/ms/raw/var
  count                13714
  domain               ((0,<PHONE_NUMBER>854773758),)
  maxdomain            ((0,<PHONE_NUMBER>854773758),)
  upgraded             False

[SparseNDArray] ms/raw/X/data
  URI file:///var/ns/pbmc3k/ms/raw/X/data
  used_shape           ((0, 2637), (0, 13713))
  shape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             False

uri = "/var/ns/pbmc3k"
tiledbsoma.io.upgrade_experiment_shapes(uri, verbose=True)

[DataFrame] obs
  URI file:///var/ns/pbmc3k/obs
  Applying upgrade_soma_joinid_shape(2638)

[DataFrame] ms/RNA/var
  URI file:///var/ns/pbmc3k/ms/RNA/var
  Applying upgrade_soma_joinid_shape(1838)

[SparseNDArray] ms/RNA/X/data
  URI file:///var/ns/pbmc3k/ms/RNA/X/data
  Applying upgrade_shape((2638, 1838))

[SparseNDArray] ms/RNA/obsm/X_draw_graph_fr
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_draw_graph_fr
  Applying upgrade_shape((2638, 2))

[SparseNDArray] ms/RNA/obsm/X_pca
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_pca
  Applying upgrade_shape((2638, 50))

[SparseNDArray] ms/RNA/obsm/X_tsne
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_tsne
  Applying upgrade_shape((2638, 2))

[SparseNDArray] ms/RNA/obsm/X_umap
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_umap
  Applying upgrade_shape((2638, 2))

[SparseNDArray] ms/RNA/obsp/connectivities
  URI file:///var/ns/pbmc3k/ms/RNA/obsp/connectivities
  Applying upgrade_shape((2638, 2638))

[SparseNDArray] ms/RNA/obsp/distances
  URI file:///var/ns/pbmc3k/ms/RNA/obsp/distances
  Applying upgrade_shape((2638, 2638))

[SparseNDArray] ms/RNA/varm/PCs
  URI file:///var/ns/pbmc3k/ms/RNA/varm/PCs
  Applying upgrade_shape((1838, 50))

[DataFrame] ms/raw/var
  URI file:///var/ns/pbmc3k/ms/raw/var
  Applying upgrade_soma_joinid_shape(13714)

[SparseNDArray] ms/raw/X/data
  URI file:///var/ns/pbmc3k/ms/raw/X/data
  Applying upgrade_shape((2638, 13714))

uri = "/var/ns/pbmc3k"
tiledbsoma.io.resize_experiment(uri, 2638, {"RNA": 5000, "raw": 13714})

[DataFrame] obs
  URI file:///var/ns/pbmc3k/obs
  Applying resize_soma_joinid_shape(2638)

[DataFrame] ms/RNA/var
  URI file:///var/ns/pbmc3k/ms/RNA/var
  Applying resize_soma_joinid_shape(5000)

[SparseNDArray] ms/RNA/X/data
  URI file:///var/ns/pbmc3k/ms/RNA/X/data
  Applying resize((2638, 5000))

[SparseNDArray] ms/RNA/obsm/X_draw_graph_fr
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_draw_graph_fr
  Applying resize((2638, 2))

[SparseNDArray] ms/RNA/obsm/X_pca
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_pca
  Applying resize((2638, 50))

[SparseNDArray] ms/RNA/obsm/X_tsne
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_tsne
  Applying resize((2638, 2))

[SparseNDArray] ms/RNA/obsm/X_umap
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_umap
  Applying resize((2638, 2))

[SparseNDArray] ms/RNA/obsp/connectivities
  URI file:///var/ns/pbmc3k/ms/RNA/obsp/connectivities
  Applying resize((2638, 2638))

[SparseNDArray] ms/RNA/obsp/distances
  URI file:///var/ns/pbmc3k/ms/RNA/obsp/distances
  Applying resize((2638, 2638))

[SparseNDArray] ms/RNA/varm/PCs
  URI file:///var/ns/pbmc3k/ms/RNA/varm/PCs
  Applying resize((5000, 50))

[DataFrame] ms/raw/var
  URI file:///var/ns/pbmc3k/ms/raw/var
  Applying resize_soma_joinid_shape(13714)

[SparseNDArray] ms/raw/X/data
  URI file:///var/ns/pbmc3k/ms/raw/X/data
  Applying resize((2638, 13714))

uri = "/var/ns/pbmc3k"
tiledbsoma.io.show_experiment_shapes(uri)

[DataFrame] obs
  URI file:///var/ns/pbmc3k/obs
  count                2638
  domain               ((0, 2637),)
  maxdomain            ((0,<PHONE_NUMBER>854773758),)
  upgraded             True

[DataFrame] ms/RNA/var
  URI file:///var/ns/pbmc3k/ms/RNA/var
  count                1838
  domain               ((0, 4999),)
  maxdomain            ((0,<PHONE_NUMBER>854773758),)
  upgraded             True

[SparseNDArray] ms/RNA/X/data
  URI file:///var/ns/pbmc3k/ms/RNA/X/data
  used_shape           ((0, 2637), (0, 1837))
  shape                (2638, 5000)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             True

[SparseNDArray] ms/RNA/obsm/X_draw_graph_fr
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_draw_graph_fr
  used_shape           ((0, 2637), (0, 1))
  shape                (2638, 2)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             True

[SparseNDArray] ms/RNA/obsm/X_pca
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_pca
  used_shape           ((0, 2637), (0, 49))
  shape                (2638, 50)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             True

[SparseNDArray] ms/RNA/obsm/X_tsne
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_tsne
  used_shape           ((0, 2637), (0, 1))
  shape                (2638, 2)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             True

[SparseNDArray] ms/RNA/obsm/X_umap
  URI file:///var/ns/pbmc3k/ms/RNA/obsm/X_umap
  used_shape           ((0, 2637), (0, 1))
  shape                (2638, 2)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             True

[SparseNDArray] ms/RNA/obsp/connectivities
  URI file:///var/ns/pbmc3k/ms/RNA/obsp/connectivities
  used_shape           ((0, 2637), (0, 2637))
  shape                (2638, 2638)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             True

[SparseNDArray] ms/RNA/obsp/distances
  URI file:///var/ns/pbmc3k/ms/RNA/obsp/distances
  used_shape           ((0, 2637), (0, 2637))
  shape                (2638, 2638)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             True

[SparseNDArray] ms/RNA/varm/PCs
  URI file:///var/ns/pbmc3k/ms/RNA/varm/PCs
  used_shape           ((0, 1837), (0, 49))
  shape                (5000, 50)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             True

[DataFrame] ms/raw/var
  URI file:///var/ns/pbmc3k/ms/raw/var
  count                13714
  domain               ((0, 13713),)
  maxdomain            ((0,<PHONE_NUMBER>854773758),)
  upgraded             True

[SparseNDArray] ms/raw/X/data
  URI file:///var/ns/pbmc3k/ms/raw/X/data
  used_shape           ((0, 2637), (0, 13713))
  shape                (2638, 13714)
  maxshape<PHONE_NUMBER>854773759,<PHONE_NUMBER>854773759)
  upgraded             True

@nguyenv I decided not to go with attrs over TypedDict -- sort of a po-tay-to po-tah-to ...


Preserve NaNs in groupby operation

In make_time_series() in data_analysis_utils.py the groupby operation drops NaNs (months with no images). We probably want to preserve these so that subsequent plots (e.g. autocorrelation) make more sense.
One solution is to (linearly) interpolate for those months in which we do not have data. Though this can gaps in the data when plotting a time series.

Here is an example of the problem showing (blue) the autocorrelation plot with missing months, and (orange) the autocorrelation plot with linear interpolation for missing data

Strictly might not need to fix this, but it is worth being aware of it


Release alpha 3

@mcollina Wrongly I pushed a merge from master to next 😅
In order to merge fastify's master to next I need this fix #96 : some tests are failing cause of this.
Could you (or may I) release an alpha-3 version?

Released!

👍


Inconsistent {CMD}: <MGET command in dice/docs isn't matching while running it on dicedb-cli>

This issue has been raised in accordance to fix issue #823
Steps to reproduce

go to the dicedb-cli after running dicedb server
on cli, SET key "value1"
on cli, LPUSH key4 "value2"
MGET key key4

Expected output (given in documentation of the command)

This is the output that given in the dicedb documentation
{expected_output}

Observed output

This is the case when keys doesn't exist previously.
NOTE: on the redis-cli and dicedb-cli it's giving the same output for the above command. one thing I have observed is that let's say key4 isn't existence then LPUSH will make a list for key4 and store its value but if keys exists then it will throw the error immediately after executing the LPUSH command. I am attaching for a case also when keys exists.

{observed_output}

Expectations for resolution
This issue will be considered resolved when the following things are done

changes in the dice code to meet the expected behavior
addition of relevant test case to ensure we catch the regression

You can find the tests under the integration_tests directory of the dice repository and the steps to run are in the README file. Refer to the following links to set up DiceDB and Redis 7.2.5 locally

setup DiceDB locally
setup Redis 7.2.5 locally
setup DiceDB CLI

Hey @lucifercr07  can you assign this to me .I would like to work on it.

This issue should be resolved once https://github.com/DiceDB/dice/issues/823 is fixed. Assigning it to the assignee of #823

Hi @shashi-sah2003. Is someone working on this issue? I would like work on this if no one else is assigned for this issue.

Assigned.


create model using python & (fix) exposure of joints

The type of joint used as a root joint can now be specified by the user in C++ and in Python

Add the possibility to create a model from scratch in python. To do so, first create an empty model using BuildEmptyModel(), then add one by one your joints with addBody(..args..)

Also fixing a missing functionality : you now have properly access to the joints of the model.

Exemple
import pinocchio as se3

from pinocchio.romeo_wrapper import RomeoWrapper

myModel = se3.Model.BuildEmptyModel()

myModel.addBody(0, se3.JointModelFreeFlyer(), se3.SE3.Random(), se3.Inertia.Random(),"j1","b1",True)
myModel.addBody(1, se3.JointModelRX(), se3.SE3.Random(), se3.Inertia.Random(),"j2","b2",True)

joint =  myModel.joints[2]
print joint.idx_q
print joint.nq

@jcarpent new version of pull request.
parseTree and buildModel have been reworked to allow to be used with or without specifying a root joint. More user friendly.
Again: there are two methods in python to build a model because the macro BOOST_PYTHON_MEMBER_FUNCTION_OVERLOADS seems not be usable because of the different scopes


client.RegisterAsync, no timer/"button not pressed"

Hello,
This could be due to my own ignorance but I'm struggling to use client.RegisterAsync.
I keep getting "Button not pressed", the light doesn't blink on the bridge, but it makes sense that the command is being sent out, doesn't get a response, and returns "Button not pressed".
Am I missing something simple, like a correct wait statement or something?
    class Program
    {
        static void Main(string[] args)
        {
            Task tasker = thisDoesntWork();

            //thisWorks();

            Console.ReadLine();
        }

        static async Task thisDoesntWork()
        {

            ILocalHueClient client = new LocalHueClient("<IP_ADDRESS>");
            string appKey = await client.RegisterAsync("mypersonalappname", "mydevicename");

            //Crash occures before this line, but I get "Button not pressed" in the debugger, so I think it's executing.

            Console.WriteLine(appKey);

        }

        public static void thisWorks()
        {

            ILocalHueClient client = new LocalHueClient("<IP_ADDRESS>");

            client.Initialize("139f12ce32a30c473368dbe25f6586b");

            var command = new LightCommand();
            command.On = true;

            client.SendCommandAsync(command);

        }
    }

I'm able to use client.Initialize with a appKey I generated from the hue debug api, and I'm able to send basic light commands with your library, so I don't think this is a connection issue.
Any help is greatly appreciated.

You first need to press the button on the bridge and then send the command within 30 seconds, see the documentation here:
http://www.developers.meethue.com/documentation/configuration-api#71_create_user
Also see this example for using async and await in a console app:
http://stackoverflow.com/a/17630538

Worked like a charm, thank you sir!


Wrong information about the scope needed to change the authentication type for a domain

[Enter feedback here]
To be able to convert a domain from Federated to Managed, the scope need to be  "Domain.ReadWrite.All", "Directory.AccessAsUser.All".
So the article need to be corrected as below :
Connect-MGGraph -Scopes "Domain.ReadWrite.All", "Directory.AccessAsUser.All"
Update-MgDomain -DomainId "test.contoso.com" -BodyParameter @{AuthenticationType="Managed"}
Regards,

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

ID: 02c07bd2-01c5-77f4-11e0-6d942c5c0885
Version Independent ID: f61490c2-0f18-4609-29f1-99ccb4215ad2
Content: Change subdomain authentication type using PowerShell and Graph - Microsoft Entra ID
Content Source: docs/identity/users/domains-verify-custom-subdomain.md
Service: entra-id
Sub-service: users
GitHub Login: @barclayn
Microsoft Alias: barclayn

@JK-MS1
Thanks for your feedback! We will investigate and update as appropriate.

@JK-MS1
Thank you for bringing this to our attention! We've initiated a pull request (PR) to address this issue. Once the author reviews the changes, they will be merged, and the updates should go live shortly. Your diligence is much appreciated!


AUR Package

Greetings,
I made an AUR package - can you please check it television-bin. I have installed tv by this and it works well. If it seems fine, maybe you can add it in readme.
Thank you.

Hi, thanks for showing interest in the project.
I think this should be added as a CD workflow to be done automatically with each release.
Also I changed version tags from vx.y.z to x.y.z in recent releases which means your PKGBUILD will need a couple of changes for these.
Are you willing to start a PR for that?

yes, i well try to get both done, i have a bit of a busy week on hand, so it might be delayed a bit

television is now an official arch package, so aur '-bin' package is not required anymore, so i am closing this.


h5netcdf new API support

Closes #1536
Support arbitrary compression plugins through the h5netcdf new API.
Done:

public API and docstrings (untested)
implementation
unit tests
What's New

I couldn't find a .stickler.yml file in this repository. I can make one for you, or you can create one by following the documentation.

I couldn't find a .stickler.yml file in this repository. I can make one for you, or you can create one by following the documentation.

@crusaderky - if you sync with master, you'll stop getting the stickler-ci messages.

@crusaderky Yes, I like the look of that better. It's slightly unfortunate that it's taking a while for the netCDF4 library to figure this out so it's not clear the API will be yet. We might not want to keep the separate new API backend with h5netcdf in the long term, but we certainly would not remove/deprecate it until everything is supported with the legacy API.
One minor concern is the name. I don't know what h5netcdf-ng stands for ("next generation?"). I would prefer h5netcdf-new.

Almost finished - only one outstanding unit test failure when specifying the "chunks" parameter

@shoyer finished and ready for final code review and integration

Generally looks great.
What if both zlib=True and compression='lzf' is set in encoding? This could potentially happen, e.g., if reading from a file with zlib encoding and attempting to write as lzf. I think the compression argument should take a precedence, but we should add a test case to verify that this works.

@shoyer done

@shoyer / @jhamman , stickler-ci has been pending for a day now - I'm afraid it should be pronounced dead

closing/reopening seemed to do the trick for @stickler-ci.

@shoyer all green and ready for merge

Thanks @crusaderky !


Newby unable to follow README instructions

Hi as a Newby I tried to follow the instructions from the README but no success.
pip install -r requirements.txt failed on the python version:none (I have python 3.6.8 installed en pip 20.something, thus as a fallback I manually installed all imported modules using pip)
python main.py, intensions of the meal bot are not included in this repository... where to get it?

Hello @pdorrestijn , sorry for the late answer.
Did you install all the required modules? If so, would you quote the exact error you get when trying to execute the main file? I think they may be some problems with Tensorflow version. But I have to check the error message to be sure about that.

Hello @pdorrestijn , sorry for the late answer.
Did you install all the required modules? If so, would you quote the exact error you get when trying to execute the main file? I think they may be some problems with Tensorflow version. But I have to check the error message to be sure about that.


Make Streamus' search input look like Google Inbox

Hi, first of all I like the product you made and I am more than happy to help/contribute as I could. I made some changes to the search input styles.
Search input focused

Search input filled

How do you think of that?

Hey,
This is a duplicate of #378 :)
I do agree that it needs to look something like that. Here's how Google's looks:

But there's reason why the 'X' icon to be there right? It dismisses the search panel when user click on it. In my opinion, no need for that 'magnifying glass' icon, in exchange keep the 'X' icon.

In my overall 'vision' of how it would look -- the magnifying glass would slide to the left as the search slides open and it would end up covering the 'playlist' button since you can only have one or the other open as it is.
Other concerns are just that there's no way to clear the text in the search input right now -- since 'X' closes the window entirely a lot of people who don't use ctrl+a + del to clear the search are left out to dry.

Here's my input:

When user clicks the Streamus icon, autofocus the search box and give it maybe some more visual precedence and/or a very subtle pulsing animation to make it obvious it is focused.  It may seem weird at first, but the two use cases I see are:

Click streamus -> click on a button like 'play', 'pause', 'next', 'volume', etc. (which would unfocus the search box anyway)
Click streamus -> click on search, type something

I don't see any reason not to assume the user will be searching off the bat as you could collapse the search box if they click a button other than the search icon/input.  Aesthetically, I love the animation idea you have with sliding the magnifying glass, but my favorite thing about Streamus is the speed of use.  Personally, I would gladly trade off some satisfying animations in order to save clicks or keystrokes.

When search input is opened and nothing is typed, show the 'X' icon and let it behave as it does now... but if the user has entered anything, replace the 'X' with an eraser icon.

If the eraser icon is shown and the user hits the ESC key, clear the text and show the 'X' icon.
If the eraser icon is shown and the user manually removes the search (ctrl-a, backspace, etc), then hide the eraser and show the 'X' again with current behavior.

I've been using it very heavily for the past week or two and it is amazing. Hands down my favorite music streaming software I have ever used.  <3  Thanks for all your work.

Porting some comments over to #524


contrastLimits possibly unbound in reader

this line here can lead to a possible error local variable 'contrastLimits' referenced before assignment depending on the dtype
https://github.com/CBI-PITT/napari-imaris-loader/blob/fccb04f4dafda8e3f16a38150db88faaa5597fc6/napari_imaris_loader/reader.py#L106

Thanks for submitting this issue.  I see how this might happen since the assignment of contrastLimits is not guaranteed.  Do you have a specific example / traceback that caused an issue?  Willing to share the .Ims file?  In any case I will make a note to fix.  Thanks!

'fraid I don't have the file or a better traceback.  Saw this as a bug that came in through https://github.com/tlambert03/napari-error-reporter ... probably won't be passing these on to plugins forever, but it's still new so I'm doing it for now :)

Understood.  Thanks for passing it along.  I will fix...  Best,
Alan

Issue fixed by ensuring that contrastLimits always receives a value default [0,1]:  v0.1.6


Unit tooling improvements

I ran into errors when loading my dashboard locally, as unit was empty ("") for some assets' sensors.
I then also solved two more smaller issues with the unit utilities.
We can also discuss protecting these utils better or at some other place against unit string being incompatible with pint.

Good improvements. Indeed I'd like to validate on input, too. I believe that would be sensor crud logic.


Full screen

When running throught a course, after the lesson is completed the new lesson opens without full screen. Ok. I can always click "F" to put it on full screen - but it never works. When I'm in one lesson and the video is on full screen, I can exit the full screen with "F" and it works with the "F" from now on but with the new lesson I always have to click on the button - even clicking anywhere on the website what could activate the ability of "F" shortcut is not working - which is really annoying as some lessons have only a few minutes.

You should first click the video area before clicking F to enter full screen after the new lecture shows up. That could work, i think.
P.S. this is not about our extension. It is related to the learning platform, Teachable.


Deprecated sub packages and ReDOS vulnerability

When I do npm i:
npm WARN deprecated<EMAIL_ADDRESS>Jade has been renamed to pug, please install the latest version of pug instead of jade
npm WARN deprecated<EMAIL_ADDRESS>ReDoS vulnerability parsing Set-Cookie https://nodesecurity.io/advisories/130
npm WARN deprecated<EMAIL_ADDRESS>Deprecated, use jstransformer
npm WARN deprecated<EMAIL_ADDRESS>ReDoS vulnerability parsing Set-Cookie https://nodesecurity.io/advisories/130
npm WARN deprecated<EMAIL_ADDRESS>to-iso-string has been deprecated, use @segment/to-iso-string instead.
npm WARN deprecated<EMAIL_ADDRESS>Jade has been renamed to pug, please install the latest version of pug instead of jade
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN prefer global<EMAIL_ADDRESS>should be installed with -g

Is it safe to ignore the warnings or should I do something?

Yes you can safely ignore them. Other than the first jade warning, there is not much I can do about sub-dependencies of Hackathon Stater's dependencies.

Can you remove/replace the packages that need those insecure sub-dependencies?
Maybe we can create issues for those packages.

I cannot remove these packages, even if I track them down somehow, I still need them for this project to function correctly.
Really, there is no need to worry, it's just a warning. When and if things break, those packages will be updated.


GTA V Crack (𝟸𝟶𝟸𝟺) +𝙾𝚗𝚕𝚢 𝙳𝚘𝚠𝚗𝚕𝚘𝚊𝚍 𝙵𝚛𝚎𝚎 𝚏𝚘𝚛 𝙿𝙲 𝚁𝚎𝚕𝚘𝚊𝚍𝚎𝚍 𝙻𝚊𝚝𝚎𝚜𝚝

Grand Theft Auto V Crack is an action-adventure video game developed by Rockstar North and published by Rockstar Games. Released in September 2013, it is set in the fictional state of San Andreas, which is based on Southern California.
➤ ►👉 Do𝚠nlo𝚊d GTA V  (𝚁𝚎𝚕𝚘𝚊𝚍𝚎𝚍) Crack
The game features an open world that allows players to explore the city of Los Santos and its surrounding countryside.


Add geometric functions, unit test improvements

This adds the SYCL geometric functions as per spec, tests them, and also includes various unit test enhancements.

Thank you, looks great!


Allow setting base directory when calling getMesh

When pwd != the directory of the entry point of the app the path of referenced is files in the mesh config would have to be changed between environments. For example:

nx + nestjs + graphql + mesh (added via getMesh)

in development when serving

pwd is {workspaceroot}
the files being served are located (by default) in {workspaceroot}/dist/apps/myapp/

in production

pwd is {rootofappfiles}
files are located {rootofappfiles}

To ensure that both environments can be served with the same config it would be helpful to be able to specify the base directory that is used for anything that gets executed when calling getMesh.
When looking at the code it seems that pwd is used as base dir across functions. There is no passing of a reference directory from the parent fucntions to its children.
The working solution we came up with is to change the pwd before execution of getMesh and afterwards changing it back to its original value. This works like a charm, but since it cost us 2 days to come up with this I felt it might make sense to have an additional option (dir) when calling getMesh that does this under the cover for the user.

Hi @ntziolis,
I believe you should already be able to do what you're after.
My recommendation is to place your Mesh Config file (.meshrc.yaml) in the director you want to set as a base-dir.
Then you can tell Mesh to use that as a base directory like this:
findAndParseConfig({ dir: yourBaseDir })
Setting base directory is indeed already supported, and it works also on CLI commands, like this:
graphql-mesh write-introspection-cache --dir ./src/yourBaseDir
Give it a try and please let us know if this doesn't address your request.

Hi @santino,
this is what we tried at first. When using findAndParseConfig({ dir: yourBaseDir }), the config is successfully found and parsed.
However the resulting configuration object does not retain the dir information (we inspected it during debugging). So when using this config object to call getMesh, the latter has no way of knowing what the dir value was that was used to find and parse the config.
That said, even if the config object would have the base dir information:
We have debugged the execution of getMesh to the point where .js files are imported (eg for resolver composition). What we found is that:

The code used to import always uses cwd() as base dir for relative paths.
There are no parameters in the calling method chain to pass in a base dir from outside to influence that.
There is no change of pwd to the place the mesh config is actually located in the chain of calls when executing getMesh.

Based these findings we saw 2 general approaches to solve this:

Changing all relevant calls in the getMesh call chain include a parameter or extend existing options to include dir and ensure it is being used instead of pwd()
Or change the pwd at the beginning of getMesh and change it back before the function completes

No matter which option is chosen we agree, it would make sense to include the base dir used as part of the parsed config object. This way the getMesh call wouldn't need an additional parameter.
Please note:

While it would be possible to use absolute paths
I our case the absolute paths for development are different than for production, hence we need to use relative paths, preferably relative to the location of the get mesh config

@ntziolis Each handler can m individually take baseDir parameter;
new OpenAPIHandler({
   // ...
   baseDir: __dirname
})

I apologize. Before filing this issue I thought I ensured to install the latest versions of all mesh libs but I must have forgotten to specify @latest.... shameonme
For anyone finding this:
The reported issue does NOT exist in the current version of the libs anymore. Providing the base dir to find and parse config as @santino suggested works exactly as intended.

I see, that makes sense now.
For reference, the issue you reported was fixed with this PR.
The release with the corrected behavior is available since the 25th of Feb 😉


Support forward/backward navigation in the types view

What is the feature you'd like to have?
As u know, we can navigate in the code window, and if we like, we can use Esc to go back to the last view.
But in Types, we will encounter many cases that many objects and members cross together, and we need double click to goto the deep defination of the member.
But now ,we can not navigate back. We have to copy paste the name to Types window and check.
Is your feature request related to a problem?
NO
Are any alternative solutions acceptable?
Bind Key maybe can solve
Additional Information:

I also sometimes wish to go to a particular offset in a structure by pressing G. Though I am not sure how this could be implemented, since it will conflict with the code view navigation


Uncaught TypeError: undefined is not a function

The line number was tree.js:77
TreeNode.addRelativePath(node-v4.2.1/test/fixtures/keys/0-dns-cert.pem)

I cant test the tree render because this still happens!

This happen when one of the folder name is 'keys' and all javascripts object have keys method built in so this resulted in retreiving the keys object method.  Which caused the problem.
Fixed by reverse back to use array instead of object for node Array.


Fix broken link

Codecov Report

Merging #67 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #67   +/-   ##
=======================================
  Coverage   77.21%   77.21%
=======================================
  Files          45       45
  Lines        2611     2611
=======================================
  Hits         2016     2016
  Misses        595      595

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 879d394...4b21a3e. Read the comment docs.


fix: Remove unnecessary sourcemap overwrite during transform

Summary
In our project we use a different loader setting for TypeScript files from what's recommended in the Recipes.
TS files are loaded by ts-loader then a babelWorkerLoader with metro preset:
// Ugly experimental code below
const babelLoaderRule = config.module!.rules[1];
const babelLoaderDef = (babelLoaderRule.use as webpack.RuleSetUseItem[])[0] as webpack.RuleSetLoader;
(babelLoaderDef.options! as any).presets = [[require.resolve('metro-react-native-babel-preset'), { disableImportExportTransform: false }]];

config.module!.rules.push({
  test: /\.tsx?$/,
  loader: 'ts-loader',
  options: {
    onlyCompileBundledFiles: true,
    transpileOnly: env.bundleTarget !== 'file',
    compilerOptions: {
      declaration: false,
      sourceMap: true
    }
  }
});

So what babelWorkerLoader receives as 'sources' is the compiled JS code, with a sourcemap that points back to original TypeScript file. However, transform.js then overwrites the result sourcemap with the compiled JS code (because it thinks it's the real source), resulting in the loss of TypeScript code from sourcemap.
By removing the overwrite, this PR preserves the original source that comes before babelWorkerLoader.
While our loader setup is somewhat unusual, I'm curious about the reason for the overwrite logic introduced by #689. If this PR is rejected, some explanation would be truly helpful 👍
Test plan

cc @NoemiRozpara (original author of #689), thank you!

@statm babelWorkerLoader is heavily based on babel-loader and they overwrite the source maps in their transform as well: https://github.com/babel/babel-loader/blob/master/src/transform.js#L24-L26

@zamotany Interesting! I'll convert this back to a draft while doing more research.

@zamotany The TypeScript paraphrase of original babel-loader's transform.js is inaccurate.
Original transform.js:
if (map && (!map.sourcesContent || !map.sourcesContent.length)) {
  map.sourcesContent = [source];
}

(Fill map.sourcesContent if it's empty.)
Haul's transform.ts with optional chaining:
// TS
if (map?.sourcesContent?.length) {
  map.sourcesContent = [source];
}

// Compiled JS
if (map && map.sourcesContent && map.sourcesContent.length) {
  map.sourcesContent = [source];
}

(Fill map.sourcesContent if it's not empty.)
I just made another push to fix the logic.


Feature: Per Monitor VRR Toggle

As per https://github.com/alex-courtis/way-displays/issues/59#issuecomment-1453204405, a per monitor VRR toggle would be quite useful.

VRR enabled is the desirable (current) default.
It could be disabled as per DISABLED e.g.
VRR_OFF:
  - "eDP-1"
  - !.*Gigabyte.*

@p0da what is the use case for disabling VRR? I think I might be missing something that should be captured/configured.

@p0da VRR_OFF has been added. I would be most grateful if you could test on master this prior to release:
git clone git@github.com:alex-courtis/way-displays.git
cd way-displays
make
sudo make install

See https://github.com/alex-courtis/way-displays/blob/master/doc/CONFIGURATION.md#vrr_off

Thanks for adding the option, worked well for me during testing.

Many thanks.


fix: Provide Endpoint hasher to work around URLQueryItem bug

Issue #

Description of changes

Scope

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

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

When I run EndpointTests on Linux without the changes to Endpoint.swift, this is the (expected!) test failure I get:
Test Case 'EndpointTests.test_hashableAndEquatable_hashesMatchWhenURLQueryItemsAreEqual' started at 2023-04-25 02:40:17.557
smithy-swift/Tests/ClientRuntimeTests/NetworkingTests/EndpointTests.swift:29: error: EndpointTests.test_hashableAndEquatable_hashesMatchWhenURLQueryItemsAreEqual : XCTAssertEqual failed: ("1202833612664289731") is not equal to ("2786467153059950206") -
Test Case 'EndpointTests.test_hashableAndEquatable_hashesMatchWhenURLQueryItemsAreEqual' failed (0.001 seconds)


Populating Hidden Field on Form with Javascript

I have a form that requires a record to be populated with the Account Id it is associated with. I use the code below to populate the Connected Account field_218. I was hiding the field_218 on the form using _hf= but the field stopped populating unless it is visible on the form. When I created this awhile back the fields was populating when hidden but something seems to have changed so that the field can't be hidden now.
$(document).on('knack-view-render.view_187', function(event, view, data) { 	// The connected record id value 	//alert(contactId) 	var connectionId = contactId;  console.log('view_187 Connection ID set to: ' + connectionId); 	// Update the view and field ids for your application 	$('#view_187-field_218').val(connectionId); 	$('#view_187-field_218').trigger("liszt:updated"); });

@djbbiz,
So, if you don't hide the field, all is working as expected?  (did you actually try that?)
BTW, this is your code, formatted for improved readability:
$(document).on('knack-view-render.view_187', function (event, view, data) {
    // The connected record id value
    //alert(contactId)
    var connectionId = contactId;
    console.log('view_187 Connection ID set to: ' + connectionId);
    // Update the view and field ids for your application
    $('#view_187-field_218').val(connectionId);
    $('#view_187-field_218').trigger("liszt:updated");
});

I tried to log in the app you gave me access a while ago (tedtasks), but I think it's another one.  I don't see anything related to view_187 or field_218.
If you could give access to the new app, it would help a lot.
Norm

That is correct when I take off the hidden keyword the account field is populating properly. And you are correct it isn't in TEDtasks it is in another app. I will contact you directly with more info.

I believe you are correct about the use of the ktl Class but _hf was different and working before for this purpose. Strangely it still appears in the DOM as you can see so it isn't truly gone from the DOM

The above was a test, here is the original issue and you can see that I have the Account field_218 hidden in the form yet it appears in the DOM and I am in the Test User and my javascript is indeed acting with the field in the DOM and setting the Account field to Test User. But when I save it isn't recording the Test User Account so I suspect maybe there is another issue that is deleting that selection before my saving the form. But not an issue when the field is visible.

@djbbiz,
Try the latest release v0.25.6 please.
Norm

Hey Norm, I just tested and everything seems to be back to normal and working. Thanks for a fast fix!

Great, very happy to read that - thanks for reporting!
Norm


Fix github CI

can you elaboarte the issue? @KarlOfDuty

The project doesnt build at the moment, so should fix that.

@KarlOfDuty hi, i have successfully build this , can i raise a PR for this and if you want you can check the build here

You are building it using the wrong dotnet version, this is supposed to be .NET Framework 4.7.2. You can check the project file for more info.

oh actually the ci  was earlier using 3. something that's why i did it let me check again

Could not find ".NET Core SDK" with version = 4.7.2 i am getting an error like this , should i use an upper version?

It is not a .NET Core version, it is a .NET Framework version.

@KarlOfDuty  just for clarification, you mean this version should be 4.7.2

I don't know the syntax for github actions, but you can look at my jenkins jobs for my scpsl plugins if you want to see how they are run.

@KarlOfDuty yeah please tell me where i can find Jenkins jobs i want to see

Check SCPDiscord or SCPermissions for instance

@KarlOfDuty i did not got it, do i have to join discord

No, just check my github repos

@KarlOfDuty hi, i am thinking of rewriting this github actions, can you please tell me sln files that are used for build?

I have finished up the smod update so I think I will fix this too now.

I would like to help you fixing this🙂 if you allow

Sorry but I kinda want to get this done now

@KarlOfDuty well i am available to do it now or its your wish if you want to proceed you can :)

I added it now, but thank you anyway.


Upgrade weave to v2.1.3, move to using the new network policy API

Fixes #997, fixes #998

LGTM! Do we need anything in the release notes so that users know they have to change their policy resources?

That sounds good, I will include it in the release notes for this issue.


programmatically call AVA tests

In the case that a developer wants to pass data to a test file programmatically (not at the command line)
with Mocha, seemingly the only way to do this would be:
function wrapTheMochaSuiteWithThisFn(data){

  describe('root suite', function(){

      describe('child suite', function(){

       data.forEach( item => {
             it('stubbed test case');
        });

     });

  });

}

if(require.main === module){
   wrapTheMochaSuiteWithThisFn([]);  // if test file is executing directly, run it
}

module.exports = wrapTheMochaSuiteWithThisFn;

will we have to do the same thing with AVA if we want to pass data / inject data into tests?

See: https://github.com/sindresorhus/ava/issues/695

Currently you have to do it the same way you do it in mocha. We are discussing an alternate solution in #695

ah thanks


Issue with the template css

Hello there,
I have an issue where whenever I try to install the template I get my page like this picture

So I'm trying to install it with laravel..
I tried to follow the instruction in coreui doc and I tried to follow a tutorial on youtube made by laravel daily but both give the same result

I have the same problem.

I even posted on stack overflow and in Coreui community and seems like no one knows what's going on

What is your version of @coreui/coreui in node_modules?

The version I had is "version": "2.0.1"

I asked about the version of @coreui/coreui library, go to node_modules/@coreui/coreui/ and check the version in package.json

The version I had  "version": "4.0.1"

You have to lock the version of @coreui/coreui in package.json
Change
"@coreui/coreui": "^3.2.0"
to
"@coreui/coreui": "3.4.0",
and reinstall using npm install

ok i will tested thanks for your feedback

I checked the node_modules/@coreui/coreui/ and check the version in package.json. The version was "3.3.0" and I changed it to "3.4.0" and ran npm install but still same problem

BTW, whenever I run npm install I get this message
up to date, audited 1994 packages in 4s

111 packages are looking for funding
  run `npm fund` for details

56 vulnerabilities (5 low, 20 moderate, 28 high, 3 critical)

To address issues that do not require attention, run:
  npm audit fix

To address all issues possible (including breaking changes), run:
  npm audit fix --force

Some issues need review, and may require choosing
a different dependency.

Run `npm audit` for details.

it working for me i changed
the link to
https://unpkg.com/<EMAIL_ADDRESS>

I finally made it work,
Thanks Mrholek and hmzaaa
I appreciate the help

with pleasure sir
thanks to you and marholek for help
what change do you make ?

Yes,faced the same problem!!
The problem was with cdn links
updated them to version 3.4.0

I got pages like this

check your cdn links for css and js.
Change the version numbers included with the links.
في الأحد، 3 أكتوبر 2021 في 5:07 م تمت كتابة ما يلي بواسطة
SmithaSadanandan @.***>:

I got pages like this
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/coreui/coreui-free-laravel-admin-template/issues/69#issuecomment-932958624,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUIO6UVWEKWESO4MZBSXIMDUFBPQTANCNFSM5AEDZHGA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Issues regarding the function 'convert to html'

When I use the markdown tools to convert .md file to .html file, a problem occurred:
"Running the contributed command:'markdown.extension.printToHtml' failed"

Is this a bug? Or did I miss something in my vscode?
Thanks for help!

Could you please open the console to see if there is any error message?
(menubar > help > toggle developer tools)

Neilsustc, did you update the plugin? I tried today (Dec 5th at UTF+8), and it works all well!
Anyway, thanks for help! I will let you know if it happen again (but I hope it will never happen again ^_^).

No updates these days. But it is okay since it works now 😉


Add stackblitz to this repo

Is your feature request related to a problem? Please describe.

We already have a stackblitz demo but it is in a separate repo. Because of that, it can be easily outdated. Let's add this stackblitz as a part of this repository so that it can be easily updated with a new version.
Describe the solution you'd like

Describe alternatives you've considered

Additional context

Just like I did here -> https://github.com/Baroshem/nuxt-security/tree/main/stackblitz
https://github.com/Baroshem/nuxt-security/blob/78cecd15263472d619dd9306b71b491139cf2d4b/package.json#L40
https://github.com/Baroshem/nuxt-security/blob/78cecd15263472d619dd9306b71b491139cf2d4b/package.json#L60


Simple gradient descent works for callables without gradient

Description
Previously, the simple gradient descent optimiser accepted both callables and callables with gradient. However, when passing a callable, it would not compute gradients with a numerical approximation routine, and the optimisation would fail. This PR wraps the incoming cost function, if it's just a callable, to convert it to a callable with gradient by using finite differences.
Please verify that you have completed the following steps

[x] I have self-reviewed my code.
[x] I have included test cases validating introduced feature/fix.
[x] I have updated documentation.

Codecov Report
Base: 94.95% // Head: 94.02% // Decreases project coverage by -0.92% :warning:

Coverage data is based on head (d4ed4e6) compared to base (76e96c9).
Patch coverage: 63.33% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #30      +/-   ##
==========================================
- Coverage   94.95%   94.02%   -0.93%
==========================================
  Files          28       28
  Lines        1011     1038      +27
==========================================
+ Hits          960      976      +16
- Misses         51       62      +11

Impacted Files
Coverage Δ

...rquestra/opt/optimizers/simple_gradient_descent.py
78.43% <62.06%> (-21.57%)
:arrow_down:

...tra/opt/optimizers/pso/continuous_pso_optimizer.py
84.21% <100.00%> (ø)

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.


Add Role Assignment when added to tech stack - 👌🔥  Įⓝ丅€ᖇ𝔢𝕤𝐭ᵉᗪ ¢Ｏᵈ𝐄𝐑  🐊♥

u can make something like when u add a tech to your stack
u get a role or smthin
and in the help channel
it will ask which tech and ping it
That’s a good idea, but we would need to make a way to “match” tech names so we don’t have an infinite amount of roles.
eg if the users enters “react” and “reactjs” he will be give 1 single role “react”

i 100% cant do it
so anyone else would be welcome to ask @al5ina5 wether u can work on this

I don't think roles are good way to do this stuff.
First of all everyone will have a lot of roles. And it is hard to somehow merge 'react' and 'reactjs', into one thing. It will require time to manually watch for tech added and merge them into something one, like Frontend developer (react/vue/angular/jquery idk),
What can be useful I though about, is creating new help rooms when someone asks joie about help and close them later.
Someone calls joie, +helpme [tech] -> +helpme angular
Joie searches for users that have this tech, creates help room and adds everyone with [tech] to this channel, and maybe it can be closed? So only one who asked and who can help are in it. If no one has this [tech] send message "Sorry no one has [tech] in their stack, ask for help in help room".
About closing channels, close them only when

person who asked calls +thankyou all or @person1 @person2 command. (And everyone, or only one who is included in command will receive points for help)
admin closes it (since people can just leave without finishing process)

If persons closes it, joie will ask if he is sure (since everything will be removed) and ask to mark message as ✅.
And it may be good idea to search only people who agreed to help, maybe they added tech stack just to show what they know and don't want to be included in every help room.

yeah..... that too is a good idea

if anyone wants to do it, dm me on discord - @lolvsc or @Sebastian


move jenkins file and docker file in build folder

Description
Please describe the changes in your pull request in few words here.
Changes
List the changes done to fix a bug or introducing a new feature.
How to test
Describe the steps required to test the changes proposed in the pull request.
Please consider using the closing keyword if the pull request is proposed to
fix an issue already created in the repository
(https://help.github.com/articles/closing-issues-using-keywords/)

LGTM @ChakshuGautam

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
436

Relevant Lines:
1897

💛  - Coveralls


Forced combinations but instead of match everything, randomly match from one of the list

How do i do it if i only wanted to randomly match from one of the element
const forcedCombinations = {
  "Male_Base1": ["Blonde_Slickback", "Red_Buzzcut", "NONE"] /* blonde slickback and red buzzcut both belong to one folder which is the hair folder, if male_base1 is picked I want it to force combine from only one of the element*/
};

For random picking like this, you can use nested folders with weights.
An "allow list" for features like this is not supported through config.js


Implement Registration Indicator

As requested here:
https://community.mybb.com/thread-224305-post-1334866.html#pid1334866

Actually would br much better to return a variable - google, facebook, github etc, you can then use it as a css class to implement this everywhere in forun (profile, postbit etc).

That exactly is the plan.

Well, we need to discuss on this.
Few people indicated this as a privacy exposure.
Do you really need to know how he has registered? On permanent basis? When we are ensuring the security in entrance ...
If at all admins wanna know, there can be a log, prunable.
Thoughts?

Well, we need to discuss on this.
Few people indicated this as a privacy exposure.
Do you really need to know how he has registered? On permanent basis? When we are ensuring the security in entrance ...
If at all admins wanna know, there can be a log, prunable.
Thoughts?

I think this is a security (privacy) issue. So maybe we can just add some log in ACP.

I think this is a security (privacy) issue. So maybe we can just add some log in ACP.


Update README.md

Update the way to import method. Updated by common JS to EJS

Thanks for your contribution, but this PR seems duplicated (see: #284)