[periscope:user] Periscope channel video count not working, not initiating channel rips

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 2<IP_ADDRESS>. 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 2<IP_ADDRESS>

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 concertize 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 ```):
$ youtube-dl -v <your command line>
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<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

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 required an account credentials please provide them or explain how one can obtain them.
Attempting to download Periscope channels yields zero videos, therefor nothing is downloaded:
youtube-dl --verbose --ignore-config --skip-download https://www.periscope.tv/PositiveSmash
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', '--ignore-config', '--skip-download', 'https://www.periscope.tv/PositiveSmash']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 3.5.2 - Linux-4.4.0-34-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: avconv 2.8.6-1ubuntu2, avprobe 2.8.6-1ubuntu2, ffmpeg 2.8.6-1ubuntu2, ffprobe 2.8.6-1ubuntu2
[debug] Proxy map: {}
[periscope:user] PositiveSmash: Downloading webpage
[download] Downloading playlist: PositiveSmash
[periscope:user] playlist PositiveSmash: Collected 0 video ids (downloading 0 of them)
[download] Finished downloading playlist: PositiveSmash

thanks


MultiDex: VM has multidex support, MultiDex support library is disabled

`public class iApplication extends Application {
@Override
public void onCreate() {
super.onCreate();
        FlowManager.init(new FlowConfig.Builder(this)
                .addDatabaseConfig(
                        new DatabaseConfig.Builder(iDataBase.class)
                                .openHelper(new DatabaseConfig.OpenHelperCreator() {
                                    @Override
                                    public OpenHelper createHelper(DatabaseDefinition databaseDefinition, DatabaseHelperListener helperListener) {
                                        return new SQLCipherHelperImpl(databaseDefinition, helperListener);
                                    }
                                })
                                .build())
                .build());

}

@Override
protected void attachBaseContext(Context base) {
    super.attachBaseContext(base);
        MultiDex.install(this);
}

}`
`public class SQLCipherHelperImpl  extends SQLCipherOpenHelper
{
public SQLCipherHelperImpl(DatabaseDefinition databaseDefinition, DatabaseHelperListener listener)
{
    super(databaseDefinition, listener);
}

}`
     <application         android:name=".iApplication" ...
When i use dbflow sql cipher the following is occurred and app is closed
07-23 11:48:23.356 8412-8412/? I/art: Late-enabling -Xcheck:jni 07-23 11:48:23.710 8412-8412/ir.hanaabonyan.plasscoapp I/MultiDex: VM with version 2.1.0 has multidex support 07-23 11:48:23.710 8412-8412/ir.hanaabonyan.plasscoapp I/MultiDex: install 07-23 11:48:23.710 8412-8412/ir.hanaabonyan.plasscoapp I/MultiDex: VM has multidex support, MultiDex support library is disabled.

this does not look like a DBFlow issue. This is a log output that has nothing to do with DBFlow.

I think this is not an error , The vm has already have the support of multidex that is why they are disabling , try to check the log in below 5.0. I guess the the ART vm have already have the multidex support


Missing empty spaces in if statement

Hi Thakkaryash,
I just tried out your repo by building it myself with Docker and triggering backup.sh manually inside the container. The shell complained about missing spaces in the if statement (but the script ran through).
The issue is in :
https://github.com/thakkaryash94/docker-postgres-backup/blob/b2c58e5f506e59907e16dc40a1c42541df9579ad/backup.sh#L15
Adding empty spaces solved the problem:
if [ -z "$S3_BUCKET" ] || [ -z "$S3_ACCESS_KEY" ] || [ -z "$S3_SECRET_KEY" ]; then
I am not really familiar with shell, but this blog says sh always expects empty spaces.
Regards

Hey @Xenotech81 , I agree. Thank you for the issue and for finding the fix as well. I have updated the code. It's fixed and docker image is updated as well.


Show local time in tooltip instead of UTC?

It's better to show the time in local time zone instead of UTC time in the time tooltip.

The team is pretty neutral on this. It actually wouldn't add visual clutter, but we don't really want to invest time in a PR. If you wrote up a PR for us, we'd probably consider merging it though :)

Hey there! The underlying data received from the NYT source is in UTC. We don't really want to add more client-side formatting to wrangle our JSON blob into local times.

Displaying timestamp tooltips does not require modifying the underlying data. I have a PR #253 that only modifies the timestamps in the html.tmpl.


improvement: use light-dark() css function for easier overrides

https://developer.mozilla.org/en-US/docs/Web/CSS/color_value/light-dark
This makes it easier for consumers to override css variables and make it easier for devs to manage this.
However it only reach all 80% of users: https://caniuse.com/?search=light-dark

/marimo create-test-release

going to hold off on this change, given light-dark() adoption


Meteor method to add new field with type to collection

I'm using node simple schema to define a schema to use for an API for example:
export const Data = new Mongo.Collection('data');
schema = {}

export const schema.data = new SimpleSchema({
    name: {
        type: String,
        label: "Device's Name",
    },
    message: {
        type: String,
        label: "Message",
    },
    data: {
        type: Date,
        autoValue: function() {
            if (this.isUpdate) {
                return new Date();
            }
        },
    },

}, { tracker: Tracker });

Data.attachSchema(data.schema);
Now I want users to be able to define and add new field with type to the data collection using a predefined method from the server that user can call on the API-Client. This method should add a new field with type to the data collection. Ex:
api.call.addData( surName: {type: Array, label: "User Surname", OtherSimpleSchema_attribute...}). But the defined field should be available only to the actual user.
Any help

Although you can do schema.data.extend(theUsersField), that would add it for all users. The best would be to store the user's extra desired fields in their user doc, and then build a schema on the fly by merging the base schema with the user schema whenever you need to validate. It's probably best not to attach it to Data collection but rather do the validation yourself. You could try reattaching for each user, but I'd be worried it might use the wrong user's schema if you're not careful.


Permission popup appears during app startup on ios

I used this library to get the notification permission on device but every time i install the app or rebuild it, permission popup appears automatically on app startup, however it was not happening when i was on version 1.1.1 . can anyone shed some light on this?
React native version: 0.59.4
ios device: iOS 12.4.5 (ipod touch)
Development Versions: macOS Catalina 10.15.4
React Native Permissions: "^2.0.0"

Where do you trigger the permissions request?
Provides some code or this issue will be closed.

@zoontek Thanks for reply.
Here is the wrapper around the lib functions according to need.
import {checkNotifications, requestNotifications, openSettings} from 'react-native-permissions';
import Log from './log';

module.exports = {
  requestPermissions() {
    return new Promise(resolve => {
      requestNotifications(['alert', 'sound', 'badge']).then(response => {
        let permission = false
        if (response.status === 'granted') {
          permission = true
        }
        const logData = {
          title: 'App asked for notification',
          message: {
            notificationStatus: permission
          }
        }
        Log.save(logData);
        if (permission) {
          resolve('Allowed');
        } else {
          resolve('Not Allowed');
        }
      })
    });
  }
}

and i call the function requestPermissions when user press on the permission dialogues delivered from the server and then this function call open the permission popup. here the things are working fine. no complaints at all!!
here is the function call: (this will be called when user press on particular dialogues for the permission)
if (inputRequest.type === 'permission_popup_open') {
          const response = await Permission.requestPermissions();
          this.handleActionPermissionButton(response);
          this.resetExecutionTime();
        }

but the problem is when i reinstall the app(by uninstalling) or update the app, then permission popup appears regardless the requestPermissions function call.
here is the pods:
  pod 'RNPermissions', :path => '../node_modules/react-native-permissions'
  permissions_path = '../node_modules/react-native-permissions/ios'

  pod 'Permission-Notifications', :path => "#{permissions_path}/Notifications.podspec"

However, it was working before when i was on version 1.1.1 but after upgrading to 2.0.0 i'm facing this.
I reckon, this is not the problem with the code. this is more with the delegates or pods.Thanks!!!

@zoontek  I just came across this a swell. I'm not calling requestNotifications or even checkNotifications() anywhere. The popup comes up every time i delete the app and reinstall
"react-native-permissions": "3.6.1"
"react-native": "0.70.4"

device is iOS 16.1


Formatter::asCurrency() different results depending on decimal part

Editing... [Sorry, I saved before the time]
Formatter behavior is different depending on the number has non zero decimal part (like .55) or not;
What steps will reproduce the problem?
In a console app (in a test), run:
$v1 = \Yii::$app->formatter->asDecimal('960000.00');
$v2 = \Yii::$app->formatter->asDecimal('960000.55');

$I->assertEquals('960.000,00', $v1); // passing
$I->assertEquals('960.000,55', $v2); // failing

What is the expected result?
Both tests true.
What do you get instead?
 Fail  Failed asserting that two strings are equal.
- Expected | + Actual
@@ @@
-'960.000,55'
+'960,000.55'

Scenario Steps:

 2. $I->assertEquals("960.000,55","960,000.55") at tests/functional/EspecificacaoDeGarantiasCest.php:56
 1. $I->assertEquals("960.000,00","960.000,00") at tests/functional/EspecificacaoDeGarantiasCest.php:55

More details
The problem also happens when using asCurrency() in similar situations.
My formatter configs:
'formatter' => [
    'defaultTimeZone' => 'America/Sao_Paulo',
    'currencyCode' => 'BRL',
    'numberFormatterOptions' => [
        NumberFormatter::MIN_FRACTION_DIGITS => 2,
        NumberFormatter::MAX_FRACTION_DIGITS => 2,
    ],
    'numberFormatterSymbols' => [
        \NumberFormatter::CURRENCY_SYMBOL => 'R$' . (PHP_OS != 'Linux' ? ' ' : ''),
    ],
],

Additional info

Q
A

Yii version
2.0.20

PHP version
7.1.16

Operating system
Linux Deepin

Formatter::normalizeNumericValue() cast the number using $value = (float) $value; // line 1724, but the cast does not work fine for non-zero decimal part value in console.
Note: in web application the problem never happens, not sure why. All locale, ICU, Intl are the same (I looked in phpinfo for both apps). The same configs for formatter and language are being used for console and web.

What do you mean by "not work fine"?

Closing for now because in Win10 I was not able to reproduce the problem. When I am at home (linux) I will investigate a bit more and give more examples about "not working".


Integrate Protein Interactions summary widget and detail view

Please integrate the existing Protein Interactions summary widget and detail view from the PoC version into the alpha version.

This ticket is pending a decision from @d0choa about whether we integrate the existing OmniPathDB database dump or move to using the networks data generated by OTAR044

Ticket moved to next sprint. Decision to be made about whether to proceed with OmniPathDB implementation for alpha release.

Ticket closed as work completed and reviewed and merged into main


Replace "i.e." with "e.g."

Replace "i.e." with "e.g."

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @nishithgadhiya, welcome and thank you for your effort.
These are actually places where "i.e." was correct


feat: Add tool for exporting mask images

:tada: This PR is included in version 1.0.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Theme fails to load in Seamonkey 2.40 Linux

I just loaded Authentic Theme 18.31 in Virtualmin and I am now faced with a spinning wheel that never stops. Doesnt matter what I choose on the left (Virtualmin/webmin) I get nothing but the spinning wheel.
I tried this theme in Chrome and Firefox and it works fine, so its just Seamonkey that its not working in.
Thanks

Hi,
I don't officially support SeaMonkey.
What is the error in console? Did you try incognito mode? Old cache or browser's extensions can cause issues.

Hi
Only extensions is weather and enigmail, so I think that should be ok. Cache cleared.
Console error is this
Timestamp: 23/12/16 10:30:38
Error: SyntaxError: missing ; before statement
Source File: https://vader.shirehosting.com.au:10000/unauthenticated/js/content.min.js?1831
Line: 6, Column: 128575
Source Code:
n}t___wi.location.href=$(this).find("a").attr("href")});let DELAY=340,clicks=0,timer=null;$("body").on("click",'.icons-r
Timestamp: 23/12/16 10:30:38
Error: TypeError: window.page.f__g is not a function
Source File: https://vader.shirehosting.com.au:10000/unauthenticated/js/parent.min.js?1831
Line: 6
Timestamp: 23/12/16 10:30:42
Error: TypeError: f.data(...) is undefined
Source File: https://vader.shirehosting.com.au:10000/unauthenticated/js/parent.min.js?1831
Line: 6

Did you try latest SM 2.46?

No I havent, I'll give it a go and report back

Ok, SM 2.46 was just made available from repos and with a large smile I can confirm that it works.
Thanks for your effort.

Great to hear! Merry Christmas!


stage0: Create an mtab symlink if not present

Several Linux tools expect this to exist, but several images do not
include it. This creates it when it does not exist.
Fixes #3264

/etc/mtab has been deprecated some time ago, but you are right that some older images still rely on it so I'm ok in adding it.
I've just some doubts with the approach here which modifies the image rootfs before execution, while for all other cases we have been doing this in prepare-app. Is this case different somehow or are there reasons to do it separately instead?

I thought about prepare-app vs stage0 and ended up with it in stage0 because I saw this behaviour as a "generic across stage1s" thing. It also isn't too different from setting up resolv.conf, which is done in stage0.
I don't think I have a strong reason beyond the desire to keep prepare-app small and be generic across stage1s.
Happy to move it if you think it should be.

@lucab ping, is @euank's explanation ok?

I acknowledge that we have an hidden problem here: an image running under stage1-fly is currently not getting the same environment setup as other stage1 pods, as we have duplicate logic for this in prepare-app and in other places.
In the long term we may need to rethink this part of image setup, possibly moving some of this to the rendering phase (similarly to here).
However, this PR alone is fine so LGTM.


Firefox Ignores Rows

We seem to be having an issue using match heights in Firefox ( latest release although it did do the same in previous version ).
Basically we have 5 rows, on each row are 4 divs ( its just a product collection page ). The JS is set so that it should apply different heights to each row, which it is doing perfectly, however in firefox it seems to be ignoring this and setting all the divs at the same height.
Let me know if you need any more specific info

Are you still having this issue? If so could you provide an example jsfiddle? Thanks

Hey
Yea were still having this issue however we think we have worked out what
is causing it, we just wont have a perfect solution yet.
The issue is being caused by how firefox handles images within a table cell.
Basically we have a div with display table set on it, and the eqheights
plugin adds the heigt to this div.
Inside this is another div set as display table-cell, and vertically
aligned, this centers the image no matter what the height.
In firefox an image inside this setup will overflow the box, to fix this
you add table-layout: fixed; to the div with display table; This fixes the
image overflowing.
So what it appears is happening is that even though the image now scales,
the jquery is still reading the height at the full size image height,
rather than its scaled height.
e.g if the full size image height is 200px, but on this page it is scaled
to 100px, the jquery still reads the height as 200px, causing the height to
be large than it actually needs to be.
Let me know if this helps you pin down the issue
Thanks
Ben
On 9 November 2015 at 20:30, Liam<EMAIL_ADDRESS>wrote:

Are you still having this issue? If so could you provide an example
jsfiddle? Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/liabru/jquery-match-height/issues/76#issuecomment-155183576
.

--
Ben Henry
Front End Developer
Squashed Pixel Ltd
t : 01535 517 729
<EMAIL_ADDRESS>www.squashedpixel.co.uk
*Connect with us on :Facebook
http://www.facebook.com/pages/Bradford/Squashed-Pixel/202190017196 |  Twitter
http://twitter.com/squashedpixel |  LinkedIn
http://www.linkedin.com/companies/squashed-pixel?trk=co_search_results&goback=.cps_1260370812220_1
*
Squashed Pixel Limited, The Studio, Bridge House, 13 Devonshire
Street, Keighley,
BD21 2BH
Registered in England & Wales Company No: 6980425 | VAT No: 978 8323 58.
The unauthorised use, disclosure, copying or distribution of this email or
any of the information it contains is prohibited and may be unlawful. If
you are not the intended recipient please notify the sender immediately.
[image: kbid-signature-support]

How is the image set to scale? If the image is bigger than the container, it will always overflow. When I tried adding any kind of scaling to the image, jQuery always reports the correct height.
Can you provide a jsfiddle?


Add error responses to route schemas

fastify-swagger doesn't appear to support declaring separate media types for responses at present


add UICI domain

Official website: https://uici.ac.id
Informatic program: https://uici.ac.id/program-studi-informatika-s1/

@hynra First stage of verification passed, official domain is uici.ac.id as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@hynra Thank you for your patience while waiting for a review and processing your request. Pull request merged.

Thank you @philipto


Add > Cadence Check > Workspace user activity

Checklist

[ X] Issue(s) linked

@misraved can you pls expedite this?

@RahulSrivastav14 could you please add the missing docs page for this report?
Thanks!!


Fix border drawn over children on iOS

Summary:
Fixes https://github.com/facebook/react-native/issues/44690
In the code responsible for drawing border on iOS there's a comment saying:

iOS draws borders in front of the content whereas CSS draws them behind the content. For this reason, only use iOS border drawing when clipping or when the border is hidden.

The condition that follows checks whether the content is clipped and the width and alpha channel of the border: https://github.com/facebook/react-native/blob/e0a2e86d0346bd7e40adf69311daa538ca8c9c5f/packages/react-native/React/Fabric/Mounting/ComponentViews/View/RCTViewComponentView.mm#L643-L644.
The problem is when the color is not set at all - colorComponentsFromColor(borderMetrics.borderColors.left).alpha will be equal to 0 since the relevant SharedColor is null: https://github.com/facebook/react-native/blob/e0a2e86d0346bd7e40adf69311daa538ca8c9c5f/packages/react-native/ReactCommon/react/renderer/graphics/platform/ios/react/renderer/graphics/HostPlatformColor.mm#L76-L86
Then it uses the path with the default iOS behavior (drawing the border on top of the content) instead of the custom one (with the border below) and it seems like it defaults to drawing black when the passed color is nil.
This PR simply adds one more check to make sure the color is actually set before choosing the default platform behavior.
Changelog:
[IOS] [FIXED] - Fixed border being drawn over children when no color was set
Test Plan:
Tested on the code from the issue.

Before
After

Are the failing actions something I should be concerned with 😅?

Are the failing actions something I should be concerned with 😅?

Nope it's a flaky internal test


Add: Movie Stream Sources ( webview or internal player )

https://sora.is-really.cool/
Or
https://documenter.getpostman.com/view/18986031/UVXdNeFD

Hello @OshekharO, Jetflix is a project to try out Jetpack Compose android UI Toolkit.
Unfortunately, for now, I don't intend to add any feature for streaming the movies. Obviously for legal reasons and not detouring from the main scope of this repository.

Hello @OshekharO, Jetflix is a project to try out Jetpack Compose android UI Toolkit. Unfortunately, for now, I don't intend to add any feature for streaming the movies. Obviously for legal reasons and not detouring from the main scope of this repository.

Ok


New out of box Photon detected as Core on Linux

The serialNumber: 'Particle_Photon' doesn't match for a new photon straight out of the box since it has 'Spark_Devices_Photon' as the serial number. Because of this, I can't set up a new Photon using the CLI.

Since you have the Linux box :smile: do you want to submit a PR?

Yeah. I just put that in so I don't forget about it.


#2712 fixed name of credentials file, removed obsolete checks

Closes #2712
Additional changes: removed outdated checks from the test scripts

Codecov Report

Merging #2713 (9bd9c79) into master (b21f0b8) will decrease coverage by 13.37%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #2713       +/-   ##
===========================================
- Coverage   36.25%   22.88%   -13.38%
===========================================
  Files         161       50      -111
  Lines        9285     1097     -8188
  Branches      197      197
===========================================
- Hits         3366      251     -3115
+ Misses       5380      819     -4561
+ Partials      539       27      -512

Flag
Coverage Δ

moduleA
22.88% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cli/pkg/websockethelper/websocket_helper.go

api/ws/hub.go

gatekeeper-service/pkg/handler/handler.go

helm-service/controller/delete_handler.go

lighthouse-service/event_handler/common.go

api/handlers/evaluation.go

cli/cmd/check_endpoint_status.go

api/handlers/metadata.go

cli/cmd/root.go

jmeter-service/common.go

... and 75 more

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 9cd0b8e...9bd9c79. Read the comment docs.


Add some debug properties to BackdropFilterLayer

Fix https://github.com/flutter/flutter/issues/101102
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 this PR is test-exempt.
[ ] All existing and new tests are passing.

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

This PR requires test

Please add all properties and this PR requires a test.

cc @TahaTesser Could you have look at this again? It needs two flutter members to review.

@Piinks Could you do a secondary review?

@wangying3426
Can you rebase or add an empty commit so Google Testing passes to get this merged?

@wangying3426 Can you rebase or add an empty commit so Google Testing passes to get this merged?

The Google testing seems has some problem, and failed many times.


[IAML/GENRES] Adding minimal hierachy

Hello,
I would like to add the following relations to the vocabulary and create 2 new concepts.
The reason are more or less the same of the ones of the issue #38 .
<http://data.doremus.org/vocabulary/iaml/genre/lit> skos:narrower <http://data.doremus.org/vocabulary/iaml/genre/vrl> , <http://data.doremus.org/vocabulary/iaml/genre/lid> , <http://data.doremus.org/vocabulary/iaml/genre/lod>, <http://data.doremus.org/vocabulary/iaml/genre/lyh> .

<http://data.doremus.org/vocabulary/iaml/genre/i> a skos:Concept ;
skos:prefLabel "Instrumental music"@en ;
skos:narrower
<http://data.doremus.org/vocabulary/iaml/genre/tri>, <http://data.doremus.org/vocabulary/iaml/genre/aub> , ...

<http://data.doremus.org/vocabulary/iaml/genre/v> a skos:Concept ;
skos:prefLabel "Vocal music"@en ;
skos:narrower
<http://data.doremus.org/vocabulary/iaml/genre/czp>, <http://data.doremus.org/vocabulary/iaml/genre/due> , ...

My points:

lit has the note "When any more specific code can’t be used"
vrl, lod, lyh have a skos:note "Liturgical", while lid has it in the name
lots of concepts have the skos:note "Instrumental" or "Vocal"

I have also to say that we have some concept from other vocabularies that we can reuse:

For instrumental we have already http://data.doremus.org/vocabulary/itema3/genre/84 , http://data.doremus.org/vocabulary/itema3/genre/musdoc/81 and http://data.doremus.org/vocabulary/diabolo/genre/musique_instrumentale
For vocal music we have http://data.doremus.org/vocabulary/itema3/genre/musdoc/76 and http://data.doremus.org/vocabulary/diabolo/genre/musique_vocale

I'm not questioning the need for those new skos:Concept. However, I found awkward to define those new terms in the <http://data.doremus.org/vocabulary/iaml/genre/> concept scheme because there are not (yet) defined in IAML.
Should we try first to ask IAML whether they would be interested to add those concepts too? If not, should we not define those concepts in the reconciled concept scheme (pivot graph) that aligns IAML with other genre vocabularies?

Should we try first to ask IAML whether they would be interested to add those concepts too?

It would be interesting. It needs a discussion in the modelling group.

If not, should we not define those concepts in the reconciled concept scheme (pivot graph) that aligns IAML with other genre vocabularies?

Yes, but the pivot graph does not exists in this moment :)

@kgtodorov Can you let us know the status regarding the validation of all alignments between controlled vocabularies, and in particular, for genres and mop? Do we have a chance to have a reconciled graphs before the next general meeting?

Luckily, my student in charge of that is back to work after some hard times. We have now all new alignments (mop and genres), which are of significantly higher coverage than the old ones thanks to the new algorithm using directly SKOS (so no conversion to OWL). However, we need to go again through the validation process before we leverage the final alignments, this would definitely take more time than the couple of weeks before the AG.

So I postpone it after #22

Modelling meeting:

add it in the pivot vocabulary
propose to IAML the adoption of this hierarchy (I am writing an email with Francoise)


Allow serializer to serialize "Any"

I can't even use the serializer on:
let msg: [String: JSONRepresentable] = ["name": "Robert"]

It may be significantly slower but it'd be good if the serializer could take Any and just recursively serialize it throwing an error if something isn't JSONRepresentable.

Closed via #16


Logging configuration

This PR accomplishes two primary goals:

Normalize logging usage in pyOCD. Every module that writes log messages now uses a module-specific logger with the name of that module, for instance pyocd.coresight.cortex_m.
Enable logging configuration to be customized. A logger user option can be set to a logging configuration dictionary to fully configure all aspects of logging. Full documentation is included.

A few related changes were included:

Fatal errors are logged by the pyocd tool at the CRITICAL level.
The various LOG_X flags in some modules to control trace logs were converted to trace loggers that have a default level of CRITICAL, so they won't output anything unless overridden by a logging configuration. This lets you control trace output without having to modify source.
GDB server and packet threads set the thread name (useful if you set a log formatter that prints the thread name).

/morph test

@hakehuang You asked for better log control long ago, it's finally here! 😀

@flit nice work! thanks a lot


forc fmt hangs on an empty project

If Forc.toml is missing from a folder, forc fmt just hangs instead of emitting a clean error message.

Would you mind adding which project you ran this on? I've run it on projects without a Forc.toml and it does produce the error message 🤔

Ohhh I see what was happening. I had an empty project that lived inside another project that wasn't empty and so the Forc.toml from the parent project was kicking in, so forc fmt was taking a long time so I thought it was hanging. I will close this issue.


progressUpdateInterval Not Working

I'm trying to do this:
<Video
  source={{uri: this.props.uri}}
  repeat
  resizeMode="cover"
  progressUpdateInterval={16}
  onProgress={play => this.setState({
    playProgress: play.currentTime / play.playableDuration
  })}
/>

But onProgress is still being called on the default 250ms interval.
I added some logging in this component and found setSource() is being called before setProgressUpdateInterval(), so setSource() is always using the default.
I couldn't figure out how to fix it :(
Versions
iPhone 7+
react-native-video: 1.0.0
react-native: 0.42.0

+1

I fiexd ract-native-video/ios/RCTVideo.m  line63
// _progressUpdateInterval = 250;
_progressUpdateInterval = 500;

It works.

+1

+1

+1 please address this


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter apcups for common errors and appropiate suggestions to keep this adapter up to date.
Please see the result of the check below.
ioBroker.apcups
   -

:thumbsup: No errors found
WARNINGS:

[ ] :eyes: [E851] .npmignore found but "files" is used at package.json. Please remove .npmignore.
[ ] :eyes: [W130] Many "common.news" found in io-package.json. Repositorybuilder will truncate at 7 news. Please remove old news.

Please review issues reported and consider fixing them as soon as appropiate.
Warnings reported by repository checker should be reviewed. While some warnings can be considered as a suggestion and be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence

Fixed

Thanks for processing and closing this issue.
I will recheck for new issues later. So it's no problem if this issue has been closed premature.
You may start a new check at any time by adding the following comment to this issue:
@iobroker-bot recheck
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.  Let's work together for the best user experience.
your
ioBroker Check and Service Bot


[PHP] Add PHP 8.1

Follow-up to #196185 (PHP 8.4), #196728 (PHP 8.3), and #196743 (PHP 8.2)
This PR requests to add PHP 8.1.
PHP 8.1 is currently the oldest supported PHP version, reaching EOL on 2025 December.
Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.9 schema?

Microsoft Reviewers: Open in CodeFlow

Verify:

(This is for an ancellary EXE, because I couldn't find how to have the main EXE display its version number.)


Suggestion: Add a deep learning team

Deep learning is very popular
But Trust did not invest in the development of artificial intelligence in a timely manner
Causing a lack of artificial intelligence ecology
Artificial intelligence cannot achieve effective development through third-party means
The Deep Learning Framework of the Current Community
Many of them cannot be used normally or are troublesome to use
Suggest adding a team for deep learning
Implement inference frameworks for various models
The development roadmap requires the addition of artificial intelligence teams
Provide reasoning for various models
No need to train the model in the early stage
Just need an inference framework
Why Python is becoming increasingly popular
Because of the development of the artificial intelligence industry
Why doesn't Rust value artificial intelligence?

Anyone is free to use Rust to develop artificial intelligence systems. However, the Teams of the project exist to handle the specific needs of the project, not just to develop Rust code in general.
Here's an example: Even though Firefox uses a lot of Rust code, we don't have a Firefox specific team within the rust project.

I don't know of any mainstream programming language with an official deep learning library. These libraries are always developed by third parties.
In 2017, the Rust project invested in improving and polishing some important libraries in the crates.io ecosystem. Since this wasn't repeated after 2017, I assume it was not considered a good use of their time. People in the Rust project have more than enough to do as it is.
If you want a team to improve the deep learning ecosystem, that team doesn't need to be an official part of the Rust governance. And if many people share your view of how important this is, it shouldn't be hard to find contributors.

I have to say that this is  not a rust official team should be responsible for.  If you are interested in this field, you can see the tag Artificial Intelligence from this list, then find the one you most interested in and contribute your effort.


Touchables inconsistently call press handlers

Is this a bug report?
Yes (and maybe)
Have you read the Bugs section of the Contributing to React Native Guide?
Yes
Environment

react-native -v: snack.expo.io
Target Platform: iOS, Android, (and Web)

Steps to Reproduce

Add press handlers to a Touchable
Press Touchable
Observe which handlers are logged

Expected Behavior
I expected expected each of the different Touchables to call handlers in the same way by default. Initially I assumed they might call pressIn, then press, then pressOut. Or simply press for taps.
Actual Behavior
When tapping on TouchableOpacity and TouchableWithoutFeedback it will log pressIn and press, whereas TouchableHighlight will only log press.
(For longer presses, all Touchables appear to exhibit the same behaviour and log pressIn, then longPress, then pressOut. Or if you press and then slide off the Touchable, it logs pressIn and then pressOut.)
This inconsistency is down to each touchable setting slightly different defaults for the touchable delay getters. TouchableHighlight ends up falling back to a default delay of 130ms defined in Touchable.
Is it intentional to have this difference and for onPressIn to be called without onPressOut following?
Reproducible Demo
https://snack.expo.io/rJLY6nCVZ

Turns out the bug lies in this example's logger.
Fixed: https://snack.expo.io/HkBPXykrZ


 seem to be missing even use Edge version

Description
In my project I cannot import  and it also do not have in node_modules/@nuxt/ui/dist/runtime/components/forms

Would you mind providing a reproduction and specifying which Edge version you're using?


[BUG] getTrustedAgentAccessToken don't work with private custom proxy.

Looks like the getTrustedAgentAccessToken doesn't work when we use a private proxy in code & the Postman i.e. http://localhost:3000/mobify/slas/private/shopper/auth/v1/organizations/f_ecom_bldj_dev/oauth2/trusted-agent/token. (its working when we use URL: https://{shortCode}.api.commercecloud.salesforce.com/shopper/auth/v1/organizations/{organizationId}/oauth2/trusted-agent/token manually in postman)
  `const api = useCommerceApi()
  const slasClient = api.shopperLogin
  slasClient.clientConfig.proxy = ${getAppOrigin()}/mobify/slas/private
  const base64data = btoa(${clientId}:${secret});
  const tokenResponse = slasClient.getTrustedAgentAccessToken({
    body: {
      channel_id: siteId,
      grant_type: 'client_credentials',
      idp_origin: 'ecom',
      login_id: loginId,
      enc_user_id: encUserId
    },
    headers: {
      'Authorization': Bearer ${authCode},
      '_sfdc_client_auth': base64data,
      'Content-Type': 'application/x-www-form-urlencoded'
    }
  })`

The following are the server error logs that we have after triggering this:
Failed to proxy SLAS Private Client request - 400 Please make sure you have enabled the SLAS Private Client Proxy in your ssr.js and set the correct environment variable PWA_KIT_SLAS_CLIENT_SECRET. SLAS Private Client Proxy Request URL - http://localhost:3000/mobify/slas/private/shopper/auth/v1/organizations/f_ecom_bldj_dev/oauth2/trusted-agent/token.
API returns following error (but we've entered correct value in _sfdc_client_auth because its working when we trigger URL: https://{shortCode}.api.commercecloud.salesforce.com/shopper/auth/v1/organizations/{organizationId}/oauth2/trusted-agent/token
manually in postman):
{     "status_code": "400 BAD_REQUEST",     "message": "Illegal base64 character 20" }
We've tried putting both the correct secret key as well as SLAS_SECRET_PLACEHOLDER. None appears to work.
We do have PWA_KIT_SLAS_CLIENT_SECRET set in env variables and useSLASPrivateClient enabled in ssr.js, so the error that we are getting is irrelevant. Is there something else, which is missing from docs that we need to do?

False alarm. It worked after re-installing node modules.


Bumped version to 7.0.1

I think we should release a new version at this point before merging more PRs. We already have 10 updates on this minor version.

I'd be fine with some release automation, probably with some manual triggering. It could just post a new commit with an updated changelog and tag it appropriately. I would suggest git-cliff for the job if we try to automate it.

I'll start working on it whenever I find some free time, It may take a weekend or two for me to read all of the documentation and play around with small examples so I have to postpone it until I'm finished with my current project.
I need to test it in my own repository but I'll make sure to create a draft merge request when I've got a working version so you can give feedback on the implementation.


Using mock_s3 breaks instance IAM permissions

Tests run automatically on the AWS instance. Some tests are mocking the S3 service but some of them are actually downloading some test related assets from AWS bucket.
If there are any tests with mock_s3 used before accessing the AWS bucket, IAM permission for instance won't work and traceback will be printed:
Traceback (most recent call last):
  File "/venv/local/lib/python2.7/site-packages/botocore/client.py", line 314, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/venv/local/lib/python2.7/site-packages/botocore/client.py", line 612, in _make_api_call
    raise error_class(parsed_response, operation_name)
ClientError: An error occurred (InvalidAccessKeyId) when calling the ListObjectsV2 operation: The AWS Access Key Id you provided does not exist in our records.

Though there is no AWS related environment variables or ~/.aws/credentials available as instance itself has IAM role assigned for accessing required buckets.
In tests I use the mock_s3 like that:
class Test(unittest.TestCase):
    def setUp(self):
        self.mock_s3 = mock_s3()
        self.mock_s3.start()

    def tearDown(self):
        self.mock_s3.stop()

    def test_something(self):
        cli = boto3.client('s3', config=Config(signature_version='s3v4'))
        .create_bucket(Bucket='foobar')

Everything works as expected if I change the order of tests and the ones with real S3 calls are executed before tests where S3 service is mocked.
Any thoughts / workarounds you have in mind?

# requirements.txt

moto==1.3.3
boto3==1.7.26

I'm receiving the same issue while running tests using moto on an EC2 instance using an IAM Role.
Test file
# test.py
import moto
import boto3

@moto.mock_s3
def test_a():
    s3 = boto3.client('s3')  # if you comment this out, the tests will pass

def test_b():
    s3 = boto3.resource('s3')
    b = s3.Bucket('some-bucket')
    b.download_file('some-key', '/some/path')

Command
pytest test.py

Versions
boto3==1.7.60
botocore==1.10.60
moto==1.3.5

I believe this is a dupe of https://github.com/spulec/moto/issues/1793. Let me know if not.


Create GCD.java

@Sonal-Modi-Git  It seems you are newbie to contribution. Next time when you contribute to any repository create pull request with proper description,


Externalize CSS for newLink, login, user/index, and header PHP Pages 🎨💡

Description
The newLink, login, user/index, and header PHP pages are currently utilizing embedded and inline CSS, potentially resulting in code redundancy and increased maintenance complexity. To optimize code organization, enhance maintainability, and adhere to coding best practices, our aim is to transfer all internal and inline CSS to dedicated external files for these pages. By segregating the CSS into separate external files, we can establish improved modularity and facilitate a more streamlined development process.
Key Modifications:
Relocate embedded CSS to distinct external CSS files for the newLink, login, user/index, and header PHP pages. 🔗📂
Update the corresponding PHP pages to include references to the newly externalized CSS files. 🔄
Through this effort to externalize CSS, we endeavor to foster a more structured and readily maintainable codebase, ultimately contributing to optimized performance.
We value your collaboration and insights as we continue to refine our codebase and elevate our development methodologies. 🌟📝🚀
Screenshots
No response
Additional information
No response

@apoorvaron Please Review this Issue

@Nishitbaria Include all externalise here ( Don't make another issue for externalising css )
Assigned

means all other files ?

@Nishitbaria How many pages are left ? for externalising css ?

forget-pass
footer
edit-profile
delete-link
changepassword
includes/footer
includes/header

@Nishitbaria Don't do for includes folder
Do all externalising css in 2 PR  (this PR and Make another one)
forget-pass
footer
edit-profile
delete-link
changepassword


CMD+⇑ for "Go forward"

I wanted to bind the shortcut "CMD+⇑" to the command "Go forward" and have tried the following combinations for the shortcut:
CMD+↑
CMD+⇑
⌘+⇑
⌘+↑
⌘+up
⌘+UP
CMD+UP
CMD+up
Nothing worked for me. Can you please give me a hint how to bind this shortcut? Im using the latest FF 57 and parallel the DEV Version, with the latest version of the extension with OsX 10.13.1

The solution is command+up

ah, great stuff, thanks for posting the solution here


Undefined symbols for architecture x86_64 - Xcode 7.3

Hello,
Issue exists in Xcode 7.3. I tried 3.4.0 and also from master branch. Issue exist on both.
Undefined symbols for architecture x86_64:
"Alamofire.upload (Alamofire.Method, Alamofire.URLStringConvertible, headers : [Swift.String : Swift.String]?, multipartFormData : (Alamofire.MultipartFormData) -> (), encodingMemoryThreshold : Swift.UInt64, encodingCompletion : (Alamofire.Manager.MultipartFormDataEncodingResult) -> ()?) -> ()", referenced from:

Not an issue with Alamofire. Fixed when cleared derived data.


Travis doesn't appear to be running tests

Describe the bug
Travis isn't running tests on builds. (I think we disabled this when we had to package the jar 100 times while setting up dockerization/remoting).
To Reproduce
Look at any travis build:
DiscoverySuite:
Run completed in 289 milliseconds.
Total number of tests run: 0
Suites: completed 1, aborted 0
Tests: succeeded 0, failed 0, canceled 0, ignored 0, pending 0
No tests were executed.
[INFO]
[INFO] --- scalatest-maven-plugin:2.0.0:test (alltests) @ evvo ---
Discovery starting.
Discovery completed in 191 milliseconds.
Run starting. Expected test count is: 0
DiscoverySuite:
Run completed in 302 milliseconds.
Total number of tests run: 0
Suites: completed 1, aborted 0
Tests: succeeded 0, failed 0, canceled 0, ignored 0, pending 0
No tests were executed.

Expected behavior
All tests should be run on mvn build, and therefore run by travis before approval.
Screenshots

@julian-zucker, you're definitely right, the tests aren't running on Travis. The weird part is that when I run mvn test locally on the latest version of master (746d06e) it runs the tests twice. After mvn test fails to discover tests (see below), the mvn Scalatest plugin does this twice (below below):

I reproduced locally: if you don't have pre-built code, test fails. So, if you run mvn clean test, the tests aren't found:
evvo (feature/build-speed|✚2) $ mvn clean test
[INFO] Scanning for projects...
[INFO]
[INFO] ---------------------------< com.evvo:evvo >----------------------------
[INFO] Building evvo 0.0.0
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ evvo ---
[INFO] Deleting /Users/julianzucker/Documents/projects/evvo/target
[INFO]
[INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ evvo ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 3 resources
[INFO]
[INFO] --- maven-compiler-plugin:2.3.2:compile (default-compile) @ evvo ---
[INFO] Nothing to compile - all classes are up to date
[INFO]
[INFO] --- scala-maven-plugin:3.3.2:compile (default) @ evvo ---
[INFO] Using incremental compilation
[WARNING] Pruning sources from previous analysis, due to incompatible CompileSetup.
[INFO] Compiling 19 Scala sources to /Users/julianzucker/Documents/projects/evvo/target/classes...
[INFO]
[INFO] --- maven-resources-plugin:2.6:testResources (default-testResources) @ evvo ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory /Users/julianzucker/Documents/projects/evvo/src/test/resources
[INFO]
[INFO] --- maven-compiler-plugin:2.3.2:testCompile (default-testCompile) @ evvo ---
[INFO] Nothing to compile - all classes are up to date
[INFO]
[INFO] --- maven-surefire-plugin:2.7:test (default-test) @ evvo ---
[INFO] Tests are skipped.
[INFO]
[INFO] --- scalatest-maven-plugin:2.0.0:test (test) @ evvo ---
Discovery starting.
Discovery completed in 229 milliseconds.
Run starting. Expected test count is: 0
DiscoverySuite:
Run completed in 329 milliseconds.
Total number of tests run: 0
Suites: completed 1, aborted 0
Tests: succeeded 0, failed 0, canceled 0, ignored 0, pending 0
No tests were executed.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  18.385 s
[INFO] Finished at: 2019-06-01T15:53:18-04:00
[INFO] ------------------------------------------------------------------------

Actually, after running more tests locally, it appears that running clean even once makes the tests stop running. I have a theory about why this is, if it works I'll make a PR


Downloading set of natural products

Hi - I'd really like to get the set of all CHEBI products that are derived from a natural source. I don't want to have to install the DB locally.
I tried searching for 'natural product' which gave me ~3000 results - brilliant - but I am only allowed to view or download the first 1000 (can you not allow download of all results, even if the display is limited?)
So I then had a look in your downloads and found a file compound_origins.tsv which seems to contain 8000 entities originating from organisms, so I've used this in combination with compounds.tsv to get the CHEBI names.
What I really need in addition though is the CHEBI synonyms, is there a way I can get these?
thanks.

Hi,
for your first question, you could try to go to Advanced Search
in Text Queries and search for * in ALL SPECIES.
increase the amount of results to 30k on top left.
and search.
when you get the results, on the top right you have 3 options of download format: TSV, XML and SDF.
for your second question, you need the names.tsv file.
the link between all 3 files will be the compound_id
so names and origins compound_id point to id in compounds.tsv
and within the compounds.tsv id points to parent_id until is null.
I hope it helps, you can read the manuals here and here:
http://www.ebi.ac.uk/chebi/userManualForward.do
http://www.ebi.ac.uk/chebi/developerManualForward.do
If you need further help, please let us know,
Best Regards,
Adriano.

Thanks Adriano, that’s useful.
As a quick follow-on Q - can you tell me what '(rel)-‘ means after a term name? Is it safe to strip these?
thanks
Jane

On 28 Nov 2016, at 15:58, Adriano Dekker<EMAIL_ADDRESS>wrote:
Hi,
for your first question, you could try to go to Advanced Search
in Text Queries and search for * in ALL SPECIES.
increase the amount of results to 30k on top left.
and search.
when you get the results, on the top right you have 3 options of download format: TSV, XML and SDF.
for your second question, you need the names.tsv file.
the link between all 3 files will be the compound_id
so names and origins compound_id point to id in compounds.tsv
and within the compounds.tsv id points to parent_id until is null.
I hope it helps, you can read the manuals here and here:
http://www.ebi.ac.uk/chebi/userManualForward.do http://www.ebi.ac.uk/chebi/userManualForward.do
http://www.ebi.ac.uk/chebi/developerManualForward.do http://www.ebi.ac.uk/chebi/developerManualForward.do
If you need further help, please let us know,
Best Regards,
Adriano.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ebi-chebi/ChEBI/issues/3221#issuecomment-263309606, or mute the thread https://github.com/notifications/unsubscribe-auth/AGVQwxiWFI4gYPQDIE3G0a2e3uD6LO5Lks5rCvojgaJpZM4K95Fi.

Hi Jane,
Further to Adrano's reply - Searching in the species table as he describes is a good start, but there are still a significant number of natural products that have not yet been included in the species table. Some of these can be found quite easily:
In the Advanced Search page, create and run the search as described by Adriano above (it found 13,049 answers when I tried it a few minutes ago).
Click on Edit Search. In the Filter by Database section, and add the query:
OR contains a database cross-reference in KNApSAcK
Then run the search again (currently finds 16,087 answers).
The (rel)- at the end (or the beginning) of a name is mostly found on 2-star entries. It means that only relative stereochemistry is shown. It generally means that there are two possibilities - either the stereochemistry is as shown in the structure and name (say, 1R,2R,5S) or it is the complete opposite (1S,2S,5R) - they don't know which. Determining which one is correct can be very difficult and time consuming - often it is not established until many years after a natural product was first reported in the literature.
Cheers,
Gareth


need to improve the front -end.

web-sit require an small small changes on various  places like nav bar or front page and last description.


QSVM classification tutorial fails with Scipy 1.6.0

Information

Qiskit Aqua version: 0.8.1
Python version: 3.8
Operating system: Linux

What is the current behavior?
When running the qsvm tutorial with the latest scipy (1.6.0) the tutorial fails in the qp_solver module trying to run cvxpy to solve the quadratic programming problem with a mismatched dimension error:
ValueError                                Traceback (most recent call last)
<ipython-input-3-3863988dcf61> in <module>
      5 quantum_instance = QuantumInstance(backend, shots=1024, seed_simulator=seed, seed_transpiler=seed)
      6
----> 7 result = qsvm.run(quantum_instance)
      8
      9 print(f'Testing success ratio: {result["testing_accuracy"]}')

~/git/qiskit/qiskit-tutorial/test_notebooks/lib/python3.8/site-packages/qiskit/aqua/algorithms/quantum_algorithm.py in run(self, quantum_instance, **kwargs)
     69                 self.quantum_instance = quantum_instance
     70
---> 71         return self._run()
     72
     73     @abstractmethod

~/git/qiskit/qiskit-tutorial/test_notebooks/lib/python3.8/site-packages/qiskit/aqua/algorithms/classifiers/qsvm/qsvm.py in _run(self)
    470
    471     def _run(self):
--> 472         return self.instance.run()
    473
    474     @property

~/git/qiskit/qiskit-tutorial/test_notebooks/lib/python3.8/site-packages/qiskit/aqua/algorithms/classifiers/qsvm/_qsvm_binary.py in run(self)
    133     def run(self):
    134         """Put the train, test, predict together."""
--> 135         self.train(self._qalgo.training_dataset[0], self._qalgo.training_dataset[1])
    136         if self._qalgo.test_dataset is not None:
    137             self.test(self._qalgo.test_dataset[0], self._qalgo.test_dataset[1])

~/git/qiskit/qiskit-tutorial/test_notebooks/lib/python3.8/site-packages/qiskit/aqua/algorithms/classifiers/qsvm/_qsvm_binary.py in train(self, data, labels)
     80         labels = labels * 2 - 1  # map label from 0 --> -1 and 1 --> 1
     81         labels = labels.astype(np.float)
---> 82         [alpha, b, support] = optimize_svm(kernel_matrix, labels, scaling=scaling, lambda2=lambda2)
     83         support_index = np.where(support)
     84         alphas = alpha[support_index]

~/git/qiskit/qiskit-tutorial/test_notebooks/lib/python3.8/site-packages/qiskit/aqua/utils/qp_solver.py in optimize_svm(kernel_matrix, y, scaling, maxiter, show_progress, max_iters, lambda2)
     93         [G@x <= h,
     94          A@x == b])
---> 95     prob.solve(verbose=show_progress, qcp=True)
     96     result = np.asarray(x.value).reshape((n, 1))
     97     alpha = result * scaling

~/git/qiskit/qiskit-tutorial/test_notebooks/lib/python3.8/site-packages/cvxpy/problems/problem.py in solve(self, *args, **kwargs)
    288         else:
    289             solve_func = Problem._solve
--> 290         return solve_func(self, *args, **kwargs)
    291
    292     @classmethod

~/git/qiskit/qiskit-tutorial/test_notebooks/lib/python3.8/site-packages/cvxpy/problems/problem.py in _solve(self, solver, warm_start, verbose, parallel, gp, qcp, **kwargs)
    567
    568         self._construct_chains(solver=solver, gp=gp)
--> 569         data, solving_inverse_data = self._solving_chain.apply(
    570             self._intermediate_problem)
    571         solution = self._solving_chain.solve_via_data(

~/git/qiskit/qiskit-tutorial/test_notebooks/lib/python3.8/site-packages/cvxpy/reductions/chain.py in apply(self, problem)
     63         inverse_data = []
     64         for r in self.reductions:
---> 65             problem, inv = r.apply(problem)
     66             inverse_data.append(inv)
     67         return problem, inverse_data

~/git/qiskit/qiskit-tutorial/test_notebooks/lib/python3.8/site-packages/cvxpy/reductions/matrix_stuffing.py in apply(self, problem)
     84         # Form the constraints
     85         extractor = CoeffExtractor(inverse_data)
---> 86         new_obj, new_var, r = self.stuffed_objective(problem, extractor)
     87         inverse_data.r = r
     88         # Lower equality and inequality to Zero and NonPos.

~/git/qiskit/qiskit-tutorial/test_notebooks/lib/python3.8/site-packages/cvxpy/reductions/qp2quad_form/qp_matrix_stuffing.py in stuffed_objective(self, problem, extractor)
     47         # extract to x.T * P * x + q.T * x + r
     48         expr = problem.objective.expr.copy()
---> 49         P, q, r = extractor.quad_form(expr)
     50
     51         # concatenate all variables in one vector

~/git/qiskit/qiskit-tutorial/test_notebooks/lib/python3.8/site-packages/cvxpy/utilities/coeff_extractor.py in quad_form(self, expr)
    156         for var_id, offset in offsets:
    157             if var_id in coeffs:
--> 158                 P = sp.block_diag([P, coeffs[var_id]['P']])
    159                 q = np.concatenate([q, coeffs[var_id]['q']])
    160             else:

~/git/qiskit/qiskit-tutorial/test_notebooks/lib/python3.8/site-packages/scipy/sparse/construct.py in block_diag(mats, format, dtype)
    691     row = np.concatenate(row)
    692     col = np.concatenate(col)
--> 693     data = np.concatenate(data)
    694     return coo_matrix((data, (row, col)),
    695                       shape=(r_idx, c_idx),

<__array_function__ internals> in concatenate(*args, **kwargs)

ValueError: all the input arrays must have same number of dimensions, but the array at index 0 has 1 dimension(s) and the array at index 1 has 2 dimension(s)

Steps to reproduce the problem
Run https://github.com/Qiskit/qiskit-tutorials/blob/master/tutorials/machine_learning/01_qsvm_classification.ipynb with scipy 1.6.0 installed.
What is the expected behavior?
The tutorial works without error
Suggested solutions
It looks like this is being caused by a mismatched dimension between P we pass into cvxpy as part of the quadratic programming solver and when cvxpy tries to make a sparse block diagonal array with the input it errors because the dimensions differ, probably (n, 1) != (n,). We should either fix the issue in qp_solver, or if it's an issue in cvxpy's internals report this upstream and try to get a fix.

We run that same tutorial on the Aqua CI and it passes on python 3.8. For a recent run:
https://github.com/Qiskit/qiskit-aqua/runs/1824072555?check_suite_focus=true
Also, our CI saves the tutorials as artifacts that can be downloaded. I downloaded the artifact tutorials3.8.zip and looked at the notebook in question and it seemed fine to me. Our tutorial CI is using scipy 1.6.0:
Requirement already satisfied: scipy>=1.4 in /opt/hostedtoolcache/Python/3.8.7/x64/lib/python3.8/site-packages (from qiskit-aqua==0.9.0) (1.6.0)

The difference in the qiskit/qiskit CI is that it runs on released aqua and not master  (since they are run to verify that the versions in the metapackage are compatible and don't regress/are backwards compatible) Looking at the git log I'm wondering if https://github.com/Qiskit/qiskit-aqua/commit/739e3302942a66070566ad28e3708a5f7be93550#diff-373b76176a94d6c7d7584735fbb83cfde7d8b4f678431361d7ae85958f5b323a fixed this issue. I know we're in a weird place being in the middle of a transfer but maybe we want to push that out as part of a 0.8.2

I'm also working on trying to fix the CI jobs in qiskit/qiskit, the numpy 1.20.0 release and cvxpy 1.1.8 release broken things have it stuck pretty good. But, once it's working again I can try removing the constraints pin and seeing if the job works

@mtreinish Did you get a chance to check as per above?

No longer relevant


Use github action instead of bash uploader for codecov

T/O

Codecov Report

Merging #288 (2fc49a2) into master (6c7f27d) will decrease coverage by 1.62%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #288      +/-   ##
==========================================
- Coverage   81.31%   79.69%   -1.63%
==========================================
  Files          47       47
  Lines       15563    17589    +2026
==========================================
+ Hits        12655    14017    +1362
- Misses       2343     3007     +664
  Partials      565      565


Fix documentation for makefile changes

If we merge #207 we need to carry out updates to the docs to reflect this, particularly in structure.md.

Same goes for updating the style guidance if we merge #204, depending on our decisions, so the style guide is more prescriptive than descriptive. Same applies to the laptop setup guide depending on our tech choices in #207 and others. I might leave this issue as one to review all of our documentation once we've merged the recent PRs in the cookiecutter.

Ok good point, I will edit the title to reflect this


feat: S3

i would like to rename this PR to feat: asset storage instead of s3.
Any objections?

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add collapsable_if_else

What it does
Checks for collapsible else { if ... } expressions
that can be collapsed to else if ....
Why is this bad?
Each if-statement adds one level of nesting, which
makes code look more complex than it really is.
Example

if x {
    …
} else {
    if y {
        …
    }
}

Should be written:
if x {
     …
} else if y {
    …
}

https://rust-lang.github.io/rust-clippy/master/index.html#/collapsible_else_if

Hi @0xLucqs is this a good first issue? Can I hop in?

hey! @0xLucqs PR ready for review 😊


Squatters on names

Just received a request today to put my library on Bower. 1st time coming here and looked to see if the name was available. I tried searching typer and then typer-js. Both are taken by the same user and both point to the same repo. Pretty sure that's not the thing to do. #OhWell

That's something we allowed to do unfortunately


AdGuard integration unable to add some sensors/switches

The problem

After adding the AdGuard integration there are immediately 3 entities not available:
sensor.adguard_dns_queries_blocked
sensor.adguard_rules_count
switch.adguard_query_log
Statistics and logs are enabled in AdGuard Home and there are no error logs in HASS.
AdGuardHome installation version 0.100.9 in lxc container.
Environment

Home Assistant release with the issue: 0.105.5
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.): Docker
Integration causing this issue: AdGuard
Link to integration documentation on our website: https://www.home-assistant.io/integrations/adguard/

Problem-relevant configuration.yaml

Traceback/Error logs

Additional information

Those sensors are disabled by default, you can enable them again on the device if you like to.

Oh that is true, sorry. Didn't realize that.


ARC: fault: Fix uninitialized memory access

Found this via static analysis. When CONFIG_MULTITHREADING is not set, guard_end is left uninitialized and is subsequently used in a comparison.

Thanks, this indeed looks like potentially causing random results for the check if an exception was due to stack checking or not when multithreading is disabled.
@evgeniy-paltsev, any way to determine the stack bounds in that case?

the Cortex-M port does the following:
	if (IS_MPU_GUARD_VIOLATION((uint32_t)z_main_stack,
			guard_len,
			fault_addr, psp)) {
		/* Thread stack corruption */
		return (uint32_t)K_THREAD_STACK_BUFFER(z_main_stack);
	}

Which I think would translate to:
guard_start = z_main_stack - Z_ARC_STACK_GUARD_SIZE;
guard_end = z_main_stack;

I'm happy to update the PR with the above, but would appreciate someone confirming that this sounds right given how little I know about the ARC code.

That sounds about right, yes. In arch/arc/core/thread.c the main_stack is set up similarly but with some macros for alignment handling:
	void *main_stack = (K_THREAD_STACK_BUFFER(z_main_stack) +
			    K_THREAD_STACK_SIZEOF(z_main_stack));

Though not sure if anyone is using the stack guards without multi-threading, I believe it can be configured and it would make sense, e.g. for a bootloader.

FWIW I made the suggested change

Hi @franc0is,
thank you for the change.
Currently you have different email in commit author (Francois Baldassari<EMAIL_ADDRESS>and in Signed-off-by (François Baldassari<EMAIL_ADDRESS>which is not allowed by Zephyr projects - so the CI is blocking PR merge. Could you please fix that? Thanks!
There is a bit more details about it: https://docs.zephyrproject.org/latest/contribute/guidelines.html#commit-message-guidelines

Thank you for the note about the commit author (new laptop...). I've fixed it.

I think the original proposed solution was correct (start = end - size). Confusion is about stack top and bottom vs start and end. In the check logic here, start should be the smaller address, and end the larger one.
My point earlier was that I think some alignment and optional reserved space should be taken into account:
 * It is NOT legal to take the sizeof(sym) and pass that to the stackSize
 * parameter of k_thread_create(), it may not be the same as the
 * 'size' parameter. Use K_THREAD_STACK_SIZEOF() instead.
 *
 * Some arches may round the size of the usable stack region up to satisfy
 * alignment constraints. K_THREAD_STACK_SIZEOF() will return the aligned
 * size.

Thank you for the suggestion @ruuddw. I've made the suggested change. Note that this PR is getting a bit more complicated as a result.

Hi @franc0is, I finally had time to look a bit deeper into this, and without multithreading the MPU stack guards are not configured/supported.
So the z_check_thread_stack_fail() function can simply return false in that case, as there is no stack guard region defined for the main function.

Hah! So back to the original. I sure learned a lot in this process, thank you for your all your help!

Hi @ruuddw,

without multithreading the MPU stack guards are not configured/supported.

Are you sure about it? As for now we define main stack the same way when we don't have multi threading: https://github.com/zephyrproject-rtos/zephyr/blob/main/kernel/init.c#L53
We access the stack area via K_THREAD_STACK_BUFFER & K_THREAD_STACK_SIZEOF - so we take MPU guard area into account (if MPU_STACK_GUARD is enabled): https://github.com/zephyrproject-rtos/zephyr/blob/main/arch/arc/core/thread.c#L289-L290
It's done deep inside these macros with adding/removing of K_KERNEL_STACK_RESERVED: https://github.com/zephyrproject-rtos/zephyr/blob/main/include/zephyr/kernel/thread_stack.h#L288
And K_KERNEL_STACK_RESERVED is defined to ARCH_KERNEL_STACK_RESERVED
And ARCH_KERNEL_STACK_RESERVED is defined to Z_ARC_STACK_GUARD_SIZE
I don't see that MPU_STACK_GUARD depend from multi-threading. So I'd say it's supported even in single thread mode.

Yes, the guard area is defined, but the MPU doesn't get programmed to enforce it as that is done when a thread is initialized or swapped. At least, that is the conclusion I came to.

Yes, the guard area is defined, but the MPU doesn't get programmed to enforce it as that is done when a thread is initialized or swapped. At least, that is the conclusion I came to.

Yep, can confirm it.


About NavigationView conflicts with other libraries

Hello, when I use NavigationView 1.1.3 and https://github.com/guoyingtao/Mantis 2.21.0 at the same time, NavigationView will cause drag errors in Mantis
When NavigationView is not used：
https://github.com/user-attachments/assets/c1e6f753-5d4e-4e6a-a3a0-1f532e64df3b
After using NavigationView：
https://github.com/user-attachments/assets/5e823950-6e66-46af-998b-9747f62b8011
Code Sample：
import Mantis
import MijickNavigationView
import SwiftUI

@main
struct TestMantisDragApp: App {
    var body: some Scene {
        WindowGroup {
            ContentView()
                .implementNavigationView()
                .preferredColorScheme(.light)
        }
    }
}

struct ContentView: NavigatableView {
    @State var image: UIImage? = UIImage(named: "EnglishImage")

    var body: some View {
        ImageCropper(image: $image)
            .ignoresSafeArea()
    }
}

#Preview {
    ContentView()
}

struct ImageCropper: UIViewControllerRepresentable {
    @Binding var image: UIImage?

    @Environment(\.presentationMode) var presentationMode

    class Coordinator: CropViewControllerDelegate {
        var parent: ImageCropper

        init(_ parent: ImageCropper) {
            self.parent = parent
        }

        func cropViewControllerDidCrop(_ cropViewController: Mantis.CropViewController, cropped: UIImage, transformation: Transformation, cropInfo: CropInfo) {
            parent.image = cropped
            print("transformation is \(transformation)")
            parent.presentationMode.wrappedValue.dismiss()
        }

        func cropViewControllerDidCancel(_ cropViewController: Mantis.CropViewController, original: UIImage) {
            parent.presentationMode.wrappedValue.dismiss()
        }
    }

    func makeCoordinator() -> Coordinator {
        Coordinator(self)
    }

    func makeUIViewController(context: Context) -> UIViewController {
        makeCustom(context: context)
    }

    func updateUIViewController(_ uiViewController: UIViewController, context: Context) {}
}

extension ImageCropper {
    func makeCustom(context: Context) -> UIViewController {
        var config = Mantis.Config()
        config.cropToolbarConfig.mode = .normal
        let cropViewController = Mantis.cropViewController(image: image!, config: config)
        return cropViewController
    }
}

Test image:

Steps to reproduce:
When the line “.implementNavigationView()” is commented, it works fine. When it is added, the problem occurs
This problem does not appear when previewing with #Preview. I only discovered it when I was packaging the simulator or  debugging on the real device. I am very confused now

Hey @ByLiangCheng,
I'll check it at the beginning of next week as I'm literally swamped with work this week
Sorry for this and have a good day!

Thank you, don't worry, I found another solution. Now there is no problem in presenting the view corresponding to the Mantis library through fullScreenCover


Weave bridge not exposed after node reboot

This seems to be a regression in a 0.15.0 release candidates.

Related: https://github.com/weaveworks/weave/issues/2479


Allow to edit a bank custom fields

For ING and HSBC for example, I cannot modify the custom fields (secret question, birthday, etc.) once the account is created.

Thanks for the issue!

I am working on it.


Missing underscore in package.json dependencies

module.js:338
throw err;
^
Error: Cannot find module 'underscore'
at Function.Module._resolveFilename (module.js:336:15)
at Function.Module._load (module.js:278:25)
at Module.require (module.js:365:17)
at require (module.js:384:17)
at Object. (/Users/joaoribeiro/Documents/Projects/cloudtasks.io/node_modules/sitemap/lib/utils.js:7:9)
at Module._compile (module.js:460:26)
at Object.Module._extensions..js (module.js:478:10)
at Module.load (module.js:355:32)
at Function.Module._load (module.js:310:12)
at Module.require (module.js:365:17)
at require (module.js:384:17)

Hi @JonnyBGod Thanks for report.
Yep, seems forgot to add that dependence into bower.json.

Not just that. You also have to change the package.json as you have the underscore dependency in "devDependencies" while this should be in "dependencies".
This make it work when you do node install --production


MGDOBR-937: Verify new Google PubSub source processor

JIRA: https://issues.redhat.com/browse/MGDOBR-937
Hopefully this is the last one!
Form generated from schema works fine. Needed changes are logged inside MGDOBR-901. For now it's just the Service Account Key missing the proper password definition.

One observation: there is possible to set negative integers which is nonsense but I assume that it depends on the source definition and UI need to support full interger when the definition defines it.

Agree, that should be defined in the schema. PRs like this one are mainly intended to do a formal check about all the schemas that were introduced over time, verifying that the form is generated correctly from the json-schema definition.
We'll take care of improvements/changes to the schemas with MGDOBR-901.
I've added a note there proposing to add extra validation for positive integers on "Max Messages Per Poll" and "Concurrent Consumers"


Server crashes if init request has empty client capabilities

See https://github.com/OmniSharp/omnisharp-roslyn/issues/1499

This should be fixed via #147


Posts from Bluesky don't get delivered to the fediverse until they're boosted/reposted by a opted-in Bluesky account

Example: https://bsky.app/profile/makai.chaotic.ninja/post/3lccx2f7dss2k
I can't manually look it up (whether it's via Mastodon, Misskey, or Pleroma) either via https://fed.brid.gy/r/https://bsky.app/profile/makai.chaotic.ninja/post/3lccx2f7dss2k. I have to repost it just to see it in my Misskey's home feed.

Looking at a Misskey inbox delivery, we posted the Create activity below for this post to https://makai.chaotic.ninja/inbox at 2024-12-02 10:45:55 UTC, and it returned HTTP 202.
{
  "@context": ["https://www.w3.org/ns/activitystreams"],
  "type": "Create",
  "id": "https://bsky.brid.gy/convert/ap/at://did:plc:cu33jkvtkoxyfd45wg7hrhra/app.bsky.feed.post/3lccx2f7dss2k#bridgy-fed-create",
  "actor": "https://bsky.brid.gy/ap/did:plc:cu33jkvtkoxyfd45wg7hrhra",
  "published": "2024-12-02T10:45:54.633923+00:00",
  "object": {
    "type": "Note",
    "id": "https://bsky.brid.gy/convert/ap/at://did:plc:cu33jkvtkoxyfd45wg7hrhra/app.bsky.feed.post/3lccx2f7dss2k",
    "url": "https://fed.brid.gy/r/https://bsky.app/profile/did:plc:cu33jkvtkoxyfd45wg7hrhra/post/3lccx2f7dss2k",
    "attributedTo": "https://bsky.brid.gy/ap/did:plc:cu33jkvtkoxyfd45wg7hrhra",
    "content": "
Test post for #BridgyFed GitHub issue, please don't repost (except if you're @snarfed.org)
",
    "contentMap": {
      "en": "Test post for #BridgyFed GitHub issue, please don't repost (except if you're @snarfed.org)"
    },
    "published": "2024-12-02T10:45:53.503Z",
    "tag": [
      {
        "type": "Hashtag",
        "name": "#BridgyFed",
        "href": "https://bsky.brid.gy/hashtag/BridgyFed"
      },
      {
        "type": "Mention",
        "name"<EMAIL_ADDRESS>        "href": "https://bsky.app/profile/snarfed.org"
      }
    ],
    "content_is_html": true,
    "to": ["https://www.w3.org/ns/activitystreams#Public"]
    "cc": ["https://bsky.app/profile/snarfed.org"],
  },
  "to": ["https://www.w3.org/ns/activitystreams#Public"],
  "cc": ["https://bsky.app/profile/snarfed.org"]
}

Where is the content_is_html from? That's not defined in the Activity Streams vocabulary.

Hah, good eye. It's Bridgy Fed internal, very non-standard, not really intended for external consumption. Ideally we should remove it before sending activities.

I think that would be a good idea. It's a validation error in "safe mode" JSON-LD processing, so it could well cause issues like this.
That aside, the code you pasted isn't valid JSON either. There's a misplaced comma between lines 31 and 32.
Maybe that's the issue? (I didn't notice at first either, but https://json-ld.org/playground/ flagged it.)

Ah, sorry, that's just me, I rearrange and clean up JSON activities a bit before I add them to issues here. Will fix.

I still can't do the manual look up (and for Mastodon I test from yamabi.co and mastodon.social), however I did another test post in my bsky and I can see it delivered now to the fediverse (both Mastodon and Misskey) without any need for reposts. Seems like the fix worked!

Another Bluesky example, this time a reply to another opted-in account (and boosting doesn't seem to be working?): https://bsky.app/profile/makai.chaotic.ninja/post/3ldhsucuy722u

Not sure if it's this issue but my friend's account miseru.bsky.social stopped bridging about 4 weeks ago. I thought it might be because of "Discourage apps from showing my account to logged-out users" but https://github.com/snarfed/bridgy-fed/issues/974 suggests it would have continued bridging?

@temtemy sorry for any confusion in https://github.com/snarfed/bridgy-fed/issues/1584#issuecomment-2547689257! Sounds like it turned out to be mainly Misskey interop?
@ygg2 looking at https://fed.brid.gy/bsky/miseru.bsky.social , we're seeing their recent posts, so I'm guessing they got hit by #1595 😢

@temtemy sorry for any confusion in #1584 (comment)! Sounds like it turned out to be mainly Misskey interop?

@snarfed Yes it seems to be just Misskey interop this time, sorry for not making it clear.

@temtemy np! Feel free to reopen if you have any details on what Misskey specifically is having trouble with.

I believe this corresponds to https://github.com/misskey-dev/misskey/issues/15039. Misskey started to reject when URLs have different hosts, and thus having https://fed.brid.gy/ and https://bsky.brid.gy/ together causes rejection.

THank you @saschanaz! And ugh. I get requiring that activity, object, actor, and attributedTo ids are all on the same domain, but url isn't really relevant to authorization or other security contexts in AP, this new Misskey requirement is unfortunate and doesn't make a lot of sense to me.

Will follow up in #1570


sub menu doesn't show localized items

I've created custom flat menu using Replacing both the FlatMenu and FlatMenuItem models
The problem is, that if menu has submenus, items there from link_page, and not from link_page_fr or so. Same is for link_text.
As a result, I have:
home_en
- page_en_one
- page_en_two

in another language I have:
home_fr
- page_en_one
- page_en_two

but I need:
home_fr
- page_fr_one
- page_fr_two

class TranslatedPageField(object):
    def __init__(self, en_field, fr_field):
        self.en_field = en_field
        self.fr_field = fr_field

    def __get__(self, instance, owner):
        active_language = translation.get_language()
        if active_language == 'fr':
            return getattr(instance, self.fr_field)
        return getattr(instance, self.en_field)

class TranslatedFlatMenuItem(AbstractFlatMenuItem):
    menu = ParentalKey(
        TranslatedFlatMenu, # we can use the model from above
        on_delete=models.CASCADE,
        related_name=settings.FLAT_MENU_ITEMS_RELATED_NAME,
    )
    link_page_fr = models.ForeignKey(
        Page,
        verbose_name=_('Link to an internal page (Frenc)'),
        blank=True,
        null=True,
        on_delete=models.CASCADE,
        related_name='link_page_ru',
    )
    link_url_fr = models.CharField(
        verbose_name=_('Link to a custom URL (French)'),
        max_length=255,
        blank=True,
        null=True,
    )
    link_text_ru = models.CharField(
        verbose_name=_("Link text (Russian)"),
        max_length=255,
        blank=True,
    )
    translated_link_text = TranslatedField('link_text', 'link_text_fr')
    translated_link_url = TranslatedField('link_url', 'link_url_fr')
    translated_link_page = TranslatedPageField('link_page', 'link_page_fr')

    def translated_page(self):
        return self.translated_link_page

    @property
    def menu_text(self):
        """Use `translated_link_text` instead of just `link_text`"""
        return self.translated_link_text or getattr(
            self.translated_link_page,
            settings.PAGE_FIELD_FOR_MENU_ITEM_TEXT,
            self.translated_link_page.title
        )

    def relative_url(self, site=None, request=None):
        if self.translated_link_page:
            try:
                page_url = self.translated_link_page.get_url(
                    request=request, current_site=site)
                return page_url + self.url_append
            except TypeError:
                return ''
        elif self.translated_link_url:
            return self.translated_link_url + self.url_append
        return self.link_url + self.url_append

Any suggestions of what kind of information I need to add? Have I missed something or did wrong?

Good day. Maybe this will help you:  #242 .

Good day. Maybe this will help you: #242 .

@noparamos, yes, I've read that issue and some solutions, but no success with AbstractFlatMenuItem. Adding localized do nothing. It shows submenu for page in link_page but when I active another language and want to get submenu for link_page_fr I got nothing.

@AlexDeltax you will want to look at (and override) the methods on the menu model that gather the pages needed for generation, like the developer in the linked PR is doing.
It looks like you're using a completely different approach to translation than they are, so the code will not work like-for-like - you will need to figure out how to use your menu model's language-specific foreign key values.

Am closing this issue, as it isn't a bug with this software - more a support request


Comment if a newly submitted issue doesn't have any labels

Useful for community-created issues, but also for committers who forget to label issues.
CC #11

There should be some delay, to give the maintainers time to find the issue and label it, right?  Non-maintainers can't label things.


“Invalid_request” error when sending an device access token request with basic authorization header

Describe the issue:
Gives “Invalid_request” error when sending device access token request with basic authorization header instead of client_id as a body param in device auth code flow. The client_id param seems to be a mandatory body param for the token request even when the Authorization header is used.

Related issue: https://github.com/wso2/product-is/issues/8166


New setPayload flag to turn off automatic claims addition

Addresses #19.
When providing the payload via the constructor the iat property wasn't being added automatically so the payload would be inconsistent in that case so I adjusted that as well. Switched the other properties to use their setters for consistency.
Switching the constructor revealed a couple of issues in the JWTTests so those are fixed and improved a little to hopefully prevent regressions. The test was using the same array structure for both header and payload so the base64 representation was coming out the same, but the order of the concatenation of strings was wrong. So I made it so that they had different array values and fixed the assertion.
Unit tests included for the payload changes, though I still had unrelated breakage for other existing tests. See Namshi\JOSE\Test\Signer\ES256Test::testSigningWorksProperly (and for ES384 and ES512).

:+1:

in 2.1.0

Thanks!

After testing, this led to a regression in verification because of JWS::load not having the same flag, so the auto assignment of iat being the default means that the payload gets it even when read from the signature. This is related to a bigger change that I was going to propose and that has to do with how the header is constructed for the verification object in load anyways as the current setup prevents the user from providing their own non-recognized header values. AFAICT from the specs a user should be able to pass through any value that would be ignored by a system that doesn't know how to handle it. My attempts to do so were thwarted by the auto constructed array for the header.
I'll get another PR submitted, sorry for the inconvenience should have thought to check the test suite for a load test when I saw they were sparse.

@odino Additionnal tests should be a good idea for each new PR.

There were tests around the specific changes in the PR, but the lack of other coverage meant the regression didn't get caught.

Ok, that is what I just saw.
Unit tests using PHPUnit is a good idea, but IMO it is not enough (even if all the code is covered). This PR is a good example: tests added, all passed but there is some new issues.
I think I will add some functionnal tests soon using Behat.

I dont think the library needs functional tests though :) I am quite in favour to move over to specs so if you want to look into that it would be awesome!..but functionally testing a library that only provides units is a bit weird to me.
To be honest I think regressions might happen even with more tests -- it's a matter of double checking things. My bad for overlooking this :) Im gonna revert and publish a new tag now

btw reverted and re-tagged


Some code snippets are empty

Some code snippets are empty in the Add the final polish paragraph

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

ID: 6e3fc905-7aa1-7f5e-9bfe-fbcef461ad5e
Version Independent ID: a286e20b-1e43-dc57-e0fb-5a9dee7ec3f6
Content: Tutorial: Write your first analyzer and code fix
Content Source: docs/csharp/roslyn-sdk/tutorials/how-to-write-csharp-analyzer-code-fix.md
Product: dotnet-csharp
Technology: csharp-roslyn
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks @SaSha-K1. I'll fix that.
@gewarren Do you know why there were no build warnings?

@gewarren Do you know why there were no build warnings?

No, that is strange. I'll investigate and open a bug if necessary.

I logged internal bug https://dev.azure.com/ceapex/Engineering/_workitems/edit/397329.


Breaking change in DataRenderer

Version and Platform (required):

Binary Ninja Version: 4.2.6339-dev (d021f9d4)
OS: macOS
OS Version: current
CPU Architecture: Apple SIlicon

Bug Description:
As I just updated BN to 4.2.6339-dev (d021f9d4), my  DataRenderer for a custom named type started to report an exception for each data variable with the corresponding type in the console:
[ScriptingProvider] Exception ignored on calling ctypes callback function: <bound method DataRenderer._get_lines_for_data of <subjective.cfstring.CFStringRenderer object at 0x14f984260>>
[ScriptingProvider] TypeError: DataRenderer._get_lines_for_data() takes 11 positional arguments but 12 were given

Steps To Reproduce:
Register the following basic DataRenderer POC using a named type that suits your test file.
Here, I am expecting an Obj-C Mach-O file like /System/Applications/Contacts.app/Contents/MacOS/Contacts where we override the default rendering for __NSConstantString, i.e. the named type for the CFString structure.
During auto-analysis, you can already paste this in the Python console:
name = '__NSConstantString'  # change me if you don't want to open an Objective-C Mach-O file!

class ReproRenderer(DataRenderer):
    def __init__(self):
        DataRenderer.__init__(self)
    def perform_is_valid_for_data(self, ctxt, view, addr, type, context):
        return DataRenderer.is_type_of_struct_name(type, name, context)
    def perform_get_lines_for_data(self, ctxt, view, addr, type, prefix, width, context):
        return [DisassemblyTextLine(prefix, addr)]

ReproRenderer().register_type_specific()

The go to the __cfstring section (g __cf) and watch exceptions screaming at me from your Log view!

Expected Behavior:
No exception!
Example with actual CFStringDataRenderer:

Fixed in build 4.2.6366


Allow specifying the javascript version in the debug html window

I would like to test javascript 1.8 as this is what I would like my target to run on (greasemonkey on firefox), but currently SCRIPT_TYPE is hard-coded
        return util.format(SCRIPT_TAG, SCRIPT_TYPE[fileExt] || 'text/javascript', filePath)

without application/javascript;version=1.8 firefox runs ES5 only, so can there be some option to put this in the config?

@simonzack This is possible to do. However, in case the user uses multiple browsers, this might go wrong. If, in any case Chrome is used to run besides Firefox 4 (which supports 1.8 now), the tag will make the tests all fail. Even more, Karma will throw an Error.
I was able to make the configuration for you and was wanting to add a PR for it, but I stumbled upon the error part of Karma when trying to add the tag when running Chrome. Maybe @dignifiedquire has an idea how to make this fail-safe?

Not sure (I never had to use this in a real world example before)
But we just merged custom context and debug files in #1825  so maybe that can help you to specify exactly what you want.

I have seen that, but it is not exactly what I'm looking for. The case would be that Firefox 4 supports 1.8, but Chrome (even latest version) does not. This makes it that in the karma.js-file, when the Firefox 4 browser is run, it should add the tag of ";version=1.8", otherwise not. I'd think that there should be some check which browser is being run at that moment and perhaps the version of it.
This might be a solution to it, but takes the effort to update this whitelist of browsers to be updated everytime a new update comes out...

@dignifiedquire: Do you know a possible way to capture the current browser being run? That would solve the issue mainly.

@Moumi capture it where?

@dignifiedquire: At https://github.com/karma-runner/karma/blob/master/lib/middleware/karma.js just before the return statement:
return util.format(SCRIPT_TAG, SCRIPT_TYPE[fileExt] || 'text/javascript', filePath)

@dignifiedquire ping

I don't think that will easily work as the file is simply served to every browser the same atm. You could check the express docs how to do browser detection in a single request

@dignifiedquire I managed to get the full functionality working. I wanted to create a PR, but I wanted to write a test or 2 for it too. However, it seems very difficult to test it. The check to add the tag depends on:

If the configuration is set
If the current browser is Firefox. Which is derived from a request and the user-agent header in it.

Well, how can you force during a test that the user-agent should say "Firefox"? If you could help me with figuring this out, that'd be great and this issue would be fixed (I hope). I am just learning about Sinon and it isn't that clear to me either.

@Moumi your best bet is to use an e2e test (see test/e2e) which can specify a browser type and does a full karma run and checks the output.

I managed to create a fix and two e2e tests for it. I will provide a PR this evening for it.