Update mdo-email-entity-page.md

https://github.com/MishraSoumyaMS/microsoft-365-docs/commit/87e8ec10ef2bdb353034b5b4b9d96b1a897e2a48 reference images

Docs Build status updates of commit 8fd8470:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md
:warning:Warning

Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md

Line 169, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Take-ActionWizard-Email-entity.png'.
Line 176, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Block-URL-Email-entity.png'.
Line 178, Column 15: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/permissions-microsoft-365-security-center?view=o365-worldwide' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 178, Column 15: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/permissions-microsoft-365-security-center?view=o365-worldwide' will be broken in isolated environments. Replace with a relative link.
Line 178, Column 15: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/permissions-microsoft-365-security-center?view=o365-worldwide

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 41819f4:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md
:warning:Warning

Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md

Line 169, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Take-ActionWizard-Email-entity.png'.
Line 176, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Block-URL-Email-entity.png'.
Line 178, Column 15: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/permissions-microsoft-365-security-center?view=o365-worldwide' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 178, Column 15: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/permissions-microsoft-365-security-center?view=o365-worldwide' will be broken in isolated environments. Replace with a relative link.
Line 178, Column 15: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/permissions-microsoft-365-security-center?view=o365-worldwide

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 28b820d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md
:warning:Warning

Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md

Line 169, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Take-ActionWizard-Email-entity.png'.
Line 176, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Block-URL-Email-entity.png'.
Line 178, Column 15: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/permissions-microsoft-365-security-center?view=o365-worldwide' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 178, Column 15: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/permissions-microsoft-365-security-center?view=o365-worldwide' will be broken in isolated environments. Replace with a relative link.
Line 178, Column 15: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL.  Otherwise, remove the view parameter. URL: https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/permissions-microsoft-365-security-center?view=o365-worldwide

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit af6e262:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md
:warning:Warning

Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md

Line 169, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Take-ActionWizard-Email-entity.png'.
Line 176, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Block-URL-Email-entity.png'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit b241afe:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md
:warning:Warning

Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md

Line 169, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Take-ActionWizard-Email-entity.png'.
Line 176, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Block-URL-Email-entity.png'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 65a2071:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md
:warning:Warning

Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md

Line 169, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Take-ActionWizard-Email-entity.png'.
Line 176, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Block-URL-Email-entity.png'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 6a71f30:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md
:warning:Warning

Details

microsoft-365/security/office-365-security/mdo-email-entity-page.md

Line 169, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Take-ActionWizard-Email-entity.png'.
Line 176, Column 1: [Warning: file-not-found - See documentation] Invalid file link: '../../media/Block-URL-Email-entity.png'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[filebeat] Make chart compatible with Kubernetes-on-Windows

Describe the feature:
Although filebeat itself works perfectly fine on Windows and Kubernetes on Windows the Helm chart is not configurable enough to use it for both Linux and Windows.
There's 2 notable things that need to be possible:

Volume varlibdockercontainers needs to have hostPath /ProgramData/docker/containers to be able to find the container logs. It can still be mapped to mountPath /var/lib/docker/containers, which is actually preferable, because you can then reuse the same config between linux and windows deamonsets.
Mounting files with subPath is not possible for Windows containers, only directories can be mounted. For that reason filebeat-config and data volumes need to be mounted in different locations, without the subPath properties getting used.

Describe a specific use case for the feature:
Shipping logs from Kubernetes on Windows nodes.

For using processor add_kubernetes_metadata correctly it's useful to set mountPath: /ProgramData/docker/containers as well, so preferably it's configurable via the same property.
That works in combination with filebeat.yaml:
    filebeat.inputs:
    - type: container
      paths:
      - "C:\\ProgramData\\docker\\containers\\*\\*.log"
      processors:
      - add_kubernetes_metadata:
          in_cluster: true
          default_matchers.enabled: false
          matchers:
          - logs_path:
              logs_path: "C:\\ProgramData\\docker\\containers"

I'm curious about why the subPath was used for the config mounts, since you can actually mount a configmap to /usr/share/filebeat and another directory to nested mount /usr/share/filebeat/data. But perhaps this didn't work in older versions of Kubernetes?
If it's needed I can add a toggle value in order to support both behaviours.

 \
	Expand-Archive filebeat.zip -DestinationPath C:\filebeat; \
	\
	Write-Host 'Removing filebeat.zip ...'; \
	Remove-Item filebeat.zip -Force; \
	\
	Write-Host 'Completed installing filebeat.';

FROM mcr.microsoft.com/windows/nanoserver:1809

COPY --from=core /filebeat/filebeat-7.10.2-windows-x86_64 /filebeat
COPY --from=core /Windows/System32/netapi32.dll /Windows/System32/netapi32.dll

RUN dir c:\filebeat

USER ContainerAdministrator

ENTRYPOINT [ "c:\\filebeat\\filebeat.exe", "-c", "c:\\usr\\share\\filebeat\\filebeat.yml" ]

Still struggling with the sha verification but works fine otherwise.

Hi @jmlrt can you have a look at associated PR? I think all my changes should be fine now, just need to have the tests run again and see if it's okay to be merged now. Thx


feature: add CI/CD

Build Apple SDK (all platforms and architectures)
Build Web SDK

There's an ugly double-zip archive for Apple SDK. That's because the Posemesh.xcframework dir contains relative symlinks and the upload-artifacts GH action doesn't preserve these symlinks and instead hard links the files within its zip. That's why I had to zip manually first. There's also no way to specify for upload-artifacts to skip its own zipping.
This is probably okay for now, but let's try to find a better solution.

also another couple of possibly important things to add:

put commit hash somewhere in the lib (for js it can be the first // comment in .js/.ts file, for apple it can be within the Info.plist, one of the keys that contains description?)
ensure version stamp is everywhere (for apple its already in info.plist in one of the versioned keys, for js we may also use the comment at the start of .js/.ts)

this could come in handy when we start getting bug reports
also TODO reminder to self: add missing initialize() function ts def in Web API on Posemesh class (static func)

few more ideas TODO:

in web sdk, the js code should compare that the versions of PosemeshNetworking.wasm and Posemesh.wasm match the same version/commit id pair as embedded in the javascript code (in case we get bug reports from people mishmashing different files)
add getVersion() functions (maybe also commit id)


@ ./~/mariasql/lib/Client.js 13:10-45

Here is message details:
I am using this package with redux and react
WARNING in ./~/mariasql/lib/Client.js
Module not found: Error: Cannot resolve 'file' or 'directory' ../build/Release/sqlclient in /home/maxifjaved/Desktop/manish/node_modules/mariasql/lib
@ ./~/mariasql/lib/Client.js 11:10-47
ERROR in ./~/mariasql/lib/Client.js
Module not found: Error: Cannot resolve 'file' or 'directory' ../build/Debug/sqlclient in /home/maxifjaved/Desktop/manish/node_modules/mariasql/lib
@ ./~/mariasql/lib/Client.js 13:10-45

You're not trying to use mariasql from the browser are you?

I am experiencing a similar issue running the code with devtool.
I get the following error message:
Cannot find module '../build/Debug/sqlclient'

@flongo How are you trying to use the module? You can replace the addon = require(..../Debug/sqlclient) in the lib/Client.js with throw ex to get the real error.


RuntimeError: std::exception

When trying to use the plotting functionality a I get a "RuntimeError: std::exception"
I've testes if the drive are mounted correctly to the container and if a I read/write access as well for the /plotting and for the /plots directory this is working fine.
We I try to manually create a plot using the chia commandline I get the same error.
I'm running the :latest version of machinaris freshly installed today.

Welcome! Sorry to hear you're having an issue.  Please let us know your OS platform, and attach the logs from the following locations in your .machinaris folder:

plotman/logs/
machinaris/logs/
mainnet/log/

Also, the output of the Docker launch would be useful.  In Unraid, this the 'Logs' view for the Machinaris container.  If, this is a docker run, try launching without the -d flag and watch the terminal/console for output.
After submitting these logs and given this is a fresh install, I would probably recommend that you, first ensure your mnemonic.txt is safely saved/recorded.  Then stop the running container, delete entire .machinaris folder, and launch again.  Follow the post-launch configuration steps.  Hopefully this resolves your issue.

Thank you for the fast reply.
I found the root cause for the issue which was due to my underlying setup. I was running the docker container in a Ubuntu VM in Proxmox. Looking into the error log showed the issue with bitfield. So I've changed my setup now and remove the proxmox layer.
This issue can be closed. Thank you


add controlsList attribute for media elements

resolve #1133

Coverage remained the same at 100.0% when pulling ce3dffbdd5efc67d31992831cf1beabcd8b28596 on wayou:patch-1 into 61a540e8078d533803213104eb3caffd3d845e2a on developit:master.


updating some URLs

Thanks for contributing!
However there are some issues with these changes, see line comments.
Another thing to note: we don't merge PRs into master branch, we use dev branches for this. Please submit to v0.12.0.x (stable/current version) and v0.12.1.x (test/dev version) branches only.

Hey. Okay. (a) Can't believe my eyes just swooped right over the other URL. And (b) ... I haven't looked at the dev branches yet, but maybe the README.md really needs to be updated to mention that. Of course, there are a number of TODOs in there... as there always are.
Thanks for kindly redirecting me. I will fork from a different branch. :)


Unable to cast object of type 'EFCoreSecondLevelCacheInterceptor.EFTableRowsDataReader' to type 'Oracle.ManagedDataAccess.Client.OracleDataReader'.

