MV Patch

Fixes some issues, added "Skill Focus" options as well as "Miscellaneous Bonuses" options for all skills. Added a base value of 0 to all character stats (and base of 6 for move speed).

Looks like you're changing some attribute names that may cause users of the old version of the sheet to lose data. You might consider adding a sheet worker to update these so that users don't lose anything.

Which attribute names am I changing?

For example, where you are changing Artisans to be a repeating section, that will not preserve old data because even thought he attribute names are the same, they are now in a repeating section so they won't automatically update.

Artisan was changed to a repeating section last patch to begin with, and the code for Artisan prior to the last update was flawed to begin with. In order to change it to a repeating section, the code for the old Artisan wouldn't have been able to be carried over.
I'm sorry if this inconvenienced you or anyone else.

Ok, that sounds fine, but could you please resolve the conflicts on this branch so that I can merge this request?

Which conflicts need to be resolved, aside from the Artisan field changes as mentioned above?

Github is detecting a merge conflict in Multiverse/Multiverse Sheet.html. You should see the message This branch has conflicts that must be resolved on this page below our comments.
You can resolve the conflicts using the web editor by clicking on Resolve conflicts, or it may be simpler to discard your changes, bring your fork completely up to date, and then re-apply your changes.
I'm happy to help walk you through any of this if this isn't clear.

I've gone through and I'm pretty sure I've fixed all of the issues present.


Angular Directives Not Working Under FormattedString Span Tag

For Some reason Angular  Ng Directives are not wotking
Following is a code sample:
<Button dock="top" class="btn btn-primary btn-rounded-lg" style="margin: 5;">
            <FormattedString>

            </FormattedString>
        </Button>

When you toggle between items where isPractiveMode gives you true and false then the Formatted String becomes longer and longer. Please find the image attached.

Hi @rakeshgirase,
Thank you for reporting this issue.
I tested this case and indeed this is a real issue and the Span visibility will not be changed dynamically while using the ngIf directive.
I am logging this as a bug and for further info, you could keep track on the issue.

+1

@rakeshgirase I'd suggest you creating FormattedString in component and bind it to the Label.
  formattedString: FormattedString;

  ngOnChanges(changes): void {
    if (changes && changes.job && changes.job.currentValue) {
      this.formattedString = this.generateValuesFormattedString(changes.job.currentValue);
    }
  }

Hi @trunkovich Thank you for the workaround. I raised this bug because it is not working as expected and ideally should be fixed.

Still no fix..

still not working

@tsonevn This is still not working, should I create a new issue for this?


ISPN-10612 Support multiple permissions for GlobalRoleConfigurationBuilder

https://issues.jboss.org/browse/ISPN-10612

Thanks @diegolovison


React + TypeScript?

Does it support using React with TypeScript?  It doesn't seem to work for me...

Currently we just look for. Js and. Jsx files. I will scope out the effort and see if it would be an easy fix.


Signer Addition Modal UI

CTAs needs to be added on Modal

The Signer addition modal was missing in the design file ,have added both in light an d dark mode now. Also a CTA been added as well.
file:
https://www.figma.com/design/m5eMFTKD391AiJAXaEzgxB/Bitcoin-Keeper?node-id=2695-6551&t=iRjHxGnU63kB8i6p-1
s.s.:

@Raheel1258 can you check if its implemented we can close

@AreejKhalid21 hasn’t the key addition modal design already changed? It now has different CTAs depending on the key being added. I made this change in the branch @cakesoft-vaibhav  is working on for remote key sharing since it’s dependent on it. So, is this ticket still valid?

No this is not valid anymore. These were the old modals and were to be used till the new changes weren't implemented. If the new Signers Addition modals have been implemented as they are in design file then this issue can be closed as signer addition modals have already been updated in design as well.

I have still added the CTA just in case the new modal does not go into the release for some reason.


pythonPackages.virtualenv: 13.1.2 -> 15.0.2

Motivation for this change
Things done

[ ] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Apparently, there is a 15.0.3 version out: https://pypi.python.org/pypi/virtualenv. Maybe bump the version in nixpkgs up to that release?

79149ac12d5009169900f027de2633a843e614f0


sw-lib has large file size for basic cases

Library Affected:
sw-lib
Browser & Platform:
All
Description:
My old manual service worker that does basic caching for a set of URLs with a simple version key went from 1k to 45k. I know that's not particularly actionable, and there are already distinct modules for a la carte inclusion, but might be nice to hunt down some savings in parts baked into sw-lib or provide other, smaller targeted builds to cover the most basic use cases.

I managed to get a sourcemap tree of the built sw-lib. super useful for exploring what is in this 45k.
view it: http://standing-farm.surge.sh/

The sourcemaps generated in utils/build seem to be borked so I had to rebuild the bundle on my own. It's hacky but this is what i ended up using:

### hacky re-build of sw-lib

const fs = require('fs');
const babel = require('babel-core');

const rollup = require('rollup');
const nodeResolve = require('rollup-plugin-node-resolve');
const commonjs = require('rollup-plugin-commonjs');

rollup
.rollup({
  entry: './packages/sw-lib/src/index.js',
  sourceMap: true,
  plugins: [
    nodeResolve({ jsnext: true, main: true, browser: true }),
    commonjs()
  ]
})
.then(bundle => bundle.write({
  dest: 'swbundle.js',
  moduleName: 'goog.swlib',
  format: 'cjs',
  sourceMap: true,
}))
.catch(err => console.log(err.message, err.stack))

.then(_ => {

  const sourcejs = fs.readFileSync('swbundle.js', 'utf-8');
  const result = babel.transform(sourcejs, {
    presets: [
      ['babili', {
        comments: false,
        keepFnName: true,
        keepClassName: true,
      }],
    ],
    sourceMaps: true,
    inputSourceMap: JSON.parse(fs.readFileSync('swbundle.js.map', 'utf-8'))
  });

  fs.writeFileSync('./swbundle.min.js', result.code);
  fs.writeFileSync('./swbundle.min.js.map', JSON.stringify(result.map));

});

Woo! Thanks for the source maps breakdown, @paulirish! ⭐
Digging into our gzip sizes atm, we're coming in at just under 13KB:
$ gz sw-lib.v0.0.17.min.js
orig size    (bytes):
   44446
gzipped size (bytes):
   12824

Largest parts of our build

Our libs: sw-precaching, sw-runtime-caching, sw-cache-expiration
Indiv libraries we pull in: path-to-regexp (3.8KB), idb (4.44KB), error-stack-parser (3.1KB)

Inspecting our actual built source, we have a lot of debug level logging that's useful during iteration workflow but could be slimmed down for the final production build.
We might also be able to trim down some code using babel-preset-env to only transpile the bare minimum that modern browsers need.

AFAIK, the only transpilation we're doing is async/await to generators, and as per http://caniuse.com/#feat=async-functions, if we only care about the stable versions of browsers with SW support, we don't need to do that anymore.
I worry a bit about Samsung Internet and other random Chromium-based browsers which don't get auto-updated as frequently as Chrome does, though.

I looked into our options for doing conditional builds with imports removed.
Given that we're using ES2015 import syntax for pulling in everything, and you can't conditionally execute import, we're somewhat limited.
What seems like the most elegant (albeit still hack-y) solution is to use https://github.com/rollup/rollup-plugin-replace as part of our build process, and rewrite a few of the import statements during the build process so that they pull in modules that are effectively no-ops, or at least very minimal.
We could do this assert.js and log-helper.js, so that statements like
import assert from 'path/to/assert.js';

get rewritten as
import assert from 'path/to/assert-no-op.js';

where assert-no-op.js just exported an object that matched the assert.js interface, but didn't do anything.
For log-helper.js we could do something similar, though we might want to respond to the warn and error methods in the interface with a corresponding console.warn and console.log, instead of being a true no-op.

Here's the current breakdown of the minified, production build, with error-stack-parser omitted:
Live source-maps-explorer output

I think this is well tracked via the overall v3 work. The addition of pr-bot in particular ensures that we're keeping an eye on build sizes as part of every PR.


x/sys/windows: a way to load dll functions by ordinal (not only by name)

It would be usefull to be able to load DLL functions by ordinal in syscall package.
Example: to use the SHCreateMemStream function on Windows XP
Thanks :)

The syscall package is frozen (see its package docs) but maybe something can be added to the https://godoc.org/golang.org/x/sys/windows package (it's a bug that those docs don't show anything: #16509).

It would be usefull to be able to load DLL functions by ordinal in syscall package.

I don't see usefulness of that. You have mentioned "SHCreateMemStream", but I have never used that function. What is the problem that you are trying to solve?
And you can do it now. You can copy syscall.GetProcAddress and change the function to pass integer to syscall.Syscall instead of string. I just don't see how this can be used to justify us adding this functionality even in golang.org/x/sys/windows.
Alex

SHCreateMemStream is exported by ordinal in WindowsXP. I needed it to
create a memory stream to export/import an in-memory jpeg GpImage.
Finally, I solved the problem by using GlobalAlloc/Free and
CreateStreamFromHGlobal.
Le ven. 5 août 2016 à 07:54, Alex Brainman<EMAIL_ADDRESS>a
écrit :

It would be usefull to be able to load DLL functions by ordinal in syscall
package.
I don't see usefulness of that. You have mentioned "SHCreateMemStream",
but I have never used that function. What is the problem that you are
trying to solve?
And you can do it now. You can copy syscall.GetProcAddress and change the
function to pass integer to syscall.Syscall instead of string. I just don't
see how this can be used to justify us adding this functionality even in
golang.org/x/sys/windows.
Alex
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/16507#issuecomment-237770700, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABtU_uA2harxClZghpKKEdVTEGL5F_Kcks5qct46gaJpZM4JVvOf
.

SHCreateMemStream is exported by ordinal in WindowsXP.

Do you have some references? Just for my curiosity. Thank you.

I solved the problem by using GlobalAlloc/Free and
CreateStreamFromHGlobal.

Cool. I take it you don't need ability to find DLL procedure by cardinal number then.
Alex

Reference: MSDN
In fact, I don't need it anymore but using SHCreateMemStream would have
been easier.
Thanks.
Le Lun 8 Aoû 2016 2:43, Alex Brainman<EMAIL_ADDRESS>a écrit :

