REARv0.0.1 implementation in FLUIDOS Node workflow and data structures

Hello everyone,
With this pull request I set the basis for the implementation of the REAR protocol v0.0.1. Since the protocol requires a generalization of the flavor entity, many data structures and internal workflows of the FLUIDOS node had to be adapted to this change.
The changes are mainly related to a "generalization" of the data structures, bringing fixes and little new features. The code is a foundation for supporting other flavor types, right now it only supports the K8Slice flavor type, but once the whole process is well defined, new types can be easily added and implemented in the right section of the code, ready to accommodate that new type.

Flavor generalization
The Flavor data structure has been changed from Flavour to Flavor to be consistent between REAR documentation, FLUIDOS node components and general documentation. Also, a new alias is available for this resource, you cannot try it with kubectl get fl.
The most significant change is to its structure. It now reflects the REAR protocol data structure in a very similar way: except for the new field of different flavor properties, the Type field with two subfields now recognizes the name of the type of the Flavor (K8Slice, Service, VM, Sensors, etc.), but also its specific subdata structure. Different features, properties and policies can be found based on the type of the flavor.
Solver generalization
The Solver now has a field called Selector, which with two subfields expresses the name of the Flavor type we are looking for, but also its specific characteristics. These filters are different depending on the selected flavor type.
For each flavor type related characteristic that is filterable, you can use a Match or a Range filter. Now both are supported! You can play around with combinations of Match and Range filters for any filterable characteristic that supports them. At the end, only the flavors with all matching conditions are returned, the selector is not mandatory.
You can avoid specifying the data structure of filters and just write the type of Flavor you are looking for, or even skip it and make a Solver that looks for every possible resource available.
Reservation, Transaction and Contract generalization
The Reservation, Transaction and Contract had an optional field called Partition. This data structure was created specifically for a partition of a K8Slice flavor. Since different flavor types should be supported now, this specific field has evolved. Now the three previously mentioned data structures have the field, with the associated specific data structure `Configuration'.
Configuration
This data structure was born out of the need for a generalization of the partition data structure. It is also being studied to adapt to specific configurations of flavor types that are not partitionable but can be configured, such as a `service' with specific parameters.
Webhook implementations and CRD generic structure
With this release of the FLUIDOS node, we are now implementing Kubernetes webhooks. These are necessary to support the generalization of data structures, and to provide a trustworthy validation of them before they are allowed and created in the Kubernetes cluster. Therefore, the data structure of the generalized CRD is currently often organized as
.
+-- type
|     +-- name
|     +-- data

where the data field is often a generalized raw, so not parsed, validated and checked by the usual CRD template. You need to follow the instructions in the documentation of the node and the validating webhooks will approve your generalized CR.

You can find more detailed documentation about low-level usage of the FLUIDOS Node Custom Resource in "docs/usage/usage.md", or follow the path from the "Installation" section in the main README.md.
The changes made after PR #86 have been set aside in this code redo as no longer compatible and partially incomplete, so it is necessary to study how to re-implement those features if still desired.
Since I am aware that these big changes could lead to many new problems, I decided to make a pull request on a separate branch for evaluation by the community and to be ready to get feedback!
Best regards,
Andrea

Hi Andrea.
I have only a few question..
When you say:
"For each flavor type related characteristic that is filterable, you can use a Match or a Range filter. Now both are supported! You can play around with combinations of Match and Range filters for any filterable characteristic that supports them. At the end, only the flavors with all matching conditions are returned , the selector is not mandatory."

User can user Match OR Range filter, or he/she can use both (combination of Match and Range filters)? This is not clear to me.
This is related to the first one: the underline sentence means that a filtered flavor list is returned thanks to the Selector? Or, if the user can apply a combination of Range and Match selectors, the list is filtered only through Match selector?

Thanks.

Fix pipeline breaking checks.

Fixed

Hi Andrea.
I have only a few question..
When you say:
"For each flavor type related characteristic that is filterable, you can use a Match or a Range filter. Now both are supported! You can play around with combinations of Match and Range filters for any filterable characteristic that supports them. At the end, only the flavors with all matching conditions are returned , the selector is not mandatory."

User can user Match OR Range filter, or he/she can use both (combination of Match and Range filters)? This is not clear to me.
This is related to the first one: the underline sentence means that a filtered flavor list is returned thanks to the Selector? Or, if the user can apply a combination of Range and Match selectors, the list is filtered only through Match selector?

Thanks.

Hi @fracappa ,
Concerning your questions:

For each type of flavor you can have a set of field on which you can set a Math or a Range filter. Maybe an example is clearer:

selector:
    # The flavorType is the type of the Flavor (FLUIDOS node) that the solver should find
    flavorType: K8Slice
    # The filters are used to filter the Flavors (FLUIDOS nodes) that the solver should consider
    filters:
      # The cpuFilter is used to filter the Flavors (FLUIDOS nodes) based on the CPU
      cpuFilter:
        # This filter specifies that the Flavors (FLUIDOS nodes) should have at least 100m of CPU
        name: Range
        data:
          min: "100m"
      # The memoryFilter is used to filter the Flavors (FLUIDOS nodes) based on the Memory
      memoryFilter:
        # This filter specifies that the Flavors (FLUIDOS nodes) should have at least 1Gi of Memory, but no more than 100Gi
        name: Range
        data:
          min: "1Gi"
          max: "100Gi"
      # The podsFilter is used to filter the Flavors (FLUIDOS nodes) based on the Pods
      podsFilter:
        # This filter specifies that the Flavors (FLUIDOS nodes) should have exactly 110 Pods
        name: Match
        data:
          value: 110

When you apply a selector to a Solver, the FLUIDOS Node will request flavors to provider that meet the combination of all the filters. Taking the example I posted in the first point, you will receive a list of flavors that have:

CPU value >= 100m
Memory values >= 1Gi && <= 100Gi
Pods value == 110

Thanks @andreacv98 .
It's much clear now and I approved your changes.
Please, have a final view on your modifications if something needed otherwise you can merge them.


Unit Tests for General App Properties

Added tests for ServerSettingsFragment, AdminConfiguration Fragment and tableSetting fragment for general settings
Added custom ViewAction and ViewMatcher methods.

Hi @wbrunette for the Table-specific Settings, I'm not sure what changes were made after setting the "Use custom home screen checkbox as true",&  also in the device setting fragment the text size or languages were not changed for the device.

Hi, @wbrunette I've finished writing the test for "General app properties" and created an issue in the odk-forum for the doubt https://forum.odk-x.org/t/odk-x-services-ui-test-doubt/1839

@sanyud24 please fix merge conflicts

@sanyud24 please fix merge conflicts

Fixed it @wbrunette

@sanyud24 please fix merge conflicts


Updated email validator max length.

Trac ticket number

N/A
Branch description
This change updates the maximum length that the EmailValidator allows to 254, which is the maximum value per RFC 3696 (erratum 1690).
Checklist

[X] This PR targets the main branch.
[X] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[ ] I have checked the "Has patch" ticket flag in the Trac system.
[ ] I have added or updated relevant tests.
[ ] I have added or updated relevant docs, including release notes if applicable.
[ ] I have attached screenshots in both light and dark modes for any UI changes.

I don't think we can simply ship this as is. What should be done of existing user names and emails that were created prior to this change?
This needs a larger scope discussion and an accepted ticket at the very least.

Superseded by #17763.


Swarm 324: Bootstrap performance issues

DON'T MERGE: This is for review purposes only.
Moving to persistent jandex indexes

@kenfinnigan Here's a working example for wfs-core. Maybe we can use it as a baseline for discussing the changes needed to enable persistent indexes?

@kenfinnigan Please not that it requires these changes to be merged into the jandex plugin. Hence the 1.0.4-SNAPSHOT dependency which causes the CI build to fail.
https://github.com/wildfly/jandex-maven-plugin/pull/9

FYI, I had added the jandex plugin to parent pom so it can be shared across more than core.
Is there any benefit to customizing the index name? We're not doing anything WildFly Swarm specific in building the jandex right?

The custom index name is necessary, because otherwise it conflicts with indexes that WF expects at that location.

Is it building differently than what WF expects? Otherwise I would not expect it to be an issue

yes, the indexes seem to be different. i didn't have the time to fully track down all the inner workings of deployer chains in WF and how they leverage jandex, but I can tell that our integration tests fail with indexes provided by us that carry the default name

The latest update pushes the jandex plugin into the fractions. If this passes the tests, then we would be ready to apply the changes to the other fraction outside of core

The latest commit removes the jandex file from main/module.xml

Cleaned up and squashed the commits to make it more comprehensible

With this build setup (jandex in the core pom) all our own fractions that don't reside within core inherit everything they need to make it work.
Only fractions that don't inherit from core would need to add the jandex plugin. But I am currently not  aware of anything outside wildfly-swarm.

With regard to the choices of
a) jandex in the fraction plugin and
b) jandex in it's own plugin (like it is now)
the first solution (a) sounds nice, but it would require adding the fraction plugin to any runtime module. currently it's only attached to the main module, but indexes generated at that level don't help.
the second option (b) seems to be most versatile, doesn't require additional changes to any downstream module and creates an index with every jar.
Long Story Short
It's complicated, don't ask any further, this is the best approach I could find to balance ease of use with imposed constraints.

From my point of view this is ready to be merged

I'd say merge, and then I'll rebase my :spi stuff and get that in today, hopefully.


SMS signup intro message and first question send at the same time

The intro message and first question send at the same time despite trying to add a pause.
Related to issue https://github.com/twilio/twilio-ruby/issues/172

No great way to fix this at the moment. Possibly cut down intro message and first question so that they can fit in one message.

Here are two new versions that combine the intro message and the first question. @shua123 which one do you think sounds better?
Thanks for joining CUTGroup! We’ll ask 10 quick questions to complete your signup. Once completed, we’ll send you a $5 VISA gift card!  What is your first name?
or
Thanks for joining CUTGroup! We’ll ask 10 quick questions to finish your signup. Once completed, we’ll send your $5 gift card! To begin, what’s your first name?

We went with the second option and the SMS Wufoo form has been updated to reflect this. Closing.


General: Fix several follow up dark mode issues

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines.
[x] I added multiple screenshots/screencasts of my UI changes.

Description

This PR fixes four follow up issues regarding the dark mode
1. Dark Mode settings popover is wrongly positioned if the navbar collapses on smaller screens:
Reported by @jpbernius, @krusche, @JohannesStoehr and @Funky-Punky.
Closes #5036.

2. Lecture attachment details appear black in dark mode
Reported by @ge65cer
Happens if you select a lecture with at least one attachment as student:

3. Exercise feedback is less noticeable in dark mode
Reported by @LadnerJonas
Closes #5037.
Before:

After:

