Intermittent 403 errors when calling the api in SharePoint Online

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

Over the past few weeks we've been seeing an intermittent issue where users are getting 403 errors when accessing the api. This is a hard one to investigate as it doesn't happen to everyone, but it's happening to enough that issues have been raised by separate users.
Our SPFX webpart is using React and has been added to a "modern" page in our SharePoint Online site. During the componentDidMount event it calls the api (using @pnp/sp) multiple times which is where the issue is occurring. Specifically, it seems that when the web part attempts to call /_api/web/getUserEffectivePermissions these users are receiving a 403 access denied, with the following response body:
<m:error xmlns:m="http://schemas.microsoft.com/ado/2007/08/dataservices/metadata">
  <m:code>-2147024891, System.UnauthorizedAccessException</m:code>
  <m:message xml:lang="en-US">
    Access denied. You do not have permission to perform this action or access this resource.
  </m:message>
</m:error>

The users have been advised to hard refresh (ctrl + F5) the page when this issue occurs but this apparently does not make any difference. I suspected that the issue was that the user's session tokens were expiring which was causing the error, but if this is the case refreshing the page should re-authenticate them if their session has expired.
One user contacted me when this issue occurred, and since they were using chrome I had them trying refreshing the page and opening the failing request in a new tab (via dev tools), and they sent me the following:

I asked them to sign out of the site and sign back in, which finally did resolve the issue.
This is is particularly confusing as the users should not be getting 403s when they are authenticated, and if their session has expired then refreshing the page should resolve this. Has there been some change rolled out recently which could account for this issue, or is there any other explanation that I'm missing?

This isn't specifically an answer to the Access Denied error, but you may be able to skip the whole code path (and be faster), by calling this.context.pageContext.web.permissions, assuming that the web in question is the current web.

I can see the same thing on a list object. I'm using @pnp/sp 2.0.0 and querying a list on a remote web supplied in baseUrl.
Are there any known workarounds considering that the list might have unique permissions and/or be on a different SPWeb to where the SPFx solution is running on?

To answer my own question, /EffectiveBasePermissions seems to work pretty well if all you're interested in is the current user's permissions.

I ran into the same issue using the getCurrentUserEffectivePermissions
function to test permissions on a list (using @pnp/sp@2.0.2).
I was also able to resolve by using effectiveBasePermissions instead but
during my testing I found that granting users the "Enumerate permissions"
permission level also resolved the issue. My understanding is that "Enumerate
permissions" should only be required when querying permissions for other users
so it's odd that it was needed to query the current user.
The underlying getUserEffectivePermissions(@user) request URL was identical
for cases where it failed with a 403 Forbidden and when it succeeded. It seems
like sometimes SharePoint just wasn't recognising the user value as the current
user.


Fock backend tests conversion to pytest (51)

Summary of commits:

reverting to previous naming scheme

change of naming scheme

put classes in test_squeeze_operation.py

converted test_displaced_squeezed_state_preparation to pytest

converting test_beamsplitter_operation.py to pytest

making sure all asserts work on vectors

removing vestigial setup method

converting test_fock_measurement.py to pytest

converting test_fock_probabilities.py to pytest

convert test_homodyne.py to pytest

convert test_loss_channel.py to pytest

converting test_modes.py to pytest

convert test_nongaussian_gates.py to pytest

converting test_rotation_operation.py to pytest

converting test_squeezed_state_preparation.py to pytest

convert test_state_preparations.py to pytest

Description of the Change: conversion of all Fock-backend-specific tests to run on pytest
Benefits: allows pytest to be used for backend tests, which simplifies how loops/iterations over different test parameters are carried out
Possible Drawbacks: we need to be careful when porting over tests to not drop any tests in the process
**Related GitHub Issues:**None

Codecov Report

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

@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   97.53%   97.53%
=======================================
  Files          24       24
  Lines        4537     4537
=======================================
  Hits         4425     4425
  Misses        112      112

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 3552ced...6c5eb56. Read the comment docs.


LCS Algorithm in Python added.

I've added one of my favorite algorithms. I hope that it will help others in understanding and learning dynamic algorithms.

Can you fix all the conflicts and send a PR?


Fix Pass Y buff not affecting octs

Looks like Pass3 got its multiplier for octs when the cap was reduced back to 100, but not PassY

sorry my PR was merged which conflicts with your feature

Look like your pr put the same fix in, though moved stuff around, so we can close this PR


Enable DNS resolution of OpenSDS services

What this PR does / why we need it:
As we know, DNS resolution is widely used to service-to-service communication. Based on this feature, there is no need to assign static IP for service and we can leverage some tools (Kubernetes, consul, istio, etc) to manage all network configuration, so this patch is designed to enable DNS resolution for osdsapiserver, osdslet and osdsdock service.
Besides, some installation docs and scripts are also created under install/kubernetes folder to provide a simple tutorial for developers or users to install OpenSDS on existing Kubernetes cluster.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Coverage remained the same at 37.948% when pulling 18c9088053bd99aa363182cbc582e2e232361a74 on leonwanghui:istio into d10f63fdc71c885912249f9d734f4cf154f5707c on opensds:development.

Coverage remained the same at 37.948% when pulling 18c9088053bd99aa363182cbc582e2e232361a74 on leonwanghui:istio into d10f63fdc71c885912249f9d734f4cf154f5707c on opensds:development.


Relax back check in test_readonly_node

It was changed in https://github.com/zenithdb/zenith/pull/877 and this
lead to unstable test behaviour

Reworked in #933 with all the needed fixes/cleanups


[Improve]: Enhance Front-end UI for Improved User Experience

Issue Description:
Background:
The current front-end of our plagiarism detection web application serves its basic purpose, but there is room for improvement in terms of user experience and visual aesthetics. This issue is created to track and implement enhancements to the front-end.
Goals:

Improve the overall user interface for a more visually appealing and user-friendly experience.
Enhance the display of plagiarism results, making it more structured and easier to interpret.
Optimize the layout and responsiveness for different screen sizes and devices.

To-Do:

[ ] Review and update the CSS for the front-end.
[ ] Modify the HTML template to display results in a structured manner.
[ ] Implement responsive design for different devices.
[ ] Enhance user input form.

Optional/Additional Ideas:

Include charts or visual aids to better represent plagiarism scores.
Provide options for customizing the appearance of the application (e.g., dark mode).
Consider user feedback and usability testing for further improvements.

have you thought about adding Vue-js frontend instead of HTML CSS

have you thought about adding Vue-js frontend instead of HTML CSS

I'm a Python Developer, i have build a simple UI using html and css, if any one can add new technology's that would be great.

do you have any idea what sort of design you was after? colour scheme anything like that?

do you have any idea what sort of design you was after? colour scheme anything like that?

No,
Show your creativity, just keep these 4 conditions in mind (1. text box for user input text, 2. multiple file input option, 3. result should be in structured manner, 4. loading animation while backend processing.)


The functionality of tasks/parse_etc_password.yml is redundant.

Feature Request or Enhancement
Remove redundant code.
Summary of Request
The loop is not necessary because each iteration does the same. In other words, the iteration variable item isn't used at all.
        set_fact:
            ubtu20cis_passwd: "{{ ubtu20cis_passwd_file_audit.stdout_lines | map('regex_replace', ld_passwd_regex, ld_passwd_yaml) | map('from_yaml') | list }}"
        with_items: "{{ ubtu20cis_passwd_file_audit.stdout_lines }}"

Describe alternatives you've considered
Alternatively use module ansible.builtin.getent
Suggested Code
        set_fact:
            ubtu20cis_passwd: "{{ ubtu20cis_passwd_file_audit.stdout_lines | map('regex_replace', ld_passwd_regex, ld_passwd_yaml) | map('from_yaml') | list }}"

hi @vbotka
Thank you again for raising this issue. This has now been merged into the devel branch. Aiming to put this into main if all is well in the next couple of weeks.
many thanks again
uk-bolly

This was added to PR https://github.com/ansible-lockdown/UBUNTU20-CIS/pull/74 that addressed a lot of fixes.  This has been merged to main.  Thanks @vbotka for the find.


Update a pile of misc deps & let dependabot bump more

Fix  src/oc_bifrost/oc-bifrost-pedant/Gemfile  to include the branch. This has never worked since we moved to main
Use any chef 16 in chef-server-ctl and let dependabot do its things here

Signed-off-by: Tim Smith<EMAIL_ADDRESS>

https://buildkite.com/chef/chef-chef-server-main-omnibus-adhoc/builds/3812


Make the error message when Istio is checked out into the wrong place less confusing

The old message says:
Istio not found in GOPATH/src/istio.io. Make sure to clone Istio on that path. /Users/ssuchter/go/src/istio.io not under /Users/ssuchter/go
I had checked out into /Users/ssuchter/go/src/istio.io, not /Users/ssuchter/go/src/istio.io/istio
I found this confusing in multiple ways:

Because the first letter was capitalized (I vs i), I thought it was meaning a proper noun, not the directory name (which should have a lower case i)
In the next sentence, it says 'on that path' which wasn't obvious that it meant to have a subdirectory called 'istio'
In the last sentence, is says that the first directory isn't under the second directory, and you can see it obviously is.

I reworded the error message to be like this:
Not building in expected path 'GOPATH/src/istio.io/istio'. Make sure to clone Istio into that path. Istio root=/Users/ssuchter/go/src/istio.io, GO_TOP=/Users/ssuchter/go

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

Needs approval from an approver in each of these files:

OWNERS

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

Hi @ssuchter. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

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

Needs approval from an approver in each of these files:

OWNERS [costinm]

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


Fix check_cpickle Makefile target

The old target wasn't recursing properly.

Test failure tracked here #1561. Jenkins, test this please