Summary of the issue
Environment
.NET (Core) SDK version: 5.0
Microsoft.EntityFrameworkCore version: 5.0.3
EFCoreSecondLevelCacheInterceptor version: 2.4.1
Database: (e.g. SQL Server, SQLite, etc.) oracle 11g
Cache provider: (e.g. in-memory, redis, etc.)  in-memory

Hi VahidN;
I've got an error when I tried to update my Project from .net core3.1/EFCore3.1.8 to .Net 5.0/EF5.0.3

.Cacheable(CacheExpirationMode.Absolute, TimeSpan.FromHours(1))  <--error from this line

It  well worked on .net core3.1/EFCore3.1.8

Output:
System.InvalidCastException:“Unable to cast object of type 'EFCoreSecondLevelCacheInterceptor.EFTableRowsDataReader' to type 'Oracle.ManagedDataAccess.Client.OracleDataReader'.”

Full Stack trace:
An exception occurred while iterating over the results of a query for context type 'XH.ICIS.Models.CISDATAContext'.
System.InvalidCastException: Unable to cast object of type 'EFCoreSecondLevelCacheInterceptor.EFTableRowsDataReader' to type 'Oracle.ManagedDataAccess.Client.OracleDataReader'.
   at Oracle.EntityFrameworkCore.Storage.Internal.OracleRelationalCommandBuilderFactory.OracleRelationalCommandBuilder.OracleRelationalCommand.ExecuteReader(RelationalCommandParameterObject parameterObject)
   at Microsoft.EntityFrameworkCore.Query.Internal.SingleQueryingEnumerable`1.Enumerator.InitializeReader(DbContext _, Boolean result)
   at Oracle.EntityFrameworkCore.Storage.Internal.OracleExecutionStrategy.Execute[TState,TResult](TState state, Func`3 operation, Func`3 verifySucceeded)
   at Microsoft.EntityFrameworkCore.Query.Internal.SingleQueryingEnumerable`1.Enumerator.MoveNext()System.InvalidCastException: Unable to cast object of type 'EFCoreSecondLevelCacheInterceptor.EFTableRowsDataReader' to type 'Oracle.ManagedDataAccess.Client.OracleDataReader'.
   at Oracle.EntityFrameworkCore.Storage.Internal.OracleRelationalCommandBuilderFactory.OracleRelationalCommandBuilder.OracleRelationalCommand.ExecuteReader(RelationalCommandParameterObject parameterObject)
   at Microsoft.EntityFrameworkCore.Query.Internal.SingleQueryingEnumerable`1.Enumerator.InitializeReader(DbContext _, Boolean result)
   at Oracle.EntityFrameworkCore.Storage.Internal.OracleExecutionStrategy.Execute[TState,TResult](TState state, Func`3 operation, Func`3 verifySucceeded)
   at Microsoft.EntityFrameworkCore.Query.Internal.SingleQueryingEnumerable`1.Enumerator.MoveNext()

I don't know. It should work, because this is the signature of OracleDataReader and EFTableRowsDataReader can be converted to it.
 public sealed class OracleDataReader : DbDataReader

public class EFTableRowsDataReader : DbDataReader

I am not running Oracle and I don't have any intention to install or test it.
Just make sure you have updated all of your dependencies. It's better to run this command from the root of your solution to update all of your projects consistently:
dotnet restore
dotnet tool update --global dotnet-outdated-tool
dotnet outdated -u
dotnet restore

Hi,
We are having the same issue and we were able to narrow it down a little bit.
In our case, the .Cacheable(CacheExpirationMode.Absolute, TimeSpan.FromHours(1)) extension method is working correctly, until we use it on an query for an entity with a property that has a valueConverter (XDocument to string). On entities with 'more simple' valueConverters (stringToBool), this issue is not happening.
The failing query was working on EF Core 3.1, but since the upgrade to EF Core 5 it crashed.
I thought it was an issue with the Oracle Data Provider, but they redirected me to this issue.
Would it be possible to have another look at this?

I have the same issue after upgrading EF Core to 3.1.3 and Oracle.EntityFrameworkCore to 3.19.110.

Hi @VahidN
the last workable version is Oracle.EntityFrameworkCore 3.19.80
does there any solution exist  with this issues?
thanks

Hi @VahidN @maxborts did you guys find any fix for this issue ? the current workaround is use EF Core 3.19.80 ?

Have pretty same issue with Microsoft.Data.SqlClient.SqlDataReader.
Once I've added custom converter - Unable to cast object of type 'EFCoreSecondLevelCacheInterceptor.EFTableRowsDataReader' to type 'Microsoft.Data.SqlClient.SqlDataReader' issue appeared.
Microsoft.EntityFrameworkCore 7.0.10
Microsoft.EntityFrameworkCore.SqlServer 7.0.10
EasyCaching.Serialization.MessagePack 1.9.0
EFCoreSecondLevelCacheInterceptor 3.9.2


Fallout: V&D PNP - LK and Vehicle Page Fix, Robotics Skill Removed

Changes / Comments
A few bugs were found in the Vehicle Tab regarding Storage Space which kept disappearing whenever you inputted the number. A few of the equations for the Skills were missing LK for some reason. They were re-added.
Robotics skill has been removed and the tinkering for robots has been split between Computer Science and Engineer. Other skills can assist in this in small doses such as Blacksmith and Gunsmith.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[X] Does the pull request title have the sheet name(s)? Include each sheet name.
[X] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[X] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

This PR still contains the extra version of the sheet, the one with the old name. If you remove those files, this could be accepted. This error was correctly highlighted by the "Single Sheet Rule" check failing.

@Anduh I think it fixed it.


problematic subfigure dimensions (hight of 0)

in the subcaptions_public.jsonl file there are subfigures with problematic dimensions.
for example in line 1548 there is a subfigure with points [[34, 1.7], [34, 1.7], [35.7, 1.7], [35.7, 1.7]] so its hight is 0.

Hi Idan, sorry for the delay. Yes, I remember there are some very small boxes that are annotation errors, and you can remove these from the dataset. I think--but could be mistaken--many of these are already disregarded in the subcaption training/evaluation due to this line, which only includes a subfigure if there is a matching subcaption: https://github.com/allenai/medicat/blob/2a75c3cfbbfcc3b123851b11ca7ce209a2805633/code/subcaption/subcaption_box_dataset_reader.py#L115


Use Symbol.species?

A more canonical way to do branding in modern JavaScript is Symbol.species. Javascript only defines Symbol.species when it appears as a static in a class (or as a property on a constructor function, which is the same thing). Putting it on the instance is not defined, but allowed. Basically, this is valid JavaScript code:
type Species<U, A> = {
    "fp-ts/kind": U
    "fp-ts/type": A
}

interface HKT<U, A> {
    [Symbol.species]: Species<U, A>
}

const URI = "Test";
type URI = "Test";

class Test<A> implements HKT<URI, A> {
    [Symbol.species]: Species<URI, A>
}

No more underscores (uses namespacing with "fp-ts/" instead) and no polluting the object. Advantage is that symbols are not enumerable at runtime, and they don't show up in keyof at compile time, while still providing their function in constraints. the namespacing with "fp-ts" is optional of course, but I expect other libraries will start using Symbol.species in the future and as fantasy-land indicated, "namespace/" is kind of the canonical way to avoid collisions.

I'm not familiar with Symbol or with how Symbol.species works.
Is there a good document anywhere on Symbol.species that I can read?

The mozilla doc is pretty good and links to the ECMA spec: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Symbol/species
Basically, the trick here is that Symbol.species is only defined as a static (property on the constructor), so we are free to use it for other purposes, but you could also attach namespaced properties on the static species if you so wish I think.

Not sure how the Symbol.species may be used but we must consider its adoption (the IE story does not look that great).

Hmm, good point. There's a polyfill for IE, but maybe we should find a different spec that doesn't rely on Symbol.
As far as I can tell, Microsoft doesn't plan on supporting a bunch of new technologies with IE. I've already decided not to support Safari for any of my projects for similar reasons (people just get a page telling them to get a real browser), but I realize this is not an option for everyone.

If Symbol.species is not currently supported on Microsoft's browser, that's a conversation killer.
It's one thing for an application to decide to drop support for Microsoft stuff, quite another for a library, especially since developers might end up finding out about it after they've deployed it in a production environment and it breaks for users, because honestly, we aren't reading the freaking manual most of the time 😀

Given that TypeScript will include support for indexing via symbols (https://github.com/Microsoft/TypeScript/pull/15473), a viable (and less spec-violating) approach would be to define a custom symbol type that would then be used to brand the type application.
const type = Symbol("type");

type _ = any;

interface TypeTag<U, A> {
  readonly kind: U;
  readonly type: A;
}

interface App<U, A> {
  [type]: TypeTag<U, A>
}

abstract class Functor<F> {
  abstract map<A, B>(fa: App<F, A>, f: (a: A) => B): App<F, B>;
}

// Using _ as any makes clear our intentions about App:
// App<Container<_>, T> reads as "apply T to the partial application in Container<_>"
class Container<T> implements App<Container<_>, T> {
  [type]: TypeTag<Container<_>, T>

  constructor(public readonly get: T) {
  }
}

// In Scala this would be `extends Functor[Box[_]]` to denote higher-kind application
class ContainerFunctor extends Functor<Container<_>> {
  map<A, B>(container: App<Container<_>, A>, f: (a: A) => B): Container<B> {
    // Here we'll cast the type application to a concrete U<A> type
    const a = (container as Container<A>).get;
    return new Container(f(a));
  }
}

const f = new ContainerFunctor();
f.map(new Container(1), a => a);

This would still have the advantage of preventing access to internal properties, tooling would ignore them and, symbols being unique, would prevent naming or semantic conflicts with other libraries and conventions. Symbols are also well-supported in all modern browsers if we're not relying on a certain builtin symbol like species mentioned above.

Good idea!
Would we be better off defining a global system with a namespaced name, so we don't need a dependency library?
const type = Symbol.for("fantasy-land/type");

Not sure which namespace would be best, so I just picked fantasy-land, because it's popular. That one has the advantage of not needing a dependency to provide the symbol, as well as being unique enough.

HKT encoding is now frozen (v1.0.0)

Why not use Symbol as tags instead of string, in Option for instance?
const NoneSym = Symbol('NoneSym')
export interface None {
  readonly _tag: NoneSym
}

Comparisons would be faster, i.e. in line 126 would be return fa._tag === NoneSym

@rui-atg Perhaps because TypeScript cannot do type narrowing based on symbols.


Uncaught exception 'SpotifyWebAPI\SpotifyWebAPIException'

Hello,
I think I'm having the same issue that https://github.com/jwilsson/spotify-web-api-php/issues/29.
Here's my error:
So, when I launch index.php it asks me the authorization and when I say 'yes' I'm getting this.
I have an another page callback.php which is called once the user clicks on 'yes'.
These two pages only contain the code provided by you.
I tried to run it on both Linux and Windows :)
If you need anything, ask me !
Thanks in advance, it's really important for me to solve this problem :+1:
Audric.

