Fix ESLint issues (Errors) - react-hooks/rules-of-hooks

Running npm run lint in the command line shows various ESLint issues that we should fix. The issues are split into warnings (yellow-coded) and errors (red-coded). Within warnings and errors, the issues are split into categories by ESLint rule name.
Example:

Fix all errors under the category react-hooks/rules-of-hooks. Please make sure to regression test so that we know nothing has broken through these changes.
List of issues (May not be exhaustive so double check):

[ ] wtt_front/client/src/api/queries.js

[ ] 129:12  error  React Hook "useQuery" is called in function "preProcessor" that is neither a React function component nor a custom React Hook function. React component names must start with an uppercase letter. React Hook names must start with the word "use"  react-hooks/rules-of-hooks

[ ] wtt_front/client/src/pages/NewTree/NewTree.js

[ ] 21:25  error  React Hook "useAuth0" is called conditionally. React Hooks must be called in the exact same order in every component render. Did you accidentally call a React Hook after an early return?  react-hooks/rules-of-hooks
[ ] 49:23  error  React Hook "useForm" is called conditionally. React Hooks must be called in the exact same order in every component render. Did you accidentally call a React Hook after an early return?   react-hooks/rules-of-hooks

[ ] wtt_front/client/src/pages/Tree/treeutilities.js

[ ]    8:23  error  React Hook "useTreeQuery" is called in function "updateOrCreateTree" that is neither a React function component nor a custom React Hook function. React component names must start with an uppercase letter. React Hook names must start with the word "use"               react-hooks/rules-of-hooks
[ ]    9:26  error  React Hook "useTreeDataMutation" is called in function "updateOrCreateTree" that is neither a React function component nor a custom React Hook function. React component names must start with an uppercase letter. React Hook names must start with the word "use"        react-hooks/rules-of-hooks
[ ]   10:26  error  React Hook "useCreateTreeDataMutation" is called in function "updateOrCreateTree" that is neither a React function component nor a custom React Hook function. React component names must start with an uppercase letter. React Hook names must start with the word "use"  react-hooks/rules-of-hooks

Hello,
I can work on this issue too. Please, assign !


Script still running when stopped

I noticed that they uploaded a new version and this version will keep running even you stopped the script (ctrl+c) been 5hrs now since I stopped and deleted all the getgrass-bot files my dashboard stills says that there are devices connected and I’m earning 0

me too

this is because the web grass still has a lot of bugs, don't worry too much about it and if every morning your points are dropped it's a bug again, we all experience it and points will be normal late in the day

I noticed that they uploaded a new version and this version will keep running even you stopped the script (ctrl+c) been 5hrs now since I stopped and deleted all the getgrass-bot files my dashboard stills says that there are devices connected and I’m earning 0

Bug (glitch) on Grass dashboard.

I noticed that they uploaded a new version and this version will keep running even you stopped the script (ctrl+c) been 5hrs now since I stopped and deleted all the getgrass-bot files my dashboard stills says that there are devices connected and I’m earning 0

This my problem too


Run testing outside the project

Here is my bolt-testing.js:
var bolt = require('firebase-bolt');
var rulesSuite = bolt.rulesSuite
when I run mocha bolt-testing.js --ui tdd I will get this error
TypeError: rulesSuite is not a function
It seems that rulesSuite is not exported in bolt.js, how can I import rulesSuite?
Thanks

That looks like the way I recommended using it - perhaps I accidentally removed the export at some point.
As a work around you could try:
var simulator = require('firebase-bolt/simulator');
var rulesSuite = simulator.rulesSuite;


When AllowBulkExecution is true, ReadItemAsync and ReadItemStreamAsync will never yield NotModified on a IfNoneMatchEtag, even when it is expected

Describe the bug
The optimistic concurrency feature If-None-Match does not work when AllowBulkExecution is enabled. 304 Not Modified is never observed (even when we would expect it due to a matching ETag).
To Reproduce
// Microsoft.Azure.Cosmos v3.37.0

var myAccountEndpoint = "example";
var myAccountKey = "example";

var client = new CosmosClient(myAccountEndpoint, myAccountKey, new CosmosClientOptions() {
    // This bug does not occur if the following is false
    AllowBulkExecution = true
});

var myDatabase = "example";
var myContainerName = "example";

var container = client.GetContainer(myDatabase, myContainerName);

var myDocumentId = "1234";
var myPartitionKey = "abcd";
var myCachedETag = "xxxx";

using var response = await container.ReadItemStreamAsync(
    myDocumentId,
    new PartitionKey(myPartitionKey),
    // This If-None-Match never successfully yields a not modified,
    // even when the correct/latest ETag was supplied!
    new ItemRequestOptions() { IfNoneMatchEtag = myCachedETag },
    CancellationToken.None
);

// response.StatusCode is always HttpStatusCode.OK
// There is no CosmosException thrown where exception.StatusCode == HttpStatusCode.NotModified

Expected behavior
A 304 Not Modified should be observed (via a thrown CosmosException) when the document ETag matches the latest version.
Actual behavior
The result is always 200 OK, with the full resource present in the response body.
Environment summary
SDK Version: 3.37.0
OS Version: Windows
Additional context
I spent a bit of time stepping through Microsoft.Azure.Cosmos. I can see that the SDK is correctly writing a value for ifNoneMatch in the ItemBatchOperation.WriteOperation method, but the resulting TransactionalBatchOperationResult is always a 200 OK.

Duplicate of https://github.com/Azure/azure-cosmos-dotnet-v3/issues/1581

Oh, I see. My apologies for not identifying the existing issue.

Closing as duplicate. This is a bug on the service.


Destroy fails on an empty state

I have issues
Destroying on empty state.
I'm submitting a...

[x] bug report
[ ] feature request
[ ] support request
[ ] kudos, thank you, warm fuzzy

