IOS: View is not in the window hierarchy

Hey, thank you for this library!
I'm running into an isssue when trying to use auth:
Attempt to present <UINavigationController: 0x7fc6458b7c00> on <Tiktok: 0x7fc64526fa80> (from <Tiktok: 0x7fc64526fa80>) whose view is not in the window hierarchy.

React Native: 0.68.2.
This is my AppDelegate.mm:
- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{

[[TikTokOpenSDKApplicationDelegate sharedInstance] application:application didFinishLaunchingWithOptions:launchOptions];
  RCTAppSetupPrepareApp(application);

  RCTBridge *bridge = [self.reactDelegate createBridgeWithDelegate:self launchOptions:launchOptions];

  UIView *rootView = [self.reactDelegate createRootViewWithBridge:bridge moduleName:@"main" initialProperties:nil];

  rootView.backgroundColor = [UIColor whiteColor];
  self.window = [[UIWindow alloc] initWithFrame:[UIScreen mainScreen].bounds];
  UIViewController *rootViewController = [self.reactDelegate createRootViewController];
  rootViewController.view = rootView;
  self.window.rootViewController = rootViewController;
  [self.window makeKeyAndVisible];
   return YES;
}

I've tried forcing the TikTokOpenSDK version to be 5.0.0 but that didn't help.
Do you have any ideas for what it could be?

@dscottpi
unfortunately, I didn't test it on version 0.68+

+1 I have exactly the same issue

Has somebody solved this issue?
I'm having the same issue with react native 0.69. Although it's working just fine with some tweaks on android.

On IOS Simulator I get this error but when I build and use on my phone, it opens up TikTok app. So the issue is if you don't have Tiktok installed on your device, then it wants to open a modal which causes this issue. I tried debugging for the past hours and it all leads to TikTok.swift file, I don't know much about Swift so I am kind of stuck here. But I thought of at least help by leading to the issue.

Anything new on this? It works completely fine on my iphone physical device. But that's because I have Tiktok app installed and it isn't trying to open a web view where the user can login to tiktok. So without the Tiktok app installed on your device, you can't use the Tiktok login functionality and you get this error as the title says.

We have solved it with our own logic. We are checking if TikTok is installed and:

If it is, then we are using react-native-TikTok
if it's not, then we are redirecting the user to the Safari browser, where the user can log in. After successful login, TikTok calls redirect Url, which is our API, and this API requests the user to open our app.

Thank you for this!

Hello everyone
I am still facing the issue.
Tried on a real physical iPhone in which Tiktok is installed, but still has an error.
-canOpenURL: failed for URL: “tiktokopensdk://” - error: “The operation couldn’t be completed. (OSStatus error -10814.)”
[Presentation] Attempt to present <UINavigationController: 0x107009a00> on <Tiktok: 0x109f051d0> (from <Tiktok: 0x109f051d0>) whose view is not in the window hierarchy.

Also, the same error happens on iOS simulator.
Much appreciate it if there is someone who can solve clearly for this.
Thanks


cleanup selected when removing a device

@ariandora If this fixes an issue, can you reference the issue in the commit. This way it'll be automatically closed.


Can't run project

Hello,
I can't run project locally. In console there are errors:

Could somebody help?
Thanks

I've also discovered some issues with SAPUI5 1.101. Please try with 1.100.

You need to replace the occureces of https://ui5.sap.com/ in app/index.html with https://sapui5.hana.ondemand.com/1.100.2/

@gregorwolf thanks, works now


Should we not log with a timestamp

oc logs has timestamp functionality which definitely looks nicer than what we have in the scripts, should we strip the date -u commands from our scripts?
Example:
# oc logs --timestamps -f -n scale-ci-tooling job/scale-ci-nodevertical
2019-06-21T19:50:53.813339525Z Fri Jun 21 19:50:53 UTC 2019 Configuring pbench for Node Vertical
2019-06-21T19:50:53.818275115Z Fri Jun 21 19:50:53 UTC 2019 Done configuring pbench Node Vertical
2019-06-21T19:50:53.819025782Z Fri Jun 21 19:50:53 UTC 2019 Running Node Vertical
2019-06-21T19:50:55.003477457Z Running VIPERCONFIG=/root/workload/nodevertical.yaml openshift-tests run-test "[Feature:Performance][Serial][Slow] Load cluster should load the cluster [Suite:openshift]"
2019-06-21T19:50:56.631537971Z Jun 21 19:50:56.631: INFO: >>> kubeConfig: /root/.kube/config
2019-06-21T19:50:56.633074576Z Jun 21 19:50:56.633: INFO: Waiting up to 30m0s for all (but 100) nodes to be schedulable
...

@jmencak @chaitanyaenr Any thoughts on this one?

Any thoughts?  If not I am going to close it.

I never used --timestamps but I like it.  I noticed, however, they will mis-align columns when the fraction of a second (behind .) ends with 0, so I'd probably leave things as they are for now.

After some careful thought and experience I concur with @jmencak idea of keeping it.


CI improvements

General improvements to the CI scripts and addition of oneapi+gpu_intel checks

Looks OK!
If you're done with this, please use squash-merge. We don't need the many intermediate history of the testing commits.
Thanks!


Progress callback not passed when using the Velocity UI pack.

When passing a progress callback function using a Velocity UI pack animation, the function is not triggered.
http://codepen.io/anon/pen/alwtz
The above example is passing the complete and progress callback functions, but only complete is called.

You should mention this in the progress section also :)
http://julian.com/research/velocity/#progress


test: use non-deprecated gomock package & fix unit tests

the github.com/golang/mock package has been archived with the suggestion to use the forked version here. this pr replaces the now abandoned repo in favor of the maintained fork and regenerates the mocks.
d72e83c94350433a1f194dbfaff97980b3a274a7 also introduced some breakage in the unit tests which i've fixed.

cc @jtieri this is actually approved now with me being added to the IBCApps team

cc @jtieri this is actually approved now with me being added to the IBCApps team

thanks! just getting back around to this, i'll add that line to the Makefile target for generating the mocks then get this merged.


Algorithm wish list

Methods that we haven't implemented yet, but would like to. In no particular order:

[ ] Probablistic Global Distance Metric Learning

Probabilistic extension of MMC (pMMC?)

[ ] Discriminative Component Analysis (DCA)

reference implementation

[ ] Localized Distance Metric Learning (LDM)

reference implementation

[ ] DistBoost and KernelBoost

reference implementation

[ ] Active Distance Metric Learning

reference implementation

[ ] Large Scale Metric Learning from Equivalence Constraints (KISSME)

website (includes reference implementation)
python implementation (unverified)

[ ] Logistic Discriminant Metric Learning (LDML)

reference implementation

[x] Metric Learning for Kernel Regression (MLKR)

reference implementation
added in PR #28

[x] Mahalanobis Metric for Clustering (MMC)

reference implementation
added in PR #61

Hello, any tips or further documentation on how to go about doing this? And which one of these would be highest priority?

I don't personally have a priority, but if you want to rank them you could look at their citation counts on Google Scholar. That should give you an idea of what's more popular in the literature. I would start with one that seems easy to implement, though.
Each method should have its own Python file in the metric_learn directory, which defines a class that inherits from BaseMetricLearner. This subclass should implement a .fit() method, as well as one of the .metric() or .transformer() methods. As a really simple example, you can start with the Covariance class.

Once you have something that works, add a class to test/metric_learn_test.py which exercises the new algorithm. You should also verify that your implementation matches the reference code, if possible.

Thanks for all the details! The paper by Xing, et al is I think the most popular (and one of the first in Metric Learning) algorithm/paper among the list, I'll give it a shot.

Hey @perimosocordiae can I take up the MLKR implementation? I was thinking of using the transformer() method to use MLKR for dimensionality reduction and hence return X_new as the output. Also, I was thinking of having the A matrix stored as an attribute after the fit() operation thus enabling the user to access it independently also if needed. Does this seem fine to you?

@dsquareindia Sure, MKLR is a good choice. The transformer() method should return the matrix A (as defined the paper), and the transform() method will take care of the dimension reduction without needing to be overridden. In this way, the base class metric() method will also work as intended (corresponding to M in the paper).

KISSME metric learning would be a nice addition (no optimization).
You can find a reference Matlab implementation and the paper here: https://lrs.icg.tugraz.at/research/kissme/
There is also a python implementation (which I have not verified) here: https://github.com/Cysu/dgd_person_reid/blob/master/eval/metric_learning.py

@aschuman thanks, I've added it to the list.

LDML

hi @perimosocordiae
I would like to take up Probabilistic Global Distance Metric Learning.
thanks :)

@anirudt Thanks, added.
@RishabGoel Go for it! I look forward to your pull request.

Hello, just wanted to know where the short form PGDM came up - I can't seem to find it in the paper linked (which is the paper just called "Distance metric learning, with application to clustering with side-information"), by Xing et al.

@bhargavvader  Probably the wrong link. The metric is a non probabilistic one in the link. But find the PRobabilistic one in the following link:
http://www.cs.cmu.edu/~liuy/frame_survey_v2.pdf
I am bit busy right now so I am not able to do much, but will submit a PR soon.
Hope this helps!
Regards

@bhargavvader, @RishabGoel: yep, that's correct. I pulled the citation from the survey paper without looking too closely at it. I'll fix the link now.

The method by Xing et al. (2003) is also sometimes referred to as "MMC", e.g., in the LMNN paper by Weinberger et al. I guess, "MM" stands for "Mahalanobis Metric Learning" and "C" for "Clustering".

@perimosocordiae I would like to take up DistBoost and KernelBoost

@souravsingh Great! Feel free to open a PR whenever you'd like to get feedback. I just noticed the paper links for those methods are broken, so if you find working links to the papers let me know.

@perimosocordiae I think these are the links to the papers-
DistBoost- http://www.cs.huji.ac.il/~daphna/papers/distboost-icml.pdf
KernelBoost- http://leibniz.cs.huji.ac.il/tr/919.pdf

@souravsingh thanks, I've updated the list with your links.