Some code is appreciated :)

Here it is :)
https://github.com/AudricManaud/SpotifyAPI

Thanks!
I think your problem lies in a mismatch between the callback URI you've registered at Spotify and the one you're actually using.
In the screenshot, the URL seems to be http://localhost/promus/callback.php but in the code you posted, the redirect URI is http://localhost/callback.php. Try changing one of them, either the registered URI at Spotify or move the files so they're placed where Spotify expects them.

Owwh I'm sorry that's a previous screenshot, I corrected it and it's still doesn't work:

Alright, I've tried reproducing it but I can't find anything wrong with it.
Could you dump the whole response by adding something like
print_r($response)

on line 126 of Request.php, right after the line
curl_close($ch);

and post the result here.

Weird, it doesn't display anything at all...

Hmm, try with
var_dump(curl_error($ch));

instead. But put it before the
curl_close($ch);

line.

There it is:

I got teamviewer if you want too :)

Great, thanks!
At least we found the problem, cURL needs a local bundle of root certificates. Take a look at this answer: https://laracasts.com/discuss/channels/general-discussion/curl-error-60-ssl-certificate-problem-unable-to-get-local-issuer-certificate/replies/37017 and restart your local server and it should work.
I hope to get a better fix for this into the library.

Oh my god, it works thank you very much !
You've been so helpful to me :+1:
If you need to run some tests on my computer to solve it for everyone, just contact me :)
Audric.

Awesome!
Glad we figured it out :)

Seemed to be solved at first, but now stopped working again.

@rubenvdb Could you pull down the 0.9.0 branch, test with that code and then open a new issue with the error received.
Thanks!


no way to disable launch on startup?

Keybase GUI Version: 1.0.16-20160803190018+268b023
I can find no way to disable Keybase from launching on computer startup / account login. The normal method of auto-starting — addition in the "Users & Groups" control panel / Login Items tab, doesn't seem to be used here. Additionally, there's no preference for not autostarting.
on OSX

Open finder and press cmd+shift+g then type in ~/Library/LaunchAgents and press enter.

You will find three files in this directory that you will need to remove. (see below)

Restart computer and keybase should not be running at startup.
For more information about launchd visit: http://launchd.info/

thank you! will this also disable my ability to use keybase when i want to?

On Aug 20, 2016, at 3:45, Adam Faur<EMAIL_ADDRESS>wrote:
Open finder and press cmd+shift+g then type in ~/Library/LaunchAgents and press enter.
https://cloud.githubusercontent.com/assets/898523/17827592/fa0257b4-6644-11e6-935e-44b47441b7ae.png
You will find three files in this directory that you will need to remove. (see below)
https://cloud.githubusercontent.com/assets/898523/17827608/46d2864a-6645-11e6-9ca6-80a5a100fd43.png
Restart computer and keybase should not be running at startup.
For more information about launchd visit: http://launchd.info/ http://launchd.info/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/keybase/client/issues/3904#issuecomment-241167214, or mute the thread https://github.com/notifications/unsubscribe-auth/AGvbSAIICaLmf7KhSRRaoq1pFfb9G46Gks5qhk5EgaJpZM4Jll_r.

Very good question.  I honestly am not sure, I just tried running again and looks like it will prompt you to try and reinstall some things.  I did this myself because I figured if it didn't work next time I use it I would just reinstall, and that would be less annoying since I don't use it often.

@afaur Yes, that works.
I am guessing the developers intend to provide a "Launch at Login" option in the apps preferences. Even still, the app should use System Preferences > Users & Groups > Login Items. That way ordinary users can easily control Keybase launch behaviour.

@lukejanicke this!  Login items should always be used for this kind of thing.

It would be really great if the GUI would not pop on login as well. While I don't spend a lot of time logging out or in from restarts or otherwise... it is not always a preferable matter to let unintended viewers of your screen to know what you are running or using actively. Nor is it enjoyable having to close the keybase window upon every login. Workflow or initial login becomes a nuisance. Please provide an option to only load the status bar icon instead.

For others looking to disable key base launch options please see ( keybase help launchd )...
$ keybase launchd list
keybase.service: Running (pid=416) [Keybase]
keybase.kbfs: Running (pid=427) [KBFS]
keybase.updater: Running (pid=407) [Updater]
$ keybase launchd uninstall keybase.service
etc...

Just realized there is no option. INSTANT UNINSTALL. Let me know when your marketing strategies grow up.

Same on Debian. Of course I can delete ~/.config/autostart/keybase_autostart.desktop, but everytime I start the Keybase Client, the autostart file was created again. Quite annoying.

@ffflorian this file has following lines
# Uncomment the following line to disable Keybase autostart:
# Hidden=true

I guess uncommenting this may help.

@uapasha Thanks, good idea.

This is a problem for me too. Now that Keybase has a mobile client, I'd prefer to not have the mac client taking space on my menu bar, when I'm going to get push notifications on my phone of changes anyway.

Not sure when it changed but it is definitely in the Login Items settings as of version 1.0.34-20171027003854+8e989533a. You can close this issue.

The update here is that we've found a race-condition that sometimes prevents the setting from sticking. We'll hopefully fix it in the next release. Thanks for your patience!
cc: @patrickxb

Even after disabling Keybase in Login Items it readds itself after a restart. Very annoying.

@olivercaine on mac, please disable Keybase autostart from within the app in Settings->Advanced.


Fix Broken/Missing Links

Under the Primitive Concepts - Id - Computational Definition section the curie-spec referenced document/section does not seem to exist.

A CURIE-formatted string that uniquely identifies a specific instance of an object within a document.

The Identifier Construction also has the same missing section in the W3C CURIE link below

A W3C CURIE format has the form:

@reece this should be fixed within the PR #147. I added a subsequent commit to fix this issue.


Add support for nushell

Description
Currently, neofetch looks like this for nushell users

This fixes the issue so that it now looks like this

What does the full nu --version output look like?

I'd like to see this fixed! Although this PR doesn't seem to fix this for me either. Grep on my system doesn't match on \d, so it doesn't output the version.
Here is the full output:
> nu --version
───┬────────────────────┬────────────────────────────────────────────────────────────────────────────────────
 # │      Column0       │                                      Column1
───┼────────────────────┼────────────────────────────────────────────────────────────────────────────────────
 0 │ version            │ 0.37.0
 1 │ build_os           │ macos-aarch64
 2 │ rust_version       │ rustc 1.55.0
 3 │ cargo_version      │ cargo 1.55.0
 4 │ pkg_version        │ 0.37.0
 5 │ build_time         │ 2021-09-14 19:45:30 +00:00
 6 │ build_rust_channel │ release
 7 │ features           │ clipboard-cli, ctrlc, dataframe, default, rustyline, term, trash, uuid, which, zip
 8 │ installed_plugins  │
───┴────────────────────┴────────────────────────────────────────────────────────────────────────────────────

It would likely be best for nu itself to process the version from the table, since it works with this format natively. It should be as simple as nu -c 'version | get version'. I can test or submit the change if need be.

Just to verify, the changes look great and work great for me!

Thanks!


Make Schemas Match Lucidchart Diagram

I went through and compared HydroServer's models with our LucidCharts diagram to make sure everything matches and found the following discrepancies. For any changes, we'll need to make sure the backend, frontend, and Lucidchart are all the same:

[ ] DataSource.URL -  This is the same everywhere, but the other tables use 'link' as the field name when referring to a web URL. Do we want to update this?
[ ] ThingAssociation.followsThing - This should be deleted everywhere since we're not using it anymore.
[ ] Observation.validBeginTime & validEndTime - present in Lucid but not the db. But there's a "suggest not using" note in Lucid. Do we care that the fields are missing in the db?

Created an issue cleaning up follower functionality: https://github.com/hydroserver2/hydroserver/issues/170


Add extra debugging information for failed stageout

Impact of the new feature
WMAgent
Is your feature request related to a problem? Please describe.
As discussed yesterday in the CompOps meeting and in a private thread with Stephan, we could dump a few other extra information to the job log. Otherwise the SS team and the site admin might have a hard time debugging stage out issues and finding the root cause of the problem.
Describe the solution you'd like
Fetch and add to the job log file the following information:

the absolute path to the binary, e.g.:

amaltaro@vocms0192:/data/amaltaro/Agent128_Val $ which xrdfs
/usr/bin/xrdfs