4. Category dropdown background is white
Reported by @bottbenj
Closes #5039.
Before:

After:

Steps for Testing

Please check that each of the described bugs do not appear anymore.
Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

/ #5037 solution is looking great 🚀

@nadinesamyy @alaraozdenler @valeriebucher
Hey, I've added 5. and 6. to this PR as well. Could you check them as well? Thank you very much!

@joaquinmurcia That looks correct. The difference is whether or not the diamond is filled, and it would be filled with the foreground color. In dark mode, that would be white, so in your case the composition diamond is filled, while the aggregation diamond is not filled (has the background color), which is correct.

@jbllhff There is this view which also support all the different diagram types. Reachable if you go to the exercise list of a course in the management area and click "Create Drag and Drop Model Quiz" in the Quiz Exercises section.


File views do not account for folded lines.

Extension version: 0.6.0
VSCode Version:1.31.0
OS:Windows

Steps to Reproduce:

Clone the vscode-azurefunctions extension. Then open this PR: https://github.com/Microsoft/vscode-azurefunctions/pull/1204/files
View FunctionCreateStepBase.ts.
Look at the line numbers in github web, and the extension. You'll notice that lines are missing, leading to lines of code that don't seem to belong to any class or function.

@PrashanthCorp can you give an example of how the lines should be folded to reproduce this issue? That PR is old so I cannot use it to repro the issue now.


feature: report pod alloc of Guaranteed pod and cpu manager policy

Signed-off-by: Yue Zhang<EMAIL_ADDRESS>Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/lgtm


Generate module name from file path

${TM_DIRECTORY/(^.+\\\/test|\\b)\\\/(\\w+)/${1:? :.}${2:/pascalcase}/g}
When TM_DIRECTORY was '~/Document/some_project/test/some_long/path', it translate to ' SomeLong.Path'. (note that it has first space I can not reduce it 😓 )
${TM_FILENAME_BASE/(.+)_test/${1:/pascalcase}/}
When filename was some_module_test it translate to SomeModule

Thank you!!!


Draft: Feature/#73 use esm

I'm trying to use the new version in some of my recent projects. I'm still running into some errors...
This one for example:
"The component with the tag name "undefined" has no ElementFactory"
Also I had some trouble defining the factory. Would it be possible to set the factory by reference rather than just a string?

I also tried to use this new version for the ocean-crm demo project. I'm running into some weird bugs. I'm not getting any error messages. The luna startup just stops before registering components and prints out a bunch of bundled code for one single imported file (but not all of it...)
See: https://github.com/webtides/ocean-crm/tree/feature/type-module

I'm trying to use the new version in some of my recent projects. I'm still running into some errors...
This one for example:
"The component with the tag name "undefined" has no ElementFactory"
Also I had some trouble defining the factory. Would it be possible to set the factory by reference rather than just a string?

If I add some more logging:
The component with the tag name "undefined" has no ElementFactory {
component: { element: [class index], ElementFactory: undefined },
attributes: {}
}
I think this is the index page?
Also the luna-renderer package is still "commonjs" and bundles a bunch of require stuff - is that supposed to be like that?

@lukas-schardt @quarkus I think this should be good now! Please have a look 😆
I will update the changelog and write a small upgrade guide next week


Limit height for possible multiple rows column on application list page

What this PR does / why we need it:
https://user-images.githubusercontent.com/32532742/164584887-6675527b-4165-405f-9f73-6d646a821881.mp4
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Nice improvement.
/lgtm

/hold

/hold cancel

/lgtm


support remotes with preston.tar.gz and preston-1.tar.gz archives

Currently, Preston allows for discovering resources in tar.gz files on remotes.
for instance,
when retrieving content associated with -
hash://sha256/2a5de79372318317a382ea9a2cef069780b852b01210ef59e06b640a3539cb5a
using some remote, preston digs out the content from
preston-2a.tar.gz
if exists. The naming convention is preston-[first two hash ids characters].tar.gz .
The reason for this feature is to bundle resources to keep the file count low. For instance, when a remote only provides up to 100 files (like Zenodo), resources can be bundled into these tar balls.
Suggest to support naming conventions:
preston-2.tar.gz (first content hash character)
as well as
preston.tar.gz (all content to be found in this archive).

As of Preston 0.8.5, remotes are queried for "preston.tar.gz" in addition to the "preston-[a-f0-9]{2}.tar.gz" patterns.
For example usage, see
Poelen, J. H. (2024). A biodiversity dataset graph: Biological Associations in TaxonWorks hash://sha256/e4a47c067d6c125da60c9a1b92b5eecdea539cb8666cd3aed99db347ae5b8ed0 hash://md5/686007de79cc2a49ab23fd3debe56e3f (0.3) [Data set]. Zenodo. https://doi.org/10.5281/zenodo.11151783
this enables stuff like:
preston clone --remote https://zenodo.org/records/11151783/files

which would use preston.tar.gz to clone the dataset.


Up version

Coverage remained the same at 98.61% when pulling 8a7e160d22cd94854a01dad18580febc2b961407 on version into f898f198586f7300e233ff69d94519017825e149 on master.


Add Performance Issue Template

This adds a new Issue Template specifically for Performance Issues.

Looks good to me. Do you think it would be helpful to mention zprof or is it overkill?

Yes, I think zprof would be too much.. Thx for the review.


Its possible to change the bill login?

Hello ¡Well Done!, I installed it out of curiosity in beta mode on windows, now I want to know if it is possible to change the "payout coopers" login to receive donations from coopers, take action skip map, replay map for my same account in which I play and not from the server login, since it cannot be used!! and that it is not the same as that
Can i change it in the root files?
I await your response, maybe it's hidden there  (?=

it is possible as all payments go through the same method. if you want to change it, edit this line (270) in /src/Utils.ts
https://github.com/lythx/trakman/blob/7ca9c46a18e4e002c340ea3ba40e7e8f120c7305/src/Utils.ts#L270
switch targetLogin: string = '' for targetLogin: string = 'your_tm_login'
after that, rebuild and launch again, the controller will then send bills with your login as the one receiving coppers.

Thank you very much for your attention, but I have changed the indicated line 270 targetLogin: string = ''MY LOGIN" targetLogin: string = 'your_tm_login'
restart npm run stop & npm run daemon(background) and it didn't work, any other ideas?


Add tag announcement functionality

Should be a rich message with as much interesting information as is reasonable, all in a dedicated channel.

Done! Could use finesse, but works alright.


Chat Prefix doesn't work anymore in 4.0.0

Hello! I'm using Paper 68.
Basically, when prefixchat is set to true, it doesn't add anything to the chat syntax. Otherwise I'm having no problems at all!
FIND: the prefix does show in the console, though!
Thank you!

EDIT: After I rebooted, it came back for some reason... I really have no idea what I did to fix it tho xD

Self-solved.


Issue #8 - alternate /tests/ setup for 100% maven

Deploys jetty9:testing (tagged) image in /jetty9/
module during integration-test phase
Adds /tests/
Adds /tests/webapps/
Adds /tests/gcloud-testing-core/ for common testing lib

This is an alternate approach for Issue #8 and should be considered a replacement for  pull request #17

Will create new issue-8 branch locally to this repo and issue a new pull request


Delete response

This adds two dangerous things to the query page:

delete response when you do a _find query, you now have the option of deleting the response, which we were doing a lot manually. (a _deleted: true delete). it prompts you with a warning first and downloads the data to your system as well before deleting.

bulk-docs this "query" is actually just short hand for setting up a post to bulk docs

there's also a script i use to post lookout to my local couch, which we may want to do some day on some of our envs

:tada: This PR is included in version 1.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


-- (double dash) in comment error in Thymeleaf

defaultContainer.js has this part around line 8

<title>__title__</title>
  <!--[if !mso]><!-- -->
  <meta http-equiv="X-UA-Compatible" content="IE=edge" />

The middle line generates error as apparently there shouldn't be double dashes in comments. Should this be fixed? If so I can submit a PR.

How do you propose to FIX this whithout breaking the "reverse conditional comment"?

Hi there @pglotov
It allows to fix some image issue on Live mail see https://www.emailonacid.com/blog/article/email-development/prevent-ie-edge-meta-tag-from-breaking-images-in-live-mail and you can see related issue here : #306


Add InterfaceType to EC2 LaunchTemplate

https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-ec2-launchtemplate-networkinterface.html#cfn-ec2-launchtemplate-networkinterface-interfacetype

I believe this should be just "basestring" instead of "[basestring]" (which means a list of strings). Could you confirm and correct? Thanks!

@markpeek fixed

Thanks!


[WFLY-18682] Upgrading WildFly Http Client to 2.0.3.Final

https://issues.redhat.com/browse/WFLY-18682

Just noticed a couple of errors in the Windows job looked directly related to the HTTP Client, running that job again.

I just upgraded to 2.0.4.Final version that should sort out the Windows issues @darranl

@ropalka FYI I am still seeing the same CI failure on the run from today.

Thank you very much @ropalka and @tadamski - CI is green so merging.


Error including latest Unity package ("Could not load signature of Google.Protobuf.ByteString:get_Span")

What version of gRPC and what language are you using?
grpc_unity_package.2.29.0-dev csharp
What operating system (Linux, Windows,...) and version?
Windows 10
What runtime / compiler are you using (e.g. python version or version of gcc)
Unity 2019.3.3f1
What did you do?
I downloaded the last unity package (2.29.0) from https://packages.grpc.io/, extracted it to the Asset/Plugins folder.
What did you expect to see?
No errors, normal unity editor environment.
What did you see instead?
Two errors, multiple occurrences of the former, one off the latter:

Error: Could not load signature of Google.Protobuf.ByteString:get_Span due to: Could not load file or assembly 'System.Memory, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51' or one of its dependencies. assembly:System.Memory, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51 type: member:(null) signature:

Unloading broken assembly Assets/Plugins/Google.Protobuf/lib/net45/Google.Protobuf.dll, this assembly can cause crashes in the runtime

Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
See TROUBLESHOOTING.md for how to diagnose problems better.
Anything else we should know about your project / environment?

This seems to happen after grpc_unity_package.2.26.0-dev, so grpc_unity_package.2.26.0-dev and prior do not elicit the same error message

This may not be on grpc, but on protobuf? Although I think we're building that package.

https://issuetracker.unity3d.com/issues/could-not-load-file-or-assembly-error-when-assembly-dependency-is-in-another-folder?_ga=2.195835948.1013030992.1586163171-425932875.1567394600
I not sure it clearly solve a bug.
I gather all of dll in same folder then error message disappeared.
platform build testing in progress, but editor is clearly fine.