Another suggestion that we could: the triplet-based method proposed in the paper below (heavily cited, very useful for information retrieval applications):
https://www.cs.cornell.edu/people/tj/publications/schultz_joachims_03a.pdf
Given that we currently do not have any triplet-based weakly supervised approach I think we could make this a priority. The implementation should not be very difficult

How about add the extension to the existed LSML algorithm, in the original paper and code, there is the extension of the vanilla algorithm called spLSML, which seems to be is just to add a L1 norm of the learned Mahalanobis matrix in the loss function, thus with a novel gradient update method, it should be quick to implement.


Update instructions

The instructions under the heading  "Added Tools Don't Show" are incorrect.
Existing:
Added tools don't show
Make sure you have the right project selected. In the Azure Migrate hub > Servers or in Databases, click on Change next to Migrate project (Change) in the top-right corner of the screen. Choose the correct subscription and project name > OK. The page should refresh with the added tools of the selected project.
Revised
Make sure you have the right project selected.  From the Azure Migrate hub, select 'Servers, databases and web apps', then in top left click on the blue highlight project name and use the pull-down to select the correct subscription and project name.  The page will then refresh with the selected project and associated tools.

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

ID: 37d76345-6904-0553-3bc2-d01b48de9e69
Version Independent ID: ee6c40e8-ae48-9ed1-81d4-cbbc6e94a5c3
Content: Troubleshoot Azure Migrate projects - Azure Migrate
Content Source: articles/migrate/troubleshoot-project.md
Service: azure-migrate
GitHub Login: @MaryMichael-MS
Microsoft Alias: v-michaelar

@MGodfrey50
Thanks for your feedback! We will investigate and update as appropriate.

@MGodfrey50 thank you for the feedback.
@MaryMichael-MS, could you please review this feedback and share your thoughts. Thank you

Hi @MGodfrey50,
Thank you for your suggestion. The document has been updated. Hence, closing the issue.
#please-close


undefined is not a function - RNN

I am trying to get this package to work and I am getting the error below.
I basically cloned the wix react native navigation example project and added this package to it.
Carefully followed the install instructions listed here:
https://github.com/wix/react-native-notifications/blob/master/docs/installation.md
and
https://github.com/wix/react-native-notifications/wiki/Android:-working-with-RNN
To reproduce,
clone this repo:
https://github.com/odesey/rn-native_navigation-native_notifications
npm install
react-native run-android
will generate this error:
TypeError: undefined is not a function (evaluating '_reactNativeNotifications.NotificationsAndroid.addEventListener('remoteNotificationsRegistered', _this.onPushRegistered.bind(_this))')

This error is located at:
    in NavigationTypes (at Navigation.js:51)
    in _class (at renderApplication.js:35)
    in RCTView (at View.js:112)
    in View (at AppContainer.js:102)
    in RCTView (at View.js:112)
    in View (at AppContainer.js:122)
    in AppContainer (at renderApplication.js:34)
NavigationTypes
    NavigationTypes.js:12:42
<unknown>
    createClassProxy.js:98:23
instantiate
    createClassProxy.js:96:6
constructClassInstance
    ReactNativeFiber-dev.js:1572:264
updateClassComponent
    ReactNativeFiber-dev.js:1657:143
beginWork
    ReactNativeFiber-dev.js:1786:44
performUnitOfWork
    ReactNativeFiber-dev.js:2528:33
workLoop
    ReactNativeFiber-dev.js:2554:141
_invokeGuardedCallback
    ReactNativeFiber-dev.js:73:23
invokeGuardedCallback
    ReactNativeFiber-dev.js:47:40
performWork
    ReactNativeFiber-dev.js:2593:41
scheduleUpdateImpl
    ReactNativeFiber-dev.js:2728:101
scheduleUpdate
    ReactNativeFiber-dev.js:2711:38
scheduleTopLevelUpdate
    ReactNativeFiber-dev.js:2992:91
updateContainer
    ReactNativeFiber-dev.js:3003:39
render
    ReactNativeFiber-dev.js:3948:53
renderApplication
    renderApplication.js:34:4
run
    AppRegistry.js:120:10
runApplication
    AppRegistry.js:207:26
__callFunction
    MessageQueue.js:299:47
<unknown>
    MessageQueue.js:111:26
__guard
    MessageQueue.js:262:6
callFunctionReturnFlushedQueue
    MessageQueue.js:110:17

Any help would be greatly appreciated.
Thanks.
react-native-navigation: 1.1.334
react-native-notifications: 1.1.17
react: 16.2.0
react-native: 0.51.0

hi @odesey were you able to fix this error? appreciate it.

Unfortunately no.
I ended up using this package instead:
react-native-firebase
https://github.com/invertase/react-native-firebase

@odesey . thanks for the response.

@odesey you tried on a device right?

Physical and emulator.


Camel 15349

Updated the XmppConsumer to "chatWith" the Participant instead of the ChatId. This solves an error when configuring a route to consume from a participant with a domain id and without a room defined the call to 'JidCreate.entityBareFrom(endpoint.getChatId())' would throw and exception because of the ill-formed JID returned by getChatId.
Added a function to XmppEndpoint to resolved the Fully qualified JID of the configured participant. This function when used by the XmppConsumer for direct chats will allow route participants to be defined without their domain and assume the domain of the server being connected too. This solves an error when configuring a route to consume from a participant without a domain id and without a room defined the call to 'JidCreate.entityBareFrom(endpoint.getChatId())' would throw and exception because of the ill-formed JID returned by getChatId.

Thanks for the PR. As @oscerd says then the code formatting need a little which you can run with
mvn compile -P sourcecheck

in the camel-xmpp folder. Then fix those errors reported and push to this PR

Sadly the gh action for backporting doesn't work anymore. Need to dig a bit. Let me do this manually.

Thanks for the PR


VSIXPackage for redhat.java 1.2.0 is actually 1.1.0

Describe the bug
The VSIXPackage for redhat.java 1.2.0, found on https://redhat.gallery.vsassets.io/_apis/public/gallery/publisher/redhat/extension/java/1.2.0/assetbyname/Microsoft.VisualStudio.Services.VSIXPackage - is actually 1.1.0.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Could you file the issue with Java extension team please? https://github.com/redhat-developer/vscode-java

Starting with 1.2.0, redhat.java only published platform-specific versions, and the Marketplace page correctly displays these versions. Also, we were told not to publish a generic version if a platform-specific version was released.

It looks like using the API to request 1.2.0 without specifying the target platform, it just returns the latest available generic version (e.g. 1.1.0). This seems a bug in the marketplace API.
@prashantvc Could you pls reopen the issue and take a look on your side? Also could you pls clarify what action, if any, should be taken on the extension publisher side?
// cc: @isidorn

If I replace "1.2.0" in the URL with "latest", it does grab the vsix I want. Does that track with your statements above?

I'm curious why you need to use API to grab the vsix, would you mind sharing your use case? @justinkb

I ran into this while using NixOS. Because of its design, you need to
declare the extensions you want to use.
It then grabs the file, per
https://github.com/NixOS/nixpkgs/blob/master/pkgs/misc/vscode-extensions/vscode-utils.nix
which references
https://github.com/NixOS/nixpkgs/blob/master/pkgs/misc/vscode-extensions/mktplcExtRefToFetchArgs.nix
and that's ultimately where the URL it's grabbing is defined.
I've switched to the vscode-fhs (sets up a virtual FHS environment and
chroots into it) since, so this is no longer an issue for me. I just
figured I'd report the odd thing I noticed.
On Tue, Jan 4, 2022, 8:07 AM Jinbo Wang @.***> wrote:

I'm curious why you need to use API to grab the vsix, would you mind
sharing your use case? @justinkb https://github.com/justinkb
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vsmarketplace/issues/288#issuecomment-1004571884,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABVHKDIRPCHXULGAYAHWOTUUKMBPANCNFSM5LEBZEYA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@justinkb thanks for sharing the use case. I'll wait for the VS Code Marketplace team to clarify the issue first, then provide fixes if necessary.

Let me reopen and assign to @prashantvc so he can comment.

@justinkb We expect the API to be called with appropriate targetPlatform that the version supports. The VSCode client takes care of this.
Falling back to the latest generic version is intentional, this was done to support older VSCode clients which doesn't know anything about targetPlatform and doesn't support platform specific extensions.
Also can you add some details about the scenario where you are using the API directly ?

I've detailed how I ran into this in my previous comment in this thread.

Closing this issue, it works as designed (https://github.com/microsoft/vsmarketplace/issues/288#issuecomment-1006279371)


Build of the workspace failed

Following the instructions in the README, I've encountered this error when building the workspace:
In file included from /root/catkin_ws/src/go2_robot/go2_driver/src/go2_driver/go2_driver.cpp:31:
/root/catkin_ws/src/go2_robot/go2_driver/include/go2_driver/go2_driver.hpp:45:10: fatal error: nlohmann/json.hpp: No such file or directory
   45 | #include "nlohmann/json.hpp"
      |          ^~~~~~~~~~~~~~~~~~~
compilation terminated.

How can I solve this?

Hi, I was also experiencing this issue. I managed to get the WS to build by cloning the JSON repo here and copying the include/nlohmann folder into ~/ros2_ws/src/go2_robot/go2_driver/include/

Hi @tgodfrey0, thanks for the hint, everything builds fine now. Closing for the time being.


MongoDBCsv empty first column parsing fix

when printed a record with an empty first column with MongoDBCsv predefined format, the csv file cannot be then parsed with the same MongoDBCsv predefined format.
this is due to the conflict between 'escape' and 'quote' character branches in Lexer. They are both set to ' " ' for MongoDBCsv.
fix: in Lexer reordered 'escape' and 'quote' branches.

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

Project coverage is 97.89%. Comparing base (08b09ed) to head (df7c340).
Report is 79 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #412      +/-   ##
============================================
- Coverage     98.35%   97.89%   -0.46%
- Complexity      541      550       +9
============================================
  Files            10       10
  Lines          1152     1189      +37
  Branches        202      212      +10
============================================
+ Hits           1133     1164      +31
+ Misses            7        6       -1
- Partials         12       19       +7

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

TY @ikfid for your PR. Merged. Please note the changes I made to the test to convert it into a proper JUnit test; there is no need to handle exceptions "manually": https://github.com/apache/commons-csv/commit/70c9418e71c32a6a8248f9b2e62aad6b3892a485

@garydgregory , thanks !) the proper test is way better :)