What is the current behavior?

  on .terraform/modules/eks/terraform-aws-modules-terraform-aws-eks-6ea3582/local.tf line 6, in locals:
   6:   cluster_security_group_id = coalesce(
   7:
   8:
   9:
    |----------------
    | aws_security_group.cluster is empty tuple
    | var.cluster_security_group_id is ""

Call to function "coalesce" failed: no non-null, non-empty-string arguments.

If this is a bug, how to reproduce? Please include a code sample if relevant.
Run it on empty state with no infrastructure provisioned.
What's the expected behaviour?
Do nothing and exit without error.
Are you able to fix this problem and submit a PR? Link here if you have already.
Can do if nobody beats me to it :smile:
Environment details

Affected module version: v5.1.0
OS: Linux
Terraform version: 0.12.8

Any other relevant info
/

Closing as duplicate https://github.com/terraform-aws-modules/terraform-aws-eks/issues/402 and resolved by https://github.com/terraform-aws-modules/terraform-aws-eks/pull/459
Feel free to reopen the issue if the provided PR doesn't resolve your problem.


[FEATURE_REQUEST] Auto-Fill credentials when opening a Dashy item

Is your feature request related to a problem? If so, please describe.
Because a lot of my items are self-hosted, the programs/services use the same IP address, just differentiated by the port when accessing them via web browser. Because of this, credentials can not be stored on the browser as browsers do not differentiate between services through the ports (ex: <IP_ADDRESS>:1001 looks the same as <IP_ADDRESS>:5005 to the browser, and thus autofills the same credentials to both).
Describe the solution you'd like
Adding an option to pre-fill credentials on the Dashy side so that they are filled when opening the item would be useful. Having credentials filled per profile would be invaluable to security as well, but not needed.
Priority
Medium (Would be very useful)
Is this something you would be keen to implement
None

I cannot speak for Lissy but I guess this is way out of scope and not at all what Dashy is all about. Dashy is a Dashboard not a password manager.
Why don't you consider using a reverse proxy? Then you could have like dashy.myserver, nas.myserver, webserver.myserver, whatever you like and there are plenty of guides on youtube.

Yeah, this wouldn't be physically possible to do from Dashy.
It'd need to be done through a browser extension - you're better of submitting a FR/PR to your password manager, adding the ability to differentiate between different hosts.
Or easier/better still as @CrazyWolf13 mentioned, setup a reverse proxy or use your DNS to configure custom local domains. That way your password manager would be able to auto-fill correctly.


Add unit tests that exercise issue #234

This PR adds unit tests that exercise issue #234.  They're (deliberately) minimal, since license comparison is expensive and already a sizable portion of the runtime of the unit tests, but they at least show what's going on with that issue as of v1.11 of the library.
Note: the tests are failing because this PR only adds (previously missing) unit tests to demonstrate issue #234, but not logic to fix it.

Thanks @pmonks - creating unit tests will help in diagnosing/fixing the issues.
It may be a while before I can look at these issues - I'm prioritizing getting the SPDX 3.0 supported.

@goneall yep absolutely no problem - I know just how busy you've been with the 3.0 release and updating the library to match!

@goneall I believe this PR can be closed without merging, as the changes here were merged as part of PR #249.

Thanks @pmonks - closing the issue


Using Analytics under macOS

Describe your environment

Xcode version: 9.4.1 (9F2000)
Firebase SDK version: 5.6.0
Firebase Component: Core
Component version: 5.1.1

Describe the problem
I'm trying to use Firebase Analytics in a macOS Swift project.
Installed FirebaseCore via Cocoa Pods, but there are missing symbols, in particular :

Analytics
and all the parameter names constants, for example AnalyticsParameterItemID.

It seems I'm missing FirebaseAnalytics where those are declared and defined.
Steps to reproduce:

install FirebaseCore via Cocoa Pods
import FirebaseCore
try compile some code : Analytics.logEvent(event, parameters: ...)

The compiler complains about undefined identifiers : Use of unresolved identifier 'Analytics'
Relevant Code:
Any code that tries to access Analytics or any parameter names constants.
Maybe I'm missing something, but anyone knows how to solve this ?

I was able to fix this issue by adding this pod:
pod 'Firebase/Analytics', '5.6.0'

@dylanmaryk : I already tried installing Analytics along via Cocoa Pods, but it complains it does not support macOS :

[!] The platform of the target MyApp (macOS 10.11) is not compatible with Firebase/Analytics (5.6.0), which does not support osx.

Ah yes sorry, I was thinking about iOS...

See the macOS documentation. FirebaseAnalytics is not currently supported for macOS.

Same here. It would be great to have this, along with FirebasePerformance added to the mix. FirebasePerformance is perhaps one of the things that's exciting me about the move from Fabric to Firebase.

Same here!!

+1. FirebaseAnalytics support for macOS would be fantastic!

+1 to this!

+1 to this!

+1 to this!

Any updates about FirebaseAnalytics support for macOS?

+1 please! It’s important now that we have to migrate away from Fabric.

We are investigating. Tracked internally at b/138987204

hate to say 'me too!', but 'me too!

Just received an email from Google announcing a deadline where Answers and Fabric will stop working by March 2020.
We heavily rely on Answers (i.e. now Google Analytics) for our macOS apps and since there's no macOS support yet, I don't see how "Migrate your apps to Firebase now" makes any sense. This deadline should either be increased to allow a port of Firebase Analytics over to macOS or Google needs to prioritize bringing it over to macOS as soon as possible.
Please don't break our workflows. We loved Fabric, there's nothing wrong with it. Don't take it away from us.

+1 to Firebase Analytics support on macOS.

+1

+1

+1

+1

I wrote to Google Developer Support (for Fabric) after receiving a “Fabric is shutting down” email yesterday and they’ve clearly said there are no plans of bringing analytics to macOS in the foreseeable future. It’s officially not going to happen and repeatedly asking for it and waiting for it won’t help. I’ve recently discovered AppCenter by Microsoft. I suggest you guys move away too.

any plans?

+1

+1

@paulb777 this thread should be a reply to your questions asked here

+1

Please upvote the issue instead of leaving +1 comments which send a notification to everyone subscribed. Thanks!

Any news on this one? Is this the blocking issue for macOS community-support för Analytics?

@appfrilans This is the right issue to track. Note that Catalyst support for Analytics is a higher priority for us and will likely happen before traditional macOS support.

@appfrilans This is the right issue to track. Note that Catalyst support for Analytics is a higher priority for us and will likely happen before traditional macOS support.

@paulb777 Do you have a date?

@emreoktem Sorry, by policy, we don't provide dates for future features.

@appfrilans This is the right issue to track. Note that Catalyst support for Analytics is a higher priority for us and will likely happen before traditional macOS support.

Oh, I was actually wondering about Catalyst support. Which issue is the right one to follow then?

@appfrilans See #4563 to track FirebaseAnalytics and Catalyst

Fabric shuts down in 8 days.
It's pretty shoddy to not have a MacOS solution to replace Answers yet.

Guys, I would really appreciate your advice here.
We are porting our iOS app for the first time to Macos. Google Firebase analytics was incredibly helpful on iOS to observe users' behaviour and of course to see the crashes. Now with Fabric gone, what are our options for Macos?
Is there a free Analytics/crash reporting for Macos you could recommend? Ideally, one that supports both iOS and Macos. It makes no sense for us to split the analytics between iOS and Macos. We need to understand the entire market for both users in one dashboard.
Many Thanks,

@houmie Firebase crash report support both iOS and Macos,but analytics doesn't support it

@houmie Microsoft App Center supports free Analytics, Crashes and Distribution for iOS, macOS and Android. I've moved due to lack of property macOS support in Firebase (not just missing analytics - even the Firebase console shows these apps as "iOS" and you can no longer see version adoption as you could with Fabric, so you have absolutely no idea how many people are actively using the build vs. those crashing). Firebase has had more than two years to support macOS but for some very odd reason, they chose not to. I still don't understand why when everything worked just fine in Fabric. Any way, MS App Center is worth a try. It's even must faster to load and a lot less bloated than Firebase Console.

@guidedways I switch to Microsoft App Center about a year ago and I agree with your comments. I  was never able to make fabric/firebase events work on macOS but it works very nicely on MS App Center. I created events to track which macOS versions the existing and new users use which is super useful.

@guidedways Thank you for the detailed reply.  I have tried Microsoft App Center and it looks good.
Is there a way to obtain the country of the Mac Users? I looked at the FAQ and it seems that is not possible. A shame that a simple IP check could have just done that. Thanks

+1

+1

Is Firebase/Analytics now works on macOS ?

Anything new about Firebase Analytics for macOS?

Looks like its an gone case….they are at all not i guess looking into it.

On 07-Aug-2021, at 3:07 PM, Nathan Fallet @.***> wrote:

Anything new about Firebase Analytics for macOS?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Sorry for the delays on this one. Work is in progress and we hope to share more details soon.

@paulb777 Any news on Firebase Analytics Mac Catalyst support?

@sstennis1 Active progress and hope to share an update in the near future.


prepare ia_tbl etc. in unified routines

I prepare ia_tbl etc. in unified routines.

Thank you for merging, @syamada0 san.


Does Gatsby .cache work ?

** Please describe which feature you have a question about? **
Trying to cache gatsby .cache directory but does not seems like it working.
I have following Build settings
version: 1.0
frontend:
  phases:
    preBuild:
      commands:
        - cd gatsby
        - nvm use $VERSION_NODE_12
        - yarn install

    build:
      commands:
        - nvm use $VERSION_NODE_12
        - yarn build

  artifacts:
    baseDirectory: gatsby/public
    files:
      - '**/*'
  cache:
    paths:
      - gatsby/.cache/**/*
      - gatsby/node_modules/**/*

** Provide additional details**

its working but to reduce build time need to cache
- gatsby/public/static/**/*

I can confirm that this setting doesn't work, failing with:
2021-01-04T02:08:24.521Z [WARNING]: !TypeError: Cannot read property 'cache' of undefined

in the building phase

I can confirm that this setting doesn't work, failing with:
2021-01-04T02:08:24.521Z [WARNING]: !TypeError: Cannot read property 'cache' of undefined

in the building phase


FR: Choose between checkboxes and select field

I'd like the ability to choose in the field settings whether this field displays the sections as a list of checkboxes, as it does now, or a select field type (dropdown) to select just one option. This would be a nice addition if you find time.

Thank you, I appreciate that info. In my case, this is less about being able to select one, or more than one option, and more about the field layout for content authors. Here's my use case.

OK, I was under the impression you had over 50+ sections. :)
It might be worth considering that sometimes visual distinction is more beneficial than trying to compress everything into a minimal space. This approach can help users more quickly understand and navigate the layout.

I appreciate you assuming what is best for my client. I'll take that as a no.


Java Native Interface

Proposed title of article
Java Native Interface
Introduction paragraph (2-3 paragraphs):
As we know, one of Java's key advantages is its portability, which means that when we create and compile code, the outcome is platform-independent bytecode. Simply put, this can run on any machine or device that can run a Java Virtual Machine, and it will run as smoothly as possible.
However, there are situations when we need to use natively compiled code for a specific architecture. In this article the reader will be able to understand  Java Native Interface and how to implement it.
Key takeaways:
The reader will understand the following;

Native Methods.
JNI Elements in Code (Java).
Creating the Java Class.
Compiling and Linking.
Adding Parameters To Our Native Methods.
Using Objects and Calling Java Methods From Native Code.
Advantages and disadvantages of Using JNI.

References:
N/A

Okay. Topic approved. Let's ensure that it adds value to the community. Thanks


Fix frame count calculation on journal recovery

This PR fixes a bug where journals are not recovered correctly if the journal header has not been sync'd and the frame count field is zero.

@kentcdodds This PR should fix the issue you were hitting in production yesterday. I did a lot of local manual testing and I'll backfill some test coverage on Friday. I went ahead and cut a v0.3.0-beta5 if you want to use it.
I still have more a lot more testing to do overall before cutting the final v0.3.0 so insert standard beta software disclaimer here. :)


Question about the concat_all_gather function

Hi! In your code the function "concat_all_gather" is only used in enqueue_and_dequeue function, and the function is processed on CPU. I have 3 questions: 1) why are the prototypes not gathered by using this function? I think the prototype should remain same cross GPUs; 2) when do we need to use this function? 3) why is the keys detached and transfered to CPU before gathering (code link is here ? Thanks!

Taking the vanilla cross-entropy loss as an example. When we are computing the objective across GPUs, it is not necessarily to ensure that the segmentation logits remain the same. For the same reason, prototypes are not necessarily the same across GPUs, as are queries and their negative keys.
The queue that stores all negative keys should be the same across GPUs, and thus this function is needed.
All negative keys are expected to be detached from the gradient.


FIX Parse shortcodes in template helpers

Fixes #5661
Fixes #1381
See #5653
Basically back-porting the approach from the above PR.
Use RAW to parse the shortcodes and rely on RAW for all the other helper methods.

As for https://github.com/silverstripe/silverstripe-framework/pull/5761#issuecomment-230085407, please add some unit tests.

closed because we merged into 3.2


Add popup customization

This PR adds width and height popup customization, requested in https://github.com/nicolasbeauvais/vue-social-sharing/issues/240

Hello @CarinaChenot,
Thank you for your contribution, your changes have been tagged in the latest release.


CycleFinderTask and TranslateTask Unit Tests

Testing Improvements:

Dependency injection for Project.exec {…} method
MockProjectExec testing utility class
First use of the Groovy MockFor system
CycleFinderTaskTest good coverage
TranslateTaskTest only basic coverage

Other Changes:

exec args to have proper iteration for adding as args
J2objcHome() standardize capitalization

@advayDev1, @confile - this is something that I've long been wanting to to... adding solid testing around the calls to the J2ObjC command line tools, e.g. cycle_finder and j2objc.
The test are somewhat between a unit test and integration test. Certainly the TranslateTask could do with better unit testing and having the code broken up further in to more testable components.

Closing to merge in to branch instead


bump version to 0.0.28

Codecov Report

:exclamation: No coverage uploaded for pull request head (bump@9a00a86).

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 b81b914...9a00a86. Read the comment docs.


Update ktlint version

Signed-off-by: Rupal Mahajan<EMAIL_ADDRESS>Description
Updating ktlint version from 0.33 to 0.45.1
Issues Resolved
CVE-2020-8908<EMAIL_ADDRESS>><EMAIL_ADDRESS>CVE-2018-10237<EMAIL_ADDRESS>><EMAIL_ADDRESS>CVE-2015-5262<EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>CVE-2020-13956<EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #521 (ff18963) into main (139e6e5) will increase coverage by 0.19%.
The diff coverage is 91.26%.

@@             Coverage Diff              @@
##               main     #521      +/-   ##
============================================
+ Coverage     74.27%   74.46%   +0.19%
  Complexity      383      383
============================================
  Files            94       94
  Lines          3984     4022      +38
  Branches        641      641
============================================
+ Hits           2959     2995      +36
- Misses          891      893       +2
  Partials        134      134

Flag
Coverage Δ

dashboards-reports
76.85% <ø> (ø)

reports-scheduler
72.27% <91.26%> (+0.41%)
:arrow_up:

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

Impacted Files
Coverage Δ

...nsearch/reportsscheduler/ReportsSchedulerPlugin.kt
100.00% <ø> (ø)

...search/reportsscheduler/action/PluginBaseAction.kt
31.34% <ø> (ø)

...tsscheduler/model/CreateReportDefinitionRequest.kt
73.07% <ø> (ø)

...sscheduler/model/CreateReportDefinitionResponse.kt
100.00% <ø> (ø)

...tsscheduler/model/DeleteReportDefinitionRequest.kt
67.85% <ø> (ø)

...sscheduler/model/DeleteReportDefinitionResponse.kt
87.50% <ø> (ø)

...sscheduler/model/GetAllReportDefinitionsRequest.kt
62.85% <ø> (ø)

...scheduler/model/GetAllReportDefinitionsResponse.kt
84.61% <ø> (ø)

...rtsscheduler/model/GetAllReportInstancesRequest.kt
62.85% <ø> (ø)

...tsscheduler/model/GetAllReportInstancesResponse.kt
84.61% <ø> (ø)

... and 43 more

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


[WebPubSub] Resolve dependency and GA API review comments.

All SDK Contribution checklist:
This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] Please open PR in Draft mode if it is:

Work in progress or not intended to be merged.
Encountering multiple pipeline failures and working on fixes.

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code. (Track 2 only)
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK. Please double check nuget.org current release version.

Additional management plane SDK specific contribution checklist:
Note: Only applies to Microsoft.Azure.Management.[RP] or Azure.ResourceManager.[RP]

[ ] Include updated management metadata.
[ ] Update AzureRP.props to add/remove version info to maintain up to date API versions.

Management plane SDK Troubleshooting

If this is very first SDK for a services and you are adding new service folders directly under /SDK, please add new service label and/or contact assigned reviewer.

If the check fails at the Verify Code Generation step, please ensure:

Do not modify any code in generated folders.
Do not selectively include/remove generated files in the PR.
Do use generate.ps1/cmd to generate this PR instead of calling autorest directly.
Please pay attention to the @microsoft.csharp version output after running generate.ps1. If it is lower than current released version (2.3.82), please run it again as it should pull down the latest version.

Note: We have recently updated the PSH module called by generate.ps1 to emit additional data. This would help reduce/eliminate the Code Verification check error. Please run following command:
  `dotnet msbuild eng/mgmt.proj /t:Util /p:UtilityName=InstallPsModules`

Old outstanding PR cleanup
Please note:
If PRs (including draft) has been out for more than 60 days and there are no responses from our query or followups, they will be closed to maintain a concise list for our reviewers.

API changes have been detected in Microsoft.Azure.WebJobs.Extensions.WebPubSub. You can review API changes here
API changes
-         public string Format();
-     public enum WebPubSubPermission {
-         SendToGroup = 1,
-         JoinLeaveGroup = 2,
-     }
-         public string OperationKind { get; }

API changes have been detected in Microsoft.Azure.WebJobs.Extensions.WebPubSub. You can review API changes here


Added RawOutputDataConfig in ExecutionSpec

Signed-off-by: Kevin Su<EMAIL_ADDRESS>TL;DR
Added RawOutputDataConfig in ExecutionSpec
https://github.com/flyteorg/flyte/issues/2230
Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
fixes https://github.com/flyteorg/flyte/issues/
Follow-up issue
NA

RST files restored.


[NSE-297] Disable incremental compiler in GHA CI

Guys. If any build issues faced after merging this please let me know. So far I have tested it locally.


Fumedev Implemented Copy Functionality and Improved User Feedback with Toast Notifications

Update Chat.tsx to Include Copy Functionality
frontend/src/components/chat/Chat.tsx
Import React Icon for Copy Button
import { MdContentCopy } from 'react-icons/md';

Add Utility Function for Copy-to-Clipboard Action
const copyToClipboard = async (text: string) => {
  try {
    await navigator.clipboard.writeText(text);
    console.log('Message copied to clipboard');
  } catch (err) {
    console.error('Failed to copy text: ', err);
  }
};

Modify Message Rendering to Include Copy Button
return (

    {messages.map((message, index) => (

        <ChatMessage message={message} />
        <button
          onClick={() => copyToClipboard(message.text)}
          className="ml-2 p-1 rounded hover:bg-gray-200 focus:outline-none"
          aria-label="Copy message"
        >
          <MdContentCopy size={24} />
        </button>

    ))}

);

Ensure Accessibility with ARIA Attributes
Ensure the button is accessible by adding appropriate ARIA attributes. The aria-label attribute has been added to the button for accessibility purposes.
Modify ChatMessage Component for Better Styling
frontend/src/components/chat/ChatMessage.tsx
Adjust Styling to Accommodate Copy Button
function ChatMessage({ message }: { message: Message }) {
  return (

{message.text}

  );
}

Ensure Proper Use of Space and Padding
Ensure proper use of space and padding to ensure that message text within each ChatMessage component does not overflow or clip due to the addition of a copy button.
Enhance User Feedback on Copy Action
frontend/src/components/Toast/ToastContainer.tsx
Implement Toast Component for Notifications
import React from 'react';
import { createPortal } from 'react-dom';

interface ToastProps {
  message: string;
  onClose: () => void;
}

const Toast: React.FC<ToastProps> = ({ message, onClose }) => {
  React.useEffect(() => {
    const timer = setTimeout(onClose, 3000);
    return () => clearTimeout(timer);
  }, [onClose]);

  return createPortal(

      {message}

,
    document.body
  );
};

This implementation enhances the user experience by providing a visual feedback when a message is copied to the clipboard.

Hey there. I tried this issue on FumeDev. Pls give me a feedback if this is correct or not.

I don't think Fume is quite ready to open a PR without some human intervention, so unless you want to work through the lint issues, I'm going to close this one

@PierrunoYT unfortunately there are a bunch of lint problems here.
I don't think Fume is quite ready to open a PR without some human intervention, so unless you want to work through the lint issues, I'm going to close this one

Okay sure you can close it.


Only call focus() if we had it before [issue 495]

To address this issue.

Hi @alexkrolick , I'm not sure if you're the right person to ping for this but if you could I'd appreciate.

👀

waiting for this to be merged

Also waiting. Additionally the Travis-Check was successfull, but github doesnt see that

What's the best way for me to go about getting this merged?

Committed that change. Looking at it, I'm not sure why the original focus() call added with this commit was needed, as the setSelection() would focus or not depending if there was a cursor in the editor or not. Maybe 3 years ago quill setSelection() didn't behave the way it does now?
I'm hesitant to remove the focus() call completely. I'm not sure if there's a possibility for editor to have focus without having a cursor in it, if it can then it's better to keep the focus() call.

Any updates?

@alexkrolick What's the status on getting this merged and shipped?

Any update on getting this merged in and released?

This should be merged and released, it works well

I created a fix for this according to the latest master branch [v2] here: https://github.com/zenoamaro/react-quill/pull/594.

@rockwotj's change has been merged to master. Can you verify that it solves your issues? Cloning and npm link should do the trick.


Showing table metadata should sort metadata in alphabetical order

When displaying the table metadata, SQL Lab should automatically sort the metadata in alphabetical order:
Current state:

https://github.com/airbnb/caravel/pull/1464


Open Pilot now unuseable

Describe the bug
I own a 2018 Honda c-rv. I've used open pilot for 4 months now. Every two or three weeks, on starting the card, all of the vehicle warning messages are displayed in sequence on the screen, accompanied by a warning tone every 2 or 3 seconds. After driving for 15 mins. or so the problems disappear and everything works normally.
Unfortunately, since the most recent update, the problem with the warning etc are permanent, meaning the system is now unusual.
Has anyone else experienced this? Is there a fix or should I return the system to Comma.ai
Thanks for any suggestions.
Steve
Which car does this affect?
Honda C-RV 2018
Provide a route where the issue occurs
?
openpilot version
Latest version
Additional info
See above

We'll need your device's dongle ID to look into this.

Thanks for your swift reply. Where do I find this ID?Thanks, SteveSent from my iPadOn Apr 24, 2024, at 17:33, Adeeb Shihadeh @.***> wrote:
We'll need your device's dongle ID to look into this.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

It's in comma connect or the device settings.

Any updates @Deltic9009?


feat: bundle extension with webpack, add MathJax 3.x support

🚧 Currently work in progress. Issues to address:

[ ] Update MathJax to 3.x beta for webpack support. (Needs refactoring @Silverster98.)
[ ] Update Webpack build scripts.
[ ] Update CI and tests.
[ ] Check for dependency issues before publishing.

This will be published for version v0.1.0.

Currently extension will fail when invoked action Math to Image: ... (local). Webpack can't bundle MathJax libraries synchronously.

Useful links:
MathJax

MathJax 3.x source repo: https://github.com/mathjax/MathJax-src
MathJax Webpack demo: https://github.com/mathjax/MathJax-demos-web/blob/master/custom-build/webpack.config.js

Webpack

Webpack external dependencies documentation: https://webpack.js.org/configuration/externals/
VS Code demo - Add support for webpack PR: https://github.com/microsoft/vscode-references-view/pull/50/files
Official VS Code docs: https://code.visualstudio.com/api/working-with-extensions/bundling-extension

Closing in favor of recent updates.


Widgets with defined width

As is, widgets take the full width of the column they've been added to.
Does anyone know if it's possible to have widgets define their own width? So that layouts like below are possible:

Looking at the widget template I can't see a semantic way to achieve this, but if anyone's faced this or has suggestions your input would be greatly appreciated! I must confess, I'm new to bootstrap, but I'm coming from foundation so I understand the basics, so I may be missing an easy win.

No, widgets can not define their own width. But your picture looks like the following structure:
  dashboardProvider
    .structure('6-6 12 4-8', {
      rows: [{
        columns: [{
          styleClass: "col-md-6"
        }, {
          styleClass: "col-md-6"
        }]
      },{
        columns: [{
          styleClass: "col-md-12"
        }]
      },{
        columns: [{
          styleClass: "col-md-4"
        }, {
          styleClass: "col-md-8"
        }]
      }]
    });

The dashboard (enabled editMode), will look like the following:

Ok thanks. The sample layout unfortunately doesn't suit what I need, as it simple fixes the columns/rows and scales the widgets in them. I need the widgets to be fixed width and for them to wrap onto new rows where appropriate.
I will see if I can tweak this use case into existence. Thanks all the same though.


Sample App Update

Remove mention of model-driven-app Samples within the documentation.

Per a requirement, based on issues with Signed dll files and outdated Sample apps, these connections and the corresponding documentations need to be removed.
There is a sibling PR out within PPUX to feature flag and disable the display of these Sample Apps until such time they can be created in a manner that is beneficial to our user base.

@DigitalKrony - Thank you for your contribution. The article has already been removed via another pull request.
Closing without merging.


simple executeSingleCommand need 6s

#6s
PowerShellResponse powerShellResponse = PowerShell
.executeSingleCommand("Get-CimInstance -ClassName Win32_Printer -Property Name|Select-Object -Property Name;");
#1s
Process exec = Runtime.getRuntime()
.exec("powershell.exe Get-CimInstance -ClassName Win32_Printer -Property Name|Select-Object -Property Name;");
#1s
powershell> ....
i try some commands,all need 6s. There seems to be a configuration error it spend 5s. i dont know what happen

