Better Map Activity

Rather than displaying only the location of the event, the app should display some critical information too like venue for Sub Events that'll be happening alongside the main event, Nearby Hotels and the Airports.
Going forward, we can add a button that'll open up the maps application allowing the user to navigate successfully to the venue.
What should the map activity look like :

To implement this, we'll be needing to create a custom overlay to be masked on top of the map.
@mananwason @juslee @championswimmer Thoughts ?

@the-dagger great idea. I was also thinking of something like this but how do you plan to make it customisable for different events?

@manawason that's the problem. We can't generalize the overlay for every event so I guess what we can do is take the latitide and longitude for the event, subevents, nearby hotels and airports from the API and place markers there.
The end product won't be as neat as the attached map, but again we are generalizing it and not making a custom overlay for each event.

@the-dagger great project. Wanted to contribute to this thread. I was thinking of implementing marker clustering , which I had successfully implemented in my previous project .

Hey @rohanagarwal94 ,
That's awesome!
You can feel free to go ahead and start working on this issue. :+1:
Cheers.

@the-dagger the map in the image is not exactly the map in the upstream/development. Even the markers are not added as of yet. Just notify me when it gets updated. Or if you want me to implement a better map activity/fragment, I can do that too.

@rohanagarwal94 The map in the image is a prototype. It has been taken from the Google IO app. We need the map activity in our app to be similar to this.

https://lh3.googleusercontent.com/HY77Ed5C-eRpVcpMVvmIcvskf57RFrx0puwsWDlr-Rij0hMtH_m9hkeSt_fsGFl1mw=h900-rw Is this acceptable? This is my implementation through with an autocompletetextview search box and marker clustoring.

image @mananwason is it acceptable? This is my implementation of the map with marker clustering and autocompletetextview search.

Yes this looks fine but keep in mind that we want something customisable. I mean if we have a list of locations and levels, the app should be able to generate a map on the fly without any lags and the rooms on different levels should be easily distinguishable.

I will try to implement it in the best possible way.

Sorry for the late interaction, caught up on something in college. 2 things @the-dagger @mananwason . First please generate and provide the map API key for the project. Secondly, please provide me with the JSON file for the markers of the building with the objects title, subtitle(optional), lat, long, level. Otherwise almost done.

@rohanagarwal94 We can't share the key we're using for the map. Why don't you generate one for yourself with a different package name and start working on this. Please use this microlocations markers for now https://github.com/fossasia/open-event/blob/master/sample/FOSSASIA16/microlocations

#564


two tower feature tests

Description

1. Why the change?

2. User API changes

3. Summary of the change

4. How to test?

[ ] N/A
[ ] Unit test
[ ] Application test
[ ] Document test
[ ] ...

5. New dependencies

[ ] New Python dependencies
- Dependency1
- Dependency2
- ...
[ ] New Java/Scala dependencies and their license
- Dependency1 and license1
- Dependency2 and license2
- ...

@hkvision

merged in intel-analytics:main, closing


A crazy widget combination causes icon to jump

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Might be related to #83050.
On Windows:

Install font_awesome_flutter: ^10.3.0 package
Run app from the code snipped
Hover the icon with your mouse
See icon jumping when hovered

This is pretty crazy, but every single Widget in the snippet seems to be important for causing this bug to appear.
Expected results
The icon should stay in the same place when hovered (since the layout doesn't change).
Actual results
The icon moves by a few pixels to the right when hovered.
Code sample
Code sample
var hovering = false;

Widget colorFiltered({required Widget child}) {
  return hovering
      ? ColorFiltered(colorFilter: ColorFilter.mode(Colors.black.withOpacity(0.5), BlendMode.srcATop), child: child)
      : child;
}

runApp(MaterialApp(
  home: Scaffold(
    body: Padding(
      padding: const EdgeInsets.all(29.600000000000023),
      child: AnimatedOpacity(
        opacity: 1,
        duration: const Duration(milliseconds: 50),
        child: StatefulBuilder(builder: (_, setState) {
          return MouseRegion(
            onEnter: (_) => setState(() => hovering = true),
            onExit: (_) => setState(() => hovering = false),
            child: colorFiltered(
              child: Container(
                width: 24,
                height: 24,
                alignment: Alignment.center,
                child: const AnimatedOpacity(
                    opacity: 1,
                    duration: Duration(milliseconds: 50),
                    child: FaIcon(FontAwesomeIcons.pause, color: Colors.orange)),
              ),
            ),
          );
        }),
      ),
    ),
  ),
));

Screenshots or Video

Screenshots / Video demonstration
https://github.com/flutter/flutter/assets/35242233/e4d246f7-da9e-4fef-9320-5a6a15a6553e

Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.9, on Microsoft Windows [Version 10.0.22621.2428], locale pl-PL)
    • Flutter version 3.13.9 on channel stable at C:\tools\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d211f42860 (2 weeks ago), 2023-10-25 13:42:25 -0700
    • Engine revision 0545f8705d
    • Dart version 3.1.5
    • DevTools version 2.25.0

[✓] Windows Version (Installed version of Windows is version 10 or higher)

[!] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\Gieted\AppData\Local\Android\sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = C:\Program Files\Chromium\Application\chrome.exe

[✓] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.4.3)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.4.33205.214
    • Windows 10 SDK version 10.0.22000.0

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2)
    • IntelliJ at C:\Users\Gieted\AppData\Local\Programs\IntelliJ IDEA Ultimate
    • Flutter plugin version 76.3.4
    • Dart plugin version 232.10072.19

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22621.2428]
    • Chrome (web)      • chrome  • web-javascript • unknown
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 119.0.2151.44

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

This looks like a package or specific font issue as I don't see it with an Icon widget:
https://github.com/flutter/flutter/assets/104349824/d0f96672-f3ec-4043-927a-ed5b239b575c

Sample code
import 'package:flutter/material.dart';

void main(List<String> args) {
  runApp(const MaterialApp(
    home: NewWidget(),
  ));
}

class NewWidget extends StatefulWidget {
  const NewWidget({
    super.key,
  });

  @override
  State<NewWidget> createState() => _NewWidgetState();
}

class _NewWidgetState extends State<NewWidget> {
  var hovering = false;

  Widget colorFiltered({required Widget child}) {
    return hovering ? ColorFiltered(colorFilter: ColorFilter.mode(Colors.black.withOpacity(0.5), BlendMode.srcATop), child: child) : child;
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Padding(
        padding: const EdgeInsets.all(29.600000000000023),
        child: AnimatedOpacity(
          opacity: 1,
          duration: const Duration(milliseconds: 50),
          child: StatefulBuilder(builder: (_, setState) {
            return MouseRegion(
              onEnter: (_) => setState(() => hovering = true),
              onExit: (_) => setState(() => hovering = false),
              child: colorFiltered(
                child: Container(
                  width: 24,
                  height: 24,
                  alignment: Alignment.center,
                  child: const AnimatedOpacity(opacity: 1, duration: Duration(milliseconds: 50), child: Icon(Icons.pause, color: Colors.orange)),
                ),
              ),
            );
          }),
        ),
      ),
    );
  }
}

Could you narrow down the issue and confirm this?

Yeah, but it might also be due to the fact that the widget has to be in a very specific position for this bug to appear (dropping the last 2 digits from 29.600000000000023 is enough to break the setup). The Icon widget has completely different size than FaIcon, so this might be the reason why the issue don't happen to it. Of course, it might also have to do with something about how FaIcon is rendered, but it most likely is some problem with the Flutter engine.

Could you try reproducing the issue without using font_awesome_flutter package? It would be appreciated if it's a minimal sample code in a main.dart file (along with custom font files if any)

It would be hard to do so, because I don't know what exactly causes the issue (one would probably have to debug the rendering process to find the root cause)

Since font_awesome_flutter package is a widget with customization and fonts, I'm not sure if the issue is from one of them or from the Flutter framework, I would recommend you file an issue there for preliminary analysis first. Once you have more information, please also share here so that it's helpful for us to continue investigating the issue. Thanks!


[FEATURE] Knob value entry

It has been asked before, in someone's list of suggestions. This has been closed without too much explanation.
-Clicking the values under the knobs lets us input values with the keyboard
-Holding shift (or another modifier) lets us have finer control when using the mouse to turn knobs -> now working
Steve's answer: I can check whether the iPlug2 framework supports these, but if not, then it's not something I'd be developing under this repo.
Is manual value entry possible in the iPlug2 framework, if so: can we have this in NAM?
If not i'll be in the iPlug repo bothering Oli :)

it's easy to enable manual entry, I made a PR #247

it's easy to enable manual entry, I made a PR #247

🙌🏻


Discord Link going to Github

Just a small one here, but there's a link on the /dev route for the Discord button that just sends you to BitEscrow Github instead. Specifically this one:
https://github.com/user-attachments/assets/182f8539-899d-40e3-8d2f-dec556d4a418

Can't seem to find it in the code on cursory look, but it might be that you've already fixed it in the lower environment branches. :)


feat: use global thread pool in async state root calculation

a node will need to be ran off this pr for a week or smthg though, to make sure it doesn't mess smthg up that we don't see right away,

absolutely, I'm already running it on a node focusing on the reth_engine_rpc_new_payload_v3 metric for comparison, will update this PR when we have more data

since we don't have a diagram anywhere of the current thread model of the node - unless you made one?

not yet, will try to prepare something, thx @emhane !


Single Logout Service never called

Hi,
When I log out of my Django application that is a service provider, I am correctly redirected to the logout page of the IDP that I am using (Salesforce in this case). However, when I then navigate back to my site (i.e. the service provider), I am not presented with the login page, but instead it seems like I'm already logged in. I then figured out that my singleLogoutService url is never being hit, and so auth.process_slo() is never being executed (so my session is not being deleted). Do you know what could be causing this?
Thanks.

Were you guys ever able to figure this out? I am experiencing the same issue.


mac specific behaviour - maximize, minimize, close buttons not displayed

Hey,
i am using version 3.1.0 in my project.
It seems to work great & as expected on linux, but i realize issues on mac (running in a non-packaged version of my electron 5 based app)
While the configured

maximize
minimize
close
buttons are shown on linux, they aren't shown on mac os.

