Disable webhooks + start application after initial setup

I'm developing a prototype based on v10.1.0 which is supposed to listen for POST requests and create a new pull request with POSTed content, a similar service to what Staticman is providing: https://github.com/shaftoe/static-comments/blob/master/index.js
I list here a few things that are popping up hoping to give an overview from a new user perspective:

I've not found documentation about how to disable webhooks, in my case at the moment I don't make use of any
would be nice to start the application automatically after initial install is completed, right now if I get it right I have to restart the process. I see a comment in setup.ts that says "If using glitch, restart the app" which might be related

I've not found documentation about how to disable webhooks

You mean disable them in probot? I'd argue that is out of scope for probot. I'd just use @octokit/core with @octokit/app-auth in this case
const { Octokit } = require("@octokit/core");
const { createAppAuth } = require("@octokit/auth-app");

const auth = new Octokit({
  auth: {
    id: 1,
    privateKey: "-----BEGIN PRIVATE KEY-----\n...",
    installationId: 123
  },
  authStrategy: createAppAuth,
});

right now if I get it right I have to restart the process

That is correct.
Right now I think we should move out the setup logic from Probot, instead be more explicit about what is happening. When the app is not configured, stop the process and inform the user to run probot setup. When the setup is done, we should stop the server and show a page that informs the user that they can run probot run ./index.js now (and log the same to the console).
It's a nice user experience to have the two combined, but it causes all kind of problems, makes probot hard to test, and is a problem for the upcoming v11 "serverless"-ready release of Probot

You mean disable them in probot? I'd argue that is out of scope for probot. I'd just use @octokit/core with @octokit/app-auth in this case

When I start Probot in develpment I see it's creating a connection to smee.io, e.g. Forwarding https://smee.io/someHook to http://localhost:3000/ and I'd like to disable that. I guess when Probot is run as production this won't be an issue though.
If I follow the above recommendation though it means I can't make use of app.auth() which has a much nicer API so I guess I'll live with an extra unused connection in this case 😆

Right now I think we should move out the setup logic from Probot, instead be more explicit about what is happening. When the app is not configured, stop the process and inform the user to run probot setup. When the setup is done, we should stop the server and show a page that informs the user that they can run probot run ./index.js now (and log the same to the console).
It's a nice user experience to have the two combined, but it causes all kind of problems, makes probot hard to test, and is a problem for the upcoming v11 "serverless"-ready release of Probot

All right, understood, it makes total sense. I'd also like to be able to put a link to my probot app GH repo in the GET /probot page once the setup is done, not sure if it's already possible at the moment.

When I start Probot in develpment I see it's creating a connection to smee.io, e.g. Forwarding https://smee.io/someHook to http://localhost:3000/

Stupid me, I see now it's just a matter of removing WEBHOOK_PROXY_URL from the environment and Probot will ignore smee.io seutp.


Name your form link doesn't work as expected

The link to name your form under the share option does not shift focus over to the form file name.

Closing as a duplicate of #2115


Flutter 2.0 problem (Gradle task assembleDebug failed with exit code 1)

Hello everyone, so I have installed flutter SDK 2.0.1 and when I run my personal project I got this error. I don't know why but it happened after I upgraded my flutter to 2.0.1. Any solutions would be appreciated
Flutter Doctor:
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 2.0.1, on Microsoft Windows [Version 10.0.19042.804], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[√] Chrome - develop for the web
[√] Android Studio (version 4.0)
[√] VS Code (version 1.54.1)
[√] Connected device (3 available)

• No issues found!

Console error
/C:/src/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_svg-0.19.0/lib/src/picture_provider.dart:52:59: Error: No named parameter with the name 'nullOk'.
        context != null ? Localizations.localeOf(context, nullOk: true) : null,
                                                          ^^^^^^
/C:/src/flutter/packages/flutter/lib/src/widgets/localizations.dart:413:17: Context: Found this candidate, but the arguments don't match.
  static Locale localeOf(BuildContext context) {
                ^^^^^^^^
3

FAILURE: Build failed with an exception.

* Where:
Script 'C:\src\flutter\packages\flutter_tools\gradle\flutter.gradle' line: 991

* What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.
> Process 'command 'C:\src\flutter\bin\flutter.bat'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 26s
Exception: Gradle task assembleDebug failed with exit code 1

It worked, I just upgraded my flutter_svg package to flutter_svg: ^0.19.3


test(modal): test dismissAll

That is 1% more of covering :)
Didn't create a helper method to dismissAll because it is just one test.

Strange we never had a test for this - LGTM


feat: export reports as PDF

Description
fix ZEN-278
https://github.com/zeno-ml/zeno-hub/assets/5690524/c684ade3-844f-44c4-8373-9152a811bb68
The PDF output is not always super clean. E.g. Vega charts sometimes to weird things with sizing. Do we generally want this?

Yeah not sure if it's worth given the quality of the PDF...


Using cmdk with Remix

Issue
I am trying to integrate this cmdk tool with a remix application, but I am facing some styling issues:

The menu always appears at the bottom of the page instead of a pop over on top of whatever is currently on the page
The cmdk menu does not match the styling showed in the demo
Is there any documentation that can help to integrate the css more appropriately, thanks

References
Here is a link to the stackblitz I have been working in: https://stackblitz.com/edit/remix-run-remix-sjsqzh?file=app/root.tsx

There are some good style examples here:
https://github.com/pacocoursey/cmdk/tree/main/website/styles/cmdk


File extension trimming

Is it possible to include the file extension when selecting a file? It's being trimmed currently. Is that the intended behavior?

Add this line to settings.json:
"path-autocomplete.extensionOnImport": true
This can be found under Options in the documentation.
It would be nice if we could choose by file language (ie, true for html, false for js, etc).

It would be nice if we could choose by file language (ie, true for html, false for js, etc).

@teamsteamdev you should be able to do that with transformations
something like
"path-autocomplete.transformations": [{
    "type": "replace",
    "parameters": ["\\.html$", ""],
    "when": {
        "fileName": "\\.html$"
    }
}],


with atom 1.34.0 lauch debugger from pull down menu does not work

I'm not sure if this issue should be at atom or go-plus or this plugin but,
lauch debugger from pull down menu does not work.
F5 works, but this is not clear from documentation.

If you mean the Debugger menu entry next to the Help - this comes from atom-ide-ui and is not related to go-debug.
A little bit of info:
atom-ide-ui (which is a bundle of several utility UIs for atom) provides a debugger UI. Apparently, it did not provide the necessary features I needed for go-debug. Unfortunately, atom-ide-ui has been archived earlier this year, so I will definitely not use this feature in the future.
You can disable the atom-ide-ui debugger (and the menu entry) in its settings -> set Enabled Features - Enable Debugger to Never enabled.

Hi
Thank you for responding, I found the disable setting.
You package work well.. really needed feature.
I would still clarify the documentation how to start it.
What I was left missing was dlv cmd prompt, I would like to just print a
variable with full lenght.. e.g string.
Br, Ari Mattila
On Tue, Feb 26, 2019 at 9:53 PM Lukas Beranek<EMAIL_ADDRESS>wrote:

Closed #150 https://github.com/lloiser/go-debug/issues/150.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lloiser/go-debug/issues/150#event-2165978098, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATmkFFVEVOTvjoDb4uen08NP57hbnk0Wks5vRZCggaJpZM4bS4__
.

I'm always open to merge requests to improve the documentation or the plugin itself :)

I have never made a pull request.. now is good time to try.. I'll do it ;)
On Wed, Feb 27, 2019 at 9:24 AM Lukas Beranek<EMAIL_ADDRESS>wrote:

I'm always open to merge requests to improve the documentation or the
plugin itself :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lloiser/go-debug/issues/150#issuecomment-467753192,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATmkFNGjDtSZ0Q4G8ejiDYm8l7UFfb7tks5vRjLKgaJpZM4bS4__
.


[th/common-kw-only] host: use common.KW_ONLY_DATACLASS in host.py

So far this was avoided, because it wasn't clear whether "host" module should depend on "common" module or vice versa.
That was also, because in the past netdev.ip_addrs() was inside the "common" module, so back then, "host" could not have imported "common" (but the other way around).
Instead, clarify that "host" very much can use "common" module (and never the other way around).
At this point, use common.KW_ONLY_DATACLASS instead of reimplementing it.

LGTM


DO_NOT_MERGE - CPF-3169 Adding PasswordPolicy

Add PasswordPolicy to Group
UpdatePassword method cleanup

Note that test cases still need to be added for the PasswordPolicy class.


Analytics-friendly route names

Looks like Firebase Analytics has made great strides in making screenName (and screenClass) more marketer-friendly but the current route annotations are somewhat rigid in that they're only path-based.  As far as I can tell, the only way to set the screenName (following the FilledStacks tutorial) is through the [path] parameter, but that has some limitations:
// app.dart - route annotations
// #1.
// routes break runtime ([navigateTo]) if they have spaces in them
CupertinoRoute(page: CreateAccountView, path: 'Create Account'),
CupertinoRoute(page: ForgotPasswordView, path: 'Forgot Password'),

// #2.
// [initial] doesn't seem to matter unless [path] is also set to '/'.
// Setting it to anything else requires "home: StartupView()" to be set on MaterialApp.
MaterialRoute(page: StartupView, initial: true, path: 'Startup')

// main.dart
MaterialApp(
      title: 'Compound',
      navigatorKey: StackedService.navigatorKey,
      onGenerateRoute: StackedRouter().onGenerateRoute,
      navigatorObservers: [
        locator<AnalyticsService>().getAnalyticsObserver(),
      ],
      // required only if "initial" path != '/'
      home: StartupView(),
      ...
    );