3.1.1

Trace it down in jPowerShell - I could imagine that you've got a problem here, there you see the extra +5 seconds.
In Java 9+, one can get the process PID in a better way.


Run isort in all files.

Summary
This is an extended PR of #169, I decided to create a separated PR for this.

Checklist

[ ] If code changes were made then they have been tested

[ ] I have updated the documentation to reflect the changes
[ ] I have formatted the code properly by running black .

[ ] This PR fixes an issue
[ ] This PR adds something new (e.g. new method or parameters)
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

BTW, might be worth adding config for isort.
Most importantly: https://pycqa.github.io/isort/docs/configuration/profiles.html#black


Redis should be listed as a prerequisite

Mysql is listed as a prerequisite but redis is not which seems inconsistent

It's actually listed under "Dependencies":

Sidekiq is used to run jobs in the background, and in turn depends on Redis to queue data.


[RTL] Add MultiplierImplementation parameter in top level

I thought that it would be nice to have this switch in top level, so that everything can be configured from here. Slow multiplication is already described in documentation, but there is no information how to enable it.

Also I think that there should be added test case for it for CI. I've seen some linter errors when choosing slow multiplier when run with my Docker image.
Maybe I should open separate issue about it, with all the details?

Also I think that there should be added test case for it for CI.

Yes, that would be great. Should be easy enough by just adding a second call to fusesoc to the azure-pipelines.yml file which sets the --MultiplierImplementation=slow. Please feel free to do so in a PR or file a new issue for it.

Also I think that there should be added test case for it for CI.

Yes, that would be great. Should be easy enough by just adding a second call to fusesoc to the azure-pipelines.yml file which sets the --MultiplierImplementation=slow. Please feel free to do so in a PR or file a new issue for it.

I think it would be better to do it in separate PR as there are some linter errors present, which I didn't look into too much and I didn't look into the architecture of slow multiplier yet..

IMHO it looks better now :stuck_out_tongue_winking_eye:


Fixing handling of null payloads

Client could cause a runtime error by sending a message with null payload:

running example from readme modified to connect with unix socket

$ csi example.scm
CHICKEN
(c) 2008-2021, The CHICKEN Team
(c) 2000-2007, Felix L. Winkelmann
Version 5.3.0 (rev e31bbee5)
linux-unix-gnu-x86-64 [ 64bit dload ptables ]

Type ,? for help.
; loading example.scm ...
; loading /nix/store/pk3xa2milqiv3fp5v22srfbx8hqidwb5-chicken-5.3.0/lib/chicken/11/chicken.blob.import.so ...
; loading /nix/store/pk3xa2milqiv3fp5v22srfbx8hqidwb5-chicken-5.3.0/lib/chicken/11/chicken.string.import.so ...
; loading /nix/store/i0m9f0ppgq62rxihvd7z6m6im17ybsbv-chicken-mosquitto-0.1.4/lib/chicken/11/mosquitto.import.so ...
; loading /nix/store/i0m9f0ppgq62rxihvd7z6m6im17ybsbv-chicken-mosquitto-0.1.4/lib/chicken/11/mosquitto.so ...
; loading /nix/store/bxxf61yi5nnj4qsvvfb5qkkjvzs44xng-chicken-srfi-1-0.5.1/lib/chicken/11/srfi-1.so ...
Yay, we are connected!

send message with null payload:
$ mosquitto_pub --unix ./test/mosquitto.sock -t topic1 -m ''