SHCreateMemStream is exported by ordinal in WindowsXP.
Do you have some references? Just for my curiosity. Thank you.
I solved the problem by using GlobalAlloc/Free and
CreateStreamFromHGlobal.
Cool. I take it you don't need ability to find DLL procedure by cardinal
number then.
Alex
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/16507#issuecomment-238124333, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABtU_l3aMWEFRjmRfr1pPHbSQ0xV-tQ4ks5qdonUgaJpZM4JVvOf
.

Reference: MSDN

How do I use this reference? I was hoping you have some official links on the Internet. I tried implementing this, but I cannot even write a test for it - what should my test do?
Alex

Example: https://msdn.microsoft.com/en-us/library/windows/desktop/bb773795.aspx tells me to import shlwapi.dll:IsOS by ordinal in order to be compatible across OS versions.

Example: https://msdn.microsoft.com/en-us/library/windows/desktop/bb773795.aspx tells me to import shlwapi.dll:IsOS by ordinal in order to be compatible across OS versions.

@hillu thank you for the reference. I will try to implement this when I have spare time. Unless someone beats me to it.
Alex

@hillu I have found your CL 70690. I will review it when I have time. Thank you.
Alex


Mention the feed url:
{
    url: "https://thinkdobecreate.com/feed/"
}

https://github.com/murtuzaalisurti/rssed/commit/65b68423b6ec88d678d78fa94ca11b62ed4bcb6b


P8B Display Flipped

Have flashed your zip as I'm not sure the state of the PRs into the standard Infinitime release. I believe I have a p8b based on magnetic charger. Ordered recently from Aliexpress and the stock firmware worked okay.
Everything seems to be working except ... the display is flipped left-to-right. Same behaviour was seen in the bootloader.

I don't see anything immediately obvious for build options or config. The touchscreen interactions aren't flipped. Before I start digging in driver code, do you have any suggestions?

Oh wow, that is a new one :D Where did you buy that one? I am interested in collecting more variants.
What was your firmware identification string? E.g. something like MOY-TFK5-1.7.7 .
As for fixing this, I think the display controller itself has an option/register to flip the display. I am not sure weather the driver exposes this function. Maybe LVGL has a function to mirror everything in software as well.

Touch is not flipped. I didn't take note of the firmware string before trying to flash wasp-os (also was flipped). I'll start having a dig through LVGL and the display driver.
This was the model purchased:
https://www.aliexpress.com/item/4000557708951.html

I see. So your touch inputs don't line up with the elements on the display you want to tap?
I guess the manufacturer decided to mount the LCD in reverse, or used a slightly different model.

