Module 1 Blocker

Module: 01
Lab/Demo: Setup
Exercise: 01
Task: 01
Step: 07
Description of issue
Hello, our tester has run into a blocker during his review. Below are his notes and a screenshot of his results while running this step.
Unrecognized PowerShell command. Environments cannot be created for future use in the module, qualifying the issue as a blocker.
I believe the PowerShell command may have been superseded. However, I am not fully sure.
Are there alternative options?

Repro steps:

Are you following the instructions in M09_ALM_Lab_setup.md>
The setup.ps1 script creates the Powershell function Start-PL-600-Setup
The error is that it can't find he Powershell function Start-PL-600-Setup
Are you running PowerShell as an Administrator?
Are you running PowerShell command line or PowerShell ISE?
Have both commands
a. Set-ExecutionPolicy RemoteSigned
b. Set-PSRepository -Name PSGallery -InstallationPolicy Trusted
been run
Were you prompted to sign in with your Tenant administration credentials?

Are you performing these steps from within an ALH environment on the Virtual Machine?

@MOC-Labs-Review can this be closed?  I don't see anything in response to Julian's point.

Closing due to no response from lab tester.


Unable to load tflite model in iOS.

Hi, I am trying to add this library to my project. For android given instructions are working. But for iOS i tried many ways to add library but i am getting null is not an object evaluating (TfliteReactNative.loadModel).
I tried below ways induvidually and failed to load model.

Xcode -> MyProject -> ProjectFolder -> Right Click -> Add filed to "My project" -> Disabled Copy items if needed and Enabled Create groups -> Selected models folder(which contains model.tflite and label.txt files).
Xcode -> MyProject -> ProjectFolder -> Right Click -> Add filed to "My project" -> Enabled Copy items if needed and enabled Create groups  -> Selected models folder(which contains model.tflite and label.txt files).
Tried to add Directly .tflite and .txt without models folder with above 2 steps.
Drag ad dropping models folder/ files.

Where am i going wrong? Please anyone help me out.
My configurations are:
macOS                        - 10.15.7
react-native               - 0.64.2
tflite-react-native     - 0.0.5
pods:
TensorFlowLite (1.13.1)
TfliteReactNative (0.0.4)
Thanks in advance.

@RaviKiranMakala did you ever get this resolved?
I am having the same issue.
My versions are:
macOS - 11.5.2
react-native - 0.65.1
tflite-react-native - 0.0.5
pods:
TensorFlowLite (1.12.0)
TfliteReactNative (0.0.4)

Same here


Document more of the REST API

Can you limit requests returning multiple results to a single,
representative item?

Is this PR ever going to be merged? It's impossible to use the REST API without this documentation. I had to build it myself.

Sorry, I missed that my concerns were addressed. Merged!


Document how to keep internal action calls trackable

What problem is this solving
When defining stores with the setup syntax, it quickly happens that one action calls another.
Similarly you are allowed to setup a watcher inside a store, which again triggers an action.
These action calls trigger the plain function directly and pinia cannot notice it.
Thus you cannot observe them with $onAction and they are not visible in the devtools.
Proposed solution
I am working around the issue by calling the action as if it was on another store:
export const useCounterStore = defineStore('counter', () => {
  const count = ref(0)
  const doubleCount = computed(() => count.value * 2)
  function increment() {
    count.value++
  }

  const self = useCounterStore()
  function add(n: number) {
    for (const i = 0; i < n; i++) {
      self.increment()
    }
  }

  watch(count, (currentCount) => {
    if (currentCount > 10) {
      self.square()
    }
  })

  return { count, doubleCount, increment, square }
})

Most importantly this is something that should be documented as the recommended way to call actions from other actions even within the same store, so users won't run into the issues described above.
Is it possible to add self as a parameter to the setup function even? I am a bit uncertain whether type-inferrence works there. E.g.:
export const useCounterStore = defineStore('counter', (self) => {
  …
})

Describe alternatives you've considered
I looked through all kinds of issues and discussions to see whether this issue has been discussed before, but suprisingly couldn't find any mention of it.

This is a design limitation… making it possible requires a verbose wrapping of the actions that’s why it wasn’t added. Within the store, actions are just functions after all
I wouldn’t do the nested call of the store so I cannot recommend it in the docs

I wouldn’t do the nested call of the store so I cannot recommend it in the docs

It also wouldn't be my first choice, but I don't see any alternative.
Arguably you could also forgo seeing these action calls in the devtools / handling them with $onAction.
Shouldn't the user at least be made aware of this limitation and given the option to use such a nested call?
Are there any major downsides to doing the nested call? It is after all just how actions in different stores already call each other.

I think it might be worth leading a proper RFC in the discussions in this repo. There are many factors to take into consideration like type safety, possible runtime errors and error prone patterns for nested store calls. All for the sake of $onAction. In other words: it might not be worth
If it’s for dev tools, I believe creating build plugins could be the key

Hello!
I have encountered a similar issue in my project, where we have numerous internal action calls, particularly for prefetching data during store initialization. This became problematic when I attempted to emit an event for every store action to handle updates.
Adding this feature would greatly improve the Pinia user experience!


Deprecate warn/2, add warning/2

For library changes, see CHANGELOG.
Upgraded CI Elixir from 1.10.4 OTP 23.0.4 to 1.16.1 OTP 26.2.1.
Upgraded dev dependencies:

:credo from 1.1 to 1.7 (1.1 breaks on Elixir 1.16.1)
:dialyxir from 1.1.0 to 1.4 (1.1.0 breaks on Elixir 1.16.1)
:ex_doc from 0.25 to 0.31 (to sync with the updated design and features)

This should be a minor release (from 1.6.1 to 1.7.0) since it is adding MetaLogger.warning/2 and deprecating MetaLogger.warn/2.

Closing, duplicate of #20


[FEEDBACK][TypeScript][Virtual Assistant & Skill] Implement Dependency Injection

Purpose
What is the context of this pull request? Why is it being done?
Changes
Are there any changes that need to be called out as significant or particularly difficult to grasp? (Include illustrative screenshots for context if applicable.)
Tests
Is this covered by existing tests or new ones? If no, why not?
Feature Plan
Are there any remaining steps or dependencies before this issue can be fully resolved? If so, describe and link to any relevant pull requests or issues.
Checklist
General

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated the appropriate tests
[ ] I have updated related documentation

Closing this PR as it was promoted in MS


Issue with hiera lookup and boolean value

I'm currently using hiera to create my program:
foo_autostart: true

supervisor_programs:
    'foo':
        command        : "foo"
        [...]
        autostart      : "%{hiera('foo_autostart')}"

And for some reason, I have to use a hiera lookup for the autostart value.
With puppet 3.8.4 and hiera 1.3.4 and the code above, this ends up with autostart = "true". As the module is checking if the value is a real boolean, the compilation fails because "true" is a string.
Based on my knowledge, it's not possible to do a hiera lookup with boolean.
So I'm wondering if we cannot replaced this check:
if $autostart { validate_bool($autostart) }

by this one (already used for the autorestart param):
if $autostart { if !is_bool($autostart) { validate_re($autostart, ['true', 'false']) } }

or even if we cannot use the str2bool func like this (not tested, but you get the idea):
if $autostart { if !is_bool($autostart) { $autostart=str2bool($autostart) } }

See https://forge.puppetlabs.com/puppetlabs/stdlib#str2bool
I'm happy to do the PR, but need to know which solution you want to be implemented...

This is strange, are you sure that doing "%{hiera('foo_autostart')}" isn't just added the quotes?
We use json for our hiera backend and haven't seen any problems using booleans for autostart.

You are right, doing "%{hiera('foo_autostart')}" adds the quotes but when using the yaml format, you have to use the quote for lookup, see:
https://docs.puppet.com/hiera/3.1/variables.html#in-data

Note: With YAML data sources, interpolating into hash keys means those hash keys must be quoted.

I've already tried to remove the quote but then the catalog compilation fails...

Is there a reason you would want to do a lookup for this? Just taking this to it's logical conclusion, wouldn't this mean all booleans would need this change to avoid similar problems?

My use case is not very simple but basically, I want to be able to override the autostart value.
As I don't use deep merging, with this lookup, I just need to write foo_autostart: false in another hiera file which has bigger priority in the hierarchy.
If I don't use the lookup, I have to override the full supervisor_programs: hash, so that mean the code is completely redundant and hard to maintain.
TL;DR; I reckon all the booleans might need this change too. But on the other hand, I believe the hiera lookup works fine for boolean in puppet 4.x (not sure of that however)

I've merged #91 to solve this as there are very few only boolean values.


Consider implementing faketime

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem?
Needed to test something in the future.
Describe your solution
The solution described on https://stackoverflow.com/questions/67279712/how-can-i-temporarily-change-the-time-in-my-ddev-web-container worked perfectly, but would be even better if ddev had native solution to the problem.
Describe alternatives
No response
Additional context
No response

Hi @alanmels - We try to keep from adding extra things to DDEV that don't benefit a significant audience. However, you could easily create a ddev add-on using the recipe in Stack Overflow. I highly recommend it, super easy, see

https://ddev.readthedocs.io/en/latest/users/extend/additional-services/
https://github.com/drud/ddev-addon-template

I'll be happy to help. If you do that, you have the ability to create a good README that explains how to use it for your exact use-case. Please report the URL here when you do it!

As I explained the solution on the link worked perfectly for me. Thanks!

I was hoping you'd create the add-on :)


Type hints: toolbar/middleware

Description
This PR adds type hints to the middleware and toolbar modules
Fixes #1705
Checklist:

[x] I have added the relevant tests for this change.
[ ] I have added an item to the Pending section of docs/changes.rst.

I haven't touched on the changes.rst file. I wasn't quite sure if it's required for this type of change.

I'm not quite sure how to resolve this lint error. Can anyone give me a hint?