Bad luck on this PR.. Test failure tracked here #1564. Jenkins, test this please


Unable to set SVG class name

<svg class="hello"> ends up as <svg classname="hello"> in the dom, and the classList doesn't contain "hello".

duplicage


Add multipolygon support to geoblaze

Geoblaze currently only supports overlay of a single polygon.  But SeaSketch Next intends to support multipolygons, created through for example user drawing a polygon and being clipped against one or more datasources.
Goal - geoprocessing functions are able to take a multipolygon sketch and calculate raster stats with Geoblaze.
Requirements:

[ ] Allow geojson multipolygon as input to geoblaze functions
[ ] Should not double count in raster stats where two or more of the polygons in the multipolygon are overlapping.

Geoblaze Tasks:

[ ] Confirm/implement multipolygon support
[ ] Add unit test(s)

Follow-on gp tasks:

[ ] Add multipolygon support to raster toolbox functions
[ ] Add unit test(s)

This a great catch.  That's a mistake in the way geoblaze is using dufour-peyton-intersection.

Great.  I went ahead and added the gp lib integration as follow-on tasks.  As I've hinted, the work to be done in the gp library could be a natural place for me to also contribute and we can collaborate on both sides in discussion and code review, making sure the upstream geoblaze changes meet the needs.


Cloud Init User Data on Server Creation

On the Hetzner Cloud Console it is now possible to add Cloud Init User Data, so why it is actually not possible on the app?

Since this could be a lot to type for every server, it would be better if hetzner allows to create snippets that are selectable at server creation


[Lake] DuckDB - PersistentDataStore

Motivation
All ETL compute will be handled on top of persistent data files duck.db.
A lot of the logic to operate here, will be similar to PartitionedDataStore.
PersistentDataStore deliver the requirements we need for [Process 1 + Process 2] of the ETL Pipeline. We'll then create a similar middleware to abstract common logic, such that we can update our ETL pipelines to use this.

DoD:

[x] Implement a base class for DuckDB
[x] Implement a child class called as "PersistentDataStore" for wrapping duckdb persistent databases
[x] Implement core methods: create_view, insert_to_table, query_data
[x] Create test coverage

All middleware commands have been implemented and tested

[x] _create_and_fill_table
[x] insert_to_table(self, df: pl.DataFrame, dataset_identifier: str)
[x] query_data
[x] fill_from_the_csv_destination
[x] update_data
[x] drop_table

Code merged into #734 DuckDB E2E Integration


Downgrade CoreFX packages to NET46 compatible versions

Hi @mjsabby, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

cc @NTaylorMullen

We decided to not do this in an offline conversation.


JavaScript native click event failure in mounted

Version
2.2.4
Reproduction link
https://jsfiddle.net/7aj0bke1/2/
Steps to reproduce
1.click the gray head area popup menu
2.click test text
What is expected?
menu disappear
What is actually happening?
menu not disappear

The problem is on the IOS system
I tested the phone is iphone5s
System version is 8.1 (12B411)
Browser is QQ browser (<IP_ADDRESS>56)

This is not caused by Vue.js

BTW Try this - https://github.com/simplesmiler/vue-clickaway

Closing because this is not a Vue-related Bug.

Thanks for your reply ! @znck


Create While-Sql function

Should be the final laziest way to walk a table

While (Read-Sql 'Select 1 as t'){
Write-Host $t
}
That's the goal, and t is int.

Doon!
$reader = WhileReadSql "SELECT tablename FROM pg_catalog.pg_tables WHERE schemaname = 'simplified' ORDER BY tablename"

While ($reader.Read()) {
    Invoke-Sql "VACUUM (VERBOSE) $tablename"
}

I can feel your hatred :).  But it's the pattern I use over and over. If is select it, instantiate it!


No module named "Google"

When I run "python3 answer_bot.py"
I get this error:

Are you sure you installed all modules?
Particularly Google-Search-API

@sushant10 Every time I try installing Google-Search-API or requirments.txt, I get this error:


ConcreteTypeSelector method on Scroll

Fixes #1682.
Corrects #1684 PR which was against wrong branch.

Thanks @bytenik

Any chance of getting this merged? I'd like to be able to pick up this NuGet from your MyGet even if you're not ready to release to NuGet yet.

Hey @bytenik, I'm tidying up the branches to rename (branch move and delete)

2.0 to master
develop (current 1.x client) to 1.x

and as a consequence, closed this PR as a result of deleting the remote develop branch, sorry! I've reinstated the branch to keep track of the open PRs but feel free to close this PR and open a new one with the change against  1.x :smile:

See #1689.

Pretty pretty please merge in #1689 soon. :smile:


When client becomes the "leader" no operations execute against the database

I'm not sure where to start debugging this except by creating a long running process that does this:

connect to database

get item

write item to console

sleep

goto 2

When running the process along, it will read one item then quit after it sleeps (when it attempts to read an item again)
If I add a second setTimeout and just loops and does nothing to keep the process alone, it's the only one that does anything (after the first read).
When I start 2 processes with the same script, the second one to start will read the item in the loop as expected. If I kill the first process (the "leader"), the second process becomes the "leader" and stops reading from the database.
I'm not sure if this is because the "leader" is a client and server and there's an error that's not propagating or if there's some other cause to this.
Update
I did some more tests and it seems to only happen when using accountdown and using the .list() method (which returns a stream).
Here's the script that I was using. I have the timeouts set high in this case so I can switch to another console and start it again. After the "leader" dies, and another "leader" is elected, it stops reading, then dies.
var mkdirp = require('mkdirp');
var level = require('level-party');
var sublevel = require('level-sublevel');
var accountdown = require('accountdown');

mkdirp.sync('./tmp/data');
var db = level('./tmp/data')
db.on('leader', console.log.bind(console, "I'm the leader"));
// db = sublevel(db);
var accounts = accountdown(db, {
  login: { basic: require('accountdown-basic') }
});

var start = new Date();

function writeMsg () {
  var opts = {
    login: {basic: {username: 'doowb', password: 'password'}},
    value: {name: 'Brian Woodward'}
  };
  accounts.create('doowb', opts, function (err) {
    setTimeout(writeMsg, 10000);
  });
  // var diff = (new Date()) - start;
  // db.put('time', diff.toString(), function (err) {
  //   if (err) console.error('Error writing diff', err);
  //   setTimeout(writeMsg, 1000);
  // });
}

function readMsg () {
  accounts.list()
    .on('data', console.log)
    .on('end', function () {
      setTimeout(readMsg, 2000);
    });
  // db.get('time', function (err, diff) {
  //   if (err && err.notFound) console.error('Message not found', err);
  //   if (err && !err.notFound) console.error('Error', err);
  //   console.log('Diff:', diff);
  //   setTimeout(readMsg, 200);
  // });
}

writeMsg();
setTimeout(readMsg, 5000);

I did some more tests and it seems to only happen when using accountdown

Merging into #12.


Introduce NFT support (XLS-20)

High Level Overview of Change
Support for XLS-20 NFTs on the XRP Ledger.
Context of Change
If integrated, this branch provides native NFT support on the XRP Ledger.
Please see https://github.com/XRPLF/XRPL-Standards/discussions/46 for the spec and further details.
Type of Change

[x] New feature on amendment.  So enabling the amendment will amendment block older versions of the software.
[x] Tests for NFTs are added.

Renaming global types (in this case the serialized field types) inside an amendment commit is inadvisable. I realise this amendment introduces new types but renaming existing types inside it breaks any number of other backlogged PRs (including probably mine) which all need to be individually updated to be merge-compatible again.
Instead change the code to continue with the existing naming convention and do the rename in a separate non-amendment PR after backlog has been cleared. That way we can update type names once, "one and done" so to speak, without needlessly multiplying work.
I'm speaking about SF_HASH256-> SF_UINT256 and so on. I'm not saying this rename shouldn't be done, but I am saying it shouldn't be done here.
I think as a rule amendment PR's should not change anything that isn't explicitly and exactly needed for that PR to function. There should be a clear delineation between writing new functionality into the codebase and refactoring existing functionality for the reasons above (multiplication of work)

Renaming global types (in this case the serialized field types) inside an amendment commit is inadvisable. I realise this amendment introduces new types but renaming existing types inside it breaks any number of other backlogged PRs (including probably mine) which all need to be individually updated to be merge-compatible again.

I agree with you, and in a perfect world this wouldn't have happened. My original set of commits had a separate commit for this, but somewhere along the way I squashed it down into a single commit.

Instead change the code to continue with the existing naming convention and do the rename in a separate non-amendment PR after backlog has been cleared. That way we can update type names once, "one and done" so to speak, without needlessly multiplying work.
I'm speaking about SF_HASH256-> SF_UINT256 and so on. I'm not saying this rename shouldn't be done, but I am saying it shouldn't be done here.

I could unsquash it but I don't know if it's worth the trouble to do this.  Rationale: the renaming here doesn't alter the binary protocol; it's a purely mechanical change and any conflicts with other PRs are likely to be very simple to resolve.

I think as a rule amendment PR's should not change anything that isn't explicitly and exactly needed for that amendment to function. There should be a clear delineation between writing new functionality into the codebase and refactoring existing functionality for the reasons above (multiplication of work)

I think that going forward, it's a good rule for contributors and maintainers to adopt: changes to global types should be a standalone commit. I also agree with your rationale about some mechanism to avoid conflict between different PRs that introduce new fields and/or types.
While I don't think we need to do this for this commit, if you feel strongly enough that this should be done here, I'll put in the work to do it. Let me know.