I couldn't find anything in LVGL to flip, only rotate. Setting MX bit in MADCTL register fixes the display
void St7789::MemoryDataAccessControl() {   WriteCommand(static_cast<uint8_t>(Commands::MemoryDataAccessControl));   WriteData(0x40);  //  MY(0) MX(1) MV(0) ML(0) RGB(0) 0 0 0 }
in St7789 header. Uploading the firmware, it shows correct for about 10 seconds before reverting back to the bootloader. More digging for another day.
I'm at a loss how I have a flipped screen, seems like a very strange sku change. Unless they have a bad production run and worked around it?

Built from clean <IP_ADDRESS> source and it is now working and (seemingly) stable. I'm surprised how precise and responsive the touchscreen is. The stock firmware was nowhere near this reliable (for the 15 minutes I played with it).
Accelerometer works, heart rate works, time and gadgetbridge sync works. The step counter doesn't work.
I'm not really sure the best way to incorporate this. There's no real way to detect screen flip, so compile flag/yet-another-version or a setting in the watch are seemingly the options?

Great to hear! If you want to create a PR, please make one to https://github.com/StarGate01/InfiniTime/tree/p8b-base , since that is the one that will get upstreamed via https://github.com/InfiniTimeOrg/InfiniTime/pull/1128 (see also: https://github.com/InfiniTimeOrg/InfiniTime/pull/1050). The p8b branch in my InfiniTime fork is a big merge of all these PRs.
I recommend adding an entry to https://github.com/StarGate01/InfiniTime/blob/p8b-base/src/CMakeLists.txt#L796= , e.g. P8B_FLIP. I will replace that name once I figured out its original MOY name.
Then, you can define a variable: add_definitions(-DDRIVER_DISPLAY_FLIP), and check for that one in your code.
For testing, you can just merge your PR fork back into p8b.

The step counter doesn't work.

Step counting in hardware is not supported by the SC7A20 accelerometer used in the P8b, and no software implementation exists yet.

I have integrated the changes from your fork into the branch p8b-mirror (https://github.com/StarGate01/InfiniTime/tree/p8b-mirror), and created a PR at https://github.com/InfiniTimeOrg/InfiniTime/pull/1200 . It is also linked to https://github.com/InfiniTimeOrg/InfiniTime/pull/1050 .
If you want to make further changes, please rebase and PR to p8b-mirror.

Please test the new <IP_ADDRESS> MOY-UNK release: https://github.com/StarGate01/p8b-infinitime/releases/tag/v<IP_ADDRESS> .

Have flashed <IP_ADDRESS> and confirm happily running.

The model I received from your AliExpress link appears to be a atock MOY-TON5 P8b, colors are correct, however the SPI Flash might have changed. I will have to debug this in depth (already ripped it open). I suspect the manufacturer mixes and matches parts depending on availability.
In any case, your model will continue to be called MOY-UNK.


Arguments to pass to fbsimctl

Is there any documents around what parameters to pass to fbsimctl to launch simulator with udid or something similar to unit tests you guys have for FBSimulatorControl?

fbsimctl is lacking in the documentation department right now. What may be of use is the the Unit tests for the command parsers., there's examples of the commands and what they should parse into.
Loosely speaking the format of a fbsimctl command is:
fbsimctl [global-options] [query-or-simulator-UDID] [list-of-actions]

Thanks a lot. I will give a try

Closing due to inactivity

Thanks. hat adding udid first was a trick as bootstraponline pointed out.
./fbsimctl  boot :boot simulator with udid
./fbsimctl create "iPhone 6" "9.1" : crete new simulator with model and iOS version
./fbsimctl   terminate com.apple.mobilesafari : launch app on sim with udid
./fbsimctl   launch com.apple.mobilesafari : terminate app on sim with udid


Authentication

I'm currently able to log in to Zooniverse as usual but not into Chimpandsee. Have tried in different browsers, etc. The 'forgot your password?' option goes to to https://www.zooniverse.org/#/reset-password, which doesn't work when one is still logged into the main site.

I'd like to add that there have been a disturbing number of reports to this effect:  able to sign in on the main page, but not into certain projects.  Although it has been for a couple of different reasons.
Question: are you using your username?  Not your email address?  Email address login only works on the main page.

Yes, username, almost always via a password manager (so it's not a matter of misspellings).

I get the same problem on Chrome 44.0.2403.125.

On 30 Jul 2015, at 03:47, Peter Green<EMAIL_ADDRESS>wrote:
Yes, username, almost always via a password manager (so it's not a matter of misspellings).
—
Reply to this email directly or view it on GitHub https://github.com/zooniverse/Chimp-Zoo/issues/74#issuecomment-126166651.


Enables reacting to user pin attempts in AppLockActivity.

This is a squashed version of "Feature/attempt listening #12".
By overriding
public void onPinSuccess(int attempts)
and/or
public void onPinFailure(int attempts)
you can react to a user passing or failing a pin code challenge.
Example: after 5 incorrect pin attempts log a user out and close the app.

Normally for this type of feature, we proceed with an interface to make the callback pattern.
Could you change a little bit this pattern?
make onPinFailure and onPinSuccess part of an interface (OnLockListenerInterface for instance). This way the interface can be implemented in a lot of ways, not only by extending AppLockActivity.
Add a setOnLockListener(OnLockListenerInterface) to the AppLock manager for the user to provide his own implementation of the OnLockListenerInterface
Then on the view call like you do the AppLock.getOnLockListener().onPinSuccess and onPinFailure when necessary.
Don't hesitate if you have any question or remark.
Thanks in advance.
Cheers,
Olivier

I had originally designed it that way but switched it because this way is much simpler. Also, pin success and failure events all happen IN the AppLockActivity so it makes sense to have those events fired FROM AppLockActivity. For instance, I need a reference to my AppLockActivity inside the onPinFailure() so that I can finish() the Activity and log a user out if they fail the pin challenge 5 times. This is easily done when this happens inside of AppLockActivity.
Thoughts?

Alright let's keep in that way for now. Just a modification before the merge, could you make the onPinFailure and onPinSuccess abstract?
Thanks in advance.
Cheers,
Olivier

We cant make them abstract. AppLockActivity is no longer abstract. Because forgot and handling pin successes and failures are all optional the AppLockActivity is completely stand-alone now. Even if AppLockActivity was still abstract we probably don't want to make optional methods required by making them abstract.

The purpose was not to make AppLockActivity a StandAlone.
The purpose is to extend it. Moreover, having empty methods just to override them is not super well in term of design. I am making a few changes on the main branch to make it Abstract again.
Cheers,
Olivier

I made it abstract again.
It's better that way than to propose a stand-alone with empty features (like clicking on the forgot button does nothing by default, that's weird).
Cheers,
Olivier

Ah ok. It just seemed weird to keep it abstract when forgot went optional. You were requiring a class implementation to fill in a forgot method when they may have ticked it off.

Yeah it's just that at least they are forced to override it, so they are aware of the feature and how to implement it.
It's easier for us to maintain, otherwise we will have issues and questions about that.

Cool, I get it. Just pushed the new code.

Thanks, pull request merged.


Revert changes in the lib/job.js file from 39217f8ddcf3046cd3a0fb7a77e0bf6924c48292

Fixing issue #217

Thanks a lot @jbachelet


Please, add Universidad De La Salle Bajío

https://bajio.delasalle.edu.mx
https://bajio.delasalle.edu.mx/oferta/oferta1.php? n=1&p=35

@isravazquez I have to close this request without merging, because you requested to add a non-existent domain to this repository: ;; connection timed out; no servers could be reached Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Fix Android app stuck in splash screen when killed by the OS

Summary
There is a bug in RNN that causes the app to enter an infinite loop between the SplashActivity and the first activity of the app giving the sensation that the app is stuck in the Splash Screen, I've reported the Bug and there seems to be a workaround for it (referenced in the issue).
But when closing the app with the hardware back button and re-open the app indeed was stuck, that is why I've added the changes in the Makefile to handle the back button with a different behavior and solving the issue.
RNN = react native navigation
RNN are saying that they are fixing this and other issues in their v2 but is not yet ready but we should be on the look out.
Ticket Link
https://mattermost.atlassian.net/browse/ICU-592
cc: @grundleborg just to take a look at the PR and if you have some feedback is always welcome

Eww that's gross. But if it fixes the bug I'll be very happy as that's one of the bugs that has caused me a lot of pain on my phone.


[folding][json] show number of collapsed array items

Issue Type: Feature Request
When the mouse hovers an ellipsis sign that represents a collapsed literal array, the editor would display a small popup window showing the number of elements in the array.
VS Code version: Code 1.56.2 (054a9295330880ed74ceaedda236253b4f39a335, 2021-05-12T17:13:13.157Z)
OS version: Windows_NT x64 10.0.19042

@faicalcarvalho Sorry I can only make the upvote 2 now. The only alternative I know of is to select the line of the opening bracket [, then Cmd/Ctrl + F to find the first closing bracket ] (or prefix with the number of spaces if it's a nested array)... and finally select the last array item to see the last index here:

I don't know why they couldn't just put the length inside these brackets:

Would be nice to see something like the GitLens "grey text" with the number of elements next to the folded array:


MDS analysis plot

Thanks for your great work! I found the MDS analysis plot on Fig. 4, could you show us the code of MDS analysis? I think this plot is very useful , and I would like to create same plot on my own data. I am just interested in how to create distance matrix represent sequence distance and number of mutations simultaneously.

Thanks for patience with slow reply. Here's the code for the MDS plot https://github.com/debbiemarkslab/sequenceMDS/blob/main/sequenceMDS example 2 - alignbio tournament 2023.ipynb


Is it possible to compile to .app on macOS? Also, is there a way to hide terminal when building

Go version: latest
Go-SDL2 version:
SDL2 version:
OS: macOS
Architecture: Apple Silicon

Hi @Mariownyou, Go doesn't build an .app file but you can create it yourself.
You can try building the program statically so all dependencies is contained using:
CGO_ENABLED=1 GOOS=darwin GOARCH=arm64 go build -tags static -ldflags "-s -w" -o app

After that you can create the following directory structore for the app bundle. Here the name is GoSDL2.app as example.
GoSDL2.app
└── Contents
    ├── Info.plist
    ├── MacOS
    │   └── app
    └── Resources
        ├── app.icns
        └── example.bmp

The content of Info.plist is the following:
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
  <key>CFBundleExecutable</key>
  <string>app</string>
  <key>CFBundleIconFile</key>
  <string>app</string>
</dict>
</plist>

Then you can copy the executable app to GoSDL2.app/Content/MacOS/ and if you have any resource like image files, you can put them under GoSDL2.app/Content/Resources/.
As for hiding terminal when building, could you elaborate more on that? Is that a problem somehow?

Thank you for providing information regarding creating .app bundle.
About terminal issue, I would like to hide terminal when builing app

HI @Mariownyou, do you mean that you want to hide terminal when running the app instead of building the app? I think you could create the app bundle like above and run it. In that case it won't show the Terminal

Thanks, I will try this :)


Remove underscore/lodash dependency

This might just be a personal preference, but I don't feel like it makes sense to require underscore/lodash for this library, especially when a lot of the functions already have Angular or jQuery alternatives. The fewer external libraries needed to get up and running, the better, and I just don't have a pressing need for those libraries in my apps.
I'd be glad to help rewrite the necessary bits if you're okay with moving away from this dependency.

+1

Will you be using lodash for 2.0?

I do not like the idea of having an implicit dependence on Loadsh in Restagnular too.
This brings me problems when I want to use other libraries as Lazy.js, which for some functionality have better performance. And if in my project I just need these features that has better performance, I need to be required to have the problems of conflicts? Why that? Why force the use of extra library is the objective of the main library is not the same dependency library? Open source is not freedom? Restangular is not open source?
PS: I use Restangular in almost all my projects with angularjs and HTTP requests and I love to do this to be easy with Restangular! Thanks @jdalton.

lodash v3 supports lazy evaluation btw.

+1


Investigate useOnyx infinite loading state related to the Connection Manager

This issue was first noticed here -> https://github.com/Expensify/App/pull/49509.
I recently implemented a fix in Onyx to solve the infinite loading states after a Onyx.clear() operation, but in this case they are happening without the clear operation, so we must investigate it again.
Report of the problem: https://github.com/Expensify/App/pull/49509#issuecomment-2462699104
Reproducible steps: https://github.com/Expensify/App/pull/49509#issuecomment-2403667996
My initial investigations: https://github.com/Expensify/App/pull/49509#issuecomment-2477373813

Hi @gijoe0295 Could you confirm for me if you are still being able to reproduce this loading state issue? I'm having difficulties to reproduce it. If you are able could you give me the detailed test steps again? Thanks

@ishpaul777 maybe you can help me here with reproducing the problem from your side? We need to remove reuseConnection: false from src/pages/workspace/withPolicy.tsx before testing.

Update: Created a WIP Draft PR. Will proceed with testing soon.

@ishpaul777  PR open to review!

Offer sent @ishpaul777

Payment complete.


Design Tabs

goal: design the tabs component.
Prior art

Pain points

The current tabs component transform to a Dropdown on mobile
Tabs are horizontally centered
Tabs doesn't look like actual tabs

Tasks

[ ] Define use case
[ ] Define mobile behaviour
[ ] Visuals
[ ] Document on the website

Use Cases

To divide page content under logical grouping.
Divide a long form into logical groups of inputs.

Wireframes

Desktop

On tablet sizes and up the tab options are displayed in a row, with the content of every tab underneath.

Mobile

On mobile the tabs options are also displayed in one single row but the user can scroll the options horizontally.

@LeoT7508
I think we are good to go with this one

@jinjiahuangsap
Im adding Jinjia to this issue since he is working on it now.

@joseegm @LeoT7508 Hey guys, below are Tabs and how it works on mobile.

@jinjiahuangsap
Good first draft of the tabs :)
Still some stuff to be be improved:

We only need the left aligned ones, we decided that yesterday on the design meeting.
The tabs doesn't look like tabs. That is one of our pain points that we want to solve

For the examples with more content, the tabs should be above the toolbar, not inside the collection content.

@joseegm @jinjiahuangsap

We included left aligned tabs and centered tabs to cover both instances if needed.
We experimented with different colored shapes behind the tabs, it didn't work out, this fits in better with the overall design style we have so we will move forward with this design.

@LeoT7508  @jinjiahuangsap
Guys, we can definitely do better than this.
The design needs to solve the pain points and one of them is the tabs not looking like tabs.
You can try something like this

It is easily to see the one selected and it "encapsulates" the content bellow visually.
Please note: we are reserving the ALL CAPS only for the actions, all SAP interfaces don't use all caps, but we can get away if we use that wisely.
Let's give this another try, it can be so much better than now.

@joseegm
This isn't about "doing better than this", its about creating a consistent design pattern that fits the existing components and design style we have created to this point.
This solution for tabs is widely used in design systems everywhere so there is no fundamental issue with a 1px rule below a line of clickable labels with a highlight below the active section.
Now as it pertains to our design system, this solution works best for reasons listed below:

Our text button links work exactly the same way, on hover there is a 2px rule that highlights the button.

The tabs presented fall right in line with this, it creates less variation for button styles, and creates a repeatable visual experience.

This should also be easier to implement for developers, with this solution they don't have to create a whole new button set for tabs, they can just clone our Text Button and add in the tab feature.

This solution also scales better than other tab structures, the wider the button shape (like the example you created above) the more space this uses horizontally on the page. The tabs we proposed more effectively use the horizontal space they occupy. This holds especially true on mobile where there isn't much room horizontally and the space needs to be used more efficiently.

Now pertaining to ALL CAPS. We are keeping consistent with our buttons using all caps labels. However I do see some value in creating a version with title case labels since some of the names can get longer. We can create a variation for this and see how it comes out. I would like to stay with ALL CAPS wherever possible. So we will iterate on this and come back with the best solution.


UI Improvement

Change color-pallete for both light & dark mode

Updated color-pallete for dark mode


Example with abnormals

It could be nice to have an example with abnormals to show how to deal with $p^0$. We can also augment the system.

@ocots also nice to make a shooting function with a normalisation ($|p(0)|=1$, e.g.) that works for both normal and abnormal extremals.


Use assert.h instead of cassert

This fixes building for android differently than in f5e483ce.
On android,  isn't available in the normal include path,
only when the STL headers are available.
We intentionally avoid using STL within the main libopenh264.so, to
simplify dependency chains for users of the library (which otherwise
could run into conflicts if the surrounding app would want to use
a different STL implementation).
The previous fix only provided headers, not actually linking
against STL, so at this point it's not a real issue yet, but it's
still a very slippery slope towards accidentally starting relying on
STL within the core library.
Instead explicitly avoid using STL within the core library, by not
even providing the include path.
Review at https://rbcommons.com/s/OpenH264/r/1642/.
FYI @saamas @GuangweiWang @HaiboZhu

Thank you very much. Your comments are very professional.


qcloud bucket task

这个 PR 实现什么功能/修复什么问题:
腾讯云 bucket website/cors/referer设置

是否需要 backport 到之前的 release 分支:
--release/3.5

/area region
/cc @ioito @zexi

Codecov Report

Merging #8795 (53ffd1b) into master (df82c08) will increase coverage by 11.90%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #8795       +/-   ##
===========================================
+ Coverage    5.94%   17.84%   +11.90%
===========================================
  Files         818       88      -730
  Lines      127071     8846   -118225
===========================================
- Hits         7553     1579     -5974
+ Misses     118841     7094   -111747
+ Partials      677      173      -504

Flag
Coverage Δ

aFlag
17.84% <ø> (+11.90%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/compute/models/loadbalancerqcloudcachedlbb.go

pkg/util/s3auth/interface.go

pkg/compute/models/secgroupcache.go

pkg/compute/models/cloudregionschedtags.go

pkg/cloudcommon/policy/resources.go

pkg/hostman/guestfs/fsdriver/macos.go

pkg/cloudcommon/db/sharedresource.go

pkg/baremetal/tasks/prepare.go

pkg/multicloud/ucloud/image.go

pkg/mcclient/options/loadbalancercertificates.go

... and 718 more

Continue to review full report at Codecov.

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

/lgtm

/lgtm

/lgtm

/lgtm
/approve

/lgtm
/approve


feat: update default value of git_ref :safety_pin:

By default use github.base_ref and use master as a fallback.
This will primarily help when running on pull_request_target.

Also, you also have access to these variables within the typescript code:
https://github.com/actions/toolkit/blob/main/packages/github/src/context.ts#L6-L24
Might be good, because there you can actually check what trigger it was and so on.

[test]

[test]

[test]

I think this is a breaking change. I would consider releasing it with tfaga 3.0 sometime in the future (although 3.0 is not yet even planned).

I don't think this is a breaking change, but it changes behavior a bit and might surprise some users.

Initially I thought so too, but it is a breaking change because it changes from master -> github.ref, which at minimum will affect PR builds.

Initially I thought so too, but it is a breaking change because it changes from master -> github.ref, which at minimum will affect PR builds.

But only in the sense that TF would be using plans from the PR branch, right?

Initially I thought so too, but it is a breaking change because it changes from master -> github.ref, which at minimum will affect PR builds.

But only in the sense that TF would be using plans from the PR branch, right?

As far as I can think of that would be the only implication, but even there that is quite limited for repos that have master as the main branch as that is being discouraged. You may also have discrepancy when it is run on tag.
But, why guess, there are only 28 instances of the action usage: https://sourcegraph.com/search?q=context:global+++"uses:+sclorg/testing-farm-as-github-action"+&patternType=keyword&sm=0

For sure I would add it into v3. Let's add it there for sure.

[test]


centos7  ncm2-go   how can i use it

If you're having trouble with NCM2, Please provide the following information
along with your detailed issue description:
OS
(If you're using Linux, sharing a reproducible, vimrc minimized docker image,
can definitely help solving the issue)
neovim :version output
Minimal vimrc for reproducing the issue
neovim :messages output when the error occurs
Log files if you think the issue is a bug
export NVIM_PYTHON_LOG_FILE=/tmp/log
export NVIM_PYTHON_LOG_LEVEL=DEBUG
nvim
# reproduce the issue

i use nvim  and vim-plug     bug  it not works

Make sure that https://github.com/mdempsky/gocode is installed and gocode binary is available in your PATH.


Unable to Release OCRAN 1.3.16

I am unable to release OCRAN version 1.3.16. Could someone guide me on what steps to take to resolve this issue? Any assistance would be greatly appreciated. Thank you.

https://github.com/Largo/ocran/releases/tag/1.3.16
I have created the 1.3.16 gem, but I was unable to upload it to RubyGems.org.

Thanks a lot! I released it!

Thank you. Now I can go to sleep.


fix: Switching back from comment mode the selection canvas doesn't work.

fixing bug where switching back from comment mode the selection canvas doesn't work.
Fixes #7709

if no issue exists, please create an issue and ask the maintainers about this first

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions, so we can reproduce.
Please also list any relevant details for your test configuration.

Test A
Test B

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
No changes to code coverage between the base branch and the head branch

@marks0351 Working fine on Deploy Preview.


ModuleNotFoundError: No module named 'selective_scan_cuda_oflex'

Can you help me? How can I solve it

I think that you should reinstall the selective_scan with the order "cd kernels/selective_scan && pip install ." until "Successfully built selective_scan
Installing collected packages: selective_scan
Successfully installed selective_scan-0.0.2"

Yes, I have done this by following the steps you described, but it is still not resolved.
---Original---
From: @.>
Date: Wed, May 22, 2024 11:17 AM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [MzeroMiko/VMamba] ModuleNotFoundError: No module named'selective_scan_cuda_oflex' (Issue #203)
I think that you should reinstall the selective_scan with the order "cd kernels/selective_scan && pip install ." until "Successfully built selective_scan
Installing collected packages: selective_scan
Successfully installed selective_scan-0.0.2"
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

oh... Perhaps you can then install a 'mamba_ssm' and the problem will be solved

oh... Perhaps you can then install a 'mamba_ssm' and the problem will be solved

I have also installed the package of mamba_ssm.
---Original---
From: @.>
Date: Wed, May 22, 2024 11:55 AM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [MzeroMiko/VMamba] ModuleNotFoundError: No module named'selective_scan_cuda_oflex' (Issue #203)
oh... Perhaps you can then install a 'mamba_ssm' and the problem will be solved
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

oh... Perhaps you can then install a 'mamba_ssm' and the problem will be solved

One of my current situations is that I have selective_scan_cuda_core, but no selective_scan_cuda_oflex

oh... Perhaps you can then install a 'mamba_ssm' and the problem will be solved

I also tried pip install mamba-ssm but the problem of https://github.com/MzeroMiko/VMamba/issues/55 still persists.


Bower depency list must be dictionnary

Hi there,
So I'm trying to install and use this with bower.
The problem is, when I start my meteor server after installing this with bower, it will give this error :
error: Bower dependencies list must be a dictionnary in bower_components/angular-data-table/bower.json
Anyone have a clue on how it can be fix or is it something to be fixed in the table ?

Any update on this? I don't think this is an error on my end.


Cannot read property 'OAuth2' of undefined

In the Google sheets api sample code:
https://github.com/gsuitedevs/node-samples/blob/e426bd6af84904ea49ebd043787659936264727a/sheets/quickstart/index.js#L20
Expected Behavior
The following should be working properly.
const google = require('googleapis');
const OAuth2Client = google.auth.OAuth2;
Actual Behavior
Instead, it is showing
Cannot read property 'OAuth2' of undefined
Steps to Reproduce the Problem

Just create a quickstart.js with the code linked above.
Install required packages.
Run node quickstart.js.

Solution
After debugging the code, I found out that it is using the google property in the google variable that we required from googleapis. This bug was resulting in the undefined behavior as google.auth is not defined. What actually defined is google.google.auth.
So what I did was extract the google property directly be using ES6 destructuring which fixed this issue and all the further issues which were created by it.
const {google} = require('googleapis');
Specifications

Node version - v8.10.0
OS - Linux(Ubuntu v16.04)

I already fix this problem at #3
but I didn't mention about this prob... sorry

No worries. :+1:
By the way, this problem is still there in many gsuitedev/node-samples quickstart codes and READMEs. If you want I can fix them on all the places.

@grant I found this issue on many of the Node.js documentation on the Google website also. I would be happy to contribute to those also.

Thanks @paigekehoe.
Looks like I haven't published the updated GitHub snippets for all the pages with changes in the above PR.
I will leave this issue open until I do.

Thanks, good spot!

Published.


chore: bdd-spring-1582439046 to 0.0.1

chore: Promote bdd-spring-1582439046 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Containers reconnect

Docker Compose Gradle plugin (I'm the author) has a nice feature of reconnecting to already running containers. It's very useful when iterating over an integration test because e.g. RabbitMQ Windows Container is starting few minutes and you don't want to wait for the container start again and again.
How it could work in TestContainers:

When creating a new container, add an unique label that indicates why the container is being started, e.g. full test (suite) name.

If there is already a container with this label, don't create a new container, just perform the waiting policy instead.

When a container should be removed, no action is performed (so the container is still running). Or alternatively, the container is just paused/stopped.

Obviously, this shouldn't be the default behavior. It would be fine if this behavior could be enabled using an environment variable, or using a properties files.
If this feature request makes sense for you, could you please point me to the right direction how to implement this feature (where to start) ?
Thanks!

Duplicate of #781 (see the answers/progress there)


Fork template

Overview
When I forked a template from templates.eth , instead of just the targeted template coming across to the new DAO (i.e. process payroll template) all the templates came with it
Additional Info

Navigate to templates.eth - https://app.dev.fractalframework.xyz/daos/0x83BF3E2949b3aB5714BBc2D4Fda64a949D8932b1.

See the templates there

Check out the forked individual 'Process Payroll' template proposal here - https://app.dev.fractalframework.xyz/daos/0x167bE4073f52aD2Aa0D6d6FeddF0F1f79a82B98e/proposals/7

Review how all the templates now movved to this DAO - https://app.dev.fractalframework.xyz/daos/0x167bE4073f52aD2Aa0D6d6FeddF0F1f79a82B98e/proposal-templates

Version
Dev (app.dev.fractalframework.xyz/)

@mudrila very low priority

@adamgall this got sorted by an earlier PR right?

@decentdao/engineering @decentdao/product
I would like to remove Proposal Templates from the app completely.

Whats the status on the thoughts on this?


Add inspirational quote

I just read a great book on the Wright brothers where I found this gem.

:bird: :bulb:


Custom Data files not being copied to relevant directories

I've followed the instructions to use my own data however I can't get it to work. The .dat and .xml files are not copied to the directories on both iOS and Android. I'm using Cordova 6.1.1 along with the latest version of Ionic framework. The plugin works with the test data.

Duplicate of #15 I think - we are currently testing this

Sorry didn't see that!

It's cool 😀


Add "Report-To" header

@myartsev @cak Looking like bug introduced: #8 fix by #9

@myartsev @cak Looking like bug introduced: #8 fix by #9


Repeated "Parameter default value should be constant" with whole file context

I'm not sure if this is a bug, or misuse due to newbness.
I'm trying to mock an object:
import mockatoo.Mockatoo;
using mockatoo.Mockatoo;
...
	@:mock var areaMock:FlxScrollableArea; // or var areaMock = Mockatoo.mock( FlxScrollableArea );

That code alone (either variation) produces 11 identical warnings:

HaxeWrapper.hx:73: C:\HaxeToolkit\haxe\std/haxe/macro/MacroStringTools.hx:73: characters 82-87 : Parameter default value should be constant
HaxeWrapper.hx:73: src/gimmicky/FlxScrollableArea.hx:15: lines 15-231 : Defined in this class

I tried providing some constant default values for my class constructor's parameters and for my public and private class variables, but nothing reduces the number of warnings.  Could someone help me with this?
BTW, haxe 3.2.1/mockatoo 3.2.1, and I'm using the --no-inline compiler flag.

Is this because I'm on haxe 3.2.1, maybe?


Fixes BUCK

visbility looks like a typo of visibility. Also, is prebuilt_cxx_binary meant to be prebuilt_cxx_library? My buck (v2<IP_ADDRESS>) is complaining about unknown rule type prebuilt_cxx_binary.

Also, would it be nicer to use
exported_headers = {
    'json.hpp': 'src/json.hpp',
},

instead? It would allow #include <json/json.hpp>, which feels more natural than #include <json/src/json.hpp>. But this would be breaking, and might be subject to individual's preference. So I left it out of this PR.

Yep, this was a mistake on my part! Fixing it now.

Thank you for merging in the PR!
On the other note, do you have any thoughts on #include <json/json.hpp> vs #include <json/src/json.hpp>? In terms of the change being breaking, I think we can have both available at the same time. Just wanna get your thoughts on it :)

Never mind my previous comment, I just realized that you made the header namespace change already ;)


Loading popup move behind debug

The loader popup has to move behind the debug popup.

fixed in https://github.com/flagrow/bazaar/commit/cab1f156fdf4eda2f4c7fa7a4ab5347ddb767751


Framebuffer rendering

This PR also includes the instruction for installing the driver on the Waveshare 2.8" DPI LCD. The code is not adjusted for this yet, it lacks support for variable resolution, as well as touch input.
The input timings also need some tweaking, this is especially noticeable in the IO test screen.

Thank you for the review, good comments!

I'm closing this PR because I'm pretty confident the seedsigner project will not move to framebuffer in the near future. I do appreciate the work in this PR.


Gcp readme

Just add two clarifications in the READMEs for gcp and mnist example with gcp.
Thanks,

Should be good now, thanks,


pipelineX Support sort node

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.3 seconds
stream load tsv:          553 seconds loaded<PHONE_NUMBER>9 Bytes, about 129 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.1 seconds inserted 10000000 Rows, about 343K ops/s
storage size:<PHONE_NUMBER>9 Bytes


feat: Support additionalProperties of type any in Python and TS (box/box-codegen#453)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6586

Relevant Lines:
8386

💛  - Coveralls


fix(cobertura): removes rounding

The rounding was previously removed one the package level. This commit also removes it for the sources level.
Fixes #6

Hi @VannaDii, is that something you can merge and release by any chance?

Yes, I'll make a point to get this merged and released today. Thank you for the fix 🙏🏼

This is now released on https://pub.dev/packages/cobertura

Thanks!


Btrx 587 int cohorts

Addresses
BTRX-587: ORSP: Add International Cohorts Questionnaire to NE and NHSR Projects Review and Edit flow
Changes

Due to project extra properties already contains a variable named feeForService we used feeForServiceWork name to differenciate one and another. In order for this change to be updated on  'issue' object, we had to map this variable in the backend

Created a new component for International Cohorts review so we save boilerplate code in Project and Consent Group

Some minor tweaks had to be done on questionnaireWorkflow. Essentially to differentiate when this questionnaire is being used for creation or edition

Keep centralizing the use of isEmpty method placed in Utils

[ ] Submitter: Verify all tests go green, including CI tests
[ ] Submitter: Get a thumb from @rushtong
[ ] Submitter: Merge to develop using github's "Squash and Merge" feature
[ ] Submitter: Delete branch after merge

@rushtong i Corrected the code, please check again.
Also added a validation scenario for Legacy projects. In these cases International Cohorts and Info Security shouldn't be required to submit an edit/suggestion


Analyzer Feedback from IntelliJ

Analyzer Feedback from IntelliJ
Version information

IDEA IU-183.5912.21
2.2.0
IU-183.5912.21, JRE 1.8.0_152-release-1343-b28x64 JetBrains s.r.o, OS Mac OS X(x86_64) v10.14.3, screens 2560x1440, 1200x1920

Exception
Dart analysis server, SDK version 2.2.0, server version 1.23.0, FATAL error: Failed to handle request: {id: 3, method: analysis.setAnalysisRoots, params: {included: [/Users/rayk/critprojects/repo], excluded: [/Users/rayk/critprojects/repo/.idea, /Users/rayk/critprojects/repo/app/.dart_tool, /Users/rayk/critprojects/repo/app/.pub, /Users/rayk/critprojects/repo/app/build, /Users/rayk/critprojects/repo/appCfg/.dart_tool, /Users/rayk/critprojects/repo/appCfg/.pub, /Users/rayk/critprojects/repo/appCfg/build, /Users/rayk/critprojects/repo/appLog/.dart_tool, /Users/rayk/critprojects/repo/appLog/.pub, /Users/rayk/critprojects/repo/appLog/build, /Users/rayk/critprojects/repo/appState/.dart_tool, /Users/rayk/critprojects/repo/appState/.pub, /Users/rayk/critprojects/repo/appState/build, /Users/rayk/critprojects/repo/authN/.dart_tool, /Users/rayk/critprojects/repo/authN/.pub, /Users/rayk/critprojects/repo/authN/build, /Users/rayk/critprojects/repo/calcs/.dart_tool, /Users/rayk/critprojects/repo/calcs/.pub, /Users/rayk/critprojects/repo/calcs/build, /Users/rayk/critprojects/repo/drapht/.dart_tool, /Users/rayk/critprojects/repo/drapht/.pub, /Users/rayk/critprojects/repo/drapht/build, /Users/rayk/critprojects/repo/fetcher/.dart_tool, /Users/rayk/critprojects/repo/fetcher/.pub, /Users/rayk/critprojects/repo/fetcher/build, /Users/rayk/critprojects/repo/knocon/.dart_tool, /Users/rayk/critprojects/repo/knocon/.pub, /Users/rayk/critprojects/repo/knocon/build, /Users/rayk/critprojects/repo/lokistore/.dart_tool, /Users/rayk/critprojects/repo/lokistore/.pub, /Users/rayk/critprojects/repo/lokistore/build, /Users/rayk/critprojects/repo/sim/.dart_tool, /Users/rayk/critprojects/repo/sim/.pub, /Users/rayk/critprojects/repo/sim/build, /Users/rayk/critprojects/repo/sled/.dart_tool, /Users/rayk/critprojects/repo/sled/.pub, /Users/rayk/critprojects/repo/sled/build, /Users/rayk/critprojects/repo/out]}, clientRequestTime:<PHONE_NUMBER>847}
NoSuchMethodError: The getter 'fullName' was called on null.
Receiver: null
Tried calling: fullName
#0      Object.noSuchMethod (dart:core/runtime/libobject_patch.dart:50:5)
#1      _DartNavigationComputerVisitor._addUriDirectiveRegion (package:analysis_server/src/domains/analysis/navigation_dart.dart:361:47)
#2      _DartNavigationComputerVisitor.visitPartDirective (package:analysis_server/src/domains/analysis/navigation_dart.dart:232:5)
#3      PartDirectiveImpl.accept (package:analyzer/src/dart/ast/ast.dart:9453:49)
#4      _DartNavigationComputerVisitor.visitCompilationUnit (package:analysis_server/src/domains/analysis/navigation_dart.dart:149:12)
#5      CompilationUnitImpl.accept (package:analyzer/src/dart/ast/ast.dart:2439:49)
#6      computeDartNavigation (package:analysis_server/src/domains/analysis/navigation_dart.dart:28:10)
#7      ServerContextManagerCallbacks._computeNavigationParams (package:analysis_server/src/analysis_server.dart:1034:5)
#8      ServerContextManagerCallbacks.addAnalysisDriver.<anonymous closure>.<anonymous closure> (package:analysis_server/src/analysis_server.dart:843:19)
#9      new Future.<anonymous closure> (dart:async/future.dart:176:37)
#10     _rootRun (dart:async/zone.dart:1120:38)
#11     _CustomZone.run (dart:async/zone.dart:1021:19)
#12     _CustomZone.runGuarded (dart:async/zone.dart:923:7)
#13     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:963:23)
#14     _rootRun (dart:async/zone.dart:1124:13)
#15     _CustomZone.run (dart:async/zone.dart:1021:19)
#16     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:947:23)
#17     Timer._createTimer.<anonymous closure> (dart:async/runtime/libtimer_patch.dart:21:15)
#18     _Timer._runTimers (dart:isolate/runtime/libtimer_impl.dart:382:19)
#19     _Timer._handleMessage (dart:isolate/runtime/libtimer_impl.dart:416:5)
#20     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

For additional log information, please append the contents of
file:///private/var/folders/k3/rpf30xp576gd3b7wdpvvyg2r0000gn/T/report.txt.

Duplicate of #36299.


Capture task input files when plugin not applied

This PR changes the behavior such that task input files are captured when the environment variable is explicitly specified and for the cases when the plugin is not applied.

Changes LGTM. However, can you please raise an issue describing the problem that is fixed by this PR, and the changed behaviour? That way it won't be missed from the release notes.

Sure, I've created the issue and linked it.


Manually execute indicator

[ ] Button to execute indicator
[ ] Button to execute indicator group
[ ] Display execute result to user

Button to execute indicator group implemented as part of PR #213


Crash by ConcurrentModificationException

Describe the bug
I don't have a prior logs on clue how did this happened. Stack trace does not show my code and I am not able to reproduce the issue, so sorry for that.
Stack Trace
Caused by java.util.ConcurrentModificationException
       at java.util.ArrayList$Itr.next(ArrayList.java:860)
       at java.util.AbstractCollection.toString(AbstractCollection.java:461)
       at java.lang.String.valueOf(String.java:2924)
       at java.lang.StringBuilder.append(StringBuilder.java:132)
       at com.twilio.audioswitch.AudioSwitch.enumerateDevices(AudioSwitch.java:274)
       at com.twilio.audioswitch.AudioSwitch.access$setLogger$p(AudioSwitch.java:29)
       at com.twilio.audioswitch.AudioSwitch$bluetoothDeviceConnectionListener$1.onBluetoothHeadsetStateChanged(AudioSwitch.java:47)
       at com.twilio.audioswitch.bluetooth.BluetoothHeadsetConnectionListener$DefaultImpls.onBluetoothHeadsetStateChanged$default(BluetoothHeadsetConnectionListener.java:4)
       at com.twilio.audioswitch.bluetooth.BluetoothHeadsetManager.onReceive(BluetoothHeadsetManager.java:140)
       at android.app.LoadedApk$ReceiverDispatcher$Args.lambda$getRunnable$0$LoadedApk$ReceiverDispatcher$Args(LoadedApk.java:1588)
       at android.app.-$$Lambda$LoadedApk$ReceiverDispatcher$Args$_BumDX2UKsnxLVrE6UJsJZkotuA.run(-.java:2)
       at android.os.Handler.handleCallback(Handler.java:883)
       at android.os.Handler.dispatchMessage(Handler.java:100)
       at android.os.Looper.loop(Looper.java:214)
       at android.app.ActivityThread.main(ActivityThread.java:7615)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:492)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:964)

and second one, probably on another thread or worker:
Fatal Exception: java.lang.RuntimeException: Error receiving broadcast Intent { act=android.media.ACTION_SCO_AUDIO_STATE_UPDATED flg=0x10000010 (has extras) } in com.twilio.audioswitch.bluetooth.BluetoothHeadsetManager@df0a9a7
       at android.app.LoadedApk$ReceiverDispatcher$Args.lambda$getRunnable$0$LoadedApk$ReceiverDispatcher$Args(LoadedApk.java:1606)
       at android.app.-$$Lambda$LoadedApk$ReceiverDispatcher$Args$_BumDX2UKsnxLVrE6UJsJZkotuA.run(-.java:2)
       at android.os.Handler.handleCallback(Handler.java:883)
       at android.os.Handler.dispatchMessage(Handler.java:100)
       at android.os.Looper.loop(Looper.java:214)
       at android.app.ActivityThread.main(ActivityThread.java:7615)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:492)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:964)

Expected behavior
Probably some thread protection should be there or maybe immutable list should be used. you know the best.
Android Device (please complete the following information):

Device: LGE Stylo 6
API Version: Android 10

Version

Version: 1.1.0

Additional context
Happens rarely.

Hi @guness . From the ConcurrentModificationException it appears that your application is accessing an AudioSwitch instance from multiple threads. We currently only support access to this class from a single thread as specified in the documentation here. Please make sure that access to this class in your application is single threaded. Thanks!

hey Alton, I will make this single threaded. thanks for the tip.

Hey,
I think the README should refer to this situation of only supporting access from a single thread.
Besides that I think there are two situations that should also be informed:

AudioSwitch.start will perform the register of WiredHeadsetReceiver to listen to ACTION_HEADSET_PLUG changes, and this is sticky broadcast so when you do astart the enumerateDevices will be trigger twice (one directly from the AudioSwitch.start and another from the callback WiredDeviceConnectionListener
It should also be indicated that the lib should be used from the MainThread because, WiredHeadsetReceiver will send updates through the MainThread. If you startAudioSwitch.start in a different thread than the Main, you can end in situation where two different threads are calling enumerateDevices and leading to ConcurrentModificationException


Vs Code File search not working for .SQL Files.

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

Open the Folder that contains .sql files.
Press CTRL + P
Then search for the desired file
It Does not show the file while the file exists in the folder

bug link ... (https://youtu.be/7B0y9d9LDQ8)

Can you try disabling all extensions? Also, what do you have for your search.excludes setting?

Search.Exclude option is giving this error.

by disabling extensions found out that this extension was causing the problem.

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


openssl wrapper gets stuck if client is started before network is ready

in my client I set up a connection using code copied from the pubsub sample, which uses the openssl wrapper:
https://github.com/aws/aws-iot-device-sdk-cpp/tree/master/samples/PubSub
if I start my program before the system's network is available then it gets stuck forever on the iot_client->connect() call (see this line for example):
https://github.com/aws/aws-iot-device-sdk-cpp/blob/master/samples/PubSub/PubSub.cpp#L181
the error returned is -300, NETWORK_TCP_CONNECT_ERROR, with a log line "TCP Connection error" which means it came from this bit of code:
https://github.com/aws/aws-iot-device-sdk-cpp/blob/master/network/OpenSSL/OpenSSLConnection.cpp#L290
and this specific system call:
https://github.com/aws/aws-iot-device-sdk-cpp/blob/master/network/OpenSSL/OpenSSLConnection.cpp#L158
I'm going to instrument the system call to get its error code.  What happens is the connection fails, my program waits N seconds and retries (with a fresh network::OpenSSLConnection object), and continues to get the same error even though my computer can ping and the network is now up.  If I restart my client after the network is up it works fine.

error from connect() is ECONNREFUSED

figured this out.  glibc caches /etc/resolv.conf from the program's start and you need a call to res_init() to update it.  gethostbyname() was returning a local IP for my aws endpoint instead of failing, and will do so forever because res_init() is never called.  I am working on a patch that I'll submit as a PR.  I suggest a review of all the C system calls being used because I found a few other details wrong (not calling close() after connect() failed, for example, and some deprecated syscalls).

Hi @larsonmpdx,
Thanks for figuring this out. We have decided we need to simulate this in our release pipeline somehow and make sure we cover this issue for our future releases. I will also be spending some time working only on the network code to make sure your C system call issues are addressed. I think the wrappers can be improved substantially for each of the platforms. Please do let us know if you have any further suggestions.
Rahul


AttributeError when publication was not yet cited

When trying to use pub.citedby on a publication that has not yet been cited, scholarly throws an AttributeError.
I believe that initializing "citedby" per default with 0 during init would be beneficial.
-->
    self.bib = dict()
    self.source = pubtype
    """Init citedby with 0 to avoid AttributeError"""
    self.citedby = int(0)

Confirming the bug. Minimal code to replicate:
search_query = scholarly.search_pubs_query("Demand-Aware Career Path Recommendations: A Reinforcement Learning Approach")
p = next(search_query)
filled = p.fill()
print(filled.citedby)

---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-22-f7831d7fdac9> in <module>
----> 1 filled.citedby

AttributeError: 'Publication' object has no attribute 'citedby'

I have addressed this bug here (line 81)

The bug is there at v.0.3 as well.


zap_auth doesn't support logins with username with type=email

Assuming html login form username is actually an email, this control:
<input name="email" placeholder="Email" type="email" class="form-control">
won't be identified, even when run with auth.username_field="email"
It would seem that this happens as the code in zap_auth.py explicitly looks for @type='text' or not(@type) controls.

@iTaybb I just made a commit which includes username fields with type=email. Can you pull the latest image and try again?

It seems that with the latest image, the code crashes:
 Traceback (most recent call last):    File "/zap/zap_auth.py", line 77, in login      self.auto_login(config)    File "/zap/zap_auth.py", line 163, in auto_login      self.submit_form(username_element)    File "/zap/zap_auth.py", line 169, in submit_form      if self.config.auth_submitaction == "click":  AttributeError: 'ZapAuth' object has no attribute 'config'

@iTaybb oops... fixed the bug. Can you try again?

Looks great! 10x


Line rate should accept float value

Now line rate should be of type integer. But in some cases we need to provide line rate less than 1.

line rate in Flow.Rate is now modeled as a number which allows for floats
        percentage:
          description: >-
            The percentage of a port location's available bandwidth.
          type: number

line rate in Flow.Rate is now modeled as a number which allows for floats
        percentage:
          description: >-
            The percentage of a port location's available bandwidth.
          type: number


Problems raised when a PNG image was being processed.

When I tried to expand a PNG image with an alpha channel, somewhere translucent seemed unlikely to retain its transparency but more likely to transform into white background with other colors on it. If there are difficulties I'd like to know what can I do by myself instead.

アルファチャンネルがあるPNGを処理したときに、半透明部分がおかしな色になることがあるということですよね？
その現象が起こるPNGをアップロードしていただけないでしょうか。


onAnimationComplete doesn't work after show tooltip.

I think we need to add the revoke of onAnimationComplete at the end of showTooltip() function, see following fix:
showTooltip : function(ChartElements, forceRedraw){
......
this.options.onAnimationComplete.call(this);
return this;
},

@tannerlinsley what do you think?

@ricemouse Huzzah! The first alpha of Chart.js 2.0 has landed and should fix this issue. Check out the release and try it out! We've got a lot of momentum right now, so please help us test so we can launch 2.0 Gold by the end of the month.
https://github.com/nnnick/Chart.js/releases/tag/v2.0-alpha
I'm closing this issue for now, but if you have implementation questions or find bugs, please create a jsfiddle and post the link here and we'll reopen this issue and get it fixed.


dist.sh: use go build option to strip binaries

30% release binary size reduction - from 10487848 bytes to 7041472 bytes for the linux x86_64 build.
I also bumped the version for dev, I can rebase and omit that commit if a different version bump is done in master.

Nope, this is all I had in mind. (Any requests?)


Deny-PublicPaaSEndpoints contains a policy with no deny effect

API Management should disable public network access to the service configuration endpoints  policy (/providers/Microsoft.Authorization/policyDefinitions/df73bd95-24da-4a4f-96b9-4e8b94b402bd), which is a part of Deny-PublicPaaSEndpoints initiative does not have deny effect. It should be either removed from the initiative or deny effect should be added to the built in policy.

Hi @achechen, thanks again for raising you issues and concerns here!
We're busy packaging the next policy refresh, which is large and contains many updates including to that particular initiative. Can I ask we review this in a week or so?

The Policy's effect parameter has allowedValues of AuditIfNotExists and Disabled, so it can be changed in the Initiative or Assignment parameters without an override.
Trying to override an AuditIfNotExists Policy with the Deny effect should generate an
error and prevent the update. Otherwise, the existenceCondition will not evaluated and all APIM resources matching the if condition would be denied!
The corresponding APIM enforcement policy below has a DeployIfNotExists effect and so it would probably be in different initiative named Deploy-:
Configure API Management services to disable access to API Management public service configuration endpoints (7ca8c8ac-3a6e-493d-99ba-c5fa35347ff2)
Many thanks for all the excellent work.

@achechen and @s4parke as part of the policy refresh just released, we have a new set of policies and initiatives that can help further secure workload specific features like for APIM (under "Workload Specific Compliance" in the portal accelerator). The DINE policy you mention above is included for API Management specifically.
We'll review if we should remove the AuditIfNotExists policy from the overall infra initiative.

Thanks! I noticed most of the enforcing policies don't have an Audit/AINE effect so my workaround has been to add them to a separate Initiative and either Assign them with Enforcement mode=Off or without the SAMI roles.
The override effect in the Assignment would be sweet, but it only works if AINE is one of the AllowedValues in the effect parameter for that policy,
But maybe it's possible to create an Initiative Parameter called "Effect" and tell the Policy to use that instead of its own "Effect" param in the Initiative Definition. That's something to try.


Mantainement

Is this mantained?

I've not been working on the project that required it for a while.  It's something I'd like to get back to though.  If you submit a PR I'd be happy to merge it and add you as a maintainer.

Try https://github.com/tmds/Tmds.DBus


undefined macro variable '_arg0'

class Foo
  macro def bar(*args) : Nil
    nil
  end
end

Foo.new.bar(1, 2, 3) # see title

Thanks for reporting! Default args didn't work either.


Usersnap Feedback - "Authorize" button on American Gut did not work.

Comment: "Authorize" button on American Gut did not work.
Open #8 in Usersnap Dashboard

Browser: Chrome 41 (ChromeOS)
Referer: https://stealth.openhumans.org/member/me/research-data/
Screen size: 1366 x 768 Browser size: 1366 x 633
Powered by usersnap.com

Authorizing now works, I've created another issue for the linking issue.


fix(ui): bottom sheet reduced motion

What this PR does / why we need it:

Possible fix on bottom sheet not opening on some devices
There are cases wherein the battery is low or reduced motion is enabled, the bottom sheet is not opening at all

Which issue(s) does this PR fixes?:

Fixes #
Additional comments?:
Developer Checklist:

[ ] Read your code changes at least once
[ ] Tested on iOS/Android device (e.g, No crashes, library supported etc.)
[ ] No console errors on web
[ ] Tested on Light mode and Dark mode*
[ ] Your UI implementation visually matched the rendered design*
[ ] Unit tests*
[ ] Added e2e tests*
[ ] Added translations*

Codecov Report
Attention: Patch coverage is 16.66667% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 31.65%. Comparing base (bf3cd26) to head (f97a544).

Files
Patch %
Lines

...creens/TransactionAuthorization/PasscodePrompt.tsx
33.33%
2 Missing :warning:

mobile-app/app/components/BottomSheetInfo.tsx
0.00%
1 Missing :warning:

mobile-app/app/components/BottomSheetWithNav.tsx
0.00%
1 Missing :warning:

mobile-app/app/components/BottomSheetWithNavV2.tsx
0.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #4183       +/-   ##
===========================================
- Coverage   60.05%   31.65%   -28.40%
===========================================
  Files         442      442
  Lines       12577    12581        +4
  Branches     4129     4129
===========================================
- Hits         7553     3983     -3570
- Misses       4964     8554     +3590
+ Partials       60       44       -16

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Completing P107 for finite spaces

Essentially the final part of #902. In light of newly added properties, some of the finite spaces have unknown traits again. Specifically, six spaces have "Has a closed point" marked as unknown. Unfortunately, none of the additions make any of the currently known traits for these spaces redundant, as far as I can tell.

It seems that the following should be true, and is not currently derivable:
A T_0 space with a focal point must have a closed point.  (The focal point must be a closed point.)
That would allow to derive many of theses traits.

@prabau Ah, true, that's definitely correct and immediately deduce 4 of these. I'll check if this theorem generalizes something currently known.

Doesn't seem like it's connected to any known theorems. And no spaces will have their traits redundant due to this theorem, as far as I can tell. (S150 is the only one assigned P107 manually, but its P202 is deduced from P107. I've similarly checked those spaces that have P202 assigned manually.) I'll just rewrite this PR to add the suggested theorem and reduce the manually added traits to the two that cannot be deduced from this.

Wait.  Maybe the result is not true?
The dual: [has a generic point + T0 + ~ has an isolated point] seems not to be true.
π-Base, Search for generic point + T0 + ~isolated point
Maybe I am missing something.  The proposed result above would be true for finite spaces, but not sure now in general.

@prabau No, it's true. The difference is an arbitrary intersection of open sets may not be open, but an arbitrary intersection of closed sets is always closed. If you have a focal point, just take the intersection of all nonempty closed sets. It has to be closed, contain all focal points, but can only contain one point as otherwise the space won't be $T_0$. This can only work for the dual theorem if the space is Alexandrov, in which case you can take arbitrary intersection of open sets and still be open.

@prabau Suggestion implemented. (Though I realized only two of the proposed traits got automatically deduced, since only two in the search are finite.)

@david20000813 This PR is causing a git problem when doing git pull .  The files for the P107 traits are causing phantom differences when updating my repository on the command line.  Exactly the same problem as described in PR #649.
I don't know exactly how this happens, @StevenClontz probably knows, but it's happened in the past and the remedy is exactly what's described in PR #649.  Even better: whenever creating a new file, check the correct LF setting in the editor.
Let me know if that makes sense.  For now I'll create a PR to fix it and you can approve.

@prabau I think I understand this to some extent. If you create a file from scratch, sometimes the file uses the wrong control character for the line endings? I'll be more mindful about this in the future.

Yeah.  All you need to do while editing a file with the wrong setting is to click on CRLF and change it to LF.

https://github.com/pi-base/data/blob/main/.vscode/settings.json should be helping us somewhat, but it's always technically possible to set the wrong line endings. Unfortunately, GitHub's PR review system doesn't make weird line endings obvious unless you open the PR in GitHub.dev (and even then, you have to inspect each file). For now, my most clever solution is to fix things ad hoc as we're doing now.


Aleph randomizer can place alephs such that a rock or other fixed object is directly infront of the exit

Randomized alephs can get placed such that their exit is blocked by an asteroid or other custom map object.
Suggest that when placing an aleph that a check is performed to see that there is nothing in front of it within X meters (or perhaps within a radius Y, X meters in front of the aleph). If there is something there then move the aleph a certain distance in a random (or otherwise) direction / pick a new random location.
Imported from trac ticket #82,  created by madpeople on 04-27-2010 at 13:19, last modified: 07-19-2010 at 06:53

reopen with screenshots or soemthing, anything, like a map name?
Trac comment by Imago on 07-19-2010 at 06:53


Remove checks 229 and 242

These treat opinion-emum and encryption_algo_enum as if they were open
vocabs instead.
Fixes #104.

Codecov Report

Merging #105 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master   #105      +/-   ##
========================================
+ Coverage   93.99%    94%   +<.01%
========================================
  Files          61     61
  Lines        5615   5607       -8
========================================
- Hits         5278   5271       -7
+ Misses        337    336       -1

Impacted Files
Coverage Δ

stix2validator/util.py
89.8% <ø> (ø)
:arrow_up:

stix2validator/v21/shoulds.py
90.88% <ø> (+0.09%)
:arrow_up:

stix2validator/v21/enums.py
94.77% <ø> (-0.14%)
:arrow_down:

Continue to review full report at Codecov.

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


fix: Skip ClusterServingRuntime deletion if deployed using --namespace-scope-mode

Motivation
delete.sh script attempts to delete ClusterServingRuntime resources when they don't exist after deploying with --namespace-scope-mode, interrupting the script and leaving multiple resources behind.
Modifications
Set and use NAMESPACE_SCOPE env variable to check whether to skip ClusterServingRuntime deletion.
Result
Resolves #420

Thanks @rafvasq


Clean up version definition

Goal

Testing

Release Notes

WHAT:
WHY:

#1003
#1002  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bidetofevil and the rest of your teammates on  Graphite


start parameter calculation change at visible_emails_pre function

The markup for email's pagination has been changed at gmail recently:

Was:
111–114 of 114
New:
111–114 of 114
Also if there are more than 1000 emails the number may be "1,234". The  .replace(",", "") has been added to fix it.
Note: i don't know why we need duplicated start parameter.

Thanks for the patch! I've done some rudimentary checking and can confirm that:

the existing code doesn't detect page-number.
the new code does.

And by those standards, this looks good to me.

Note: i don't know why we need duplicated start parameter.

This was introduced in the following commit: a5c9928d152ccb7416725ab672c0991f33376be6
As far as I can tell, this is actually a code-error. See the previous state: It only used one start-parameter. Could you test if this works as it should if you don't setup duplicate starts?
If so, it would be nice to clean that up at the same time.
@avzabr: Any comment on this?

@josteink
Good finding! Actually based on the mentioned commit: a5c9928 the second parameter should be the sstart instead of duplicated start.

I renamed the second parameter to sstart with the b4e1e76 commit

Unsure about this one. That sstart was supposedly a typo as can be seen in this follow-up commit: 0e9822088ce10ddfbcbd383181793bbb5bba811d by Lucas Shculze (Github ID unknown).
However, the original code, before @Navega and Lucas made their changes only had one single start-value. I honestly think the second sstart (or start in the typo-fixed version) is a mistake. I don't think it should be there at all.
What I tried to ask you (but obviously failed!), was if you would be able to test without the second start entry at all, and see if that works. Could you do that please? :)
Because, if it's a bug, we should clean this up at the same time IMO.
@Navega: Do you have any opinion on this?

@josteink Ok, i was able to do several tests.
Test procedure:

Download gmail-chrome-extension-boilerplate
Update the gmail.js with new version from pull request
Update the num parameter to 10
Update the gmail inbox settings to show 10 emails (as i had only 70 emails in my mailbox)
Ensure the gmail.js with sstart parameter
Get copy(gmail.get.visible_emails()) from Primary tab first/last pages and Social tab first/last pages
Remove the sstart parameter from gmail.js
Get copy(gmail.get.visible_emails()) from same pages
Compare results from 6) and 8) with jsondiff

Test results are attached: GmailJsPull337.zip
The result is following:

At Primary first page, Social tab first/last pages results are same
At Primary last page the json received without sstart parameter does not correspond to the emails displayed on a screen

Thus from the test results the sstart parameter is needed.

That's great testing! Thanks for putting in the effort! (y)
If it sstart is indeed needed, then we keep it. Simple as that.
Consider this merged. Thank you for your effort! :)

Hi guys,
I have been debugging the problem. Gmail modifies its structure continuously. My commit does not work because Gmail replaces label "" with "" where number of offset page is positionated. I used 2 parameters: "start" and "sstart" because they depend of the used version. Older versions used the parameter "start", but now the correct parameter is "sstart" to ask mails with offset.
Certainly. good catch with offset > 1.000, thanks!
Do you agree?

@josteink The changes are perfect, thanks. The solution is to replace find("b:first") by find("span:first").

Thanks for chiming in about the history of these parameters. That makes things much more clearer :)
And just to be clear: In your opinion, no more changes needs to be done or reverted, right?