7/27 Publish

@Taojunshen,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-autogen/feature.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/group.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/group/deployment.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/group/deployment/operation.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/lab/custom-image.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/policy.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/policy/assignment.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/policy/definition.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/provider.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/provider/operation.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/resource.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/resource/link.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/tag.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/vmss.pycliyml
:white_check_mark:Succeeded
View

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


Not work method .send()

From gitter chat:

Installed the new @adonisjs/mail provider, and for some sort of reason got Cannot find module 'Mail', then added to aliases Adonis/Addons/Mail, and again no luck, when I try to send something it drops Mail.send is not a function

In doc https://github.com/adonisjs/adonis-mail/blob/develop/instructions.md example use Mail.send, but it's not worked.
I use this for bypass problem:
const sender = await Mail.connection()
await sender.send('welcome', {
  user: user
}, (message) => {
<EMAIL_ADDRESS>  message.to(user.email, user.name)
  message.subject('Test, Hello world!')
})

Can u verify this on the latest version? It should work

Yep, wait

Well done, it's worked with last changes!


Global admin credential requirement -- ongoing?

Hello,
Can you clarify whether Global Administrator credentials are required after the initial setup phase? The documentation specifies that "These credentials are only used during the installation and are not used after the installation has completed. It is used to create the Azure AD Connector account used for synchronizing changes to Azure AD. The account also enables sync as a feature in Azure AD." However, I'm unclear whether we can Block Sign-in / remove the Global Administrator role from this user after AD Connect is set up. My read of the documentation is that the only account needed for ongoing synchronization is the Azure AD Connector account -- named "On-Premises Directory Synchronization Service Account" with the Directory Synchronization Accounts special role. Is this correct?
Thanks,
Kevin

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

ID: f2c5e335-4367-b1fe-2e14-311a6f2cf81d
Version Independent ID: a35eed80-eb37-e59c-07a4-43af4a985991
Content: Azure AD Connect: Accounts and permissions
Content Source: articles/active-directory/hybrid/reference-connect-accounts-permissions.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@kschach Thanks for your feedback! We will investigate and update as appropriate.

Hey @kschach in the doc it says :

Azure AD Global Admin credentials
These credentials are only used during the installation and are not used after the installation has completed. It is used to create the Azure AD Connector account used for synchronizing changes to Azure AD. The account also enables sync as a feature in Azure AD.

Hey @kschach you're correct you shouldn't need the global admin role after the initial setup and the only required item is the On-Premises Directory Synchronization Service Account.
It seems like you were able to infer this from the doc. Are you saying that you'd like a more clear sentence that says this?
thanks,

Frank Hu

Thanks, Frank. Yeah that would be great.
If the Global Admin role is no longer necessary once the On-Premises Directory Synchronization Service Account is created, perhaps a step could be added to the docs to specify this? Currently, we have a Global Admin called AD Sync that was specifically created for/by AD Connect and the On-Premises Directory Synchronization Service Account created by AD Connect. For security purposes, I'd like to disable the AD Sync Global Admin account if it's no longer required. I just wanted to confirm this before proceeding.

Note, that if anything goes wrong with the connector then you won't be able to re-run it with that account. It's better to just change the role to a less powerful role as totally removing the account could introduce issues if you ever need to perform a re-initialization.
I'll go ahead and put in that note, if you have anymore questions within the scope of this git issue please let me know as I will be closing out this git issue by end of day tomorrow.

What was the recommended role if not Global Admin? @kschach did you settle on a less powerful role?


Serialize xsd:integer to plain JavaScript int

Fixes https://github.com/rdfjs/serializer-jsonld/issues/4

Would like to see this merged :)

@angelo-v Sorry for the late feedback. I had already a look, but your PR is more complex than it looks.
First I would like to know why you need that feature. I tried with the JSON-LD Playground and it looks like jsonld.js is not converting literals with a integer datatype to a plain number. I expect you wanted to run the compact or frame function on the output of the serializer and the output of jsonld.js was not like expected. From my point of view, this is something which should be fixed in jsonld.js.
@dlongley Please correct me if I'm wrong about the behavior of jsonld.js. Did you have any reasons to not implement that feature?
Just a little bit of background: The idea of this serializer was to be simple, fast and output valid JSON-LD. The not yet ported package from RDF-Ext rdf-serializer-jsonld-ext should handle the use cases where more control over the generated JSON object is required.
I see why people want to have this feature in the rdfjs/serializer-jsonld package. Also keep in mind that for a use case like Data Cube, where there are 1000x of integer literals, the full check needs to be done quite often. If we add it, it should be possible to enable it with a flag in the options. I'm not sure yet about the best way to make it optional with minimal drawback on performance when it's disabled.

@bergos I want to build browser apps that are able to make use of plain JSON as much as possible
Something like
{
  "someValue": {
    "@value": "42",
    "@type": "http://www.w3.org/2001/XMLSchema#integer"
  }
}

is too complex to handle, at least more complex as necessary, since it could be as well like:
{
  "someValue": 42
}

The JSON-LD Playground converts the above json-ld (given a context of course) to something like this:
_:b0 <http://.../someValue> "42"^^<http://www.w3.org/2001/XMLSchema#integer> .

Parsing that triple back to JSON-LD I would expect to get the orginal form back.

@bergos,

I tried with the JSON-LD Playground and it looks like jsonld.js is not converting literals with a integer datatype to a plain number.
@dlongley Please correct me if I'm wrong about the behavior of jsonld.js. Did you have any reasons to not implement that feature?

There is no conversion to/from JSON native types in the compaction, expansion, flattening, and framing algorithms in the JSON-LD API spec. The only time JSON native type conversion is done is when converting from/to RDF. The playground doesn't have a feature to import from, say, N-Quads -- so you can't try this out on the playground, unfortunately. However, if you were to use jsonld.js to parse  N-Quads via its fromRDF call, then you could see output with JSON native types (e.g., JSON numbers).
So when @angelo-v says:

Parsing that triple back to JSON-LD I would expect to get the orginal form back.

(Where the triple uses a value of 42 with a datatype of xsd:integer), he is correct, it would be parsed back by jsonld.js to a native JSON number.

@angelo-v I fear after adding the integers, we have to add support for doubles, etc. At the end we are reimplementing jsonld.js. So I would prefer to not merge this PR. For more control of the output format/structure the serializer-jsonld-ext package is the right choice, which I cleaned up and ported to rdfjs this afternoon. It's using the fromRDF from jsonld.js and supports compact, flatten and frame in one step. It supports other features besides the one from jsonld.js like skipping the @graph property when possible. If we need even more control, we can add it to that package. That way we don't duplicate code from jsonld.js and keep this package small and simple. I hope this is OK for you. If it works for your use case, I will release it on npm.

I do not really care which package realizes the functionality as long as I have a way to produce the simple JSON-LD form with native number. Unfortunately I won't find time to test this in the near future I guess. But it's ok then if you close this.


“Copy link” from experiment removes the /palabrate from EPFL link

I don't think there is a builtin way to detect subroutes like palabrate, however the url looks like https://modemos.epfl.ch/palabrate/#/experimenter so we could take the whole url and split it manually at #

Yes, fixed in https://github.com/PAIR-code/llm-mediation-experiments/commit/4660580384df621f28fda28d9aa447a93845a70a


Fix issue related to PhilJay/MPAndroidChart#1121

Setting default skip to 1 might result in unexpected behaviour.

commit 088b10d actually want it to be 1

If it will be zero then it will cause an infinite loop. Should be closed

Should not cause infinite loop. You already fixed this: https://github.com/danielgindi/ios-charts/blob/master/Charts/Classes/Renderers/RadarChartRenderer.swift#L182
Currently it will (per default) skip 1 web line.

Oh right. This property was very confusing to begin with...

Yes it was :-)
I sure had my trouble with it :-D

Yeah there was a good reason why I wanted to change its behavior to not add/subtract the 1 on the property. There were many bugs and misunderstandings. I hope its stable now...


Doesn't work on some country

I'm Belgian and I tried to watch a video restricted for French viewers but the extension didn't find any proxy to bypass the restriction for the France.
So I found an app that proxy YouTube and I hosted it on a French server: https://github.com/dmhacker/dmhacker-youtube
Maybe we could discuss integrating servers like this into your extension?
And unfortunately this app doesn't give any choice to choose the quality of the video.
The URL of my proxy is https://fr.ytproxy.unixfox.eu and here is the URL of the creator of the app: https://dmhacker-youtube.herokuapp.com/ (hosted in US I think).
I think this issue will be the case for others countries other than France.
I can deploy others proxy in UK and Austria if you want to.
EDIT: I found another proxy app: https://github.com/licson0729/node-YouTubeStreamer and there is lot's of others here: https://github.com/search?o=desc&p=1&q=youtube+proxy&s=stars&type=Repositories&utf8=✓

@unixfox Hi, Emilien.
Thanks for opening this issue. Of course we can discuss integrating this into the extension. Do you allow me to use your proxy as another source to find mirrors? I'd be happy to handle the integration part without any problem.
It would be fantastic to have other proxy in UK and Austria as well. If you can provide that, we'd be very grateful.
About the choices of quality of video, this is a known issue (see #11) and we'll be working on it in the near future.
As there is a lot of proxy apps, I think the best way to handle this would be to gradually add the ones that we feel are the best suited for our project.
Thanks for showing interest in helping the project grow. You support is greatly appreciated.

Thank you for your reply!
If you don't mind I'll take the time this weekend to setup a bunch of others Youtube proxies properly with a separate domain, using cloudflare to hide the origin IP of the server and so on.
I'm doing all of this because I want to have the least possible link to me in case YouTube do lawsuits.
I let my instance online for testing and if you can don't push it to the production extension.

Thank you for your help.
Take as much time as you feel is needed to setup everything. Your support is very much appreciated on this project and I totally understand your concerns regarding lawsuits and everything. Again, feel free to take as much time as you want to do everything in your own terms. Whenever you feel you are getting close to finishing it, just let me know so I can start the integration part, ok? :)
Your instance online has just been pushed to production for a quick test in a real environment but if you want I can take it out at anytime. Just say the word and I'll pull it back.
I know this sounds repetitive, but thanks again for your help :)