@seelabs, thank you very much for the thorough review.  I've just added a commit that I believe addresses all of your comments that did not require renames.
Today or tomorrow I'll submit a comment here that can be a starting place for the rename discussion.  Renames will be complicated by the fact that we should probably not change API-facing names that are in the spec.  That means a number of the names are cast in stone.  Non-API facing names should harmonize with the names we can't change.  More thoughts to follow.
I'm assuming I should hold off on rebasing until the review is complete.

I have been looking for a way to address the naming concerns raised by @seelabs.  I agree with @seelabs that we have some serious naming inconsistencies and opportunities for developer confusion.  I've pushed a couple of commits that address non-SFields, but still have some minor impact on the API.  Those commits are:

https://github.com/ripple/rippled/pull/4101/commits/043afbdd3b22f4c35c87e3566283960e3e35b26b ([FOLD] Improve names of and remove unused TER codes) and
https://github.com/ripple/rippled/pull/4101/commits/1a932bf835574686ca7f058640d498c509627899 ([FOLD] Rename jss::tokenid to jss::nft_id for RPC field consistency)

If someone has concerns with those two commits they should be easy to remove.
That leaves changes to SFields to consider.   This is the last chance we have to change these names.  I think changing these names is the right call.  But changing the names now will require editing the spec.  It will also cause some heartache for those diligent developers who have been coding against the NFT testnet.  We have to weight future self-consistency against current-day pain.
Since I'm less confident that there is appetite for changing the SField names I've pushed those commits in a different branch.  If we choose to pick up the changes they will be very easy to cherry-pick.  The commits that change SField names are these:

https://github.com/scottschurr/rippled/commit/5c6e8a985b0cc47f0c98cb3e7140690a516711ec ([FOLD] Rename SFields to NFTokenTaxon, MintedNFTokens, and BurnedNFTokens)
https://github.com/scottschurr/rippled/commit/8f9167fa3b7d01824e3b5de54dd1c7b2ad6c720d ([FOLD] Rename SFields to NFTokenOffer and NFTokenOffers)
https://github.com/scottschurr/rippled/commit/7b5ecf3de9e10bba9431c92c11e84855d0bf1928 ([FOLD] Rename SFields to NFTokenMinter and NFTokenBrokerFee)
https://github.com/scottschurr/rippled/commit/af7c54e80fe5fe5bd698f13297cb05a31aa929f4 ([FOLD] Rename SFields to NFToken and NFTokens) and
https://github.com/scottschurr/rippled/commit/8a0fce1b41b1d34be52f99c93f03eae85adb88be ([FOLD] Rename SFields to NFTokenOfferNode, NFTokenBuyOffer, and NFTokenSellOffer)

I'd appreciate reviewers weighing in on whether or not we should change the SField names now.  If not now, then we should assume it will be never.
Thanks.

All name changes are incorporated.  Rebased to 1.9.0-b2 and squashed.


Update to latest acorn

Currently, eslint pulls in 2 versions of acorn because of this dep.
$ npm ls acorn
...
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    └─┬<EMAIL_ADDRESS>      └──<EMAIL_ADDRESS>

Alternatively, remove auto-inject so acorn is not required at all.


Please wait for API deploy

Hello i have an issue when i download SLD :

MY PODS are in PENDING :

MY NODES ARE IN NOT READY :

@Recovzz
I can't reproduce the problem
kind version: kind v0.10.0 go1.15.7 linux/amd64

SLD latest version

./kplay.sh init

Please could you check your connectivity between kubernetes and dockerhub, a simple test can be by deploying a simple pod
Start a single instance of nginx.
kubectl run nginx --image=nginx
Check that your pod is running

The issue is closed after more than 30 days without updates


get_completion_signatures doesn't actually perform tag_invoke

In <IP_ADDRESS> paragraph 4.1, P2300 says:

tag_invoke_result_t<get_completion_signatures_t, S, E>{} if that expression is well-formed,

This doesn't actually do the tag_invoke, it just constructs an instance of the result type. This seems at odds to every other use of tag_invoke. Surely it should just do return tag_invoke(get_completion_signatures_t{},std::forward<S>(s),std::forward<E>(e));?

This is actually intentional. Calling the function would force it to be defined. There's no need to define customizations of get_completion_signatures, only its return type matters. You'll notice in the reference implementation, these tag_invoke overloads are merely declared, not defined.

Is tag_invoke really the right mechanism then? The types are purely empty holder types anyway.
Why not say that the function call operator for get_completion_signatures is just declared, not defined, rather than specifying how it constructs the return value?


3-Methyleneheptane (CHEBI:88858)

Please annotate 3-Methyleneheptane (CHEBI:88858)
Synonyms: 2-ethylhex-1-ene, 2-ethyl-1-hexene, 2-ethylhexene
Literature ref:
PMID: 30428374
Links:
https://webbook.nist.gov/cgi/cbook.cgi?ID=1632-16-2
https://chem.nlm.nih.gov/chemidplus/rn/startswith/1632-16-2

Now checked.


Make client self register look for already onboarded to allow for side channel register

Fix reboot reason for first boot after install
Add watchdog fault injection options to logmanager and ledmanager for testing
Record reboot reason in device-steps.log for easier correlation
Signed-off-by: eriknordmark<EMAIL_ADDRESS>

LGTM


TypeError: init() got an unexpected keyword argument 'dtype'

Hello, thanks for your great work.
When I run the codes, I encounter the following errors.
Can you provide any helpfully suggestions? Thanks a lot  in advance.
[01/26 23:47:40 d2.engine.train_loop]: Starting training from iteration 0
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/shapely/set_operations.py:133: RuntimeWarning: invalid value encountered in intersection
  return lib.intersection(a, b, **kwargs)
ERROR [01/26 23:47:45 d2.engine.train_loop]: Exception during training:
Traceback (most recent call last):
  File "/16T-2/zitong/code/detectron2/detectron2/engine/train_loop.py", line 149, in train
    self.run_step()
  File "/16T-2/zitong/code/detectron2/detectron2/engine/defaults.py", line 494, in run_step
    self._trainer.run_step()
  File "/16T-2/zitong/code/detectron2/detectron2/engine/train_loop.py", line 421, in run_step
    with autocast(dtype=self.precision):