Ideally, routes would have both a path as well as a screenName.  Whether could be declared explicitly and/or implicitly on the actual View (say, for dynamic screen names like "Product XYZ" - see #226).  There's the foundations of a workable way here, but it only works for Stateful widgets.
Anyway, I suppose this might be considered a feature request, but I wanted to flag this as it will undoubtedly become more problematic for folks in the future as analytics becomes increasingly less developer-centric.

Thanks @manafire for the details. We're also pretty analytics centric. Well our clients are and this hasn't been a problem yet. But it's a good thing to keep note of and add in. We could add an additional parameter for ViewName and use that in the analytics observer instead of the path.
Will see where I can fit this in when high priority issues are dealt with.


fixed the code to not add a new paper if this paper doesn't have any …

…valid XREF IDs

Thanks @azurebrd


xoutput closes by it self when i click start

I set up my controller bindings and when I clicked start and launched fortnite it didnt work. so i restarted my computer and when i pressed start it closed by it self. for 3 times in a row it crashed. i restarted my pc again and it is still happening plz help

What controller are you using? Is it the twin gamepad?


Not able to run Rcpp due to space in R root directory: 'C:/Program' not found

OS: Windows
VSCode Version: 1.58.2
R Version: 4.1.0
vscode-R version: 2.1.0

Don't know whether it is a vscode-R issue or vscode issue.
r$> library(Rcpp)

r$> evalCpp("2*2")
Error in system(cmd, intern = !showOutput) : 'C:/Program' not found
Error in system(cmd, ignore.stderr = TRUE, intern = TRUE) :
  'C:/Program' not found

Is there anyway to fix this without re-installing in a non-space directory?

Does it occur with

Offical R terminal instead of radian in vscode
radian outside vscode

Does it occur with
* Offical R terminal instead of radian in vscode

* radian outside vscode

It occured with radian in vscode.

Does it occur with
* Offical R terminal instead of radian in vscode

* radian outside vscode

Just tried to disable the radian, and used the official R terminal.
It works on it.

Then it should be radian's issue.

Then it should be radian's issue.

Thanks, I will raise the issue over there then.

Found a way to fix this by defining R path in settings.json
"r.rterm.option": ["--r-binary=C:\\Program Files\\R\\R-4.1.0\\bin\\R.exe"]


[dev] Nidoran Error because of special character

I thibk this error is caused by the special characters for female / male in Nidoran's name
Traceback (most recent call last):
File "pokecli.py", line 464, in
main()
File "pokecli.py", line 68, in main
bot.tick()
File "/Users/toremielck/Desktop/PokemonGo-Bot-silosilosiloPTC/pokemongo_bot/init.py", line 84, in tick
if worker.work() == WorkerResult.RUNNING:
File "/Users/toremielck/Desktop/PokemonGo-Bot-silosilosiloPTC/pokemongo_bot/cell_workers/move_to_fort_worker.py", line 43, in work
logger.log('Moving towards fort {}, {} left'.format(fort_name, format_dist(dist, unit)))
File "/Users/toremielck/Desktop/PokemonGo-Bot-silosilosiloPTC/pokemongo_bot/logger.py", line 21, in log
print('[' + time.strftime("%H:%M:%S") + '] ' + string)
File "/Users/toremielck/Desktop/PokemonGo-Bot-silosilosiloPTC/lib/python2.7/codecs.py", line 369, in write
data, consumed = self.encode(object, self.errors)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 49: ordinal not in range(128)

There are some work in progress i think
PR: https://github.com/PokemonGoF/PokemonGo-Bot/pull/1809

This is working for me.
change line in logger.py to the following
print('[' + time.strftime("%H:%M:%S") + '] ' + string.decode('utf-8'))
credit to another commentator on a previous issues


Automatic rounding doesn't play nice with years

_ Is refactor
\x Is new feature
_ Concerns documentation

Description of suggestion or shortcoming:
I am modeling a variable for the year when human-level machine intelligence arrives. The variable has p5 = 2019 and p95 = 2074. Squiggle Hub's UI rounds off to two significant figures, so all the numbers get rounded off to 2000 or 2100 even though this is not useful.

This screenshot shows two copies of the variable, based on the AI Impacts surveys from 2022 and 2023. I can see the exact numbers for each quantile (p5, p50, etc.) by hovering over them, but without hovering, it shows every number as 2000 or 2100, which is not what I want.
I see how rounding is useful in general and it's not clear to me how you would fix this special case, but just wanted to raise the issue in case you think it's fixable.

Yea, this is an annoying problem. The issue is one of rounding and ranges, rather than years specifically.
Right now our rounding math rounds to 2 places. But really, in situations where the entire range (5th to 9th) is all in the same first 2 places, we should round to 3. But this requires more complexity.
I would flag that you can often use the @format() tag in a pinch, with the number of decimal places you want it to go.

Gr... I spent some time on this and wound up thinking that I'm confused on what I really want.
Here's some tests that an LLM suggested for how we should round. Think one challenge is that we sometimes want to use different rounding techniques.
If the range is (1000.2, 1000.8), then we want to be sure to round 1000.833 to 1000.5.  If the range is 1.0002 to 1.0008, then we want to round 1.00063 to 1.0006.
But if the range is (1, 1000), then we want to round 282.3 to maybe 300, or 280, if we default to two levels of precision.
If the range is (23.2883, 29.3523), then we probably want to round 25.52354 to at least the 1s place.
I'm going to stop working on this now, will think more about it.
describe("roundToStepSize", () => {
test.each([
// [min, max, testValue, expectedStepSize, expectedRounded]
[10005, 10009, 10007.12345, 1, 10007],
[1.0001, 1.0009, 1.00054321, 0.0001, 1.0005],
[1, 10, 5.6789, 1, 6],
[100, 200, 156.789, 10, 160],
[1, 1000, 567.89, 100, 600],
[-10009, -10005, -10007.12345, 1, -10007],
[-200, -100, -156.789, 10, -160],
[-10, 10, 5.6789, 1, 6],
[0.00001, 0.00009, 0.0000532, 0.00005, 0.00005],
])(
"value %f in range [%f, %f] rounds to %f with step size %f",
(min, max, testValue, expectedStepSize, expectedRounded) => {
const stepSize = calculateStepSize(min, max);
expect(stepSize).toBeCloseTo(expectedStepSize, 10);
  const rounded = roundToStepSize(testValue, stepSize);
  expect(rounded).toBeCloseTo(expectedRounded, 10);

  // Log the results for visual inspection
  console.log(`\nRange: [${min}, ${max}]`);
  console.log(`Original: ${testValue}`);
  console.log(`Step Size: ${stepSize}`);
  console.log(`Rounded: ${rounded}`);
}

);
});

If the range is (1000.2, 1000.8), then we want to be sure to round 1000.833 to 1000.5.

Don't you mean 1000.8? Why would you round to 1000.5?
I think a reasonable rounding rule would be:

Round to the largest significant figure that's smaller than abs(max - min) / 10.

You want to round appropriately whether the range scales arithmetically or geometrically.
Python implementation:
def place_value(x):
    return int(floor(log10(abs(x))))

def sqround(min, max, x):
    return round(x, 1 - place_value(abs(max - min)))

Results:
In [22]: sqround(1000.2, 1000.8, 1000.833)
Out[22]: 1000.83

In [23]: sqround(1, 1000, 282.3)
Out[23]: 280.0

In [24]: sqround(23.2883, 29.3523, 25.52354)
Out[24]: 25.5

This gives 1 more digit in the [1000.2, 1000.8, 1000.833] case, but I think that's good actually.


Make v-model work with props

What problem does this feature solve?
It would be nice, if v-model works when using a prop as its target. By allowing this, we would reduce error prone boilerplate code, when having "pass-through" input/model components.
A similar request was made in https://github.com/vuejs/vue/issues/8403 , however there is acutally no need to introduce a new modifier or anything.
The v-model logic would be modified in the following way:

If the target of v-model is not a prop: Use the current behavior
Else, the target of v-model is a prop:

If the prop is the model of this component (by looking at the components options.model.prop), emit an event of the type declared in  options.model.event with the new value
Else, emit an event of type update:PROP_NAME with the new value

Similar logic can be done for the .sync modifier targeting props.
What does the proposed API look like?
No API changes needed

I don't think I understand what you are asking for. Please follow the RFC process to better describe your proposal.


Formula Looping

First off, I love this plugin. However, I've noticed with larger tables there tends to be many repeated formulas to accomplish basic tasks.
Take the following table as an example.
| test 1         | test 2 | test 3 | test 4 |
| ----------- | ------ | ------ | ------ |
| Banana      | 1        | 2        | 3        |
| Orange      | 2        |           | 2        |
| Cherry       | 2        |           | 2        |
| total          | 5        | 2        | 7        |
<!-- TBLFM: @2$4=sum(@2$2..@2$3) -->
<!-- TBLFM: @3$4=sum(@3$2..@3$3) -->
<!-- TBLFM: @4$4=sum(@4$2..@4$3) -->
<!-- TBLFM: @>$2=sum(@I..@-1) -->
<!-- TBLFM: @>$3=sum(@I..@-1) -->
<!-- TBLFM: @>$4=sum(@I..@-1) -->

It takes 2 formulas, repeated 3 times each to accomplish both horizontal and vertical total calculation and only gets worse as the table grows in size, forcing repeated formulas to be added and leading to potential error.
Ideally, there would be some function that could iterate so many times similar to a for loop.

The row sums can be merged by setting last column at once. This works because omitting the row from the destination applies the formula to all rows after the header.
| test 1 | test 2 | test 3 | test 4 |
| ------ | ------ | ------ | ------ |
| Banana | 1      | 2      | 3      |
| Orange | 2      |        | 2      |
| Cherry | 2.     |        | 2      |
| total  | 5      | 2      | 7      |
<!-- TBLFM: @>$2=sum(@2..@-1) -->
<!-- TBLFM: @>$3=sum(@2..@-1) -->
<!-- TBLFM: $>=sum($2..$3) -->

I don't think this is possible here for the column sums because the first column is non-integer. Trying to use <!-- TBLFM: @>=sum(@2..@-1) --> parses (silently) but shows an error in the console:

[DecimalError] Invalid argument: banana

What I would like is a way to include ranges in the destination specification. Eg <!-- TBLFM: @>$2..$3=sum(@2..@-1) -->


Advisor details improvements

Please have a look at the individual commit messages for the details.

Merging despite the unrelated YarnFunTest failure.


Search dictionary by other fields besides name

Would like to support searching by:

role
contributor
definition
perspective

@novellac
I just added a search bar on the dictionary page
https://zpl.io/Vkgpgjv


This line only works in .vim files

I'm not sure why this bit of code works in .vim files and not in anything else.
https://github.com/wincent/wincent/blob/4e2e685628b778ae197c9f64c39425d03b09925f/aspects/nvim/files/.config/nvim/init.lua#L142
I took it from one of your videos. So what I have in my set up is:
vim.opt.colorcolumn = '79'
vim.opt_local.colorcolumn = '+' .. vim.fn.join(vim.fn.range(0,254), ',+')

I made a mistake. Instead of setting 'textwidth' to 79 I set 'colorcolumn' to 79. Sorry for the noise.

Yep, those + values are relative to the textwidth.


Error message on some DOB dates should be different

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open staging dot
Go connect bank account -> Personal information 3 / 5 page
Type in the date of Birth as “04/21/1873” .
Notice that the error message is “Must be over 18 years old”
Notice that the DOB shows user is above 18 years old. Thus the error message must be different.

Expected Result:
The error text should be something different that corresponds to the user input
Actual Result:
The above-mentioned error text doesn’t truly indicate the invalid DOB that the user has typed.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number:
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43996225/234352438-411c58fc-4774-42d8-9fae-23eaf693b6ec.mov
Expensify/Expensify Issue URL:
Issue reported by: @ashimsharma10
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1682349870593299
View all open jobs on GitHub

Triggered auto assignment to @sonialiap (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Seems so.

Reviewing label has been removed, please complete the "BugZero Checklist".

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.3.7-3 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/17474

If no regressions arise, payment will be issued on 2023-05-05. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] [] The PR that introduced the bug has been identified. Link to the PR:
[ ] [] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] [] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [] Determine if we should create a regression test for this bug.
[ ] [] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
[ ] [@sonialiap] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