System.Memory uses <IP_ADDRESS> (.net sdk 4.5.3) for grpc and <IP_ADDRESS> (.net sdk 4.5.2) for protobuf.
Unity3D cannot have two versions of dependencies.
in Google.Protobuf.csproj
...
  <ItemGroup>
    <PackageReference Include="System.Memory" Version="4.5.2"/>
    <PackageReference Include="Microsoft.SourceLink.GitHub" PrivateAssets="All" Version="1.0.0"/>
    <!-- Needed for the net45 build to work on Unix. See https://github.com/dotnet/designs/pull/33 -->
    <PackageReference Include="Microsoft.NETFramework.ReferenceAssemblies" PrivateAssets="All" Version="1.0.0"/>
  </ItemGroup>

  <!-- Needed for netcoreapp2.1 to work correctly. .NET is not able to load the assembly without this -->
  <ItemGroup Condition=" '$(TargetFramework)' == 'netstandard2.0' ">
    <PackageReference Include="System.Runtime.CompilerServices.Unsafe" Version="4.5.2"/>
  </ItemGroup>
...

in Grpc.Core.csproj
...
  <ItemGroup>
    <PackageReference Include="System.Memory" Version="4.5.3" />
  </ItemGroup>
...

workaround here
@rem System.Memory <IP_ADDRESS> -> <IP_ADDRESS>
@rem warn: this remove sign from Google.Protobuf.DLL
@rem .netfx 4.6.1 tools tested
"ildasm" /UNICODE /all /text Google.Protobuf.dll /out=Google.Protobuf.il
powershell -Command "(gc Google.Protobuf.il) -replace '.ver 4:0:1:0', '.ver 4:0:1:1' | Out-File -encoding UNICODE Google.Protobuf-mod.il"
@rem need admin permission or Could not create output file, error code=0x80070005
"ilasm.exe" /DLL "%CD%\Google.Protobuf-mod.il" "/output:Google.Protobuf.dll" "/resource=Google.Protobuf.res"
del /q Google.Protobuf*.il
del /q Google.Protobuf.res

Thanks for driving this, ZCube!

This seems to happen after grpc_unity_package.2.26.0-dev, so grpc_unity_package.2.26.0-dev and prior do not elicit the same error message

Using 2.26-0-dev causes another error when I import generated csharp files:
error CS1729: 'GeneratedClrTypeInfo' does not contain a constructor that takes 7 arguments
Environment and tools used:
Unity 2019.2.13f1
protoc 3.12.4
grpc_csharp_plugin (from grpc.tools 2.31.0)
What is the consensus on a solve for these related issues?

This seems to happen after grpc_unity_package.2.26.0-dev, so grpc_unity_package.2.26.0-dev and prior do not elicit the same error message

Using 2.26-0-dev causes another error when I import generated csharp files:
error CS1729: 'GeneratedClrTypeInfo' does not contain a constructor that takes 7 arguments
Environment and tools used:
Unity 2019.2.13f1
protoc 3.12.4
grpc_csharp_plugin (from grpc.tools 2.31.0)
What is the consensus on a solve for these related issues?

I ended up getting things working by using an older version of protoc and grpc_csharp_plugin:
Mac OSX Mojave

Protoc (v3.8) and grpc_csharp_plugin (grpc-protoc_macos_x64-1.26.0-dev.tar.gz) was used to compile and generate code files from proto.
grpc_unity_package.2.26.0-dev.zip

This seems to happen after grpc_unity_package.2.26.0-dev, so grpc_unity_package.2.26.0-dev and prior do not elicit the same error message

Using 2.26-0-dev causes another error when I import generated csharp files:
error CS1729: 'GeneratedClrTypeInfo' does not contain a constructor that takes 7 arguments
Environment and tools used:
Unity 2019.2.13f1
protoc 3.12.4
grpc_csharp_plugin (from grpc.tools 2.31.0)
What is the consensus on a solve for these related issues?

I ended up getting things working by using an older version of protoc and grpc_csharp_plugin:
Mac OSX Mojave
Unity 2019.2.13f1

Protoc (v3.8) and grpc_csharp_plugin (grpc-protoc_macos_x64-1.26.0-dev.tar.gz) was used to compile and generate code files from proto.
grpc_unity_package.2.26.0-dev.zip

@mchiang-goo Having the exact same issue. Is that the latest version of protobuf that this will work with?

@cloutiertyler Yup, just reference protobuf/src with -I. My very messy script to generate files looks like this:
$PROTOC_PATH \
-I ~/workspace/github/googleapis \
-I ~/workspace/github/protobuf/src \
-I ~/Downloads/protoc-3.11.1-osx-x86_64/include \
--csharp_out $OUT_PATH \
--grpc_out $OUT_PATH \
$PROTO_PATH/assistant/embedded/v1alpha2/embedded_assistant.proto $PROTO_PATH/type/latlng.proto $PROTO_PATH/api/annotations.proto $PROTO_PATH/api/http.proto \
--plugin=protoc-gen-grpc=~/Downloads/grpc-protoc_macos_x64-1.26.0-dev/grpc_csharp_plugin

* grpc_csharp_plugin

I ran into same issue. grpc_unity_package.2.26.0-dev.zip resolve the error messages as mentioned.
I have a couple of questions, repo documentation does not say how to actually generate proto files. C# main repo does mention it but it mentions to use dotnot package manager, something not fully available in unity.
If you are able to provide support for this:
How do you compile proto files? How did you installed grpc_csharp_plugin?

workaround here
@rem System.Memory <IP_ADDRESS> -> <IP_ADDRESS>
@rem warn: this remove sign from Google.Protobuf.DLL
@rem .netfx 4.6.1 tools tested
"ildasm" /UNICODE /all /text Google.Protobuf.dll /out=Google.Protobuf.il
powershell -Command "(gc Google.Protobuf.il) -replace '.ver 4:0:1:0', '.ver 4:0:1:1' | Out-File -encoding UNICODE Google.Protobuf-mod.il"
@rem need admin permission or Could not create output file, error code=0x80070005
"ilasm.exe" /DLL "%CD%\Google.Protobuf-mod.il" "/output:Google.Protobuf.dll" "/resource=Google.Protobuf.res"
del /q Google.Protobuf*.il
del /q Google.Protobuf.res

It's close, but the command doesn't output a res file.

Protobuf has been upgraded to the same version of System.Memory as Grpc here: protocolbuffers/protobuf#7940. So once a new version of Google.Protobuf is released, things will work without requiring any workarounds.

This seems to happen after grpc_unity_package.2.26.0-dev, so grpc_unity_package.2.26.0-dev and prior do not elicit the same error message

Using 2.26-0-dev causes another error when I import generated csharp files:
error CS1729: 'GeneratedClrTypeInfo' does not contain a constructor that takes 7 arguments
Environment and tools used:
Unity 2019.2.13f1
protoc 3.12.4
grpc_csharp_plugin (from grpc.tools 2.31.0)
What is the consensus on a solve for these related issues?

I encountered the same error, it turned out that I was using new version of protoc.exe, try to execute "protoc.exe --version" to know if you are using correct one, it should be 3.8.0

I still get the same error. Can someone confirm?

Just commenting here to say the same thing. I get the same errors as everyone else here. Downgrading to the 2.26.0 version works

+1

+1


Merge metadata tables

The current implementation uses multiple backing tables to store metadata information - e.g. list of indexed keys for both edge and vertex separately, names of index tables. This seems excessive as they could all be stored in a single metadata table given that these will be small. They should be merged into a single table.

Merged changes in 1018d48382e488a9f0757768c5230d8d0e05ecf2


Support FastGit as alternative download url

Is your feature request related to a problem? Please describe.
GitHub Raw has block or jamed in China. Using fastgit will accelerate it.
Describe the solution you'd like
using an env var to enable using fastgit as aleternative download url
or check the download status. when download from github raw fails, switch to fastgit as fallback.
Additional context
also cloud provide an env var for user to define their own mirror/alternative download urls

Interesting. How does fastgit work from a user perspective? Do you simply replace the github.com domain in the checkout url?

Interesting. How does fastgit work from a user perspective? Do you simply replace the github.com domain in the checkout url?

almost is.
but the domain has a little different for different github services
https://doc.fastgit.org/en-us/guide.html#the-usage-of-web

almost is.
but the domain has a little different for different github services
https://doc.fastgit.org/en-us/guide.html#the-usage-of-web

Thanks for googling it for me ;)
I'm genuinely curious about fastgit.
So. For cloning this here repo for example you'd need to:
git clone https://download.fastgit.org/zdharma/zinit.git

While doing that I noticed that I got redirected to github.com:
$ git clone https://download.fastgit.org/zdharma/zinit.git
Cloning into 'zinit'...
warning: redirecting to https://github.com/zdharma/zinit.git/ # <-- HERE
remote: Enumerating objects: 130, done.
remote: Counting objects: 100% (130/130), done.
remote: Compressing objects: 100% (58/58), done.
remote: Total 19905 (delta 81), reused 116 (delta 72), pack-reused 19775
Receiving objects: 100% (19905/19905), 10.52 MiB | 10.61 MiB/s, done.
Resolving deltas: 100% (12854/12854), done.

Does fastgit lookup the source IP and redirect if not originating from China?

Thanks for googling it for me ;)
I'm genuinely curious about fastgit.
So. For cloning this here repo for example you'd need to:
git clone https://download.fastgit.org/zdharma/zinit.git

While doing that I noticed that I got redirected to github.com:
$ git clone https://download.fastgit.org/zdharma/zinit.git
Cloning into 'zinit'...
warning: redirecting to https://github.com/zdharma/zinit.git/ # <-- HERE
remote: Enumerating objects: 130, done.
remote: Counting objects: 100% (130/130), done.
remote: Compressing objects: 100% (58/58), done.
remote: Total 19905 (delta 81), reused 116 (delta 72), pack-reused 19775
Receiving objects: 100% (19905/19905), 10.52 MiB | 10.61 MiB/s, done.
Resolving deltas: 100% (12854/12854), done.

Does fastgit lookup the source IP and redirect if not originating from China?

maybe yes. this is a test from a hk vps.

maybe yes. this is a test from a hk vps.

Unfortunately, hosts doesn't always work. And its speed is much slower than proxy.

I think you misunderstood what I meant.
What I suggested was editing the hosts file so that github.com points to the fastgit IPs

Well, this doen't work. After I alter the IP of github.com as fastgit.org IP, remote server will ask a certification when clone files. Server rejects direct IP access.
git clone https://github.com/zdharma/zinit.git
Cloning into 'zinit'...
fatal: unable to access 'https://github.com/zdharma/zinit.git/': SSL: certificate subject name (isaacchen.cn) does not match target host name 'github.com'
fatal: unable to access 'https://github.com/zdharma/zinit.git/': gnutls_handshake() failed: Error in the pull function.

Yeah, you'd obviously also need to disable TLS cert validation. Not exactly ideal :-/