example crashed:
Error: (move-memory!) bad argument type - not a block object: #f

	Call history:

	<syntax>	 (message (string-append "Topic: " (mqtt-message-topic msg) " " "Payload:" (blob->string (mqtt-messag...
	<syntax>	 (string-append "Topic: " (mqtt-message-topic msg) " " "Payload:" (blob->string (mqtt-message-payload...
	<syntax>	 (mqtt-message-topic msg)
	<syntax>	 (blob->string (mqtt-message-payload msg))
	<syntax>	 (mqtt-message-payload msg)
	<syntax>	 (mqtt-publish client "topic2" "message received, thanks!")
	<eval>	 (set-mqtt-client-message-callback! client (lambda (cl msg) (message (string-append "Topic: " (mqtt-m...
	<syntax>	 (mqtt-connect client "./test/mosquitto.sock" #:port 0)
	<eval>	 (mqtt-connect client "./test/mosquitto.sock" #:port 0)
	<syntax>	 (mqtt-loop-forever client)
	<eval>	 (mqtt-loop-forever client)
	<eval>	 (message "Yay, we are connected!")
	<eval>	 [message] (display message)
	<eval>	 [message] (newline)
	<eval>	 [message] (flush-output)
	<eval>	 (mqtt-subscribe client "topic1")	<--

Also, I have added some duplicate exports during #1, so fixing this too, sorry :)

Commit ea38158 "reformat sources" has indentation issue - spaces have been replaced with tabs. There should only be spaces

Thanks!


Allow for graceful Agent stopping

This changes the behaviour of the existing Agents page, which treats the "Stop" button as an immediate, ungraceful, force stop. This means we don't have any user-facing way to gracefully stop an agent!
Following this change, the "Stop Agent" button is a graceful stop. We'll ask the agent politely to stop.
If the Agent takes more than ten (±3) seconds to stop, we then show a "Force Stop Agent" button in its place. This warns that it'll cancel the running job (as the current warning does) and will function the same as our current behaviour.

There's a few improvements I can spot, that'd be great to get in here:

Maybe keep the "Stopping…" disabled button and section visible until the agent actually stops. So you don't have to scroll up, you can just stay there waiting for 3 sections until a new "Force Stop Agent" section appears below it?
We can probably move the "If the agent doesn't respond..." words down to the bottom w/ the "Stopping…" button.
In table of attributes, "Stopped" is shown in the "stopping" state… which looks inconsistent/lies because the agent hasn't stopped. Maybe we rename to "Stopped By" and rejig the sentence to "Jessica Stokes, today at 6:03 PM"

Looks like this change doesn’t work after all due to backend limitations. Closing for now.


Use native Jest matchers

Jest supports asynchronous matchers and asymmetric matchers, two required things to port expect-puppeteer matchers into Jest matchers. Some problems still persist:
Some matchers are returning an element like toMatchElement()
A matcher should not return anything, yes. But it is very convenient to assert that something is in the page and get the element matched in the page. I think we probably need to rethink this part. I am open to propositions.
Some matchers are already used by Jest like toMatch
I think it is not a big problem, we can change the name. Also open to propositions.

Thanks for your suggestion but I didn't get it, can you show me an example of code?

👍 on the idea of allowing matchers to receive element refs instead of selectors. It would be nice to be able to do:
await expect(page).toClick($btn)
Where $btn is already an element handle.

@gregberge so is this essentially a wontfix?

Sorry it was a mistake.


minor wordsmith of README

a trivial edit to verify that we can merge a valid commit that passes travis and has 1 approving review

Required review configuration working as expected:

Required review configuration working as expected:


Update dllhost_with_no_command_line_arguments_with_network.yml

Removal of PID collision false positives using host in join. Added other improvements for context.
Details
What does this PR have in it? Screenshots are worth 1000 words 😄
Checklist

[ ] Validate name matches <platform>_<mitre att&ck technique>_<short description> nomenclature
[ ] CI/CD jobs passed ✔
[ ] Validated SPL logic.
[ ] Validated tags, description, and how to implement.
[ ] Verified references match analytic.

thank you for the PR. added your changes here for testing : https://github.com/splunk/security_content/pull/2528


MSI B450 Tomahawk - Colors do not match

I have an MSI B450 Tomahawk and trying to figure out how to work with this tool as it behaves differently from the instructions in README.
For me the output of msi-rgb FFFFFFFF 00000000 00000000 alternates between blue and white, the output of msi-rgb 00000000 FFFFFFFF 00000000 alternates between purple and white, and the output of the msi-rgb 00000000 00000000 FFFFFFFF alternates between yellow and white.
Other than the fact that I expect different colors to be displayed, I expect the color to be fixed and not changing between different ones.
Does anyone have any suggestions on how to configure the program to show the correct colors and to stop it from changing it?

Try initializing the B450 with the MSI Gaming App (turn LEDs on).
You will have to boot WIN to do this.

Thanks for the response. But isn't not having/wanting to install Windows the reason for using this code. Is it possible to use a virtual environment, wine, or playonlinux for to do the initial setup?

But isn't not having/wanting to install Windows the reason for using this code.

This is quite spot-on, but it is also the case that the only really supported board is my own, and the rest of the boards were only reported as working by other people. I’m unable to provide support nor fix the code for other boards and what @dcerisano suggests is a troubleshooting suggestion, I guess.

Is it possible to use a virtual environment, wine, or playonlinux to do the initial setup?

No.

@nagisa Thanks for your timely answer. It makes perfect sense. Just as a suggestion, if you have any instructions or useful resources on how one can develop such code with some slight details, I am sure there are many people, including myself, that are willing to spend time to extend this library so that it works properly on a wide range of motherboards.

There is something you can use as a starting point described in my blog post linked from the README. I know that https://github.com/ixjf has quite successfully decompiled the MSI code and using msi-rgb as an inspiration made a C# application as well.
In general helping out involves running windows on bare-metal and then looking at how the MSI’s official app behaves with some particular hardware, but perhaps checking whether alternative apps work and cross-referencing code paths would also help.

As @nagisa stated, msi-rgb is only guaranteed to run on one mobo (MSI Mortar Arctic).
For many others, the WIN based MSI app must be run once per power cycle to init the RGB header.
Also, running the MSI app via WINE is a no go.


Fix directory pull image name for OCI images

This is a breaking change and modifies the resulting image name when
pulling from an directory via oci:....
Without this patch, the image names pulled via a local directory got
processed incorrectly, like this:
> podman pull oci:alpine
> podman images
REPOSITORY      TAG      IMAGE ID       CREATED       SIZE
localhost/oci   alpine   4fa153a82426   5 weeks ago   5.85 MB

We now use the same approach as in the corresponding buildah fix to
adapt the behavior for correct localhost/ prefixing.
After applying the patch the same OCI image pull looks like this:
> ./bin/podman pull oci:alpine
> podman images
REPOSITORY         TAG      IMAGE ID       CREATED       SIZE
localhost/alpine   latest   4fa153a82426   5 weeks ago   5.85 MB

End-to-end tests have been adapted as well to cover the added scenario.
Relates to: https://github.com/containers/buildah/issues/1797

LGTM
@mheon @giuseppe @baude @TomSweeneyRedHat @vrothberg @QiWang19 @ashley-cui PTAL

Changes LGTM
On Mon, Aug 19, 2019, 07:19 Daniel J Walsh<EMAIL_ADDRESS>wrote:

LGTM
@mheon https://github.com/mheon @giuseppe https://github.com/giuseppe
@baude https://github.com/baude @TomSweeneyRedHat
https://github.com/TomSweeneyRedHat @vrothberg
https://github.com/vrothberg @QiWang19 https://github.com/QiWang19
@ashley-cui https://github.com/ashley-cui PTAL
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/containers/libpod/pull/3849?email_source=notifications&email_token=AB3AOCAA5SF6HINKBUCV763QFJ6VLA5CNFSM4IM5E4Z2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4SR5ZQ#issuecomment-522526438,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB3AOCH6SDQMIL6H4WKUAVDQFJ6VLANCNFSM4IM5E4ZQ
.

/lgtm
/hold
Cancel hold when tests pass.

/approve

@saschagrunert Is this broken for the dir: transport as well?

@saschagrunert Is this broken for the dir: transport as well?

No, this works and worked as before (images are prefixed with localhost)

I would expect docker.io/library/alpine:latest to be listed and not localhost/alpine.
I'd prefer setting org.opencontainers.image.ref.name when using the oci transport as a destination and use it when using oci as a source.
update: I didn't check it but that might require touching containers/image.

This sounds like a new feature to me. I'm not sure if it's cool to modify the image manifest on push, but we could make it configurable. I guess it should definitely go into containers/image because it would work with all consumers of the API...

I would expect docker.io/library/alpine:latest to be listed and not localhost/alpine.
I'd prefer setting org.opencontainers.image.ref.name when using the oci transport as a destination and use it when using oci as a source.
update: I didn't check it but that might require touching containers/image.

This sounds like a new feature to me.

We're about to change behaviour in any case. I want to make sure we're not patching symptoms. The 'localhost/ prefix was usually used to avoid defaulting to a registry (traditionally docker.io) for unqualified names. In case of the dir:/ transport, we don't (maybe even can't) store the name but in case of oci: we can do that which is why I think we should tackle it in c/image rather than in libpod.
If others prefer the localhost/ prefix, I don't block the PR. But maybe we need to check other transports as well (oci-archive, docker-archive, docker-daemon, etc.).

I'm not sure if it's cool to modify the image manifest on push, but we could make it configurable.

That's fine. We're converting to OCI which supports embedding the reference in the manifest.

Well I am fine with this for now and then fixing it correctly in the long run.  Creating
localhost/oci is definitely a bug.

/hold cancel

Alright, then let's get this in :)


mem-ruby: Fix deadlock in GPU_VIPER when issuing atomic requests

When a compute unit issues several requests to the same line,
the requests wait in the L2 if it is a writeback cache. If the line is
invalid initially and the first request is atomic in nature, the L2
cache issues a request to main memory. On data return, the cache line
transitions to M but doesn't wake up the other requests, resulting in
a deadlock. This commit adds a wakeup call on data return for atomics
and fixes potential deadlocks.

@v-ramadas : it looks like you pulled a lot of commits other than your own.  You'll need to fix that before this can go in.

@v-ramadas : I think part of the problem is you tried to push to stable instead of develop

Finally, it looks like your commit message is missing a ChangeID?

Oops. I didn't realize that I was merging with stable. I changed it to develop now.

Thanks @v-ramadas !


Add Version to User agent Headers

Adding a version allows the devs to block older incompatible versions of the API instead of flatout blocking all requests.
Also removed the duplicate header kwargs, by adding it to default headers of the session.

Thanks for this improvement! I appreciate your efforts and merged that PR.


docs: make example easier to copy/paste

This might turn all the long classes to scroll bar mess.
Instead try Enhanced GitHub plugin. It adds copy paste button to all the code snippets.

my main idea was more related to: make code copy pastable in a single step.
as is - when integrating the snippet into a pre-existing rector.php - I need copy/paste 2 times. once for the use .. and once for the actual code lines.

I got that, but it would turn docs into mess of randoms classes that some are copy-pasteable and some now.
We use the PSR-12 standard here.
This should be handled on PHPStorm side, I know it worked before very well.


Understanding why a version of a package is missing

Hello,
I'm trying to understand why the package why<IP_ADDRESS> is not currently available for mingw. I'm assuming packages from opam-repository are automatically made available here by default, but I'm not sure.
I was using the old fdopen-based URL, so I changed to this ocaml-opam one, did opam update, but still nothing. I tried creating a new ocaml-variants.4.14.1+mingw64c switch, but it also didn't help.
I assumed the package must not be building under Cygwin, but if I clone it from git and compile it, it seems to work. It also doesn't have any windows-specific constraints.
So, in the end, I don't know why it is not available, and if there is something I could do to make it available. I need it as a dependency to another package, so manually compiling it is not very useful in the end.

What does opam remote -v give you ? do you have the default repo enabled ?

Could it be related to https://github.com/ocaml/opam-repository/issues/23954 ..

Probably! I commented there, in case the symbolic link information could be helpful.

The underlying issue was indeed the symlink problem (and thanks again for the diagnosis on that, @maroneze 🙂)

I'm assuming packages from opam-repository are automatically made available here by default, but I'm not sure.

That's not the case - apart from point releases of 4.14.x, the only changes in this repository are to existing package versions, mirroring changes in upstream ocaml/opam-repository. To get new releases of packages, you need to ocaml/opam-repository (see also this Discuss post).

Sorry for coming back to this question, but I just want to confirm that I understand the expected workflow.
So, I'm still trying to get why<IP_ADDRESS> installed in Cygwin. I added https://opam.ocaml.org as a remote, and now the package is available. I tried to install it and the ./configure script failed. While debugging it, I found the cause:
      OCAMLFINDLIB=$(ocamlfind printconf stdlib)
      if test "$OCAMLFINDLIB" != "$OCAMLLIB"; then
         found_ocamlfind=no
         reason_ocamlfind=" (incompatible with OCaml)"
         echo "but your ocamlfind is not compatible with your ocamlc:"
         echo "ocamlfind: $OCAMLFINDLIB, ocamlc: $OCAMLLIB"

In the first line, we assign the result of ocamlfind printconf stdlib to OCAMLFIND. And this *seems* to work, even if we print the variable afterwards. But this being ~~hell~~ Windows, *of course* it had to add a \rto the end, nearly invisibly, and making the test"$OCAMLFINDLIB" != "$OCAMLLIB"` fail miserably.
Ok, great, so the code is buggy on Windows. What should be done in this case?

Should there be some magical tool or script that, on Cygwin, automatically removes such \rs and makes things work as people expect?
Should I report to the why3 developers, so that they can fix it to work on Windows as well? This could entail lots of busywork for packages using such shell scripts, and it's hard to debug without having Cygwin yourself; this is like a tricky interview question, unless you are used to these things you will never think of them in advance;
Should I add a patches file with a windows-specific patch for such cases, and submit it to opam-repository?

I believe solution 2 is the expected one, but before reporting bugs upstream, I'd like to confirm that this is the case. I had the impression that, in the past, Cygwin tools automatically did such CR-stripping, but maybe I misremember it.

Should there be some magical tool or script that, on Cygwin, automatically removes such \rs and makes things work as people expect? And that I somehow skipped or disabled, and so I should do it otherwise to get it working?

Indeed - the magical tool here is either to avoid shell-scripting for configuration (e.g. using configurator, or even an OCaml script) or in this case simply adding OCAMLFINDLIB=$(ocamlfind printconf stdlib | tr -d '\r') which is portable.

Should I report to the why3 developers, so that they can fix it to work on Windows as well?

Yes, that's the fix that's necessary. If targeting Windows is a concern, the effort spent avoiding shell scripts usually pays off further down the line (this kind of thing doesn't happen anywhere near as often if you're processing the output using OCaml's string processing, for example).

It's probably the best choice, but this could entail lots of busywork for packages using such shell scripts

Yes, indeed, but...

and it's hard to debug without having Cygwin yourself;

... ocaml/setup-ocaml, GitHub Actions etc. can be used here.

this is like a tricky interview question, unless you are used to these things you will never think of them in advance.

That definitely sounds like the interview from hell 🙂

Still, if the idea is "test on Windows or add {os != "windows"} to available seems to be the "expected" one nowadays;

There's definitely no need to add os != "win32" just because it's not yet been tested on Windows (i.e. I think it's very bad to constrain away a working version of a package from Windows just because it hasn't been tested). opam-repository's fine to have availability and depends updated further down the line if the first person to try it on Windows finds it's broken.

Should I instead add an entry to the patches opam field, with a Windows-specific patch for such cases, and submit it directly to opam-repository ? This solution places less burden on the developers, but forces Cygwin users to find and fix issues by themselves. Still, it seems like a viable solution;

This is the change (which was already policy in opam-repository) - the patch must go upstream first. There might then be a decision to patch the existing package pending a new release, but it's very unusual for patches to be carried only in opam-repository. The normal thing here would be to patch why3 upstream, release a new version and update all the old versions in opam-repository to have os != "win32" (unless there's a reason to need to back-port the patch).

Should I just wait for opam 2.2 to be released, and such issues will be automatically handled by it? At least these \r ones, which seem often present?

opam 2.2 isn't magically going to fix this. This worked before because opam-repository-mingw carries a vast amount of non-upstreamed patches in packages which made changes like this. It's not sustainable, though - it means that why3 before didn't work on Windows, it worked specifically on "OCaml for Windows as packaged through opam-repository-mingw". That was fine while someone else was willing to maintain it, but it's a lot of moving parts.
As far as I can see, why3 has always worked in opam-repository-mingw ... that line in configure has been present since the oldest version of why3 in opam (11 years ago), so the rationale here is that it could have been fixed a long time ago, but a non-upstreamed workaround in another package, means that no one knew that it was broken.

I believe solution 2 is the expected one, but before reporting bugs upstream, I'd like to confirm that this is the case. I had the impression that, in the past, Cygwin tools automatically did such CR-stripping, but maybe I misremember it.

Cygwin did change some handling with CRs and became more pedantic (some time ago, now), but that wouldn't have affected this particular case. In particular, Cygwin's grep used to strip out CRs but they stopped on portability grounds.
On the "workarounds" (there are various internal tweaks like this in patches in opam-repository-mingw), the problem I have (and is why I'm certainly not willing to maintain them myself) is that they don't actually fix the underlying problems or, sometimes worse, they actually make diagnosing other bugs harder, as they add layers of "difference" that are easy to forget about (like when your packages are inexplicably selecting the wrong C compiler...).
The very real issue in the past (I mean in pre-opam days) is that getting patches for Windows upstreamed was very slow and often impossible. However, that's not where we are today, so while I realise there will be a fair old bit of pain dealing with these papercuts, I anticipate that it will be possible to get patches in upstream so that we don't have to carry all these downstream patches any longer.


Feature - Opening up ParameterEncoding

This PR changes the queryComponents and escape method ACLs from internal to public. This will make it easier for those implementing a .Custom parameter encoding. Right now if you need to use either function in a .Custom case, you have to implement both of these methods on your own.

cc @kylef and @kcharwood


Make storage efficient in terms of redundant information being saved on disk

@2020saurav can you please suggest a few sane techniques to go forward with this?

File size be made in multiples of disk page size.
Not storing redundant data such as AriaKey or AriaValue etc. But this will make it harder to parse the content. Presently, I am using read and show, which is cool.
Compression (must be reversible)

May not implement this


[Flaky Test] E2E Test Suite: [It] Shoot Tests Create, Hibernate, Wake up and Delete Shoot [Shoot]

How to categorize this issue?

/area testing
/kind flake
Which test(s)/suite(s) are flaking:
E2E Test Suite: [It] Shoot Tests Create, Hibernate, Wake up and Delete Shoot [Shoot]: https://github.com/gardener/gardener/blob/49427df9174f995daf8a782168b1ed84212f00f9/test/e2e/shoot/create_hibernate_wakeup_delete.go#L29
CI link:

https://prow.gardener.cloud/view/gs/gardener-prow/logs/ci-gardener-e2e-kind/1503527318235123712#1:build-log.txt%3A1656
https://prow.gardener.cloud/view/gs/gardener-prow/logs/ci-gardener-e2e-kind/1503587717034807296#1:build-log.txt%3A1725
https://prow.gardener.cloud/view/gs/gardener-prow/logs/ci-gardener-e2e-kind/1503587717034807296#1:build-log.txt%3A1725

Reason for failure:
Tests are failing in step Wake up Shoot with errors related to a broken worker health check, errors similar to:
    {"level":"info","msg":"Shoot wake-up-7cccz not yet reconciled successfully (condition type EveryNodeReady is not true yet, had message Worker extension (shoot--local--wake-up-7cccz/wake-up-7cccz) reports failing health check: failed to execute 1 health check: unable to check nodes. Failed to list shoot nodes: Get \"https://kube-apiserver.shoot--local--wake-up-7cccz.svc/api/v1/nodes\": dial tcp <IP_ADDRESS>:443: connect: connection refused. with reason WorkerUnhealthyReport)","ts":"2022-03-15T00:40:10.856Z"}

or
    {"level":"info","msg":"Shoot wake-up-7cccz not yet reconciled successfully (condition type ControlPlaneHealthy is not true yet, had message Worker extension (shoot--local--wake-up-7cccz/wake-up-7cccz) reports an outdated health status (last updated: 9m0s ago at 2022-03-15 00:37:00 +0000 UTC). with reason WorkerOutdatedHealthCheckReport)","ts":"2022-03-15T00:46:10.919Z"}

Anything else we need to know:

I reproduced this issue locally (first try) by hibernating and waking up a shoot.
The worker status was in the following state after the wakeup operation had been completed:
status:
  conditions:
  - lastTransitionTime: "2022-03-15T16:14:58Z"
    lastUpdateTime: "2022-03-15T16:22:04Z"
    message: Shoot is hibernated
    reason: HealthCheckSuccessful
    status: "True"
    type: SystemComponentsHealthy
  - lastTransitionTime: "2022-03-15T16:15:28Z"
    lastUpdateTime: "2022-03-15T16:22:04Z"
    message: Shoot is hibernated
    reason: HealthCheckSuccessful
    status: "True"
    type: ControlPlaneHealthy
  - lastTransitionTime: "2022-03-15T16:17:20Z"
    lastUpdateTime: "2022-03-15T16:22:04Z"
    message: Shoot is hibernated
    reason: HealthCheckSuccessful
    status: "True"
    type: EveryNodeReady
  - lastTransitionTime: "2022-03-15T16:17:08Z"
    lastUpdateTime: "2022-03-15T16:17:08Z"
    message: No rolling update in progress
    reason: NoRollingUpdate
    status: "False"
    type: RollingUpdate
  lastOperation:
    description: Successfully reconciled worker
    lastUpdateTime: "2022-03-15T16:25:31Z"
    progress: 100
    state: Succeeded
    type: Reconcile

So it seems like the health check controller doesn't pick up the checks again after waking up the shoot.
For the ControlPlane resource, this was not the case:

Interesting, thanks for checking @timebertt! Will you continue to look into this?

/assign


fix server validate

then it must be so?

Yes, thank you.
I was pretty tired when I did the changes, I will add some tests and will publish a new version tomorrow if everything works fine 👍

good rest!

Codecov Report

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

@@          Coverage Diff          @@
##           master   #516   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          11     11
  Lines         450    450
=====================================
  Hits          450    450

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 b051e74...6c0f138. Read the comment docs.


[efr32] configurable board

As discussed in issue #2324, the support for Silabs Thunderboard Sense 2 will be added through this pull request.
The user can now  provide the board, for example for WSTK:
make -f examples/Makefile-efr32 COMMISSIONER=1 BOARD=BRD4161A

or for Thunderboard Sense 2
make -f examples/Makefile-efr32 COMMISSIONER=1 BOARD=BRD4166A

I signed it!


Please provide the BiFPN model in EfficientDet to the neck file.

My question has been writed in the title.

Hi! We do have plans to work on that in the future. Meanwhile, your PR is very much welcomed!


Distribution tests - Fix cleanup

[x] All tests passed. If this feature is not already covered by the tests, I added new tests.
[x] This pull request is on the dev branch.
[x] I used gofmt for formatting the code before submitting the pull request.

Delete old release bundles before deleting old repositories. This fix handles the following error:
{
  "errors": [
    {
      "status": 403,
      "message": "Failed to delete repository 'cli-tests-dist1-1598249974' because it contains artifact/s related to release bundle"
    }
  ]
}

@yahavi, you'd probably want to merge this to the dev branch as well.

Thanks. Rebased to dev.


#56 - Remove magnet pattern in obj()

per #56 -- make obj() a non-magnetic impl. I bailed out trying to make arr() for now, but I do think it's possible but not elegant.

Looks good, thanks!


Run travis tests against 3 different ansible versions

run tests in parallel with 3 different ansible versions
use docker package instead of docker-py as it was renamed (http://docker-py.readthedocs.io/en/stable/)

@atosatto can we remove test-requirements.txt?

Why don't we use tox as done, e.g., in http://github.com/atosatto/ansible-caddy?

I don't have much experience with tox. @atosatto can you parallelize tox execution over multiple travis instances?

@atosatto tox looks so much better! I need to port it to @cloudalchemy :smile:

😄


Support for POST parameters

I cannot favorite something because the URL has to be as defined here https://developer.twitter.com/en/docs/tweets/post-and-engage/api-reference/post-favorites-create
POST https://api.twitter.com/1.1/favorites/create.json?id=243138128959913986
The URL is only constructed by the resource, and doesn't support params https://github.com/Preposterous/twitter-lite/blob/master/twitter.js#L81

Hey @gnomeontherun post parameters are supported in 0.4.0
Thanks so much for letting me know!

I've found it counter-intuitive to pass a null body, then my actual parameters to client.post:
await client.post("friendships/create", null, {
    screen_name: 'dandv'
  });

This also breaks drop-in compatibility with twit.
Is the body parameter reserved for certain things, like uploading media? If so, there should be a note in the README about this. In its README, twit doesn't seem to ever use any other parameters than parameters.
CC @tylerbuchea


Could you provide camera focal length?

Hello, could you provide camera focal length? This is very important to my project！
Thanks a lot!
Best wishes!

I want to know focal value, too.
Thanks!


Command PhaseScriptExecution failed with a nonzero exit code

Getting Command PhaseScriptExecution failed with a nonzero exit code when trying to build. What's wrong?

Needed to install SwiftLint


1.7.0: pytest fails in tests/test_pyproject_toml_fmt.py::test_help_invocation_as_script unit

I'm packaging your module as an rpm package so I'm using the typical PEP517 based build, install and test cycle used on building packages from non-root account.

python3 -sBm build -w --no-isolation
because I'm calling build with --no-isolation I'm using during all processes only locally installed modules
install .whl file in </install/prefix> using installer module
run pytest with $PYTHONPATH pointing to sitearch and sitelib inside </install/prefix>
build is performed in env which is cut off from access to the public network (pytest is executed with -m "not network")

Consequence of using above procedure is that pyproject-fmt command during the testing is present on $PATH and not in exact location (/usr/bin/pyproject-fmt).

        # Data format: "exception name:hex errno:description"
        # Pickle is not used; it is complex and involves memory allocation.
        errpipe_read, errpipe_write = os.pipe()
        # errpipe_write must not be in the standard io 0, 1, or 2 fd range.
        low_fds_to_close = []
        while errpipe_write < 3:
            low_fds_to_close.append(errpipe_write)
            errpipe_write = os.dup(errpipe_write)
        for low_fd in low_fds_to_close:
            os.close(low_fd)
        try:
            try:
                # We must avoid complex work that could involve
                # malloc or free in the child process to avoid
                # potential deadlocks, thus we do all this here.
                # and pass it to fork_exec()

                if env is not None:
                    env_list = []
                    for k, v in env.items():
                        k = os.fsencode(k)
                        if b'=' in k:
                            raise ValueError("illegal environment variable name")
                        env_list.append(k + b'=' + os.fsencode(v))
                else:
                    env_list = None  # Use execv instead of execve.
                executable = os.fsencode(executable)
                if os.path.dirname(executable):
                    executable_list = (executable,)
                else:
                    # This matches the behavior of os._execvpe().
                    executable_list = tuple(
                        os.path.join(os.fsencode(dir), executable)
                        for dir in os.get_exec_path(env))
                fds_to_keep = set(pass_fds)
                fds_to_keep.add(errpipe_write)
                self.pid = _posixsubprocess.fork_exec(
                        args, executable_list,
                        close_fds, tuple(sorted(map(int, fds_to_keep))),
                        cwd, env_list,
                        p2cread, p2cwrite, c2pread, c2pwrite,
                        errread, errwrite,
                        errpipe_read, errpipe_write,
                        restore_signals, start_new_session,
                        gid, gids, uid, umask,
                        preexec_fn)
                self._child_created = True
            finally:
                # be sure the FD is closed no matter what
                os.close(errpipe_write)

            self._close_pipe_fds(p2cread, p2cwrite,
                                 c2pread, c2pwrite,
                                 errread, errwrite)

            # Wait for exec to fail or succeed; possibly raising an
            # exception (limited in size)
            errpipe_data = bytearray()
            while True:
                part = os.read(errpipe_read, 50000)
                errpipe_data += part
                if not part or len(errpipe_data) > 50000:
                    break
        finally:
            # be sure the FD is closed no matter what
            os.close(errpipe_read)

        if errpipe_data:
            try:
                pid, sts = os.waitpid(self.pid, 0)
                if pid == self.pid:
                    self._handle_exitstatus(sts)
                else:
                    self.returncode = sys.maxsize
            except ChildProcessError:
                pass

            try:
                exception_name, hex_errno, err_msg = (
                        errpipe_data.split(b':', 2))
                # The encoding here should match the encoding
                # written in by the subprocess implementations
                # like _posixsubprocess
                err_msg = err_msg.decode()
            except ValueError:
                exception_name = b'SubprocessError'
                hex_errno = b'0'
                err_msg = 'Bad exception data from child: {!r}'.format(
                              bytes(errpipe_data))
            child_exception_type = getattr(
                    builtins, exception_name.decode('ascii'),
                    SubprocessError)
            if issubclass(child_exception_type, OSError) and hex_errno:
                errno_num = int(hex_errno, 16)
                child_exec_never_called = (err_msg == "noexec")
                if child_exec_never_called:
                    err_msg = ""
                    # The error must be from chdir(cwd).
                    err_filename = cwd
                else:
                    err_filename = orig_executable
                if errno_num != 0:
                    err_msg = os.strerror(errno_num)
>               raise child_exception_type(errno_num, err_msg, err_filename)
E               FileNotFoundError: [Errno 2] No such file or directory: '/usr/bin/pyproject-fmt'

/usr/lib64/python3.9/subprocess.py:1837: FileNotFoundError
================================================================================== short test summary info ==================================================================================
FAILED tests/test_pyproject_toml_fmt.py::test_help_invocation_as_script - FileNotFoundError: [Errno 2] No such file or directory: '/usr/bin/pyproject-fmt'
=============================================================================== 1 failed, 98 passed in 0.83s ================================================================================

We only support testing with tox.


Implement chatrooms

this will finish off #7 as well

ngl i completely forgor i made this issue lmfao


az ad app update command does not work for az cli version 2.0.49/2.0.50

az cli version 2.0.42 supports az ad app update command.. Post upgrade to any version of az cli does not support az ad app update.
$ az ad app update --id $appId--reply-urls https://$hostname/.auth/login/aad/callback
Update to existing credential with KeyId '$keyid' is not allowed.
az ad app update --id $appId --set groupMembershipClaims=All
Property 'groupMembershipClaims' not found on root. Send it as an additional property .
Update to existing credential with KeyId '$keyid' is not allowed.
az ad app update --id $appId --required-resource-accesses @manifest.json
Update to existing credential with KeyId '$keyid' is not allowed.
This scenario can be reproduced. It does not work on any az cli version after 2.0.42
To reproduce this scenario, create keys manually under Azure App Registration > settings > API > keys and then run the app update command as shown above.
This is a major bug to use az cli for any update.

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

ID: 921aa920-c25f-5227-aabb-49116da38fa7
Version Independent ID: 9b5a53ad-edf8-4d89-f8e6-cde01b7866f6
Content: az ad app
Content Source: src/command_modules/azure-cli-role/azure/cli/command_modules/role/_help.py
Service: active-directory
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

dupe #7728


Update 05-04-augmented-deliberation.md

Added a sentence about examples of models of deliberation.
This was done in collaboration with Tracy Kunkler - she doesn't have a git hub account (yet)

Should I assume you want to share credit with Tracy and she will sign up?

I'm going to nudge her to sign up in the coming days.

Kaliya

On Mon, Mar 4, 2024 at 1:41 PM Glen Weyl @.***> wrote:

Should I assume you want to share credit with Tracy and she will sign up?
—
Reply to this email directly, view it on GitHub
https://github.com/pluralitybook/plurality/pull/425#issuecomment-1977508820,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW3CC7CGEIG6MNMJJELMHTYWTS7DAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXGUYDQOBSGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks!

Research Lead, Plural Technology Collaboratoryhttps://aka.ms/plural, Research Special Projects

Collaboratehttps://github.com/pluralitybook/plurality with Digital Minister Audrey Tang, myself and dozens around the world to tell the story of Pluralityhttps://www.plurality.net/

From: Kaliya - Identity Woman @.>
Sent: Monday, March 4, 2024 10:48 PM
To: pluralitybook/plurality @.>
Cc: Glen Weyl @.>; State change @.>
Subject: Re: [pluralitybook/plurality] Update 05-04-augmented-deliberation.md (PR #425)
I'm going to nudge her to sign up in the coming days.

Kaliya

On Mon, Mar 4, 2024 at 1:41 PM Glen Weyl @.***> wrote:

Should I assume you want to share credit with Tracy and she will sign up?
—
Reply to this email directly, view it on GitHub
https://github.com/pluralitybook/plurality/pull/425#issuecomment-1977508820,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW3CC7CGEIG6MNMJJELMHTYWTS7DAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXGUYDQOBSGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHubhttps://github.com/pluralitybook/plurality/pull/425#issuecomment-1977917700, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABIRKSGNCDIN4LAYXL5SGI3YWU6BNAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXHEYTONZQGA.
You are receiving this because you modified the open/close state.Message ID: @.***>

Her handle is @bioregionaltracy
https://github.com/bioregionaltracy

Thanks!
___________________________________________________ Research Lead, Plural Technology Collaboratoryhttps://aka.ms/plural, Research Special Projects
___________________________________________________ Collaboratehttps://github.com/pluralitybook/plurality with Digital Minister Audrey Tang, myself and dozens around the world to tell the story of Pluralityhttps://www.plurality.net/
________________________________ From: Kaliya - Identity Woman @.> Sent: Monday, March 4, 2024 10:48 PM To: pluralitybook/plurality @.> Cc: Glen Weyl @.>; State change @.> Subject: Re: [pluralitybook/plurality] Update 05-04-augmented-deliberation.md (PR #425) I'm going to nudge her to sign up in the coming days. - Kaliya
On Mon, Mar 4, 2024 at 1:41 PM Glen Weyl @.> wrote: Should I assume you want to share credit with Tracy and she will sign up? — Reply to this email directly, view it on GitHub <#425 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAW3CC7CGEIG6MNMJJELMHTYWTS7DAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXGUYDQOBSGA . You are receiving this because you authored the thread.Message ID: @.>
— Reply to this email directly, view it on GitHub<#425 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABIRKSGNCDIN4LAYXL5SGI3YWU6BNAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXHEYTONZQGA. You are receiving this because you modified the open/close state.Message ID: @.***>

Can you either give me an email for her or tell her how to sign up for Gov4Git?

Research Lead, Plural Technology Collaboratoryhttps://aka.ms/plural, Research Special Projects

Collaboratehttps://github.com/pluralitybook/plurality with Digital Minister Audrey Tang, myself and dozens around the world to tell the story of Pluralityhttps://www.plurality.net/

From: Kaliya - Identity Woman @.>
Sent: Tuesday, March 5, 2024 9:17 AM
To: pluralitybook/plurality @.>
Cc: Glen Weyl @.>; State change @.>
Subject: Re: [pluralitybook/plurality] Update 05-04-augmented-deliberation.md (PR #425)
Her handle is @bioregionaltracyhttps://github.com/bioregionaltracy
https://github.com/bioregionaltracy
Thanks!
___________________________________________________ Research Lead, Plural Technology Collaboratoryhttps://aka.ms/plural, Research Special Projects
___________________________________________________ Collaboratehttps://github.com/pluralitybook/plurality with Digital Minister Audrey Tang, myself and dozens around the world to tell the story of Pluralityhttps://www.plurality.net/
________________________________ From: Kaliya - Identity Woman @.> Sent: Monday, March 4, 2024 10:48 PM To: pluralitybook/plurality @.> Cc: Glen Weyl @.>; State change @.> Subject: Re: [pluralitybook/plurality] Update 05-04-augmented-deliberation.md (PR #425https://github.com/pluralitybook/plurality/pull/425) I'm going to nudge her to sign up in the coming days. - Kaliya
On Mon, Mar 4, 2024 at 1:41 PM Glen Weyl @.> wrote: Should I assume you want to share credit with Tracy and she will sign up? — Reply to this email directly, view it on GitHub <#425 (comment)https://github.com/pluralitybook/plurality/pull/425#issuecomment-1977508820>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAW3CC7CGEIG6MNMJJELMHTYWTS7DAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXGUYDQOBSGA . You are receiving this because you authored the thread.Message ID: @.>
— Reply to this email directly, view it on GitHub<#425 (comment)https://github.com/pluralitybook/plurality/pull/425#issuecomment-1977917700>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABIRKSGNCDIN4LAYXL5SGI3YWU6BNAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXHEYTONZQGA. You are receiving this because you modified the open/close state.Message ID: @.***>
—
Reply to this email directly, view it on GitHubhttps://github.com/pluralitybook/plurality/pull/425#issuecomment-1978875153, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABIRKSH5YLARGK2XWIQOSZDYWXHWHAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZYHA3TKMJVGM.
You are receiving this because you modified the open/close state.Message ID: @.***>

Sure... and Git4Gov was super hard for me...and I'm still not sure I can
actaully use it but I will try.
Next week when I'm back I will hand hold it through the process.

Kaliya

On Tue, Mar 5, 2024 at 6:08 PM Glen Weyl @.***> wrote:

Can you either give me an email for her or tell her how to sign up for
Gov4Git?

Research Lead, Plural Technology Collaboratoryhttps://aka.ms/plural,
Research Special Projects

Collaboratehttps://github.com/pluralitybook/plurality with Digital
Minister Audrey Tang, myself and dozens around the world to tell the story
of Pluralityhttps://www.plurality.net/

From: Kaliya - Identity Woman @.>
Sent: Tuesday, March 5, 2024 9:17 AM
To: pluralitybook/plurality @.>
Cc: Glen Weyl @.>; State change @.>
Subject: Re: [pluralitybook/plurality] Update
05-04-augmented-deliberation.md (PR #425)
Her handle is @bioregionaltracyhttps://github.com/bioregionaltracy
https://github.com/bioregionaltracy
Thanks!
___________________________________________________ Research Lead, Plural
Technology Collaboratoryhttps://aka.ms/plural, Research Special Projects
___________________________________________________ Collaboratehttps://
github.com/pluralitybook/plurality with Digital Minister Audrey Tang,
myself and dozens around the world to tell the story of Pluralityhttps://
www.plurality.net/
________________________________ From: Kaliya - Identity Woman @.> Sent:
Monday, March 4, 2024 10:48 PM To: pluralitybook/plurality @.> Cc: Glen
Weyl @.>; State change @.> Subject: Re: [pluralitybook/plurality] Update
05-04-augmented-deliberation.md (PR #425<
https://github.com/pluralitybook/plurality/pull/425>) I'm going to nudge
her to sign up in the coming days. - Kaliya
On Mon, Mar 4, 2024 at 1:41 PM Glen Weyl @.> wrote: Should I assume you
want to share credit with Tracy and she will sign up? — Reply to this email
directly, view it on GitHub <#425 (comment)<
https://github.com/pluralitybook/plurality/pull/425#issuecomment-1977508820>>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW3CC7CGEIG6MNMJJELMHTYWTS7DAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXGUYDQOBSGA
. You are receiving this because you authored the thread.Message ID: @.>
— Reply to this email directly, view it on GitHub<#425 (comment)<
https://github.com/pluralitybook/plurality/pull/425#issuecomment-1977917700>>,
or unsubscribehttps://
github.com/notifications/unsubscribe-auth/ABIRKSGNCDIN4LAYXL5SGI3YWU6BNAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXHEYTONZQGA.
You are receiving this because you modified the open/close state.Message
ID: @.***>
—
Reply to this email directly, view it on GitHub<
https://github.com/pluralitybook/plurality/pull/425#issuecomment-1978875153>,
or unsubscribe<
https://github.com/notifications/unsubscribe-auth/ABIRKSH5YLARGK2XWIQOSZDYWXHWHAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZYHA3TKMJVGM>.
You are receiving this because you modified the open/close state.Message
ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/pluralitybook/plurality/pull/425#issuecomment-1978990850,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW3CC7RROVFRLKIYZWKID3YWXNX3AVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZYHE4TAOBVGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

The desktop interface or even signing up on GH?

Research Lead, Plural Technology Collaboratoryhttps://aka.ms/plural, Research Special Projects

Collaboratehttps://github.com/pluralitybook/plurality with Digital Minister Audrey Tang, myself and dozens around the world to tell the story of Pluralityhttps://www.plurality.net/

From: Kaliya - Identity Woman @.>
Sent: Tuesday, March 5, 2024 3:24 PM
To: pluralitybook/plurality @.>
Cc: Glen Weyl @.>; State change @.>
Subject: Re: [pluralitybook/plurality] Update 05-04-augmented-deliberation.md (PR #425)
Sure... and Git4Gov was super hard for me...and I'm still not sure I can
actaully use it but I will try.
Next week when I'm back I will hand hold it through the process.

Kaliya

On Tue, Mar 5, 2024 at 6:08 PM Glen Weyl @.***> wrote:

Can you either give me an email for her or tell her how to sign up for
Gov4Git?

Research Lead, Plural Technology Collaboratoryhttps://aka.ms/plural,
Research Special Projects

Collaboratehttps://github.com/pluralitybook/plurality with Digital
Minister Audrey Tang, myself and dozens around the world to tell the story
of Pluralityhttps://www.plurality.net/

From: Kaliya - Identity Woman @.>
Sent: Tuesday, March 5, 2024 9:17 AM
To: pluralitybook/plurality @.>
Cc: Glen Weyl @.>; State change @.>
Subject: Re: [pluralitybook/plurality] Update
05-04-augmented-deliberation.md (PR #425)
Her handle is @bioregionaltracyhttps://github.com/bioregionaltracy
https://github.com/bioregionaltracy
Thanks!
___________________________________________________ Research Lead, Plural
Technology Collaboratoryhttps://aka.ms/plural, Research Special Projects
___________________________________________________ Collaboratehttps://
github.com/pluralitybook/plurality with Digital Minister Audrey Tang,
myself and dozens around the world to tell the story of Pluralityhttps://
www.plurality.net/
________________________________ From: Kaliya - Identity Woman @.> Sent:
Monday, March 4, 2024 10:48 PM To: pluralitybook/plurality @.> Cc: Glen
Weyl @.>; State change @.> Subject: Re: [pluralitybook/plurality] Update
05-04-augmented-deliberation.md (PR #425<
https://github.com/pluralitybook/plurality/pull/425>) I'm going to nudge
her to sign up in the coming days. - Kaliya
On Mon, Mar 4, 2024 at 1:41 PM Glen Weyl @.> wrote: Should I assume you
want to share credit with Tracy and she will sign up? — Reply to this email
directly, view it on GitHub <#425 (comment)<
https://github.com/pluralitybook/plurality/pull/425#issuecomment-1977508820>>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW3CC7CGEIG6MNMJJELMHTYWTS7DAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXGUYDQOBSGA
. You are receiving this because you authored the thread.Message ID: @.>
— Reply to this email directly, view it on GitHub<#425 (comment)<
https://github.com/pluralitybook/plurality/pull/425#issuecomment-1977917700>>,
or unsubscribehttps://
github.com/notifications/unsubscribe-auth/ABIRKSGNCDIN4LAYXL5SGI3YWU6BNAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXHEYTONZQGA.
You are receiving this because you modified the open/close state.Message
ID: @.***>
—
Reply to this email directly, view it on GitHub<
https://github.com/pluralitybook/plurality/pull/425#issuecomment-1978875153>,
or unsubscribe<
https://github.com/notifications/unsubscribe-auth/ABIRKSH5YLARGK2XWIQOSZDYWXHWHAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZYHA3TKMJVGM>.
You are receiving this because you modified the open/close state.Message
ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/pluralitybook/plurality/pull/425#issuecomment-1978990850,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW3CC7RROVFRLKIYZWKID3YWXNX3AVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZYHE4TAOBVGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHubhttps://github.com/pluralitybook/plurality/pull/425#issuecomment-1979578472, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABIRKSA44O565V55UPT4D4LYWYSZVAVCNFSM6AAAAABEEIY3O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZZGU3TQNBXGI.
You are receiving this because you modified the open/close state.Message ID: @.***>


projj sync help message is wrong

see https://github.com/popomore/projj/blob/master/lib/command/sync.js#L22

PR is welcome


Bootstrap 4 Availability

Hi,
Although its not ready, how can I down load Boostrap version of Append grid?
I am using the append grid at its basic form.
Q

Hi cottonfieldusa,
As you know it is still under development, the final version may be complete different to current working version. You may take a look the Bootstrap 4 sample here and use it by downloading the jquery.appendGrid-development-v2.js and jquery.appendGrid-development-v2.css. And sorry that I cannot provide support at this stage. Please come back and check the status later :)

Hi cottonfieldusa,
The new version of AppendGrid that can work with Bootstrap 4 is out now!
You may check demo and docs on AppendGrid website.
And I planned to add more demo cases in future. I hope you’ll enjoy it!

Thank you for working on version 2.
I have been using version 1 in the application and its been great!
Thanks..
I would like upgrade to version 2 (pure javascript with bulma) from version 1 and keep the same
look of it from version 1 (Jquery UI default blue)
What is the best way to do this?
Q

Hi cottonfieldusa,
There are several breaking changes in version 2 (such as removed jQuery as dependence) so I am afraid you need to change your code and test the grid one by one. I tried to keep the name of options and methods unchanged to help developers a bit easier to upgrade.
For example, if your grid is simple enough:
// Version 1
$('#tblAppendGrid').appendGrid({
  columns: [/* Your columns */],
  initData: [/* Your data */]
});

You may just modify the first line and add element / uiFramework to make it work with Bulma:
// Version 2
var myAppendGrid = new AppendGrid({
  element: "tblAppendGrid",
  uiFramework: "bulma",
  iconFramework: "fontawesome5", // Optional
  columns: [/* Your columns */],
  initData: [/* Your data */]
});

The event handling on generated input elements are changed quite a lot. Please check the demo and documentation for more. Thanks for using AppendGrid!


fix #51

This PR resolves #51.
All I've done is simply remove the .internal-link class from .original-internal-link element.
It seems to suffice to fix the issue because the MarkdownPostProcessor of Strange New Worlds finds its target elements with querySelectorAll("a.internal-link:not(.snw-link-preview)"):
https://github.com/TfTHacker/obsidian42-strange-new-worlds/blob/48a4d547a653aaefab76214ab9b0e573f167babe/src/view-extensions/references-preview.ts#L153
@zhaoshenzhai Hi, in my quick test, Dataview inline fields (which were the only concern) are rendered without any problems after this change.
I'd like to check the compatibility issues with other plugins as well, but the plugin folders in the test vault seem to lack main.js.
What is the intention of this? How do you usually test them in the test vault? (I'm aware of makefile, but it has nothing to do with other plugins than MathLinks, so)

After a PR that I sent to Dataview is merged, I will probably be able to fix #51 in a more "proper" way.

What is the intention of this? How do you usually test them in the test vault? (I'm aware of makefile, but it has nothing to do with other plugins than MathLinks, so)

Ah sorry, I must'hv mistakenly included all the main.js's in the .gitignore. I'll fix this now.

Thanks! I also deleted MathLinks's main.js (once a file starts being tracked, it will be kept tracked even after adding it to .gitigore so we have to explicitly delete it from the repo: https://stackoverflow.com/a/45400404).

After a PR that I sent to Dataview is merged, I will probably be able to fix #51 in a more "proper" way. So it's a good idea to keep this PR untouched for now.

A recent Dataview update contains this pull request, so I believe now this issue can be fixed in a better way. I will close this PR and create a new one later.

It turned out I was wrong. Dataview's inline fields are rendered in a (seemingly) strange way, and it makes the problem harder.


BUG-dict variable state not sent to front-end

Let this test example for Taipy / Chalk'it : https://github.com/bengaid/covid_taipy_chalkit
Please clone it to get it. Use latest Taipy / Chalk'it setup for running it.
In this example, a new JSON-like Python dict option_e is recomputed after country-list user selection:
def on_change(state, var, val):
    if (var == 'selected_countries'):
        state.selected_countries = val
        state.covid_filtered = get_covid_filtered(covid_data_frame, val)
        state.option_e = echarts_option(state.covid_filtered, val)

But websocket does not send the updated value to front-end. The ECharts chart is not updated.

Dict are blocked by default
Workaround: [dict()]

@bengaid
@FredLL-Avaiga suggestion is on point. You can try his workaround to see if it works for you or not.


Daemon: policy add rules to DNS when is locked.

On PolicyAdd rules are copied to dns manage. It can be a race where the
same rule can be pushed to the DNS without lock and overwrite old ones.
With this change the StartdNS will happens under the lock, and race will
be skipped.
Fix #7105
Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

This change is

test-me-please

test-me-please

test-me-please


Cannot read property "namedValues" from undefined.

Hello, I was following https://developers.google.com/workspace/solutions/content-signup to set the script editor but then met this error:
TypeError: Cannot read property "namedValues" from undefined.
onFormSubmit

Any clue about how to resolve this?
Thanks!

I had the same problem. Looking for a solution as well.