You are welcome! I appreciate puting my free time into great projects.
I prefer if you remove my testing instance on the production extension.
I'll come back to you as soon as I've tested every software and deploying properly some proxies. I've a bunch of ideas to cover a lot's of countries to bypass the YouTube restriction.
PS: I'm a Firefox user and I'm very looking forward to this issue #32 :).

Thanks for being so willing to help. Unfortunately Google has removed the extension from the Chrome Web Store just now.
So I just wanted to tell you to not spend your time developing it unless I find a way to talk to them about this and maybe get it republished.
They said the extension did not comply with their policies. I'll see what I can do about it :(
Thanks!

I knew it, someone reported the extension.
I don't think Google will allow this extension, this goes against their rules.
You've got two options for Google Chrome users:

Publishing this extension on a separate website.
Redevelop the whole extension into an userscript for making compatible with Tampermonkey/Greasemonkey, this way Google can't circumvent your extension, this makes very easy to install and auto update.

I also don't think Google will allow it. Thanks for providing input on the options to move forward :)
I've managed to get the extension working already on Firefox, so, I should be publishing tomorrow. At least that way people can continue to install the extension in another browser.
About the options for Chrome, I'm thinking about developing both. What are your thoughts on it?

I'm happy to see that you got the extension working on Firefox, if you need a tester I'm here.
About the options for Chrome: I would say that maintening two types of extensions (extension and userscript) can be hard but it's up to you if you've enough free time to spend on these two options.
I also tell you about the first option that by installing manualy an extension the user can't have auto updates from the developper so he will have to manualy update the extension each time there is a new version. This issue will not be the case for the userscript if the user uses Tampermonkey/Greasemonkey because these extensions supports natively auto updating the userscript.

You know what, you're absolutely right. I had not considered the problem that the extension doesn't autoupdate on a manual install. And I feel like that's a pretty big deal.
So I think it's pretty clear then that we will be moving forward with the Tampermonkey/Greasemonkey solution coupled with the Firefox release. I feel this way we can still offer the product to those who are interested in a relatively easy way reaching a decent amount of people. It's not like the Chrome Web Store but we will improvise, adapt and overcome this. Thanks for being so supportive in this tough situation.

I'm not saying that you have to drop support for the Chrome extension if you want to continue working on the extension for Firefox but take note of this issue, what I mean is that if the code base is similar to Firefox web extensions you could only release big updates for Chrome users as long as you don't have to provide a huge amount of work to readjust the Firefox extension to Chrome and you just have to send a notification to the users to ask them if they want to update the extension when there is a new version out.
By doing this way you can also publish the extension for Firefox on the same separate website in case the extension is taken down by Mozilla. So if the extension is taken down just send a notification to Firefox users that if they still want to have updates for this extension they will have to visit the website.
Moreover, by supporting the Chrome extension, you can also publish the extension on the extension store of Opera and others Chrome ium fork.
PS: Do you want to discuss on Gitter instead of being off topic (of the original issue): https://gitter.im? Just create a room linked to the repo and invite me.

@unixfox I've created the room on Gitter and already invited you :)


Bearish or Bullish: which token?

Add this to the sentiment modifier buttons - should also respect price flip

Done


Avoid compilation error when passing in heap_t to C++ allocators

Before it would not work to create the mi_heap_stl_allocator types with passing in a mi_heap_t*, since sizeof is used and it gives a compilation error. This change fixes that.
Steps to reproduce the current error:
include <mimalloc.h>
#include <vector>

int main() {
    mi_heap_t *hp = mi_heap_new();
    mi_heap_destroy_stl_allocator<int> thisOne(hp);
    mi_heap_destroy(hp);
}

Compilation error is:
====================[ Build | mimalloc-stuff | Debug ]==========================
/home/rhermes/.local/share/JetBrains/Toolbox/apps/clion/bin/cmake/linux/x64/bin/cmake --build /home/rhermes/commons/projects/measure-everything/cmake-build-debug --target mimalloc-stuff -j 22
[1/2] Building CXX object CMakeFiles/mimalloc-stuff.dir/mimalloc-stuff/main.cpp.o
FAILED: CMakeFiles/mimalloc-stuff.dir/mimalloc-stuff/main.cpp.o
/usr/bin/c++  -I/home/rhermes/commons/projects/measure-everything/cmake-build-debug/_deps/mimalloc-src/include -g -std=gnu++20 -fdiagnostics-color=always -MD -MT CMakeFiles/mimalloc-stuff.dir/mimalloc-stuff/main.cpp.o -MF CMakeFiles/mimalloc-stuff.dir/mimalloc-stuff/main.cpp.o.d -o CMakeFiles/mimalloc-stuff.dir/mimalloc-stuff/main.cpp.o -c /home/rhermes/commons/projects/measure-everything/mimalloc-stuff/main.cpp
In file included from /usr/include/c++/13.2.1/bits/shared_ptr.h:53,
                 from /usr/include/c++/13.2.1/memory:80,
                 from /home/rhermes/commons/projects/measure-everything/cmake-build-debug/_deps/mimalloc-src/include/mimalloc.h:489,
                 from /home/rhermes/commons/projects/measure-everything/mimalloc-stuff/main.cpp:5:
/usr/include/c++/13.2.1/bits/shared_ptr_base.h: In instantiation of ‘std::__shared_ptr<_Tp, _Lp>::__shared_ptr(_Yp*) [with _Yp = mi_heap_s; <template-parameter-2-2> = void; _Tp = mi_heap_s; __gnu_cxx::_Lock_policy _Lp = __gnu_cxx::_S_atomic]’:
/usr/include/c++/13.2.1/bits/shared_ptr.h:214:46:   required from ‘std::shared_ptr<_Tp>::shared_ptr(_Yp*) [with _Yp = mi_heap_s; <template-parameter-2-2> = void; _Tp = mi_heap_s]’
/home/rhermes/commons/projects/measure-everything/cmake-build-debug/_deps/mimalloc-src/include/mimalloc.h:497:50:   required from ‘_mi_heap_stl_allocator_common<T, _mi_destroy>::_mi_heap_stl_allocator_common(mi_heap_t*) [with T = int; bool _mi_destroy = true; mi_heap_t = mi_heap_s]’
/home/rhermes/commons/projects/measure-everything/cmake-build-debug/_deps/mimalloc-src/include/mimalloc.h:550:91:   required from ‘mi_heap_destroy_stl_allocator<T>::mi_heap_destroy_stl_allocator(mi_heap_t*) [with T = int; mi_heap_t = mi_heap_s]’
/home/rhermes/commons/projects/measure-everything/mimalloc-stuff/main.cpp:10:50:   required from here
/usr/include/c++/13.2.1/bits/shared_ptr_base.h:1472:26: error: invalid application of ‘sizeof’ to incomplete type ‘mi_heap_s’
 1472 |           static_assert( sizeof(_Yp) > 0, "incomplete type" );
      |                          ^~~~~~~~~~~
ninja: build stopped: subcommand failed.

I'm testing more now, and this doesn't really work at all, even with the given fix.
The current approach of using a std::shared_ptr to manage the heap only works if we are going to call a destructor on it. We also have to change the deleter when constructing. I'll do that and add tests for this and push again.

@microsoft-github-policy-service agree

@daanx requesting a PR here, if you have the time :)

Hi @rHermes -- just to confirm, this is only a problem for  mi_heap_destroy_stl_allocator right? (On msvc 2022 I only get errors on the "destroy" variant).
This would be great to fix.. but we cannot pull in mimalloc/types.h as that breaks the abstraction barrier where mi_heap_t should be abstract. I guess you pulled it in to make the sizeof operation in the static assertion pass?  Is there another way we could make this work where we do not include the internal mimalloc/types.h header file?
Maybe we should not use std::shared_ptr ?  Or perhaps use a wrapper struct around a mi_heap_t* ?

Hey!
Yes the problem is only for mi_heap_destroy_stl_allocator and it's only when you pass in a heap type. The problem here is that shared_ptr is a perfect type for this, as the allocator might live on in objects created with it, like std::vector or others. I think the approach works well and I don't think it should be changed.
I don't really know about the internals of mimalloc, so I don't know about why it needs to be upheld, but I will try some ways to do it, without including the type! I'll comment back when I either push a new version or give up ;)

Ok, that was quick. I don't know why I included the type initially, but it's not needed. I've removed it now and it's good!

ah that's great. Testing now

Thanks so much!


index: diff: sort when detecting renames

Codecov Report
Base: 55.05% // Head: 55.20% // Increases project coverage by +0.14% :tada:

Coverage data is based on head (aefa4cb) compared to base (5bcaf47).
Patch coverage: 85.71% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #217      +/-   ##
==========================================
+ Coverage   55.05%   55.20%   +0.14%
==========================================
  Files          49       49
  Lines        2870     2884      +14
  Branches      490      493       +3
==========================================
+ Hits         1580     1592      +12
- Misses       1218     1226       +8
+ Partials       72       66       -6

Impacted Files
Coverage Δ

src/dvc_data/index/diff.py
80.95% <85.71%> (+0.95%)
:arrow_up:

src/dvc_data/cli.py
32.10% <0.00%> (ø)

src/dvc_data/hashfile/build.py
17.74% <0.00%> (ø)

src/dvc_data/hashfile/state.py
34.48% <0.00%> (ø)

src/dvc_data/hashfile/db/index.py
0.00% <0.00%> (ø)

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

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


Quick fix for child ordering problem in tree

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
768

Relevant Lines:
768

💛  - Coveralls


Update docfx.json for PDF

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

Skype/UCMA/activating-a-ucma-5-0-trusted-application.md
:white_check_mark:Succeeded

Skype/UCMA/additional-scenarios.md
:white_check_mark:Succeeded