the content of LD_LIBRARY_PATH (in a user friendly format), e.g.:

cmst1@vocms0193:/data/srv/wmagent/current $ echo $LD_LIBRARY_PATH
/data/srv/wmagent/v1.2.8/sw/slc7_amd64_gcc630/cms/wmagent/1.2.8/lib:/data...

and maybe the list of dependencies for the binary, e.g.:

amaltaro@vocms0192:/data/amaltaro/Agent128_Val $ ldd /usr/bin/xrdfs
	linux-vdso.so.1 =>  (0x00007ffdbfde5000)
	libpthread.so.0 => /lib64/libpthread.so.0 (0x00007f9c71acf000)
	libXrdCl.so.2 => /lib64/libXrdCl.so.2 (0x00007f9c717a7000)

Anything else that is important and not listed above?
Note that this information is only supposed to be fetched for permanent stageout failures (via subprocess calls).
Describe alternatives you've considered
no clue
Additional context
none

This has already been resolved with Andrea's PR: https://github.com/dmwm/WMCore/pull/12081


double is not supported in Complex type

when create table using double type in  complex type, creating will failed.
i have raised a pr to handle this.

pr is merged. close it.


add signing unit test

add signing unit test

Hello @jsun28! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file packaging/package/jar_jdk_signer.py:

Line 83:1: W293 blank line contains whitespace

In the file packaging/tests/test_jar_signer.py:

Line 3:32: W291 trailing whitespace
Line 10:20: E225 missing whitespace around operator
Line 15:1: W293 blank line contains whitespace
Line 17:5: E303 too many blank lines (2)
Line 19:80: E501 line too long (81 > 79 characters)
Line 20:80: E501 line too long (89 > 79 characters)
Line 25:80: E501 line too long (81 > 79 characters)
Line 26:80: E501 line too long (89 > 79 characters)
Line 31:80: E501 line too long (82 > 79 characters)
Line 32:80: E501 line too long (89 > 79 characters)
Line 38:80: E501 line too long (89 > 79 characters)
Line 40:1: W391 blank line at end of file

In the file packaging/tests/test_package.py:

Line 26:9: E303 too many blank lines (2)
Line 26:80: E501 line too long (135 > 79 characters)
Line 26:134: E202 whitespace before ')'