Sadly, now seems the easiset way is to repalace the clone url.

reference to this document  https://zdharma.org/zinit/wiki/Private-Repositories/
we can use from ice
like this
zinit lucid wait'1' for \
    from"hub.fastgit.org" \
    matthieusb/zsh-sdkman

Hey, guys. I found an elegant way to solve this. Just config the git url:
git config --global url."https://hub.fastgit.org/".insteadOf "https://github.com/"
A fast mirror you will get!

Hey, guys. I found an elegant way to solve this. Just config the git url:
git config --global url."https://hub.fastgit.org/".insteadOf "https://github.com/"
A fast mirror you will get!

But this method didn't work for this kind of zinit config
zinit snippet OMZ::plugins/git/git.plugin.zsh
zinit snippet PZT::modules/helper/init.zsh


[Do not merge] Remove jquery from mobile navigation

Rewrites the mobile navigation JavaScript to avoid using jQuery
Worth reading this commit by commit since I needed to refactor the structure of this before rewriting it to jQuery.
TODO:

Test on mobile
Double check IE8-IE11
Test cross browser
Check ARIA behaviour is consistent with previous implementation

https://trello.com/c/IqK2orrv/1328-update-mobile-navigationjs-to-follow-frontend-conventions-and-remove-dependency-on-jquery

You can preview this change here:
Built with commit 71eb4dc194d76808dd87ebda85035b6ef03f1a37
https://deploy-preview-595--govuk-design-system-preview.netlify.com

You can preview this change here:
Built with commit d49350c5a1b33ddda698ac63159420e6e6a2db54
https://deploy-preview-595--govuk-design-system-preview.netlify.com

Assuming the tests pass once this is rebased against #600, those last two commits look good and I think this is good to go 👍


Add the option to hide the generated datetime from the generated files

On some projects I want to be sure that we didn't miss anything in the generated files. That is why I prefer to run the generate commands in the pipeline.
Because of the timestamps the files are always changed, with this PR it is possible to hide the datetime from the generated files so it is possible to see automated if there is any change found and a new file need to be committed.

@mfn thanks for the feedback, I will wait on the answer from @barryvdh on your question about removing instead of adding another config. If that is clear I will update this PR.

Makes sense!

FTR, here are the reasons we've it in the first place:

Basically since the beginning the date was added without man fanfare
=> https://github.com/barryvdh/laravel-ide-helper/commit/91033293b9fd879d1951838f913f0a6a3fbd852c

Later the time was added:

Show time in generated files / I think this is useful for debugging.

=> https://github.com/barryvdh/laravel-ide-helper/pull/574

The "think … is …useful" sounds a bit fishy to me 😄 I think debugging should happen on different levels and favouring CI workflows in 2020 is more appropriate (aka just removing it).
I think this whole thing may also comes from a time when those files were "shared" online via gists for specific Laravel version, a practice I'd say shouldn't be endorsed at all anymore.

Fun fact: I already mentioned that the _ide_helper_models.php does not include them.
In fact the test suite was recently refactored to use snapshots for the rewritten/generated PHP files and this wouldn't be possible if a date was written in them…
** :+1: ** for just removing them from my side

I think I added it because in the earliest versions, the actual gist was used more often. That's why the version and time were included. I don't see a benefit of including it now. Might as well remove it altogether.

Sent in https://github.com/barryvdh/laravel-ide-helper/pull/1000

Thanks

Sent in #1000

You are fast! 👍


Implement Day 4 Part 1

https://adventofcode.com/2023/day/4

Ok, I've done this algorithm once in JS and once in C# already. For this pass, I'm going to try to make it as efficient as reasonably possible. Unlike with C#, Go doesn't have many data structures with O(1) lookup times. In fact, pretty much all I have to work with (as far as I can tell) is map... Unless I want to import some packages, which I don't really want to do.
Nope, map[string]bool is what we're going to use in order to get the quick lookup when we're looking for matches.

I'm kind of torn on whether or not I want to use the fact that all of the numbers in the cards are only 1 or 2 digits, with the 1-digit numbers having an extra space in front of them. I could use that to do faster parsing, but it feels like a bad idea if we're pretending that this is code that could be used on unknown datasets in the future. The problem description definitely does not give any indication on whether or not the numbers follow any kind of structure or pattern, so I'm going to play it safe. Split the string using space ' ' character, and remove any empty strings you end up with in the resulting string slice.


Add SmoothAnimatedColapseArrow (local widget) to the sections in the settings

What

[ ] Add SmoothAnimatedColapseArrow (local widget) to the sections in the settings

Part of

#960

Sure @bhattabhi013 we already have a prebuild widget SmoothAnimatedColapseArrow, but on the Profile/Settings page we currently don't use it, just normal Icons. the issue is about switching to the animated one


ci: Bump Node.js to v18

Seems I have bumped into https://github.com/nock/nock/issues/2397. Added workaround...


[tech] BlockExplorer autodetection

The blockExplorer used to show the transactions is bound to the environment: https://github.com/KlimaDAO/klimadao/blob/staging/carbonmark/lib/constants.ts#L116
This is disturbing for testers:

The blockExplorer should be chosen dynamically using the connected wallet.
Or testers should always test on mumbai ?

This will go away after wallet abstraction work is done


Do Scanned Maps fit?

Given that:

The source metadata lives in Voyager at MARC
The delivery mechanism isn't the repo and could/should be Blacklight (perhaps GeoBlacklight and OrangeLight, which makes this similar to Plum's emphasis on reuse/embedding).
IIIFification is a requirement (correct?)
Model behaviors are similar (names are wrong)
Workflow requirements seem similiar

ScannedBook could be ScannedResource and the notion of "pages" could be abstracted a bit more, as "surfaces".
I think this could work.
@eliotjordan and @terrellt could we do a hangout to discuss this on Monday?

Yes. Rename ScannedBook to ScannedResource.


Resolves  RicoSuter/NJsonSchema#1322 - Added uuid format for schema validation

Created new uuid format validator
Added testing for format

Resolves RicoSuter/NJsonSchema#1322

Thanks for the PR.


CVE-2019-16701

https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2019-16701
is this cve really a vulnerability ? seems like someone has cve'ed a bad feature
Can someone verify that this CVE is actually a vulnerability?

The line is correct. If the userid is set AND the user is not 0 (admin) AND that user does not have privileges to access the page, then they are denied. That CVE is invalid, it's not a bug, and it's an intentional (not bad) feature. If you grant someone access to run commands and code, don't be surprised when they can, in fact, run commands and code.

thanks for confirming this


Bug on install using poetry

 File "/root/.cache/pypoetry/virtualenvs/discollama-gIPUCJZT-py3.11/lib/python3.11/site-packages/discord/client.py", line 849, in runner await self.start(token, reconnect=reconnect) File "/root/.cache/pypoetry/virtualenvs/discollama-gIPUCJZT-py3.11/lib/python3.11/site-packages/discord/client.py", line 777, in start await self.login(token) File "/root/.cache/pypoetry/virtualenvs/discollama-gIPUCJZT-py3.11/lib/python3.11/site-packages/discord/client.py", line 609, in login raise TypeError(f'expected token to be a str, received {token.__class__.__name__} instead') TypeError: expected token to be a str, received NoneType instead
any ideas how to resolve this are much appreciated.

This is an error when discollama starts without DISCORD_TOKEN set to a valid value. Details are in a note in the README


Bootstrap 3.3.0 modal bug

Hello,
I'm new on bootstrap and I found a modal bug: when I press the modal button, the background goes right of 15px about. My bootstrap template is 3.3.0 version.
I tried more possible solutions on GitHub but none of these works for me, so I open this topic. Probably I wrong anything!
Please, can you tell me what files I have to modify and what parameters I have to change?
Thank you in advance to all!
Gianfranco

Before any contributor tell you, update your version to latest (3.3.4). JSbin maybe needed.
Sorry my bad english

Hello,
I also tried the upgrade to version 3.3.4 but doesn't work. Probably I wrong the upgrade procedures....
Can you tell me what files I have to substitute for the upgrade?
Thank you for your help!
Gianfranco

Please follow our issue tracker policies and bug reporting guidelines in the future:
https://github.com/twbs/bootstrap/blob/master/CONTRIBUTING.md#using-the-issue-tracker
https://github.com/twbs/bootstrap/blob/master/CONTRIBUTING.md#bug-reports
How-to questions like "how to upgrade?" should be asked on StackOverflow instead on this issue tracker.
Bug reports should include a simplified, reproducible test case demonstrating the problem, preferably as a JS Bin or JS Fiddle.
Thanks, and good luck!


Feature request: outstanding queryFee summary via indexer-agent

As discussed with @fordN
It would be great to have the agent output a summary of outstanding queryFees per allocationId in a table, so an Indexer can get a visual representation of what they have outstanding in the queryFee pool and make decisions on how they want to set their batch claim thresholds.

https://github.com/graphprotocol/indexer/pull/403
This PR creates a table on allocations, one of the column is query fee collected. I think the table is sufficient for making decisions, or we can add a command for filtering (maybe something like get outstanding x_amount)


remove priority goods to align with PEGA

Looks good to me, except for the fact that package-lock.json shouldn't be in the repo.


Uninstall feature not showing up

Running the most recent pre-release version
⚡ marcusf@MARCUS-SURFACE  C:\src  winget --version
v0.2.3162 Preview
The first time I run winget features I see Uninstall available and disabled, so I enabled all the features of interest in the settings file below.
Setting file
{
"$schema": "https://aka.ms/winget-settings.schema.json",
// For documentation on these settings, see: https://aka.ms/winget-settings
"source": {
   "autoUpdateIntervalInMinutes": 5
},
"experimentalFeatures": {
    "experimentalMSStore": true,
    "list": true,
    "uninstall": true,
    "upgrade": true
}

}
Come back and check features again and it seems Uninstall has disappeared from the list.
⚡ marcusf@MARCUS-SURFACE  C:\src  winget features
The following experimental features are in progress.
They can be configured through the settings file 'winget settings'.
Feature                 Status   Property            Link
Command Sample          Disabled experimentalCmd     https://aka.ms/winget-settings
Argument Sample         Disabled experimentalArg     https://aka.ms/winget-settings
Microsoft Store Support Enabled  experimentalMSStore https://aka.ms/winget-settings
List Command            Enabled  list                https://aka.ms/winget-settings
Upgrade Command         Enabled  upgrade             https://aka.ms/winget-settings

@devmarcusf would you mind sharing the output for winget --info?

@devmarcusf would you mind sharing the output for winget --info?

@devmarcusf would you mind sharing the output for winget --info?

I can't access the uninstall feature either. I can add it to the settings file but the feature doesn't seem present at all in the currently available version. Here is my output:
Windows Package Manager v0.2.3162 Preview
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19042.746
Package: Microsoft.DesktopAppInstaller v1.11.3162.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
--------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget

@devmarcusf would you mind sharing the output for winget --info?

I can't access the uninstall feature either. I can add it to the settings file but the feature doesn't seem present at all in the currently available version. Here is my output:
Windows Package Manager v0.2.3162 Preview
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19042.746
Package: Microsoft.DesktopAppInstaller v1.11.3162.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
--------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget

@MattBDev can you reproduce with Windows Package Manager v0.2.10191 Preview?

@MattBDev can you upgrade to https://github.com/microsoft/winget-cli/releases/tag/v-0.2.10191-preview
If you are on a Windows Insider build or the Windows Package Manager Insider Preview, you should be able to upgrade "App Installer" via the Microsoft Store.

The current client version is now 0.2.10771.
We have not been able to reproduce this issue.
If you are still having this problem on a current version, feel free to re-open this issue.


When to install exporter service required dependency/resource?

On branch /dev/refactor-multiple-exporter-support
The current install function of BaseExporter class is:
class BaseExporter:
    def install(self) -> bool:
        """Install the exporter."""
        logger.info("Installing %s.", self.exporter_name)
        config_success = self.render_config()
        service_success = self.render_service(str(self.charm_dir), str(self.exporter_config_path))
        if not (config_success and service_success):
            logger.error("Failed to install %s.", self.exporter_name)
            return False
        systemd.daemon_reload()

        # Verify installed
        if not (self.exporter_config_path.exists() and self.exporter_service_path.exists()):
            logger.error(f"{self.exporter_name} is not installed properly.")
            return False

        logger.info("%s installed.", self.exporter_name)
        return True

and the timing to trigger this install function is on

def get_exporter(exporter: BaseExporter):
    """Install and get an exporter"""
    if exporter.exporter_service_path.exists():
        return exporter
    installed = exporter.install()
    if not installed:
        logger.error(f"Failed to install {exporter.exporter_name}")
        return
    return exporter

class HardwareObserverCharm(ops.CharmBase):
    def __init__(self, *args: Any) -> None:
        ...
        self.exporters = []

        hardware_exporter = exporter_helpers.get_exporter(
            HardwareExporter(self.charm_dir, self.model.config)
        )
        if hardware_exporter:
            self.exporters.append(hardware_exporter)
        ...

The logic only include the systemd service config file generate, which is because we hide the prometheus-hardware-exporter dependency in charm's packaging, which is working fine.
But this become a issue when now we need to include more exporter services which are not a python package. There is no hook we can use to handle the install process of the required resource/dependency for exporter.

I suggest we split the get_exporter into get_installed_exporters (for getting the installed exporters instances, used in places other than install even) and install_exporters (for installation only, used in install or upgrade events) functions


Add new stream formatters for cert chain identifiers

Commit Message: Adds stream formatters to print the fingerprints and serial for entire downstream cert chain
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]: Fixes #35452
[Optional Deprecated:]
[Optional API Considerations:]

/retest

/assgin @wbpcode @ggreenway

I think the test failures are a flake we're seeing on main as well (quic socket singleton cleanup).
/retest

I am fine to the formatter. But I think the core are these new connection methods. So, will let @ggreenway to review this PR first.

/retest

/retest

I have absolutely no idea why tar is failing:

tar: 9fac9f54c87c48c99d33de59dc23ea66_archive.tar: Wrote only 2048 of 10240 bytes

/retest

I have absolutely no idea why tar is failing:

tar: 9fac9f54c87c48c99d33de59dc23ea66_archive.tar: Wrote only 2048 of 10240 bytes

test infrastructure flake. I'm re-running it.

/retest

/retest

/retest

/retest

Coverage is too low; retry won't fix this. Please try to improve coverage.

https://github.com/envoyproxy/envoy/pull/35513/commits/42615205d5005a5871544358e6f6ed7630fbc6aa was my attempt at approving coverage; the retest I did was for a flake (timeout). I'm looking for other branches to test, but there aren't very many left from what I can see


Scrolling issue with Top aligned Modals

Bug Report
Steps

Go to Top Aligned Modal demo and open modal by clicking on "Show Modal" button.

Start resizing browser window to a smaller size so that the window height is slightly less then a height of a modal.

Observe that its possible to resize window that is hides a lower part of a modal, and no scroll bars are displayed. This doesn't happen to modals without centered={false} setting.

Expected Result
Vertical scrollbar should be displayed, when bottom of the modal is obscured.
Actual Result
Scroll bar is not displayed, and its impossible to reveal obscured bottom part of the modal by scrolling.
Version
0.81.1

I am not able to reproduce. Please check again, if issue is reproduced, please add browser version.

Its Chrome v67.0.3396.99. I think in your case the window is a bit too small, if you slowly resize window by dragging it with the mouse, gradually increasing it's height, you should be able to observe the buggy behaviour.
For example: small window, scroll bar appears:

Slightly taller window (but not tall enough to fit the contents), no scroll bar:

Thanks 👍 As I see that the issue has a same place as #2221, however I will reopen this issue.

Thanks!


Deleted SharePoint Online Client Extensibility App in AzureAD. How to get it back?

Category

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

Hi,
I've accidentally deleted the SharePoint Online Client Extensibility application in AzureAD.
Is there any way to get this back?
Thanks

@Seiyasu could you point me to where I can find the Recycle Bin? Just can't find it anywhere :$

Have you tried to redeploy your package? When you accept the permissions in Central Admin it should recreate the app registration in your tenant. You might have to redo all previous permissions manually through the Azure portal if you don't want to reinstall all packages if you have multiple.

It it the Enterprise App registration that's missing or the 'regular' App registration that's missing? The first one you might be able to find using search in the catalog? I can't test this as mine is already added there. For the App registration the above worked for me, so no other ideas there sorry. You might be able to add it manually and then past in the manifest but I have no idea what else is happening behind the scenes for this App registration that you'd then be missing.

Same issue with me.
"SharePoint Online Client Extensibility web application principal" is missing.
Powershell (SPOTenantServicePrincipal) gives me the error: Unknown Error
Is it possible to Enable the "SharePoint Online Client Extensibility web application principal" back?

@lahuey - is there a workaround for people in this state at the moment?

In the meantime I have restored the App through azure powershell.
It would however be nice if there was some sort of "reset" functionality to ensure the correct settings are set to the app (e.g. the empty app url, single/multi-tenant, ...)

In the meantime I have restored the App through azure powershell.
It would however be nice if there was some sort of "reset" functionality to ensure the correct settings are set to the app (e.g. the empty app url, single/multi-tenant, ...)

Can you please send me the steps to restore the APP through azure powershell

@DnsSrinath sure:
Connect-AzureAD
Get-AzureADDeletedApplication
Restore-AzureADDeletedApplication -ObjectId <your_object_id>

It worked thank you so much..
This saved my day bro :) thanks once again.


400 error when (resumable) uploading files of size > 20 MB

I'm writing an upload script that uploads CSV files to a Google Drive folder using resumable uploads - am using this as a reference
https://developers.google.com/api-client-library/python/guide/media_upload
See the code below.
It seems to work file for files of size <= 20MB, but bigger files are causing problems.
I've got a 43 MB CSV file which I am trying to upload using a chunk size of 5 MiB but am getting a 400 error on the last chunk upload:
file_meta = {'title': 'test.csv', 'description': 'Test CSV', 'mimeType': 'text/csv', 'parents': [{'id': myfolder_key}]}
convert_to_Google_Sheets = True
chunk_size = 1024*1024*5
content = <MediaIoBaseUpload object, resumable, 5 MiB chunk size>
req = drive.files().insert(body=file_meta, media_body=content, convert=convert_to_Google_Sheets)
response is None
while not response:
  status, response = req.next_chunk()
  if status:
    print "Uploaded %d%%." % int(status.progress() * 100)

The upload seems to progress fine until we get to the last chunk, when it terminates with a 400 error:
Uploaded 11%.
Uploaded 23%.
Uploaded 34%.
Uploaded 46%.
Uploaded 57%.
Uploaded 69%.
Uploaded 81%.
Uploaded 92%.
Traceback (most recent call last):
  File "<console>", line 2, in <module>
  File "/usr/local/lib/python2.6/dist-packages/oauth2client/util.py", line 137, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/usr/local/lib/python2.6/dist-packages/googleapiclient/http.py", line 880, in next_chunk
    return self._process_response(resp, content)
  File "/usr/local/lib/python2.6/dist-packages/googleapiclient/http.py", line 907, in _process_response
    raise HttpError(resp, content, uri=self.uri)
HttpError: <HttpError 400 when requesting https://www.googleapis.com/upload/drive/v2/files?uploadType=resumable&convert=true&alt=json returned "Bad Request">

Did you ever mange to solve this issue? I'm getting the same thing.

Oh god, this was a long time ago. I'm not able to recall 100% but I think it was that convert=convert_to_Google_Sheets argument, i.e. had to remove it.

For anybody else reviewing this I can confirm that it was the convert flag. Conversion is still possible, but you need to do something like this...
convert = True
original_mimetype = 'text/csv'
body_mimetype = 'application/vnd.google-apps.spreadsheet' if convert else original_mimetype
file_meta = {'name': 'test.csv', 'mimeType': body_mimetype, 'parents': [{'id': myfolder_key}]}
chunk_size = 102410245
content = <MediaIoBaseUpload object, resumable, 5 MiB chunk size, original_mimetype>
req = drive.files().insert(body=file_meta, media_body=content, convert=convert_to_Google_Sheets)
response is None
while not response:
status, response = req.next_chunk()
if status:
print "Uploaded %d%%." % int(status.progress() * 100)

Thanks.


What is a regional scale set?

The term regional scale set is never defined on this page or linked.
Is a regional scale set, just a scale set with Availablity Zone set to none?

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

ID: e5880e85-75e2-21b0-0f7e-b7541bdc04ae
Version Independent ID: 762f79cf-563d-1bb7-ddae-8c53131abaf6
Content: Create an Azure scale set that uses Availability Zones
Content Source: articles/virtual-machine-scale-sets/virtual-machine-scale-sets-use-availability-zones.md
Service: virtual-machine-scale-sets
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for the feedback! We are currently investigating and will update you shortly.

@MSIH Sorry for the confusion. A regional scale set is a non-zonal scale set i.e no availability zones selected. I have update the doc. I ave created a PR and changes will be live shortly.
You can also find a mention of it in the following doc:
https://docs.microsoft.com/en-us/azure/virtual-machine-scale-sets/virtual-machine-scale-sets-faq#do-scale-sets-work-with-azure-availability-sets
Thanks for bringing this to our attention. We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


Fix compile error with questa, for issue #242

This PR fixes #242 by using a deferred constant. It shouldn't change anything except allowing to compile the package with Questa, which failed before.