Skype/UCMA/administering-a-ucma-5-0-application.md
:white_check_mark:Succeeded

Skype/UCMA/alert-notification-system.md
:white_check_mark:Succeeded

Skype/UCMA/audio-devices-feature.md
:white_check_mark:Succeeded

Skype/UCMA/audio-devices.md
:white_check_mark:Succeeded

Skype/UCMA/audiovideocall-samples.md
:white_check_mark:Succeeded

Skype/UCMA/audiovideoflow.md
:white_check_mark:Succeeded

Skype/UCMA/autoprovisioning-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/avflow-applychanges-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/avflow-hold-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/avflow-initialize-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/avflow-mute-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/back-to-back-user-agent-architecture.md
:white_check_mark:Succeeded

Skype/UCMA/back-to-back-user-agent-feature.md
:white_check_mark:Succeeded

Skype/UCMA/basicaudiovideocall-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/basiccallhandling-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/basicconferencing-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/build-a-bot-sample.md
:white_check_mark:Succeeded

Skype/UCMA/call-park.md
:white_check_mark:Succeeded

Skype/UCMA/callmodalityaddition-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/calls.md
:white_check_mark:Succeeded

Skype/UCMA/calltransferbasic-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/calltransfersupervised-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/client-platforms.md
:white_check_mark:Succeeded

Skype/UCMA/conference-features.md
:white_check_mark:Succeeded

Skype/UCMA/conferenceescalation-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/conferencemanagement-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/contact-center-sample.md
:white_check_mark:Succeeded

Skype/UCMA/contact-center.md
:white_check_mark:Succeeded

Skype/UCMA/conversationcontextchannel-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/creating-a-generic-bot.md
:white_check_mark:Succeeded

Skype/UCMA/creating-an-excel-bot.md
:white_check_mark:Succeeded

Skype/UCMA/declineincomingcall-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/deploying-ucma-4-0-and-ucma-5-0-applications-in-a-skype-for-business-server-2015-environment.md
:white_check_mark:Succeeded

Skype/UCMA/dns-load-balancing.md
:white_check_mark:Succeeded

Skype/UCMA/endpoint-draining.md
:white_check_mark:Succeeded

Skype/UCMA/extending-the-ucma-platform.md
:white_check_mark:Succeeded

Skype/UCMA/forwardincomingcall-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/general-application-activation.md
:white_check_mark:Succeeded

Skype/UCMA/infrastructure-enhancements.md
:white_check_mark:Succeeded

Skype/UCMA/installing-ucma-5-0-sdk.md
:white_check_mark:Succeeded

Skype/UCMA/instantmessagingcall-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/key-features-of-ucma-5-0.md
:white_check_mark:Succeeded

Skype/UCMA/managing-ucma-5-0-applications.md
:white_check_mark:Succeeded

Skype/UCMA/manualprovisioning-quickstart.md
:white_check_mark:Succeeded

Skype/UCMA/media-stack-performance-counters.md
:white_check_mark:Succeeded

Skype/UCMA/migration-coexistence-and-upgrade-of-ucma-applications.md
:white_check_mark:Succeeded

Skype/UCMA/negotiating-declined-offers.md
:white_check_mark:Succeeded

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Skype/docfx.json
:white_check_mark:Succeeded

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


Update reset.yml

Enable bootstrap-os for reset.yml

@matt-freelancing What is the point of having bootstrap-os role run in the reset playbook? It is supposed to have been played at cluster setup...

@matt-freelancing What is the point of having bootstrap-os role run in the reset playbook? It is supposed to have been played at cluster setup...

I use the ansible script on CoreOS instances where I need to specify the bin_dir variable for python. CoreOS-specific variables are set as part of the bootstrap-os role. Hence I added that role to reset.yml to have OS-specific variables available before reset runs. As an alternative, I assume the bin_dir variable can also be set in the inventory file.

@matt-freelancing I agree that we shouldn't need to rerun bootstrap-os. If a variable set by bootstrap-os is missing, maybe the dynamic bin_dir logic should be moved to kubespray-defaults?

@matt-freelancing I agree that we shouldn't need to rerun bootstrap-os. If a variable set by bootstrap-os is missing, maybe the dynamic bin_dir logic should be moved to kubespray-defaults?

Good idea to do so


Changes code behavior on .split()?

Ruby version: 3.0.3
Rubyfmt git sha: 266b47b (latest homebrew version)

Input file
if source.include?('something/')
	source.split('something/').last
end

Rubyfmt's output
if source.include?("something/")
  source.split "something/".last
end

This breaks because
Expected rubyfmt to simply replace the single quotes with double quotes, but instead it left out the parens on the call to #split which changes the behavior of the program. 😨

Have you tested this on the current master branch? I'm fairly certain this is fixed by #415 and that this is a duplicate of #418 and #414.

(cc @fables-tales, might not be a bad idea to do a patch release since there are several other fixes on trunk now)

Doing some cleanup: this is now out in the most recent release (v0.10.0)


Add impredicative set example

Example to answer questions about the impredicative set option.

Codecov Report

Merging #194 (bf40b3e) into master (320e556) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #194   +/-   ##
=======================================
  Coverage   69.19%   69.19%
=======================================
  Files          31       31
  Lines        4295     4295
=======================================
  Hits         2972     2972
  Misses       1323     1323

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 320e556...bf40b3e. Read the comment docs.


Information ASR functioning

Hello,
Thank you for your work, it works pretty well ! :)
I just have a technical question on how the automatic speech recognition works on this Youtube transcript api, is it using an acoustic and language model ? Is BERT the language model use for it ?
I searched for documentation but I didn't find anything, if you have papers, I'm interested.
Thanks !
Camille

Hi @CamilleSchr,
this module just retrieves the subtitles which have automatically been generated by YouTube. I don't think that information on how YouTube generates those is publicly available.

Hi @CamilleSchr,
this module just retrieves the subtitles which have automatically been generated by YouTube. I don't think that information on how YouTube generates those is publicly available.


Fixed #199 - Rotated sprites are not dumping a rotated sprite

Object rotations have to be done before positioning them.

While testing the result I noticed that rotating an object also modifies its position (in the tooltip), even though on screen it appears as rotating in place. Isn't this a bug too?

Yup since the position of a sprite is determined by the TopLeft corner :)
I don't know if that should be considered a bug

Oh ok, then it makes sense to change on rotations.

Tested locally, it works perfectly, thanks again :)


[DEHVAS-259 Redux] Use patch files to set replicas for HAS

Follow up from https://github.com/redhat-appstudio/infra-deployments/pull/1724. See that PR for context on this change.
It seems that adding the replicas field to the staging and production kustomization.yaml didn't result in any changes on the cluster, despite my version of kustomize showing the replicas set to 3. Updating my change to use a patch file instead.

/lgtm


ferret and PyFerret updates on the Mac

Reported by @AnsleyManke on 19 May 2015 23:20 UTC
There was interest in this at GFDL and there definitely is at PMEL. We need a pathway to Mac support for Ferret/PyFerret.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2287

Available through Anaconda


Could not load file or assembly Nlog <IP_ADDRESS>

Hi,
We are getting this error

We have the logentries nlog dll which is version 2.6.0 and the nlog dll which is 4.5.8. The assembly binding redirect is set in the config files for the nlog dll. The installed build gives the above error. Has anyone experienced this before?
My understanding is logentries.nlog has a specific reference to 3.2.1 version of nlog. How can I make the latest version of nlog work?
Thanks
Sowmya

@ddisys Try changing to this package instead:
https://www.nuget.org/packages/r7Insight.nlog
P.S. Think you should contact Rapid7-support if having issue with the service. See also: https://github.com/rapid7/r7insight_dotnet/

The assembly binding redirect is set in the config files for the nlog dll.

Any way, this should work. If you need that, please share the redirect config and fill in the template

Is this still an issue?

This is no longer an issue. Had to make sure the build exe looks at the
output config file for the assembly bindings.
Thanks
On Mon, Nov 5, 2018 at 12:37 PM Julian Verdurmen<EMAIL_ADDRESS>wrote:

Reopened #2977 https://github.com/NLog/NLog/issues/2977.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NLog/NLog/issues/2977#event-1946886703, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AQ1pAVhzOdMN2fUQJDyrAFWF_ywtF4E9ks5usHdtgaJpZM4YALCm
.

--
Sowmya Chidambaram
Software Developer
GUI Team
DDI Systems
p:<PHONE_NUMBER>
a: 75 Glen Road
Sandy Hook CT
w: www.ddisys.com  e<EMAIL_ADDRESS>


Updated error-boundaries.md

Just added a one liner, to be more verbose.

Hello @siddrc!
As the sentence you modify no longer exists in the latest of react.dev, I will close this PR as outdated.
Thank you for taking time to improve the docs!


testing: failures in Cleanup functions do not trigger -failfast if the test includes a parallel subtest

When run with -failfast, the following test should run only Test1, which fails.
Instead, it reports the failure from Test1 and then proceeds to run Test2.
https://go.dev/play/p/OiVQBuQ7fJl:
package main

import "testing"

func Test1(t *testing.T) {
	t.Cleanup(func() {
		t.Errorf("fail in Cleanup")
	})

	t.Run("sub", func(t *testing.T) {
		t.Parallel()
	})
}

func Test2(t *testing.T) {
	t.Fatalf("This test should not run with -failfast.")
}

Without the call to t.Parallel(), it skips Test2 as expected.
This is because the numFailed variable used for -failfast is incremented  before cleanup, only for tests that have parallel subtests.
I do not see a compelling reason for the difference in codepaths between tests with parallel subtests and those without. But I also don't see why we need to keep track of the number of failed tests explicitly — it seems like we could instead just set a boolean when any test fails.

Reopening because the fix was reverted.

The CL originally submitted for this (which was later reverted) also fixes #49929


Add a pinch of CI/CD (no CD yet)

Ditto
(Kinda wanted this as I intend on testing some tings my side)

Very nice! Been wanting to do this forever, and you came to the rescue :)