Working through checklist

Reviewed the issue and looks like this was a dupe. Closing out


[wip]test cluster glossary definition

testing tutorial cluster basic page

Page preview:
https://deploy-preview-29712--kubernetes-io-main-staging.netlify.app/docs/tutorials/kubernetes-basics/create-cluster/

/close


i2c-nonblocking: how to use?

What is the state of non-blocking i2c, can it be used? I tried
cargo add stm32g0xx-hal --no-default-features --features stm32g071cb,rt,i2c-nonblocking
but compilation fails with could not find blocking in i2c in prelude.rs.

Non-blocking things are under i2c::nonblocking https://github.com/stm32-rs/stm32g0xx-hal/blob/096121af08f320a3e115945c176c843e7526eff0/src/i2c/mod.rs#L5
PS: have you looked into embassy. It has the most comprehensive driver support for all STM32 families.


community: Extend Baichuan model with tool support

Description: Expanded the chat model functionality to support tools in the 'baichuan.py' file. Updated module imports and added tool object handling in message conversions. Additional changes include the implementation of tool binding and related unit tests. The alterations offer enhanced model capabilities by enabling interaction with tool-like objects.

@nobbbbby Can you add an example to the doctoring of using a tool? Other than that lgtm

@nobbbbby Can you add an example to the doctoring of using a tool? Other than that lgtm

Hi, I just add one example


engine: rm dead code in podLogManager

Hello @landism, @nicks,
Please review the following commits I made in branch maiamcc/clean-up-pod-log-watch:
c2fc560669dd3211c118f87897947bc926099d90 (2019-03-05 17:52:39 -0500)
engine: rm dead code in podLogManager
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

Rolling back a chunk of changes from #776 -- seems that all that was needed to fix the problem in question was the HardCancelReader


Cannot submit kata

All the tests pass, it just does not let me submit. There is no error message. The kata question doesn't even show up under "unfinished."
THe question I'm working on is "Playing with digits"

Update: it's not just the question, none of the kata I'm trying to submit are being submitted.

And I can't even bookmark questions

Can you post some screenshots? Also please open up your console and refresh the page, and report any errors that may come up. Thanks.


onSlideNextEnd in firefox not executed

No matter which callback i use - onSlideChangeEnd or onSlideNextEnd, when sliding to the right the callback is not executed in Firefox. However, in the opposite direction it works fine. I don't have this problem with Chrome and IE, just in Firefox.

Can't replicate it. Do you have live example or JSFiddle with the issue?

Have absolutely the same issue. If I have time, I will try to prepare an example.

But i still can't replicate it :) Need to see example with the issue

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Handle named wildcards in REST routes

Description

Prevents registering multiple REST routes with the same wildcards which would throw an exception on OpenSearch RestController
Match incoming rest requests to the wildcard key for the dictionary

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

Note to self: looks like I need fnmatch.

I think I am misunderstanding the feature. When would this be used?

Here's one example of many.

It looks like this lets me arbitrarily add named routes by making requests to OpenSearch which seems like a DDoS waiting to happen.

It's no more arbitrary than plugins adding them.  And we're already adding routes, the only difference here is how we are treating routes with a specific synax.
I don't understand the DDOS concern.
Registering /hello/{name} registers exactly one path on OpenSearch's RestController, where internally in the PathTrie the {name} is converted to a * for matching purposes, but also populates the param with key "name" when it does match.


Underscore in method signature meaning

Hello,
what's the meaning of the underscore in some method signatures?
For example _template.
I'm looking for a way to configure this rule of ESLint.
Something like:
"no-underscore-dangle": ["error", {
        "allow": [
          "_template",
          "_setup",
          "_setState"
        ]
}]

There is a list of this methods?
Eventually, this list should be maintained in some way?
Best regards
Mattia

Hey @MattiaPontonioKineton
There is no complete list per se. Underscored methods include:

component hooks like _template _init, _setup, _setState (docs)
keys like _handleRight, _handleUp (docs)
signals, which can be completely user defined (docs)
focus hooks (docs)
as well as hooks that that are part of SDK

As you can see, there seem to be no better way forward than disabling the rule you mentioned altogether.
Best regards


Can you help with this error?

When I run this command line:
$ sudo /usr/bin/java -jar -Dserver.port=8080 -Dconfig.file=/home/pi/habridge/data/habridge1.config -Dupnp.device.db=/home/pi/data/device1.db /home/pi/habridge/ha-bridge-4.5.6.jar
I get this and the HA Bridge fails:
2017-07-07 02:13:04,849 [main] INFO  com.bwssystems.HABridge.HABridge - HA Bridge (v4.5.6) starting....
2017-07-07 02:13:04,960 [main] INFO  com.bwssystems.HABridge.BridgeSettings - reading from config file: /home/pi/habridge/data/habridge1.config
Exception in thread "main" java.lang.NullPointerException
at com.bwssystems.HABridge.BridgeSettings.buildSettings(BridgeSettings.java:150)
at com.bwssystems.HABridge.HABridge.main(HABridge.java:51)

Does this issue still occur? Seems I took too long to repsond on this and you have other questions as you have it running currently.

I believe I should have closed this issue. Things are resolved.
Sent from my iPhone

On Aug 29, 2017, at 1:47 PM, BWS Systems<EMAIL_ADDRESS>wrote:
Does this issue still occur? Seems I took too long to repsond on this and you have other questions as you have it running currently.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[dev]  ./genome/genome.fa link not created in output dir

Hello,
I have run:
nextflow run . -profile test,singularity --outdir ../out_test_nf_chipseq

While loading IGV session session:
cd ../out_test_nf_chipseq/igv/broadPeak/
igv.sh
#Loading:
igv_session.xml

IGV reported:
Error loading session session: Cannot locate genome: /full/path/to/out_test_nf_chipseq/genome/genome.fa

I have found ( in one of the subdir in ./work) and soft-linked the genome.fa to out_test_nf_chipseq/genome/genome.fa
after which IGV displayed the results as expected.
This is a fairly minor nuisance, but IMHO it should be addressed.
Many thanks for creating and maintaining such complex pipeline.
Darek Kedra

Thanks for reporting @darked89, we should address it before releasing 2.0!

Fixed in #279. I close this issue now but let us know if you find any other issue @darked89 Thanks!


Derive Clone, Copy for Event.

Building glue fails with rustc dev 2015-04-06:
glue/src/lib.rs:115:1: 115:23 error: the trait core::clone::Clone is not implemented for the type Event [E0277]
glue/src/lib.rs:115 impl Copy for Event {}

Thanks


(Build) Fail on dev branch.

This issue is created automatically by CI.
Linux Build State: failure
Windows Build State: failure
Commit: 6c51a69ba88270129dbb77395ca67a3c43202c6f
Commit Branch: dev
Commit Author: innerviewer

Auto-closing as build in this branch is successful now. Commit: d51f9b5be774adf93d3096cdb824093eed135a68


Fix gcs storage driver

Based on https://github.com/distribution/distribution/pull/3702.
Also fixes the OSS Cancel method which was missing a context parameter.

Who can help with the 2nd approval @milosgajdos ?

CC: @distribution/maintainers

bumping this as it's still pending a review.

great to see this getting merged! thanks to all for driving this effort 👏🏼

Big thanks go to @jmontleon for the initial effort and @flavianmissi for getting it across the finish line 🙇


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1717

Questions ?
Please refer to the Backport tool documentation


How can I put a Scanner inside a struct, are there any examples to learn from?

During the development of a detection engine based on yara-x in Rust, I encountered a lifetime issue because the scanner depends on a reference to rules, but I want to put the scanner inside a struct. How should this be implemented? Are there any corresponding examples to learn from?

I am not sure where you're encountering the lifetime issues without seeing the code, but here is an example that places the Scanner into a struct with named lifetime parameters.
use yara_x::Scanner;

struct NestedScanner<'a> {
    inner_scanner: yara_x::Scanner<'a>,
}

fn test_function(scanner_struct: &mut NestedScanner) {
    let result = scanner_struct.inner_scanner.scan(b"test");
    println!("{} rules matched", result.unwrap().matching_rules().len());
}

fn test_function_two(scanner_struct: &mut NestedScanner) {
    let result = scanner_struct.inner_scanner.scan(b"test");
    println!("{} rules not matched", result.unwrap().non_matching_rules().len());
}

fn main() {
    let mut compiler = yara_x::Compiler::new();

    compiler
        .add_source(
            r#"
        rule always_true {
            condition: true
        }"#,
        )
        .expect("Rule should've added without issue")
        .add_source(
            r#"
        rule always_false {
            condition: false
        }"#,
        )
        .expect("Rule should've added without issue");

    let rules = compiler.build();

    let scanner = Scanner::new(&rules);

    let mut ns = NestedScanner {
        inner_scanner: scanner,
    };

    test_function(&mut ns);
    test_function_two(&mut ns);
}

@xvsfekcn, you may find lifetime issues if you are trying to put both the rules and the scanner inside the same structure, because the scanner references the rules. That's a self-referential struct, and they are problematic in Rust. If that's your problem, you may find more information if you search for "rust self-referential structs" in Google.

I'm closing the issue because it doesn't look like a real issue, but feel to provide more information about your use case here and I'll try to help you as much as I can.


Expose the SemVer package publicly

I’ve worked on more than one project that needed to programmatically configure environments using versioned packages, or simply log which versions are being used for future auditability. Tasks like parsing and comparing SemVer versions were pretty common.
The implementation used for the Go11 modules cmd/go/internal/semver seems stable and complete enough to be made public.
I suggest we move the semver package one level up so it’s not longer under internal.

Dup of #28101.


SSDT-SLEEP-PTXH.aml