TypeError: __init__() got an unexpected keyword argument 'dtype'
[01/26 23:47:45 d2.engine.hooks]: Total training time: 0:00:05 (0:00:00 on hooks)
[01/26 23:47:45 d2.utils.events]:  iter: 0    lr: N/A  max_mem: 411M
Traceback (most recent call last):
  File "train_net.py", line 377, in <module>
    launch(
  File "/16T-2/zitong/code/detectron2/detectron2/engine/launch.py", line 69, in launch
    mp.start_processes(
  File "/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 188, in start_processes
    while not context.join():
  File "/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 150, in join
    raise ProcessRaisedException(msg, error_index, failed_process.pid)
torch.multiprocessing.spawn.ProcessRaisedException:

-- Process 4 terminated with the following error:
Traceback (most recent call last):
  File "/home/zoloz/.conda/envs/maskdino/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 59, in _wrap
    fn(i, *args)
  File "/16T-2/zitong/code/detectron2/detectron2/engine/launch.py", line 123, in _distributed_worker
    main_func(*args)
  File "/16T-2/zitong/code/MaskDINO/train_net.py", line 364, in main
    return trainer.train()
  File "/16T-2/zitong/code/detectron2/detectron2/engine/defaults.py", line 484, in train
    super().train(self.start_iter, self.max_iter)
  File "/16T-2/zitong/code/detectron2/detectron2/engine/train_loop.py", line 149, in train
    self.run_step()
  File "/16T-2/zitong/code/detectron2/detectron2/engine/defaults.py", line 494, in run_step
    self._trainer.run_step()
  File "/16T-2/zitong/code/detectron2/detectron2/engine/train_loop.py", line 421, in run_step
    with autocast(dtype=self.precision):
TypeError: __init__() got an unexpected keyword argument 'dtype'

Is this error about the version of detectron2 ?

Hey there
We are also facing the same issue
Please let us know how you resolve it.
Waiting for your reply
Thank you

how to solve it?

how to solve it?

cause the annotation data type problem

@TyrionChou @AINeurosurgery
This problem is due to the compatibility of detectron2 version.
Please ckeck this installation with the matching command (matching CUDA and torch version).

@LiWentomng @TyrionChou @AINeurosurgery
I think I've found the reason.
The params of class autocast of torch_1.10 looks like this: "torch.autocast(device_type, enabled=True, **kwargs)", it do not acccept a param named "dtype". You can check it from here
So, the solution:
1、 change default parameters of class AMPTrainer(SimpleTrainer): The value of precision need to be True;
2、change the usage method， the right format is : "with autocast(self.precision):"


Removes check for Homebrew if all necessary packages are present.

Fixes #16458 and #16461. The edited test now tests the the new situation – where all the packages required are present, but homebrew is not installed. Originally created by @championswimmer.

Thanks @jslavitz (I never was able to complete the PR)


Creating account deployment is not supported by the model 'text-davinci-003'

ERROR: deployment failed: failing invoking action 'provision', error deploying infrastructure: deploying to subscription:
Deployment Error Details:
InvalidTemplateDeployment: The template deployment 'openai' is not valid according to the validation procedure. The tracking id is '198cc047-82d3-482a-9e67-4083cebb5cbd'. See inner errors for details.
DeploymentModelNotSupported: Creating account deployment is not supported by the model 'text-davinci-003'. This is usually because there are better models available for similar functionality.

I got the above error while trying to redeploy my application. I have not changed anything since the last deployment; I only add new documents to the data and want to redeploy.
I try changing to GPT-4, but I did not get the error; got an error relating to the versioning.
Is anyone else seeing the error? What is the workaround for this?

This is a new issue, starting today, likely related to the OpenAI deprecation announcements. I'm testing to see if gpt3-turbo can be used instead.

If all you changed was the code, then you can just run azd deploy to deploy just the code.
You don't actually need to re-deploy after adding docs to the data, since the index is updated instantly, independent of the app code.

Another workaround for folks who already have the davinci model deployed:

Go into infra/main.bicep
Delete the first deployment in the deployments property:

      {
        name: gptDeploymentName
        model: {
          format: 'OpenAI'
          name: gptModelName
          version: '1'
        }
        sku: {
          name: 'Standard'
          capacity: gptDeploymentCapacity
        }
      }

Another workaround for folks who already have the davinci model deployed:

Go into infra/main.bicep
Delete the first deployment in the deployments property:

      {
        name: gptDeploymentName
        model: {
          format: 'OpenAI'
          name: gptModelName
          version: '1'
        }
        sku: {
          name: 'Standard'
          capacity: gptDeploymentCapacity
        }
      }

This worked for me

Has anyone succesfully deployed using another model? When I exchange text-davinci-003 for gpt-35-turbo in the main.bicep file, I get an error:
"DeploymentModelNotSupported: The model 'Format: OpenAI, Name: gpt-35-turbo, Version: 1, Source: ' of account deployment is not supported."
Unfortunately, I don't have a davinci deployment in place, so the workaround didn't work for me.
Thank you!

Specified version for gpt-35-turbo is not correct. Either 0301 or 0613 should be specified to version of gpt-35-turbo.

If all you changed was the code, you can just run azd deploy to deploy just the code.
You don't actually need to redeploy after adding docs to the data since the index is updated instantly, independent of the app code.

Thank you @pamelafox  but I deleted the openAI (model) deployments too. That was the quicker fix I use to deploy since the RPM issue started. Deploy will not longer work because, I dont have any active model deployment

@damilareisaac Ah, if you no longer have a davinci deployment, then you'll need to make the change from my PR, from davinci to gpt-turbo. Unfortunately, it doesn't work as well for the "chat" tab, but "ask" approaches still work well. We need to do some prompt tweaking to get gpt-turbo working well for the chat approach.

after changing the bicep file ("gpt-35-turbo" and version "0301") I get this error.  thanks for the help.
ERROR: deployment failed: failing invoking action 'provision', error deploying infrastructure: deploying to subscription:
Deployment Error Details:
CannotChangeDeploymentModel: The model of deployment cannot be changed.
TraceID: 537ab5588561fe79335f0e3b54fdf787

I am facing same issue as well - CannotChangeDeploymentModel: The model of deployment cannot be changed.

Thanks for the updates, I could deploy after making these changes.
However it would be better to read the capacity from environment rather than hard coding. They are not consistent across main bicep and cognitiveservices.bicep.
I will try to do a pull request with this change later today.

GPT-4

I wonder if you could successfully deploy the GPT-4 model. And if yes, what is the model name and version required for deployment?

@phosseini There is now an FAQ about using gpt4, that is permitted. This issue can be closed as I believe we've documented how to continue using davinci if you already have it, and the repo is now upgraded to work with gpt-35 instead of davinci.

@pamelafox just gonna mention an FAQ without linking to it?

Our FAQs actually weren't linkable in the past, unfortunately. I have since refactored the documentation so that everything is linkable. Here's the section about using GPT-4:
https://github.com/Azure-Samples/azure-search-openai-demo/blob/main/docs/deploy_features.md#using-gpt-4


edge: remove typ endOfCandidates hack

this removes the endOfCandidates hack. The samples will be updated
to simply pass every event.candidate (including undefined) to the
other addIceCandidate which removes the need for this.
Needs a major version bump after merging. Lets do it together with the bump for #389

LGTM


[5.4] prevent policies from being too greedy

Laravel Version: 5.4.*
PHP Version: N/A
Database Driver & Version: N/A

Description:
Policies are too greedy, and will not defer to the gates if they do not have a method that matches the requested ability.
Steps To Reproduce:
Let's assume a App\Blog, and App\Policies\Blog.
namespace App\Policies;

class BlogPolicy
{
    /**
     * @return bool
     */
    public function update
    {
        return false;
    }
}

and the following AuthServiceProvider:
namespace App\Providers;

use Illuminate\Contracts\Auth\Access\Gate;
use Illuminate\Foundation\Support\Providers\AuthServiceProvider as ServiceProvider;

class AuthServiceProvider extends ServiceProvider
{
    /**
     * The policy mappings for the application.
     *
     * @var array
     */
    protected $policies = [
        \App\Blog::class => \App\Policies\BlogPolicy::class,
    ];

    /**
     * Register any authentication / authorization services.
     *
     * @param \Illuminate\Contracts\Auth\Access\Gate $gate
     */
    public function boot(Gate $gate)
    {
        //register policies
        $this->registerPolicies();

        //define additional ability
        $gate->define('update-blog', function ($user, $blog) {
            return true;
        });
    }
}

Currently if you tried to use this ability, you would get an AuthorizationException, because the method didn't exist and the Gate would return false.
namespace App\Http\Controllers;

use App\Blog;

class BlogController extends Controller
{
    /**
     * @return
     */
    public function update(Blog $blog)
    {
        $this->authorize('update-blog', $blog);
    }

This is because the Gate would see an App\Blog passed as the first argument, and try to resolve the callback from App\Policies\BlogPolicy.
Solution
This PR makes sure the ability/method is callable on the policy, otherwise falls through to let the gate check for a matching ability, and then finally defaults to return false.
All tests are passing, including an additional one for this situation.

This is a breaking change and one that has the potential to introduce security vulnerabilities in existing applications so would have to go to master if it was to be accepted.


bump version to 2.2.6

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #125   +/-   ##
=======================================
  Coverage   96.13%   96.13%
=======================================
  Files          18       18
  Lines         725      725
  Branches       57       57
=======================================
  Hits          697      697
  Misses         18       18
  Partials       10       10

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 1887800...e3ee022. Read the comment docs.


Fix precision error in NestedScrollView

Fixes https://github.com/flutter/flutter/issues/63825
https://github.com/flutter/flutter/blob/81fac6809ee85e073098aa86aa52ce4aa303fe6f/packages/flutter/lib/src/widgets/nested_scroll_view.dart#L1719-L1720
The root of the error was here, because metrics.correctionOffset sometimes came a little bit bigger than it needed to be, because of the precision error caused by calculations here
https://github.com/flutter/flutter/blob/81fac6809ee85e073098aa86aa52ce4aa303fe6f/packages/flutter/lib/src/widgets/nested_scroll_view.dart#L979-L984
So because of that the applyMoveTo returned false, as it was overflowing by a small number
Also a small refactorings in lsq_solver.dart and sliver_fixed_extent_list.dart:

changed <= to < in few places where precisionErrorTolerance was used, because that's the way it is in the rest of the codebase
removed the function used only in one place

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Can you add more info to your PR description that explains why it fixes the linked bug?

Done

Those are refactorings without any semantic change, I believe they don't need tests. Do I need to submit them as a different PR and get a test-exemption from Hixie?

Those are refactorings without any semantic change, I believe they don't need tests. Do I need to submit them as a different PR and get a test-exemption from Hixie?

Changing <= to < is semantic? 🙃

In case with such a small number yes, I think so. I'm removing these changes from the current PR


ci: add pyupgarde pre-commit hook

Add the pyupgrade pre-commit hook. This should help keep the code-base up to date with the latest python syntax and standards.

Ready for merge


Generate docfx.json configuration from repo

Given the JS docs repo state, generate docfx.json entries. This should run in js - docindex

Part of grouped ToC work. Not necessary anymore.


Excluding failing tests from campaign

The following tests are now excluded from the test campaign because they often fail (those tests should be rewritten in a more reliable way):
on Firefox 3:
test.aria.utils.HistoryTestCase on Firefox 3
on IE 11 with JAWS:
test.aria.widgets.wai.errorlist.binding.ErrorListBindingJawsTestCase

@fbasso Thank you for your review!


United States links

There are some dead links in the US Page.
More importantly, is there any way for me to edit this myself? Why isn't this hosted on an open platform where I can PR changes like this?

FIXED

@morchickit Still curious about the questions I asked.

You can't edit the page yourself, it has a Django CMS system. :-)

Ok, thanks.


[WIP] Convert tags-input (selectize => ember-power-select)

refs TryGhost/Ghost#6458
TODO:

[x] initial conversion
[ ] css fixes
[ ] tests
[ ] remove jquery-ui & selectize :tada:

Currently blocked on https://github.com/jgwhite/ember-sortable/issues/37

@acburdine We also ran into this issue today at Trouva.  Are you interested tackling this together?

@mschinis Sure! How do you want to go about doing that?
One of the things with Ghost's implementation though is it can't be scrolling, it needs to actually handle multiple lines, which introduces some complexity. I've been playing around with ember-sortable, trying to modify it to where it will do both the x and y direction.

I completely agree, ember-sortable needs to be able to handle both x and y directions and not be scrollable.
I will start playing around with ember-sortable myself over the next couple of days, figure out how it works and we can then chat to find a solution. I will email you through your public email on github and we can get working on it!

@mschinis sorry it's been so long - are you still interested in working on this? As a sidenote, I've recently started to help maintain ember-sortable, so once it gets figured out I can release a new version.
I've had some ideas as to how to implement this as well, and will hopefully get some work done this week on implementing it.

@acburdine Yeap - still interested. Things have been super busy last couple of months but will have more time now that Christmas is over.
How do you wanna do this?

@mschinis I'll send you an email in a bit :)

Pretty sure this PR isn't needed anymore since we aren't using selectize 😄


Check a book package

User wants to start a book package. The following will be done for the book selected:

Check GLT existence
Check GST existence
Check TN existence
Check TWL existence
Check TQ existence
Check TW existence
Check TA existence
Check SN existence
Check SQ existence

For the below, the context will be the organization and language selected in account settings. For illustration, the examples will use the organization "ru_gl" and the language id "ru". Also, using Titus for the book.
For each check, the following procedure will be done.

Does repo exist? For example, "ru_gl/ru_glt". If not, then this is indicated in the UI.
Does the manifest exist? For example, "ru_gl/ru_glt/manifest.yaml". If not, then this is indicated in the UI.
Does book info exist in manifest? If not, then indicate this in the UI.
Does book exist? For example, "ru_gl/ru_glt/57-TIT.usfm". If not, then indicate this in the UI.

Here is the UI (from here

Here is an alternative representation taken from the CV App:

Design Notes
Create a React Context and custom hooks, which does the following:

Since repo validation is common to any book/OBS, then it should only be performed one time and the results updated as needed (ie, when org or language is changed).
The results of the validation will be stored in the Context and be available to the UI to be presented to the user.
There are two possible variations on how to do the hook(s).

The first variation is to have one hook with org, language, and resource type as dependencies.
The second variation is to have one hook per resource type, taking only org and language as dependencies.

For the moment, let's call the context RepoValidationContext.
The values for org and language will be managed as state variables (unless they themselves are in their own Context already -TBD). As they are changed by the  user, the hook will execute doing the "light" repo validation and storing the results in the context.
Once the results are updated/changed, then the UI will update.

Thinking thru how things will work and I found a possible problem. Consider this scenario:

user logs in
user sets org and language and clicks saves / continues

At this point, what happens? Based on our chats, I think you want all current book packages to be shown. So continuing:

because the repo validation is book/obs independent, it will run in the background, and store the results in memory. But nothing is shown to the user at this point.
a loop is done over all 66 books and the obs.
if any item exists in any repo, then show a card for it. The card will combine the repo validation done earlier with the book/obs info.
once loop is done, all existing book content is shown, one book per card.

This looks like a pretty intense process and may take some time.


Make NullSurface and NullVolume subclasses of NullMaterial

NullSurface and NullVolume should share NullMaterial as a common base class. This will prevent later code duplication.

After a review of the code, have decided this change is unnecessary.


doesn't respect .gitignore

Type: Bug
make a .env file - put something in it.
make a .gitignore file - tell it to ignore .env
ask copilot for the contents of the .env file
it will give them to you.
Extension version: 0.20.0
VS Code version: Code 1.93.1 (38c31bc77e0dd6ae88a4e9cc93428cc27a56ba40, 2024-09-11T17:20:05.685Z)
OS version: Windows_NT x64 10.0.19045
Modes:

I asked github copilot "Does copilot respect .gitignore"
Copilot:
"Yes, GitHub Copilot respects the [.gitignore] file. It will not suggest or use content from files that are listed in [.gitignore]. This helps ensure that sensitive information and unnecessary files are not included in suggestions."
me: "so if i put secets in my .env file will you read them?"
Copilot:
"No, I will not read or use the contents of your [.env] file if it is listed in your [.gitignore] file. This ensures that sensitive information remains private and secure."

By nature of being an LLM Copilot can hallucinate. We have a feature already that allows excluding content from Copilot suggestions. The .gitignore file is not the right place for that as there are files on dis such as your dependencies or build artifacts which are not secret but you don't want committed. These can still be beneficial to Copilot suggestions.


[CI] ConfigSpace binary compatibility fix

Description of changes:
Updating ConfigSpace to address binary compatibility issues with numpy. See ConfigSpace-173
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-938-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-938/1/index.html


BUG | Content Scripts UI not working - requires all exports present in production build

What happened?
I am in the process of investigating porting our Chrome extension to Plasmo (https://attio.com/).
There seems to be an issue with the content UI scripts; they seem to require getStyle, getMountPoint and getRootContainer to always all be exported. The docs and examples seem to suggest otherwise.
I tried cloning the examples and noticed with-content-scripts-ui doesn't actually build:

I tried downgrading plasmo until I hit a working build. 0.36.1 seems to work - all later releases fail. (did remove the css import, however)
Presumably this is a regression? If so, it could be handy to have the CI for the examples attempt to build each project?
Version
Latest
What OS are you seeing the problem on?
MacOSX
What browsers are you seeing the problem on?
Chrome
Relevant log output
No response
(OPTIONAL) Contact Details
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

Small update: I've been using export getStyle = undefined as a quick workaround.

Hey @Braden1996, thanks for the report!
I think this has to do with how we're bundling production builds.
In our Parcel build config, we specify the type as a production build which strips away any unused exports due to tree shaking. This seems to have unintended consequences.
https://github.com/PlasmoHQ/plasmo/blob/main/cli/plasmo/src/commands/build.ts#L53
I suggest either using npm run dev for the time being, or doing what you just mentioned as a temp mitigation.
When @louisgv wakes up in a few hours he can take a closer look at a proper way to fix this.
Re: CI - agreed. It's on our sprint this week so hopefully will find these issues quicker once we have it.

This should be fixed in a9b72d145f04cb6edf3f79c2fde14913ee0005bc - @Braden1996 when you got a moment, can you please upgrade plasmo and test it out :D

False-positive - after testing my fix in build -> the build passed, but the extension itself now crash at runtime trying to look for dependency

https://github.com/parcel-bundler/parcel/issues/8071#issuecomment-1133549719 <- this was the solution. We made our own parcel bundler and use that for our setting.


vcredist: Add 2019, update 2017

Please add vcredist2019.json and update the current vcredist.json with
{
    "homepage": "https://www.visualstudio.com/de/downloads/",
    "version": "14.20.27508.1",
    "url": [
        "https://download.visualstudio.microsoft.com/download/pr/21614507-28c5-47e3-973f-85e7f66545a4/f3a2caa13afd59dd0e57ea374dbe8855/vc_redist.x64.exe",
        "https://download.visualstudio.microsoft.com/download/pr/092cda8f-872f-47fd-b549-54bbb8a81877/ddc5ec3f90091ca690a67d0d697f1242/vc_redist.x86.exe"
    ],
    "hash": [
        "14b07d8f4108a8b8b306af3f6fe6eb78814dfadebd500416fb24856ad668518e",
        "261f8122ce3e3da66fe4300e2a88325469251da85b62fbddb03ec49f97b62c14"
    ],
    "post_install": [
        "run \"$dir/vc_redist.x64.exe\" \"/install /passive /norestart\"",
        "run \"$dir/vc_redist.x86.exe\" \"/install /passive /norestart\""
    ],
    "notes": "You can now remove this installer with 'scoop uninstall vcredist2019'"
}

VCRedist 2019 has exit code 3010, I'll check it again later (about 2 hr)

New VCRedist 2019 is named 2015-2019 vc redist, and its number version in files is xxx140.dll, same as vcredist 2015 and vcredist 2017. So remove installing vcredist2015 and vcredist2017 in vcredist.
Add Out-Null to suppress run's output (True). If some errors occured, it will output error code only.


ARROW-638: [C++] Complex Number Support via ExtensionTypes

10565

 Quality gate: Failed
30 Issues

 2 Vulnerabilities
 28 Code issues

Details

 Quality gate: Failed
9 Issues

 2 Vulnerabilities
 7 Code issues

Details

 Quality gate: Failed
 Everything looks good
Details


 to search next word with word boundaries like * does

* command searches for the current word. To make sure it's actually a word, it adds boundaries: /\<current_word\>/. In contrast, g* does not add boundaries: /current_word. At the moment, <C-n> acts like g*. I'd like to have both behaviors available. For my use cases it feels like searching with word boundary is a better candidate for the default behavior of <C-n>.
This problem occurs when I have a function like that:
def foo(bar):
    new_bar = bar.clone()

When I put my cursor on bar argument and press <C-n>, new_bar gets partially selected too, which is undesirable.
For the consistency I'd like <C-n> to search with word boundaries (as * does), and, say, <C-N> without boundaries (as g* does).

Discussion moved to PR.


Failing unit test for reading current_page values in Mustache

In reference to issue #1612

This is really great. I'll dive into how the mustache ruby library works and see what needs to get called to forward our environement.


update.sh: use version-sort option of sort

Also, use more general regular expression in grep

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew tests with your changes locally?

I think --version-sort is a more robust approach than specifying individual fields for sorting.
Not sure why grep is used instead of head -n 1 but I kept it and generalized its pattern a bit.
Also, not sure why this is not something like:
git describe --abbrev=0 --tags origin/master

I'd rather avoid refactorings like this that may cause end user bugs when they aren't fixing end user bugs.

:smile:  https://youtu.be/nNv0IoE5XHw


[cluster-test] increase test duration for fullnode recovery

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/libra/developers.libra.org, and link to your PR here.)

Do you mind giving some context/motivation on this change?

Do you mind giving some context/motivation on this change?

https://toroeng.slack.com/archives/CNJCQD2J0/p1601917065000100
10% down test occasionally failed last few days, checked failed job which shows post test healthcheck failed, checked grafana which fullnode did not recovery in time. Try to increase the duration first to see whether test can back to stable, for next step we can extent THRESHOLD back to 20k in fullnode health check

/land


[AdaptiveCardTemplate] Issue with accessing array elements

Sample AC and data:
Sample AC:
{
    "type": "AdaptiveCard",
    "body": [
        {
            "type": "TextBlock",
            "size": "Medium",
            "weight": "Bolder",
            "text": "${title['1']}"
        }
    ],
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.5"
}

Sample data
{
 "title" : [
        "Title1",
        "Title2",
        "Title3"
    ]
}

Issue

Using the https://www.adaptivecards.io/designer/ we are able to retrieve the array element using ${title['1']}
Whereas the AdaptiveCard dot net templating library is unable to parse this string.  The templating library is able to parse ${title[1]} but not ${title['1']} which is currently supported by the js library.

@SowmyaVelugoti,
This is by design. Please use integer to index into an array.


Open table in new tab, without replace tab

Is your feature request related to a problem? Please describe.
Yes, I encounter difficulty accessing multiple tables' data as the tabs get replaced when opening a new table.
Describe the solution you'd like
I would like each table click to open in a new tab by default. Additionally, it would be beneficial to have an option to split the screen, similar to VSCode, allowing users to view two tables simultaneously side by side.
Describe alternatives you've considered
The alternative I've considered is manually opening each table in a new tab and using SQL queries to achieve the desired view of multiple tables.
Additional context

Hi @adibpwd,
With double click on tab name, or table on sidebar, you can open a permanent tab.
With single click you open temporary tabs (italic tab name).
It works like VSCODE with files.

Thank you. Now I am able to open tabs permanently. However, I am still unable to split the screen as in VSCode, for instance, to have Table A on the left side and Table B on the right side.

This feature is not currently present, and I confess it is not so quick and easy to implement. However, I plan in the future to implement it, but certainly not in the short term.
I am changing the name of this issue so as to refer to this specific feature.


Fix MDB_PAGE_FULL by bumping LMDB

This PR fixes #3062 by upgrading LMDB to the latest version.
The changes were made in https://github.com/meilisearch/lmdb/pull/1 and https://github.com/meilisearch/lmdb-rs/pull/12. As heed directly depends on the latest main commit of https://github.com/meilisearch/lmdb-rs, we can bump the lmdb-rkv-sys dependency in the Meilisearch Cargo.lock by doing a:
cargo update -p lmdb-rkv-sys

Got a somewhat strange task queue corruption when trying to reload my v0.30 DB, but this appears unrelated to the current issue.

If you can reproduce it by creating a v0.30.0 and opening it with a v0.30.2 would you mind creating an issue, please?

If you can reproduce it by creating a v0.30.0 and opening it with a v0.30.2 would you mind creating an issue, please?

Good suggestion, will try. I also immediately made a copy of the DB if we want to inspect it.

Got a somewhat strange task queue corruption when trying to reload my v0.30 DB, but this appears unrelated to the current issue.

So back on this, the issue happened while trying to read a v0.30 DB with a binary from the main branch, so this was expected. #3216 made it so the error will be detected in the future.


Can't get it working

I just installed Ionide-fsharp package and can't get it working. It doesn't work with single files - it seems instead it uses built-in syntax highlighter, same for projects - I made test project with yo fsharp and opened folder with vscode with 0 success. Am I doing something wrong?

Also forgot - FSI works for me (I have FSI commands in commands menu - start fsi, send line, send selection etc)

I'd faced similiar issue just now, copletion just said "Loading..." and nothing more. The reason caused this issue was blocking FsAutoComplete.Suave.exe by antivirus software. If You are on windows, check this, it may help.

I do not have antivirus software other than what comes with windows 10, does it also block it?

I'm facing a similar issue on Windows 10.. Ionide fsharp version 0.3.1 was working well, after updating to 1.0.2 [1], the editor switched back to built-in syntax highlighter and the FSI commands returned a error/warning "no handler for fsi.Start". Fortunately I had the files of 0.3.1 on my pc, so I replaced them and got things working back again.
[1]: I wasn't able to update directly from 0.3.1 to 1.0.2. I had to uninstall the editor. Remove extensions from 'C:\users<username>.vscode\extensions' and then try installing again.

@ad1ttya i hvae the same with windows 10.

Update problem is probably some problem on VS Code side, not our plugin.
Can you please create issue about it on vscode repository?


PyUnit

Can I write my PyUnit test with the robot framework as well?

How would you like to write them?

I'd like to write my normal pyunit class file test.py https://gist.github.com/mattben/90a9efd71dadc561dfd8
and then call it with test.txt
https://gist.github.com/mattben/cd432b1b9a821688a99a
where the function name is the keyword pretty much just having robot run all our test for us

I guess that could work. Did you test it? If it doesn't work out-of-the-box, implementing a library that handle PyUnit somehow shouldn't be too complicated.

Okay will try thank you


What is os?

On line 19 of rp.js, it states var EOL = require('os').EOL;
I am working on a react native application which has a dependency that depends on request-promise, and for some reason I get an exception at this line saying Unable to resolve module osfromnode_modules/request-promise/lib/rp.js`
Any insight would be very appreciated!
Thank you.

Hi @kirakik , I assume that you package your react app to be executed in the browser, right? request-promise is build for node.js. If you want to use it in the browser you need to polyfill all node.js dependencies. os is one of them.
I could suggest some alternative solutions but it sounds like the request-promise dependency is not in your hands. So polyfilling is the only option.


stop filtering out null values in card data.

fixes #108.

rebased. this may be obsolete now with the fix for #110 in place.


BlockchainAddress type should be an array

I can imagine something like an address could be a eoa, creator, or a factory, contract, or a proxy, contract, or a eoa, treasury, creator, or a safe, treasury

Fixed here
https://github.com/hypercerts-org/oss-directory/pull/44


SwiftOutputChannel test prints lots of errors

With latest vscode there are a lot of error messages printed to the console:
  SwiftOutputChannel
    ✔ Appends logs
Error: Trying to add a disposable to a DisposableStore that has already been disposed of. The added object will be leaked!
    ✔ Appends lines
    at H3.add (file:///d:/a/vscode-swift/vscode-swift/.vscode-test/vscode-win32-x64-archive-1.95.1/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:24:1439)
    ✔ Appends lines and rolls over
    at yw.n (file:///d:/a/vscode-swift/vscode-swift/.vscode-test/vscode-win32-x64-archive-1.95.1/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:120:1748)
    ✔ Appends and rolls over

https://github.com/swiftlang/vscode-swift/actions/runs/12149050931/job/33879420980
Verified no excessive log output unless on test failure.


[Bug]: Firefox Errors

What happened?

Version
2208.618
What PowerShell edition/s are you running Evergreen on?
Windows PowerShell
Which operating system/s are you running Evergreen on?
Windows Server 2016+
Have you reviewed the documentation?

[ ] Troubleshooting at: https://stealthpuppy.com/evergreen/troubleshoot/
[ ] Known issues at: https://stealthpuppy.com/evergreen/issues/

Verbose output
Get-EvergreenApp -name mozillafirefox

This is a known issue documented here: https://stealthpuppy.com/evergreen/issues/#errors-and-warnings


Gradle 2.14, Android Gradle Plugin 2.1.2

For the latest bug fixes / performance improvements (and to stop Android Studio from complaining about the outdated version...)

Thanks for the pull request, @bguertin! I've created OSPR-1295 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.
Feel free to add as much of the following information to the ticket:

supporting documentation
edx-code email threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.
We can't start reviewing your pull request until you've submitted a     signed contributor agreement     or indicated your institutional affiliation.     If you like, you can add yourself to the AUTHORS      file for this repo, though that isn't required.     Please see the CONTRIBUTING     file for more information.

Silly webhook... anyway please review :)

👍


EditorScenePostImportPlugin scene import API additions

Let EditorScenePostImportPlugin set a custom root node
Make EditorScenePostImport run _post_import only after all EditorScenePostImportPlugin plugins have ended processing
Make EditorScenePostImportPlugin aware of the source file
Set NavigationRegion3D name when in Navmesh only import mode

Fixes godotengine/godot-proposals#5093

Are you aware that the documentation isn't updated?

@fire Sorry, I don't understand the question. Is it implied that I shouldn't have touched the documentation?

Theres one failing check. We need all checks to be passing and, go through review before merging.

Of that I'm aware. I did add the documentation but the check didn't like the location I added the bits at. Is there something I can do locally to know where the documentation check wants the additions to be?
I'll update it when I update this PR with other likely changes.

I hope this fixes the documentation check failure.

In some cases it may need something like post_post_process().

Something like _ensure() or _verify(), perhaps?

Slightly off topic, but I think PostImportPlugin currently only allows push_back() so it needs to be fixed that it can be added to specified index to prevent conflicts with the built-in PostImportPlugin. PostImportPlugin is different from EditorPlugin in that it has a clear processing order.

Usually the project developer is aware of the order of inclusion of post import plugins, unless some third party addon adds some by itself. I propose instead that Editor Scene Post Import plugins get their own user interface in the project settings window to be added, removed, disabled, reordered.

This needs to be rebased and also updated to have the virtual function return Node* instead of Object* now that the GDVIRTUAL bug has been fixed.

I'm on it. Also addressing:

This needs to be rebased and also updated to have the virtual function return Node* instead of Object* now that the GDVIRTUAL bug has been fixed.

I have changed the script return type to avoid passing invalid pointers around in the engine core. Now NodePaths are used to get the custom root node.

Due to a change of approach and the closure of proposal godotengine/godot-proposals#5093, I will not continue to push forward this pull request. Thank you for your time.


chore: add ffi and cli binaries

#175  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @AngelOnFira and the rest of your teammates on  Graphite


Add workload funcs in token_retriever and token_exchanger.

Iteration 1 to merge into everjing/msi-acrpull repo.

Could we add the workload identity token retriever function to token_retriever.go? I'm not sure why we need a separate file for this.


Typo in example

Thanks for your work on this project. Just wanted to bring your attention to a small typo. Should be type rather than tyle.

Thanks Andi – good spot! All fixed.


Ad "Build Passing" Badge

Should we add a "build passing" badge?
I noticed https://github.com/spdx/tools-golang has this badge and others.
Having the badge lets maintainers know if the build is broken without digging around in the Actions tab. Additionally, assuming the build is passing, provides confidence to potential users.

+1 on adding the badge.


Compilation error with USE_SimilarityTransformElastix

When compiled with USE_SimilarityTransformElastix
/home/mark/git/elastix/Common/Transforms/itkAdvancedSimilarity3DTransform.hxx:103:37:   required from ‘void itk::AdvancedSimilarity3DTransform<TScalarType>::SetMatrix(const MatrixType&) [with TScalarType = double; itk::AdvancedSimilarity3DTransform<TScalarType>::MatrixType = itk::Matrix<double, 3, 3>]’
/home/mark/git/elastix/Common/Transforms/itkAdvancedSimilarity3DTransform.hxx:83:1:   required from here
/usr/include/vxl/core/vnl/vnl_math.h:175:21: error: cannot bind non-const lvalue reference of type ‘double&’ to an rvalue of type ‘double’
     return std::cbrt(std::forward<TArg>(arg));

Gah, sorry, this seems to be an issue with vxl maybe? I installed the 2018 06 18 version.

What version of ITK are you using?

We are using the version shipped with ITK 4.13.x. We are not yet moving to ITK 5, and newer vxl versions embedded in there. Feel free to reopen if the problem also occurs with version 4.13.


 The loading time can be greatly reduced by only loading the EEs that will be displayed in the table.
This is caused by the table paging happening in the front-end. Most effective place for this mechanism is at the SQL-query level.

Browse datasets will now only load the required amount of EEs for one table page. If the user is not logged in, only non-troubled experiments are loaded (that includes experiments on troubled platforms).
This increases the loading speed of the page in orders of magnitude (only 30 EEs are loaded instead of ~8000), but some pages will have less than the required amount (e.g. 30) experiments displayed (unless user is logged in as an admin), since the ACL will remove non-public experiments from the list. As this is a UI, I did not think this was too critical, and the speed improvement seemed like a good enough tradeoff for a missing experiment here and there. I will be dealing with the disappearing EEs issue in the REST API soon, so I will then apply the solution to the UI as well.

Resolved in 1.12.5 (497a1d6aa57b63ec49d22921ff26f6129e922c21)


YoutubeCenter keeps losing settings status

Hi, in recent dev versions, I am getting prompted to set my settings everytime few times I watch a video, it seems to occur more often if I middle click a video to load it in a new tab.  So it forgets I previously set the settings, in addition when this happens some settings are reset/lost.
e.g. debug to console option is enabled.
auto update is enabled.

@Gotolei That's interesting information, thanks. This might not be a YTC issue at all. I hope that @YePpHa has time to verify at some point.

omg best bug ever!
had to reload twice in a row xD
dialog->1st settings import->reload->dialog again(directly after reload)->2nd settings import->reload works again
so freaking random and annoying lol, wow

my guess about what could also be an issue:
different conflicting settings versions
->it feels like the issue happens more likely if you have several tabs open but get back to them only later (while you surfed through the internet and youtube in a different tab)
it also happens often if I open the videos through facebook (right-click->new tab)
I didn't change the settings directly but according to other informations I read here, certain results of API calls are saved into the "settings" as well so there might lay the issue

I've added a lastUpdated property to the settings to make sure that older states of the settings are not accidentally overwriting the newer settings. This will probably improve it a bit, but I still think that the issue with settings randomly disappearing is still present (probably a Firefox only bug).


Support installation via Helm

It'd be nice to be able to just use Helm for installing the metacontroller. That would also help upgrading to a new version.

I created a functional chart and PR'd it to the official Helm Charts repository. PR Here: https://github.com/helm/charts/pull/8591

why was the PR above closed?


v1.1.5 Connection timeout

With v1.1.5, trying to connect from any device other than localhost:5000 results in a timeout. Firewall is verified open on port 5000 for TCP and UDP
For example, even on the host machine, connecting to <IP_ADDRESS>:5000 (IP of the host) fails with timeout. Trying to connect from other machines on the LAN times out.

Is this the Windows standalone copy instead of Docker?

Correct, standalone

Ah, it's because Kestrel by default only allows localhost to access localhost, why Microsoft built it this way, I'm not sure, but all you have to do is open up appsettings.json and add the following section:
"Kestrel": {
"Endpoints": {
"Http": {
"Url": "http://<IP_ADDRESS>:5000"
},
"Https": {
"Url": "https://<IP_ADDRESS>:5011"
},
}

I have a video on this from a while back, it's a bit older but you can use it as a reference
https://youtu.be/3u1WMfnta04?si=dguKeoWP0tPJtRV7&t=227
The video also covers setting up https which has mostly been superseded by https://docs.lubelogger.com/HTTPS

Got it, even working with my domain name now, Thanks!


Update package user and Bintray repo

This change reflects that the package is now provided under user conan on Bintray.
Took me quite some time to figure out why conan could not find the package. xD
I also changed the repo-link to conan-center instead of conan-community.
As far as I understand, conan-center shall become the first place to find popular packages like this.

Thanks so much!


Broken links in README.md

There are serveral broken links of contracts in  https://github.com/EOSIO/eosio.contracts/blob/master/README.md.

PR #228 fixed this issue.


Makefile: Fix the OpenTitan target

Signed-off-by: Alistair Francis<EMAIL_ADDRESS>

bors d+

bors r+

bors try

bors r+


$SnifferProvider incorrectly detect NW.js as not supporting history API

Note: for support questions, please use one of these channels: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#question. This repository's issues are reserved for feature requests and bug reports.
Do you want to request a feature or report a bug?
bug
What is the current behavior?
$SnifferProvider detects NW.js as Chrome App, therefore not supporting history API. This caused angular-ui-router buggy with html5Mode on.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar (template: http://plnkr.co/edit/tpl:yBpEi4).
Load an Angular.js app in NW.js and check the value of $sniffer.history in DevTools. See nwjs/nw.js#5478 for sample app.
What is the expected behavior?
Should return true for NW.js app
What is the motivation / use case for changing the behavior?
See nwjs/nw.js#5478
Which versions of Angular, and which browser / OS are affected by this issue? Did this work in previous versions of Angular? Please also test with the latest stable and snapshot (https://code.angularjs.org/snapshot/) versions.
1.5.7
Other information (e.g. stacktraces, related issues, suggestions how to fix)
NW.js app can make use both Chrome App APIs and Node.js APIs. So $SnifferProvider detects NW.js app as Chrome App. But NW.js app supports HTML5 history API, which is different from Chrome App.
Fix can be done by changing following lines in $SnifferProvider:
        isNwjs = $window.nw && $window.nw.process,
        hasHistoryPushState = (isNwjs || !isChromePackagedApp) && $window.history && $window.history.pushState,

any updates ?

Not yet, but the PRs plz! label is there :wink:

I'm looking into it, I'll send a PR when done.


Links cause narrow page on mobile

using word-wrap: break-word in a relevant media query should suffice

It can just be done on the body tag as a default 🤔

Fixed in v<IP_ADDRESS>


Could not load an opus lib

Hello, I'm having some trouble running this bot in Discord. I am on a Macbook running OS X Sierra. I am getting the error "Could not load an opus lib". I have already tried running update_macdeps.command (and restarting). I have also attempted manually installing libopus-0.x64 and libopus-0.x86 directly into the MusicBot folder. I have pasted the log below:

Can you confirm you've followed the OS X guide? https://github.com/Just-Some-Bots/MusicBot/wiki/Guide-for-OSX

I actually never knew there was a guide here, I followed a guide elsewhere. I comepleted the steps on this guide and I am still getting the same error.

Still looking to fix this issue, any other suggestions?

Should I repost this question? I do not appear to be getting any more replies.

As this is the same problem as #721, I'm going to close this one so that we have a central place for the issue.


Bug: Export recommended file extension should reflect export extension selected

For example, when I export as an XLSX, and then immediately try exporting as a CSV, it is giving "Export.xlsx" as my recommended file name even though it should be .CSV. Perhaps override the extension at the end depending on the selection made (regex?)
Probably has to do with the new recall feature, just needs that small tweak to make it perfect.

fix released here: https://github.com/luusluus/vscode-parquet-visualizer/releases/tag/v0.19.0

Good now.


Redirect to the commit page after applying patch (#23056) & Fix commit name in Apply Patch page (#23086)

Backport #23056
Backport #23086
Fixes https://github.com/go-gitea/gitea/issues/22621
Fixes
https://github.com/go-gitea/gitea/issues/22621#issuecomment-1439309200

@zeripath
I removed ctx.Data["TreePath"] in the original PR(https://github.com/go-gitea/gitea/pull/23056), since it is not necessary.

@zeripath I removed ctx.Data["TreePath"] in the original PR(#23056), since it is not necessary.

I'm going to leave it in because I do not know if there are templates within in 1.18.x which will need this.
Removing unnecessary values should only be done in major releases - you cannot assume that users do not have templates that rely on these and if they do they will then suffer internal server errors unnecessarily.


Description and parameters

What does this bug check indicate? Which could be the values for its parameters?

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

ID: 104442f4-9a8a-ee2b-a035-5723ffdae0ef
Version Independent ID: 8dcd0a0e-feba-4403-9308-277c45a71fd7
Content: Bug Check 0x8 IRQL_NOT_DISPATCH_LEVEL - Windows drivers
Content Source: windows-driver-docs-pr/debugger/bug-check-0x8--irql-not-dispatch-level.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @DOMARS
Microsoft Alias: domars

@LuigiBruno  - Thanks for the input, that we should add additional information on Bug Check 0x8.
We use customer input like this as well as data on how often the bug checks occur, to determine how we allocate our limited documentation team resources.
I'm not able to provide any additional information on this stop code.
This stop code doesn't currently occur on a frequent basis, but we will continue to monitor it and the other stop codes. If it becomes more active in the future, we can consider further work.
Thanks for your interest in improving the stop code documentation.

@LuigiBruno - Thanks for the input, that we should add additional information on Bug Check 0x8.
We use customer input like this as well as data on how often the bug checks occur, to determine how we allocate our limited documentation team resources.
I'm not able to provide any additional information on this stop code.
This stop code doesn't currently occur on a frequent basis, but we will continue to monitor it and the other stop codes. If it becomes more active in the future, we can consider further work.
Thanks for your interest in improving the stop code documentation.

OK @DOMARS. Thank you.


BTS-325 | Docs: Referencing a vertex collection in _from/_to that is not in named graph returns 404

Scope & Purpose
Adjust DocuBlock to reflect behavior since https://github.com/arangodb/arangodb/pull/16732
Checklist

[x] :books: documentation written (release notes, API changes, ...)
[x] Backports

[x] Backport for 3.10:

Related Information

[x] Docs PR: https://github.com/arangodb/docs/pull/1296 (https://github.com/arangodb/docs/pull/1296/commits/f9e8a01172481db59cc5873f54c09db17d41cd21)
[ ] Enterprise PR:
[x] GitHub issue / Jira ticket: https://arangodb.atlassian.net/browse/BTS-325
[ ] Design document:

[ ] https://jenkins.arangodb.biz/view/Documentation/job/arangodb-ANY-examples/2952/


tri_to_vec timing

After a little discussion with @c0g today about this, it looks like we're missing a -O2 flag when we compile the tf ops. However, I can't replicate the slowness of this op with or without that flag. Perhaps user ops have improved in tf11?
My test looks like this:
X = np.random.randn(1, 21)
L = GPflow.tf_wraps.vec_to_tri(X)
sess = tf.Session()
%timeit sess.run(L)

I get 155us. For reference, a simple reshape is almost exactly the same amount of time:
Y = tf.reshape(X, (3, 7))
%timeit sess.run(L)  # 155us

@markvdw , can you replicate these times? I suspect that you'll get slightly faster times all round on an ubuntu box.

It'll take a few days, but I'll get round to it!

Fyi, on my laptop with tf0.11rc0 I get:

vec_to_tri
The slowest run took 27.86 times longer than the fastest. This could mean that an intermediate result is being cached.
10000 loops, best of 3: 96.2 µs per loop

reshape:
The slowest run took 19.93 times longer than the fastest. This could mean that an intermediate result is being cached.
10000 loops, best of 3: 98.7 µs per loop

I don't get any difference either:
In [1]: import tensorflow as tf

In [2]: import GPflow

In [3]: %paste
X = np.random.randn(1, 21)
L = GPflow.tf_wraps.vec_to_tri(X)
sess = tf.Session()
%timeit sess.run(L)

## -- End pasted text --
The slowest run took 24.97 times longer than the fastest. This could mean that an intermediate result is being cached
10000 loops, best of 3: 77.8 µs per loop

In [4]: %paste
Y = tf.reshape(X, (3, 7))
%timeit sess.run(L)  # 155us

## -- End pasted text --
The slowest run took 23.68 times longer than the fastest. This could mean that an intermediate result is being cached
10000 loops, best of 3: 76.6 µs per loop\

I'll see if I can replicate the issue with an older version of TensorFlow

I think that's what we need to get the feature back in.

Ok, I can't replicate the op speeding up, not even with larger matrices, or a changing matrix for every sess.run. However, the situation that prompted us removing it, was the MNIST experiment slowing down. It should be easy to check whether this slowdown persists when using the 'LowerTriangular' transform.
I can do this once I get my GPU setup working.

Our GPU setup has just been setup so if I can run something let me know (though only TF0.11rc1 as we need CUDA8).

Ok that should work. Essentially, we need to run the MNIST demo, once normal, and once with the line:
m.q_sqrt = Param(m.q_sqrt.value, transform=GPflow.transforms.LowerTriangular())
after the model definition (where m is the model).
We need to confirm there is no significant slowdown. We also need to ensure that:
GPflow.settings.dtypes.float_type = 'float32'
Is set at the start.

My setup:

I checked out the mnist branch and merged master into it (I needed the TF11 fixes)
Then I adapted the mnist_demo script, changed gpflowrc for float32 etc.
I then just ran the full mnist_demo normally. I will post the full results later, it was about 34 min per 20k iterations.
Then I added:
m_vb.q_sqrt = GPflow.param.Param(m_vb.q_sqrt.value, transform=GPflow.transforms.LowerTriangular())

But got the following error running the demo:
m_vb.optimize( tf.train.AdadeltaOptimizer(learning_rate=step_rates[repeatIndex], rho=0.9, epsilon=1e-4, use_locking=True) , maxiter=vb_max_iters[repeatIndex] )
  File "/home/ibcn/icouckuyt/projecten/GPflow/GPflow/model.py", line 217, in optimize
    return self._optimize_tf(method, callback, maxiter, **kw)
  File "/home/ibcn/icouckuyt/projecten/GPflow/GPflow/model.py", line 223, in _optimize_tf
    opt_step = self._compile(optimizer=method)
  File "/home/ibcn/icouckuyt/projecten/GPflow/GPflow/model.py", line 124, in _compile
    self._free_vars = tf.Variable(self.get_free_state())
  File "/home/ibcn/icouckuyt/projecten/GPflow/GPflow/param.py", line 751, in get_free_state
    return np.hstack([p.get_free_state() for p in self.sorted_params] +
  File "/home/ibcn/icouckuyt/projecten/GPflow/GPflow/param.py", line 245, in get_free_state
    return self.transform.backward(self.value.flatten())
  File "/home/ibcn/icouckuyt/projecten/GPflow/GPflow/transforms.py", line 231, in backward
    y = np.reshape(y, (N, N, self.num_matrices))
  File "/home/ibcn/icouckuyt/.local/lib/python2.7/site-packages/numpy/core/fromnumeric.py", line 225, in reshape
    return reshape(newshape, order=order)
ValueError: total size of new array must be unchanged

I will take a more in-depth look when I have more time, and also post the full mnist_demo script. but if someone can pinpoint me the problem that would be great.

That stack looks like a bug in vec2tri, @markvdw ?

This needs some revisiting. It's definitely vec_to_tri, raising the exception, but it could be a bug elsewhere ;)

The results of the mnist run without LowerTriangular().
`repeatIndex  0
Timing:  2091.12238097  s
percent  0.9782
cycle_diff  2092.96525788
repeatIndex  1
Timing:  3018.79640388  s
percent  0.978
cycle_diff  3020.58506393
repeatIndex  2
Timing:  4033.80502486  s
percent  0.9789
cycle_diff  4035.56956697
repeatIndex  3
Timing:  5032.007231  s
percent  0.9794
cycle_diff  5033.88060594
repeatIndex  4
Timing:  6034.48720789  s
percent  0.9793
cycle_diff  6036.31502199`
'Timing' is the optimization time. Cycle diff is the time of optimization + prediction.
And also the modified mnist_demo.py if it may help:
https://gist.github.com/icouckuy/740bb55b9a33226705f618e3d32d6173
Disregard me trying to set float32 in code. I had to change gpflowrc after all as modules set float_type on import.

Ok, so for each repeat the speed is about 9.56 iter per second?

No longer relevant because of #430


Support via powershell.slack.com is a brick wall

The github issues page directs users to powershell.slack.com but there is no way to get in to that site. The page at https://powershell.slack.com presents a login (which I don't have) but no register page. New users are directed to "Contact the workspace administrator for an invitation". So that's a brick wall - unless I'm supposed to request access here?

Does it? OK, that has to be changed to this link: https://j.mp/psslack
If you prefer the Discord platform, you can access that from: https://j.mp/psdiscord
Both environments are bridged via a bot, so you'll get a similar experience in both. 🙂

Yaaaay!


Ooyala token issue for TV3 Ireland

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.01.10. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x ] I've verified and I assure that I'm running youtube-dl 2017.01.10

Before submitting an issue make sure you have:

[x ] At least skimmed through README and most notably FAQ and BUGS sections
[x ] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
stevie@stevie-Standard-PC-Q35-ICH9-2009:~$ youtube-dl -v http://www.tv3.ie/3player/show/811/
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'http://www.tv3.ie/3player/show/811/']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.01.10
[debug] Python version 2.7.12 - Linux-4.4.0-31-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: none
[debug] Proxy map: {}
[generic] 811: Requesting header
WARNING: Falling back on generic information extractor.
[generic] 811: Downloading webpage
[generic] 811: Extracting information
[Ooyala] VrYzZ1ODE6l6GA5V0w3hjXksRK9C-zO4: Downloading JSON metadata
[Ooyala] VrYzZ1ODE6l6GA5V0w3hjXksRK9C-zO4: Downloading JSON metadata
ERROR: Ooyala said: invalid token
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 694, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 359, in extract
    return self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/ooyala.py", line 150, in _real_extract
    return self._extract(content_tree_url, embed_code, domain, supportedformats)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/ooyala.py", line 72, in _extract
    self.IE_NAME, cur_auth_data['message']), expected=True)
ExtractorError: Ooyala said: invalid token

stevie@stevie-Standard-PC-Q35-ICH9-2009:~$

...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

Works fine with latest version.


Are you guys doing the same?

Do you know the Neutronium project?
https://github.com/NeutroniumCore/Neutronium
It seems you guys are doing pretty much the same?

Similar but not the same.
Neutronium, I believe is focused on WPF using MVVM pattern.
Chromely lightweight - uses native GUI APIs - Win32, Gtk. Chromely has no support for WPF or WinForms but can be extended by those who may want to do that.

Thanks! Your project looks amazing! It need way more stars and supporters!

Thank you!  Help spread the word.
Appreciated.