Hello @jsun28! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:
* In the file [`packaging/package/jar_jdk_signer.py`](https://github.com/tableau/connector-plugin-sdk/blob/b4f0e9aa2dfee4c57e5bbfb734c07928a5ba33a1/packaging/package/jar_jdk_signer.py):

Line 83:1: W293 blank line contains whitespace

* In the file [`packaging/tests/test_jar_signer.py`](https://github.com/tableau/connector-plugin-sdk/blob/b4f0e9aa2dfee4c57e5bbfb734c07928a5ba33a1/packaging/tests/test_jar_signer.py):

Line 3:32: W291 trailing whitespace
Line 10:20: E225 missing whitespace around operator
Line 15:1: W293 blank line contains whitespace
Line 17:5: E303 too many blank lines (2)
Line 40:1: W391 blank line at end of file

* In the file [`packaging/tests/test_package.py`](https://github.com/tableau/connector-plugin-sdk/blob/b4f0e9aa2dfee4c57e5bbfb734c07928a5ba33a1/packaging/tests/test_package.py):

Line 26:9: E303 too many blank lines (2)

Line 26:134: E202 whitespace before ')'

These are worth fixing.


Add dependencies for golang.org/x/net

Fix #154

I was able to build the container image using the Dockerfile in this branch.


4.x: MP quickstart archetype has unnecessary metrics dependencies

Helidon Version: 4.0.0-RC1
Helidon MP

Problem Description
Generate a 4.0.0-RC1 MP quickstart using the CLI and the pom.xml has:
        <dependency>
            <groupId>io.helidon.metrics</groupId>
            <artifactId>helidon-metrics</artifactId>
            <scope>runtime</scope>
        </dependency>

And
        <dependency>
            <groupId>io.helidon.metrics</groupId>
            <artifactId>helidon-metrics</artifactId>
        </dependency>

I don't think it should have either of these.

I believe Joe is correct here.
At one point we planned to use bundles such as helidon-metrics but, at least for now, we have moved away from that.
For MP, depending on helidon-microprofile-metrics and org.eclipse.microprofile.metrics:microprofile-metrics-api should be what we want for API dependencies. The MP QuickStart depends on the full Helidon MP bundle which includes those already.
And (as we've discussed earlier), these need to be present also:
        <dependency>
            <groupId>io.helidon.webserver.observe</groupId>
            <artifactId>helidon-webserver-observe-metrics</artifactId>
            <scope>runtime</scope>
        </dependency>
        <dependency>
            <groupId>io.helidon.metrics</groupId>
            <artifactId>helidon-metrics-system-meters</artifactId>
            <scope>runtime</scope>
        </dependency>


multiple definitions of SoxEffect.

SoxEffect is defined both in
https://github.com/pytorch/audio/blob/00d38203e401b8d9472a8f8394a10e2c309be02c/torchaudio/torch_sox.h#L50-L54
and
https://github.com/pytorch/audio/blob/00d38203e401b8d9472a8f8394a10e2c309be02c/torchaudio/torch_sox.cpp#L73-L77

torch_sox.h can be included into torch_sox.cpp so that SoxEffect needs to be defined
only once in the header.

Good catch. Do you want to open a PR for this?


add Consul mocks via mockery and reinstate the Environments page smoke test

This was originally part of #11.
The make generate task effectively requires an external dependency, mockery, which can be grabbed via go get github.com/vektra/mockery/v2. This is required to keep the mocks in sync with the interfaces they target.
The introduction of such a dependency might constitute a case for starting to think about providing a Dockerfile, to perform development and build task without having to install packages on a development box.

can you just update the README.md with the go get github.com/vektra/mockery/v2 cmd and/or the Makefile check for it?

can you just update the README.md with the go get github.com/vektra/mockery/v2 cmd and/or the Makefile check for it?

done!


Update Monaco dependency in the example

textDocument/documentLink requests no longer work in the example because the Monaco bridge code uses 0.17.x APIs but the example still depends on 0.15.5.

thank you @rcjsuen !

@akosyakov No worries, thank you for reviewing!


ActionView::Template::Error (undefined method `protect_against_forgery?'

Using Rails 5 beta3 API mode.
Getting this error when trying to do a get request to /users/sign_in route

Hey! I'm having the same problem
How did you fix this?

+1. How did you fix this?

I got this error

protect_against_forgery? is defined in ActionController::RequestForgeryProtection module. This module is included by ActionController::Base.
By default, DeviseController extends 'ApplicationController'.

ApplicationController usually extends ActionController::Base.
On Rails5 API mode, however, ApplicationController extends ActionController::API, which does not include ActionController::RequestForgeryProtection.
In order to fix, insert config.parent_controller = 'ActionController::Base' into config/initializers/devise.rb.

How you configure to all others actions to respond to JSON instead HTML when you changed to config.parent_controller = 'ActionController::Base'?

@rodriggochaves
I don't longer use devise with Rails5 API mode, so not remember well what I did...
I recognize that config/initializers/devise.rb effects only modoules which relate to devise,
so I think config.parent_controller = 'ActionController::Base'  not change other controller's work.
if you will use Rails5 API mode, I recommend to use devise_auth_token. I already use it.


Tweaks weed node damage threshold

About the PR
closes #2375
Why / Balance
requested tweak, for reference it takes two hits with a bayonet knife to destroy nodes now.
Technical details
increased the threshold from 5 to 30 to destroy nodes; takes 2 hits with a standard bayonet knife and more with entities that have lower damage output. Similar to cm13.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
N/A
Changelog
:cl:

tweak: increased the required damage threshold to destroy weed nodes

I need to ask Skarlet about this but from testing in CMSS13 the weed nodes do take one hit to destroy with a bayonet for me, not two


Trying to get UUP for Windows 10 ARM, gets exception error

BEFORE SUBMITTING YOUR ISSUE, PLEASE LOOK AT THE PINNED ISSUES AND USE THE SEARCH FUNCTION TO MAKE SURE IT IS NOT ALREADY REPORTED. ALWAYS COMMENT ON AN EXISTING ISSUE INSTEAD OF MAKING A NEW ONE.
Describe the issue
Was trying to download the UUP Windows 10 ARM, and gets an exception error.
Downloading converters...
Exception caught
Exception: [download_helper.cc:562] errorCode=1 Failed to open the file files/converter_multi, cause: File not found or it is a directory
We have encountered an error while downloading files.
Looking it up, looks like it's an unknown error.
I tried using Windows, Ubuntu, and a virtural macOS using Monterey, still gets the issue.
Configuration

UTM Version: NA
macOS Version: 13.0
Mac Chip (Intel, M1, ...): M1

Crash log
N/A
Debug log
N/A
Upload VM
N/A
Picture

I haven’t tried running it on Linux before, can you try on macOS?

I haven’t tried running it on Linux before, can you try on macOS?

I did. Still gives me the error

I also found a log on Ubuntu with this:
2022-10-15 15:15:37.176959 [INFO] [Context.cc:179] <<--- --- --- ---
2022-10-15 15:15:37.177043 [INFO] [Context.cc:180]   --- --- --- ---
2022-10-15 15:15:37.177068 [INFO] [Context.cc:181]   --- --- --- --->>
2022-10-15 15:15:37.177078 [INFO] [Context.cc:182] aria2 1.36.0
2022-10-15 15:15:37.177086 [INFO] [Context.cc:183] gcc 11.2.0
built by  aarch64-unknown-linux-gnu
on        Aug 21 2021 15:26:28
2022-10-15 15:15:37.177092 [INFO] [Context.cc:184] Linux 5.15.0-41-generic #44-Ubuntu SMP Thu Jun 23 11:20:13 UTC 2022 aarch64
2022-10-15 15:15:37.177097 [INFO] [Context.cc:185] zlib/1.2.11 libxml2/2.9.12 sqlite3/3.36.0 GnuTLS/3.7.1 nettle GMP/6.2.1 c-ares/1.17.1 libssh2/1.10.0
2022-10-15 15:15:37.177100 [INFO] [Context.cc:186] Logging started.
2022-10-15 15:15:37.177105 [INFO] [SocketCore.cc:1659] Checking configured addresses
2022-10-15 15:15:37.177214 [INFO] [SocketCore.cc:1710] Not considered: <IP_ADDRESS>
2022-10-15 15:15:37.177224 [INFO] [SocketCore.cc:1707] Found configured address: <IP_ADDRESS>
2022-10-15 15:15:37.177228 [INFO] [SocketCore.cc:1710] Not considered: ::1
2022-10-15 15:15:37.177232 [INFO] [SocketCore.cc:1707] Found configured address: fdb2:2c26:f4e4:0:21c:42ff:fee4:7043
2022-10-15 15:15:37.177256 [INFO] [SocketCore.cc:1710] Not considered: fe80::21c:42ff:fee4:7043%enp0s5
2022-10-15 15:15:37.177261 [INFO] [SocketCore.cc:1714] IPv4 configured=1, IPv6 configured=1


:sparkles: Sign the agent certificate by the global hub operator

Summary
Related issue(s)
Prerequisites:

Since the cert sign flow is difficult to cover on unit and integration tests. We will enable the built-in Kafka in e2e to cover it: https://github.com/stolostron/multicluster-global-hub/pull/1023

Reference: https://issues.redhat.com/browse/ACM-12388
Fixed: https://github.com/stolostron/multicluster-global-hub/issues/950

/test test-e2e

https://github.com/openshift/release/pull/54606
/test test-e2e

/cc @clyang82 @ldpliu


add pkg/errors

DRAFT:
Adding errors wrapper

@whomobile Hello kim, i have a propose about log. want to make it more

If there is a one way to find all user visible logs(debug/info/warn/error), we could make llm to generate more readable/consistent logs.  Lat time when I debug, there were couple of variants.
When user face issues, they should able to report/fix.

Yes, you can find log file in ~/.kubevpn/daemon.log.

On Jul 28, 2024, at 03:18, Man Kim @.***> wrote:
If there is a one way to find all user visible logs(debug/info/warn/error), we could make llm to generate more readable/consistent logs. Lat time when I debug, there were couple of variants.
When user face issues, they should able to report/fix.
—
Reply to this email directly, view it on GitHub https://github.com/kubenetworks/kubevpn/pull/90#issuecomment-2254232235, or unsubscribe https://github.com/notifications/unsubscribe-auth/AICAMA6VAMD4ZKDAIHRDRRLZOPW7LAVCNFSM6AAAAAA7CIFBWWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJUGIZTEMRTGU.
You are receiving this because you commented.

Current my design is, when start connect/proxy/dev/clone command, daemon process will set logrus.out to conn, during this command executing, we can use logrus.info/debug/warning http://logrus.info/debug/warning to print to user terminal. After command finished, set logrus.out to daemon.log file.

On Jul 28, 2024, at 03:18, Man Kim @.***> wrote:
If there is a one way to find all user visible logs(debug/info/warn/error), we could make llm to generate more readable/consistent logs. Lat time when I debug, there were couple of variants.
When user face issues, they should able to report/fix.
—
Reply to this email directly, view it on GitHub https://github.com/kubenetworks/kubevpn/pull/90#issuecomment-2254232235, or unsubscribe https://github.com/notifications/unsubscribe-auth/AICAMA6VAMD4ZKDAIHRDRRLZOPW7LAVCNFSM6AAAAAA7CIFBWWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJUGIZTEMRTGU.
You are receiving this because you commented.


Organize tests (unit tests vs integration)

Description of issue
Currently, integration tests and unit tests are mixed together in the same project. Separating unit test cases and integration tests into their own directories or projects would improve maintainability.
Additional context
Provide any additional information

Closing- not needed


don't sync svcaccounts if namespace is terminating

https://github.com/rancher/rancher/issues/12266
https://github.com/rancher/rancher/issues/12497

LGTM


Initial split of tokenization transform into ray and python

Why are these changes needed?
Continuing work to split transfom runtimes.  This time it is universal/tokenization
Related issue number (if any).
Issue #185

Sorry, I do not like this whole thing. Its way to complex and cumbersome for users


Custom timer feature

Feature suggestion:
Users can adjust the timer according to their preference.
Rationale:
Users should have flexibility to increase the breaks or the working time.

Completed in contrihub


⚠ GitLab Server has degraded performance

In 69ac57e, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11862 ms

Resolved: GitLab Server performance has improved in 0413fe4.


Changing disparity shift in c++

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
D435

Firmware Version
5.9.14

Operating System & Version
Win 10

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi/ NVIDIA Jetson / etc..

SDK Version
LibRealsense SDK 2.0

Language
c++

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
Hello I'm trying to change disparity shift via code in my project but couldn't find an efficient way so far.
What I want to do is to change disparity shift to the desired value such as 100 if needed.
I know that disparityShift parameter is in the structure STDepthTableControl but I cannot figure out a way to get access to this structure in my project and modify that value so that i can get depth stream with modified diaparityShift value.
Thanks

Something like:
using namespace rs2;
context ctx;
auto dev = ctx.query_devices().front().as<advanced_mode>();
STDepthTableControl table = dev.get_depth_table();
// modify table
dev.set_depth_table(table); // write changes

If you are using pipeline:
pipeline p;
auto profile = p.start();
auto dev = profile.get_device().as<advanced_mode>();
// same as before

@dorodnic  It works well and I really appreciate your help, thanks.
One more question, Is there any way that I can handle presets AND the small manipulation like disparity shift at the same time?

Most presets will reset all advanced mode controls, but then you can change the controls back.
You can also download the preset you want in JSON form and remove the parts you wish to control yourself. In general, switching presets during streaming (without stop / start) is not recommended (not all registers can be modified without stop/start). In the Viewer we require stop / start because of it.

@MechanicJay7
Does this answer your question?

Feel free to open more issues if you have more questions.


GGXF format documentation

Attached is a very preliminary format specification just to act as a straw man for further discussion. I would expect much of it to be changed, some radically. If potentially useful, probably needs to be transferred into a communal editable document.
GGXF v0.1 spec 2020-06-13.docx

Thanks for the draft. Below are a few observations:
Requirement 2
This requirement suggests to use −360 <= λ <= 0 or 0 =< λ =< +360 longitude ranges when the grid crosses the 180° meridian. Another approach could have been to keep the [-180 … +180] range but specify that the [λ₁ … λ₂] range is interpreted as "start from λ₁, then move toward increasing longitude values until we reach λ₂" where possibly λ₂ < λ₁ if the grid crosses the 180° meridian:

This approach is used by ISO 19115-1 for EX_GeographicBoundingBox and is also used in EPSG database. But I presume that the [−360 … 0] or [0 … +360] approaches are preferred in the context of this standard because they make easier to interpolate in the grid, is that right?
I ask because having 3 different kinds of longitude range complexifies a little bit the mixing of different data on the same map, while a single [-180 … +180] range with above interpretation complexifies interpolations in the grid. So it is a matter of choosing where to put complexity (I'm not suggesting either side, just seeking to make explicit which side we choose to make easier).
Metadata
In addition of extent metadata, I would suggest a CI_Citation metadata with the URL where the grid can be downloaded. A user may have a copy of a copy of a copy and lost the original provenance.
File no data flag
Is the IEEE 754 representation of floating point numbers ubiquitous enough for recommending the use of NaN (Not-a-Number)?
Parameter values
The requirements saying "The grid data in a GGXF file shall be a sequence of space-separated values" and "The end of the file shall be terminated by an EOF character" suggest a text format. A binary format such as NTv2, NetCDF or GeoTIFF would have no separator (because all values are of fixed length).
Except for above-cited two requirements, current specification seems format-neutral in current state.
If the above-cited EOF character is an ASCII control code ^C, ^D or ^Z (depending on the OS), they are not needed any more. They are vestige of a time when the CP/M system (even older than MS-DOS) didn't knew the exact size of a file (it knew only its multiple of 128).

A comment on #1 saying "Higher resolution may not always imply higher accuracy" raises a new question for me: does the operation_accuracy parameter applies only to values in the grid (i.e. gives accuracy only at the exact node positions, where no interpolation are needed), or does it applies also to the result of applying interpolations between nodes? I ask because the purpose of a denser but less accurate grid is not clear to me, unless that grid provides more accurate interpolated values (because of shorter distances from the nodes).

On the list of interpolation methods (bilinear, etc.), should "Geocentric translation by Grid Interpolation" (EPSG::1087) be added? It is bilinear interpolation but applied on Tx, Ty, Tz geocentric translation terms instead than on latitude and longitude values.

Intended to be provided for. In there in table C.2 as geocentric translations and this is cross-referenced from Cartesian 3D offsets.
But the IGN method also has a parameter (Standard CT code) to avoid iteration in the reverse. EPSG 1087 formula covers cases with and without this. But maybe these two cases could have separate provision.


Errors in the API reference

I've been using the official reference to generate some Metadesk bindings for another language, and I've found a few issues through that process:

MD_PushNewReference is missing the arena parameter.
arg_string in MD_TagArgFromString is listed as type int, when it should be MD_String8.
MD_MakeDetachedError is listed in the reference, but does not exist at all in the source.
The reference lists a string_hash field on MD_Node that does not exist in the source.

(I'm sure this is not exhaustive, but this is what I've found so far.)

Fixed, thanks!


make sure np.random.seed can be set in one place globally

right now, it needs to be set within .fit()

I added this to version 0.1 milestone as it seems critical to reproducibility of our results. We can add an argument to GLM called random_state or seed which gets used by all the methods in GLM. What do you think?

Yeah, lets add random state to GLM

@pavanramkumar I can't fix it without knowing first the reasoning behind it.
It seems that it is used for ensuring reproducibility but I don't know where and therefore making it a parameter of GLM may not fix the problem since the seed setting is also here, which is really strange

@daniel-acuna setting it outside the class alone was not sufficient to produce reproducible results. that's why I had to set it inside the fit function as well. i thought if we accept RandomState as a parameter, it can be used to set the seed within each function that uses np.random


Added AbstractFloat and AbstractString.

Closed by https://github.com/JuliaLang/julia-vim/commit/4b5ffe05d59dfe13ea25c6266f7c4d2617758930


Create logo for extension

Something with a file and then something that represents "automatically opening"

Closed in #26


Move metadata fetch to lazy reader

Does this PR need a docs update or release note?

[ ] :white_check_mark: Yes, it's included
[ ] :clock1: Yes, but in a later PR
[ ] :no_entry: No

Type of change

[ ] :sunflower: Feature
[ ] :bug: Bugfix
[ ] :world_map: Documentation
[ ] :robot: Supportability/Tests
[ ] :computer: CI/Deployment
[ ] :broom: Tech Debt/Cleanup

Issue(s)

https://github.com/alcionai/corso/issues/3998

Test Plan

[ ] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

Closing this in favor of a simpler fix here - https://github.com/alcionai/corso/pull/4001


When clicking on a new date the previous clicked date decorator is still remaining there

Thanks for this library.... Is there a way to remove the previous clicked date decorator ? When I am clicking a new date the previous dates background is not changing

Can anyone please help???

The library does not handle decoration. You have to call notifyDateChanged() on the previous date and remove the decoration yourself.

ok


Text alignment

New Mockup
Please check assets/css/global.css, design-assets folder images, and figma files for updated style guides. Any new pull request should use the updated color, spacing, and sizing variables.
No Bootstrap or jQuery
Please refrain from adding Bootstrap or jQuery to the site. These pr's will be closed. They overwrite global styles. Vanilla CSS and JS is way better now anyways 😊
New Task:
What is the task
at the "OUR CLIENT LOVE US BECAUSE" section the text isn't aligned properly to the left
Who is the task for?

[x] I created this task for me to complete, I will create a PR soon.
[ ] I created this task for someone else to claim.
[ ] I think this is a task that multiple people can do

Confirm all of the following:

[x] I have looked at all the issues currently available and this does not exist or overlap with another ticket

⚠ PLEASE ONLY SUBMIT A PR FOR THIS TASK IF THIS TASK WAS AVAILABLE FOR YOU TO CLAIM

text-align: left is not what we want there, it is centered correctly on the main version of the site.


Issue 12: Create list of services with name. protocol, host, port and path

Added -l flag to include list of discrete services in the generated json file. Reworded help output. Fixed a bug or two.

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


empty cloud credential fields in nodetemplate config if credID is set

we shouldn't store cred field data in nodetemplate config if credID is provided,
for instance - on nodetemplate edit of upgraded setups
https://github.com/rancher/rancher/issues/18205

LGTM


HookEmit  [DO NOT MERGE]

#365

High Level Overview of Change

Context of Change

Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (non-breaking change that only restructures code)
[ ] Tests (you added tests for code that already exists, or your new feature included in this PR)
[ ] Documentation update
[ ] Chore (no impact to binary, e.g. .gitignore, formatting, dropping support for older tooling)
[ ] Release

API Impact

[ ] Public API: New feature (new methods and/or new fields)
[ ] Public API: Breaking change (in general, breaking changes should only impact the next api_version)
[ ] libxrpl change (any change that may affect libxrpl or dependents of libxrpl)
[ ] Peer protocol change (must be backward compatible or bump the peer protocol version)

hopefully we'll get this into the release after next

hopefully we'll get this into the release after next

Due to waiting for the probability of a deterministic build, I don’t add enough tests at this time.


Changed Modal to align with new styles (WIP)

Removed unused library from ScrollShadow;
What was implemented/already was in this component:

Modals 150px from the top of the screen
Modal height determined by the content
Scrolling occur as needed based on the users viewport
The buttons separated from the content by a dividing line that's 30px from the last bit of content
The buttons 15px from the dividing line and from the bottom edge of the modal
The bottom area with the action button should become fixed

What was skipped:

Modals use primary and secondary buttons
Modal buttons are right aligned
Primary action is farthest to the right
Modal height should never go beyond 150px from the bottom of the viewport
If modal has a cancel/close secondary button, we do not show an ‘x’ in the top right corner

To implement all related to the buttons, I think we need to have one of the variants:

Allow only to use the specific types of the buttons (to do so, we should allow send only the array of buttons, or buttons wrapped by <React.Fragment />)
Create new component Dialog which will deal with buttons
Deal with buttons locally in the project where you are using SCL

PR Checklist
Check these before submitting your pull request:

[x] Visual and behavioral components are separated
[x] Exported components are documented with examples
[x] Props have JSDoc comments
[x] All relevant visual sub-components can be overridden via props
[x] Any extra props are spread into the outermost rendered element

Changes to Existing Components
Changed styles of Modal component regarding requirements

I think we can rely on users to only use primary / secondary buttons, no enforcement needed.
I do think a layout component to do the right alignment and proper spacing would be beneficial. Would Horizontal maybe be helpful here?

From the new changes:
I've added 2 flags:
withCancelButton and withCloseButton, the only difference in the text they provide to button and both of them trigger onClose function
The reason is that we need to remove X icon in case if cancel or close button was provided, so I should have been handled this
Also, I've changed the CloseButton to CloseIcon because I think this the name this prop should have, most of the time we would render icon in the right corner, not button and from onClose={onClose} to onClick={onClose}, because I don't think that this is clear for developer, that he should provide onClose prop in case if he would use custom CloseIcon


[SPARK-37732][SQL][TEST] Improve the implement of JDBCV2Suite

What changes were proposed in this pull request?
When I reading the implement of JDBCV2Suite, I find we can improve the code by define checkSchemaNames and checkFiltersRemoved.
Why are the changes needed?
Improve the implement of JDBCV2Suite
Does this PR introduce any user-facing change?
'No'. Just update the test code.
How was this patch tested?
Exists test.

Test build #146552 has started for PR 35010 at commit 10463cf.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/51027/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/51027/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/51027/

Test build #146552 has finished for PR 35010 at commit 10463cf.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146552/

ping @huaxingao cc @cloud-fan

thanks, merging to master!

@cloud-fan @huaxingao Thank you for review.


Fix token on job reload in dev mode

This bug prevented reloaded agents from publishing transcriptions and from updating room metadata.
Cause: The SFU does more than set video grants when generating tokens: https://github.com/livekit/protocol/blob/main/agent/token.go and it has apis that are not available in the client library. (i.e. SetKind)
Fix: In reloaded jobs now we simply reuse the previous token. Could be some token ttl issues but I think that's the lesser of two evils.

I think we could add https://github.com/livekit/protocol/blob/32c577d805ed1c885a4fa3fff3fc37e10f920111/auth/accesstoken.go#L60 to our Python SDK on livekit-api? wdyt?

I think we could add SetParticipantKind to our Python SDK on livekit-api? wdyt?

I'm not sure. I wasn't sure of the reason we rely on kind in the SFU even though video grants has an agent property. But generally, I kind of like just having the SFU be the source of truth for token generation. It's reasonable to assume that there will be more/different permissions in the future that breaks this again.
We should probably refresh the token periodically but I'm leaning towards reusing SFU generated tokens. Another approach, since the agent has the API Key/Secret. Is just copying the fields of the old jwt into a new one with a new ttl. Wdyt?

We should probably refresh the token periodically but I'm leaning towards reusing SFU generated tokens. Another approach, since the agent has the API Key/Secret. Is just copying the fields of the old jwt into a new one with a new ttl. Wdyt?
Copying the JWT data into a new one seems like the best solution in this case.


Broken requirements.txt

Hello,
Current requirements.txt produce errors when installing with pipenv on python 3.9.5.

SQLAlchemy is not installable sqlalchemy==1.3.0,>=0.9.0,>=1.3.2
htmlentities has been moved to html.entities in python 3
markupsafe is too old, so its setup.py has broken imports from setuptools

Can you please bump the versions of your dependencies?

** replace requirements.txt with this and it will successfully install
alembic==1.0.0
banal==0.3.7
chardet==3.0.4
Click==7.0
dataset==1.3.2
Flask==1.0.2
frida==12.2.6
htmlentities
itsdangerous==0.24
Jinja2==2.10.1
Mako==1.0.7
MarkupSafe
normality==0.6.1
python-dateutil==2.7.3
python-editor==1.0.3
six==1.11.0
SQLAlchemy
termcolor==1.1.0
Werkzeug==0.15.3

alembic==1.0.0
banal==0.3.7
chardet==3.0.4
Click==7.0
dataset==1.3.2
Flask==1.0.2
frida==12.2.6
htmlentities
itsdangerous==0.24
Jinja2==2.10.1
Mako==1.0.7
MarkupSafe<2.1
normality==0.6.1
python-dateutil==2.7.3
python-editor==1.0.3
six==1.11.0
SQLAlchemy
termcolor==1.1.0
Werkzeug==0.15.5


python311Packages.uqbar: 0.7.0 -> 0.7.1

Changelog: https://github.com/josiah-wolf-oberholtzer/uqbar/releases/tag/v0.7.1
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 288760 run on x86_64-linux 1

2 packages failed to build:

python312Packages.abjad
python312Packages.abjad.dist

6 packages built:

python311Packages.abjad
python311Packages.abjad.dist
python311Packages.uqbar
python311Packages.uqbar.dist
python312Packages.uqbar
python312Packages.uqbar.dist

abjad will get Python 3.12 support with the next release.


Blog for Hacktoberfest : Traversing the realms of Quantum Network

Checklist

Please note that in case of below checklist not updated accordingly, the maintainers have the right to immediately close your pull request.

[x] You created a branch from dev branch.
[x] You PR is raised on dev branch and not onmaster.
[x] Blog follows Blog Guidelines.
[x] You have read the Contribution Guidlines before creating this PR.
[x] Checked for any duplicate blog entry.
[x] Checked for all grammatical errors in the blog.
[x] You have not published the blog content anywhere else on the internet on platforms like medium, dev.to etc.
[x] The content is not copy-pasted from anywhere else on the Internet.

Description of change

Author Details for Blog Post Entry

id: Devansh Markan
bio: I am a Student and Computer Science Enthusiast
github: https://github.com/markandevansh99
linkedin: https://www.linkedin.com/in/devansh-markan-65940817b/
medium: https://medium.com/@markandevansh99

@puneetsingh24 ...Can you please review it??

@puneetsingh24 ..is there anything that I need to modify?


Pulumi IaC support

What is your feature idea?
Kubefirst currently natively supports Terraform as its IaC engine. I would love to see Pulumi offered as an alternative, especially since I migrated most of my current infra stacks from TF to Pulumi.
Note that Pulumi has native support for migrating Terraform HCL to Pulumi source code, which may be useful if this is integrated.
Why is it needed?
It would be nice to have pluggable architecture options for other parts of the Kubefirst ecosystem (IaC included, naturally), much like the option of GitHub and GitLab feature support (CI runners, container registry, etc.).
Is this missing feature preventing you from using kubefirst?

[X] Yes

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks for the feature suggestion @coopbri!
For now there is no plan to offer Terraform alternatives, but we will use this issue to gage the community interest. Note that with the latest license changes from HashiCorp, it does not affect Kubefirst, nor our community so far, but we are keeping our eyes on the situation, as it may be one reason we move away or offer Terraform alternatives.


Media queries: change to mobile first & unify breakpoints

We have 768 and 640 css/js breakpoints, we need to unify them in the codebase.
We use max-width, let's use min-width to follow the mobile-first approach

@vault-developer
MR for this issue
https://github.com/vault-developer/event-loop-explorer/pull/24


Add possibility to attach custom logger

It can be used for messages logged in the events.

Now, we can add custom logger implementation during object creation as follows:
new NetworkEvents(context, bus, logger);

If we don't pass it, library will use default logger, which is NetworkEventsLogger. It logs debug messages to the LogCat.


class name been added multiple times as you scroll

Hi, thanks for the awesome script.
Btw I found that script keeps on adding the class name as you scroll up down. maybe need a condition to check class availability?

Thanks

hmm, that's weird. the lib removes all event names after finishing the animation. can you please share your code?

I'll share mine later, but you can check from your demo as well for infinite pulse

hmm yeah sure, your code can help fixing it faster though. thanks in advance.


move string-based PEM loading to callbacks

Note - PR is to branch opened for #283
Suggestion to move string-based PEM loading to callbacks, which will enable other functionality in callbacks too - different cert formats, private keys with passwords, load from external store, etc

Odd to be getting a test failure here - https://github.com/nats-io/nats.net.v2/actions/runs/7184322773/job/19565121688?pr=284#step:7:138
@mtmk is there anything special that needs to be done timing-wise - like wait for server shutdown?

Odd to be getting a test failure here - https://github.com/nats-io/nats.net.v2/actions/runs/7184322773/job/19565121688?pr=284#step:7:138
@mtmk is there anything special that needs to be done timing-wise - like wait for server shutdown?

it looked like an intermittent server start error. Other versions passed so, not sure. Running again will have a closer look if it persists.


Model saving and loading

Add ability to save and load models

I was able to save a model into a file but I can't seem to figure out how to load it next time application starts.
SAVE:
learningModel = Learner.Learn(data, .80, 1000, generator);
learningModel.Model.Save(modelFileName);
LOAD: (not working, throwing NullRefException)
learningModel = new LearningModel();
learningModel.Model.Load(modelFileName);

I'm stuck on the same.
Saving works fine, but I don't know how to load it.

@PippoAe I saw that recently @sethjuarez committed some code to better support serialization for NaiveBayes. Have you tried with the latest version ? Which generator did you use ? Have you seen this issue, it seems that after loading a model you still need to initialize the Descriptor.
If that doesn't solve your issue I would suggest you to open a new issue, including some code and steps to reproduce your issue.

Hi rold!
Thanks for the fast answer.
I now worked around the error.
I'm still using "model.Save(modelPath);" to save the TreeDecisionModel the learner has after learning.
To load I now use:
DecisionTreeModel model = (DecisionTreeModel)numl.Utils.Xml.Load(modelpath, typeof(DecisionTreeModel));
This returns me a working model again that I can use .Predict() on.

Yay!!


bump aiobotocore to 2.4.0

If welcome, I'm happy to provide a PR :) This is currently failing some dependency resolvement between boto3, botocore, metaflow and s3fs, whereas bumping here seems to be the easiest fix (and waiting for it helped in the past).

Please do PR


[BUG] Blackjack - Win after 21 doesn't work

Need to fix Blackjack, if win is 21

Need to update user money

Need to update balance after user's value is equal to 21

Fixed


Dead code used repeatly

The body of this function is marked DEAD, but this function is called multiple times throughout the code base.
Should this piece of code be revived? @clyne
https://github.com/NCAR/VAPOR/blob/33109f31b22d87ca520cd55169f65028f3a03300/apps/vaporgui/StartupEventRouter.cpp#L464-L473

Resolving this issue could potentially resolve issue #63

Lower its priority because that's a deprecated class.


Fix to only remove specified ks output files

Delete all kilosort output files except continuous.dat and -input.json

replaced with #128


Run cargo test on generated code as part of stubgen tests.

Needs a new trustfall and trustfall_core release, because the executed cargo test needs to be able to access the invariant-checking functionality.

Superseded by #407.


Is it possible to improve the model's performance by switching the base model to SDXL?

Hi,
First of all, thank you for your great work!
During my testing, I encountered the need for higher resolution depth images. I noticed that the SDXL model has a much higher resolution compared to the SD2 model. I would like to ask if it’s possible to improve the overall model's performance at higher resolutions by replacing the pre-trained base model from SD2 to SDXL.
Looking forward to your thoughts!

Hi, thank you for the question.
I did try it at some point. I can confirm that it is technically possible to replace SD2 with SDXL as a backbone. However, I cannot comment on the performance gains, since we did not have large enough GPU's to properly train it. I susspect that it will indeed work.
Hope that helps.
Nando


Can't change a collection while iterating

This line when creating a custom user factory throws an error.
identity.RemoveClaim(existingClaim).
Making the roleclaims collection a new copy of the identityclaims collection should fix it
var roleClaims = identity.FindAll(identity.RoleClaimType).ToArray

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

ID: 9638772f-0f1e-c2a9-8a58-ae8c3e37ae28
Version Independent ID: 7a5b169e-08cb-cdba-4501-88d3da2723bb
Content: Secure a hosted ASP.NET Core Blazor WebAssembly app with Identity Server
Content Source: aspnetcore/blazor/security/webassembly/hosted-with-identity-server.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

@Frankofoedu ... I can't repro that error with 3.x. With the identity.RemoveClaim(existingClaim) line, final claims are (my test roles are "CustomRole" and "Another Role") ...

Comment out the line, and the original role claim is there ...

throws an error

Can you provide the stack trace? ... and confirm that you're using the code exactly as it appears in the topic?
Also, let's confirm versioning ...

Are you reporting this for framework version 3.x or 5.0 RC1?
Latest version of the IdS package (Microsoft.AspNetCore.ApiAuthorization.IdentityServer) for 3.x is 3.1.8, but my test app is on 3.1.3. That's just happens to be where I left off with testing. What version of that package are you using?

Also, I do see one thing (unrelated to your issue here) that should be done. The part of the document that explains how to add the factory to the client app doesn't also tell readers that they must configure Identity Server in order to make roles work (i.e., they must come down from the server in order to be processed, but the section doesn't make that clear).

@guardrex Thanks for replying back.

Can you provide the stack trace? ... and confirm that you're using the code exactly as it appears in the topic?

Here is the stack trace from the browser.
Microsoft.AspNetCore.Components.WebAssembly.Rendering.WebAssemblyRenderer[100]
Unhandled exception rendering component: Collection was modified; enumeration operation may not execute.
System.InvalidOperationException: Collection was modified; enumeration operation may not execute.
at System.Collections.Generic.List1.Enumerator[[System.Security.Claims.Claim, System.Security.Claims, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a]].MoveNextRare()    at System.Collections.Generic.List1.Enumerator[[System.Security.Claims.Claim, System.Security.Claims, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a]].MoveNext()
at System.Security.Claims.ClaimsIdentity.FindAll(String type)+MoveNext()
at CustomUserFactory.CreateUserAsync(RemoteUserAccount account, RemoteAuthenticationUserOptions options) in C:\Users\Frank\source\repos\GISProject\GISProject.Web\Client\CustomUserFactory.cs:line 29
at Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteAuthenticationService3.<GetAuthenticatedUser>d__26[[Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteAuthenticationState, Microsoft.AspNetCore.Components.WebAssembly.Authentication, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60],[Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteUserAccount, Microsoft.AspNetCore.Components.WebAssembly.Authentication, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60],[Microsoft.AspNetCore.Components.WebAssembly.Authentication.ApiAuthorizationProviderOptions, Microsoft.AspNetCore.Components.WebAssembly.Authentication, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60]].MoveNext()    at Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteAuthenticationService3.d__25[[Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteAuthenticationState, Microsoft.AspNetCore.Components.WebAssembly.Authentication, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60],[Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteUserAccount, Microsoft.AspNetCore.Components.WebAssembly.Authentication, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60],[Microsoft.AspNetCore.Components.WebAssembly.Authentication.ApiAuthorizationProviderOptions, Microsoft.AspNetCore.Components.WebAssembly.Authentication, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60]].MoveNext()
at Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteAuthenticationService3.<CompleteSignInAsync>d__19[[Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteAuthenticationState, Microsoft.AspNetCore.Components.WebAssembly.Authentication, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60],[Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteUserAccount, Microsoft.AspNetCore.Components.WebAssembly.Authentication, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60],[Microsoft.AspNetCore.Components.WebAssembly.Authentication.ApiAuthorizationProviderOptions, Microsoft.AspNetCore.Components.WebAssembly.Authentication, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60]].MoveNext()    at Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteAuthenticatorViewCore1.d__84[[Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteAuthenticationState, Microsoft.AspNetCore.Components.WebAssembly.Authentication, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60]].MoveNext()
at Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteAuthenticatorViewCore`1.d__82[[Microsoft.AspNetCore.Components.WebAssembly.Authentication.RemoteAuthenticationState, Microsoft.AspNetCore.Components.WebAssembly.Authentication, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60]].MoveNext()
at Microsoft.AspNetCore.Components.ComponentBase.CallStateHasChangedOnAsyncCompletion(Task task)
at Microsoft.AspNetCore.Components.ComponentBase.RunInitAndSetParametersAsync()
at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task taskToHandle)

Are you reporting this for framework version 3.x or 5.0 RC1?
Latest version of the IdS package (Microsoft.AspNetCore.ApiAuthorization.IdentityServer) for 3.x is 3.1.8, but my test app is on 3.1.3. That's just happens to be where I left off with testing. What version of that package are you using?

dotnet 5.0.100-preview.7.20366.6
I will try it in 5.0RC and give feedback.
These are the packages I have installed in the server and wasm project

Also, I do see one thing (unrelated to your issue here) that should be done. The part of the document that explains how to add the factory to the client app doesn't also tell readers that they must configure Identity Server in order to make roles work (i.e., they must come down from the server in order to be processed, but the section doesn't make that clear).

DOes that mean we have to configure services like this
`services.AddIdentityServer()
.AddApiAuthorization<ApplicationUser, ApplicationDbContext>(options => {
options.IdentityResources["openid"].UserClaims.Add("name");
options.ApiResources.Single().UserClaims.Add("name");
options.IdentityResources["openid"].UserClaims.Add("role");
options.ApiResources.Single().UserClaims.Add("role");
});
JwtSecurityTokenHandler.DefaultInboundClaimTypeMap.Remove("role");`
and still configure a profile service class?

Just tested with rc1 and the error still came up

and still configure a profile service class?

No, configuring a Profile Service counts as "configuring Identity Server." There are two approaches, and that's one of them.
It might be explained by the version ... maybe. I'm a bit surprised that I can't repro that with a 3.x app.
I'll add a cross-link to my issue for the 5.0 docs work and deal with this issue when I work that. I should be starting work on that this week. Go ahead and use ...
var roleClaims = identity.FindAll(identity.RoleClaimType).ToArray

... since that's working for you.

I marked it on the issue. I'll place this on hold to see if the change ends up in the doc. If so, this issue will automatically close when the PR merges.
Thanks for reporting this.


Stack model-pulling cancellation warning

Model-pull jobs throw a warning
apscheduler.scheduler  - Execution of job "_run_model_pulling_worker (trigger: interval[0:00:02], next run at: 2019-03-22 07:20:00 UTC)" skipped: maximum number of running instances reached (1)

fixed on production-server branch


Fixing date -- also only looking back 60 days

Type of change

[ ] Refactor
[ ] New feature
[x] Bug fix
[ ] Optimization
[ ] Documentation Update

Description
Dashboard is broken with the current date.
Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please describe the System Under Test.
Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

/hold


Typo in v0.13.0 changelog on the repo's releases page

"PowerReanme dialog is no longer appearing in top corner all the time"
Should be "PowerRename"

Thanks. Fixed.


Update outputs and readme

This PR adds two new options to fix #17 as well as cleans up the readme a bit.
One thought I had while drafting this - do we really need an example per option or would it be easier to give some examples following the options? I think this would make the readme easier to read, e.g. below.
Examples
The options above can be combined to achieve the desired result.
Example 1: output only miner statistics in CSV format for mainnet data
node report -c /tmp/stacks-testnet-5c87e24790411516

Example 2: output block data, miner statistics sorted alpha, and statistics for xenon data
node report -a -x /tmp/stacks-testnet-5c87e24790411516

Example 3: skip output of block data, output miner statistics sorted alpha, block distances, and transactions for mainnet data, between btc blocks 665250 and 667300
node report -a -b -d -s 665250 -e 667300 /tmp/stacks-testnet-5c87e24790411516

Example 4: long form names are interchangeable with the single letter options
node report --alpha --no-blocks --distances -s 665250 -e 667300 /tmp/stacks-testnet-5c87e24790411516

unfortunately, some of the recent changes I made on my side created a merge conflict, so you could you please rebase with the latest.

unfortunately, some of the recent changes I made on my side created a merge conflict, so you could you please rebase with the latest.

rebased and added my changes back, should be good to go!

rebased and added my changes back, should be good to go!

Fixed again, had to add the space from 57a82caafaf4242a68f8757b91c97121c3d09719 while rebasing with my own changes.

Fixed again, had to add the space from 57a82caafaf4242a68f8757b91c97121c3d09719 while rebasing with my own changes.


pioneer: 20210203 -> 20210723

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/pioneerspacesim/pioneer/releases.
meta.description for pioneer is: "A space adventure game set in the Milky Way galaxy at the turn of the 31st century"
meta.homepage for pioneer is: "https://pioneerspacesim.net"
meta.changelog for pioneer is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/75qy5cqmpcggnz8df9p2295vsyzz6san-pioneer-20210723/bin/pioneer passed the binary check.

/nix/store/75qy5cqmpcggnz8df9p2295vsyzz6san-pioneer-20210723/bin/modelcompiler passed the binary check.

Warning: no invocation of /nix/store/75qy5cqmpcggnz8df9p2295vsyzz6san-pioneer-20210723/bin/savegamedump had a zero exit code or showed the expected version

2 of 3 passed binary check by having a zero exit code.

0 of 3 passed binary check by having the new version present in output.

found 20210723 with grep in /nix/store/75qy5cqmpcggnz8df9p2295vsyzz6san-pioneer-20210723

found 20210723 in filename of file in /nix/store/75qy5cqmpcggnz8df9p2295vsyzz6san-pioneer-20210723

directory tree listing: https://gist.github.com/ae6b5db38086cdf24e05f2701aa320fa

du listing: https://gist.github.com/030edab483ae9a436dd9087db1912f23

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
0 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
pioneer

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/75qy5cqmpcggnz8df9p2295vsyzz6san-pioneer-20210723 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A pioneer https://github.com/r-ryantm/nixpkgs/archive/acc9fd541590bfa06e3df5e731a31325ee7085a6.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/75qy5cqmpcggnz8df9p2295vsyzz6san-pioneer-20210723
ls -la /nix/store/75qy5cqmpcggnz8df9p2295vsyzz6san-pioneer-20210723/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

pioneer

Maintainer pings

Result of nixpkgs-review pr 134447 at acc9fd54 run on x86_64-linux 1

1 package built successfully:

pioneer

2 suggestions:

warning: unclear-gpl
gpl3 is a deprecated license, please check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/games/pioneer/default.nix:31:5:
   |
31 |     license = with licenses; [
   |     ^

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/games/pioneer/default.nix:28:3:
   |
28 |   meta = with lib; {
   |   ^

Result of nixpkgs-review pr 134447 run on x86_64-linux 1

1 package built:

pioneer


feat(plugin): add grpc plugin

Which problem is this PR solving?

#158 - Add gRPC plugin

Short description of the changes

Adds trace collection plugin for gRPC

TODOs

Determine what to do with attributes. They were in the OC impl, but not in any grpc spec I've seen.
https://github.com/open-telemetry/opentelemetry-specification/blob/master/work_in_progress/opencensus/gRPC-stats.md
rebase for TODOS in #161 (ProxyTracer, etc)
rebase for PluginConfig

LGTM. @OlivierAlbertini @bg451 @danielkhan Could you please approve this one, if everything looks good, Thanks :)

A note about this and #216, I intend to complete this PR and then stub out the internalFilesExports functionality in a separate PR which will also rip out that functionality from this plugin (https://github.com/open-telemetry/opentelemetry-js/pull/215/files#diff-671cbf0fed27b96717e412e39a4dfc30R65). I have no other changes pending for this PR, so if it looks good to everyone else, it can be completed.