What is the purpose of SSDT-SLEEP-PTXH.aml
It seems that I have some strange issues with USB on Ventura 13.5.1. My network keeps disconnecting on builtin eth and USB adapter. Wifi works fine.

https://dortania.github.io/Getting-Started-With-ACPI/ssdt-methods/ssdt-easy.html#so-what-can-t-ssdttime-do


Support sort column as hexadecimal?

This plug-in looks really cool. Can the sort function (:Sort) by the hexadecimal value of a column? Much like :sort does.

Thanks for the feedback. I pushed a commit. Please check.


Automating generate Mobility Service configuration file

Hello,
Could you share documentation or script to automate generate Mobility Service configuration file ?
In the browser asks for page: /Home/CreateMobilityAgentConfigFile
in response receives json config file.
Attached simple base64 has information about biosID and hostname of virtual machine in json format.
The response has the same information plus a certificate and etc.

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

ID: d179f813-5826-7ec3-30c5-9b6b9c0a9acc
Version Independent ID: 3141a3e5-ea3b-cba1-7e34-4df69b8e1aa3
Content: About the Mobility service for disaster recovery of VMware VMs and physical servers with Azure Site Recovery - Azure Site Recovery
Content Source: articles/site-recovery/vmware-physical-mobility-service-overview.md
Service: site-recovery
GitHub Login: @ankitaduttaMSFT
Microsoft Alias: ankitadutta

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

@aszykm Does this help? - https://learn.microsoft.com/en-us/azure/site-recovery/vmware-azure-mobility-install-configuration-mgr

@SadiqhAhmed-MSFT It is for Classic, I am looking for Modernized.

@aszykm Check the documentation section here - https://learn.microsoft.com/azure/site-recovery/vmware-physical-mobility-service-overview#generate-mobility-service-configuration-file
Below is the set of commands which also can be used to generate the file:

Get the source config input from source machine after installing mobility agent. (need admin/root user login)
a.	Windows: \agent\AzureRcmCli.exe --getagentconfiginput
b.	Linux: /Vx/bin/ AzureRcmCli --getagentconfiginput
c.	Check for below in output:
getagentconfiginput:
Generate source config file on appliance (need admin user login)
a.	C:\Program Files\Microsoft Azure RCM Proxy Agent\RcmProxyAgent.exe -createmobilityagentconfigfile -encodedinput "" -outputfilepath "C:\Users\Administrator\Desktop\conffile.json" -clientrequestid  -activityid
Complete agent configuration by copying the conffile.json to source VM (need admin/root user login)
a.	Linux: "/Vx/bin/UnifiedAgentConfigurator.sh" -q -S “/configfile.json” -c CSPrime
b.	Windows: "\agent\UnifiedAgentConfigurator.exe" /SourceConfigFilePath “\configfile.json /CSType CSPrime”

@SadiqhAhmed-MSFT Thanks for the answer.
I want to run the second point, but I got a message about missing input in clientrequestid and activityid.
When I removed these arguments, I got feedback about missing arguments.
I understand, client request id is required, but what I should set ?

@aszykm The clientrequestid and activityid can be same or unique. Keeping them unique help triage any issues. You should use new guids in both the places. You can generate it online or using below PowerShell command.

@aszykm We will now close this issue. Please tag me in your reply if you have any questions.

@SadiqhAhmed-MSFT : I tried the steps mentioned to generate the config file. I am getting the error:
C:\Program Files\Microsoft Azure RCM Proxy Agent>".\RcmProxyAgent.exe" -createmobilityagentconfigfile -encodedinput "----------------------" -outputfilepath ".\configfile.json" -clientrequestid "afd3c3a4-0518-4a3d-a29f-f43939603bda" -activityid "2ab97790-4ee5-4e19-b3d2-034294f22286"
Exception: System.ArgumentNullException: Value cannot be null.
Parameter name: findValue
at System.Security.Cryptography.X509Certificates.X509Certificate2Collection.FindCertInStore(SafeCertStoreHandle safeSourceStoreHandle, X509FindType findType, Object findValue, Boolean validOnly)
at System.Security.Cryptography.X509Certificates.X509Certificate2Collection.Find(X509FindType findType, Object findValue, Boolean validOnly)
at RcmProxyAgent.CertificateUtilities.CertStoreHelper.FindCertificateByThumbprint(String certThumbprint, StoreName store, StoreLocation location, Boolean mustExist)
at RcmProxyAgent.Config.Configurator.CreateMobilityAgentConfig(CreateMobilityAgentConfigInput createMobilityAgentConfigInput)
at RcmProxyAgent.Main.Program.Main(String[] args).
Exception: System.ArgumentNullException: Value cannot be null.
Parameter name: findValue
at System.Security.Cryptography.X509Certificates.X509Certificate2Collection.FindCertInStore(SafeCertStoreHandle safeSourceStoreHandle, X509FindType findType, Object findValue, Boolean validOnly)
at System.Security.Cryptography.X509Certificates.X509Certificate2Collection.Find(X509FindType findType, Object findValue, Boolean validOnly)
at RcmProxyAgent.CertificateUtilities.CertStoreHelper.FindCertificateByThumbprint(String certThumbprint, StoreName store, StoreLocation location, Boolean mustExist)
at RcmProxyAgent.Config.Configurator.CreateMobilityAgentConfig(CreateMobilityAgentConfigInput createMobilityAgentConfigInput)
at RcmProxyAgent.Main.Program.Main(String[] args).
Operation '-createmobilityagentconfigfile -encodedinput --------------------- -outputfilepath .\configfile.json -clientrequestid afd3c3a4-0518-4a3d-a29f-f43939603bda -activityid 2ab97790-4ee5-4e19-b3d2-034294f22286' failed with error
'{
"ErrorCode": "UnhandledException",
"Message": "The operation failed due to an internal error.",
"PossibleCauses": "The operation failed due to an internal error.",
"RecommendedAction": "Retry the operation. If the issue persists, contact support.",
"ErrorSeverity": "Error",
"ErrorCategory": "SystemError",
"IsRetryableError": false,
"MessageParameters": {},
"IsRcmReportedError": false
}'.
{
"ErrorCode": "UnhandledException",
"Message": "The operation failed due to an internal error.",
"PossibleCauses": "The operation failed due to an internal error.",
"RecommendedAction": "Retry the operation. If the issue persists, contact support.",
"ErrorSeverity": "Error",
"ErrorCategory": "SystemError",
"IsRetryableError": false,
"MessageParameters": {},
"IsRcmReportedError": false
}

@aszykm As it states in the error description, you could retry the operation after some time. It may be an intermittent issue, if it fails on retry, then it is best to contact Azure technical support team for further assistance.


Buffers can be deleted while still attached to a source

I noticed a behavioural difference between mojoal and macos's openal, and I think it suggests a bug.
Basically you can delete buffers while they are still attached to a source. In mojoal, you don't get an error on delete and when you attach new buffers to the source to play a new sound, you get a brief blip of the previously buffered audio.
In comparison to macos's openal: when you attempt to delete a buffer that is still attached, you will get an "Invalid Operation" error.
Attached is source for testing this bug. Create two different raw audio files called tone_880.raw and tone_440.raw (different so you can hear the previous buffer play a bit), and comment/uncomment out #define AL_CLEAR_BUFFER_ON_STOP to trigger the bug.
openal_test.cpp.tar.gz
Tested using mojoal 8af8cd366257e3b9d734f635f7d17d67fc4d48f7

@sonneveld I think it's a different case, but from this PR here they also appear to have solved for their use a similar blip case: https://github.com/gosu/gosu/pull/619

So this wasn't the buffer or source state being incorrect (the test app was correctly unqueueing buffers before deleting them), it's that the internal buffer the source uses for resampling wasn't being cleared during alSourceStop(), so when you restart it, it would still have a little data from the last buffer sitting in there, resampled and ready to be played, before it would start using data from the new buffer.
alSourcei(...AL_BUFFER...) happens to forcibly destroy and recreate the SDL_AudioStream it uses for resampling, which is why that would work around the issue.
This is fixed now.


Exception on quit

My Setup

Operating System: macOS 10.13.3 (17D25b)
App Version: 2.10.1

[x] I have tested with the latest version
[ ] I can simulate the issue easily

Description
Quit Rocket Chat.
Current Behavior
Exception is thrown.
Uncaught Exception: Error: Object has been destroyed     at Object.saveState (/Applications/Rocket.Chat+.app/Contents/Resources/app.asar/app/background.js:241:18)     at invokeFunc (/Applications/Rocket.Chat+.app/Contents/Resources/app.asar/node_modules/lodash/lodash.js:10350:23)     at trailingEdge (/Applications/Rocket.Chat+.app/Contents/Resources/app.asar/node_modules/lodash/lodash.js:10397:18)     at Timeout.timerExpired [as _onTimeout] (/Applications/Rocket.Chat+.app/Contents/Resources/app.asar/node_modules/lodash/lodash.js:10385:18)     at ontimeout (timers.js:469:11)     at tryOnTimeout (timers.js:304:5)     at Timer.listOnTimeout (timers.js:264:5)
Expected Behavior
No exception.

This is old.


Clear separation of concerns between Binary Proxy and Court

These two are overlapping.
Binary Proxy

Creating a dispute
View on court
Submit evidence
No way to view primary document

Court

View primary document
No way to submit evidence

A suggested way:
Binary Proxy - dispute creation only - all the rest in the court
Court - add the ability to submit evidence.

The suggested way is not possible to implement because evidence submission functionality is on an arbitrable contract. But the Court is an arbitrator.


Added all the experiments documentation

Added documentation(.md as well as .html files), schematics, screen shots(of graphs) for all the experiments which are available in the PSLab android app.

@mariobehling Please review this pr and let me know if this is in accordance of what is expected.


[snapshot_download] upgrade to canonical separator

ping @severo

Think it will however force all repos to be downloaded & cached again

yes but we already did that recently no? :)
but yeah i feel like now we should be at the final format so shouldn't change after that

pinging @severo for a final review then let's merge!


FIX: Vet for Supervisor Tests

Current coverage is 55.50% (diff: 100%)

Merging #75 into master will not change coverage

@@             master        #75   diff @@
==========================================
  Files            19         19
  Lines          1825       1825
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1013       1013
  Misses          711        711
  Partials        101        101

Powered by Codecov. Last update 68fa14f...4f2e6d8


Adding ROV example mesh

Should be able to run gazebo...
roslaunch gazebo_ros empty_world.launch

and then use the GUI to insert the erov model.
Image of what you should see in Gazebo.

And here is what the collision looks like.