Yes, only a change: Spanish version uses "." in place of "," to write the offset. Coul you add this?

var start = $(".aqK:visible .Dj").find("span:first").text().replace(",", "");

var start = $(".aqK:visible .Dj").find("span:first").text().replace(",", "").replace(".","");

Sounds fair enough. I'll get it done.


create virtual cluster: Internal error occurred: failed calling webhook "virtualcluster.validating.webhook"

getting this error when using k3s backed base cluster and installing virtual cluster
create virtual cluster: Internal error occurred: failed calling webhook "virtualcluster.validating.webhook": Post "https://virtualcluster-webhook-service.vc-manager.svc:9443/validate-tenancy-x-k8s-io-v1alpha1-virtualcluster?timeout=30s": x509: certificate signed by unknown authority

manager logs shows following
{"level":"info","ts":1610709680.4287176,"logger":"controller-runtime.controller","msg":"Starting EventSource","controller":"virtualcluster-controller","source":"kind source: /, Kind="}
E0115 11:21:20.429401       1 event.go:260] Server rejected event '&v1.Event{TypeMeta:v1.TypeMeta{Kind:"", APIVersion:""}, ObjectMeta:v1.ObjectMeta{Name:"vc-manager-leaderelection-lock.165a63eba2391b28", GenerateName:"", Namespace:"vc-manager", SelfLink:"", UID:"", ResourceVersion:"", Generation:0, CreationTimestamp:v1.Time{Time:time.Time{wall:0x0, ext:0, loc:(*time.Location)(nil)}}, DeletionTimestamp:(*v1.Time)(nil), DeletionGracePeriodSeconds:(*int64)(nil), Labels:map[string]string(nil), Annotations:map[string]string(nil), OwnerReferences:[]v1.OwnerReference(nil), Finalizers:[]string(nil), ClusterName:"", ManagedFields:[]v1.ManagedFieldsEntry(nil)}, InvolvedObject:v1.ObjectReference{Kind:"ConfigMap", Namespace:"vc-manager", Name:"vc-manager-leaderelection-lock", UID:"4e5a1c91-4610-4a4b-ae8c-f6a257c6d512", APIVersion:"v1", ResourceVersion:"1981", FieldPath:""}, Reason:"LeaderElection", Message:"vc-manager-76c5878465-vt7zp_c51ae5b1-98a5-4691-a65b-e2cb951e2588 became leader", Source:v1.EventSource{Component:"vc-manager-76c5878465-vt7zp_c51ae5b1-98a5-4691-a65b-e2cb951e2588", Host:""}, FirstTimestamp:v1.Time{Time:time.Time{wall:0xbff87c8c198a3b28, ext:16177898643, loc:(*time.Location)(0x24f0d80)}}, LastTimestamp:v1.Time{Time:time.Time{wall:0xbff87c8c198a3b28, ext:16177898643, loc:(*time.Location)(0x24f0d80)}}, Count:1, Type:"Normal", EventTime:v1.MicroTime{Time:time.Time{wall:0x0, ext:0, loc:(*time.Location)(nil)}}, Series:(*v1.EventSeries)(nil), Action:"", Related:(*v1.ObjectReference)(nil), ReportingController:"", ReportingInstance:""}': 'events is forbidden: User "system:serviceaccount:vc-manager:vc-manager" cannot create resource "events" in API group "" in the namespace "vc-manager"' (will not retry!)
{"level":"info","ts":1610709680.5291405,"logger":"controller-runtime.controller","msg":"Starting Controller","controller":"virtualcluster-controller"}
{"level":"info","ts":1610709680.529191,"logger":"controller-runtime.controller","msg":"Starting workers","controller":"virtualcluster-controller","worker count":10}
{"level":"info","ts":1610709680.5291576,"logger":"controller-runtime.controller","msg":"Starting Controller","controller":"clusterversion-controller"}
{"level":"info","ts":1610709680.5292292,"logger":"controller-runtime.controller","msg":"Starting workers","controller":"clusterversion-controller","worker count":1}
{"level":"info","ts":1610709680.5292704,"logger":"clusterversion-controller","msg":"reconciling ClusterVersion..."}
{"level":"info","ts":1610709680.5294147,"logger":"clusterversion-controller","msg":"new ClusterVersion event","ClusterVersionName":"cv-sample-np"}
2021/01/15 11:23:09 http: TLS handshake error from <IP_ADDRESS>:49530: remote error: tls: bad certificate```

fyi i also did this based on error logs
k create clusterrolebinding cluster-admin-binding --clusterrole=cluster-admin --user=system:serviceaccount:vc-manager:vc-manager

 k delete po -n vc-manager --all

fyi i also did this based on error logs
k create clusterrolebinding cluster-admin-binding --clusterrole=cluster-admin --user=system:serviceaccount:vc-manager:vc-manager

 k delete po -n vc-manager --all

@charleszheng44 PTAL

@charleszheng44 PTAL

@debianmaster thanks for filing the issue. What is the version of the super cluster?

@debianmaster thanks for filing the issue. What is the version of the super cluster?


Add shadowsocksr-csharp

#1121

Looks good, it's just missing the .json file extension :smile:

Will not work because: "extract_dir": "ShadowsocksR-win-$version". Move it into autoupdate block and in main block change it to actual version


Extension to generate Google commerce schema

It would be great to have an extension which can output the Google commerce / product schema

Will be added in V1.1


Add another RunProject function with args to script.

I only tested this with SikuliX (using the "-- " format). I didn't change the original RunProject() function as I already have projects that are using that.

Merged and pushed to NuGet :) Thanks!


new feature: add tags with programmatically generated values such as time/date

It should be possible to add tags such as {hour}, {month}, {date} or {phpdate [any php date string]} to include timestamps and stuff like that in the bot's code.
These should be processed before the tracery parser is run, so that one could e.g. write #{month}# and have the bot call the #January# trace.
For this to work properly, the user should be able to set their timezone in the bot's control panel.

for clarity, would this allow us to set the bot to post from certain selections at certain times?
say I have a set of options for morning posts vs evening posts?
how would that work tracery code wise?

Tracery wise, since this would clearly not be vanilla tracery, I'm thinking it would make sense to use the curly brackets syntax. I'm thinking a logical format would be e.g. {date string} where string is a php date string – with aliases for common commands such as {month}, {hour} or {weekday} – and these would also work when used in tags. So you could write e.g. It is {monthName}!to return It is November!, or I feel #mood{month}#, which would be converted to I feel #mood11# which Tracery would then turn into I feel gloomy.


Update UG and add screenshots

Codecov Report
Base: 52.62% // Head: 52.62% // No change to project coverage :thumbsup:

Coverage data is based on head (caae225) compared to base (9982c69).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #135   +/-   ##
=========================================
  Coverage     52.62%   52.62%
  Complexity      708      708
=========================================
  Files           159      159
  Lines          3202     3202
  Branches        394      394
=========================================
  Hits           1685     1685
  Misses         1370     1370
  Partials        147      147

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Period cycle calculator

Description
I propose adding a period tracker calculator to the project to help users monitor their menstrual cycle and manage related information more effectively.
Below is the reference to what type of calculator I wish to contribute.
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[X] I am a SWOC'24 contributor.

Code of Conduct

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

We already have one Calculator for this. You can add more features in this if you want @Aniketmehraa

while both calculators involve dates related to reproductive health, the pregnancy due date calculator focuses on estimating the due date for childbirth during a pregnancy, while the period cycle calculator predicts the start date of menstrual periods for tracking menstrual cycles. @Rakesh9100

Kindly assign me this issue @Rakesh9100

You can do enhancements and add new features in that calculator only instead of creating a new one @Aniketmehraa


Added the use Firefox banner

This PR adds the firefox banner to all pages in the service

quick question, are you thinking of having this only on the dashboard page or everywhere? i lean towards everywhere personally

Hi. re. Firefox.
I'm concerned some people won't know what 'browser' means. Given this banner is generated for IE8 only, could we say '...using internet explorer' which they probably know from using MS Office.
After that initial sentence the reader is just left hanging a bit and has to go back up to the title to understand what to do. Could there be a second instructional sentence, even just a restatement 'Change .....'.  which is nice and short.

@JaneMOJ I think calling out Internet Explorer is a good idea. I'll update the content
cc @janestead

@ushkarev everywhere


kbfs_common: add error type for NoSigChainError

Issue: KBFS-1252

@cjb: I don't think the GUI actually has to do anything special for this error if you don't want to.  KBFS will already fill in the error with the suggested text from @malgorithms (in keybase/kbfs#184).  Though if you wanted to treat it specially, of course you could.

@strib cool, sounds okay, I won't push a handler for it straight away :thumbsup:


added in information about source map

Reviewed documents and added in a basic configuration with source maps wired up.
included links to examples and documentation

I'll fix these errors and resubmit this

@montogeek
changes made
Thanks!

Thanks!


flutter build aar should also build plugins as AARs

Description
Currently, flutter build aar doesn't build the plugins as Maven artifacts. AGP requires all libraries subprojects such as Flutter plugins to be published as Maven artifacts.
I broke this behavior when I removed the sync execute(..) call from the Flutter Gradle plugin.
Related Issues
Fixes https://github.com/flutter/flutter/issues/43919#event-2761279931
Tests
I added the following tests: integration test build_aar_module_test.dart
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

I think I tried turning on --enable-build-plugin-as-aar while testing too. Does the repro in #43919 work now with this PR?

@xster you don't need  --enable-build-plugin-as-aar. In fact, that flag doesn't do anything anymore.


net/haproxy: Initial support for HAProxy cache

This adds support for a single HTTP cache in HAProxy.
This currently supports the configuration options that HAProxy 1.8 allows for the cache ("total-max-size" and "max-age") but not the "max-object-size" option supported by HAProxy 2.0.
Only a single cache can be defined and this initial version does not allow caching to be based on an ACL.
Configuration is a two step process by firstly enabling and configuring the cache and then enabling caching in a backend pool.
Enabling caching in a backend pool without first enabling the cache under Settings will have no effect on the config.

@andrewheberle Merged, thanks!