Thanks for fixing!


SplitEx/Substr/Reverse support BinaryString as input parameter

Currently GeaFlow uses BinaryString as its implementation for string, which provides better performance. However, at present, the built-in String Function uses string as the input parameter, and a direct conversion from BinaryString to String is required in the calculation phase, which has performance overhead. Implement the SplitEx/Substr/Reverse function with BinaryString as an input.

Hi @pengzhiwei2018  may I know if I can work on this issue?  and to make it clear, is it about adding an extra implementation for SplitEx/Substr/Reverse or updating the existing funtions?


[PERF TODO] Log and output more elegance.

Requirement：

colorful
uniform interface
do not specify \t \n repeatedly in code
especially, in Evaluate module

cdk output not all in stdout:
➜  cdk ./cdk_linux_amd64_thin_upx eva > /tmp/1
2022/05/18 10:33:51 current dir: /root/cdk
2022/05/18 10:33:51 current user: root uid: 0 gid: 0 home: /root
2022/05/18 10:33:51 hostname: VM-4-6-ubuntu
2022/05/18 10:33:51 debian ubuntu 20.04 kernel: 5.4.0-65-generic
2022/05/18 10:33:51 sensitive env found:
	SSH_CLIENT=<IP_ADDRESS> 36890 22
2022/05/18 10:33:51 sensitive env found:
	SSH_CONNECTION=<IP_ADDRESS> 36890 <IP_ADDRESS> 22
2022/05/18 10:33:51 sensitive env found:
	SSH_TTY=/dev/pts/1
2022/05/18 10:33:51 service found in process:
	800	1	sshd
2022/05/18 10:33:51 service found in process:
	845	1	dockerd
2022/05/18 10:33:51 service found in process:
	1431	845	docker-proxy
2022/05/18 10:33:51 service found in process:
	65871	1	docker
2022/05/18 10:33:51 service found in process:
	65896	845	docker-proxy
2022/05/18 10:33:51 service found in process:
	3141627	800	sshd
2022/05/18 10:33:51 service found in process:
	3143372	800	sshd
2022/05/18 10:33:51 available commands:
	curl,wget,nc,netcat,kubectl,docker,find,ps,python3,php,node,npm,apt,dpkg,apache2,ssh,git,vi,capsh,mount,fdisk,gcc,g++,make,base64,python2,python2.7,perl,sudo
2022/05/18 10:33:51 Capabilities hex of Caps(CapInh|CapPrm|CapEff|CapBnd|CapAmb):
2022/05/18 10:33:51 net.ipv4.conf.all.route_localnet = 0
2022/05/18 10:33:51 checking if api-server allows system:anonymous request.
2022/05/18 10:33:52 failed to dial Alibaba Cloud API.
2022/05/18 10:33:53 failed to dial Azure API.
2022/05/18 10:33:53 failed to dial Google Cloud API.
2022/05/18 10:33:54 failed to dial OpenStack API.
2022/05/18 10:33:55 failed to dial Amazon Web Services (AWS) API.
2022/05/18 10:33:56 failed to dial ucloud API.


[flutter_tools] remove trailing eth info from fuchsia package server

Description
This is not valid ipv6

Landing on red to fix build, hopefully

Can you explain this change a bit more? The trailing info is valid there IIRC. See in particular under "Use of zone indices in URIs" in https://en.wikipedia.org/wiki/IPv6_address.

Also, can you link to the github issue that this is fixing? Thanks!

Maybe it was the URI formatting that was throwing it off?
specifically https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket.appspot.com/8882089532278067040/+/steps/Run_Fuchsia_Driver_Tests/0/steps/collect/0/logs/task_stdout_stderr:_flutter_fuchsia_driver_tests/0
[ +726 ms] executing: ssh -F /b/s/w/ir/fuchsia_ssh_config <IP_ADDRESS> amber_ctl add_repo_cfg -n flutter_tool -f http://[fe80::ec4:7aff:fecc:ea8f%25eno2]:34283/config.json
[  +48 ms] 2020/04/25 01:14:37 error adding repo: failed to GET file: Get "http://[fe80::ec4:7aff:fecc:ea8f%25eno2]:34283/config.json": dial tcp [fe80::ec4:7aff:fecc:ea8f%eno2]:34283: errno(111)
[        ] Failed to teach amber about the package server

ahh, %25, I missed that last time


fix: Category page issues

Add app to cate, can not choose app
Refactor hide create btn when category reserved

fix #1057


Flutter APK Not Compatible with Client's Samsung S24

I’ve generated a Flutter APK that works fine on my phone having android version 14, but when I sent it to my client through Whatsapp, who is using a Samsung S24, they received the error: "App not installed App not compatible with your phone."
Details:
Target SDK Version:  34
Min SDK Version: 21
Build Type: APK (Release)
Steps to Reproduce:
I generated a release APK using the following command:
flutter build apk --release --split-per-abi --no-shrink
The APK works as expected on my device.
I sent the APK via WhatsApp to my client, who is using a Samsung S24.
Upon trying to install, the client received the error: "App not installed App not compatible with your phone."
What I’ve Tried:
Verified that the APK works on other devices.
Checked that the minSdkVersion and targetSdkVersion are compatible with the client’s device.
The app should install and run successfully on the Samsung S24.
Actual Behavior:
The client receives the error: "App not installed App not compatible with your phone."
Additional Information:
Client’s device: Samsung S24
Distribution method: APK sent via WhatsApp
Questions:
Are there any known issues with Flutter APK compatibility on Samsung devices, specifically the S24?
Could there be a problem related to ABI or SDK versioning that I might be missing?
Any guidance or suggestions would be greatly appreciated!

@abbskhan
Does S24 has Android 14 version ? Can you provide S24 device details (OS version) if not 14 ?

the client received the error: "App not installed App not compatible with your phone."

Is it possible to get more details about it ? example, the error log if any ?
This may also be an issue S24, as I see similar thread here about it https://www.reddit.com/r/samsunggalaxy/comments/19evjr3/s24_old_app_not_compatible/ and https://www.reddit.com/r/oneui/comments/19eq6nt/unable_to_install_apks_in_the_samsung_s24/

the android version is 14 and the app is working on other phones having android version 14 but didnt install on s24+

This may also be an issue S24, as I see similar thread here about it https://www.reddit.com/r/samsunggalaxy/comments/19evjr3/s24_old_app_not_compatible/ and https://www.reddit.com/r/oneui/comments/19eq6nt/unable_to_install_apks_in_the_samsung_s24/

Did you get a chance to look at these links and try any solutions mentioned in them ?

Yeah,i have looked that conversations.
the main conclusion was
it seems like s24 is 64 bit arch cpu and these apps are all 32 bit.
and the
adb install --bypass-low-target-sdk-block GD.apk
how can i check that my apk is 32 bit or 64

Can you try by running flutter build apk --release --target-platform=android-arm64 and then share this apk to the client and check if it works ?

sure ill do it like this if i got back to work will inform you then
and thanks alot for your time

Any news on that?
I'm having the same issue and I've tried:
flutter build apk --target-platform=android-arm64 --obfuscate --split-debug-info=$debugInfoFolder


Feature/list views

Expand the list view and grids
Renders

TODO: fix selection on the ingredients page
TODO: debug the edit form
TODO: click through filters from ingredients > recipes (show recipes using ingredients)


Fix a null pointer in Get-ManagedInstanceGroup

I just realized I should add a test for this. Doing that now.

Fixing the tests is hard because we have problems with regional managed instance groups.


The launcher encountered Error: Assertation failed (gameSegmentZips.length > 1) on HSR. idk how to fix that

I download yaagl open it everything is fine when i install however it shows me an error (sc below)

macOS: 13.2.1 Ventura
Device:  MacbookPro 13.3 inch
CPU: Apple M1
RAM: 8GB

same to me
macOS 15.1  M1 pro

It's a 7zip decompression error. We don't support 7zip decompression. You need to get the game files elsewhere and move over.


Refactor namings form to make form available outside layout

This just makes the form available as a partial.
(The naming fields are already in their own partial, that gets included in both the naming and the observation form.
This new form partial just calls the fields partial. But we will need an intact form partial for the Observation identifier UX.)

Coverage: 93.346%. Remained the same when pulling 86e05014e5d2b68f08cee64b2b93ae8405c9828f on nimmo-naming-form-fix into d1c014af1e93ddf7c1186390d99027b2239f5648 on main.


Only send websocket data when changed

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Resolves #7


Python Datatypes

Description 📜
Hello , In This Repo We Will Be Focusing On Python Programming Language , Java Programming Language and C/C++ Programming Language

Type of change 📝

[ ] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Video (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[x] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[x] Java_#Domain
[ ] Java#Script
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[x] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

I have solved issue #245. Please merge it.

@adityabisoi Close this pr


skeleton for helm-dashboard-v2

Changes Proposed
Created the skeleton for the dashboard with mock data

Check List

[ x] The title of my pull request is a short description of the changes
[x ] This PR relates to some issue:
[ x] I have documented the changes made (if applicable)
[ ] I have covered the changes with unit tests

@chad11111 please have a look at this.

@chad11111 made the changes please have a look


Update request: fahclient 8.3.1 → 8.3.3

Package name: fahclient
Latest released version: 8.3.3

Current version on the unstable channel: 8.3.1
Current version on the stable/release channel: 7.6.21

[x] Checked the nixpkgs pull requests

Notify maintainers

@zimbatm

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

done in https://github.com/NixOS/nixpkgs/pull/295745


MacOS Installation Issues Fixed

Dependencies related to python and numpy are fixed
Instructions to add path to poetry while using a bash shell are added

@exook @fritjof-b

Hi @sanam2405 , please try to avoid editing the Python in the pyproject.toml file, we've just tailored it because there is an underlying bug in PyTorch which affects the choice of Python version (see 40acf8d and 4186f63).
There is actually some difference between .bashrc, .bash_profile, and .profile, which you can read about here. Because of this, sourcing any of those files has no effect if poetry is not correctly prepended to the PATH variable in the files themselves. This is a fault of the guide.
We've had this discussion earlier, but I don't believe we (Baler team) should own any setup of specific software such as WSL or Poetry. That setup should be left to those individual organisations. This leads me to believe that we should remove (or heavily alter) this guide completely, and rather direct users to https://python-poetry.org.
Adding numpy does not have any effect, as it is a "dependency of a dependency" (see attached screenshots).

As an endnote, I know you mentioned that you use an M1 Mac, which works fine (I do as well), and that if you had any installation issues you could reach out on Slack. Therefore I'll probably reject this PR. :-)
Wdyt @exook

Hey @fritjof-b, while installing baler on my M1 Mac, the issues I faced are as follows :-

There was no file named .profile

Fix I used - I noticed that I was using a bash terminal and I did not have a .profile file. Instead I had a .bash_profle file. On further exploring I got to know about the .bashrc .bash_profile and .profile files. Also the blog you provided cleared things more.
Check that I don't have a .profile file here but I do have .bash_profile which I used.

While running the below command

poetry run python baler --project CFD_workspace CFD_project_animation --mode train
I got the error ModuleNotFoundError: No module named 'numpy'
Fix I used - I noticed there is no numpy mentioned in thepyproject.tomlfile. So I ran the command: poetry add numpy
On running the command I got into another error that said python versions are mismatching. So I manually resolved this by assigning the python version as python = "^3.9" which fixed the issue. Then I ran the command poetry add numpy again and it ran successfully. The changes that were made once this command ran are -

numpy was added in the pyproject.toml file as numpy = "^1.25.0"
changed the poetry.lock file

Once these changes were made, I could run baler for compressing and decompressing the CFDs.
I suggest that numpy version be added in the [tool.poetry.dependencies] in the pyproject.toml file, or otherwise add an instruction saying to run the command poetry add numpy as it shall show any dependency conflicts or missing dependencies related to numpy or python version.

@sanam2405 To me it looks like you're using zsh, not bash. Since zsh is the default shell on macOS. To confirm you can run:
echo $0

You're also using conda, which can be seen from your prompt and config files. Conda probably interferes with your environment and poetry. There is no way for me to debug this on your computer.
I installed Poetry and Baler on a fresh macOS installation a couple of days ago (and before) and did not run into the problems you are facing. These facts increase the probability that there is something not "right" with your local setup. There is no way for me to debug this for you.
To give you some things to look into:

How did you install python?
How are you managing python versions and virtual environments?
Is Anaconda interfering with Poetry?

@fritjof-b I just checked and I confirmed that I am using zsh shell and not bash

I installed my python using pip. Before installing and using poetry, I have been using pip as my package manager
I am using poetry to manage versions and virtual environments as of now
I am not sure but I do not think that Anaconda should interfere with Poetry

The python versioning error popped up both in case of my macOS and Linux. The error got removed on changing the version from python = "^3.8" to python = ">=3.8, <3.11"
As an end note, I think the installation guide for python should include some direct reference link to poetry and avoid instructions that are specific to a particular machine, architecture or shell.


Fixing btoa method within cartodb.js utils

Fixes #692.

:+1: Vamos neno!
cc @iriberri @oriolbx

@ztephm: this will fix the IE9 issue :)

great thanks guys!

We just did a new release and this fix is included in CartoDB.js 3.15.8.
Great job with the fix @xavijam!

No no, was @rochoa :D

Great job with the fix @rochoa! :)

:see_no_evil:

:revolving_hearts:

:+1:


added the getwidget repo

@techtrailhead
Errors (1):
--------------------------
  x /home/runner/work/devlibrary/devlibrary/config/flutter/repos/getwidget.json has invalid authorId "pietechsolution", no such .json file exists


PIxel type compatibility

Check Brainglobe atlas kim_dev_mouse_idisco
See https://forum.image.sc/t/mri-template-for-ccfv3-space/101219/7

How does the original Allen atlas work in ABBA? That has lots of these v. high integers right?
I agree there appears no need to use these values, but they seem to have been decided by the Allen (I presume for good reasons I'm not aware of). We debated changing these within BrainGlobe, but that seemed a step too far towards standardisation.

How does the original Allen atlas work in ABBA?

I cheated because at the time I did not know how to circumvent the 16 bits unsigned int limitation. So I looked for a way and found that there was still unicity of the labels if you modulo them by 65000. So I re-indexed the labels to labels mod 65000 (https://zenodo.org/records/7492551).
At the time I did not think of supporting other atlas and this trick worked for a while. I guess it's time to move on.

Yeah, unfortunately the 32 bit integers are here to stay. New atlases are (sensibly) using Allen values for regions not originally segmented in the Allen atlas, so there are many atlases that need to be 32 bit now.

I realized I support 32 bits signed, but I need to add the unsigned ints


whois component: doesn't work with polish domains

Home Assistant release (hass --version): 0.57.2
Python release (python3 --version): Python 3.5.3
Component/platform: whois
Description of problem: doesn't get details from polish WHOIS database / polish domains
Expected: give data same as for other domains
Problem-relevant configuration.yaml entries and steps to reproduce:
- platform: whois
  domain: github.com
  name: WHOIS GitHub

- platform: whois
  domain: onet.pl
  name: WHOIS onet

- platform: whois
  domain: wp.pl
  name: WHOIS wp

Good catch. Those were due to an assumed existence of the registrar key returned from the whois lookup. This key isn't present in the Polish lookups provided, I'm just testing some changes that will check for the registrar key before setting the attribute. In the case of the ones provided, this will lead to None due to registrar not being present.
Will be submitting PR shortly.

Please see the local updates working.
Github.com attributes:

wp.pl attributes:

So now, if the whois lookup response doesn't give the following, they will not be present in the attributes:

name server list
updated date
registrar name

It will now error out if the expiration date (required for the sensor) is not present.
Essentially, a PR to improve the resilience of the whois sensor.
Thanks again for the report.

Did the fix get into the 0.59?
It seems it did, but there's still error in status:
Dec 03 22:32:24 srvr hass[23693]:   File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run Dec 03 22:32:24 srvr hass[23693]:     result = self.fn(*self.args, **self.kwargs) Dec 03 22:32:24 srvr hass[23693]:   File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/sensor/whois.py", line 47, in setup_platform Dec 03 22:32:24 srvr hass[23693]:     if 'expiration_date' in get_whois(domain, normalized=True): Dec 03 22:32:24 srvr hass[23693]:   File "/srv/homeassistant/lib/python3.5/site-packages/pythonwhois/__init__.py", line 8, in get_whois Dec 03 22:32:24 srvr hass[23693]:     return parse.parse_raw_whois(raw_data, normalized=normalized, never_query_handles=False, handle_server=server_list[-1]) Dec 03 22:32:24 srvr hass[23693]:   File "/srv/homeassistant/lib/python3.5/site-packages/pythonwhois/parse.py", line 516, in parse_raw_whois Dec 03 22:32:24 srvr hass[23693]:     data["registrar"].insert(0, match.group(1).strip()) Dec 03 22:32:24 srvr hass[23693]: KeyError: 'registrar'

I can take another look tomorrow for you. What's your config? You can email me at adam(at)genericstudent.com if you don't want to privately share.
Immediately seems to be an assumption also made in the source package.

E-mail sent

@andriej, It seems the issue here is both the package (pythonwhois==2.4.3) used and the whois data returned. On one of the domains you supplied, the whois responder (whois.eu) didn't return any expiration / renewal date (the KeyError you received). On the other non-working domain, it returned a field renewal date which doesn't seem to be parsed by the pythonwhois package.
I'm afraid this is an upstream package issue (the original dev hasn't updated it in a while either) so nothing we can do here unless an update is pushed upstream.
I'm going to send the original dev an email if they could hand the project over for more progress.
Sorry I couldn't help with this one .


Flaky test: LargestFirstMemoryManagerTest

Test name(s)

org.apache.accumulo.tserver.memory.LargestFirstMemoryManagerTest.test

Describe the failure observed
java.lang.AssertionError: expected:<1;g<> but was:<1;f<>
	at org.apache.accumulo.tserver.memory.LargestFirstMemoryManagerTest.test(LargestFirstMemoryManagerTest.java:113)

This error occurred on Jenkins while testing an unrelated PR, but when I tried to reproduce it manually, the test passed. It's not clear to me how this could happen, but should be investigated.

@ctubbsii is this different from #1789?

@ctubbsii is this different from #1789?

Looks the same. I will close as duplicate.

The line numbers in the error on this one match up with the latest revision. The line numbers in the older error in #1789 are out of date due to changes to the file on 12/8/20.


[Auto Release] Update istio.Version

Update istio.Version

@xiaolanz: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-no_auth.sh
212fa674e3391ab703302a72a0c4582f3d4310ee
link
/test e2e-suite-rbac-no_auth

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

@xiaolanz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-no_auth.sh
212fa674e3391ab703302a72a0c4582f3d4310ee
link
/test e2e-suite-rbac-no_auth

prow/e2e-suite-rbac-auth.sh
212fa674e3391ab703302a72a0c4582f3d4310ee
link
/test e2e-suite-rbac-auth

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

Are you positive the same api version is used in every repo ? Also once you run the first command, you need to wait for the build jobs to complete.

Pilot seems to be broken again.

I will make a note to release guide to "wait for the build jobs" I have a PR in progress.

abandon this PR.


[Improvement] Interview-Questions

Detailed description
In interview questions page when you select a topic it does not have a visual indicator that it is selected .
Context
It's confusing to not know which element is selected
Possible implementation
the selected topic is highlighted in orange
Additional information
No response
Code of Conduct

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

i would like to fix this

@seifnamissi could you add screenshot/short video?

Uploading screen-recorder-fri-may-03-2024-20-49-21.webm…

screen-recorder-fri-may-03-2024-20-49-21.webm

@seifnamissi please, sync the new changes.
It was already implemented in yesterday's merge, so I close this issue.

@seifnamissi By the way, thanks for trying :)

@seifnamissi I checked from mobile, weird , in mobile it works, but on desktop not. Would you like to handle this?

yes , i'll handle it .

@seifnamissi Thanks :)


Add checks to roi_heads in detection module

This PR addresses #1027 by:

adding checks on boxes, masks, keypoints and labels attributes of target passed to detection models.
updating docs

Codecov Report

Merging #1091 into master will decrease coverage by 0.72%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1091      +/-   ##
==========================================
- Coverage   64.65%   63.92%   -0.73%
==========================================
  Files          68       68
  Lines        5410     5417       +7
  Branches      830      754      -76
==========================================
- Hits         3498     3463      -35
- Misses       1662     1692      +30
- Partials      250      262      +12

Impacted Files
Coverage Δ

torchvision/models/detection/faster_rcnn.py
74.39% <ø> (ø)
:arrow_up:

torchvision/models/detection/mask_rcnn.py
81.35% <ø> (ø)
:arrow_up:

torchvision/models/detection/keypoint_rcnn.py
81.81% <ø> (ø)
:arrow_up:

torchvision/models/detection/roi_heads.py
55.77% <0%> (-1.13%)
:arrow_down:

torchvision/datasets/cifar.py
71.91% <0%> (-6.75%)
:arrow_down:

torchvision/transforms/transforms.py
78.74% <0%> (-2.76%)
:arrow_down:

torchvision/datasets/folder.py
80% <0%> (-2.5%)
:arrow_down:

torchvision/datasets/lsun.py
18.47% <0%> (-2.18%)
:arrow_down:

torchvision/datasets/voc.py
20% <0%> (-2.11%)
:arrow_down:

torchvision/transforms/functional.py
69.36% <0%> (-2.03%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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

Sorry for the delayed response. Please have a look at my response @fmassa .

@fmassa Made the changes!


angular 2 translate on ionic2 + TS behaves differently depending amount of arguments

I'm submitting a ...  (check one with "x")
[ X] bug report => check the FAQ and search github for a similar issue or PR before submitting
[ X] support request => check the FAQ and search github for a similar issue before submitting
[ X] feature request

Current behavior
When executing code with different arguments, the library returns different results based on them, althought that is never stated on the documentation. examples:
When using 2 arguments, it will return an array:
this.translate.get(['appname','loading']).subscribe((translation: [string]) => {
      SpinnerDialog.show(translation[0],translation[1],false);
    });

When using 1 argument, it will return an object based on the return type params (that's ok):
this.translate.get('appname').subscribe((translation: string) => {
      SpinnerDialog.show(translation,"",false);
    });

when using 3 arguments, it will return an object ( this is wrong, should return an array instead, right')
this.translate.get(['appname','unable_complete','ok']).subscribe((translation: [string]) => {
          console.log(translation);
          let translationLet : any =  translation;
          console.log(JSON.stringify(translationLet));
          let alert = this.alertCtrl.create({
            title: translationLet.appname,
            subTitle: translationLet.incorrect_credentials,
            buttons: [translationLet.ok]
          });
          alert.present();
        });

Expected/desired behavior
There's no need to change the behaviour of it, BUT, it should be mandatory to Write in the documentation how it exactly works so nobody gets stuck for over a quarter of a day guessing why the array position [0] will return undefined always.

Can you create a plunkr for that behavior because I'm not seeing it. When using an Array of keys, it should always return an object. Either when using 1, 2, 3 or more values.

I am unable to reproduce it, might be due to a forced upgrade of almost every plugin i did this morning for a push to prod.Still,  wouldn't it make sense that when providing a collection, the output also be in a collection? anyway, the object with properties return value is not properly documented or i  might have missed it. For sure the angular 1 version returned an array.

it might indeed be missing in the documentation, also there's a PR to change this behavior in some cases: https://github.com/ngx-translate/core/pull/412
I haven't had time to take a deep look at this yet

Checked again, can't reproduce.
the section to perfectionate is 4. , currently only shows default 1-param behaviour.

Use the service, the pipe or the directive:

You can either use the TranslateService, the TranslatePipe or the TranslateDirective to get your translation values.
With the service, it looks like this:
translate.get('HELLO', {value: 'world'}).subscribe((res: string) => {
console.log(res);
//=> 'hello world'
});

Hello, I'm closing this issue because it's too old.
If you have a similar problem with recent version of the library, please open a new issue.


A bit logic improvement

Hi there,
During a code testing with usage of the dependency-check cli we've found out that with several report types set in cli arguments it runs as much times as output formats specified (-f flag).
I'd like to suggest improvement as it dosen't make sense to run scan in this way. I believe it should be better to run scan one time and the report generation as much as was specified in cli args :)

Thanks for noticing the error and submitting the PR!  We are likely going to be pushing a release this weekend.


Firewall ports

Great work with the project.  We're in the process of testing this out in a 3 DC environment.
We've been leveraging the Dockerfile in the conf with some tweaks to make it more dynamic in loading in various configs.
That being said we're struggling with getting Dynomite running within Docker's bridge (default) network.  If we run it in the host network all seams file.  It appears as if we need to open up more then ports 8101-8102 for cross dc/rack communication.
If we simply run the container with "-p 8101:8101 -p 8102:8102", the nodes can't communicate with each other.
Can you confirm which ports need to be enabled?  It would be great if there was a new item published in the wiki.
Thanks!

Hi
8101 is a peer to peer communication port
8102 is the default client port
22122 is the default port on which dynomite talks to underlying redis (not
outside the node)
22222 is the default stats control port
What is the issue that you see?
On Mon, May 8, 2017 at 8:18 AM, slopokeit<EMAIL_ADDRESS>wrote:

Great work with the project. We're in the process of testing this out in a
3 DC environment.
We've been leveraging the Dockerfile in the conf with some tweaks to make
it more dynamic in loading in various configs.
That being said we're struggling with getting Dynomite running within
Docker's bridge (default) network. If we run it in the host network all
seams file. It appears as if we need to open up more then ports 8101-8102
for cross dc/rack communication.
If we simply run the container with "-p 8101:8101 -p 8102:8102", the nodes
can't communicate with each other.
Can you confirm which ports need to be enabled? It would be great if there
was a new item published in the wiki.
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Netflix/dynomite/issues/446, or mute the thread
https://github.com/notifications/unsubscribe-auth/ALle57Z8qacqOsycPsMytPkhLg47l6X_ks5r3zJAgaJpZM4NUGE4
.

My issue appears to be the same as described in #441.  I can only get a multi node cluster running if the container is running as --network=host
The only thing I could could come up with is that the inter-cluster communication relied on some other port other then 8101-8102


Use of EmptySchema in GET/DELETE request specification is incorrect

Noted by Gareth Sylvester-Bradley:
The 'EmptySchema' used in the APIs is intended to indicate that the request body can be empty, but in reality if you run it through a validator it requires an empty object to be present '{}'.
This should be replaced with one of the following in the RAML to fix this. It doesn't appear to be possible to fix this correctly using JSON schema directly:

Currently preferred: Body parameter removed for requests with no request payload

get:
  description: List of paths available from this API
  responses:
    200:

Body set explicitly to null. Note that when rendered as HTML this indicates that the request body must still have a content type of application/json which isn't mandatory without a request body present.

get:
  description: List of paths available from this API
  body: null
  responses:
    200:

Similar to 2, but using !! to avoid 'null' being interpreted as a literal. This usage isn't 100% clear from the RAML 0.8 spec, but does still appear to validate.

get:
  description: List of paths available from this API
  body: !!null
  responses:
    200:

Also heartbeat POST?

Yes indeed. It affects everywhere that the EmptySchema is used at present.

I think the issue is resolved well by the proposed pull request.
Pedant mode on again! For the heartbeat POST, is any kind of body allowed? What should the Content-Type header be ... if it is application/json and I post content of length zero, is this invalid JSON (i.e. not {} or [])? Perhaps something in the docs to say the POST should have an empty message indicated by Content-Length: 0 and maybe no Content-Type?
(Sorry if I missed that somewhere?)
Richard

I believe the Content-Type header can be omitted where there is no message body, and as you suggest this may well be the best option in this case to avoid getting into CORS issues with GET requests. Having said that I wouldn't necessarily want a server implementation to reject a message because it happens to have a payload, for example signalled as type text/html as it shouldn't be interpreted. Perhaps this ought to be added along the lines of 'API clients SHOULD send an empty body with no Content-Type specified for any API methods which don't specify a request body', with nothing specified on the server side?
On Content-Length I'd note that I've certainly observed client libraries in the past which fail to set this value, so I wouldn't necessarily want to mandate it and force people not to use certain HTTP client libraries.

Thanks. I like that approach to Content-Type. I assume this is an extra section in section 2.2 of the docs?
Treating Content-Length: 0 and no content length header present seems to be accepted practice or arguably even expected behaviour, e.g.:
http://lists.w3.org/Archives/Public/ietf-http-wg/2010JulSep/0275.html
Therefore, I would agree that no further documentation in NMOS is required for this aspect.

Yes I think that makes sense. I'll add a note into 2.2 as part of the same PR.


'SKIP_PREFLIGHT_CHECK' is not recognized as an internal or external command, operable program or batch file.

Hi, after yarn install done and i write yarn start i get this error !  :
'SKIP_PREFLIGHT_CHECK' is not recognized as an internal or external command, operable program or batch file.
how can i fix it ?
thanks

Hi @navidsotoudeh, I didn't have any errors when clone the repo and try to run it pls check your node version and yarn version.


Functional Spec | Winter 2019

The functional spec for the sprints is included in a link below.
Could you please review and comment on this by Tuesday 18th December in order for us to discuss in the call on Wednesday.
https://docs.google.com/document/d/1KzNNVXD7sylvr9B7GAMlOgKvpAn9aymeTwhUbWTikWk/edit?usp=sharing

@danblah We have updated the functional spec with the extra requirements around Review Roles and Review Consensus that we discussed in the last meeting.
If you could review and confirm this is how you expect it to work that would be great.

Just a further comment on this to give us the go official go ahead on this and also identify if there are any areas that are lower priority.

This looks good to me, thanks!


typing error in quick-start-with-vim-go.md file

BUG REPORT

What happened:

Inappropriate heading tags in quick-start-with-vim-go.md file which resides inside the directory documentation/techbits/. Current scenario most of the heading tags are ## it should be changes to ### to maintain the flow.

What you expected to happen:
Replace the## with ### heading tags.

@utkarshmani1997 please review

@sachinmukherjee - can you please verify and close this issue if #1511 has made the expected changes.

Is it still open?
@kmova, can I take this up?

@NisantaSahoo @kmova, is the issue still open?
If yes, can I take up the task?

@utkarsh2102 .. Can you post on the above PR requesting to take this up. You can wait for a day for them to respond and take it up.

Sure, would do as asked :)


Add quantum resource estimation demo

Title:
Quantum Resource Estimation
Summary:
Relevant references:
Possible Drawbacks:
Related GitHub Issues:

PR was closed automatically as the dev branch was recreated. Re-opened the PR and now it targets master.
If the demo does use any post-v0.26.0 features from PennyLane, the target branch should be changed again to dev.

Looks good! Couple general comments:

I know the applications for this module in PennyLane only reach QChem, but the placement of the demo (in QChem demos) seems off at first glance
Should we be more explanatory when introducing terms like "one-electron integrals" or "first-quantized Hamiltonians"?

Thanks @isaacdevlugt.
I agree with your concern that this demo is about a general topic, resource estimation, but it is too qchem-specific in its current form. The reason is that the qml.resorce module supports very advanced qchem algorithms at the moment and we cannot really have a general demo. This also explains why the demo is technical in terms of qchem jargon. For these reasons, I prefer the demo to appear in Quantum Chemistry for now. This also gives the users a hint that they should expect a lot of qchem when they want to estimate quantum resources.
In the future, when we support more general algorithms in qml.resorce, we can expand the demo and bring the qchem part to a dedicated section.
I can provide more links to our demos and wiki pages for the qchem concepts. But explaining the concepts here, very likely, changes the focus of the demo.
Please let me know what you think.

Agreed! I like the idea of re-titling the demo. Maybe, "Resource estimation for quantum chemistry"? Or, we can put it into two demo categories as was stated in a different thread.

great job! We will announce it this Wednesday morning 🚀