The single box collision may cause issues when adding thruster, manipulators and tools, but we can address that later.

Nice.  I'm wondering, would some square edges and a couple of bolts be more representative of a typical view?

Hmm, I'm not sure why it smoothes the edges like that. I'll have to look at that.

Added Ryan's new mesh.

@BruceMty Would you have time to review this PR?

This loads fine in Gazebo.  @bsb808 , do edges need to be right-angled for a cleaner look where arms and sensors are mounted?  Also, regarding my previous comment, I don't mind rounding, I'm wondering if it might be nice to add some "machine shop" look to it?

@BruceMty Yes, it is just a first cut.  We'll need to refine this and other vehicle models.
Please approve when you are ready.

This loads fine in Gazebo. @bsb808 , do edges need to be right-angled for a cleaner look where arms and sensors are mounted? Also, regarding my previous comment, I don't mind rounding, I'm wondering if it might be nice to add some "machine shop" look to it?


Application will not open

Hi, I've recently downloaded Eqmac2 however the application will not startup.  I've tried restarting my computer as well, and have tried to delete it but it says that the application is open.

Not a proper bug report. GitHub issues are not for customer support

Hey everyone. Excuse me for the lack of updates. I have been really busy on the new version of eqMac. I am happy to announce an open beta release.
You can download it here: https://eqmac.app
There are a lot of known bugs that I could not fix just yet:

Driver not installing from the first try. Sometimes need to restart eqMac a couple times.
Complex Device switching logic is not perfect (for example when switching off the device that's currently playing, plugging in new device etc.)
UI takes long to load (a lot can be done to improve that)
UI dimensions are not synching properly
There's a noticeable audio delay in the playback, so not suitable for video yet (I know where the problem lies just haven't had the time to fix yet)
Sometimes the playback doesn't start, in that case please try to switch the output to the device you want and eqMac should try and build the audio pipeline again.

So as you can see there's still some work to be done but at least it's a completely modern tech stack that I am fully willing to work on: 100% Swift, User Space driver and a Web UI, instead of C++, Kernel Drivers. As I said, should work on Catalina and should not have any pesky ear drum blasting volume bugs (at least I didn't come across any).
I'm open to feedback and issue reports here: https://github.com/bitgapp/eqmac
That repo will become the main repo for eqMac and this repo will be archived eventually.
And yes it's called just eqMac, not eqMac3 :) And it will stay that way forever.

Hey everyone 👋
I will be open sourcing the new version of eqMac, therefore, I am closing this issue and repository as I will be transfering nodeful/eqMac2 repo to bitgapp/eqMac
Please bear with me for the next couple of days as I setup that new repo. The eqMac2 code will be living in the old-eqMac2 branch if anyone is interested in referencing it for whatever reason.
Please try out the new version and report bugs to the new repo.
Thanks for using eqMac.


Openldap: Creation

WIP
For work tracking
Please dont test it ;)
Todo:

Add more instruction
Add reference
Make test

@k8s-bot ok to test

@ekozan is this still an active/WIP PR? Do you have an ETA for completion?

Closing due to inactivity. Feel free to re-open if/when ready.


tests skip sub directories and their .f90 files

Fortitude 0.5.0 (installed via by PyPI, update by yesterday November 20th) in Debian 13/trixie does not (yet) consider .f90 files in sub directories. It would be nice if one could check equally these in one run.
I have three test files r.f90, s.f90, and t.f90 which all include one real r (or s, t) -- file name reflects this variation -- of implicit kind, e.g.
program test
   implicit none
   real :: r

   print *, "hello"
end program test

If they all share the same level of hierarchy, all three are recognized, tested, and reported
$ fortitude check *.f90
r.f90:3:4: P021 real has implicit kind
  |
1 | program test
2 |    implicit none
3 |    real :: r
  |    ^^^^ P021
4 |
5 |    print *, "hello"
  |

s.f90:3:4: P021 real has implicit kind
  |
1 | program test
2 |    implicit none
3 |    real :: s
  |    ^^^^ P021
4 |
5 |    print *, "hello"
  |

t.f90:3:4: P021 real has implicit kind
  |
1 | program test
2 |    implicit none
3 |    real :: t
  |    ^^^^ P021
4 |
5 |    print *, "hello"
  |

fortitude: 3 files scanned.
Number of errors: 3

For more information about specific rules, run:

    fortitude explain X001,Y002,...

$

If I move one of them into a sub directory, the file no longer is checked:
$ mkdir sub
$ mv t.f90 ./sub/
$ ls
r.f90  s.f90  sub
$ fortitude check *.f90
r.f90:3:4: P021 real has implicit kind
  |
1 | program test
2 |    implicit none
3 |    real :: r
  |    ^^^^ P021
4 |
5 |    print *, "hello"
  |

s.f90:3:4: P021 real has implicit kind
  |
1 | program test
2 |    implicit none
3 |    real :: s
  |    ^^^^ P021
4 |
5 |    print *, "hello"
  |

fortitude: 2 files scanned.
Number of errors: 2

For more information about specific rules, run:

    fortitude explain X001,Y002,...

$ fortitude --version
fortitude 0.5.0
$

The setup of observation: Linux Debian 13/trixie, fortitude 0.5.0 (from PyPI with pipx 1.7.1); GNU bash 5.2.32(1)-release (x86_64-pc-linux-gnu), xfce4-terminal 1.1.3 (Xfce 4.18), Python 3.12.7
subdir_test.zip

@ZedThree I'm able to replicate the recommended better way.  This resolves well the issue.  Thank you.


When User: 2 offline edit the state.Then online edit the state.It cannot be edited.Some error occurred in the console

When User: 2 offline edit the state.Then online edit the state.Some error occurred
https://electric-sql.com/docs/intro/offline#merging-offline-writes
output in the console
Uncaught (in promise) Error: already performing snapshot
    at Pt._performSnapshot (5032.f7887850.js:2:310425)
    at Pt.mutexSnapshot (5032.f7887850.js:2:300604)
    at async f (5032.f7887850.js:2:315602)
    at async Pt._applyTransaction (5032.f7887850.js:2:316597)
    at async te.<anonymous> (5032.f7887850.js:2:333444)

Are you able to reproduce this consistently on the website? If so, could you share with what OS, browser, and verison?

Closing as stale, and website demos have now been updated to a more recent version. If the issue persists it can be reopened or opened as a new one.


Upgrade to league/oauth2-server 5.x

There is a new major version released.
I noticed that work has started on migrating to this new version on the develop branch.
Do you have time table for this? What needs to be done? Can users contribute?
If there was an issue with everything that needs to be done first, others could chip in and help moving this forward.

Hello @felixkiss, here are the answers to your questions:

No time table yet. My daily job keeps me from making plans.
Make it feel like it's part of laravel - make it easy to use it out of the box - make it extensible to other storage types, and in general, make everything swappable by the end user.
Yes.

Closing this since I consider it answered. @felixkiss feel free to comment if you wonder anything else.


Memo encryption doesn't work - shows ** on Bitshares and manual decryption throws error

I made a transfer using following code in python 3.6.4 / bitshares==0.1.11:
from bitshares import BitShares
bitshares = BitShares(keys={pub_key: priv_key})
bitshares.transfer('abc', 0.01, 'ABC', 'test-memo', 'def')

And it shows two asterisks as memo on BitShares dex no matter what I write as memo text:

When I try manually encrypting and decrypting memo, I got an error while decrypting:
from bitshares import BitShares
from bitshares.memo import Memo

bitshares = BitShares(keys={pub_key: priv_key})

memoObj = Memo(
    from_account=from_account,
    to_account=to_account,
    bitshares_instance=bitshares
)
enc = memoObj.encrypt('test')
memoObj.decrypt(enc)

Error:
Traceback (most recent call last):
  File "/Users/darklow/www/abc/env/lib/python3.6/site-packages/bitsharesbase/memo.py", line 116, in decode_memo
    return _unpad(message.decode('utf8'), 16)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xd5 in position 4: invalid continuation byte

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "./manage.py", line 15, in <module>
    execute_from_command_line(sys.argv)
  File "/Users/darklow/www/abc/env/lib/python3.6/site-packages/django/core/management/__init__.py", line 371, in execute_from_command_line
    utility.execute()
  File "/Users/darklow/www/abc/env/lib/python3.6/site-packages/django/core/management/__init__.py", line 365, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/Users/darklow/www/abc/env/lib/python3.6/site-packages/django/core/management/base.py", line 288, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/Users/darklow/www/abc/env/lib/python3.6/site-packages/django/core/management/base.py", line 335, in execute
    output = self.handle(*args, **options)
  File "/Users/darklow/www/abc/abc-swap/abcswap/management/commands/send_to_bitshares.py", line 13, in handle
    bitshares_sender.test(options['pub_key'], options['priv_key'])
  File "/Users/darklow/www/abc/abc-swap/abcswap/lib/bitshares_sender.py", line 36, in test
    memoObj.decrypt(enc)
  File "/Users/darklow/www/abc/env/lib/python3.6/site-packages/bitshares/memo.py", line 133, in decrypt
    memo.get("message")
  File "/Users/darklow/www/abc/env/lib/python3.6/site-packages/bitsharesbase/memo.py", line 118, in decode_memo
    raise ValueError(message)
ValueError: b'i\x1bf8\xd5A\xd2^X\xaf\xe0v'

This may well be an issue with the accounts.
If they have a memo key that is different from the active key, then you need to add both, the memo and the active key to your BitShares() instance.
Note that you can use BitShares(keys=[wif1, wif2, wi3...]) and don't need to provide pubkey (the lib should know what to do and when to use which one).
Also note that the library can technically encode anything using any key and does not check if the webwallet will be able to decode it.
Can you tell us which account names you are working with?


Command "key:generate"

Description
Added new command for encryption key generation to .env file.
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPdocs
[x] Unit testing, with >80% coverage
[x] Conforms to style guide

Tests are failing. 😢

How about making one of the methods (hex or base64) a default one? There is no much difference and it would be just easier for developers to type key:generate.
I would add support for the key length (with default value).
I'm also wondering should we support the case when there is no env file at all (even this default one shipped with the framework). Maybe it's a good idea to mention in the description that we will place a key in the env file.

Ok. I will do the default encoding and description update. For the support for key length, I don't pretty much understood what you mean. Can you explain a bit more?

The length of the encryption key should vary depending on the encryption algorithm we use.
https://codeigniter.com/user_guide/libraries/encryption.html#setting-your-encryption-key

What do you think about checking if env file actually exists before trying to making a copy?
Also, the last thing - maybe we should replace hex with hext2bin - since you made it a default method it won't be a big problem to type it and we would keep the names consistency.