@leandrodesouzadev The automatically generated documentation for panels (https://django-debug-toolbar.readthedocs.io/en/latest/panels.html#debug_toolbar.panels.Panel) will contain a reference to your new GetResponse stub. The stub itself isn't documented however, so Sphinx doesn't know where GetResponse should point to.
Something like this fixes it:
diff --git a/docs/panels.rst b/docs/panels.rst
index db4e9311..f2fe35c0 100644
--- a/docs/panels.rst
+++ b/docs/panels.rst
@@ -352,6 +352,8 @@ There is no public CSS API at this time.

     .. automethod:: debug_toolbar.panels.Panel.run_checks

+.. autoclass:: debug_toolbar._stubs.GetResponse
+
 .. _javascript-api:

 JavaScript API

I'm not sure if that's what we want. Maybe there's a different way to specify the type which doesn't require the GetResponse class to be documented as well? Or maybe documenting the stub itself (as a temporary measure, until Django ships type hints?) would be the way to go?

Will definitely take a look into it

@leandrodesouzadev :wave: I wanted to check in to see how this is coming along. Did you want to keep moving forward with it?


Add FitX Support

Added FitX support
Added /now_util command
Enabled re-using the session cookie (via .env)
Added License

Let me know if it still works on the original McFit API, I tried to keep it compatible but obv. can't test it as I don't have a membership there myself.

Yes everything is working fine. I think both version endpoints are working on mcfit like on fitx.


Incorporated feedback to adjust auto generated component names

Current coverage is 100% (diff: 100%)

Merging #34 into master will not change coverage

@@           master   #34   diff @@
===================================
  Files          21    21
  Lines         440   441     +1
  Methods        53    53
  Messages        0     0
  Branches       45    45
===================================
+ Hits          440   441     +1
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update cc71dd6...68d2a53


fix method names for onExpanded and onClosed callbacks

These methods weren't work.
Changed method names in AccordionItem component

Property
Type
Description
Default

onExpanded
Function
Callback for when item is expanded
null

onClosed
Function
Callback for when item closes
null

Thanks for the PR @murzvo but ended up updating the prop names within the README!

@brukh I think that onExpanded and onClosed are more meaningful method names, because these callback running after item expand. onClose and onExpand names is for events before expand and close.
What do you think about that?


feat:show transaction history for current wallet

Added functionality to fetch wallet transaction history

@ezequiel-rodriguez @chrisarevalodev Could you pls take a look ?

Hey @SarveshLimaye,
please submit your contribution in the following form in order to get a reward for it.
Thank you

Hey,
First of all, thanks for all the hard working you are doing and all your contributions.
Regarding this new feature, here is some feedback:

Alchemy key being asked each time, it is not a great UX.  My suggestion is that it can be included as a parameter in the command.   This way it can be reused each time the user call the command

The response from alchemy being thrown directly to the output as a json is not cool.  Check the other commands as examples.   In case for special fields or to have some flexibility if alchemy changes their response, you can add a section where you parse each pair of parameter/value and show them as they are.

@ezequiel-rodriguez Thank you so much for reviewing the changes and sharing feedback.
I have tried to address your feedback -

Now, I am taking the alchemy key from the parameter, saving it in the rootstock-wallet.json file, and using it for further calls.
Instead of returning a response directly from Alchemy api, I am parsing each pair of parameter/value and returning it to the user

It would be great if you could review the changes.
Thank you!

Hey, @SarveshLimaye! Reviewed your changes and is looking very well. I just have two comments.

For edge case please find a way to just print one error message instead of two for the same reason

I noticed that there's no indicator for which network are the tx's being queried from. So please add a console.log saying something like: Fetching transactions on Rootstock Mainnet or testnet, whatever the user chooses.
Not sure if the API makes it possible but it would be cool to show the time the tx was made.
Also it would be cool to have an optional parameter so the user can specify how many tx's wants returned (last 3, last 5, etc.)

@chrisarevalodev Thank you so much for reviewing the changes . I have addressed all four of your suggestions

Fixed edge case bug showing the same error twice
Added an indicator for which the tx's are being queried
Now the time the tx was made is also visible
Added an optional parameter so to specify how many tx's to be returned

It would be great if you could review the changes. I would love to hear any further suggestions.


Unsure how to switch account for Settings Sync

Testing #229420
On insiders.vscode.dev, I'm unsure how to switch with which account I'm using Settings Sync.

I think you go here:

then "Turn Off"

then

follow what shows up and you get to

Let me know if you were referring to something else @rzhao271
cc @sandy081

Here you go - https://code.visualstudio.com/docs/editor/settings-sync#_switching-accounts


DOCK-2299: Correctly handle thrown Error during push

Description
This PR modifies the webservice's GitHub apps push-processing code to more gracefully handle an Error thrown by the code within.  Previously, Exceptions were handled properly, but Errors slightly not: pushes resulting in Errors were logged as "success"-ful in the GitHub apps logs.  We discovered this problem because Ash created a WDL workflow that referenced itself via a http url, which Cromwell happily imported until overflowing the stack, resulting in a StackOverflowError.
The integration test processes a similarly-recursive WDL, which it assumes will trigger a StackOverflowError.  The test will break if Cromwell changes that behavior.  I tried another approach wherein Errors were triggered via mocks, but it didn't work, and it'd be fragile, anyways...
Currently, to give the user a bit of a clue as to what has happened, this PR includes the short class name of the Error in the github apps log message.  For example, StackOverflowError.  Could this leak any sensitive information?  Please consider.
We catch RuntimeException | Error on purpose, rather than Throwable, so that if we change the inner code so it throws any new check exceptions, we'll know about it.
The codebots really don't like us catching Errors, even though I believe it is the right call in this case.  They also want us to update to more modern Java features and constructs.  But I didn't see anything really objectionable in their feedback.
I updated TransactionHelper to handle Errors similarly.  There is probably other code scattered throughout our system that will skip logging/cleanup operations if an Error is thrown.
Review Instructions
Fork https://github.com/dockstore-testing/recursive-wdl and register it on both qa and staging.  Check the github apps logs on each.  qa should report that the push failed, and staging should report that the push succeeded.
Issue
https://ucsc-cgl.atlassian.net/browse/DOCK-2299
#5274
Security
There may be related issues, I have contacted the appropriate personnel.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running mvn clean install
[x] Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
[x] Follow the existing JPA patterns for queries, using named parameters, to avoid SQL injection
[x] If you are changing dependencies, check the Snyk status check or the dashboard to ensure you are not introducing new high/critical vulnerabilities
[x] Assume that inputs to the API can be malicious, and sanitize and/or check for Denial of Service type values, e.g., massive sizes
[x] Do not serve user-uploaded binary images through the Dockstore API
[x] Ensure that endpoints that only allow privileged access enforce that with the @RolesAllowed annotation
[x] Do not create cookies, although this may change in the future
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

In general, if the issue is that the cromwell/scala bridge code can throw Errors, shouldn't our catch clauses be closer to the Scala bridge code or just inside it? This approach seems to result in collateral damage or too much "by-catch. For example, this seems to affect updateUserWorkflows as well which has nothing to do with WDL parsing.
I do wonder since

Catching either Throwable or Error will also catch OutOfMemoryError and InternalError, from which an application should not attempt to recover.

could happen in big parts of "our" code too.

The integration test processes a similarly-recursive WDL, which it assumes will trigger a StackOverflowError. The test will break if Cromwell changes that behavior. I tried another approach wherein Errors were triggered via mocks, but it didn't work, and it'd be fragile, anyways...

Sounds ok, in general in favour of a high-level integration test approach without mocks. That said, if cromwell changes their code to avoid a stack overflow, I think I'm ok with that. Just make a code comment

Any of our code can throw an Error, and we should respond to that eventuality in a reasonable way.  This PR modifies the enclosing catch so that when it attempts to log an Error-related failure, it does so correctly.  After that, the resource handler ends.  It's not trying to recover, but rather provide the user with correct information about the failed push, because what it was reporting previously, if any Error was thrown, was wrong.  If we don't change the code, it's wrong, and if it simply exits without logging anything, the user won't know what has happened, because this endpoint is invoked behind the scenes via the webhook callback...
Similarly, the TransactionHelper mods are designed to do the same, to attempt some cleanup and to log some information before rethrowing the Error, so that we have more information about what happened.  If you feel super strongly about this, I'm fine with reverting the TransactionHelper changes, they're not as essential...
And sure, we can also wrap Cromwell to translate the StackOverflowError into something else.  But, I kinda feel like it's a severe enough issue that maybe we let it ripple out and terminate the thread with the other Errors.  We still need the enclosing catch to make sure that the failed push is logged correctly...

Any of our code can potentially throw an Error, and we should respond to that eventuality in a reasonable way.

Yes, but my understanding of the Java documentation is that this is usually an OS problem so "we" as a platform can respond to it but not in the Java code.
For example as previously noted

An Error is a subclass of Throwable that indicates serious problems that a reasonable application should not try to catch. Most such errors are abnormal conditions. The ThreadDeath error, though a "normal" condition, is also a subclass of Error because most applications should not try to catch it.

If our belief is that the Scala libraries are indicating a serious problem incorrectly, i.e. a non-serious problem that we can deal with, then reiterating that the Error (or StackOverflowError) catch should occur "closer" to the problem and not accidentally catch Errors that may be thrown by non-Scala code.
e.g.

In particular, if we get an OOM error and catch it, I worry that this might put the web service in a indeterminate state.

If we don't change the code, it tells the user something incorrect, and if it simply exits without logging anything, the user won't know what has happened, because this endpoint is invoked behind the scenes via the webhook callback...

This seems like a little of a different problem though. For this, maybe we should log a failure first and then only overwrite it with a success afterwards. Thinking that if the whole application crashes due to a genuine error (or if Scala does have a real problem), then we would leave behind evidence of a problem in the github app logs.

test

One other observation @svonworl I think "draft" PRs do not show up on the daily reminder, so if you wanted to keep this for inspiration/cherry-picking, that can be a route

I thought this was fixed, perhaps by a different PR/branch?

The original ticket was to address bad registration behavior due to recursive WDLs: specifically, Ash noticed that when he registered a recursive WDL, no entry appeared, and nothing appeared in the app logs.  In a different PR, we fixed the original problem by patching the code closer to where the corresponding StackOverflowError occurs, for that specific case.
However, the merged PR didn't address the general case of an Error thrown in the registration code.  This PR was intended to remedy that, but there was pushback, and I was not able to merge.  I still believe that when an Error is thrown, it is preferable for the surrounding code to attempt some basic cleanup and logging, rather than let the Error pass through, silently, unhandled.  Thus, I kept this PR around, in case the team ever subscribes to that point of view.

Thus, I kept this PR around, in case the team ever subscribes to that point of view.

In the meantime, though, we get a reminder every day about it.
I'm going to put this back into draft and I think that will stop that reminder


rename Leaf.apar to Leaf.ppar to be more accurate

breaking for packages that uses Leaf.apar

bors r+


Trace Event Logging JSON

Since there was some interest in adding some kind of TraceEvent log to BP and we also needed a JSON log containing some information about the environment for our logging I went ahead and formatted it to the TraceEvent spec. Right now it builds up the events from the xml reports but ideally I think in the future writing out events as they happen per simulator and just merging all the events into one JSON at the end would be ideal.
I also added a secondary config option to skip over successful tests and only write failed test case events, this probably isn't the ideal way to run it but was a requirement for us to keep the file size down.

Superseded by #324


gfx1100 is missing a flagsfile.

Even when specifying a local flagsfile, it tries to download one (and fails).
 python -m shortfin_apps.sd.server --fibers_per_device=4 --workers_per_device=1 --target=gfx1100 --model_config=./python/shortfin_apps/sd/examples/sdxl_config_i8.json --device=amdgpu --flagfile ./python/shortfin_apps/sd/examples/sdxl_flags_gfx1100.txt --build_preference=compile --device_ids 0 1
 ....
 ....
 OSError: Failed to fetch URL 'https://sharkpublic.blob.core.windows.net/sharkpublic/sdxl/11132024/configs/sdxl_flagfile_gfx1100.txt': HTTP Error 404: The specified blob does not exist.

I added a flagfile to azure which should let you past this -- the builder always fetches the required files even if an alternative is provided via the CLI. This is inefficient and wrong and is tracked in https://github.com/nod-ai/SHARK-Platform/issues/525


Think of a simpler example when describing setup cases

Yea, deploy contract is a bit weird. I just have a feeling that max_examples is not a simple enough cheatcode for an example, as you have to know what fuzzing is beforehand which makes learning curve steeper.
Alternative idea, which maybe tells a better story
    func __setup__():
        deploy_contract()
    end

     func setup_test_a():
        store(5)
    end

     func test_a():
        assert load() = 5
    end

     func test_b():
        assert load() = 0
    end

leaving the decision to you
Originally posted by @MaksymilianDemitraszek in https://github.com/software-mansion/protostar/pull/703#discussion_r960489050

I thought about this a little and my conclusions are:

The store&load example is not good, because you can simply put the overriding store call inside test_a directly.
Current docs state Use case hooks to configure the behavior of particular test case, for example, by calling the max_examples cheatcode.. This is actually quite correct statement describing what is this used for, and the provided example supplements it.

I don't see other feasible usage cases using current feature set. I think we should leave this as -is.

The store&load example is not good, because you can simply put the overriding store call inside test_a directly.

Yes you can, but I believe this feature (hooks per case) has been requested many times after we introduced setup hooks.  I think users like the logical separation between setup, and case as it improves readability.
IMO current docs are lacking in explanation how actually setup_case hooks work introducing them only as a space to configure strategies, even though there are more usecases.
I am not sure about the change though. Feel free to close the issue if arguments above aren't sufficient.


Live

Docs Build status updates of commit 264d3cc:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Update CategoryWidget.js to addres RFC ch150631

RFC link: https://app.clubhouse.io/cartoteam/story/150631/allow-disable-filters-and-allow-having-selected-categories-on-widget-init

Pull Request Test Coverage Report for Build 786142322

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.186%

Totals

Change from base Build 741828883:
0.0%

Covered Lines:
547

Relevant Lines:
656

💛  - Coveralls

Hi, @juandjara 2 questions to have a bit more of context for this:

Wouldn't this apply the same for other widgets (like PieWidget)? If so, then I think all adaptations should be included in this PR.
Do you have or can you prepare a small example (eg codesandbox) where you need & use this piece of code, so we understand better the use case?


[ArPow Tarball] Reconcile inline code changes

For some repos, there are code changes being made via sed, awk or a msbuild task with existing source-build.  Example: Finding and setting up the arcade logger.  These existing code changes should be reviewed and included in the new tarball if necessary.

These will be included as each repo is added to the tarball.


TUI python logging basicConfig is ignored in msg.initialize_curses_logging

logging.basicConfig can typically only be called once unless force=True (see https://docs.python.org/3/library/logging.html#logging.basicConfig). This means that when msg.initialize_curses_logging is called, the handler is not actually changed in logging.basicConfig. I confirmed this by listing the handlers after both initialize_*logging fucntions were called, and it shows that the two handlers that are set in initialize_logging are retained after the call to initialize_curses_logging.


Embed ODB into the source

So that users do not need to install ODB and/or build on Windows machines. Copy-paste ODB source to our source tree and add makefile and test with GCC and VS compiler.

Please use the same title for the ticket as you do for whatever branch you will be doing this in  following a convention where tickets are numbered in ascending order (i.e. G-1, G-2). This will make organizing tickets for releases considerably easier.

At a minimum please list the branch name in the ticket, otherwise it will be impossible to match up.

New install scripts obviate the need for this.  Will keep separate for now.


[TwitterV2Bridge] New parameter to include only media tweets

New optional parameter to include only media tweets. This satisfies issue #1401
I also remove a bunch of commented-out debug log calls, that I was using during development of this bridge.

LGTM


Add missing CONTRIBUTING.md

This repository should have CONTRIBUTING.md file with description of workflow for adding new stuff to repository.

Closed via #38


报错如下

2018-09-20 14:11:43 ERROR c.j.b.ConnectionHandle:388 - Database access problem. Killing off this connection and all remaining connections in the connection pool. SQL State = HY000
2018-09-20 14:11:44 ERROR application:742 - PreparedStatementCallback; uncategorized SQLException for SQL [SELECT SQL_CALC_FOUND_ROWS id, name, schema, source, urn, FROM_UNIXTIME(source_modified_time) as modified, rank_01 + rank_02 + rank_03 + rank_04 + rank_05 + rank_06 + rank_07 + rank_08 + rank_09 as rank FROM (SELECT id, name, schema, source, urn, source_modified_time, CASE WHEN name = 'aum' THEN 3000 ELSE 0 END rank_01, CASE WHEN name like 'aum%' THEN 2000 ELSE 0 END rank_02, CASE WHEN name like '%aum%' THEN 1000 ELSE 0 END rank_03, CASE WHEN urn = 'aum' THEN 300 ELSE 0 END rank_04, CASE WHEN urn like 'aum%' THEN 200 ELSE 0 END rank_05, CASE WHEN urn like '%aum%' THEN 100 ELSE 0 END rank_06, CASE WHEN schema = 'aum' THEN 30 ELSE 0 END rank_07, CASE WHEN schema like 'aum%' THEN 20 ELSE 0 END rank_08, CASE WHEN schema like '%aum%' THEN 10 ELSE 0 END rank_09 FROM dict_dataset WHERE MATCH(name, schema,  properties, urn) AGAINST ('aum v_aum' IN BOOLEAN MODE) ) t ORDER BY rank DESC, name, urn LIMIT ?, ?;]; SQL state [HY000]; error code [1271]; Illegal mix of collations for operation 'match'; nested exception is java.sql.SQLException: Illegal mix of collations for operation 'match'
2018-09-20 14:11:44 ERROR o.s.t.s.TransactionTemplate:167 - Application exception overridden by rollback exception
org.springframework.jdbc.UncategorizedSQLException: StatementCallback; uncategorized SQLException for SQL [SELECT FOUND_ROWS()]; SQL state [null]; error code [0]; Connection is closed!; nested exception is java.sql.SQLException: Connection is closed!
为啥这个ddl语句是 DEFAULT CHARSET = latin1;有的是utf8

dataset table
CREATE TABLE dict_dataset (
id                          INT(11) UNSIGNED NOT NULL                                                                   AUTO_INCREMENT,
db_id                       SMALLINT(6) UNSIGNED NOT NULL                                                               DEFAULT 0,
name                        VARCHAR(200)                                                                                NOT NULL,
schema                      MEDIUMTEXT CHARACTER SET utf8,
schema_type                 VARCHAR(50)                                                                                DEFAULT 'JSON'
COMMENT 'JSON, Hive, DDL, XML, CSV',
properties                  TEXT CHARACTER SET utf8,
fields                      MEDIUMTEXT CHARACTER SET utf8,
urn                         VARCHAR(500)                                                                                NOT NULL,
source                      VARCHAR(50)                                                                                NULL
COMMENT 'The original data source type (for dataset in data warehouse). Oracle, Kafka ...',
location_prefix             VARCHAR(200)                                                                                NULL,
parent_name                 VARCHAR(500)                                                                                NULL
COMMENT 'Schema Name for RDBMS, Group Name for Jobs/Projects/Tracking Datasets on HDFS ',
storage_type                ENUM('Table', 'View', 'Avro', 'ORC', 'RC', 'Sequence', 'Flat File', 'JSON', 'BINARY_JSON', 'XML', 'Thrift', 'Parquet', 'Protobuff') NULL,
ref_dataset_id              INT(11) UNSIGNED                                                                            NULL
COMMENT 'Refer to Master/Main dataset for Views/ExternalTables',
is_active                   BOOLEAN NULL COMMENT 'is the dataset active / exist ?',
is_deprecated               BOOLEAN NULL COMMENT 'is the dataset deprecated by user ?',
dataset_type                VARCHAR(30)                                                                                NULL
COMMENT 'hdfs, hive, kafka, teradata, mysql, sqlserver, file, nfs, pinot, salesforce, oracle, db2, netezza, cassandra, hbase, qfs, zfs',
hive_serdes_class           VARCHAR(300)                                                                                NULL,
is_partitioned              CHAR(1)                                                                                NULL,
partition_layout_pattern_id SMALLINT(6)                                                                                NULL,
sample_partition_full_path  VARCHAR(256)
COMMENT 'sample partition full path of the dataset',
source_created_time         INT UNSIGNED                                                                                NULL
COMMENT 'source created time of the flow',
source_modified_time        INT UNSIGNED                                                                                NULL
COMMENT 'latest source modified time of the flow',
created_time                INT UNSIGNED COMMENT 'wherehows created time',
modified_time               INT UNSIGNED COMMENT 'latest wherehows modified',
wh_etl_exec_id              BIGINT COMMENT 'wherehows etl execution id that modified this record',
PRIMARY KEY (id),
UNIQUE KEY uq_dataset_db_id_urn (db_id,urn)
)
ENGINE = InnoDB
DEFAULT CHARSET = latin1;

#1457
加微信群讨论

Dear issue owner,
Thanks for your interest in WhereHows. We have recently announced DataHub which is the rebranding of WhereHows. LinkedIn improved the architecture of WhereHows and rebranded WhereHows into DataHub and replaced its metadata infrastructure in this direction. DataHub is a more advanced and improved metadata management product compared to WhereHows.
Unfortunately, we have to stop supporting WhereHows to better focus on DataHub and offer more help to DataHub users. Therefore, we will drop all issues related to WhereHows and will not accept any contribution for it. Active development for DataHub has already started on datahub branch and will continue to live in there until it's finally merged to master and project is renamed to DataHub.
Please check the datahub branch to get familar with DataHub.
Best,
DataHub team


using "YYYY MM DD -- dddd, MMMM Do, YYYY" as calendar format causes duplications

create a blank vault, add tasks and tasks calendar wrapper
set the daily note format to YYYY MM DD -- dddd, MMMM Do, YYYY (e.g. 2024 04 30 -- Tuesday, April 30th, 2024)
do this in calendar wrapper as well.
In daily note, add a task with a future due date and it appears twice.
This does not happen to tasks not in daily notes (not obviously with other date formats...)

Screenshot

Sandbox

Obsidian Sandbox.zip

Hi @ksdavidc , I am trying your sandbox vault and I am not seeing duplicate items?

Thanks for following up.
If you add another task, it will appear, at least on my system.
(redownloaded the vault I sent you above, to be certain)
https://github.com/Leonezz/obsidian-tasks-calendar-wrapper/assets/62230004/17dfd50d-5eef-48a0-b56f-cd10be6a1e6a


Broken link http://vows.sbolstandard.org/glyph/SO:0000031/png

The broken link http://vows.sbolstandard.org/glyph/SO:0000031/png appears on https://sbolstandard.org/sbol-visual-ontology/, and it can be searched for on the page. It resolves to https://synbiodex.github.io/sbol-visual-ontology/Glyphs/SequenceFeatures/aptamer/aptamer.png and yields a 404 not found error.
@goksel what is the right fix for this? Is there something wrong with the link itself? Or is there something wrong at the SBOL Visual end? We would like to fix this link if possible.
The link http://vows.sbolstandard.org/glyph/SO:0000031/svg works, it is the /png version that does not.

Remove the /png example

Let's remove the example with the png file. The link will work if a repository folder has a png file.

https://manpages.ubuntu.com/manpages/trusty/man1/rsvg-convert.1.html

https://github.com/SynBioDex/SBOL-visual/tree/master/Glyphs

Need to run a new release of the SBOL Visual Ontology after this is done

Need to create the png's as part of the release


transparent layers?

Sorry if this a duplicate or if misunderstand the configuration.
I have the following config where holding f activates the extend layer with navigation keys on the homerow. The problem is that none of the other keys work when the extend layer is active and I'm unable to use modifiers combined with the arrow keys.
Guess I can copy the needed modifiers to extend layer as well, but I just wanted to confirm if this is the expected behavior first or if I'm missing some other config options. Thank you!
[ids]

0001:0001

[main]

leftalt = layer(meta)
leftmeta = layer(alt)
capslock = overload(C, esc)
f = overload(extend, f)

[extend]

h = left
j = down
k = up
l = right
y = home
u = pagedown
i = pageup
o = end
m = enter
; = backspace

Sorry if this a duplicate or if misunderstand the configuration.

The problem is that none of the other keys work when the extend layer is active

This is by design. See #103 if you are interested in the reasoning. I am toying with the idea of making some kind of opt-in transparent layer, but I am not yet convinced this is a good idea. Feedback is always appreciated.

and I'm unable to use modifiers combined with the arrow keys.

You need to define a nested layer which explcitly specifies the modifiers of interest. Since this seem to be a common use case I have added an example to examples/.

f = overload(extend, f)

This is probably a bad idea without a timeout. You will end up with a lot of accidental layer activations. You probably want something like overload(extend, f, 200), but I would advise against even this. If you are interested in overloading your homerow keys then I would advise you to read through #34 (and perhaps reconsider ;)).

I eventually came to my senses and made this the default behaviour. See the man page for details :).


Add AppVeyor CI (Windows) support

As of ruby-sys 0.2.3, ruru-based extensions can be built on Windows. It would be nice to add CI support for Windows builds.
A starting point can be found at my rusty_blank project: https://github.com/malept/rusty_blank/blob/master/.appveyor.yml
Some notes about my config file:

You probably want to test against stable, beta, and nightly (I only needed stable there).
I intentionally omitted MSVC builds from my AppVeyor config because the popular RubyInstaller for Windows is built with the MinGW toolchain.
Ignore the artifact/deploy sections - that is for my project to distribute Rust+Ruby extensions.

Hi @malept!
Thank you, I'll take a look at it tomorrow

Thanks, AppVeyor support enabled!


give support for android termux of amd64/arm64 even an apk

[x] I have checked that a similar feature request does not already exist.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
It cannot apt install scrcpy by termux on android arm64.
I haven't try it on android x86, yet.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Termux-ADB can make an android device wifi-adb-ly debug another.
#Referring to that#, Give support for android termux of amd64/arm64  even an apk, please.
If so, I would use my android pad to telecontrol my android TV .
All in all, what is it is that an android device control another.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
At least, it would apt install scrcpy by termux on android.
Maybe, it would pm install scrcpy.apk by termux on android.
However, being after so would be more convenient for us to telecontrol android TV than a conventional remote.
Additional context
Add any other context or screenshots about the feature request here.

#100

Some of the distro's provided by Termux's proot-distro have scrcpy packages available, though I haven't personally tested them. Being that Termux is only a terminal emulator, you'd have use something like an Android X server or VNC server


Adding pkgutil to share the opendp.whitenoise namespace

e.g. Namespace to be shared with packages generated from https://github.com/opendifferentialprivacy/whitenoise-core-python

resolved


Allow transport to indicate if read/write errors are fatal or not.

Currently, the peer loop interprets different error conditions to check if a transport is broken after a failed read or write. However, the exact rules are different per transport, so the peer loop is not in a position to properly determine this.
This responsibility should be moved to the Transport trait, which should indicate with each error if the transport is still usable or not.

PR #40 opened. Closing this.


Removing Http's close of open fd

Having Http close the fd on the peer without cleaning up Transport::peers means the following:

First client connects, and gets fd X.
Peer information is generated from this connection, and is inserted into Transport::peers<fd, Peer>
Client request is served, and Http calls close on fd X, but doesn't clean up Transport::peers
Second client connects, and fd X is reused.
Transport::peers isn't updated, as it already exists.
Now the peer associated with client 2 wrongly contains the info from client 1.

I opted to remove Http's closing of the fd, since it shouldn't be releasing resources that it doesn't own. FD closing and peers cleanup is handled by Transport::handlePeerDisconnection().
If there are clients out there that rely on the server disconnecting the TCP session after the full request has been served, this change could cause those clients to hang indefinitely. I'm not sure how many clients out there are like that, though, as the payload size is explicitly sent to them.
All of this is because Peer::hostname() will have stale information if many different servers connect serially with shortlived requests.

Codecov Report

Merging #579 into master will decrease coverage by 0.64%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #579      +/-   ##
==========================================
- Coverage   70.99%   70.34%   -0.65%
==========================================
  Files          49       49
  Lines        6957     6950       -7
==========================================
- Hits         4939     4889      -50
- Misses       2018     2061      +43

Impacted Files
Coverage Δ

src/common/http.cc
81.02% <ø> (-0.34%)
:arrow_down:

src/client/client.cc
78.81% <0%> (-6.68%)
:arrow_down:

include/pistache/net.h
83.33% <0%> (-5.56%)
:arrow_down:

src/common/peer.cc
67.74% <0%> (-1.62%)
:arrow_down:

src/common/net.cc
88.93% <0%> (-1.28%)
:arrow_down:

src/common/transport.cc
65.5% <0%> (-0.78%)
:arrow_down:

include/pistache/async.h
91.37% <0%> (-0.24%)
:arrow_down:

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 b5b1711...6852557. Read the comment docs.

I found description in this PR now, so I know why it has be done, but seems that changes are not complete.
I believe there should be added some indication from Http to Transport to remove Peer as "       Connection: close" HTTP header indicates that connection should be closed after request/response sequence.

Patches are welcome.  Please include relevant unit tests though.


Need to update

change --model llm-xpu to bigdl-llm
Missing gpu install command?
Add a description what's the relationship between llb.py and main.py
记录一下每个task要跑多久，放在表格里？方便人家估算时间。
Does arc="arc_challenge" or both challenge and easy？

For item 4, recorded in the excel, the sheet after acc test results.
For item 5, yes, arc in llb.py equals to arc_challenge in main.py. All argument mappings can be found in dict definition in llb.py.


2.x server address fixes

Moving Server to core/utils for more generic use.
Removing usage of InetSocketAddress.
Replication peer resolve to dns name only (except for A records)

NetflixOSS » eureka » eureka-pull-requests #102 SUCCESS
This pull request looks good

NetflixOSS » eureka » eureka-pull-requests #105 SUCCESS
This pull request looks good


Encrypted images support

What's necessary to support encrypted images, such as those created via Time Machine when encryption is turned on? dmg2img supposedly can decrypt these with the password provided, but it didn't work on the DMG mounted under the sparsebundlefs mount point.

I am also trying to read encrypted sparsebundles. I'm pleased to say that sparsebundlefs seems to work. However, I'm struggling to mount the .dmg file that gets produced.
When I try a standard mount, I get this error: wrong fs type, bad option, bad superblock on /dev/loop1 (as covered in your README).
When I try to use parted, I get this:
Warning: Unable to open /tmp/my-disk-image-2/sparsebundle.dmg read-write (Read-only file system).  /tmp/my-disk-image-2/sparsebundle.dmg has been opened read-only.
Warning: Unable to open /tmp/my-disk-image-2/sparsebundle.dmg read-write (Read-only file system).  /tmp/my-disk-image-2/sparsebundle.dmg has been opened read-only.
Error: /tmp/my-disk-image-2/sparsebundle.dmg: unrecognised disk label
Model:  (file)
Disk /tmp/my-disk-image-2/sparsebundle.dmg:<PHONE_NUMBER>408B
Sector size (logical/physical): 512B/512B
Partition Table: unknown
Disk Flags:

I have also tried extracting the .dmg with catacombae's dmgextractor which seems to get further, though by the time the output .iso file reaches about 650MB, I get a runtime exception of "Too many open files". :confused:
I'm thinking of upping the file descriptor limit and trying again...
That aside, thank you for the software! I'm happy to got as far as I so far have.

Let me know if you figure out what went wrong. You can try running sparsebundlefs in debug mode to see if there are any errors in the logs:
https://github.com/torarnv/sparsebundlefs#troubleshooting

I have the same problem tried troubleshooting but it sticks on
sparsebundlefs: bundle has band size 8388608 and total size<PHONE_NUMBER>000

Sorry, just returned to this, same thing happening to me as above:
sparsebundlefs: mounting `/media/capsule/jacks-macbook-pro.sparsebundle' at mount-point `/tmp/my-disk-image'
sparsebundlefs: bundle has band size 8388608 and total size<PHONE_NUMBER>408

And just hangs? Try attaching a debugger and see what the backtrace it, if possible

That would be e.g. gdb -p <pid>, and then bt all

Hi,
I made a version that support encrypted sparse bundle. It's committed here : https://github.com/jief666/sparsebundlefs.
Let me know if you'd like a pull request to integrate this in your branch.
Regards.

Hey @jief666 ! Great that you've implemented encrypted sparse bundle support. Could you give a high level overview of how an encrypted sparse bundle differs from a non-encrypted one? In particular why the decryption needs to live inside sparsebundlefs and not by whatever mounts the resulting dmg?

I even didn't think about mounting the result dmg as encrypted one. I had a quick look today and I think Linux can't mount encrypted dmg (but not sure). At least the one on my NAS.
I also had a quick look at dmg format. The first sector of an encrypted dmg looks very much like the token file. sparsebundlefs will still has to know that it's an encrypted sparsebundle and replace the content of the first sector by token, maybe fixing some offset too. And then, in my linux, I would another fuse layer to decrypt the dmg.
Looks simpler to me to have sparsebundlefs take care of decryption. Basically most of it is in sparsebundle_read_process_band_encrypted.

@jief666 I managed to get an image to compile after installing openssl-1.0 (it seems to use some deprecated methods that aren't supported in 1.1), but the mount point then just contains an empty spartsebundle.dmg file. Any idea why?
The code does seem to tell the difference between a correct and incorrect password.

No never got that. Did you try the debug non-daemon mode ?

@jief666 Thanks. I've had a bit of time to play with it and it seems the Info.plist file got lost somehow, so the code was using a block size of 0. Although it's an unlikely case it might be worth adding a little more error checking around either the block size or the whole size being 0.

Good you find.
Yes, sure. Any code to detect problems is good. Wanna do it and submit it ?

To me it seems that it is indeed the sparsebundle that is encrypted, not the resulting image. At least this seems to be the case with encrypted Time Machine backups.
Each band is encrypted and decryption requires a token located at *.sparsebundle/token. For this to works, I don't see any way around handling decryption in sparsebundlefs.
I took a quick look at rebasing @jief666's encryption branch on the current master, but they have diverged too much for me to handle. I would love to see support for encrypted sparse images in sparsebundlefs though!

Sorry, because torarnv didn't seem interested, I moved on. I created a project hdimount (https://github.com/jief666/hdimount/tree/wip) which is the union of darling-dmg and APFSFuse.
The result can mount a lot of different images (partitionned or not, HFS or APFS, encrypted or not, sparsebundle or dmg) without the need of loop devices or exploring manually partition table.
Good luck all.

Sounds like a nice project! I'll point people who need encrypted images support your way

@mafredri It seems there are alternate solutions to this: https://gist.github.com/calebgross/6f2633dc952d5ae851eda3c12a608aeb

Sounds like a nice project! I'll point people who need encrypted images support your way

Thanks.

If only it didn't require bloody Eclipse to build!

@kode54 : Congratulation for your nice comment. Free is not enough for you, you also want it ready
for your platform. Plus, you say wrong things. So please, next time, think twice before being unpleasant.

"If only it didn't require bloody Eclipse to build!"

it's a work in progress, I use Eclipse.  A free multi-platform IDE seems good to me. Feel free to use another IDE and maintain and submit project files

Even worse, it only compiles on macOS!

The goal of these project, is to mimic what macOS do. So compiling for macOS make perfect sense so I can easily compare. If found a lot of bug in darling-dmg by doing that.

Even worse, it only compiles on macOS!

Wrong. I compile for linux 32 bits and 64 bits. I test it on my linux NAS. I paid close attention to int type so they shouldn't be any warning in 32 and 64 bits. It should compile easily on windows but I didn't do it yet. You are welcome to help on that instead of criticising people who give away freely their work.
kode54, welcome to the open source world.

Please move further discussion about https://github.com/jief666/hdimount to the relevant repo/issue tracker, thanks!

Yes, sorry. There won't be any further comment about that from me.

This gist is 404, here's a mirror I pulled from a Google cached copy.

Sorry, I changed my GitHub handle a few months ago…looks like it broke my gists. Here's a working link to my instructions for decrypting/mounting an encrypted sparse bundle disk image using sparsebundlefs: https://gist.github.com/noperator/6f2633dc952d5ae851eda3c12a608aeb


all cadvisor metrics have id="/"

Description
cAdvisor metrics do not have correct cgroup path.
Describe the results you received:
The metrics can be obtain with the following command:
curl -s -k https://localhost:6443/api/v1/nodes/linuxkit-025000000002/proxy/metrics/cadvisor --cert /etc/kubernetes/pki/apiserver-kubelet-client.crt --key /etc/kubernetes/pki/apiserver-kubelet-client.key

What you will see is e.g.
container_memory_max_usage_bytes{container_name="",id="/",image="",name="",namespace="",pod_name=""} 3.03910912e+08

Note id="/".
Describe the results you expected:
On an Ubuntu install, you will see more metrics instead with different ids, e.g.:
container_memory_max_usage_bytes{container_name="weave",id="/kubepods/burstable/pod7db61ed7-e655-11e7-a92e-065f2a149e22/1c7e2c87fbdbf35542a2e060147b245455c45ca3cada8c68a9d730a12551d46e",image="weaveworks/weave-kube@sha256:07a3d56b8592ea3e00ace6f2c3eb7e65f3cc4945188a9e2a884b8172e6a0007e",name="k8s_weave_weave-net-vlw97_kube-system_7db61ed7-e655-11e7-a92e-065f2a149e22_1",namespace="kube-system",pod_name="weave-net-vlw97"} 8.4353024e+07

Additional information you deem important (e.g. issue happens only occasionally):
Kubernetes version 1.9.0 was used in both cases. Ubuntu-based cluster was installed using https://github.com/weaveworks/kubernetes-ami/pull/15.
See details here https://gist.github.com/errordeveloper/2847ea94df2b2b0cccb60f0a6aa2b20f.

This is probably to do with kubelet running in a container, and cAdvisor gets somehow confused...

I assume this is only a docker problem as handled in linuxkit/kubernetes#23 . Did you check cri-containerd?


set_position accepts FEN string

Maybe the set_position and set_fen_position methods could be conflated? or at least the input checked to see if it makes sense.
sf = stockfish.Stockfish('path-to-sf')
sf.set_position('2k5/3b4/p3p3/1p1pPn2/1Pp3pB/P3K3/2P2P2/8 w - - 2 36')
sf.get_best_move() #e2e4

I'm sorry, but it seems to me that at the moment this is unnecessary


nether load tests

Issue: #PutYourIssueNumberHere

[ ] Tick here to confirm that you have run RunCodeFormatter.ps1

Description:

Minor improvements to run load test remotely on VSTS
Use appsettings instead user ConextParameters, as this fails when running with VSTS

Test:
Update app.settings with the correct parameters to run against:
<appSettings>
    <add key="BaseUrl" value="http://localhost:5000"/>
    <add key="ClientId" value="resourceownertest"/>
    <add key="ClientSecret" value="devsecret"/>
    <add key="AdminUserName" value="netheradmin"/>
    <add key="AdminPassword" value="N3therAdm1n"/>
  </appSettings>

run any of the load tests in the project

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


Crashes on macos 13.0.1 (22A400) with latest xcode

After selecting a level, the game renders, but before my block displays:
tetris(44546,0x30ec64000) malloc: *** error for object 0x600001628010: pointer being freed was not allocated
tetris(44546,0x30ec64000) malloc: *** set a breakpoint in malloc_error_break to debug
[1]    44546 abort      tetris

Funny, 2nd time I run it, it runs fine :/


Fabricator Layout

Hey @LukeAskew -
I hope all is going well. Sorry I have been MIA recently with making some contributions to Fabricator over the last months.  I continue to love using Fabricator and we have some projects that the final deliverable is actually a style guide inside an instance of Fabricator and clients absolutely love it.
One point that I wanted to bring up quickly was an issue we have found with working with Fabricator over  6-10 projects. The issue specifically has to do with the design layout of Fabricator and the issues it causes with responsive layouts.
While I realize you can hide the side menu to test responsive within Fabricator, this causes a poor user experience because to go to another section you have to reopen the menu, click the link, and then hide the menu all over again. Also if you want to quickly navigate to another block within a section, you have to do the same thing. It would be nice if one was able to always have the menu available while still being able to test responsive layouts.
One suggestion would be to change the layout from the menu being on the side to it being on the top. That way you can always have the menu available to you no matter what you are working on.
Thoughts?

No problem! I've been crazy busy as well but have plans to release some big updates on fabricator soon.
I feel your pain with the layout issue - it's something I've come up against too. Not sure what the solution is but top nav might be a good option. I will play around with new menu styles for this next release.

Sweet man. Don't want you have to carry the full load. I am actually sketching out a few ideas myself. I will pass them along to you in when I have them down.

This is great. I think the dropdowns might be problematic with long lists, but its on the right track.
I'm thinking for v2 of Fabricator configurations like this will be "plugins" or "themes". I'm still working out the architecture details, but I definitely plan to allow more flexibility with stuff like this.

Any suggestions on how we could there could be a horizontal menu without drop downs?

We're loving Fabricator, finding it incredibly useful although we do find the UI problematic, so encouraged to see this conversation.
Something I'd love to see discussed in a future update is: Component updates that don't lose scroll position - CSS updates are seamless but when changing HTML the whole page refreshes, losing the scroll position. This is especially difficult when testing responsive breakpoints.
 We are starting a project that will be a large, deep taxonomy toolkit. I'll drop some screenshots in here as they come available.
@jameswragg For sure. I think this is a bug with how I refactored the menu. I am planning to fix it at some point. Sorry for the inconvenience!


[Feature Request]Allow idAttribute can be an array

Currently the idAttribute just can be a string, but usually the response's key of id from the api is changeable, e.g. the id of the User entity maybe is id, or userId, or authorId, or something else.
Of course we can change the key in the fetch/ajax, but that's troublesome and we should not do this because it's redux-orm's responsibility.

@markerikson  @plandem


Start behavior as setting

At the moment Zapp always starts with "Live" screen. I mostly use the "Mediathek" screen, though. Maybe it would be good to let the user choose and implement a setting to start either in "Live" or "Mediathek" screen.

Thank you very much for your suggestion!
However, I would like not to clutter the settings screen with more options most users won't use or need. It will get difficult to maintain in the long run and I like to focus on features with a broader impact for all users.


whats happen when user dont have whatsapp?

can you make it default to work like tel: ?

Hello Ranjit
First of all thanks for this amazing plugin, but each time install this plugin and run on my phonegap app, it shows the WhatsApp number you are trying to send msg to does not registered on whtsapp but I have tried with multiple numbers registered on whatsapp.
Can you please let me know, which one is the correct format of number to send whatsapp msg to?
Regards
Deftsoft Developer

Hiim not the dev of the plugin , im a user
  From: purelydeft<EMAIL_ADDRESS>
To: ranjitpandit/whatsapp-phonegap-plugin<EMAIL_ADDRESS>Sent: Friday, July 24, 2015 6:18 AM
Subject: Re: [whatsapp-phonegap-plugin] whats happen when user dont have whatsapp? (#1)
Hello RanjitFirst of all thanks for this amazing plugin, but each time install this plugin and run on my phonegap app, it shows the WhatsApp number you are trying to send msg to does not registered on whtsapp but I have tried with multiple numbers registered on whatsapp.Can you please let me know, which one is the correct format of number to send whatsapp msg to?Regards
Deftsoft Developer—
Reply to this email directly or view it on GitHub.

WhatsApp number you are trying to send msg to does not registered on whtsapp but I have tried with multiple numbers registered on whatsapp.
can you solve this problem !!?
how we can pass this massages ?!


Hover Definitions for Ruby Keywords

Set up the tooltips
Add tooltips to these keywords. The markdown files should have the description below and a newbie-friendly description.
Try to avoid having tooltips which are over-zealous (e.g. DefNode will work on any keyword within a method definition).
### Tooltips
- [ ] `__ENCODING__` - The script encoding of the current file.
- [ ] `__LINE__` - The line number of this keyword in the current file.
- [ ] `__FILE__` - The path to the current file.
- [ ] `BEGIN` - Runs before any other code in the current file.
- [ ] `END` - Runs after any other code in the current file.
- [ ] `alias` - Creates an alias between two methods (and other things).
- [ ] `and` - Short-circuit Boolean and with lower precedence than `&&`.
- [ ] `begin` - Starts an exception handling block.
- [ ] `break` - Leaves a block early.
- [ ] `case` - Starts a case expression.
- [x] `class` - Creates or opens a class.
- [x] `def` - Defines a method.
- [ ] `defined?` - Returns a string describing its argument.
- [ ] `do` - Starts a block.
- [ ] `else` - The unhandled condition in `case`, `if` and `unless` expressions.
- [ ] `elsif` - An alternate condition for an `if` expression.
- [ ] `end` - The end of a syntax block. Used by classes, modules, methods, exception handling and control expressions.
- [x] `ensure` - Starts a section of code that is always run when an exception is raised.
- [ ] `false` - Boolean false.
- [ ] `for` - A loop that is similar to using the `each` method.
- [x] `if` - Used for `if` and modifier `if` expressions.
- [ ] `in` - Used to separate the iterable object and iterator variable in a `for` loop.
- [x] `module` - Creates or opens a module.
- [ ] `next` - Skips the rest of the block.
- [ ] `nil` - A false value usually indicating “no value” or “unknown”.
- [ ] `not` - Inverts the following boolean expression. Has a lower precedence than `!`.
- [ ] `or` - Boolean or with lower precedence than `||`.
- [ ] `redo` - Restarts execution in the current block.
- [ ] `rescue` - Starts an exception section of code in a `begin` block.
- [ ] `retry` - Retries an exception block.
- [x] `return` - Exits a method.
- [ ] `self` - The object the current method is attached to.
- [x] `super` - Calls the current method in a superclass.
- [ ] `then` - Indicates the end of conditional blocks in control structures.
- [ ] `true` - Boolean true.
- [ ] `undef` - Prevents a class or module from responding to a method call.
- [x] `unless` - Used for `unless` and modifier `unless` expressions.
- [ ] `until` - Creates a loop that executes until the condition is true.
- [ ] `when` - A condition in a `case` expression.
- [ ] `while` - Creates a loop that executes while the condition is true.
- [x] `yield` - Starts execution of the block sent to the current method.

Write long-form documentation
Write newbie-friendly documentation in .md form for each of the following keywords. Once written, link to it from the tooltip. The link should open the markdown file locally in the editor.
Please see the north star comment from @vinistock below:
Expand thoughts on newbie-friendly documentation

I would expand on certain things and try to be super exhaustive with all aspects of the language. For example, in the methods documentation, what is a method?
In the modules documentation, we could explain that they are Ruby's way of supporting multiple inheritance, since it does not allow for more than on parent class. We should also mention the differences between prepending, including or extending a module

The goal of these docs is to make someone who is new to Ruby (or new to programming altogether) be able to gather a better understanding of how Ruby works from inside the editor. So a good exercise is trying to put yourself into the position of a newbie. What could be confusing? What was unclear when you first started with Ruby?

### Long-form docs
- [ ] `__ENCODING__` - The script encoding of the current file.
- [ ] `__LINE__` - The line number of this keyword in the current file.
- [ ] `__FILE__` - The path to the current file.
- [ ] `BEGIN` - Runs before any other code in the current file.
- [ ] `END` - Runs after any other code in the current file.
- [ ] `alias` - Creates an alias between two methods (and other things).
- [ ] `and` - Short-circuit Boolean and with lower precedence than `&&`.
- [ ] `begin` - Starts an exception handling block.
- [ ] `break` - Leaves a block early.
- [ ] `case` - Starts a case expression.
- [ ] `class` - Creates or opens a class.
- [ ] `def` - Defines a method.
- [ ] `defined?` - Returns a string describing its argument.
- [ ] `do` - Starts a block.
- [ ] `else` - The unhandled condition in `case`, `if` and `unless` expressions.
- [ ] `elsif` - An alternate condition for an `if` expression.
- [ ] `end` - The end of a syntax block. Used by classes, modules, methods, exception handling and control expressions.
- [ ] `ensure` - Starts a section of code that is always run when an exception is raised.
- [ ] `false` - Boolean false.
- [ ] `for` - A loop that is similar to using the `each` method.
- [ ] `if` - Used for `if` and modifier `if` expressions.
- [ ] `in` - Used to separate the iterable object and iterator variable in a `for` loop.
- [ ] `module` - Creates or opens a module.
- [ ] `next` - Skips the rest of the block.
- [ ] `nil` - A false value usually indicating “no value” or “unknown”.
- [ ] `not` - Inverts the following boolean expression. Has a lower precedence than `!`.
- [ ] `or` - Boolean or with lower precedence than `||`.
- [ ] `redo` - Restarts execution in the current block.
- [ ] `rescue` - Starts an exception section of code in a `begin` block.
- [ ] `retry` - Retries an exception block.
- [ ] `return` - Exits a method.
- [ ] `self` - The object the current method is attached to.
- [ ] `super` - Calls the current method in a superclass.
- [ ] `then` - Indicates the end of conditional blocks in control structures.
- [ ] `true` - Boolean true.
- [ ] `undef` - Prevents a class or module from responding to a method call.
- [ ] `unless` - Used for `unless` and modifier `unless` expressions.
- [ ] `until` - Creates a loop that executes until the condition is true.
- [ ] `when` - A condition in a `case` expression.
- [ ] `while` - Creates a loop that executes while the condition is true.
- [ ] `yield` - Starts execution of the block sent to the current method.

### Special Cases
- [ ] Introduce a nicer pattern in `requests/hover.rb` rather than constantly adding `elsif` statements

A false value usually indicating “no value” or “unknown”.

I'd suggest "falsey" rather than false.

A false value usually indicating “no value” or “unknown”.

I'd suggest "falsey" rather than false.

We are pulling these from the official docs, @andyw8. We will hopefully be improving these in our newbie-friendly extended docs, though.
Or do we know the right people to update the official docs with the improved definition?

Spoke to @vinistock and we won't document cases without a corresponding Prism::XNode. For example, the do keyword tends to belong to a CallNode in Prism rather than as a DoNode. In such cases, we will just not document it, but also ensure that hovering over do doesn't show the tooltip for a CallNode.


Project abandoned?

Is this project abandoned or it will be ported to python 3.12 at some point?

Is there any branch or similar for this project that's currently active on dev?


Fix config templating on Ansible 5

In Ansible 5, lookup function produces string output (new default? I couldn't find any confirmation about this). Applying to_nice_json provides a double encapsulated JSON output, which is invalid for Consul.
I'd like some feedback as I have only tested this on Linux so far with Ansible 5. Not sure if this is backwards compatible or maybe Ansible introduced a breaking change unindentionally.
Some more info here: https://github.com/ansible/ansible/issues/76443

@Blefish   What is Ansible 5?

Latest version of ansible

hmmm I guess I live too much in the world of Ansible core

Thanks for the fix and merging.


[resize-observer] Multiple callbacks on a single html element

Is your feature request related to a problem? Please describe.
Unable to add multiple callbacks on a single HTML element. When I do useResizeObserver(target, callback); and target is the same on multiple components, only the latest callback will be called.
Describe the solution you'd like
useResizeObserver should not override existing callback on elements. So I propose changing callback map from Map<any, UseResizeObserverCallback> to Map<any, Array<UseResizeObserverCallback>>. Then changes subscribe and unsubscribe callback too. More on this on the PR
Describe alternatives you've considered
I have not considered other alternatives
Additional context
None

I'm experiencing the same issue, kinda seemed like from the docs this was already handled

Try out the PR @tsmithATX, I'm using my branch for current development, hope it helps you


Job Status Serialisation issue

The Job constructor currently has a string type for the statusChanges parameter in the constructor and it appears that this causes RestEasy to output the statusChanges as a String in HTTP responses, instead of correctly serialising it into an Array (as it is declared to be in the actual class body for Job). Thanks RestEasy.
Meanwhile, the annotations picked up for the OpenAPI document (which are subsequently propagated to the automatically generated client) pick up the statusChanges field (correctly) as a List , which causes a deserialisation error on the client side when the client attempts to deserialise the JSON string primitive as an array.
This PR harmonises the two by ensuring that any inputs to the Job constructor are declared as a real List<StatusChange> which necessitates some additional manipulation to deserialise the json from the RPC layer before re-serialising the whole thing as the correct JSON in the Resources layer.
This whole situation really needs a re-design.

Seems to work:
cassandra@cluster2-dc2-r1-sts-0 /]$ curl 'http://localhost:8080/api/v0/ops/executor/job?job_id=repair-0'
{"id":"repair-0","type":"repair","status":"COMPLETED","submit_time":1695758304316,"end_time":1695758304320,"error":null,"status_changes":[{"status":"COMPLETE","change_time":1695758304320,"message":""}]}
[cassandra@cluster2-dc2-r1-sts-0 /]$


[APPS-2897] Change product id to product code

:bear:
/cc @zendesk/wombat
Description
Change product id to product code to match DB schema. https://github.com/zendesk/zendesk_database_migrations/pull/475
References

JIRA: https://zendesk.atlassian.net/browse/APPS-2897

Risks

None

:+1:

👍


fix: typescript unions not being serialized if referenced model

Description
This PR fixes a bug where a union of 1 or more types is not correctly serialized i.e. the marshal function of a referenced class is not called, and the instance is instead serialized without respecting the underlying unconstrainedPropertyName.
There is also a problem with the unmarshalling... Because at the moment the classes are not correctly created, but I will have to fix that later 🙃

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

9 of 9 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 92.434%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
4774

Relevant Lines:
5033

💛  - Coveralls

Just realized that I am not taking multiple types into account such as arrays with unions, regular arrays, regular unions, etc... Gonna add some more 😓

The sonarcloud reporting has nothing to do with the changes introduced here. Dont know why it all of the sudden reports those 🤨

Thanks @Samridhi-98 🙇
/rtm


Create all.json for getting all controlled vocabularies

Right now someone can request all of the vocabularies as a python dictionary from this URL /vocabularies/all/
It is probably time to have a json version of that same url /vocabularies/all.json but in the json format.
We should probably add an all.py variant for consistency in the urls.

Also offer JSON for what's at /all-verbose.


Error messages after page navigation

When navigate from one page to another and backwards the widget does not reset its internal widget list. This can result in the error message Events with Problems because all widgets are listed twice and thus  for a resource there are two master.

Fixed on master (v2.1.0) and on release-2.0.x (v2.0.2).


Add aliases and always run options

Allow users to define multiple domains with vagrant-hostsupdater aliases. Especially useful for WordPress multisite.
Add option to run commands always on startup with run-always.sh, not just pre/post provision.

@hegemanjr It looks good to me. Thanks!


Error while installing Autoprefixer

Hello!
Today, I was trying to install Autoprefixer in Sublime Text.
When I finished installing, it showed me this:

I tried to reinstall so many times, still no luck.
Please suggest me.

That looks like a problem with Package Control. This package probably got corrupted somehow while installing. Try asking on the Package Control support forums.


Allow for dropping tsconfig.json in arbitrary folder and specifying the source root with tsc command

Search Terms

tsconfig.json
TS18003
project root

Suggestion
I have been deving an automation publish tool, with typescript inside. Nice work for you!
Since the tsconfig.json file need to be put in a center config folder other than project/source code root. amd i have tried to use tsc --watch --project path-to-tsconfig.json  to make this future, which is not working for now.
I got TS18003 error, and i know why this happen: tsc just want to remind us we may have a wrong include or exclude config. 😄 I cannot find any tsc command should i use to conject with --project option to specify the project root.
I find in the tsc sourcecode that the compiler.parseJsonConfigFileContent(_, _, basePath); function could let us specify the project basePath when parsing tsconfig.json, so, i want to ask if we can expose a command option to let use the same function as basePath, maybe --configFileBasePath or --configContext?
Please let me know if there already is a solution for my use case.
Thank you for your works!

Examples

tsc --project path-to-tsconfig.json --configContext path-to-source-code-root
Checklist
My suggestion meets these guidelines:

[X] This wouldn't be a breaking change in existing TypeScript / JavaScript code
[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. new expression-level syntax)

You want to specify the rootDir compiler option, either in the tsconfig you're looking at or on the command line.

@weswigham no, that doesn't work.
Give the dir structure below:
app/  <--- CWD
     src/
     package.json

config/ <-- arbitrary folder
     tsconfig.json

IN app/ dir, when i invoke tsc --watch --project ../confing/tsconfig.json --rootDir ("." or "absolute path to app"), the tsc always gives me TS18003 error.
tsconfig.json content:
{
  "compilerOptions": {
    "module": "commonjs",
    "moduleResolution": "node",
    "target": "esnext",
    "allowSyntheticDefaultImports": true,
    "esModuleInterop": true,
    "sourceMap": true,
    "declaration": true,
    "declarationMap": true,
    "removeComments": true,
    "strict": true,
    "alwaysStrict": true
  },
  "include": ["src"]
}

i could use absolute path to src folder in the include field to make this work, but, it's not my best solution.

Sorry, any response for this one? Is this a bug? or Do I misunderstand something?
Thank you for help.

You want to drop a config in one folder and have the paths within that config behave as though they're from another folder? No, I don't think it's a bug, definitely a feature request; I just misunderstood what you were asking for.


prune all images if no dangling images left

this ensures disk is freed if pruned images
I believe the absence of this may be responsible for the outage on turing https://github.com/jupyterhub/mybinder.org-deploy/issues/2252 because it's causing a lot of thrashing of the disk, and lots of cordon/uncordon events, which can cause network disruptions.

That's fair. I think the condition can be more robust, but this was the simplest fix I could think of for now.


Can't compile userver through CPM

Can't compile it with CPM
signers.cpp:5:10: fatal error: 'cryptopp/dsa.h' file not found
#include <cryptopp/dsa.h>

CmakeLists.txt
cmake_minimum_required(VERSION 3.21 FATAL_ERROR)

# create project
project(aaa CXX)

add_executable(${PROJECT_NAME})

target_sources(${PROJECT_NAME}
	PRIVATE
		src/main.cpp
)

include(cmake/CPM.cmake)
# configure userver
CPMAddPackage(
GITHUB_REPOSITORY "userver-framework/userver"
NAME userver
VERSION 1.0.0
OPTIONS
	"USERVER_FORCE_DOWNLOAD_PACKAGES ON"
	"USERVER_FEATURE_CLICKHOUSE ON"
	"USERVER_FEATURE_MONGODB OFF"
	"USERVER_FEATURE_POSTGRESQL OFF"
	"USERVER_FEATURE_REDIS OFF"
	"USERVER_FEATURE_GRPC OFF"
	"USERVER_FEATURE_RABBITMQ OFF"
	"USERVER_FEATURE_UTEST OFF"
)

target_link_libraries(${PROJECT_NAME} userver-core userver-clickhouse)

main.cpp
#include <userver/components/minimal_server_component_list.hpp>

int main(int argc, char** argv)
{
	return 0;
}

Look like similar to
https://github.com/userver-framework/userver/pull/335
But it closed without any description
If I add userver as thirdparty (from add_subdirectory) and apply fixes from 335 issue, compilation done

The patch was merged in https://github.com/Riferus/userver/commit/41f176ada93517ceeb381e2723abc40c17a6ae54
Unfortunately it caused build errors on Arch Linux, so it was reverted
https://github.com/userver-framework/userver/commit/c0ea0c91af7bde2efb42aae29c086a0672c5add0
We'll take a look at the problem. Many thanks for the reproducer!


[2024] update sponsorship pages

Reference

@leon12-aaryson

We shall be using the same sponsorship packages as of last year

thanks @Leon12-aaryson


Add *.vb files to .gitattributes

Because some of us do use VB still :)

Lgtm. Thx

Are there any interesting OSS F# projects that you aren't a maintainer on? Like wow


Make constraint resolution more flexible to different concrete extension types

The go proto library struggles to resolve values for extensions if the concrete type associated with the generated source code on hand does not match that on the descriptor of the message in question (which happens in cases where the descriptor or message is produced at runtime). This manifested in the protovalidate resolver where the extension could not be found on options attached to a descriptor produced dynamically.
Unfortunately, I struggled to find a way to get the extension and its value directly. (I'm not even sure how to get the field/ext descriptor from the descriptor; protoreflect.Fields.ByNumber doesn't work). The most straightforward solution from a few iterations was to leverage proto.RangeExtensions.
There's room here for some mild optimization to short circuit in most vanilla situations but it makes it a bit less readable. Since this only happens once per descriptor, the optimization is overkill IMO.

Unfortunately, I struggled to find a way to get the extension and its value directly. (I'm not even sure how to get the field/ext descriptor from the descriptor; protoreflect.Fields.ByNumber doesn't work). The most straightforward solution from a few iterations was to leverage proto.RangeExtensions.

I think proto.RangeExtensions is fine. The alternative is to use options.ProtoReflect().Range(...), which is similar but gives you the value as a protoreflect.Value instead of any. (Not really an improvement.)
There is unfortunately no better way -- extensions are by their nature decoupled from the extended message. So the message's schema will have no knowledge of the extensions other than their ranges. protoreflect.Fields is only for directly declared non-extension fields.
Libraries typically should just use proto.{Has,Get}Extension or protoreflect.Message.{Has,Get}, as was done here before. This is the usually the appropriate way to implement this, and is why I was suggesting perhaps the complexity of dealing with dynamic schemas be pushed to the producer of the descriptors (i.e. the buf CLI), instead of having to deal with it in every library that might consume those descriptors (like this one).
In this case, since you don't have the actual extension descriptor used to populate the options, all you have is the field number, this does indeed look like the best way to handle it.


[🐞 BUG]: url link to the read documentation is invalid or showing 404 page not found..

Description 📝
when user clicks on read- documentation the link to that is showing page not found
Link 🔗
https://retlab-dev.firebaseapp.com/answers/documentation
Steps to Reproduce 🔄

go to this Tests
select the HEURESTICS created by you
scroll down to the ansers section
click on the read-documentation
see the error PAGE NOT FOUND

Screenshots 📸
https://github.com/ruxailab/RUXAILAB/assets/142114244/33d93678-777a-4a8d-bd33-c7c77d32d122
Expected Behavior 🤔
when user clicks on read-documentation the user must be taken to the actual documentation
Actual Behavior 😱
when user clciks on the read-documentation there is an error PAGE NOT FOUND
Environment 🌍
windows 11
web browser : ms edge.
Additional Information ℹ
No response

Hey there, I'm Karan Panda.
Could you please assign this task to me? I believe I can work on fixing it promptly.

Hii @Tufailahmed-Bargir ,
Great that you are interested in contributing at RUXAILAB.
It seems that no such route as /documentation exist in the project so 404 page is the expected behavior here.
Maybe documentation page will be added in the future so you can ask to implement that as a feature.

@Gaurav-Verma07,  thanks for your valuable insights, Yes it must be a feature instead of bug report


DKMS kernel modules failing to build with 5.8.3-arch1-1-surface

I am running Manjaro Linux 20 on Surface Book 2. My kernel version is 5.8.3-arch1-1-surface.
I am getting build failures for DMKS kernel modules. This has happened with surfacebook2_dgpu_hps, nvidia 450.57 (from nvidia-beta-dkms), and the virtualbox dkms modules (of which I forget the name). I am triggering builds by reinstalling linux-surface or linux-surface-headers. All of the build logs look pretty similar to below, complaining about gcc/structleak_plugin:
DKMS make.log for surfacebook2_dgpu_hps-0.1.0 for kernel 5.8.3-arch1-1-surface (x86_64)
mar 25 ago 2020 22:29:12 -05
make -C /lib/modules/5.8.3-arch1-1-surface/build M=/var/lib/dkms/surfacebook2_dgpu_hps/0.1.0/build modules
make[1]: warning: jobserver unavailable: using -j1.  Add '+' to parent make rule.
make[1]: Entering directory '/usr/lib/modules/5.8.3-arch1-1-surface/build'
  CC [M]  /var/lib/dkms/surfacebook2_dgpu_hps/0.1.0/build/surfacebook2_dgpu_hps.o
cc1: error: incompatible gcc/plugin versions
cc1: error: failed to initialize plugin ./scripts/gcc-plugins/structleak_plugin.so
make[2]: *** [scripts/Makefile.build:281: /var/lib/dkms/surfacebook2_dgpu_hps/0.1.0/build/surfacebook2_dgpu_hps.o] Error 1
make[1]: *** [Makefile:1756: /var/lib/dkms/surfacebook2_dgpu_hps/0.1.0/build] Error 2
make[1]: Leaving directory '/usr/lib/modules/5.8.3-arch1-1-surface/build'
make: *** [Makefile:9: all] Error 2

What to make of this? Have you encountered this before? In any case, do you have any fix/debug suggestions?

It seems like there is a shared library that may be built for a non-current version of gcc? In /usrlib/modules/5.8.3-arch1-1-surface/build/scripts/gcc-plugins/, there is a pre-built structleak_plugin.so file, and in the structleak_plugin.c file the comment at the top says that Usage: for 4.5/4.6/C based 4.7

Output of strings -a /usr/lib/modules/5.8.3-arch1-1-surface/build/scripts/gcc-plugins/structleak_plugin.so | grep "GCC: ("
GCC: (GNU) 10.2.0
Output of gcc --version
gcc(GCC) 10.1.0
I am using gcc from core/gcc, which appears to be v10.1.0-2 for Manjaro 20.1, but core/gcc is v10.2.0-1 on core repo site

Different gcc versions can cause that. External modules always have to be compiled with the version that the kernel has been compiled with, so you'll somehow have to get gcc v10.2.0 running on Manjaro.
Also a note: The surfacebook2_dgpu_hps driver is outdated. It's been replaced by surface_sam_hps and is already included in the kernel.

Thanks. I've switched to using gcc from the manjaro/testing branch, which uses a compatible verson. I'm gonna close this.


Rework images

https://github.com/f3d-app/f3d-superbuild/pull/68
https://github.com/f3d-app/f3d/pull/784

Also, should we rename the repository f3d-docker-images?


NoUnusedImportsFixer - fix comment handling

NoUnusedImportsFixer

[x]  crashes on comments containing the '/' character.
[x]  removes comments placed in or around use statements
[x]  removes used imports if comments are placed in or around use statements
[x] removes used function and const imports (replaces https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/3321)

The other added delimiters are not strictly needed now, however it is good practice and more future prove (and saves time checking each time).
So kind reminder;

preg_quote($a) !== preg_quote($a, '/')

Thank you @SpacePossum.


Fix edmf types

This PR fixes some edmf types by converting scalars to Covariant3Vectors in order to help fix things for our sphere example.

Lets close this one in favour of https://github.com/CliMA/ClimaAtmos.jl/pull/972


Define tfchain POS devnet deployment setup

https://github.com/threefoldtech/tfchain/blob/development_staking/substrate-node/doc/devnet_staking.md


Name change

The functionality of this package has grown quite a bit over time and I find myself laughing at lines like
from cmip6_preprocessing.postprocessing import ...

I also think the preprocessing might give many potential users the wrong impression of what the package does. Hence I would like to change the name.
 He was just always trying to be helpful haha. And +1 to jazzhands, maybe for one of the internal functions 😉
I'm biased because "preprocessing" is what I used to call the part of my Matlab code that renamed all the variables, set all the lon/lat to uniform values, etc so I think it's great 😅 Otherwise cmip6_utils sound good to me, too.
On a side note, if you rename the package what happens to the old version? If for example, we were to update our version of cmip6_preprocessing, would it just become cmip6_utils even if we specify conda update cmip6_preprocessing? Just curious :)

I like cmip6_utils

I would like to cast a low energy vote for cmip_processing, since it's the processing of cmip that end users are afraid of and why this package is so helpful.

@dcherian you are triggering me! go back to Clippy please.  Those were the days.... a helpful paperclip to guide me through life.

Hey everyone,
I just got a talk accepted at Scipy, and I think I really should change the name (and brush up the package structure, but thats another issue).
My goal is to make it not specific for CMIP6 (ultimately this should be useful for CMIP5 aswell) and not restrict it to preprocessing. It really is covering processing more broadly at this point. Also I find my self hating to type cmip6_preprocessing all the time, so a shorter name would be exellent.
I like pyMIP/MIPpy or (pyCMIP/CMIPpy)? CMIP_utils would also work nicely.
Maybe someone (@cisaacstern?) has some cat based suggestions to enter the race here 😹

My only gripe with cmip_utils is the logo (I am being serious hahah)

Not really a fan so far.
On the other hand CMIPpy looks cool IMO:

Ill keep experimenting.

Some other ideas:

I like cmip-utils (or cmip_utils) ... personally with a slight stylistic preference for dash-delimiting the package name and under-score delimiting the importable name in Python, as seems common in other projects, but delimiters is certainly not the hill I plan to die on.

Still like mippy! or catnip : Climate Analysis Tools & Nukes (?) for Inter-comparison Projects

& Nukes

Whoa! Whoa! Whoa! NO nukes over here.
catMIP? addictively easy science with Model Intercomparison Projects

catMIP!

catnip : Climate Analysis Tools for Nifty Inter-comparison Projects


Dependency locking refinement

When using the unique lock file per project, it is possible to configure
the lockfile name and location.
This enables scenarios where the lockfile name depends on some build
properties, allowing to have different lock state for different state of
the build.

This is now updated with the documentation for the new format, the release notes for it and the compatibility note for the property change around LockMode.


Chat messages are too tall

Super minor issue, but on my (full size) monitor chat messages take up way too much space. If I want to see the video and the chat at the same time, I can see at most 1-2 messages. It should be possible to show chat messages using much less space.

I've been meaning to complain about this too... :-)


LLM Quickstart

This is a WIP LLM quickstart.
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

Also left a review with NB reviewer, just a reformulation. In General I love this quickstart. Such a great way to introduce ZenML.
Great Job!

@htahir1 yep I'll move the llm quickstart out somewhere else, and also extract the code so it can run with python run.py (for testing etc). Thanks!

Closing for now in favor of #1692


docker-compose: docker-compose yaml is modified by Rancher on ingest, so export config differs from original yaml

Rancher Versions:
Server: 1.3.2
healthcheck: 0.2.0
ipsec: 0.8.1
network-services: 0.4.0
scheduler: 0.5.1
kubernetes (if applicable): NA
Docker Version:
1.12.5
OS and where are the hosts located? (cloud, bare metal, etc):
RedHat 7, AWS EC2
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
3 node HA + external (RDS) database
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:

Create a stack from dockercompose.yml, with several labels in alphabetical order:

version: '2'
services:
  nginx:
    image: nginx
    labels:
      a_key: "a_value"
      b_key: "b_value"
      c_key: "c_value"
      d_key: "d_value"
      e_key: "e_value"
      f_key: "f_value"
      g_key: "g_value"

Once created, view the stack in compose view in Rancher, or Export config.
Compare the docker-compose.yml output here, to the original docker-compose.yml
Observe ordering of labels
Perform an upgrade, adding another label
Again, observe ordering of labels

Results:
The ordering of the labels changes. We have also observed similar changes in ordering at each yaml indent level.
This interferes with third-party tools such as Terraform that use export config to maintain state.

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


add login data getter

Summary
I just used this library for bot communication. Apparently I need to now what's the logged in bot id and act as filter to avoid self replying when checking the message from rtm_read(). So I need to get the login_data which is set in server.py in parse_slack_login_data(). I cannot access that from slackclient. So I added get_login_data() for this.
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

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


Rename ContractUpdateKey

Semantically the output outcomes of the LN module are contract updates, thus I propose renaming the key LightningOutputOutcomeKey in line with the value struct saved under the key. Thoughts @mxz42?
https://github.com/fedimint/fedimint/blob/724ab5313826dff6c45ec9d829977d3d950c6c2e/modules/fedimint-ln-common/src/db.rs#L42-L52

sounds good I think.
Are you proposing a general connection between the key and the value in naming? There'd be many keys/values to change.
If we rename this, we should also take a look at the WalletOutputOutcome and MintOutputBlindSignatures in the mint module.

Closing this. Renames that take this long aren't happening!