any idea what might cause this?
Is there something mac specific i have to configure?
here is my config so far:
const customTitlebar = require('custom-electron-titlebar');

new customTitlebar.Titlebar({
    titleHorizontalAlignment: "center", // position of window title
    icon: "img/icon/icon.png",
    drag: true, // whether or not you can drag the window by holding the click on the title bar.
    backgroundColor: customTitlebar.Color.fromHex("#171717"),
    minimizable: true,
    maximizable: true,
    closeable: true
});

a iconsTheme is not set so far. Do i have to set that os-specific?
Best regards
yafp

seems to be a dup of #27


fix: Support up to 1e-20 numbers in ui.spinbox #1525

The PR fulfills these requirements: (check all the apply)

[x] It's submitted to the main branch.
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. feat: Add a button #xxx, where "xxx" is the issue number).
[x] When resolving a specific issue, the PR description includes Closes #xxx, where "xxx" is the issue number.
[x] If changes were made to ui folder, unit tests (make test) still pass.
[x] New/updated tests are included

The problem was with calculatePrecision function not working with scientific (exponential) notation numbers.
The fix converts scientific notation numbers to regular decimal numbers and it uses Number.toFixed() for conversion.
This solution is simple but works for numbers with up to 20 decimal points. To support more decimal points, it would need more complicated solution (something like this one but more reliable).
I think 20 decimal points should be enough for now and if it won't, someone will open the new issue. Wdyt @mturoci ?
Closes #1525

I think 20 decimal points should be enough for now

wdyt @pascal-pfeiffer?

20 decimals is enough for our needs. Sounds good!
If there happens to be a new case that needs more, we can reconsider alternatives.
Thank you @marek-mihok !


fix(ci): codecov report

Codecov Report

Merging #13 (f3382a3) into main (c11cbdd) will decrease coverage by 0.60%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
- Coverage   98.63%   98.03%   -0.61%
==========================================
  Files          32       32
  Lines        1982     1985       +3
  Branches      241      164      -77
==========================================
- Hits         1955     1946       -9
- Misses         27       37      +10
- Partials        0        2       +2

Impacted Files
Coverage Δ

packages/cli/src/index.ts
100.00% <100.00%> (+28.57%)
:arrow_up:

...nd-boilerplate/src/create/package-create.action.ts
95.14% <0.00%> (-4.86%)
:arrow_down:

packages/cli/src/prepare.ts
87.36% <0.00%> (-4.22%)
:arrow_down:

...mmand-boilerplate/src/create/http-create.action.ts
96.15% <0.00%> (-3.85%)
:arrow_down:

...nd-boilerplate/src/services/boilerplate.service.ts
97.36% <0.00%> (-2.64%)
:arrow_down:

packages/npm-helper/src/package.ts
99.53% <0.00%> (-0.47%)
: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 965e2d1...f3382a3. Read the comment docs.


Upgrade peer dependencies

Hi, I've tested with<EMAIL_ADDRESS>to safely remove this warning:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
Can you please check this PR and merge changes?
p.s. I've updated @storybook/react version too in examples because it matches too with my test environment

@gvaldambrini Storybook 6 has been out for a while, could you merge this?

@gvaldambrini Do you have any update on this? We are using version v6 and are also having installation problems. Thank you.


merge package template from Kronos-Tools/npm-package-template-minimal

package.json

chore(devDependencies): update<EMAIL_ADDRESS>from template

Coverage remained the same at 78.419% when pulling 470ca37c3bb720a424898766e5282c8744c1292a on template-sync-1 into e5bcdf941bdbe7f13addc41a7f52fae1361608b9 on master.


Add a recovery test for the AfterLRA annotation

#164  added support for notifications that an LRA has finished. There are already two tests for this feature but these notifications should be guaranteed to be delivered in failure scenarios and we need a TCK test to verify that.

What do you mean by in failure scenarios, please? AfterLRA annotation JavaDoc says that implementation SHOULD retry if the response code is different from 200. @rdebusscher mentioned that we shouldn't be adding TCK for SHOULD statements.
However, I agree that we should test this and we should change the wording in every participant annotation to guarantee at least some minimal transactional guarantees. This is especially not good for @Compensate annotation because it's totally valid to write an implementation that will just log failures after the first call to @Compensate. So if any microservice enlisted in LRA is down, all LRAs will start failing until it's up again.

I mean any scenario where the listener does not respond with 200 to the AfterLRA notification.
It is not desirable to force implementations to retry forever. Saying something like "MUST retry at least 7 times" is arbitrary and therefore not useful.
So SHOULD allows the implementation to give up after a while. But to mitigate such a non-transactional outcome we do stipulate that if they do that then they MUST log a warning.

But can we then include TCK tests? I am asking because I want to do so for Compensate.

Maybe we can say it must retry once? I don't want the spec to be governed by the testing strategy but I think 1 is (maybe?) less arbitrary than 7?

@xstefank We already have tests for retrying compensate etc so I would +1 @tomjenkinson and go for "MUST retry at least once and log a warning if it gives up"

But if we could formalise the WARNING then we could continue to use SHOULD in preference to MUST which ideally I'd like to avoid.

I prefer "MUST retry at least once and log a warning if it gives up"

"retry at least once is still arbitrary", is there a way we can formalise the warning?

can't we say something like "MUST make reasonable effort to get a successful response"? I am aiming just on some minimal requirement (thus MUST) to write a TCK for this and Compensate.

can't we say something like "MUST make reasonable effort to get a successful response"? I am aiming just on some minimal requirement (thus MUST) to write a TCK for this and Compensate.

That won't help with a TCK test for Compensate.
PS Don't we already have a test that validates that Compenstate is repeatedly called in failure scenarios (I'd have to check but I am sure I wrote some)?

I think that "MUST make reasonable effort" is too vague and we couldn't code it in a TCK test?
I don't understand @mmusgrov 's comment about formalising the warning - please can you provide some wording to review?

I think that "MUST make reasonable effort" is too vague and we couldn't code it in a TCK test?
I don't understand @mmusgrov 's comment about formalising the warning - please can you provide some wording to review?