If the default shipped env file is not actually existing before copy, what should be our course of action then? Should we throw an exception, or just output the $newKey and an error message to STDERR?

Yes, probably just a nice error message that says the env file is not available and then we should print the key on the screen.

This is really nice feature. Thank you!


Fix broken link

Link to peer topic is broken.

@bermudezmt - Can you PTAL and merge? Thank you!

Deploy preview for docsdocker ready!
Built with commit bd9acc0c27569dbceada8956d2512c2f045bfb00
https://deploy-preview-8801--docsdocker.netlify.com


Setup completed for tailwind

@tiagostutz I've some concerns regarding this pull request

main.css is a huge file so when I'm pushing the file to the repo it's taking a lot of time (i had to cancel the push operation) to be formatted via prettier, we need to find a way to ignore that file as it is a huge file
I have added it to the .gitignore so that it doesn't get pushed on to the repo because it's going to be outputted when we run dev server or build the app.

@tiagostutz can you merge the changes if the review is completed?

@tiagostutz can you merge the changes if the review is completed?

Hi @giteshk11 !
Reviewing it right now! =)

@giteshk11 just reviewed the PR and this 15MB css file is really a problem we will have to deal with. Not only because of the prettier thing but because of the project building/bundling. I tried to build it here and it took an insane time to finish, althought the final bundle was 1MB length, which is not the best scenario, but acceptable.
I'm doing some research here about how we could address this problem. If you find a way, please comment here. I'll do the same.
🏃🏻♂

I found this  https://tailwindcss.com/docs/controlling-file-size/ in the official tailwindcss i think i tried to use it but not sure if its working or not

Found one more link

I found this https://tailwindcss.com/docs/controlling-file-size/ in the official tailwindcss i think i tried to use it but not sure if its working or not
i think we should config prettier in such a way that it should ignore the main.css file as its outputted by build process.

I was reading it and looks like you followed the steps correctly. I checked here the build folder and I was wrong: the CSS is 15MB, 1MB is the file gzipped. So, it looks like the purge thing is not working or something like that.
I'm doing some more research here

Got this! I'm making the changes and commiting to your branch.

Reference: https://dev.to/jmhungdev/purging-tailwindcss-without-ejecting-create-react-app-4mef

I think we are good to go!
@giteshk11 take a look at changes I've made and if you agree with them we can merge this PR!

Looks good to me!
You can merge it @tiagostutz


Add instructions for clearing the cache of a Buildkite worker

Yeah, I think that’s fine
On Fri, Aug 20, 2021 at 19:59 Dilum Aluthge @.***>
wrote:

@.**** commented on this pull request.
In misc.md
https://github.com/JuliaCI/base-buildkite-docs/pull/9#discussion_r693293151
:

+ssh @.***
+```

+Step 3: Based on the worker, SSH into the appropriate CI machine. In this example, we need to SSH into amdci5:
+
+ +ssh<EMAIL_ADDRESS>+
+
+Step 4: Switch to the appropriate user. You will need to enter your sudo password.
+
+ +sudo su sabae +
+
+Step 5: Go to the scratchspaces directory and look for the scratchspace for the worker in question:
Should we just hardcode a66863c6-20e8-4ff4-8a62-49f30b1f605e in the
instructions then?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/JuliaCI/base-buildkite-docs/pull/9#discussion_r693293151,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA762DX6FGCV7FOUVSZDQLT54JAHANCNFSM5CPXXVEQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


how to compress the html code in vue file?

hello, how to compress the html code in vue file? should i change the configure of webpack.config.js?thank you~

Compression means gzip, rar, zip etc. Why compression needed also how is related with this template?
If you asking minify, when you add --release to the command, it's automatically adds uglify js, so your javascript files become uglified ( uglify = minify + naming obfuscate ). Also everytime it minifies generated html file.

cordova run browser -- --live-reload --release

i try to use this code to test my project

as you can see, when i use that code to compress, it seems to not work. but i can use the HtmlWebpackPlugin to compress html code, then these five div can display inline-block.

thanks for your help, have a greate day~

edit like that:
.actionBox {
  display: block;
  box-sizing: border-box !important;
}

.action {
  box-sizing: border-box !important;
}

very helpful! but could you teach me to change the configure of webpack.config.js?

well , i got the answer:
            rules: [{                 test: /\.css$/,                 use: [{                     loader: "style-loader"                 }, {                     loader: "css-loader"                 }]             }, {                 test: /\.useable\.css$/,                 use: [{                     loader: "style-loader/useable"                 }, {                     loader: "css-loader"                 }]             }, {                 test: /\.js?$/,                 loader: 'source-map-loader',                 enforce: 'pre'             }, {                 test: /\.(png|jpe?g|gif)$/,                 loader: 'file-loader',                 options: {                     name: '[name].[ext]?[hash]'                 }             }, {                 test: /\.(woff2?|eot|ttf|otf)(\?.*)?$/,                 loader: 'file-loader',                 options: {                     name: '[name].[ext]?[hash]'                 }             }, {                 test: /\.svg$/,                 loader: 'url-loader'             }, {                 test: /\.vue$/,                 loader: 'vue-loader',                 options: {                     preserveWhitespace: false                 }             }]         },

well , i got the answer:
            rules: [{
                test: /\.css$/,
                use: [{
                    loader: "style-loader"
                }, {
                    loader: "css-loader"
                }]
            }, {
                test: /\.useable\.css$/,
                use: [{
                    loader: "style-loader/useable"
                }, {
                    loader: "css-loader"
                }]
            }, {
                test: /\.js?$/,
                loader: 'source-map-loader',
                enforce: 'pre'
            }, {
                test: /\.(png|jpe?g|gif)$/,
                loader: 'file-loader',
                options: {
                    name: '[name].[ext]?[hash]'
                }
            }, {
                test: /\.(woff2?|eot|ttf|otf)(\?.*)?$/,
                loader: 'file-loader',
                options: {
                    name: '[name].[ext]?[hash]'
                }
            }, {
                test: /\.svg$/,
                loader: 'url-loader'
            }, {
                test: /\.vue$/,
                loader: 'vue-loader',
                options: {
                    preserveWhitespace: false
                }
            }]
        },

use preserveWhitespace: false in vue-loader options

use preserveWhitespace: false in vue-loader options


Ideas for Discussion Section and reorganize by scientific impact, not ease of implementation

We currently order the presentation of the section 7 by "ease of implementation" (sort of, it's hard to judge what is easy sometimes...)
We should probably focus the discussion on science outcomes rather than technical hurdles.
So that would mean ordering the presentation of ideas in terms of scientific impact.  For example there's not much mention about joint modeling spectroscopic binaries, or a better spelling out of multi-epoch spectroscopy, etc.

Just reviewed some of the text and think we should go even further: Let's move some of the technical hurdles into an appendix section called "Technical improvements".  Then focus the Discussion section on Science applications (of which there are already many).  We may refer to individual enabling technical improvements to the science case they unlock.  So for example adding support for GP's would unlock better pseudo-continuum modeling for Brown dwarfs.  So the main text will discuss the science of Brown dwarfs, and the appendix would describe how to implement the GPs.
I suspect this structure will flow better for most scientists, and the technically minded folks can zoom to the appendix.

Done!


Clarify low-latency model stride length

first_filter_stride_y = 4 has no effect, since first_filter_height = input_time_size. If cnn-one-fstride4 should be used, then first_filter_stride_x should be 4 instead. However, during my experimentation cnn-one-fstride4 provided significantly worse results than the current TF impl, so I think the impl is fine.
This small fix just clarifies that the model isn't striding in the time dimension.

Can one of the admins verify this patch?

Jenkins, test this please.


Explosion AOE damage incorrect

Warlock is 14m from center of AOE.  11 dice over a 16m explosion.  So Warlock should have taken at most 2 dice of damage. Also I thought the damage dice was supposed to be sorted, highest at center, lowest at edge of template.

Works for individual damage application but doesn't work for the "Apply Damage to ALL" button as that applies as if everyone were at the epicentre of the explosion. Works for non explosion AoE but not for the situation where things need to be applied differently to each target.

Resolved in #1326


Mex generation fails with the 'precision' set to 'single'

Great project, thanks for sharing the code.
Everything works fine with the precision set to 'double' though appears to fail when 'single' is used.
It seems that when setting "real  = 'single' " in the generateMotor example file, passes the tests in the "falcopt.generateCode", but then 'single' is still used internally in the code generation instead of 'float'. Thus, I get errors of the type:
FalcOpt-master\example\generatedCode\Motor_example_FalcOpt.c:673:43: error: unknown type name 'single'
static void solveConstraintSystem_invert1(single* Mi, const single* M) {
^
It looks as if 'o.precision' and 'o.real' would be used interchangeably in the code-generation phase.
Regards,
VS

Thanks VS, for finding this bug. Should be fixed in the most recent version. Let us know if more issues pop up.
Best,
Damian


Updates to License to include both MIT and LPGL-3.0-or-later

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

I've made the changes to the meta.yaml file that should have your suggestions for including both the MIT and LPGL licenses, @jakirkham.

Alright here we go @jakirkham, this is the same error I was talking about. I don't even need to download the artifact to see that the problem is that Python cannot import the package - it cannot even import the packages on the CI itself. Here's where I got stuck last time - I cannot find in the logs why Python cannot import the package if compiled with CUDA 11.

For instance, I don't know if you can view it, but here's a link to one of the logs: https://dev.azure.com/conda-forge/84710dde-1620-425b-80d0-4cf5baca359d/_apis/build/builds/833921/logs/148
You can see on following lines, the unit tests fail because the package cannot be imported into Python:
2023-11-30T17:49:00.4856063Z Traceback (most recent call last):
2023-11-30T17:49:00.4857694Z   File "/home/conda/feedstock_root/build_artifacts/seekr2_openmm_plugin_1701366120978/test_tmp/run_test.py", line 5, in <module>
2023-11-30T17:49:00.4942254Z     import seekr2plugin
2023-11-30T17:49:00.4943544Z ModuleNotFoundError: No module named 'seekr2plugin'

Thanks Lane! 🙏
That's even better

So here's what I'm seeing on CI:
copying build/lib.linux-x86_64-3.10/seekr2plugin.py -> $BUILD_PREFIX/lib/python3.10/site-packages
copying build/lib.linux-x86_64-3.10/_seekr2plugin.cpython-310-x86_64-linux-gnu.so -> $BUILD_PREFIX/lib/python3.10/site-packages

This is interesting because $BUILD_PREFIX is where command line tools are. However $PREFIX is where package contents should be installed. So for some reason they are not getting installed where we would expect

First off, thanks @jakirkham for taking the time to look at this problem and help me figure out how to solve it.
Honestly, I'm quite confused about what exactly is the purpose of each of these sections (build, host, and run) mean in meta.yaml. I think that I just added openmm to all of them, because I always wanted OpenMM to be available for every stage of the conda-forge process. I will say that OpenMM has libraries that the seekr2_openmm_plugin needs in order to build.
I'm willing to test any changes you feel might be helpful for resolving this problem. Another idea might be to remove the OpenMM 7.7 requirement - it's possible that version of OpenMM doesn't like CUDA 11...

Happy to help 🙂
Gotcha. The usual rule of thumb that I use for requirements is

build is for build tools (so compilers, CMake, etc.) anything that needs to be executed during the build, but we don't want to link to
host is for anything the build consumes (headers, libraries, etc.)
run is for anything that should show up when a user installs the package (so Python or any libraries that were linked to)

Hope that is helpful. Please feel free to ask more questions

Ok based on that here would be my suggestions

Let's drop openmm from build
Relax the version constraint on openmm (agree this is a good idea)

Think we can start with 1 and see how that goes. Then follow up on 2
It's possible other things may become clear as we iterate

Wow! Doing step 1 let all the checks pass - let me try again by removing the 7.7 dependency, and then downloading and testing one of the artifacts.

Very good news, @jakirkham, it seems that the builds are working, however I still cannot see any artifacts to download. Should we just proceed with merging this pull request, since the problem we had before seems to be gone?

Thanks Lane! 🙏
Let's try re-rendering for good measure
@conda-forge-admin , please re-render

Let me try...
@conda-forge-admin , please re-render

Ah this is an org fork. Is that right?
Unfortunately there's not a good way to fix this (GitHub limitation)
Usual way to solve this is to re-render locally
Sorry about that 😞

No problem, it was easy to re-render locally.

Thanks Lane! 🙏
Glad to hear that

Noticed on CI there is a conflict regarding numpy
When looking at the recipe there are requirements lines in host and run that have numpy commented:
https://github.com/conda-forge/seekr2_openmm_plugin-feedstock/blob/e4011600566bf78df010dbce85315f9b62bdeb51/recipe/meta.yaml#L53
https://github.com/conda-forge/seekr2_openmm_plugin-feedstock/blob/e4011600566bf78df010dbce85315f9b62bdeb51/recipe/meta.yaml#L65
Though there is a line in requirements/build that has numpy:
https://github.com/conda-forge/seekr2_openmm_plugin-feedstock/blob/e4011600566bf78df010dbce85315f9b62bdeb51/recipe/meta.yaml#L33
Could you please share some more context on this? Is numpy needed for this package? Or are these leftover from when numpy use to be a dependency? Was there some past issue when adding numpy as a dependency?

I do not believe that numpy should be a dependency, unless possibly when SWIG uses arrays. I think I was just including it because when I build OpenMM from source, it does require numpy.
Let me remove this dependency and try again...

Thanks Lane! 🙏
This looks pretty good. Thanks again for iterating on this with me 🙂
If you are happy with the changes here and CI passes, think we can bump the build/number and merge

Added a couple footnotes in the PR description and added a bit to the PR title
Hope that is ok
Also please feel free to change them as you see fit

Also one last thing, here are the artifacts for the latest CI build. Please feel free to take a look if you would like to confirm things look ok. Happy to discuss if anything comes up

Was just noticing there is not currently a bld.bat for the Windows build. Do we want to add that or skip Windows for now? It looks like only Linux packages have been built for to-date. So maybe skipping Windows is reasonable. Mentioning in part as Windows CI fails for this reason

Sounds good, @jakirkham, I will probably skip the Windows builds for now, and everything else except for linux64. I will download these artifacts tomorrow, and if everything looks good, I'll go ahead and merge.

Thanks Lane! 🙏
Makes sense. Was realizing this was turning into unintentional scope creep. So wanted to dial back on that
Happy to revisit the other OSes/architectures in a separate issue/PR after (when you are ready). Though no need to if you would prefer not to
This already seems like a significant body of work for one PR. Thanks again for working on it with me 🙂

Thanks Lane! 🙏
It can take some time after packages are built for them to mirror to the CDN that conda-forge uses, which in turn can delay getting the latest package as a user
FWIW I did just try creating a fresh environment and it seemed to work ok

$ conda create --dry-run -n tst seekr2_openmm_plugin
Channels:
 - conda-forge
Platform: linux-64
Collecting package metadata (repodata.json): done
Solving environment: done

==> WARNING: A newer version of conda exists. <==
    current version: 23.10.0
    latest version: 23.11.0

Please update conda by running

    $ conda update -n base -c conda-forge conda

## Package Plan ##

  environment location: /opt/conda/envs/tst

  added / updated specs:
    - seekr2_openmm_plugin

The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    _libgcc_mutex-0.1          |      conda_forge           3 KB  conda-forge
    _openmp_mutex-4.5          |            2_gnu          23 KB  conda-forge
    cudatoolkit-11.8.0         |      h4ba93d1_12       682.8 MB  conda-forge
    fftw-3.3.10                |nompi_hc118613_108         1.9 MB  conda-forge
    libblas-3.9.0              |20_linux64_openblas          14 KB  conda-forge
    libcblas-3.9.0             |20_linux64_openblas          14 KB  conda-forge
    libgfortran-ng-13.2.0      |       h69a702a_3          23 KB  conda-forge
    libgfortran5-13.2.0        |       ha4646dd_3         1.4 MB  conda-forge
    liblapack-3.9.0            |20_linux64_openblas          14 KB  conda-forge
    libopenblas-0.3.25         |pthreads_h413a1c8_0         5.3 MB  conda-forge
    numpy-1.26.2               |  py311h64a7726_0         7.7 MB  conda-forge
    ocl-icd-2.3.1              |       h7f98852_0         119 KB  conda-forge
    ocl-icd-system-1.0.0       |                1           4 KB  conda-forge
    openmm-8.1.0               |  py311h6d2dbb8_1        11.3 MB  conda-forge
    pcre-8.45                  |       h9c3ff4c_0         253 KB  conda-forge
    pthread-stubs-0.4          |    h36c2ea0_1001           5 KB  conda-forge
    python-3.11.6              |hab00c5b_0_cpython        29.3 MB  conda-forge
    python_abi-3.11            |          4_cp311           6 KB  conda-forge
    scipy-1.11.4               |  py311h64a7726_0        15.3 MB  conda-forge
    seekr2_openmm_plugin-0.1.6 |  py311h6818ba8_2         169 KB  conda-forge
    swig-4.0.2                 |       hd3c618e_2         1.2 MB  conda-forge
    ------------------------------------------------------------
                                           Total:       756.9 MB

The following NEW packages will be INSTALLED:

  _libgcc_mutex      conda-forge/linux-64::_libgcc_mutex-0.1-conda_forge
  _openmp_mutex      conda-forge/linux-64::_openmp_mutex-4.5-2_gnu
  bzip2              conda-forge/linux-64::bzip2-1.0.8-hd590300_5
  ca-certificates    conda-forge/linux-64::ca-certificates-2023.11.17-hbcca054_0
  cudatoolkit        conda-forge/linux-64::cudatoolkit-11.8.0-h4ba93d1_12
  fftw               conda-forge/linux-64::fftw-3.3.10-nompi_hc118613_108
  ld_impl_linux-64   conda-forge/linux-64::ld_impl_linux-64-2.40-h41732ed_0
  libblas            conda-forge/linux-64::libblas-3.9.0-20_linux64_openblas
  libcblas           conda-forge/linux-64::libcblas-3.9.0-20_linux64_openblas
  libexpat           conda-forge/linux-64::libexpat-2.5.0-hcb278e6_1
  libffi             conda-forge/linux-64::libffi-3.4.2-h7f98852_5
  libgcc-ng          conda-forge/linux-64::libgcc-ng-13.2.0-h807b86a_3
  libgfortran-ng     conda-forge/linux-64::libgfortran-ng-13.2.0-h69a702a_3
  libgfortran5       conda-forge/linux-64::libgfortran5-13.2.0-ha4646dd_3
  libgomp            conda-forge/linux-64::libgomp-13.2.0-h807b86a_3
  liblapack          conda-forge/linux-64::liblapack-3.9.0-20_linux64_openblas
  libnsl             conda-forge/linux-64::libnsl-2.0.1-hd590300_0
  libopenblas        conda-forge/linux-64::libopenblas-0.3.25-pthreads_h413a1c8_0
  libsqlite          conda-forge/linux-64::libsqlite-3.44.2-h2797004_0
  libstdcxx-ng       conda-forge/linux-64::libstdcxx-ng-13.2.0-h7e041cc_3
  libuuid            conda-forge/linux-64::libuuid-2.38.1-h0b41bf4_0
  libzlib            conda-forge/linux-64::libzlib-1.2.13-hd590300_5
  ncurses            conda-forge/linux-64::ncurses-6.4-h59595ed_2
  numpy              conda-forge/linux-64::numpy-1.26.2-py311h64a7726_0
  ocl-icd            conda-forge/linux-64::ocl-icd-2.3.1-h7f98852_0
  ocl-icd-system     conda-forge/linux-64::ocl-icd-system-1.0.0-1
  openmm             conda-forge/linux-64::openmm-8.1.0-py311h6d2dbb8_1
  openssl            conda-forge/linux-64::openssl-3.2.0-hd590300_1
  pcre               conda-forge/linux-64::pcre-8.45-h9c3ff4c_0
  pip                conda-forge/noarch::pip-23.3.1-pyhd8ed1ab_0
  pthread-stubs      conda-forge/linux-64::pthread-stubs-0.4-h36c2ea0_1001
  python             conda-forge/linux-64::python-3.11.6-hab00c5b_0_cpython
  python_abi         conda-forge/linux-64::python_abi-3.11-4_cp311
  readline           conda-forge/linux-64::readline-8.2-h8228510_1
  scipy              conda-forge/linux-64::scipy-1.11.4-py311h64a7726_0
  seekr2_openmm_plu~ conda-forge/linux-64::seekr2_openmm_plugin-0.1.6-py311h6818ba8_2
  setuptools         conda-forge/noarch::setuptools-68.2.2-pyhd8ed1ab_0
  swig               conda-forge/linux-64::swig-4.0.2-hd3c618e_2
  tk                 conda-forge/linux-64::tk-8.6.13-noxft_h4845f30_101
  tzdata             conda-forge/noarch::tzdata-2023c-h71feb2d_0
  wheel              conda-forge/noarch::wheel-0.42.0-pyhd8ed1ab_0
  xz                 conda-forge/linux-64::xz-5.2.6-h166bdaf_0

DryRunExit: Dry run. Exiting.

If you are still seeing this error, could you please fill out this issue template with info were the error was seen? That way we can dig into what is going on a bit more

Hey you're right! I just confirmed that it works - although for some reason it wants to pull the wrong version of openmm (version 8.0) when it installs, but that's OK, I can specify the users to install OpenMM 8.1, manually:
conda install -c conda-forge seekr2_openmm_plugin openmm=8.1

THANK YOU for your help with this! I knew that I'd have to get later versions of CUDA working eventually, and I wasn't sure how I was going to do it. I am very grateful that you came along and assisted me with this, @jakirkham!

That's great news! 🎉
Do the environments where this is being installed already have OpenMM 8.0 for some reason? Asking as the example I ran above grabbed OpenMM 8.1 when creating a fresh environment
Happy to help 🙂

Also this fixed the bot error referenced in issue ( https://github.com/conda-forge/seekr2_openmm_plugin-feedstock/issues/13 ). So the migrator bot has started adding PRs again

https://github.com/conda-forge/seekr2_openmm_plugin-feedstock/pull/16
https://github.com/conda-forge/seekr2_openmm_plugin-feedstock/pull/15

The CUDA 11.8 PR passes and should be mergeable without changes
The CUDA 12.0 PR likely needs changes (is failing currently). Though can follow up on that after CUDA 11.8 is in
Please let me know if you have questions on either of these

Do the environments where this is being installed already have OpenMM 8.0 for some reason?
No, it did not. I'm not sure why it grabbed that version. That's interesting that it didn't grab 8.0 for yours. Let me do some more testing and figure out what might be causing this.
As for the other PRs, sure let me merge the one for CUDA 11.8, and then I'll see if I can figure out how to fix the issues for CUDA 12. It's possible that OpenMM does not yet work on CUDA 12.

Interesting, please let me know what you find out
Sounds good
With CUDA 12, there are some more manual changes as more of the CTK is packaged now, but it is also split into smaller components. For general background please see issue ( https://github.com/conda-forge/conda-forge.github.io/issues/1963 ). Can comment on the PR with relevant details
OpenMM has been migrated ( https://github.com/conda-forge/openmm-feedstock/pull/121 ). The migrator bot won't open a PR on a feedstock until all of its dependencies have been migrated

The migration bot refreshed the CUDA 12 PR (after the CUDA 11.8 PR was merged). Have added some thoughts on next steps in comment: https://github.com/conda-forge/seekr2_openmm_plugin-feedstock/pull/17#issuecomment-1841904745


Adds Option to Copy Panel to Clipboard

Uses Android's ClipboardManager to copy an Panel to the clipboard.

Damb you Build Checks.

Well finally down to one issue and its one I'm definitely not going to attempt as I'll probably break the entire thing xD
/home/runner/work/mihon/mihon/app/src/main/java/eu/kanade/tachiyomi/ui/reader/ReaderViewModel.kt:84:7: Class ReaderViewModel is too large. Consider splitting it into smaller pieces. [LargeClass]

@AntsyLich This is basically done from my end, just let me know if you need me to remove the UI or if there is anything you need me to do

Rebase and fix build thanks


Cannot use tools in Sandbox

I cannot equip the tools in sandbox. I click on for example, wall buys, and it doesn't do anything except show me what it does in the menu.
It doesn't change the toolgun.
Any thoughts?

This shouldn't be happening as it just uses the Sandbox tool system like all other tools do. Was this in singleplayer or multiplayer? Dedicated or listen server? And can you switch to other tools, just not nZU ones, or not any tools at all?
I know that if you switch tool tab back, it'll still highlight whichever one you had selected from before, even if you selected another one from the other tab. You have to re-click it to re-equip it. Can you confirm that it wasn't just this?

I haven't seen anything like that when I was using the tools, The tools are fully functional of the ones that are present in the build

@Zet0rz
I tried both in singleplayer and multiplayer and I just cannot equip the tools.
The toolgun stays on whatever tool I was using last (defaults to rope tool)
I can spawn stuff through entities but that's it.


Adding image name in XML configuration leads to DuplicateKeyException

Description
When using the Spring Boot Http Booster, if I add the following configuration to my pom.xml
          <plugin>
            <groupId>io.fabric8</groupId>
            <artifactId>fabric8-maven-plugin</artifactId>

            <configuration>
             <!-- this images section is what causes the error -->
              <images>
                <image>
                  <name>somerepo:18444/spring/http:1.0</name>
                </image>
              </images>
            </configuration>

            <executions>
              <execution>
                <id>fmp</id>
                <goals>
                  <goal>resource</goal>
                  <goal>build</goal>
                </goals>
              </execution>
            </executions>
          </plugin>

then when I execute mvn fabric8:resource -Popenshift I get the following exception:
Failed to execute goal io.fabric8:fabric8-maven-plugin:4.1.0:resource (default-cli) on project rest-http: Execution default-cli of goal io.fabric8:fabric8-maven-plugin:4.1.0:resource failed: Duplicate key somerepo:18444/spring/http:1.0
...
...
...
Caused by: java.lang.IllegalStateException: Duplicate key
somerepo:18444/spring/http:1.0
    at java.util.stream.Collectors.lambda$throwingMerger$0 (Collectors.java:133)
    at java.util.HashMap.merge (HashMap.java:1254)
    at java.util.stream.Collectors.lambda$toMap$58 (Collectors.java:1320)
    at java.util.stream.ReduceOps$3ReducingSink.accept (ReduceOps.java:169)
    at java.util.ArrayList$ArrayListSpliterator.forEachRemaining (ArrayList.java:1382)
    at java.util.stream.AbstractPipeline.copyInto (AbstractPipeline.java:481)
    at java.util.stream.AbstractPipeline.wrapAndCopyInto (AbstractPipeline.java:471)
    at java.util.stream.ReduceOps$ReduceOp.evaluateSequential (ReduceOps.java:708)
    at java.util.stream.AbstractPipeline.evaluate (AbstractPipeline.java:234)
    at java.util.stream.ReferencePipeline.collect (ReferencePipeline.java:499)
    at io.fabric8.maven.enricher.standard.openshift.ImageChangeTriggerEnricher$1.visit (ImageChangeTriggerEnricher.java:72)

Basically what seems to be happening is that when the image name is set (in this case to artifactrepository.fanniemae.com:18444/demo-boot/demo-boot:1.0), FMP thinks there are two images (the configured one and the default one that is is formed from groupId and artifactId).
This in turn leads FMP to create attempt to create 2 different containers of the same name. The exception is thrown when FMP tries to create a map using the Streams API and having keys collide.
Info

f-m-p version : 4.1.0
Maven version (mvn -v) : 3.5.4
Kubernetes / OpenShift setup and version : 1.11, 3.11

I can add a PR for this if someone points me in the right direction.
Basically I see that the final method that adds the second image is JavaExecGenerator#customize. I would go ahead and open the PR but I am a little hesitant in doing in that since I don't understand why JavaExecGenerator#customize is being used when List<ImageConfiguration> configs is not empty.

@geoand: Sorry for the late reply. The reason why JavaExecGenerator#customize is being used because we only consider ImageConfiguration as null when it's BuildImageConfiguration is null. See:
https://github.com/fabric8io/fabric8-maven-plugin/blob/e0d6b394a7abf75b5a23f6abd23ecba93a27afe7/generator/api/src/main/java/io/fabric8/maven/generator/api/support/BaseGenerator.java#L200-L203
If you add <build> section to your image configuration then the second image would not be generated.
There is a flag also to override this behavior, but I'm not sure that works properly.
https://github.com/fabric8io/fabric8-maven-plugin/blob/e0d6b394a7abf75b5a23f6abd23ecba93a27afe7/generator/api/src/main/java/io/fabric8/maven/generator/api/support/BaseGenerator.java#L57-L58


send cache hit ratio to graphite

A couple of teams have asked for it. I wondered what you thought?

Interesting idea.
For now, we don't plan on adding new metrics in the OSS. But we do in the commercial version we're working on (we've already added a few).

Note that getting the status codes will give you a part of the cache hits (304).
Still, you would be missing the other cache hits (Expires or Cache-Control max-age).
Then, there's also the permanent redirects.

I currently do a headerRegex on X-Cache HIT|MISS and write to a file: https://github.com/bbc/gatling-load-tests/blob/master/src/test/scala/bbc/mediaselector/MediaSelectorOpen.scala#L32-L38

X-Cache is not a standard HTTP cache and even it's behavior is very implementation specific.
Is it set on resources that are served from a CDN?

Yes, I think that is the case.

Note that your PrintWriter based impl probably lessen performance (synchronized).

Thanks for the info. What library would you suggest?

Akka :) But that's probably overkill for you.
Also note that you don't close your writer, so you're leaking a fd.

I close the writer in an after block. Should I close it on each write?

My bad.

I am now updating a mutable map during the simulation
https://github.com/bbc/gatling-load-tests/blob/master/src/test/scala/bbc/mediaselector/MediaSelectorOpen.scala#L41
and using synchronous file I/O in the after block
https://github.com/bbc/gatling-load-tests/blob/master/src/test/scala/bbc/mediaselector/MediaSelectorOpen.scala#L45

A mutable Map isn't threadsafe. Use a ConcurrentHashMap instead (or a ConcurrentLinkedQueue).
You also have to make sure all your data fits in memory.

Something to be considered in the commercial product, but not in the OSS version.


slightly cleaner GetOrigin implementation

I noticed that StringExtensions.GetOrigin is a little hard to follow, and uses String.IndexOf and substring to approximate URL parsing. While it works alright, I think it's probably better to use the Uri class.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Thanks!
Coud you please add some tests to it?

Good call - I have a set of addresses I tested this against already (taking the output of existing implementation vs mine) but I'm not sure what the best way to add these are. Perhaps I should add a separate pull req with the tests (so we have a set of inputs we know pass against the existing implementation) - then it's possible to test for regressions in the new implementation? What do you think? I know it's not a hugely important part of IdentityServer, but I'd like to get it right :-)

Either a new PR or add a commit. thanks

Last chance to add tests. Thanks.

Hey Brock - sorry I have a lot of very poor excuses for not responding! I've added the tests that I put together - testing various combinations of scheme * host * port * resource against the value returned by the original implementation.

Cool, thanks!