That's really cool, didn't know it was that capable. Expanding the test suite for better coverage is definitely on the wishlist, so if you enjoy this kind of thing, I'd say it's a job ripe for taking! :)

Once the inspiration strikes I shall do so.
What I will need though is some commandline tool I can use to generate traffic on either ends of the emulated network.
Could be some tool to just send an LXMF message for example.
The assertion would be a basic one at first. Did A receive message B.
On 09 September 2024 15:26:15 SAST, markqvist @.***> wrote:

That's really cool, didn't know it was that capable. Expanding the test suite for better coverage is definitely on the wishlist, so if you enjoy this kind of thing, I'd say it's a job ripe for taking! :)
--
Reply to this email directly or view it on GitHub:
https://github.com/markqvist/Reticulum/pull/544#issuecomment-2338123493
You are receiving this because you authored the thread.
Message ID: @.***>

Tristan B. Velloza Kildaire
Web: https://deavmi.assigned.network
BNET IRC: deavmi
Email: @.***

I'll see what I can do for that for the next release!

Great!
On 09 September 2024 16:03:05 SAST, markqvist @.***> wrote:

I'll see what I can do for that for the next release!
--
Reply to this email directly or view it on GitHub:
https://github.com/markqvist/Reticulum/pull/544#issuecomment-2338215659
You are receiving this because you authored the thread.
Message ID: @.***>

Tristan B. Velloza Kildaire
Web: https://deavmi.assigned.network
BNET IRC: deavmi
Email: @.***


[MF-322] Editing Patient Demographics

Implemented objectives

Editing Demo data
After saving information, patient is redirected to where he was, in that case the patient chart (this could change)

Ticket: https://issues.openmrs.org/browse/MF-322

@samuelmale   you have some conflicting files on this that need to be resolved

In this case is it necessary? I don’t think so.
On Wed, 16 Sep 2020 at 10:24 Samuel Male<EMAIL_ADDRESS>wrote:

@samuelmale commented on this pull request.
In src/patient-registration/patient-registration.component.tsx
https://github.com/openmrs/openmrs-esm-patient-registration/pull/46#discussion_r489294054
:

@@ -256,18 +357,21 @@ export const PatientRegistration: React.FC = () => {
{props => (

         <h1 className={`omrs-type-title-1 ${styles.title}`}>New Patient

         {localStorage.getItem('openmrs:devtools') === 'true' && <DummyDataInput setValues={props.setValues} />}

         <h1 className={`omrs-type-title-1 ${styles.title}`}>{!!existingPatient ? 'Edit' : 'New'} Patient

It's the same this actually. The valid cases to use such a pattern is when
you want to assign a boolean value for a property that expects a boolean
with Typescript.
It's literately coming up with an explicit expression that returns a
boolean.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/openmrs/openmrs-esm-patient-registration/pull/46#discussion_r489294054,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACWOHPREZF363PFI34D6O3LSGB73BANCNFSM4RNXW6MA
.

--

Larry


Redesign the bar plots in the endpoint explainer

The current bar plots are showing only for the last steps, missing some information about the case counts in the first steps of the flow.
Ideally we would have some visual for each step that shows:

relative case count at this step compared to total case count
sense of increase or decrease in case count compared to previous step

Just a note that the bar plots were replaced with a table with Sara's redesign.


Update pr_BR.json

Fix some terms and add new lines to support 7.5 version.

Hello!
First of all: thank you very much for the PR!
I did noticed a few typos (it looks like some of them were already present in the previous version of the file).
Could you have a look at the commit comments and fix them (if necessary)?
That would be great :+1: :)
Thanks again and greetings from Camunda HQ,
Vale

Hi mbitencourt,
thanks for the clarifications!
can you make the changes in your pull request please?
Then I'll be very happy to accept it and merge.
Cheers,
Vale

I made the last corrections and merged your pull request


Improve shared/dynamic library search on OS X

One of the easiest and probably most common way to obtain binwalk on OS X is via Homebrew. However, the experience is suboptimal for people with a non-standard installation (i.e. install location is not /usr/local). I'd like to help address that.
The problem is inside src/binwalk/core/C.py, lines 128–131, that searches the following directories on OS X:

binwalk's private libs directory (no longer needed since there are no longer any bundled libraries?).
/opt/local/lib (standard MacPorts installation).
/usr/local/lib (standard Homebrew installation).
/usr/local/Cellar/<package>/<version>/lib (package-specific prefix of a standard Homebrew installation).

This one has an additional problem: The only shared library referenced in current HEAD is libfuzzy that is part of the ssdeep package (“formula” in Homebrew-lingo), but the <package> part is substituted with *fuzzy*, so this never yields anything.

My idea for a solution would be:

When running setup.py, offer a switch --library-search-prefix= that would default to /usr/local.
Put this into a constant in src/binwalk/core/C.py.
Use that constant in find_library in the Library class instead of the various hard-coded paths. (This could even be extended to all non-Windows platforms, not just OS X.)

If this is an acceptable solution, I can try to come up with a pull request that implements this, though my Python skills are quite limited.

Thanks for the offer! The C library dependencies have always been a PITA, on all platforms.
This is why there is no active code in the master branch that uses any C libraries; everything is now pure Python, so this should no longer be an issue. A stable release based on the current master branch is forthcoming.
Linux/OSX/Windows users can just run setup.py install to get up and running, though Python2 users are encouraged to install the python-lzma module (this module is now standard in Python3); see the INSTALL.md file for more info.

Thanks for the quick response! I had failed to see that the only code referencing a shared library (binwalk.modules.hashmatch.HashMatch using libfuzzy) was no longer active …


Sentence ID validation

Incorporate changes from https://github.com/nert-nlp/pastrie/commit/fcb2ef61f4b90b4d59bccace993b52dbb265f503 in conllulex2json.py

Fixed in 6a81436


Test running

Fix the warning which failed the CI pipeline

Close due to format issue.

Fix the warning which failed the CI pipeline


feat(previewer): support all preview scroll actions from FZF

This adds the preview-(up|down), preview-page-(up|down), and preview-(top|bottom) actions from FZF, and renames the existing page scroll actions to preview-half-page-(up|down).

Ty @toupeira! I’ll review over the weekend and share my thoughts, looks like a great addition.

https://github.com/ibhagwan/fzf-lua/commit/fcd0853770a7ef2965e2f1a8c0f1e03ad9aaea62
Ty @toupeira! Rebase-merged manually as lua_ls was complaining about the usage of single quotes.

I really like the line up/down variants, I'm thinking perhaps we should map shift-{left|right} to these as a default, what do you think?

@ibhagwan thanks for merging! Not sure if that was accidental, just double-checking if the breaking change is okay? :grinning:
I don't have a strong opinion on the defaults, happy to go with whatever you prefer. I saw Shift-Left is already mapped to preview-page-reset though.
Also note that FZF has these default mappings:

shift-left: backward-word
shift-right: forward-word
shift-up: preview-up
shift-down: preview-down

I personally use Ctrl-e/y mappings for up/down.

Not sure if that was accidental, just double-checking if the breaking change is okay? 😀

Not accidental, as I mentioned above somewhere I don’t consider this breaking (nothing breaks for anyone) the scroll just behaves a tiny bit different :)

I don't have a strong opinion on the defaults, happy to go with whatever you prefer. I saw Shift-Left is already mapped to preview-page-reset though.

Correct, but I think it would not be a big deal to change, we could also find an alternative default bind for this.

shift-left: backward-word

Wasn’t aware of the backward/forward word mappings, maybe we should consider different mappings altogether for preview scrolls?

I personally use Ctrl-e/y mappings for up/down.

ctrl-e would override the fzf’s readline defaults (end of line).

@ibhagwan ok! :+1:
What do you think about Shift-PageUp/Down for preview-page-(up|down), and Shift-Up/down for linewise preview-(up|down)?

@ibhagwan ok! 👍
What do you think about Shift-PageUp/Down for preview-page-(up|down), and Shift-Up/down for linewise preview-(up|down)?

I think it’s a great idea, I’m just concerned about laptop keyboard which do not have PageUp/PageDown keys, can we input these with a key combo?

@ibhagwan laptops and other smaller keyboards should usually have PageUp/Down with a Fn combo at least. This one does for example and I can use Shift+Fn+Up/Down for PageUp/Down, which would match nicely with Shift+Up/Down.
I'll still remap it to something more convenient (I use Ctrl-f/b for page up/down), but I think as a default it would work well :grinning:

Hmm what would be neat is having PageUp/Down directly mapped (no Shift) to scroll the preview only if it's visible, and mapped to scroll the FZF results otherwise.

Hmm what would be neat is having PageUp/Down directly mapped (no Shift) to scroll the preview only if it's visible, and mapped to scroll the FZF results otherwise.

Idk, I feel too many people already use the page up/down for scrolling the results.

@toupeira, with https://github.com/ibhagwan/fzf-lua/commit/7a2d9849fe6ee87a386a8b9486ebd2e2884184fa I removed the default shift-left as it overides the default fzf word left/right, if someone is missing this functionality they can add it in their setup.
I also added new defaults for preview (line) up/down with alt-shift-{up|down}, I avoided using shift-pageup|pagedown as these keys aren't supported by fzf and I prefer to maintain uniformity in functaionlity where possible, the alt-shift variant is supported in both fzf and neovim.

@ibhagwan thanks, works for me! :grinning:


RegexNotFoundError: Unable to extract Initial JS player signature function name;

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 this: [x])
Use the Preview tab to see what your issue will actually look like

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

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

Before submitting an issue make sure you have:

[ ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones
[ ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

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

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

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[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 2018.09.01
[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

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

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
uqNt4_B4
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.youtube.com/watch?v=jNmuqNt4_B4']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.09.01
[debug] Python version 2.7.15rc1 (CPython) - Linux-4.15.0-33-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.4-0ubuntu<IP_ADDRESS>, ffprobe 3.4.4-0ubuntu<IP_ADDRESS>
[debug] Proxy map: {}
[youtube] jNmuqNt4_B4: Downloading webpage
[youtube] jNmuqNt4_B4: Downloading video info webpage
[youtube] {22} signature length 41.42, html5 player vflvABTsY
[youtube] jNmuqNt4_B4: Downloading player https://www.youtube.com/yts/jsbin/player-vflvABTsY/en_US/base.js
ERROR: Signature extraction failed: Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1210, in _decrypt_signature
video_id, player_url, s
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1121, in _extract_signature_function
res = self._parse_sig_js(code)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1182, in _parse_sig_js
jscode, 'Initial JS player signature function name', group='sig')
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 972, in _search_regex
raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
(caused by RegexNotFoundError(u'Unable to extract \x1b[0;34mInitial JS player signature function name\x1b[0m; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1210, in _decrypt_signature
video_id, player_url, s
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1121, in _extract_signature_function
res = self._parse_sig_js(code)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1182, in _parse_sig_js
jscode, 'Initial JS player signature function name', group='sig')
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 972, in _search_regex
raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 792, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 502, in extract
ie_result = self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1803, in _real_extract
encrypted_sig, video_id, player_url, age_gate)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1220, in _decrypt_signature
'Signature extraction failed: ' + tb, cause=e)
ExtractorError: Signature extraction failed: Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1210, in _decrypt_signature
video_id, player_url, s
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1121, in _extract_signature_function
res = self._parse_sig_js(code)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1182, in _parse_sig_js
jscode, 'Initial JS player signature function name', group='sig')
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 972, in _search_regex
raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract Initial JS player signature function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
(caused by RegexNotFoundError(u'Unable to extract \x1b[0;34mInitial JS player signature function name\x1b[0m; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Duplicate Issue #17457


Read model time output

In my meta files I see this parameter
timeInterval = [  3.153559200000E+07  6.307202000000E+07 ];
that we are currently not uploading.  Instead we set the time based on the input time step, but it would be useful to just be able to read the meta file instead of having to put in the timestep.  I would be happy to work on this, but I am a bit of a python noob and need some help.  Can anyone help me with this?

I don't see that information in all *.meta files, just the ones from pkg/diagnostics. Are your *.meta files exclusively generated by pkg/diagnostics?
E.g. this is what I get in a Eta.*.meta file:
 nDims = [   2 ];
 dimList = [
    60,    1,   30,
    60,    1,   30
 ];
 dataprec = [ 'float32' ];
 nrecords = [     1 ];
 timeStepNumber = [         10 ];

I have output not from pkg/diagnostics such as Eta.*.meta, but I only use the data from pkg/diagnostics for analysis.

@mkyoungs thanks for your feedback and suggestion.
I see how it would be convenient to automatically get the time from the meta files. On the other hand, this would introduce several potential complications such as:

how to turn timeInterval into a specific time? (interval indexes are not supported by xarray yet)
what is the "reference date" for the time if inferred from .meta files?
which .meta file should be used if there are multiple files being read?
what if there is a conflict between different .meta files?
what if there is a conflict between the time inferred from the .meta file and the one suggested by delta_t?

Addressing and properly testing for all of these edge cases sounds like a significant amount of work. And for what benefit? For the convenience of not having to specify delta_t=2400 when calling open_mfdataset? In terms of cost / benefit, it doesn't feel like the best use of my time to work on this feature.
However, if it is very important to you, you are welcome to work on it and submit a PR. The first step would be to check out the way time is currently set:
https://github.com/xgcm/xmitgcm/blob/master/xmitgcm/mds_store.py#L797-L809
and the way .meta files are parsed:
https://github.com/xgcm/xmitgcm/blob/master/xmitgcm/utils.py#L64-L82
Let us know if you have any questions along the way!

Hi, to get a time coordinate without specifying "delta_t" , couldn't we also try to read the "data" file and retrieve deltaTClock or deltaT?

Hi @aulemahal,
You can get deltaT from data with a function like this:
def find_dt():
    for line in open("data"):
        if 'deltaT' in line:
            # assuming one occurence of deltaT, else needs to pick one from list
            dt= line.replace(',','').split()[-1]
    return dt

Thanks, @raphaeldussin, it is indeed quite simple. But my question was also whether there is any reason that xmitgcm doesn't read the 'data' file? I mean, it could also infer the geometry from the flags, or store some of the configuration in the dataset's attributes. I could suggest such a PR, I just wondered if this was a deliberate choice not to implement it.

Some reasons why we chose not to rely on the data file:

data file is not always kept together with the data
data file can be inconsistent with the data (e.g. if parameters are changed after the run is performed)
parsing the data file is hard

That said, we would definitely be open to a PR which could help extract relevant information from the data file, as long as it was optional to use the feature.


MultiCloud Backend Design Analysis

Issue/Feature Description:
MultiCloud Backend Design Analysis
Why this issue to fixed / feature is needed(give scenarios or use cases):
How to reproduce, in case of a bug:
Other Notes /  Environment Information: (Please give the env information, log link or any useful information for this issue)

Subtasks opened to track this.


Toggle single apps on/off

Blessing should be able to toggle single apps on and off from monitoring.

Actually, best just rename the unicorn.conf file so that Blessing won't find it


train time

训练多少epoch，200 epoch结束还有，要等多久呢？

Thank you for your interest in our work.
We train the model for 200 epochs and the training time depends on the size of the dataset.
Tiny-ImageNet and CIFAR10+ datasets take a relatively shorter time.
If you want to train the model more fastly, you can use
from torch.cuda.amp import autocast
from torch.cuda.amp import GradScaler

Since I do not speak Chinese, I hope this answers your question.

"==> Epoch 196/200
==> Epoch 197/200
==> Epoch 198/200
==> Epoch 199/200
==> Epoch 200/200
Acc: 95.93333
cifar10 Preparation
Selected Labels:  [7, 1, 0, 9, 4, 6]
Files already downloaded and verified
All Train Data: 50000
Files already downloaded and verified
All Test Data: 10000
Files already downloaded and verified
Train:  30000 Test:  6000 Out:  4000
All Test:  10000
==> Epoch 1/200
==> Epoch 2/200
==> Epoch 3/200
==> Epoch 4/200
==> Epoch 5/200"
This is the output file of my training. After training for 200 epochs, there is only accuracy. And then training next 200 epochs.

Have you created logs directory? If you check the line 352 in osr.py, it creates logs in the logs directory.
Make sure you have created the logs directory before you actually run the code.

I use background run by "nohup python -u osr.py > log 2>&1 &"
When running the code, line 355 reported an error, and I modified it according to line 352 to“     if options['dataset'] == 'cifar100':
stats_log = open('./logs/' + options['split'] + '/' + options['dataset'] + '/DIAS_%d' % (i) + str(options[
'out_num']) + '.txt', 'w')
else:
stats_log = open('./logs/' + options['split'] + '/' + options['dataset'] + '/DIAS_%d' % (i) + str(options['out_num']) + '.txt', 'w')”

Can you show me the error message?

Sorry,I can't find it.
I find the running result of the code. Is it in the /logs/F1/cifar10/DIAS_050.txt

Yes, that is the one of the result with F1 split protocol.


Annotation which is not defined every missing - Code="ErrorMissingAnnotation"

Describe the bug
I'm using custom installed AGIC (1.6.0 version). After installation AGIC is successfully applying configuration to application gateway but in log there is an error occuring every time configuration is being applied that appgw.ingress.kubernetes.io/hostname-extension is not set in ingress.
The problem is I'm not using appgw.ingress.kubernetes.io/hostname-extension annotation in my ingress definition.
How can I get rid of this error and why is it happening? I also checked ingress definition directly in my kubernets cluster and there is also no any hostname-exstension annotation.
To Reproduce
Ingress configuration:
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
name: aspnetapp1
annotations:
kubernetes.io/ingress.class: azure/application-gateway
appgw.ingress.kubernetes.io/ssl-redirect: "true"
appgw.ingress.kubernetes.io/backend-path-prefix: /
spec:
tls:
- hosts:
- myhostname.com
secretName: mysecretname
rules:

host: myhostname.com
http:
paths:

path: /
backend:
service:
name: aspnetapp1
port:
number: 80
pathType: Prefix

Ingress Controller details
AGIC version: 1.6.0
AGIC logs:
"Error while parsing hostname extension: Code="ErrorMissingAnnotation" Message="appgw.ingress.kubernetes.io/hostname-extension is not set in Ingress agic/aspnetapp1""
verbosityLevel: 5

Update: setting verbosity level <5 (e.g. 3) removes error from logs.
Problem then is agic is not logging full request it is sending to Application Gateway and makes things harder to debug in case of any problems. E.g. when agic is deployed as add-on with az command its verbosity level is 5


Mods for versions 1.19.4 and 1.18.2 are very different

Hello. I used the mod on version 1.18.2, we installed models on the server with textures for accessories and weapons, armor, everything was fine, it read any optifine formats and paths. The updated mod for version 1.19.4 does not work like that at all. I wanted to update the server to this version, but most of the accessories turned out to be without textures, and some weapon models stopped working. There are too many changes and they are critical, we have a lot of models, it is impossible to try to fix it all by hand, it will take a lot of time. The fittings are provided from various authors from mcmodels and elitecreatures, everything is done correctly in terms of paths, all textures converge and work correctly on 1.18.2 with this mod.
Could you update a mod that was on 1.18.2 to 1.19.4 without your other changes? We cannot use this mod on newer versions.

First off, absolutely no? What you're suggesting I'd do is cursed af :/
2nd, missing textures usually means that you're using textures in paths that arent in the blocks atlas as is required in all 1.19.4 resourcepacks. Vanilla or CIT packs alike.
3rd, if you believe there is an issue with how your pack loads in 1.19.4, create an issue containing both the resourcepack and your logs showing you loading it.


Expose SSL sockets to micropython

Currently it's not possible to create an SSL socket through micropython.  This is for two reasons,

the protocol argument of the constructor is not passed through in stmhal/modusocket.c socket_make_new()

There is no exposure to setup the structure for the certificates exposed to micropython yet.

This isnt entirely working yet - @drrk will close once it is.


Migration for NullSafetyAnalysisEvent to package:unified_analytics

Related to tracker issue:

https://github.com/flutter/flutter/issues/128251

This PR will use the new event from package:unified_analytics and will consolidate the 3 separate send events from the legacy analytics system to just sending one
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.

No longer necessary to migrate this event per https://github.com/dart-lang/tools/pull/202#issuecomment-1811262005


nearest_neighbour_index bugfixes

Closes https://github.com/SciTools/iris/issues/2969
There are a couple of bugs in Coord.nearest_neighbour_index(), demonstrated by the tests in the first two commits.

If the bounds of the coordinate have an integer dtype and the point argument is a float, the wrong index can be returned due to casting a float to an int.
Wrapping is not handled if the coordinate is circular and has bounds.

To be honest, I think wrapping should really be taken into account whether or not the coordinate is circular, as long as the unit has a modulus, but I haven't made that change here.

@djkirkham I just want to check that you won't be offended if I take over this PR.  Obviously if you're still interested then by all means you can fix it yourself and then I'll review it.
If however you have better things to do and don't reply before I get round to looking at this again, I'll just take it off your hands.

@pelson What's the status of this ticket now? What was the reason for you not merging it in May when you approved it?  And is it still relevant now?

What's the status of this ticket now? What was the reason for you not merging it in May when you approved it? And is it still relevant now?

@bjlittle assigned himself, and it would have been rude for me to merge something that is assigned to somebody else. There was also an action to add a unit-test. I think you can happily take this on in a separate PR (feel free to close this one when you read this).


Mocking Notification Failed

Laravel Version: ^6.0
PHP Version: 7.3.17
Database Driver & Version:Mysql &  Ver 14.14 Distrib 5.7.30

Description:
I want to fake notification but always it asserting false and say
The expected [App\Notifications\NewOrderReceived] notification was not sent.
Failed asserting that false is true.
Channel which I used is FcmChannel
Steps To Reproduce:
After Some debugging I ended up to this function
* Get all of the notifications for a notifiable entity by type.      *      * @param  mixed  $notifiable      * @param  string  $notification      * @return array      */     protected function notificationsFor($notifiable, $notification)     {         return $this->notifications[get_class($notifiable)][$notifiable->getKey()][$notification] ?? [];     }**
It generate error Undefined index: App\Models\User
**My code **
`<?php
namespace Tests\Unit;
use App\Models\Brand;
use App\Models\SubOrder;
use App\Models\User;
use App\Notifications\NewOrderReceived;
use App\Services\OrderService;
use App\States\Order\PendingPayment;
use App\States\Order\Preparing;
use Tests\TestCase;
use Illuminate\Support\Facades\Notification;
class NotficationTest extends TestCase
{
protected $orderService;
protected function setUp(): void
{
    $this->orderService = app()->make(OrderService::class);
    parent::setUp();
}
/** @test */
public function it_send_notification_in_case_of_status_is_changed()
{
    $brand = factory(Brand::class)->create();

    $subOrder = factory(SubOrder::class)->create(
        [
            'brand_id' => $brand->id,
            'state' => PendingPayment::$name,
        ]
    );

    Notification::fake();
    $subOrder->state = Preparing::$name;
    $subOrder->save();
    Notification::assertSentTo(
        User::latest()->first(),
        NewOrderReceived::class,
       1
    );

}

}
`

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

@mohammedabdallah I can't seem to see any line of code that actually sends the notification. I think you should do that before your assertions.
Since you are sending the notification to a user, you should have a reference to a user Model or collection depending on your flavor...
...
// get the user
$user = ...

$user->notify(new NewOrderReceived($subOrder));

// then you can make your assertions

hopefully, this helps.


Fix the link to the complete grammar reference.

I couldn't merge this PR because of the CLA check, so I created a new one.


http site does not enforce upgrade to https

% curl -I http://marketplace.service.gov.au/
HTTP/1.1 200 OK
Content-Length: 10265
Content-Type: text/html; charset=utf-8
Date: Sun, 25 Sep 2016 03:22:04 GMT
Dm-Request-Id: e5d2e102-c342-48e6-8d14-3df7d189c8e3
Server: waitress
Set-Cookie: dm_session=eyJfZnJlc2giOmZhbHNlLCJfcGVybWFuZW50Ijp0cnVlfQ.CsjUXA._ibcjEmArabAwCwk51L0fPq3vCI; Expires=Sun, 25-Sep-2016 07:22:04 GMT; Secure; HttpOnly; Path=/
X-Cf-Requestid: d72f9f55-58da-4045-42f8-d362f8f56818
X-Frame-Options: DENY
Connection: keep-alive
/cc @santrancisco

This has been fixed


No Error, No Timeout

I have a script that's scanning my connected USB Devices to find printers, works except for when it comes across my VR Headset.
Calling getStringDescriptor via
getManufacturerString (counter, idVendor, idProduct, index)
{
    var term = usb.findByIds(idVendor, idProduct);
    term.open();

    try
    {
        term.getStringDescriptor(index, counter.callbackClosure(counter, counter.myManufacturerStringCallback));
    }
    catch(err)
    {
        console.log(err);
    }
}

With a VR Headset (idVendor = 1118. idProduct=1625)
The call term.getStringDescriptor never returns.  No error is thrown, just goes into limbo.
term.open() doesn't throw any error (tried that)

Where are you getting the index passed to this function? Generally you would get it from the device descriptor, not know it ahead of time.
What OS is this on?
Can you provide a libusb debug log by setting the LIBUSB_DEBUG=4 environment variable?

Closing due to inactivity, please reopen this issue if it persists with the latest release.


Support passing a compiled expression to Warpath.query/3

@jonasschmidt
I closing this pr in favor of #53
Thanks


Magpie Session created and expiration should be replaced with block height

The current demo is generating a created time and expiry time on each session. This can't be verified and not able to make consensus. It has to be changed to block height.

As pointed out inside #32, I would suggest replacing all the time-related data with block heights. I think we should rename this issue an make it more generic so that it can refer to:

Post creation time
Like creation times
Session creation time
Session expiration time

This should be pretty straightforward as we can user the ctx.BlockHeight() value that should be deterministic and coherent between all the nodes.


Missing Curriculum Spells from Wizard

When selecting a Wizard class, the class feature describing Curriculum Spells is missing. Only in journals is this information present.

The text in the class feature is correct. The Player Core Wizard entry has two Arcane School items: the class feature and a supplementary article covering the details.
Those were merged into one to put everything in one place, but the supplementary article should be its own entry.


AAD-SF uses a twitter account (https://twitter.com/SFWater). Replace or remove?

Change

https://twitter.com/SFWater
To
https://twitter.com/LGROW_GVMC

Review Instructions
Swap out twitter account ( https://twitter.com/LGROW_GVMC )
Where to check:

Guidelines twitter link

on the home page > press the  "What Does it Mean To Adopt a Drain?" button > press the twitter icon
The LGROW twitter page should appear.

Preparation for emails

make sure your account has a valid email address

un-adopt all your adopted drains

First adoption email confirmation facebook link

Adopt your first drain and you should get an email with a link to twitter
click the twitter icon > The LGROW twitter page should appear

Second adoption email confirmation facebook link

Adopt your second drain and you should get an email with a link to twitter
click the twitter icon > The LGROW twitter page should appear

https://lgrow-staging.herokuapp.com

if the twitter links on the guideline page, the first and second adoption emails are found and goto LGROW twitter page then move issue #40 to Done in waffle.io


Fixing STORM-1193 - part 2

Resolves "Class "SimpleSensor" doesn't implement required "update_trigger" method from the base class" error when running sensor using "st2sensorcontainer --config-file=/etc/st2/st2.conf --sensor-name=SimpleSensor"

Merged, thanks.


Seems Langchain Wandb can not handle the ChatOpenAI object?

System Info
The llm model using default davinci example provided in https://python.langchain.com/en/latest/ecosystem/wandb_tracking.html is OK. But seems not able to handle ChatOpenAI object. How to solve this?
The code is as below:
Who can help?
No response
Information

[X] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Code:
from langchain.callbacks import WandbCallbackHandler, StdOutCallbackHandler

wandb_callback = WandbCallbackHandler(
    job_type="inference",
    project="langchain_callback_demo",
    group="test_group",
    name="llm",
    tags=["test"],
)
callbacks = [StdOutCallbackHandler(), wandb_callback]

chat = ChatOpenAI(model_name='gpt-3.5-turbo', callbacks=callbacks, temperature=0, request_timeout=20)
resp = chat([HumanMessage(content="Write me 4 greeting sentences.")])
wandb_callback.flush_tracker(chat, name="simple")

Expected behavior
Error:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In[3], line 4
      2 chat = ChatOpenAI(model_name='gpt-3.5-turbo', callbacks=callbacks, temperature=0, request_timeout=20)
      3 resp = chat([HumanMessage(content="Write me 4 greeting sentences.")])
----> 4 wandb_callback.flush_tracker(chat, name="simple")

File [~/miniconda3/envs/lang/lib/python3.11/site-packages/langchain/callbacks/wandb_callback.py:560](https://vscode-remote+wsl-002bubuntu-002d22-002e04.vscode-resource.vscode-cdn.net/home/ocean/projects/behavior/~/miniconda3/envs/lang/lib/python3.11/site-packages/langchain/callbacks/wandb_callback.py:560), in WandbCallbackHandler.flush_tracker(self, langchain_asset, reset, finish, job_type, project, entity, tags, group, name, notes, visualize, complexity_metrics)
    558 model_artifact.add(session_analysis_table, name="session_analysis")
    559 try:
--> 560     langchain_asset.save(langchain_asset_path)
    561     model_artifact.add_file(str(langchain_asset_path))
    562     model_artifact.metadata = load_json_to_dict(langchain_asset_path)

AttributeError: 'ChatOpenAI' object has no attribute 'save'

Hey @firezym , @GraesonB , our preferred LangChain integration, W&B Prompts, can be found here:
https://docs.wandb.ai/guides/prompts/quickstart
The above is an earlier callback that we'll likely be deprecating in the coming months.
Would love to know what you think about Prompts!

@morganmcg1 Get it, thanks a lot for the info!

Thanks. I think it is an outdated issue now. Please proceed. @dosu-beta