I was asking for input. Since the actual container used to execute our TCK will be different for each implementation we probably cannot access the container logs (which would contain the WARNING message).
We could require that each implementation makes the data available to the TCK (something that could be included with #175 )

However, I agree that we should test this and we should change the wording in every participant annotation to guarantee at least some minimal transactional guarantees. This is especially not good for @Compensate annotation because it's totally valid to write an implementation that will just log failures after the first call to @Compensate. So if any microservice enlisted in LRA is down, all LRAs will start failing until it's up again.

So are we ok with this as an acceptable implementation? We are developing distributed transactions in microservices so we must expect service and connection failures. Do we really want to not give any guarantees?

I don't think the discussion is about not retrying forever for compensate and complete? For AfterLRA I was all for making it non-durable like in JTA. I see no point in making any durability claims if we are just going to log a warning.

What about "MUST retry at least n times as defined by configuration parameter x.y.z"
So that we define a configuration parameter which defines how many times the implementation will retry and by setting this value to 1 in the TCK, we can effectively test it.

I like the concept from @rdebusscher  but I think something like "MUST retry until the LRA is requested to stop retrying" might be better. This could still be achieved by configuration but might be some other vendor specific API perhaps?
I think the client should be in control of the retry behaviour though (rather than the LRA vendor/host) because the client is going to expect the retry until the outcome is achieved and it is the client that would need to be notified if the AfterLRA could not be contacted - or maybe I am thinking about who the contract for the AfterLRA notification is with?

@tomjenkinson I agree, the client must be in control. But we need to standardize the way how this information is provided to the implementation for the TCK purposes.

The client could specify the retry policy using an annotation attribute of the LRA annotation.
The AfterLRA listener could do the same using an attribute of the AfterLRA annotation.

Would that cover also separate policies for Compensate, Complete, etc?

Please can someone clarify why the retry would be forever?

@tomjenkinson I don't understand. I think we are discussing including an attribute in the respective annotations that would define the number of retries. Let's say that we can add retry in the @LRA annotation that would define it for all participant methods and AfterLRA?

I meant fundamentally, I understand we are adding something because we don't want to retry forever but I lost the information about why we would ever retry forever anyway?

I think we can go with default no retries or some small reasonable number (1).

I think we should say "implementations MUST retry any callback API methods but vendors MAY provide further proprietary methods to resolve indoubt participant states and thereby prevent further retry calls"
PS I don't think we can mandate a logging methodology because certain implementations might not have a (for instance) file logging mechanism available

I like @tomjenkinson wording, we can go with this. But the configuration of max-retries and/or retry-interval is a separate question on top of that. We can take that in different issue if this one would be resolved with Tom's wording.

I like @tomjenkinson wording, we can go with this. But the configuration of max-retries and/or retry-interval is a separate question on top of that. We can take that in different issue if this one would be resolved with Tom's wording.

I think @tom_jenkinson_twitter 's sugestion (https://github.com/eclipse/microprofile-lra/issues/190#issuecomment-513796455) is that the TCK can rely on at least one retry.
Which would mean that our TCK participants would need to recover on the first retry
￼
BTW we only require this timeout behaviour to catch implementations that fail to finish an LRA. But we have the TCK property @ConfigProperty(name = "lra.tck.consistency.delay") for just this situation.
So is this a "non issue"?
￼
ie we go with Tom's text and make sure the TCK fails an implementation that does not recover on the first pass (which should be fine since we are in control of the participants).

ie we go with Tom's text and make sure the TCK fails an implementation that does not recover on the first pass (which should be fine since we are in control of the participants).

+1

@rdebusscher Do you (or someone else) want to raise a PR for this or do you want me to?

I have already something prepared for the Compensate case. I can put this preferably into one PR. Is there any other method that we want to test in TCK?

I have already something prepared for the Compensate case.
The AfterLRA case is for LRAs
The complete case is covered by the existing spec text:

Similarly if the resource cannot perform a completion activity immediately.
I can put this preferably into one PR.
One PR is prefererable
Is there any other method that we want to test in TCK?
Not that I am aware of

@mmusgrov thanks. The decision is then 2 TCK tests -- one for AfterLRA and one for Compensate with one participant failure.

@xstefank Have you started on this task yet. If not then I would like to create the two tests.
Remark: The test should also validate that it is possible to register for the notification both before and after the end phase is entered (we should probably be explicit in the spec that registering for notifications is allowed at any time that the implementation knows about the LRA - optionally even if the LRA has already ended).

yes, I have PoC already. I wanted to wait for triggerRecovery mechanism (which is now blocking development of these tests in Narayana). In case long delay solution is sufficient I can make a pr soon.

Okay no rush. So is that 2 tests for @AfterLRA and one for Compensate with one participant failure?

yes, for now, these two. We can add other scenarios later if needed.


latest release doesn't contain precompiled binaries

The links from documentation stopped working since last release.
https://taplo.tamasfe.dev/cli/installation/binary.html
example: https://github.com/tamasfe/taplo/releases/latest/download/taplo-linux-x86_64.gz .
We switched from latest to 0.8.1 in order to get binaries and use them in CI.
Is it possible to fix release so we can switch back to latest?

Sorry about this, I'll try to fix this issue.

Sorry, this is because the release is published manually without CI. I'm not quite sure how to re-release, I'll try to remove the tag and recreate them manually.

Sorry about this, I'll try to fix this issue.

All good, thanks! 👍🏼

[Invalid workflow file: .github/workflows/releases.yaml#L268](https://github.com/tamasfe/taplo/actions/runs/7742605055/workflow)
The workflow is not valid. .github/workflows/releases.yaml (Line: 268, Col: 19): Unrecognized named-value: 'env'. Located at position 1 within expression: env.RELEASE_VERSION .github/workflows/releases.yaml (Line: 270, Col: 19): Unrecognized named-value: 'env'. Located at position 1 within expression: env.RELEASE_VERSION

Seems to be https://github.com/actions/runner/issues/1189 I have no idea why/how it worked before and what changed.

See https://github.com/tamasfe/taplo/issues/535#issuecomment-1926446147. @nikaro writes:

Yep, looks like the release workflow is broken: https://github.com/tamasfe/taplo/actions/runs/7742605055/workflow#L268
And it seems to be my fault 😬 the env.RELEASE_VERSION variable is not available in the matrix.

My bad, I'm working on a PR.


Atlas Matrix Bug

Hello! I'm working on reconstructing the tissue with marker genes, and from my knowledge, the atlas matrix should be the expression matrix of cells x marker genes. It seems that the number of rows of the atlax matrix has to equal to the number of locations in the location matrix or else I get a dimensionality error. Why is the atlax matrix size dependent on location size? Shouldn't it just be the total number of cells x marker genes?

I'm commenting here because I think I'm getting the same error.
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-95-c05ffdb49fed> in <module>
----> 1 tissue_with_markers.setup_reconstruction(markers_to_use=markers_in_sc)

/Volumes/Mac-External/10x/bin/novosparc/novosparc/common/_tissue.py in setup_reconstruction(self, markers_to_use, num_neighbors_s, num_neighbors_t)
     50                 else:
     51 			cost_marker_genes = cdist(self.dge[:, markers_to_use] / np.amax(self.dge[:, markers_to_use]),
---> 52                                       self.atlas_matrix / np.amax(self.atlas_matrix))
     53                         dge = self.dge[:, np.setdiff1d(np.arange(self.dge.shape[1]), markers_to_use)]
     54                         self.num_markers = len(markers_to_use)

~/opt/anaconda3/envs/scvi-env/lib/python3.7/site-packages/scipy/spatial/distance.py in cdist(XA, XB, metric, *args, **kwargs)
   2710         raise ValueError('XB must be a 2-dimensional array.')
   2711     if s[1] != sB[1]:
-> 2712         raise ValueError('XA and XB must have the same number of columns '
   2713                          '(i.e. feature dimension.)')
   2714

ValueError: XA and XB must have the same number of columns (i.e. feature dimension.)

Okay maybe not, sorry I found my issue of course so quickly after I posted. It's because the data as I've filtered it do not contain all of the provided reference genes.
I fixed it by taking code from one of the older tutorials:

with open('../bdtnp/dge.txt') as file:
    header = file.readline()

dge_cols = header.split()

gene_cols = []
for i, dge_col in enumerate(dge_cols):
    if dge_col in gene_set:
        gene_cols.append(i)

And then setting the atlas_matrix to this:
atlas_matrix = np.loadtxt(bdtnp_path, usecols=gene_cols, skiprows=1)

The code worked for the template example that was provided but it didn't work for my lab data. The file loaded in for the atlas_matrix was hardcoded to be 3039 rows by 84 marker genes. I was wondering why the atlas matrix had to be 3039 rows, which corresponds to the size of the location matrix. I may be interpreting the code incorrectly, but shouldn't the atlas matrix be (# of cells x # of marker genes) and not (# of locations x # of marker genes)?

From what I understand, you're right that the atlas matrix is hardcoded from the bdtnp/dge.txt file. If you're using their provided 3039 cell embryo shape with this:
target_space_path = '..bdtnp/geometry.txt'
locations = novosparc.io.load_target_space(target_space_path, is_2D=True)
locations = locations[np.random.choice(locations.shape[0], 3039), :]

Then the atlas matrix has to match the locations there in order for the program to map your cells according to the reference marker genes.
Just to check if you had the same issue I had, what happens if you enter:
len(markers_in_sc)

So, as the Altas Matrix represents a spatial orientation for your (physical) target space, that means your locations, it should have the same dimensions as your locations file. The number of cells in your expression space is irrelevant for your reference Atlas, isnt it?

Hi all,
indeed the dimensions of the atlas matrix must be (# locations , # marker genes). The atlas matrix contains the spatial expression of the marker genes in the target space that you're mapping the scRNA-seq data onto, and therefore has to have the same spatial dimension. It's your guide for the spatial reconstruction. The # of cells in the scRNA-seq data is irrelevant, novoSpaRc can map many cells to fewer locations of the target space and vice versa.
Does this clarify your questions?

Thank you for the explanation! Currently, for my own data, I'm running
atlas_matrix = dataset.X[np.random.choice(dataset.shape[0], locations.shape[0], replace=False)][:, markers_in_sc]
to get my atlas matrix. Would this be the right approach?

Thanks for the clarification. In the reconstruct_bdtnp_with_markers.py example, the atlas matrix is attained by atlas_matrix = dataset.X[:, markers_to_use]. If my dataset has more cells than locations, would atlas_matrix = dataset.X[np.random.choice(dataset.shape[0], locations.shape[0], replace=False)][:, markers_in_sc] be the right approach?

In the BDTNP example, the reference atlas coincides with the "scRNAseq" data.
If you have another dataset, you have to construct the reference atlas first. This will define your target space then. And you can map your scRNAseq data regardless of their number.


Its not an issue but need help in sending outlook mail using powershell script

I am an automation engineer
I have have automated few test cases and getting the test results. This test results I wanted to forward to team using outlook using powershell. I tried many ways to do it, but not working.
I want an powershell script to send outlook mail to people who I configure.
Can anyone please help me out in this.

@vishnuprakash9845 Thank you for the question. In order to best address your question, could you provide us with the URL of the MS Doc that you were following, if any?

@vishnuprakash9845 Thank you for the question. In order to best address your question, could you provide us with the URL of the MS Doc that you were following, if any?

@vishnuprakash9845 Could you please provide the requested details?

@vishnuprakash9845 Could you please provide the requested details?

Sure Bhargavi,
Thank you for responding.
Sorry I wasn't able to get back to you sooner, I referred following URLs.
https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.utility/send-mailmessage?view=powershell-7.1
https://gallery.technet.microsoft.com/scriptcenter/Send-HTML-Formatted-email-26925a96
https://gallery.technet.microsoft.com/scriptcenter/bd0c8d87-466f-4488-92a2-0f726cb6f4cd
One small clarification, for sending the mail to team members from outlook mail id (Eg<EMAIL_ADDRESS>should we install microsoft outlook (Microsoft office) in the machine where the Poswer shell script will be ranned.

Sure Bhargavi,
Thank you for responding.
Sorry I wasn't able to get back to you sooner, I referred following URLs.
https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.utility/send-mailmessage?view=powershell-7.1
https://gallery.technet.microsoft.com/scriptcenter/Send-HTML-Formatted-email-26925a96
https://gallery.technet.microsoft.com/scriptcenter/bd0c8d87-466f-4488-92a2-0f726cb6f4cd
One small clarification, for sending the mail to team members from outlook mail id (Eg<EMAIL_ADDRESS>should we install microsoft outlook (Microsoft office) in the machine where the Poswer shell script will be ranned.

@vishnuprakash9845 Thank you for the response.
I'd like to let you know that this channel (the Azure PowerShell Docs repository) is only for driving improvements related to documentation on the Azure PowerShell module. The best place for specific answers about implementations is going to be the Microsoft Q&A and Stack Overflow platforms. We actively look through posts in the forums and will do our best to get you going in the right direction!

@vishnuprakash9845 Thank you for the response.
I'd like to let you know that this channel (the Azure PowerShell Docs repository) is only for driving improvements related to documentation on the Azure PowerShell module. The best place for specific answers about implementations is going to be the Microsoft Q&A and Stack Overflow platforms. We actively look through posts in the forums and will do our best to get you going in the right direction!

@vishnuprakash9845 At this point, it appears that there are no edits or updates being suggested to the documentation. Hence closing this issue for now. If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

@vishnuprakash9845 At this point, it appears that there are no edits or updates being suggested to the documentation. Hence closing this issue for now. If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Added error handling on start trajectory service

I tried to add some error handling functions for start trajectory service for now. If it's not enough, please let me know.

Thanks for adding the error checking! Much appreciated.
I fixed some nits in 03273efe27e463b299c7d96a396ba32dec4de60b. I also renamed Check* to something else, because Check in google-speak means crash if false, i.e. assert.
In 94547dade1c20450ed3384eb5c87d83553cc5e3f I changed the remaining case I saw where a silent error could slip through.
@wohe Also pointed out that we do not use ROS logging anywhere else. We rely on the Glog to ROS bridge that is active in the nodes. So for consistency, I changed all ROS logging to using Glog in 8c053594bc812e859ec21a1930656e891927f75e. Sorry for not noticing during the review.
I think this is good to go. Let me know if you are happy with my changes and I'll merge this in.

LGTM.
Thank you for making it better. I am always excited about your review.


Maya token send error

Describe the bug
The user is claiming he is not being able to send Maya Tokens.
To Reproduce
[Steps to reproduce the behavior:
https://discord.com/channels/1203844257220395078/1239008700065386558/1239054622023352390
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@realpaaao, @johnnyluo , @flypaper0 @Rockindash can you help me to find this error?

Are you able to reproduce the issue?

Are you able to reproduce the issue? it could be round error?

I am not. I've tried all kind of ways. Maybe the issue is because they had a previous version, and they probably need to delete the app or just remove the coins, so it can get the latest TokenStore.Coin.

Removing and adding the Chain solved the problem. Can we automate the check for TokenStore.Coin after new update?


Update cloudtrail schema to 1.08

to:
cc: @airbnb/streamalert-maintainers
related to:
resolves:
Background
AWS releases a new version of the cloud trail schema 1.08 as found in https://docs.aws.amazon.com/awscloudtrail/latest/userguide/cloudtrail-event-reference-record-contents.html
The following adds relevant fields from 1.07 & 1.08 where it seemed appropriate.
I have commented on each field added the relevant AWS documentation which was available.
Exclusions
At this time I have excluded the field edgeDeviceDetails This is exclusive to outpost and I cannot find any log examples to reference and confirm the type.
edgeDeviceDetails
Shows information about edge devices that are targets of a request. Currently, S3 Outposts device events include this field.

Since: 1.08

Optional: True

Changes
Added fields

addendum
eventCategory
insightDetails
sessionCredentialFromConsole
tlsDetails

Testing
No testing due to lack of logs to check against, I welcome examples submissions.

Rebased onto release-3-5-0

@gavinelder FYI - edgeDeviceDetails is documented to be a JSON object by AWS here


Gateway patch release april

Summary
Update the changelog for Kong Gateway 2.6.x, 2.7.x and 2.8.x releases
Testing
https://deploy-preview-3821--kongdocs.netlify.app/gateway/changelog/#2810
https://deploy-preview-3821--kongdocs.netlify.app/gateway/changelog/#2720
https://deploy-preview-3821--kongdocs.netlify.app/gateway/changelog/#2610

@tyler-ball Thanks! Let me know when the packages are released and I'll hit merge on this PR


[jest-30] Improve detectOpenHandles

Summary
Removed some previously ignored detection types.
Ref: https://github.com/facebook/jest/pull/13414

Test plan
CI GREEN

The failure in CI I should know how to fix, but I can't reproduce it locally (win) anyway, which is weird. Even I can't reproduce it using gitpod (linux). 😕

/easycla

Hey @liuxingbaoyu, finally got back to this 😀 Could you sign the CLA? 🙂

Sorry for forgetting this!
Of course, it's done!

Wonderful, thanks!

https://github.com/jestjs/jest/releases/tag/v30.0.0-alpha.1


Spellcheck of the comments in SeaHash

While reading the the documentation I found a couple of minor spelling mistakes.

thx


Update ESMA_env and ESMA_cmake

Description

This PR updates to ESMA_cmake v3.20.0 and ESMA_env v4.7.0. These are really needed by #1810 but for some reason, that PR is failing in the GEOSldas build. So I want to see if the updates to baselibs and cmake are the issue...
Related Issue

Motivation and Context

Let's me figure out what's going on!
How Has This Been Tested?

I mean, this all is zero-diff to GEOS!
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Trivial change (affects only documentation or cleanup)

Checklist:

[x] I have tested this change with a run of GEOSgcm (if non-trivial)
[x] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[x] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

Close in favor of #1810


Update simpleclient_common to 0.12.0

Updates io.prometheus:simpleclient_common from 0.9.0 to 0.12.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (0.9.0).
You might want to review and update them manually.
project/Dependencies.scala

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.prometheus", artifactId = "simpleclient_common" } ]

labels: library-update, early-semver-major, semver-spec-minor, old-version-remains

Superseded by #163.


Classes arranged in a single line for class scope

If the scope is to display only classes, the classes of a package should be arranged in a rectangle, currently it is in a single line, this is inconvenient.

Solved in v0.4.2


Unable to run the project in higher version of xcode i.e with higher swift version than compiled framework version

Fork the project
Then clone the project
On running the project in xcode 11.3.1, it gives the error "Module compiled with Swift 5.0.1 cannot be imported by the Swift 5.1.3 compiler" in file YPFiltersView, on line "import Stevia"

Hi @Shivani-Bajaj please see https://github.com/Yummypets/YPImagePicker/issues/475


Test remove unused if blocks

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build


[YUNIKORN-2305] E2E test: Upload stdout logs to Github Action artifact

What is this PR for?
In the current e2e test, there are 2 types of logs that were mixed up in Github Action Log.

ginkgo.By
ginkgo.GinkgoWriter

This PR write ginkgo.GinkgoWriter logs to local and uploads to Github Action artifact if e2e test failed.
Changes:

Each e2e suite will put ginkgo.GinkgoWriter logs to their local artifact file. (Replace default writer 'os.Stdout' by 'os.File')
Upload artifacts only if any e2e test failed.
Move some ginkgo.GinkgoWriter logs to ginkgo.By because the existing logging is not clear. (No timestamp and spec name. So I keep cluster status in artifact and move others to ginkgo.By, we can create other Jira to refactor it.)

What type of PR is it?

[ ] - Bug Fix
[x] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos
Found some issues, could create other Jira to fix it:

[ ] - Found some e2e test that didn't dump cluster status when the test failed. (recovery_and_restart/persistent_volume)
[ ] - Review exisitng ginkgo.By logs and decide which should be moved to ginkgo.GinkgoWriter.
[ ] - (Optional) Refactor fmt.Fprintf(ginkgo.GinkgoWriter,"") to ginkgo.GinkgoWriter.Printf()

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-2305
How should this be tested?
All existing e2e test should pass.
Could also check Github Action result in my shim repo:
Suceess workflow: https://github.com/chenyulin0719/yunikorn-k8shim/actions/runs/7411324558
Failed workflow: https://github.com/chenyulin0719/yunikorn-k8shim/actions/runs/7411334944
Screenshots (if appropriate)

Questions:
Currently, I upload artifact only if any e2e test failed. Please let me know if anyone suggests changing to 'always upload'.

This needs to be made optional, and off by default as it interferes with running the e2e tests interactively. My suggestion would be to use the existing implementation unless the ENV var for the path is set. This will allow doing the uploads as part of the pre-commit workflow but still allow interactive use.

I agree with @pbacsko, the current logging is fine until an error occurs, and then the dump becomes nearly unreadable, especially locally (but even on GH I've seen it overflow the allowed size). I'd be a big fan of capturing that output to files (within the source tree under output/) and then uploading those to GH if we're running in a workflow. I think separate files for state dump, yunikorn logs, and cluster diagnostics would be ideal.

Hi @pbacsko, @craigcondit

@chenyulin0719 maybe there's a misunderstanding, but I think the default logging from the tests is perfectly fine. What we need to upload is the Yunikorn log and cluster info, which is printed to the the console in the current implementation. That is a LOT of text and it's very inconvenient to read all kinds of output mixed together.
So what we need to do is writing YK logs and cluster info to a separate file (or even better, files) then upload it when the test completes.

I agree with @pbacsko, the current logging is fine until an error occurs, and then the dump becomes nearly unreadable, especially locally (but even on GH I've seen it overflow the allowed size).

Noted, will keep the default logging.
The new thought in my mind is we will have 1 new logging method for file.

ginkgo.By. (to console) (Existing)
ginkgo.GinkgoWriter.  (To console) (Existing)
File Writer (To file) (New) (file path is from github workflow ENV variable, should have default path for local run. ex: /tmp/e2e-test-artifacts)

I should move "state dump, yunikorn logs, and cluster diagnostics" from #1,#2 to #3 File Writer.(Instead of changing ginkgo.GinkgoWriter to file). Please correct me if I didn't understand it well.
The further question is the artifact's file structure.

I'd be a big fan of capturing that output to files (within the source tree under output/) and then uploading those to GH if we're running in a workflow. I think separate files for state dump, yunikorn logs, and cluster diagnostics would be ideal.

@craigcondit In my opinion, the example  in this PR is quite clear, I organized all the file logs into a single file per suite. (Printing the spec name first allows us to search by spec name.)
Keeping one file per suite is my favorite. If this is acceptible I will keep this implementation.

I'd prefer to see a directory per test, with the individual artifacts (cluster dump, YK logs, state dump) separated out because some of these are json, and others are text. Splitting them out allows for tooling to process them more easily. Also, we should not be using static names in /tmp for the output. Instead, place them in output/e2e/{test_suite}/ instead.

Hi @pbacsko, @craigcondit,
New version updated. Each failed test will generate 3 files under build/e2e/{suite}/ (Both for interactive and GitHub Action environments):

{specName}_k8sClusterInfo.txt
{specName}_ykContainerLog.txt
{specName}_ykFullStateDump.json

You can check the artifact in this workflow run:  (I make some tests in plugin mode fail.)
https://github.com/chenyulin0719/yunikorn-k8shim/actions/runs/7456536846 (
Please let me know if there have anything to be improved.

@chenyulin0719 please rebase the PR, there are some minor conflicts.

Hi @pbacsko, @craigcondit,
New version updated. Each failed test will generate 3 files under build/e2e/{suite}/ (Both for interactive and GitHub Action environments):

{specName}_k8sClusterInfo.txt
{specName}_ykContainerLog.txt
{specName}_ykFullStateDump.json

You can check the artifact in this workflow run: (I make some tests in plugin mode fail.) https://github.com/chenyulin0719/yunikorn-k8shim/actions/runs/7456536846
Please let me know if there have anything to be improved.

The approach LGTM. I just found some minor things.

@chenyulin0719 please rebase the PR, there are some minor conflicts.
Hi @pbacsko.
Just rebase and pushed.  e2e test in my side looks good.

Checked the failed test is related to preemption e2e test. I'm curretly troubleshooting it in another issue.
(https://issues.apache.org/jira/browse/YUNIKORN-2313)
https://github.com/apache/yunikorn-k8shim/actions/runs/7420644533/job/20212796775?pr=759#step:6:1483


Can't handle network requests over hls streaming in Safari

Have you read the Tutorials?
Yes
Have you read the FAQ and checked for duplicate open issues?
Yes
If the question is related to FairPlay, have you read the tutorial?

Yes
What version of Shaka Player are you using?
4.3.3
What browser and OS are you using?
Ventura - Safari (v16.3)
Please ask your question
Hello,
I have a system on my Safari browser that is protected with a DRM token and uses a CDN token for search parameters. When the CDN token expires, I regenerate it and intercept manifest requests, continuing with updated token parameters. In the playlists I play with Fairplay HLS, manifest requests are made every 6 seconds, and when the CDN token expires, I need to handle this by generating new token parameters and continuing.
However, on the DASH side, the register methods and the player's error events work correctly, but they do not work on the Fairplay HLS side. I cannot intercept manifest requests in any way, nor can I catch the 410 errors (server response due to CDN token expiration) from the network using the player's provided APIs.
I want to establish this logic on the HLS side as I have done with DASH Widevine. I would greatly appreciate your help.
Thank you.

Hello,
I have made the adjustments as you suggested, but there has been no change. I still cannot handle the requests sent over the network (manifest requests sent every 6 seconds), nor can I handle the 410 error thrown by the server when the CDN token expires. Events like onPlaying, onBuffering, and onRatechange are still working as they should.

Please, use streaming.useNativeHlsOnSafari = false (config option) Thanks!

Hello,
I have made the adjustments as you suggested, but there has been no change. I still cannot handle the requests sent over the network (manifest requests sent every 6 seconds), nor can I handle the 410 error thrown by the server when the CDN token expires. Events like onPlaying, onBuffering, and onRatechange are still working as they should.

I'm sorry, I was wrong, I didn't realize that you're using Fairplay ... try with streaming.useNativeHlsForFairPlay = false but... You have to read the tutorial carefully for FairPlay with MSE; not all providers may support it.

I'm sorry, I was wrong, I didn't realize that you're using Fairplay ... try with streaming.useNativeHlsForFairPlay = false but... You have to read the tutorial carefully for FairPlay with MSE; not all providers may support it.

Hello again, we are using Verimatrix, and our DRM integration is implemented as shown in the tutorials. However, the main thing I want to ask is this: The registerRequestFilter and registerResponseFilter methods we use over the Networking engine are not being handled in any way. This seems to be the main issue. Why are these methods, which are also used in the tutorials, not working at all for Fairplay?

When you use src= (not MSE), the native player is in charge of making the requests, not us. Verimatrix as far as I know does not support MSE, so the behavior you want to achieve is impossible (as far as I know).

When you use src= (not MSE), the native player is in charge of making the requests, not us. Verimatrix as far as I know does not support MSE, so the behavior you want to achieve is impossible (as far as I know).

Hi,
I am not using the src= attribute; instead, I am setting the video element through MSE.
The process of loading the video is as follows:
I get the video element and initialize Shaka Player with this element.
Then, I call the Player.load method with a specific URL to load the video.
I found the following code in your documentation regarding Verimatrix FairPlay support and implemented it:
javascript
Copy code
shaka.polyfill.PatchedMediaKeysApple.install();
const FairPlayUtils = shaka.util.FairPlayUtils;
player.getNetworkingEngine()
.registerRequestFilter(FairPlayUtils.verimatrixFairPlayRequest);
player.getNetworkingEngine()
.registerResponseFilter(FairPlayUtils.commonFairPlayResponse);
player.configure('drm.initDataTransform',
FairPlayUtils.verimatrixInitDataTransform);
However, when I implement this code, the registerRequestFilter and registerResponseFilter methods are never invoked. This indicates that Shaka Player is not intercepting and handling the requests.
Do you have any other suggestions or solutions regarding Verimatrix and MSE support? Is there any additional configuration or setting on the browser side that I need to apply?
Thanks.

According to your code you should be using src= :) Please check the documentation: https://shaka-player-demo.appspot.com/docs/api/tutorial-fairplay.html

Hello,
I set the src= method as suggested, but unfortunately, I didn't notice any change in behavior. I'm still facing issues with playing FairPlay HLS content using Shaka Player. I'm wondering if there's something crucial that I might be missing in my implementation.
Could you please provide any additional insights or steps that could help me troubleshoot this further?
Thank you.

It is not possible to do what you want with Shaka Player with what you comment, sorry!

My goal is to ensure that the registerRequestFilter and registerResponseFilter methods work as expected with FairPlay content in Shaka Player. I need to handle network events effectively for my implementation.
As you suggested , i switched to use src= , nothing changed.

You might be able to accomplish this with a service worker.


Resolves #53

Before using this, make sure you have https://github.com/wildfly/wildfly-core/pull/825 merged into your wildfly-core branch.

Looks good but can you please just update the commit message so it says 'Resolves #53' instead of referencing ELY-213 then I will go ahead an update the projects and merge.

Would be good to merge the wildfly-core commit into the incubator branch to avoid this extra step when setting things up.

Am in the process of rebasing both forks so the commit will be there.

Merged


[#80] Baundrate type implementation

Base impl. for discussion

My suggestion would be to start working on this for one board only (preferably some hardware you have and can test things on).  Once discussion about the design has settled and all problems are ironed out, it can be transferred to all other crates as well.  This way, you don't have to make each change for all the crates every time.

@Rahix tested default on arduino leonardo. Do you have any specific scenarios for tests?
Right now Baudrate will contain some logic which we test with some units. Do we have any rules for this? Should I define some unit tests?

@explicite, my suggestion was to remove the changes from all board code except arduino-leonardo and its example for now.  This way the MR doesn't contain so many changes for now and once everything is looking good, you can readd the changes for the other boards.  Otherwise you'll have to update each file again and again which is a bit cumbersome ...

Do you have any specific scenarios for tests?

We should test all the well known baudrates, ideally also with different core clock speeds.  Not sure how to easily try other clock speeds though (maybe by setting the clock prescaler?).

Should I define some unit tests?

I guess this would be the right thing to do but we don't really have a testing infrastructure yet ... I think manual testing is enough for now.

Oh, and feel free to remove the "Draft" state from this PR :)

@Rahix  should I implement same changes for other chips and boards?

@Rahix  I think all fixed as required. Should I made any changes/testing?

Looks like you'll need to fix merge conflicts/rebase your branch once more, then I can merge it :)


Update Integraality lists to include email, and phone numbers

The feature
The Integraality lists should include stats about phone numbers and email addresses.
Why?
We currently include popular social media platforms in the stats but to our users, email/phone numbers are just as important and we should signal this to data contributors.
Examples and inspiration
No response

Here is an example of a dashboard that becomes relevant once the group of agencies has three stars or more: https://www.wikidata.org/wiki/Wikidata:WikiProject_Govdirectory/Sweden/Coverage_of_second_level_agencies

I have added documentation now.


Fonts name should be in somewhere

This info should be at each README.md
xmb/flatui/font.ttf  (Open Sans Regular)
xmb/monochrome/font.ttf  (M+ 1p Regular)
xmb/neoactive/font.ttf  (Metrophobic Regular)
xmb/retroactive/font.ttf  (Metrophobic Regular)
xmb/systematic/font.ttf  (Metrophobic Regular)
glui/font.ttf  (M+ 1p Regular)

Along with the font licenses too.

Let me know if this works for NeoActive, RetroActive and Systematic.  I simply provided the basic font information with a link to the font license.

Looks great!

xmb/monochrome/font.ttf  (M+ 1p Regular)
I just took this one for dot-art. @ #124

I added the M+ font info to the Monochrome README.md file along with fixing a spelling mistake for theNeoActive, RetroActive and Systematic font info.  Pixel and FlatUI currently do not have a README file.  I'll leave that to @CoalaJoe and @RobLoach to add as they continue to make progress on those themes.

@sergiobenrocha2 I went ahead and made minimal Read Me files for all the themes in XMB plus the GLUI directory.  These files contain font info as requested.  I believe this addresses your issue, but if you could vet it, I would appreciate it.

I'm closing this issue as it appears to be addressed.  If you feel this issue is not resolved, please let us know.


Add defaultValue option to variables in text

Description
I would like there to be an option to set a default value in variables in text resources. This would allow us to provide sensible default whenever the data model returns a null value for a given path.
Additional Information
Related issue: #754. The issue that is reported here can be resolved by providing empty strings as default values.

Duplicate of #754?


docs: update the version compat matrix for KIC 2.0

[x] Ensure that the version compatibility matrix is updated for KIC 2.0.

Resolved by https://github.com/Kong/docs.konghq.com/pull/3081

This is resolved in that the documentation is written, but there is a follow up for the final merge to main: https://github.com/Kong/kubernetes-ingress-controller/issues/1719

Blocked on https://github.com/Kong/kubernetes-ingress-controller/issues/1683
Close this only after:

the blocker (1.22 tests) has been resolved
the 1.22 row has been added to the compat matrix

Tentatively added the new table row: https://github.com/Kong/docs.konghq.com/pull/3099/commits/2b4b23abea5b9a99edd10a69da2f94175c7e9424

Preview is working properly, 1.22 available for 2.0.0 (but not earlier): https://deploy-preview-3099--kongdocs.netlify.app/kubernetes-ingress-controller/2.0.x/references/version-compatibility/

Done


⚠ Bofrost.de has degraded performance

In 22c6ac0, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2504 ms

Resolved: Bofrost.de performance has improved in 53b30a2.


Make zero as invalid value for message_content_delete_limit_seconds.

We make zero invalid value for message_content_delete_limit_seconds and
for handling the case of "Allow to delete message any time", the API-level
value of message_content_delete_limit_seconds is "anytime" and "None"
as the DB-level value.
Testing plan:  Updated the tests accordingly.

@timabbott this is ready for review. Have made some inline comments.

@timabbott this is ready for review.

I merged the prep commits through here: d09da692c668ddd40bd088c97c1156db067c06f9.

I did my first full review of this, and I think it looks great!
I'm not sure the semantics are quite right for the "Moderators not affected by time limit" stuff, and am worried about the explanation.  Are we planning to add a delete_any_messages setting?  Maybe we just want to not change the moderator situation in this PR, and just have a "Who can delete all messages" setting with a default of Moderators?  And that second setting would just communicate clearly that the time limit only applies to the "delete your own messages" feature.

Yes, I have some work done on the delete_any_message part. I have removed the moderators don't have limit part from this PR.

If we want to set the default for setting of Who can delete all messages to Moderators then we should also change the default of this setting to Moderators for the reasons described here.

I merged #18528; can you do a rebase on this following that?  I'll also do a review now.

So, I'm having second thoughts about the name anytime for the string here, which will be part of the long-term API for this endpoint. I pushed back to this PR after some small edits, and will start an "api design" thread.

@timabbott do we still want to add this setting or hold it till we do the Groups permissions work?

I think this setting split is still useful as preparation for that.

@timabbott Updated the PR. Done all the changes as asked above and also renamed anytime to unlimited.
Also, as you asked above to put the condition for checking whether the value is everyone in user_has_permission in frontend, I have done it but I think there can be confusion because everyone value is there only in common_message_policy_values and not common_policy_values and other conditions in user_has_permission use common_policy_values object. Logically it will work fine but there can be problems due to readability. If we are fine with this, I can also change other similar functions.

Previous pushes was only for rebasing, last changes done are mentioned in the quoted comment.

Merged, sorry for all the delay on this one!  Thanks for doing this migration and cleanup.
I made a few changes before merging:

Added the final assertion commit in the settings_data code.  (We probably should make use of similar assert statements in similar code paths).
Adjusted several strings in the API documentation and the labels for the the setting.

We should remember to cross-reference the API documentation descriptions of "Who can delete any message" with this setting when we add that.

I reverted that last commit; see 171703e4803ad3f0095b8ca6bdede4d740e0bd68.  I would like us to do a proper version of that, which might involve restructuring these policy check functions to not try conditionals the way that they presently do.
(E.g. perhaps we should just have a switch statement for the policy value, and then do the test for that policy within each case).


How can I select an element that appears below another element?

How can I select an element that appears below another element?

Try holding down the ctrl/cmd key when you click on the element.


Run name mismatch between uploads and results folders

While working on #69, I found that I was not able to associate run names between /data/wastewater/uploads and /data/wastewater/results for a substantial number of runs.  For example:
art@Langley:~$ ls /data/wastewater/uploads/guelph/220121_MN01421 | wc -l
ls: cannot access '/data/wastewater/uploads/guelph/220121_MN01421': No such file or directory
0
art@Langley:~$ ls /data/wastewater/results/guelph/220121_MN01421 | wc -l
93

Is it possible that the run folders have been renamed?
art@Langley:~$ ls /data/wastewater/uploads/guelph | grep MN
20220121_MN01421
20220126_MN01421
20220131_MN01421
20220210_MN01421
20220513_MN01421
art@Langley:~$ ls /data/wastewater/results/guelph | grep MN
20220121_MN01421
20220126_MN01421
20220131_MN01421
20220210_MN01421
20220512_MN01421
20220513_MN01421
220121_MN01421
220126_MN01421
220131_MN01421
220210_MN01421

Old entries (e.g., 220121_MN01421) should be removed from results
Inquire with Guelph lab about what to do with missing run folder 20220512_MN01421

[ ] Move old files from results into some backup folder, just in case

Moving the results for the following runs into a separate backup folder
guelph/220131_MN01421
guelph/20220512_MN01421
guelph/220121_MN01421
waterloo/run22_GTAA
guelph/220210_MN01421
guelph/220126_MN01421
guelph/20101020_GB2point2


fmt or comparison fault

while tinkering I did something that isn't technically valid:
[use "fmt"]

[var x ""]

[if [!= 0 [fmt.expand x]] [block
    [putln "If you see this there wasn't a segfault"]
]]

This should have thrown an exception as the result of fmt.expand is a list, but instead I got a segfault.
The proper code is :

[use "fmt"]

[var x ""]

[if [!= 0 [len [fmt.expand x]]] [block
    [putln "If you see this there wasn't a segfault"]
]]

which works fine. Figure out where the oob issue is from the first code snippet and repair

https://github.com/bosley/sauros/pull/74


Add CreatePackageRevision Test

create Git repository server for the test
create a package (clone from upstream)

Build and/or tests may fail on this one since it depends on few other PRs #2797 and #2799.
I rebased it on #2797 only to make it easier to review.


No wheel for latest version 6.6

setup.cfg has the wheel config, but python setup.py bdist_wheel upload wasn't run for the release, so there's no wheel up at https://pypi.python.org/pypi/click/6.6 . Please add one to make everyone's installs a bit faster 👍

I'm not sure it's possible to upload a wheel without uploading a sdist. We can't just run this command on the version tag, that's for sure.

Hmmm I'm pretty sure I've uploaded wheels after sdists in the past on other projects. Make sure you python setup.py clean first?

@untitaker It is, you can do something like the following:
$ git checkout <the version you want to upload wheel for>
$ python setup.py bdist_wheel
$ twine upload dist/click-*.whl # where this is the path to the wheel

Note, you need twine (pip install twine) the PyPA tool https://github.com/pypa/twine

You can also just do 'python setup.py clean bdist_wheel upload'
On 19 November 2016 at 18:48, Kyle Fuller<EMAIL_ADDRESS>wrote:

@untitaker https://github.com/untitaker It is, you can do something
like the following:
$ git checkout
$ python setup.py bdist_wheel
$ twine upload dist/click-*.whl # where this is the path to the wheel
Note, you need twine (pip install twine) the PyPA tool
https://github.com/pypa/twine
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pallets/click/issues/671#issuecomment-261728422, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA0WCfLA-Yy5oiKLbwyWJT2y_P0KMQtlks5q_zaLgaJpZM4KV_GZ
.

--
Adam

upload tries to upload an sdist which fails

Oh ok my bad
On Sat, 19 Nov 2016 at 19:00, Markus Unterwaditzer<EMAIL_ADDRESS>wrote:

upload tries to upload an sdist which fails
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pallets/click/issues/671#issuecomment-261732642, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA0WCS6_i0D8JbbnLLZSVOe066lawPYFks5q_0dTgaJpZM4KV_GZ
.

Twine does not have this limitation. The steps I provided would still work.

Twine appeared to have worked, thanks @kylef! 6.6 now has a wheel.


Possibility of rosifying the process

@jbehley @tano297
Hallo, thank you for your excellent work and making the code open source. Students in the field of robotic vision such as myself are always fascinated by such work. Also i have tried to contribute by helping out with the remaining open issues.
So far everything has worked, however when fed with a custom dataset, the results are not so great. Would it be possible to rosify the process so that feeding in data could be efficient so as to debug why the algorithm does not work so well with a certain dataset.
For @XuRobotics,
Hi. I have noticed your work and i was wondering if you had used the ouster raw lidar data with this algorithm, if so, how were the results for you ?
Best regards,
Rohith

sorry for the late reply.
It's certainly possible to put the code into a ros node, but we currently have no plans to perform this.
Regarding the performance on custom datasets: Unfortunately, the performance of the range image-based approaches is strongly affected by the sensor configuration (i.e., the extrinsic calibration), which can lead to very different patterns. We had also a paper on this "domain gap", where we showed that the model trained on SemanticKITTI deteriorates when applied to nuScenes (see Langer et al.) Therefore, domain adaptation approaches or different scan representations (i.e., voxel-based approaches) are maybe a way to go to close the "domain gap".
Langer et al. Domain Transfer for Semantic Segmentation of LiDAR Data using Deep Neural Networks, IROS, 2020. http://www.ipb.uni-bonn.de/pdfs/langer2020iros.pdf


Separate the floors

Hello, I would like to know if in web-ifc-three, there is some way to separate the IFCBUILDINGSTOREY, at a configurable distance, something like this:

🟢🟢🟢

::take

Hi, @aka-blackboots! Thanks for taking this bounty! The due date is December 19, 2022 UTC.
If you need to submit some pull requests (PR) to complete the tasks, make sure that the last and only the last PR has a title that either starts with the bounty ID or is exactly the same as the bounty name. After the PR is merged, this bounty’s status will automatically changed to done.
If you do not need to make a PR, tell the manager @agviegas to run ::done command after your tasks is confirmed to be done.
Good luck!

🟢🟢🟢

::drop

Hi, @aka-blackboots! Thanks for giving it a try! It’s now once again available for anyone to take.

::take

🟢🟢🟢

::take

Hi, @antoniohof! Thanks for taking this bounty! The due date is March 2, 2023 UTC.
If you need to submit some pull requests (PR) to complete the tasks, make sure that the last and only the last PR has a title that either starts with the bounty ID or is exactly the same as the bounty name. After the PR is merged, this bounty’s status will automatically changed to done.
If you do not need to make a PR, tell the manager @agviegas to run ::done command after your tasks is confirmed to be done.
Good luck!

Dear elias2089,
I managed to do it like this:
`            this.model = this.IFCManager.scene.ifcModel
this.spatialStructure = await this.IFCManager.scene.ifcModel.getSpatialStructure()
        // storeys are in this hierarchy
        const floors = this.spatialStructure.children[0].children[0].children

        const floor1ids = []
        floors[0].children.forEach((c)=> {
            floor1ids.push(c.expressID)
        })

        const floor2ids = []
        floors[1].children.forEach((c)=> {
            floor2ids.push(c.expressID)
        })

        // we don't want the subsets to appear directly into the scene, we first need to clone them, so pass a dummy object3d in the scene parameter
        const emptyObject = new Object3D()

        // create subset of first floor
        const floor1 = this.IFCManager.scene.ifcModel.ifcManager.subsets.createSubset({
            modelID: modelID,
            ids: floor1ids,
            material: null,
            scene: emptyObject,
            removePrevious: true,
        });

        // clone subset of first floor and add to threejs scene
        const floor1Mesh = new Mesh( floor1.geometry.clone(), floor1.material );
        this.IFCManager.scene.scene.add(floor1Mesh)

        // create subset of second floor
        const floor2 = this.IFCManager.scene.ifcModel.ifcManager.subsets.createSubset({
            modelID: modelID,
            ids: floor2ids,
            material: null,
            scene: emptyObject,
            removePrevious: true,
        });

        // clone subset of second floor and add to threejs scene
        const floor2Mesh = new Mesh( floor2.geometry.clone(), floor2.material );
        this.IFCManager.scene.scene.add(floor2Mesh)

        floor2Mesh.position.set(0,5,0)

        // remove original model's mesh
        this.IFCManager.scene.scene.remove(this.IFCManager.scene.originalModelMesh)

`

Let me know if that helps you,
If you want I can upload a demo with that.
All the best,
Antonio

@antoniohof The end goal over here would be integrate this as a functionality so that other user can directly use it as an API from web-ifc-three

I see! I'll try to do that soon and send a PR

ah wait, I found config.customID in createSubset. Then it's much easier.

this.spatialStructure = await this.IFCManager.scene.ifcModel.getSpatialStructure()

            // storeys are in this hierarchy
            const floors = this.spatialStructure.children[0].children[0].children

            const floor1ids = []
            floors[0].children.forEach((c)=> {
                floor1ids.push(c.expressID)
            })

            const floor2ids = []
            floors[1].children.forEach((c)=> {
                floor2ids.push(c.expressID)
            })

            // create subset of first floor
            const floor1 = this.IFCManager.scene.ifcModel.ifcManager.subsets.createSubset({
                modelID: modelID,
                ids: floor1ids,
                material: null,
                scene: this.IFCManager.scene.scene,
                removePrevious: false,
                customID: 'floor1'
            });

            // create subset of second floor
            const floor2 = this.IFCManager.scene.ifcModel.ifcManager.subsets.createSubset({
                modelID: modelID,
                ids: floor2ids,
                material: null,
                scene: this.IFCManager.scene.scene,
                removePrevious: false,
                customID: 'floor2'
            });

            floor2.position.set(0,5,0)

            // remove original model's mesh
            this.IFCManager.scene.scene.remove(this.IFCManager.scene.originalModelMesh)

What about a function in IFCManager.ts called "createSubsetsOfType(config: TypeSubsetsConfig, IfcType : string)"?
It can returns an array of the subsets of a specific ifcType. (e.g IFCBUILDINGSTOREY).

Thank you very much @antoniohof , sorry for the delay, in the end I generated a subset for each BUILDINGSTORE and each one with its own customID, and then I used a gsap library that allows me to move the subsets with a transition of certain seconds, depending on the position of a slider.
Slider:

Ifc:

Using gap, passing the slider position
timeline.to(subset.position, { duration: 0, y: position });

It's quite easy to do it with the current API, do we really need to have a separate method for this? @agviegas

🔴🔴🔴
Hi, @antoniohof! The date is past due, your assignment has been revoked. It’s now available for anyone to take.

@antoniohof you are right! I'm going to shut this down.


Custom Module repeating fields

Is there documentation on how to create fields that can be repeatable. Mainly the "add new slide" feature. https://cl.ly/0p3H1N3U421R
What i would like to build is a basic touch enabled slider since the default slider only has the fading transitions.
a example like the default hello world (php and jsx) would be sure nice.

Only by creating child_slug object.
Check in CustomCtaChild in https://github.com/elegantthemes/divi-extension-example/tree/master/includes/modules/CustomCtaChild
So, you create main element like antonkorSlider with full config and antonkorSliderSlug with repeatable fields (suppose: background, text, photo etc.).

Each of those "repeatable fields" are child items. So, for the slider there are two linked files, the slider.php file and the slideritem.php file. In the Divi Extension Example on GitHub you can follow the example of the CustomCtaParent and CustomCtaChild modules to create a new slider.

It would be great to have repeater fields though. For example in Elementor you can have more than one repeater per module which is really awesome. Multiple childs per module would be great addition to Divi as well

Thanks guys.
I got the example module working by also including the parent module CustomCtaParent. I guess i can reverse engineer things from here.


⚠ TADA Wallet Web has degraded performance

In c07ec7a, TADA Wallet Web ($WALLET_WEB_URL) experienced degraded performance:

HTTP code: 200
Response time: 16202 ms

Resolved: TADA Wallet Web performance has improved in 596caf7.


redo CodeCell class

rename BeakerCodeCell -> CodeCell
make sure all its fields are filled out by beakerx.getCodeCells('tag')

give it a decent representation


[Feature] Add Chat Typing Indicator

Purpose
This PR added typing indicator presentation layer:

https://user-images.githubusercontent.com/109105353/197132130-211bb94a-4ec5-4433-bf02-0f95e5616d1b.mov
Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type
What kind of change does this Pull Request introduce?

[] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

hard code participant list as the following:

send actions like these:

Test the code

Other Information

UI behaviour validation pending with Alex (wording for 2/3/+ participants and number of max avatar shown)

merging this since I had enough approvals prior to resolving conflicts and Jim is OOO.


Issue 219 calculated rates base class

Created for ci

This in progress PR obsoletes #228 right?

@JamesRamm anything obvious sticking out to you as to why this branch is failing

This in progress PR obsoletes #228 right?

Yes but I do not seem to be able to close it

@JamesRamm let me know if you are happy with this and I will make the merge

Yeah we can go ahead

@JamesRamm Just replaced laptop that got crushed. Trying to make this merge.  Is the current master requirements.txt correct?  Had to manually resolve the conflict.

@JamesRamm GitHub UI will not allow me to merge without an approved review. Do you prefer to merge with the UI or should I manually merge and push?

Created development branch and merged this PR into it with commit e7b1652b51cbb6c47df92827e53bb1ddb994b28a

Eh, just not allowed to merge this either way:
Output from git:
Repository https://github.com/JamesRamm/longclaw.git
protected branch hook declined
error: GH006: Protected branch update failed for refs/heads/master.
error: At least 1 approving review is required by reviewers with write access.

Ok merged! I will test and resolve any issues

Hi
Hows it going? I don't know if you ever managed to get any proper access,
so I have changed the branch restriction to just requiring the Travis build
to pass before it can be merged. Previously it required a review and maybe
there was extra permissions on who can review preventing you from being
able to do this?
In any case, I am pretty sure that now you can merge anytime (as long as
Travis passes).
I don't know if I need to/am able to give you admin access to Travis also?
It may require that I setup an 'organization' for Longclaw.
On Wed, 30 Oct 2019 at 14:40, thenewguy<EMAIL_ADDRESS>wrote:

Eh, just not allowed to merge this either way:
Output from git:
Repository https://github.com/JamesRamm/longclaw.git
protected branch hook declined
error: GH006: Protected branch update failed for refs/heads/master.
error: At least 1 approving review is required by reviewers with write
access.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JamesRamm/longclaw/pull/230?email_source=notifications&email_token=AAVH4N7QJW5RCGP3LZFLIALQRGFDBA5CNFSM4HAUCIS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECUGR2Y#issuecomment-547907819,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVH4N246HOH4TRZRLYXGETQRGFDBANCNFSM4HAUCISQ
.

@JamesRamm sounds good.  I wasn't able to before.  I will make some time to try it out soon


Update README.md with global dependency versions

Update to match versions used in documentation for: https://github.com/OpenZeppelin/starter-kit/issues/62
npm install -g<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Thanks a lot!


Support only preloading

Add function to only preload an already populated/initialized model. Very small and simple change. Using to preload simple fields of an model with custom sql on a performance-critical context.

Hello @gstvg
Could you let me know your use case?  and can the Related method help you?

Hello @jinzhu, thanks for replying.
Use it for querying products on category page. Its a custom query working on a slice of Products.
So, i couldn't use Related since it would require a loop with 1 query for every item, right? Using this pull, i can use performance optimizations from Preload. Thanks again!

Thank you for your explanation, that makes sense.


failed to convert model with "FusedBatchNorm" to TFLITE format

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
NO
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
14.04
TensorFlow installed from (source or binary):
both source and binary tried
TensorFlow version (use command below):
1.4
Python version:
2.7
Bazel version (if compiling from source):
7
GCC/Compiler version (if compiling from source):
4.8
CUDA/cuDNN version:
8.0-5.1
GPU model and memory:
gtx1080-8G
Exact command to reproduce:

###Problem###
My original model is with node "FusedBatchNorm", when I run the script  bazel build tensorflow/contrib/lite/toco:toco and bazel run --config=opt tensorflow/contrib/lite/toco:toco -- --input_file=/home/wz/Desktop/rt-mobilenet.pb --input_format=TENSORFLOW_GRAPHDEF  --output_format=TFLITE --output_file=/home/wz/Desktop/tflite-pose.lite --inference_type=FLOAT --input_type=FLOAT --input_arrays=image --output_arrays=Openpose/concat_stage7 --input_shapes=1,368,368,3
Everything seems fine, but the result file is empty. And I traced the code, founding that code returned at file 'resolve_constant_binary' 's function 'EvaluateBinaryOperatorOnConstantInputs'.
My model can be get here.
God help me! Thanks a lot!

It looks like you only provided a checkpoint. Could you provide python source code for the model?

It has been 14 days with no activity and this issue has an assignee.Please update the label and/or status accordingly.

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

Closing this one. Please reopen if you are still having problems.


[PWS] Check page has some content before parsing

Go to http://url-caster.appspot.com/webui, enter http://www.orange.fr and enjoy seeing the final page being a blank page which used to raise an exception in GetContentEncoding().
Hopefully this patch addresses this issue.

Thanks Francois.  I noticed some 500's only our PWS due to this.
However, while this will prevent the 500, it will still result in non-cached endpoint.  I think this is fine, but may require more investigation..
Also, we should handle more than just empty reply, we need to handle all manner of malformed/missing replies.


pgcli: 3.3.0 -> 3.3.1

Motivation for this change
Update to latest release https://github.com/dbcli/pgcli/releases/tag/v3.3.1
Also needed to update cli-helpers: https://github.com/dbcli/pgcli/commit/a981564f9d0d67e9b583e2bff8f24f613084fbba
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

6 packages updated:
litecli mycli pgcli pgcli python310Packages.cli-helpers (2.2.0 → 2.2.1) python39Packages.cli-helpers (2.2.0 → 2.2.1)

6 packages built:
litecli mycli pgcli python310Packages.cli-helpers python310Packages.pgcli python39Packages.cli-helpers

@GrahamcOfBorg eval


Fix android codeblock rendering inconsistency

Relevant Issues

Closes #615
Description

Added newlines before and after codeblocks that didn't have them already. From testing it seems they're only technically required before to fix the android rendering issue, but I added them afterwards as well for consistency.
Did you:

[x] Join the Python Discord Community?
[x] Read all the comments in this template?
[x] Ensure there is an issue open, or link relevant discord discussions?
[x] Read and agree to the contributing guidelines?

For the eggs I don't think there should have been any leading whitespace so I removed that too.


Problem with IE11

The directive dont work well with ie and with chrome and firefox yes. he dosent display the time.
i get this error only in ie:
Error: Invalid argument.
at Anonymous function (https://localhostvendor/angular-timer.min.js:57:24)

relates to https://github.com/angular/angular.js/issues/8659


Enable -Wc++11-compat-reserved-user-defined-literal when building with clang on linux

This allows catching issues that causes the build to fail on MSVC 2015 RC,
without having to include such a configuration in travis.
Currently including an extra commit that should trigger a travis failure, to test that things behave as intended, I will remove that commit once the travis build shows that things work.

Removed the extra commit, when travis seemed to report it as an error as intended. This is now good to be reviewed and merged if seemed acceptable.
Review at https://rbcommons.com/s/OpenH264/r/1209/.


Feature Request: Granularity of fired events

As we discussed in the other issue, there's need for granularity option for events. I guess there's use case for setting this in every event. For instance, I would love to have instant response when I resize circle. It's what we're used to - reactivity that comes with vue. In circle, this could be implemented as step-size="1000" (in meters), which would also change step of drag/circle expansion.

I agree. The issue is this granularity changes with zoom. If you say 1000m, you cannot move it on zoom 22, but on zoom 0 it will be every pixel. As I said, I made a formula, but that was me testing, writing down step-size against zoom level, and then finding a formula that was close to those points. Do you have any ideas for how to overcome this?

I have the same issue now with the circles and markers.
I think it's simple to change
return Math.abs(newPosition.lat - oldPosition.lat) > 0.001 || Math.abs(newPosition.lng - oldPosition.lng) > 0.001

So that the number 0.001 can be changed by an option from somewhere.
I hope this change will be available soon.
Thanks.


pkg/components/aws-ebs-csi-driver: fix building

In f0f21f383e8e6ddfe43ef5ecd065da7ea66ea0e0 we removed
util.RenderTemplate function and replaced it with template.Render.
However, due to base branch being outdated, this broke buliding master
branch, so this commit fixes it.
Signed-off-by: Mateusz Gozdek<EMAIL_ADDRESS>

I will wait until test pass, but change looks fine to me.


can now run local commands

Allows you to run local commands on the provisioner.

Merged, thanks!


add gh-projects

https://github.com/github/gh-projects official extension for managing Projects V2

@mntlty thank you! its been added.


Ipaddress parse fix to handle X-Forwarded-For with ports

I had the same problem as https://github.com/stefanprodan/WebApiThrottle/issues/34 with X-Forwarded-For containing port numbers. This pull request adds add check for port numbers and removes them before using the same parse logic as previously.
It also adds a test project for the parse code.

What about Ipv6?

What about Ipv6?
the simplest test case ::1 would get pretty butchered, yes. Thanks, I'll submit an update!

Also IsPrivateIpAddress does not handle Ipv6 I think.

added ipv6 handling

Thanks, your fixes are included on v<IP_ADDRESS> and I've deployed the packages on NuGet.


Can we get an update?

The repo has issues above Node 14, which will no longer be supported three months from now. I also had to do some serious work fixing dependencies, but this is a great builder.io integration for nextjs. I think a seamless install will push many people towards using builder.io

Updated to next 13 and latest packages


[QUESTION] Release Date for 2.0?

Alex, we want to port Parallax 2.0 to our platform. I think it is an excellent idea to make it cross platform. Do you have a planned date for the release?

Awesome, thanks


Does not highlight Fenced code blocks in Markdown in Web Worker

test.md

**BOLD**
``` javascript
var foo = bar;
function test() {
  console.log("hello");
}
```

components/prism-markdown.js and Prism.highlight(text, grammar) do not seem to produce highlighted fenced code blocks code in Web Worker.
If i delete prism-markdown.js#L160-L162 then everything work correct:
Prism.hooks.add('after-tokenize', function (env) {
	if (env.language !== 'markdown') { <-delete
		return; <-delete
	}  <-delete

Could you please tell me the value of env.language?
I'm asking because I can image 2 possible values right now:

"md": The after-tokenize hook can't handle the language-md alias ATM. (I will fix that soon)
undefined: This means that Prism.highlight was called without (or with an invalid) language parameter.

Hi RunDevelopment, this is what i got:
input:
Prism.hooks.add('after-tokenize', function (env) {
	console.log(env);
	if (env.language !== 'markdown') {
		return;
	}

output:
...
grammar: {comment: //, blockquote: {…}, code: Array(3), title: Array(2), hr: {…}, …}
language: undefined
tokens: (20) [_.Token, "↵",
...

Without Patch:

With Patch:

The problem seems to be that the Worker doesn't get the language. env.language isn't supposed to be undefined.
I cannot reproduce your problem. Workers are working fine on my side.
Could you please provide a full example reproducing the bug?
The only real cause for env.language == undefined I can image is plugins. Some hook is probably accidentally overwriting env.language which leads to this bug.
So, what plugins are you using? Did you add some custom hooks?

I make a simple test without to use web worker and plugins and got the same result:
example.html
<!DOCTYPE html>

    <title></title>

[![test](https://example.com)](https://example.com)
**BOLD**

```python
if True:
    print "hi"
`` ` (note: 3 back ticks w/o space)

``` javascript
var foo = bar;
function test() {
  console.log("hello");
}
`` `(note: 3 back ticks w/o space)
**BOLD**
## 12345

Log without Patch, no highlighting in fenced code blocks:
examples.html:36 [![test](https://example.com)](https://example.com)
**BOLD**

```</span><span class="token code-language">python</span>
<span class="token code-block">if True:
    print "hi"</span>
<span class="token punctuation">```

```</span><span class="token code-language"> javascript</span>
<span class="token code-block">var foo = bar;
function test() {
  console.log("hello");
}</span>
<span class="token punctuation">```
**BOLD**
## 12345

Log with patch, correct highlighting in fenced code blocks::
examples.html:36 [![test](https://example.com)](https://example.com)
**BOLD**

```</span><span class="token code-language">python</span>
<span class="token code-block language-python"><span class="token keyword">if</span> <span class="token boolean">True</span><span class="token punctuation">:</span>
    <span class="token keyword">print</span> <span class="token string">"hi"</span></span>
<span class="token punctuation">```

```</span><span class="token code-language"> javascript</span>
<span class="token code-block language-javascript"><span class="token keyword">var</span> foo <span class="token operator">=</span> bar<span class="token punctuation">;</span>
<span class="token keyword">function</span> <span class="token function">test</span><span class="token punctuation">(</span><span class="token punctuation">)</span> <span class="token punctuation">{</span>
  console<span class="token punctuation">.</span><span class="token function">log</span><span class="token punctuation">(</span><span class="token string">"hello"</span><span class="token punctuation">)</span><span class="token punctuation">;</span>
<span class="token punctuation">}</span></span>
<span class="token punctuation">```
**BOLD**
## 12345

There's the problem:

Prism.highlight requires that you also pass the name of the language as the third argument. That's why the language is undefined. So Prism.highlight(text, Prism.languages.markdown, 'markdown') will work.
I can't blame you. The docs don't mention this.

Fixed the docs. #1774

Thank you, so important that piece of the 3. phantom.


hello-world

Description

Configuration

Regression?

Other information

@Paddy613 did you mean to report an issue with .NET?


bump dependencies

Applied mkpr on undefined
/tmp/mkpr.sh

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Just fixing some typos

Just fixed a typo.  Nothing else.

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

@lotekmedia : Thanks for your contribution to the Azure documentation! The author, @cherylmc, has been notified to review your proposed change.

#sign-off  Thanks, @lotekmedia

@tysonn  please merge this PR.


之前能用，今天报错：Error: Process completed with exit code 1

之前能用，今天报错：Error: Process completed with exit code 1，请问该如何解决哈

补充一下：详细的提示为：Error response from daemon: pull access denied for xxxxx/yyyyy, repository does not exist or may require 'docker login': denied: requested access to the resource is denied


[Feat]: Add Accessories page to the webpage

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I want to add Accessories section to the website with trendy accessories that users can buy.
Use Case
It makes very attractive when people can buy the matching and trending accessories along with the clothes.
Benefits
Attracts more visits to the webpage
Priority
High
Agreements

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC contributor
[X] I'm a Hacktoberfest contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Proceed!

@hars-21 @Tejashri-Taral I would be grateful if you can please change the level to level 3 in PR #442  as I have taken much efforts to search for the right accessories to match with the theme of the webpage and have created a new feature altogether and added a whole extra page in the website.
Thank you


Copy track update mode when adding reset key

When adding a reset key (by right-clicking a key in animation track), the value is copied over to RESET animation, but not the update mode, and always created continuous track. After AnimationMixer changes this can result in a warning spam if the update mode does not match.
This PR ensures that when creating a reset key from an existing animation track, the track's update mode is copied to the RESET track (even if it already existed).

InterpolationType and LoopWrap should be copied in the same way

Should I do it in the same PR?

I think should be done if possible.

Thanks!

Cherry-picked for 4.2.2.


Follow symlinked directories (fixes #28)

@toddanglin any information on the changes of this being merged? As far as I understand this is a simple fix for a regression, so I would think this wouldn't be controversial. Please let me know if there is something that I'm missing (which of course is entirely possible).

Thanks @ludwiktrammer! Sorry fro the delay. Didn't get a notification for original PR.
Merged in to master. Will bump NPM package in a moment.

Thank you!