SAML Front Channel Logout and clearing cookies in sites on different domains

Currently in our project, we have sites that are on the same domain, let's say example.com. All these sites are integrated with an Identity Provider (IDP) using SAML, enabling Single Sign On (SSO) and Single Log out (SLO) mechanisms. The IDP is also on the same domain, idp.example.com. These sites use a cookie to indicate if a user is logged in. When a user logs out from one of the sites, the IDP initiates a Front Channel Logout with the other sites. It displays a page with a hidden iframe containing a SAML Logout request. In response to the SAML Logout request, we clear the cookie that was created during login at the site level.
The plan is to have sites on different domains than example.com, and then also the IDP. In the logout flow, the cookie that we want to clear will be treated as a cross-site cookie, and the browser will not allow it to be cleared. As a result, the user will be logged out from the IDP and the site where they triggered the logout, but they will still be logged into sites that have a different domain and where the clearing of cookies was blocked. For example, this would occur for site1.newexample.com.
What is your advice here? How could we clear the cookie from the site?

Thanks for reaching out! It would be good if you could help me understand your setup: Do you control both the IdP and the Relying Parties (the top-level sites)?
As I understand it there is an interest in finding developers who would be interested in trying out front-channel logout through FedCM, but that would require a certain level of control over the IdP (support FedCM endpoints) and the RP (integrate with the RP via FedCM and support front-channel logout via FedCM).
cc @samuelgoto

Thanks for response.
My primary intent is to determine if there is a simple solution or plan that can accommodate the scenario I have described.
We are controlling Relying Parties but IdP will not support API that only works for one browser. Also FedCM sounds as a bigger change in our environment regarding a way to only clean cookies on front channel logout.
Do you have other rehomendations?

I'd like to ask some questions to get a clearer picture for your use case:
If I understand your use case correctly, before your plan of change, all the sites are on the same domains, i.e.  a.example.com, b.example.com, c.example.com ... and they all rely on idp.example.com for logging in. When a user logs out from a.example.com, the page would navigate to the idp.example.com, which embeds a hidden iframe that sends a SAML logout requests (that clear log in cookies) to a.example.com as well as other sites, so that logging out from one site also logs the user out from other sites. Is this description accurate?
And the plan is to have sites on different domains other than example.com, i.e. now there will be example-a.com. example-b.com, example-c.com ... and example-idp.com - how many sites will there be in total?

The description is almost perfect, with one difference: the iframe will not send a logout request to the site that triggered the logout (a.example.com). The rest of the description is accurate.
Regarding the number of sites, there could be up to about 10 different domains involved.

@johannhof — any further follow-up here?

No updates at this point from me - Front-Channel Logout is still a challenging issue with third-party cookies blocked, and finding a good solution (interoperable nonetheless!) will take time.

Thanks @johannhof.
@michal-tobiasz-wttech I'll close this, as there haven't been any further comments for >8 months, but feel free to reopen.


Fix dump of tracing legend

(cherry picked from commit a84e382f2445a01d68c5e54fafdac5b9b54f96c8)

Turns out I can just reconstruct it from the trace files.


Connection to pod can only be established once

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
After upgrading from podman v1.6.4 to v1.8.2  we experience the problem that any newly created pod allows exactly ONE connection attempt while subsequent connection attempts are
rejected.
Steps to reproduce the issue:

Create a new pod from scratch with just one port mapping:
podman pod create -p 4545:8080 --name testpod

Start the pod:
podman pod start testpod

Connect to the pod using ncat:
nc -v localhost 4545
  Ncat: Version 7.70 ( https://nmap.org/ncat )
 Ncat: Connected to ::1:4545.

Terminate the ncat connection and repeat step 3, now the connection attempt is rejected:
nc -v localhost 4545
Ncat: Version 7.70 ( https://nmap.org/ncat )
Ncat: Connection to ::1 failed: Connection refused.
Ncat: Trying next address...
Ncat: Connection refused.

Any subsequent connection attempt fail as well UNLESS the pod gets restarted:
podman pod stop testpod ; podman pod start testpod

Describe the results you received:
A connection to a pod can only be established once, subsequent connection attempts fail (unless the pod is restarted).
Describe the results you expected:
Several connections can be made to a running pod.
Additional information you deem important (e.g. issue happens only occasionally):

Issue can be reproduced
Never expierenced this problem in v1.6.4
podman is used/running in rootless mode
We upgraded to v1.8.2 using the instuctions from the podman website respectively the Kubic project/stream:

sudo dnf -y module disable container-tools
sudo dnf -y install 'dnf-command(copr)'
sudo dnf -y copr enable rhcontainerbot/container-selinux
sudo curl -L -o /etc/yum.repos.d/devel:kubic:libcontainers:stable.repo https://download.opensuse.org/repositories/devel:/kubic:/libcontainers:/stable/CentOS_8/devel:kubic:libcontainers:stable.repo
sudo dnf -y install podman

Output of podman version:
Version:            1.8.2
RemoteAPI Version:  1
Go Version:         go1.12.12
OS/Arch:            linux/amd64

Output of podman info --debug:
debug:
  compiler: gc
  git commit: ""
  go version: go1.12.12
  podman version: 1.8.2
host:
  BuildahVersion: 1.14.3
  CgroupVersion: v1
  Conmon:
    package: conmon-2.0.6-1.module_el8.1.0+272+3e64ee36.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.6, commit: 7a4f0dd7b20a3d4bf9ef3e5cbfac05606b08eac0'
  Distribution:
    distribution: '"centos"'
    version: "8"
  IDMappings:
    gidmap:
    - container_id: 0
      host_id: 1020
      size: 1
    - container_id: 1
      host_id: 296608
      size: 65536
    uidmap:
    - container_id: 0
      host_id: 1020
      size: 1
    - container_id: 1
      host_id: 296608
      size: 65536
  MemFree:<PHONE_NUMBER>8
  MemTotal:<PHONE_NUMBER>84
  OCIRuntime:
    name: runc
    package: runc-1.0.0-64.rc9.module_el8.1.0+272+3e64ee36.x86_64
    path: /usr/bin/runc
    version: 'runc version spec: 1.0.1-dev'
  SwapFree:<PHONE_NUMBER>
  SwapTotal:<PHONE_NUMBER>
  arch: amd64
  cpus: 2
  eventlogger: file
  hostname: daisyt
  kernel: 4.18.0-147.5.1.el8_1.x86_64
  os: linux
  rootless: true
  slirp4netns:
    Executable: /usr/bin/slirp4netns
    Package: slirp4netns-0.4.3-23.2.el8.x86_64
    Version: |-
      slirp4netns version 0.4.3-beta.1
      commit: b04291ba84ca35ccc60bd009372a28f9ea7ef841
  uptime: 22h 31m 39.37s (Approximately 0.92 days)
registries:
  search:
  - registry.access.redhat.com
  - registry.fedoraproject.org
  - registry.centos.org
  - docker.io
store:
  ConfigFile: /home/fwd/.config/containers/storage.conf
  ContainerStore:
    number: 6
  GraphDriverName: overlay
  GraphOptions:
    overlay.mount_program:
      Executable: /usr/bin/fuse-overlayfs
      Package: fuse-overlayfs-0.7.2-1.module_el8.1.0+272+3e64ee36.x86_64
      Version: |-
        fuse-overlayfs: version 0.7.2
        FUSE library version 3.2.1
        using FUSE kernel interface version 7.26
  GraphRoot: /home/fwd/.local/share/containers/storage
  GraphStatus:
    Backing Filesystem: extfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 5
  RunRoot: /tmp/run-1020/containers
  VolumePath: /home/fwd/.local/share/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
podman-1.8.2-1.1.el8.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
We run podman on a bare metal CentOS 8 server.
cat /etc/os-release:
NAME="CentOS Linux"
VERSION="8 (Core)"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="8"
PLATFORM_ID="platform:el8"
PRETTY_NAME="CentOS Linux 8 (Core)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:8"
HOME_URL="https://www.centos.org/"
BUG_REPORT_URL="https://bugs.centos.org/"

CENTOS_MANTISBT_PROJECT="CentOS-8"
CENTOS_MANTISBT_PROJECT_VERSION="8"
REDHAT_SUPPORT_PRODUCT="centos"
REDHAT_SUPPORT_PRODUCT_VERSION="8"

Running as root, or rootless?

Hi @mheon,
thanks for the quick response. We operate podman in rootless mode.

Possibly related to #5740

#5740 talks about the port mappings being lost. This seems not to be the case here as
podman port -l
still lists the ports exposed by the pod even if the pod becomes inaccessible after the first connection attempt.

Hm.
@giuseppe @AkihiroSuda PTAL

I can reproduce with 1.8.1, but not with the version from git master.
A bisect says it is fixed with 4841cd1630d892cd2e1353c88502235e79f2556f
@mlezajic could you check if it is solved for you as well?

@giuseppe
how am i supposed to apply 4841cd1 ?

@giuseppe
how am i supposed to apply 4841cd1 ?

you could try directly the last version from git master

@giuseppe
how am i supposed to apply 4841cd1 ?

you could try directly the last version from git master

@giuseppe
thanks for getting back to me. I am quite new to this so i have never run podman directly from git master. I guess i have to download/clone the repo and then build podman somehow or run it from within the cloned repo? Any advice would be highly appreciated.

@giuseppe
thanks for getting back to me. I am quite new to this so i have never run podman directly from git master. I guess i have to download/clone the repo and then build podman somehow or run it from within the cloned repo? Any advice would be highly appreciated.

yes exactly.  You'd need to clone the git repository at $GOPATH/src/github.com/containers/libpod.  A make podman should be enough to build podman (assuming you have installed all the dependencies).
Since you are running Centos 8, you can fetch the list of dependencies here: https://github.com/containers/libpod/blob/master/Dockerfile.centos#L3-L30
Some packages might have a different name (e.g. go-md2man).

@giuseppe
thanks for getting back to me. I am quite new to this so i have never run podman directly from git master. I guess i have to download/clone the repo and then build podman somehow or run it from within the cloned repo? Any advice would be highly appreciated.

yes exactly. You'd need to clone the git repository at $GOPATH/src/github.com/containers/libpod. A make podman should be enough to build podman (assuming you have installed all the dependencies).
Since you are running Centos 8, you can fetch the list of dependencies here: https://github.com/containers/libpod/blob/master/Dockerfile.centos#L3-L30
Some packages might have a different name (e.g. go-md2man).

Thanks Giuseppe! I was able to pull all dependencies as well as building and installing a rpm with the latest code from git master using
make package-install resulting in:
Version: 1.8.3-dev
RemoteAPI Version: 1
Go Version: go1.12.12
Git Commit: 12ff9c2648a0aa7d0727f57ff808fb88570c8fe6-dirty
Built: Thu Apr  9 11:17:26 2020
OS/Arch: linux/amd64

With this code level i no longer observe the problem. Do you have an estimate on when v1.8.3 will officially be released?
Once again many thanks for your help and Happy Easter.

We are planning on releasing v1.9.0 in the next week or two.  Not sure when this will bubble down to Centos.


FEATURE: Possibility to define custom assets base path. Very usefuly on

big projects with a lot of developers. See core/Core.php for usage.

This is indeed a useful feature!  I have a few questions:

I'm not sure why we have ASSETS_BASE_PATH, ASSETS_PATH, and ASSETS_DIR?  It seems like 2 config options would sufficient here; the 3rd is confusing.
Does all of the CMS functionality still work?  I believe that there is hardcoding in the JS that would need to be addressed?
Can this be used to define an assets folder of something other than assets?

It might be worth explaining a bit more how this is used?

ASSETS_PATH and ASSETS_DIR were preexisting and not configurable. I've introduced ASSETS_BASE_PATH as this was the least obtrusive way to introduce config option for assets base path. If done from scratch it could have been done simpler I guess
All of the CMS functionallity still works. We have this patch in production use for more than half a year on more than 10 sites. There is no other hardcoding than that in patch.
I don't understand the question. Can you please rephrase?

Yes, I should have explained a bit better... I added a bit of documentations to the code too.
This feature enables seamless use of shared database for developers. Without it, assets go out of sync... It has to be configured in two steps, first is defining ASSETS_BASE_PATH in your _ss_environment.php (in our installations, it maps local project folder to shared project folder), second, you have to make sure, that your web server serves the ASSETS_DIR folder from your customized location. You can do that easily with setting up Virtual Folder in your IIS or alias or even reverse proxy on your Apache. Our developers have a installation script for projects, which takes care of this part.
We use this feature when developing all our SS sites. I guess it could also be used for load balancing purposes...

Reopened on post_2.4


substratee-worker-test --ecall fails with chain_relay_db not found

above test fails if we have not synced to a node and therefore above file is not created.
This is due to the onchain read that in the encalve in get_state performs. This onchain read is redundant as we sync the relevant data each block anyhow. We should remove that

after discussing this with @clangenb , we decided to remove the entire section in get_state that syncs blocks


Implement a more robust version of getClientBoundingRect

This attempts to fix #584. Here’s what we need for this:

[x] Find the union bounding box for multiple elements relative to a common parent and calculate its left/top/bottom/right offsets.
[x] Split spanning elements into multiple client rects and calculate the union bounding box from them. This is needed if, for example, a <span> starts in the middle of a sentence but stretches over multiple lines. In that case, offsetLeft will report its offset only from where the <span> starts.
[ ] Add tests for various scenarios like nested scrolling containers.

Note: This PR also adds a toArray utility function that’s used for converting ClientRectLists into proper arrays but can be reused for any non-Array list, e.g. NodeList.
Naming: I gave the function name some more thought and I think I would be for naming it getRelativeBoundingRect. I haven’t renamed it yet but I’d argue that the name should reflect that the bounding box (as well as the the offsets) is relative to a parent element.

Looking good!
Few thoughts:

span in the middle: maybe you can use span.getBoundingClientRect() there too, instead of el.offsetLeft? And then subtract the parent rectangles from it until relativeEl is reached? (but likely overlooking something here)
toArray seems useful. just wonder it's necessary here as lodash/forEach should be able to iterate over NodeList ClientRectList etc. Haven't tested but is lodash/toArray able to do the same trick?
top-center positioning of overlay: Fine with that. My only concern would be the edge case when you are selecting on the very top of a container and the overlay would not fit above the selection. I guess we would need to detect that case and position the overlay below.

👍  for getRelativeBoundingRect!
Let me know when this is ready to merge (we can push the test suite into a separate task) want to do naive click testing first. ;)


Selections GUI updates: Adding select by time or warming level approach

Description of PR
This PR is linked to PR #449 in climakitae; please see that PR for more information on the changes here.
Summary of changes and related issue
Relevant motivation and context
Dependencies required for this change?
Fixes # (issue), delete if not necessary
Type of change

[ ] 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 not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

[ ] Test A
[ ] Test B

Checklist:

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

This looks great! I love the UI and I think it makes a lot of intuitive sense and will be super helpful for WL integration for notebooks. I wanted to ask about a couple things:

Would it be possible to toggle the entire "Warming Levels Approach" or "Time-Based Approach" panels to be visible or invisible depending on which approach you choose at the top? Currently, choosing WL will make time-based selections into n/a, but I'm wondering if it might be even cleaner to toggle the entire panel away (i.e. when choosing WL approach, the time-based approach disappears, and vice versa).
When toggling between the two approaches, my former approach's params get reset. For example, if I set WL's to 2.0 and 2.5, then toggle the time-based approach and come back to the WL approach, my params get reset to just the 2.0 WL. Is it possible to keep the params from resetting between toggles?

These are just questions/suggestions, but nothing breaking when I've been testing either. This looks good to me to merge if you'd like to keep it as is!

This looks great! I love the UI and I think it makes a lot of intuitive sense and will be super helpful for WL integration for notebooks. I wanted to ask about a couple things:

Would it be possible to toggle the entire "Warming Levels Approach" or "Time-Based Approach" panels to be visible or invisible depending on which approach you choose at the top? Currently, choosing WL will make time-based selections into n/a, but I'm wondering if it might be even cleaner to toggle the entire panel away (i.e. when choosing WL approach, the time-based approach disappears, and vice versa).
When toggling between the two approaches, my former approach's params get reset. For example, if I set WL's to 2.0 and 2.5, then toggle the time-based approach and come back to the WL approach, my params get reset to just the 2.0 WL. Is it possible to keep the params from resetting between toggles?

These are just questions/suggestions, but nothing breaking when I've been testing either. This looks good to me to merge if you'd like to keep it as is!

No, it's not possible to make the parameters dissappear :( I agree that would be a huge improvement but I currently don't think it's an option with panel/param. I too find it "messy" but it's the best option we have at the moment.

Hmm... that may be possible but I'm not sure how to do it. I agree that would be a nice feature. Would be nice to look into later down the line (will add it to my mental backburner list)


Projects with existing projectMonitors shouldn't show in the dropdown

I tested it locally and it works as advertised. Assuming the merge conflict is fixed, LGTM


Reimplement matrixSums functions in Rcpp

Just got a C-level memory leak error when running simulateCells.celda_CG() for the following parameters:

model="celda_CG"
K = 5
alpha = 0.1
beta = 0.1
G = 1000
S = 5
gamma = 5
C.Range = c(1000000, 1000001)

I'm not sure if this will reproduce (these types of bugs can be finnicky), but reimplementing in C++ may help avoid memory allocation bugs

Was fixed by using logSumExp function in matrixStats package in commit 4f38f00502c2ac88e97815cdff3d183b5a42ec7e


Create Antares-Cygnus.netkan

find requires spec_ver v1.4.
To give previous users of AntaresCygnus an upgrade path, can you change the identifier to match and also give an epoch bump "x_netkan_epoch": "1",.

The .zip  has ships in it, we can install those find: "VAB", install_to: "Ships"

Understood, the mod author isn't maintaining the ships anymore, so he'd rather they weren't installed by default.  This goes for the rest of the Raidernick mods in the recent set as well.

Passws local and Jenkins. Merging with my thanks.


Roughness ECMA 418-2

New version of roughness calculation according to ECMA 418-2.
The code uses Sottek's Hearing Model implemented in loudness_ecma. The final roughness values are not validated yet due to some weighting errors.

Thanks to a collaboration with @fchirono the code has been updated and the model now seems very close to the ECMA 418-2 results. Validation plots have been created :

And all tests are running:

Excellent work Salomé, the new validation results look great! 😃

Hi @mglesser,
Requested changes have been made ;)
Tks for your review


Artifacts in high pitched audio

Hello, thank you for your amazing work! I have a question regarding the inference. It seems that the  audio generated using FreeV model includes artifacts if the audio is high pitched even though my model is trained for around 800k iterations.
Any advice on why this happens?
Thank you

Thank you for your feedback!
Do you have some samples with these artifacts? And have you tried to compare the predicted amplitude spec with real amp spec? And reconstruct the waveform with (real amplitude spec, predicted phase spec) and (predicted amplitude spec, real phase spec) pairs, and check which branch caused this problem.
Moreover, are there some high pitched audio in your training set? Maybe this problem is caused by out of distribution problem.

Thank you for your response. I will check for the above mentioned things 👍🏻.
I am training on libritts so maybe it is a distribution problem.


Typography menu name showing of upload control field in control dropdown menu in document section of website

Issue description:
Version used:
(Did you try using the develop branch from github? There's a chance your issue has already been adressed there)
Using theme_mods or options?
Code to reproduce the issue (config + field(s))

I seriously did not understand a word from that title.

Typography menu name showing of upload control field in control dropdown menu in document section of website

Please elaborate. explain with as many details as possible.

AH ok, I understood now.
Fixed in the docs site, thank you for reporting this!


Modifications required for CI

Link T-650
Now that we're interacting with VSCode primarily as a submodule of mnode we need to update the ts-plugin reference to just be a sibling of vscode. I've also made a change to the build script and added a fix where the postinstall script was causing CI to fail due to some git config commands running in the submodule.
Note: My commits aren't verified here because I'm pushing them from mk1

cc @juancampa. I'm going to roll forward on this because it's a small change but necessary for CI to run. I can fix any feedback in a follow up PR.


Is the amount of emails/notifications going to the applicants to much?

Team is wondering if we're sending to many msgs out to applicants? We should def. ask around and design a survey for folks. If the result ends up being yes, one discussed alternative is to identify which states are most important for a notification to go out and only notify on those, making it clear in the first notification that they will only get notifications at these stages and then building features that allow them to get an update of recent activity at any time (and/or let them choose default notification settings in their profile).

We should incorporate this into our discussion around 'applicant friendly states' for the application timeline. This is planned to be covered in the work we are doing in the Winter 2019 Sprints. Adding to project board.


I tried everything but no API endpoint work with continue.dev

Before submitting your bug report

[ ] I believe this is a bug. I'll try to join the Continue Discord for questions
[ ] I'm not able to find an open issue that reports the same bug
[ ] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
I need know how i can connect to API endpoint.. i tried everything but nothing work.. i used text-generation-webui but with --api parameter and i dont know path to this api endpoint i tried on vast.ai and on runpod.io nothing work.. maybe some proxy?? or what i can do to work?
I tried llama.cpp python it work little bit but i have error 404 on request.. i tried ggml class etc.. nothing work
Description
No response
To reproduce
No response
Log output
No response

We have an option that is made specifically for text-generation-webui: https://continue.dev/docs/reference/Models/textgenui
Alternatively, if you use the openai option in text-generation-webui, you could then use the OpenAI class in Continue
I notice that we don't have TextGenUI as an option in the model selection page, so I will add this

Thank you very much!

of course - did this end up solving your problem?

I don't know about them but, this solved the problem for me.
Thanks for planning to add it to the selection page, I thought the options there were the only ones available.
Sidenote: The "Other OpenAI-compatible API" option lists text-gen-webui as a supported.

@badcodecat When you had the problem with "Other OpenAI-compatible API" option, were you using the OpenAI-compatible API extension for text-gen-webui, or just the normal "--api" flag? (https://github.com/oobabooga/text-generation-webui/tree/main/extensions/openai#an-openedai-api-openai-like)
Only the former will work with the first method, something I'll try to clarify on the model selection page

I think so (not near my pc right now), but I also recently updated/migrated to the new version so I may have broken something then.

ok, no worries. I'm going to close this issue since it does seem that this is solved - I've tested with the OpenAI-compatible server and it seemed to be working, but text-generation-webui will also be a first-class option in the UI selector in our next release. Feel free to re-open if something else goes wrong


import/exports-last only(!) last

Hi there!
I couldn't find a solution, so I decided to check with you
I really like this rule, but is it possible to make it so that the exporters are only at the bottom of the file (even if the file has only one export)?
Now this is valid syntax, but I really want to avoid it

Thanks guys 🙂

I'm not clear on what you're asking for. In your example, there's one export and it's at the bottom of the file, so what would be warned on?
The exports-last rule shouldn't care if there's 0, 1, or a thousand exports - it will warn if any non-export statement happens before any export statement.

I'm not clear on what you're asking for. In your example, there's one export and it's at the bottom of the file, so what would be warned on?
The exports-last rule shouldn't care if there's 0, 1, or a thousand exports - it will warn if any non-export statement happens before any export statement.

I'm not clear on what you're asking for. In your example, there's one export and it's at the bottom of the file, so what would be warned on?
The exports-last rule shouldn't care if there's 0, 1, or a thousand exports - it will warn if any non-export statement happens before any export statement.

Thanks for your answer 🙂
Let me explain this way:
Сan I make it so that "export" cannot be written near the declaration of variables? (Even if there is only one variable in the file, as in the screenshot above)
So that "export" can be written only at the bottom of the file (in any situation)

I'm not clear on what you're asking for. In your example, there's one export and it's at the bottom of the file, so what would be warned on?
The exports-last rule shouldn't care if there's 0, 1, or a thousand exports - it will warn if any non-export statement happens before any export statement.

Thanks for your answer 🙂
Let me explain this way:
Сan I make it so that "export" cannot be written near the declaration of variables? (Even if there is only one variable in the file, as in the screenshot above)
So that "export" can be written only at the bottom of the file (in any situation)

Meaning, you want to ban usage of the export keyword with declarations?
You could use no-restricted-syntax for that, but it seems like a really strange subset of the language to prohibit.

Meaning, you want to ban usage of the export keyword with declarations?
You could use no-restricted-syntax for that, but it seems like a really strange subset of the language to prohibit.

Meaning, you want to ban usage of the export keyword with declarations?
You could use no-restricted-syntax for that, but it seems like a really strange subset of the language to prohibit.

Okay, thanks for all your answers 🙏
We can close this if needed
Good Luck 🙂

Meaning, you want to ban usage of the export keyword with declarations?
You could use no-restricted-syntax for that, but it seems like a really strange subset of the language to prohibit.

Okay, thanks for all your answers 🙏
We can close this if needed
Good Luck 🙂

If there’s a compelling reason to ban those export forms, it might be worth creating a rule, but i can’t conceive of one at the moment.
Thanks for the discussion!

If there’s a compelling reason to ban those export forms, it might be worth creating a rule, but i can’t conceive of one at the moment.
Thanks for the discussion!


Use compiler assumptions

Closes https://github.com/kensho-technologies/babel-preset/issues/106.
~WIP. Won't continue until the upstream issue mentioned in the issue is resolved.~ A fix was released in<EMAIL_ADDRESS>Adds a top-level assumptions field for all non-precompiled modules, and removes the loose: true default from the @babel/preset-env config. The former is a replacement for the latter. (Though not an exact replacement, so this is a breaking change. It's also breaking because passing loose: false no longer has any effect.)
The Kensho preset intentionally doesn't take an overriding assumptions option because it's a top-level config and can therefore be overridden by specifying them in the consumer config:
{
  "presets": ["@kensho-technologies/babel-preset"],
  "assumptions": {"constantSuper": false}
}

The specific assumption settings are chosen to roughly match the previous behavior (trading off on absolute spec compliance for smaller/faster output) while smoothing over some rough edges. All assumptions are enabled except:

enumerableModuleMeta: This is only used when transpiling to CJS, where output size is not a concern, so we might as well use a safer transform. We've actually had to work around this one in an internal test suite.
constantReexports: Same as above.
iterableIsArray: This combines loose in transform-destructuring and transform-spread (which we enabled) with assumeArray in transform-for-of (which we did not), so neither setting matches the previous behavior exactly. However, loose mode in transform-spread was the only optimization we found annoying in practice, because it prevented code like this from working properly. So let's disable this assumption.
arrayLikeIsIterable: This was not covered by loose, so let's start with it disabled to match. We rarely iterate over DOM collections where this would be useful.

also totally random q: what are the side-effects of enabling skipForOfIteratorClosing? afaict it has to do with calling the native generator finisher methods, but I'm actually not clear on when that is a good idea or not

where do we enable transform-destructuring and transform-spread

They're standard syntax, so part of @babel/preset-env.

what are the side-effects of enabling skipForOfIteratorClosing?

The trade-off is between a smaller output when transpiling for...of in general and absolute correctness in the specific case where the iterator is a generator that implements non-empty finishing functions. The latter is so uncommon (we don't even really use generators) that it's not worth incurring the cost in general.
Btw, the assumptions are all documented on this page and each one has an example of the difference between enabling and disabling it.


Kartikeyamani

The Readme file is currently lengthy and complex, making it challenging for newcomers to comprehend how to contribute effectively to the project.
To enhance accessibility and clarity, I propose relocating the Contributing section to the beginning of the Readme file and streamlining its content where feasible.
I kindly request assistance from anyone willing to dedicate time to address this issue and improve the project's accessibility for all contributors. Thank you!

70

Hello Myself Karthikeya Gangisetty. I'm so much intrested in Machine learning and computer vision techniques. I want to build projects using these tools. I want to learn open source because it is an easy and rapid way to grow our projects according to market needs. I selected this issue because ReadMe file is like base to any projects that we see and if it's lengthy it is difficult for contributors to understand so we make it as easy as possible.

Hey @connman4027  to deal with this issue I need your help because you and I are the only participants who assigned this issue and I would like to gain some insightful knowledge about this project. I sincerely appreciate any help that I can get from you in solving the issue. Thanks in advance.

Hey @caiton1  to deal with this issue I need your help because you and I are the only participants who assigned this issue and I would like to gain some insightful knowledge about this project. I sincerely appreciate any help that I can get from you in solving the issue. Thanks in advance.

The issue is closed, Thanks everyone for their contributions


Fix timestamp format '%s'

Please review this patch.
Detail
https://gist.github.com/hiroyuki-sato/61e5da934ccf41a258ca
Epoch time :<PHONE_NUMBER> is '2015-02-24 18:34:10 +0900'
But, preview result is show 47119-03-07 17:26:40 UTC.

Right. Thank you

I updated some other code to be able to handle microseconds (there was millisecond limitation before).


Make sure subquery range is taken into account for selection

This would make sure that remote_read and other downstream library users get the right SelectParams to select the data.

This should go against master.

@brian-brazil Addressed all comments.

Done :)


Split ramdangular fork from floydsoft/angular-underscore?

I wanted to write this issue and another on the ramdangular repo, but can't since it's a fork from angular-underscore.
If we do want to keep it as-is, then I kind of wanted to make a plug for changing the name to ramda-angular or something unmerged. It trips me up every time!

+1


🚸(frontend) move room public switch to the security section

Purpose
It appears that it makes more sens to list this switch in the security section.
Proposal
Move widget to the security section.

it works fine

it works fine


Icon request: Hurricane Electric Internet Service

URL of the site or app (required): https://he.net/
Logo resources for the icon (optional, but recommended): https://pbs.twimg.com/profile_images/348659971/HurricaneElectric-Logo-001_-_med.png

Here you go!
For current Aegis (recommended for now):
https://github.com/aegis-icons/aegis-icons/blob/master/PNG/Hurricane Electric Internet Service.png
For Aegis beta version:
https://github.com/aegis-icons/aegis-icons/blob/master/SVG/Hurricane Electric Internet Service.svg


Update ci to generate a dotslash file

What
We started using dotslash to launch pks. We named the dotslash file pks and put it in the Rails bin directory. We now launch pks with bin/pks.
The reason we decided to take this approach was to ensure that the correct version of pks is always used and to provided a consistent installation for both mac and linux.
How
This PR use the facebook/dotslash-publish-release@v1 github action to create and upload a dotslash file into the release assets. The file is simply named pks.
I also broke out a separate upload-mac-binary job that is run in parallel with upload-linux-bin
Test
https://github.com/perryqh/packs-ci/releases/tag/v0.2.8

@alexevanczuk we're already using it our monolith.  We manually created the dotslash file and saved it to bin/pks. Our precommit hooks are bin/pks check and bin/pks validate. CI is also using bin/pks to run its checks.
This PR generates the dotslash file so we don't need to do it manually.
Users just need to have the dotslash cli installed.

Cool thanks for the info @perryqh !! Awesome to hear.


Devices keeps Opening... or Closing...

The blinds keeps their status on Opening.... or Closing... in the Home app.
They respond, but they still keep showing opening or closing, even when they done setting up the position.

Logs warning appears:
[4/1/2022, 5:12:57 PM] [homebridge-motionblinds] This plugin generated a warning from the characteristic 'Target Position': characteristic value expected valid finite number and received "undefined" (undefined). See https://git.io/JtMGR for more info.

This should be improved in 2.0.0, please give it a try and reopen if you encounter issues. I still see a delay of about 10 seconds in the Home app before the Opening/Closing state goes away, and I can't seem to overcome it. It could be a limitation in homebridge / HapJS / HomeKit / Home app.

Thanks for the update! Keep up the good work.
However; Im still encoutering this problem. Even aftyer updating to 2.0.0.
After closing or opening the blind sometimes says it's closed or open; but changing the position to another percentage it wont work. Video: https://youtu.be/qRbW7AYEii0
Any tips I can try?

Thanks for the update indeed. Now invert mode works perfectly which was not the case in previous version.
However blinds keeps "OPENING" forever in HOMEKIT since upgrade which was not the case in previous version.
Status works very OK in Homebridge itself, just the update in HomeKit not working properly.
Any suggestions ?

Also running into this issue. I'm on version 2.0.0 from the start. The accessory keep in the "Opening..." or "Closing..." state, even after restarting the blind, bridge or homebridge plugin.

Same for me.

Also just now realized it only happens for me when inverted=true. When not inverted homekit works as expected (just opening / closing states are wrong)


Re-write ideas

References should be resolved & imported when parsing the file. Right now we add the import when converting to a type node :(
Separate File from Context, since some handlers take multiple files (type takes type file, schema file). For some things, it's not clear which SdkFile to use, and it doesn't matter (e.g. resolveType)
Represent types at a higher level, so you can call .construct() to create one and .declare() to declare one. Same for functions (.invoke(), .declare()). This will help with examples

closing as stale


Plasma-Browser-Integration Vivaldi not working

I get the error Failed to connect to the native host. Make sure the 'plasma-browser-integration' package is installed correctly and that you are running Plasma 5.13 or later. whenever I click the icon for the extension.
It seems to work partially. On my phone, I can skip through videos, but I cannot change the volume.

I am using Plasma desktop and have the libforqt5 version installed as a system package.
I'm not sure if this is some setup I did, but it's been like this for a while. I also tried removing all of my Vivaldi files and starting from scratch and the same issue is present regardless if Vivaldi is a fresh install or not.

This needs a similar fix to the other browsers in #232528

I was able to solve this issue for Vivaldi by just adding this to my configuration file:
environment.etc."chromium/native-messaging-hosts/org.kde.plasma.browser_integration.json".source = "${pkgs.plasma-browser-integration}/etc/chromium/native-messaging-hosts/org.kde.plasma.browser_integration.json";

When 24.05 comes out, I think just using programs.chromium.enablePlasmaBrowserIntegration should just work

Good to know, that this got fixed and can be closed.

Brave have this issue also

which one is the Vivaldi config file so I can add that line and solve?

which one is the Vivaldi config file so I can add that line and solve?

which one is the Vivaldi config file so I can add that line and solve?

I was able to solve this issue for Vivaldi by just adding this to my configuration file:
environment.etc."chromium/native-messaging-hosts/org.kde.plasma.browser_integration.json".source = "${pkgs.plasma-browser-integration}/etc/chromium/native-messaging-hosts/org.kde.plasma.browser_integration.json";

When 24.05 comes out, I think just using programs.chromium.enablePlasmaBrowserIntegration should just work

which one is the Vivaldi config file so I can add that line and solve?


Nimodipine no longer in workflow query C.2 results

In April 2021, query C.2 results (from ARAX) included nimodipine, a particularly interesting result: https://arax.ncats.io/beta/?r=8461. However, this result no longer shows up. It was thought this may be due to updates/issues in the ARAX ranking algorithm, prompting the following issue in the RTX repo: https://github.com/RTXteam/RTX/issues/1627. Subsequently, the follow issue (in ranking) was found: https://github.com/RTXteam/RTX/issues/1630.
Both issue have been fixed and nimodipine still does not appear in the top 100 ARAX results. As Amy Glen noted (https://github.com/RTXteam/RTX/issues/1627#issuecomment-906899598), nimodipine is well-connected: https://arax.ncats.io/?r=23288. Furthermore, she notes that less well-connected results show up in the top 100.
Given Amy's findings, this warrants further investigation. Adding this issue here to keep attention on it.

An update on this - the best I've managed to do is get nimodipine to show up as result number 2414. And this result I found shows up as "(s)-nimodipine" and doesn't have many genes connected to it (1 or 2). It doesn't appear to be the same result that Amy tracked down when she specified nimodipine for node 3. There seem to be some other interesting results. We might need to give up on the nimodipine result of yore....

This was addressed through development of a revised queries for Demo C.


Version 12 of Tailored Room Description error

According to a thread at intfiction.org (https://www.intfiction.org/forum/viewtopic.php?f=7&t=26232), the public library version of Tailored Room Description (version 12) gives an error when you try to include it in a project.
There are some more recent versions of Tailored Room Description on Github. Should one of those go in the Public Library instead? (I have not tested any of these--just reporting on what others have said.)

This was fixed with 3bd8635 (in August 2016.)

I realise now that this was a request to update the version currently in the public library, not the version here. I don't know if this is a good place for such requests, but that is not for me to decide. Sorry about that.

The new version is now in Graham's new Public Library.  I do not expect old versions of the public library to be updated so I guess that's done.


nintendo pro controller support

add bluetooth support for nintendo pro controller
try to use bluepad32 library?

bluepad32 needs some substantial work to coexist with our current platform, I am going to move to an ESP-NOW based control instead of bluepad32.


Metrics endpoint not available

Goodday.
I've tried to utilize the Metrics endpoint which accoding to the documentation should be under {pipelines_base_url}/metrics , however this is not present.
I imported the swagger file into postman to make it a bit easier to execute, but still getting the following jsons return :
{     "message": "Route POST:/metrics not found",     "error": "Not Found",     "statusCode": 404 }

This was due to the swagger import not setting the Accept-Version, which should be set to 1.0.0.
After adding this to the header it routes correctly.
Still would be nice to have this part of the original postman collection in the project.

Thanks for raising as an issue @arnefromsa. We're tracking this and will update here when released.

Resolved in release v.1.9.0. Closing issue.


how to perform mouse right click

Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

RobotJS version:
Node.js version:
npm version:
Operating System:

var robot = require("robotjs");

robot.mouseClick("right");

mouseClick is documented here. It might be unclear since it doesn't explicitly show the values as strings.

I'm going to mark this as closed, if you still need help please let us know.


feat: #2513 - product page - moved higher the action bar

Impacted file:

new_product_page.dart: moved higher the action bar

What

In the product page, the action bar (list / edit / share) is now between the summary card and the health card. Instead of being down under, mate.
Might be problematic as each list takes room (cf. screenshots)

Screenshot

top
...scrolling down...

Fixes bug(s)

Closes: #2513

Codecov Report

Merging #2615 (e6ec70f) into develop (2ea0da3) will decrease coverage by 1.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop   #2615      +/-   ##
==========================================
- Coverage     8.86%   7.52%   -1.34%
==========================================
  Files          161     212      +51
  Lines         6623   10148    +3525
==========================================
+ Hits           587     764     +177
- Misses        6036    9384    +3348

Impacted Files
Coverage Δ

...kages/smooth_app/lib/widgets/attribute_button.dart
0.00% <0.00%> (-92.00%)
:arrow_down:

...s/smooth_app/lib/data_models/user_preferences.dart
8.73% <0.00%> (-23.57%)
:arrow_down:

packages/smooth_app/lib/themes/smooth_theme.dart
60.00% <0.00%> (-22.98%)
:arrow_down:

...p/lib/generic_lib/dialogs/smooth_alert_dialog.dart
15.11% <0.00%> (-19.10%)
:arrow_down:

...mooth_app/lib/data_models/product_preferences.dart
24.65% <0.00%> (-6.78%)
:arrow_down:

packages/smooth_app/lib/main.dart
14.16% <0.00%> (-3.73%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/welcome_page.dart
0.00% <0.00%> (-3.13%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/scan_example.dart
0.00% <0.00%> (-2.28%)
:arrow_down:

...p/lib/pages/onboarding/consent_analytics_page.dart
0.00% <0.00%> (-1.57%)
:arrow_down:

...smooth_app/lib/pages/product/new_product_page.dart
0.00% <0.00%> (-0.88%)
:arrow_down:

... and 223 more

Continue to review full report at Codecov.

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

Thank you @M123-dev for your review!
If only all PRs could be that easy...

Taking a look at #2396 later or at the latest tomorrow

I wonder if we shouldn't use a FAB in this screen to have access to the three features without even scrolling


Unable to use this adapter : No "exports" main defined in<EMAIL_ADDRESS>

I am trying to use this adapter within my project, but I encounter this issue:
No "exports" main defined in<EMAIL_ADDRESS>Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: No "exports" main defined in<EMAIL_ADDRESS>    at throwExportsNotFound (internal/modules/esm/resolve.js:290:9)
    at packageExportsResolve (internal/modules/esm/resolve.js:479:7)
    at resolveExports (internal/modules/cjs/loader.js:432:36)
    at Function.Module._findPath (internal/modules/cjs/loader.js:472:31)
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:867:27)
    at Function.Module._load (internal/modules/cjs/loader.js:725:27)
    at Module.require (internal/modules/cjs/loader.js:952:19)
    at require (internal/modules/cjs/helpers.js:88:18)
    at Object.<anonymous> (D:\(...)\svelte.config.cjs:4:16)
    at Module._compile (internal/modules/cjs/loader.js:1063:30)

(...) is an alias here for the root folder

my svelte.config.cjs :
const { mdsvex } = require("mdsvex");
const mdsvexConfig = require("./mdsvex.config.cjs");
const sveltePreprocess = require('svelte-preprocess');
const static = require('@sveltejs/adapter-static');

/** @type {import('@sveltejs/kit').Config} */
module.exports = {
	extensions: [".svelte", ...mdsvexConfig.extensions],
	// Consult https://github.com/sveltejs/svelte-preprocess
	// for more information about preprocessors
	preprocess: [
		mdsvex(mdsvexConfig),
		sveltePreprocess({
		  defaults: {
			style: 'postcss',
		  },
		  postcss: true,
		}),
	  ],

	kit: {
		// By default, `npm run build` will create a standard Node app.
		// You can create optimized builds for different platforms by
		// specifying a different adapter
		adapter: static(),
		// hydrate the <div id="svelte"> element in src/app.html
		target: '#svelte'
	}
};

Could you help my ?
Thanks in advance.

If anyone bumps into this problem, here's the example of how the config should look like
https://kit.svelte.dev/docs#configuration

Not as simple as copying the config - as everything is in flux, sometimes the adapter@next goes out of sync with kit@next. Use:
pnpm view @sveltejs/kit versions
pnpm view @sveltejs/adapter-static versions

To view what's available. For example<EMAIL_ADDRESS>and<EMAIL_ADDRESS>and reformatting svelte.config.js worked for me.

Had the same error just a moment ago and no solutions helped, but turned out I installed the dependency not as dev dependency. Moved it and now works fine.


install_dir_path logic has a mistake in it

If install_dir_path is provided via environment variable, it remains a string:
https://github.com/philipperemy/stanford-openie-python/blob/master/openie/openie.py#L23
If install_dir_path is provided via argument, it becomes a Path:
https://github.com/philipperemy/stanford-openie-python/blob/master/openie/openie.py#L25
This leads to an error on the next line if install_dir_path is provided via environment variable:
https://github.com/philipperemy/stanford-openie-python/blob/master/openie/openie.py#L26

Good catch will release a quick fix.

https://github.com/philipperemy/stanford-openie-python/pull/63

Done, you can upgrade the lib. I released 1.3.2.
pip install stanford_openie --upgrade


not throwing error?

var file = request.get(path)
  .auth(auth.user,auth.passwd)
  .end(function(err){
    if(err) throw err;
  });

if auth.user and auth.password is wrong it still proceeds anyways and doesn't throw errors

I'm unable to reproduce. You get an error when server responds with status 401. If the server rejects the password, but only prints a message without setting status code, then superagent won't know that it's an error.


Explicitly import olympia from manage.py. Fixes discovery of local …

…management commands.

Never mind. I added this to my rebase of #681 instead.


boot2docker support

C:\>docker-machine create --driver virtualbox --engine-env http_proxy=http://<IP_ADDRESS>:8118 --engine-env https_proxy=http://<IP_ADDRESS>:8118 --engine-env no_proxy=<IP_ADDRESS>,loopback plugins
...
C:\>docker-machine ssh plugins
...
docker@plugins:~$ uname -a
Linux plugins 4.4.27-boot2docker #1 SMP Tue Oct 25 19:51:49 UTC 2016 x86_64 GNU/Linux
docker@plugins:~$ curl -fsSL https://raw.githubusercontent.com/CWSpear/local-persist/master/scripts/install.sh | sudo bash
sudo: bash: command not found
curl: (6) Could not resolve host: raw.githubusercontent.com
docker@plugins:~$ export http_proxy=http://<IP_ADDRESS>:8118 https_proxy=http://<IP_ADDRESS>:8118 no_proxy=<IP_ADDRESS>,loopback
docker@plugins:~$ curl -fsSL https://raw.githubusercontent.com/CWSpear/local-persist/master/scripts/install.sh | sudo bash
sudo: bash: command not found
(23) Failed writing body
docker@plugins:~$ curl -fsSL https://raw.githubusercontent.com/CWSpear/local-persist/master/scripts/install.sh | sudo sh
NOTE: sudo needed to set up and run start service
docker@plugins:~$ sudo -H -i
Boot2Docker version 1.12.3, build HEAD : 7fc7575 - Thu Oct 27 17:23:17 UTC 2016
Docker version 1.12.3, build 6b644ec
root@plugins:~# export http_proxy=http://<IP_ADDRESS>:8118 https_proxy=http://<IP_ADDRESS>:8118 no_proxy=<IP_ADDRESS>,loopback
root@plugins:~# curl -fsSL https://raw.githubusercontent.com/CWSpear/local-persist/master/scripts/install.sh | sh
NOTE: sudo needed to set up and run start service

As you can see - sudo is supported in boot2docker environment.

This is obviously very old, and boot2docker is deprecated. To run on Mac or Windows, see this section in the docs


where i can find org/mybatis/example/BlogMapper.xml and Blog.java code

Hi i am xingyun, i want to learn mybatis from website  at
http://www.mybatis.org/mybatis-3/zh/getting-started.html
but i can't find org/mybatis/example/BlogMapper.xml and Blog.java code
could you show me these two files?
thanks a lot.

We use GitHub Issues only to track bugs and feature requests.
Please post your question to the mailing list or Stack Overflow.
If you are looking for an executable example, there is a demo project.
Thank you!


Static 0xe5 on Honda Bosch

Block 0xe5 (Bosch only) and send only static values. Wheel wobble functionality is disabled on vehicles that support it. This important safety change is crucial for Bosch cars because the wobble overrides any torque command on 0xe4 sent by OP (or stock LKAS).
Panda PR for safety. 0xe5 controls wheel wobble on Bosch and also has a bit for braking request during RDM. https://github.com/commaai/panda/pull/515
Opendbc changes are a requirement too https://github.com/commaai/opendbc/pull/250
Has been tested on Civic Hatch and Insight. Waiting for Accord.

Failing replay test because the needed opendbc changes in https://github.com/commaai/opendbc/pull/250

Can you rebase with master? I merged the opendbc PR.

@pd0wm Looks like the panda pr is ready to go too. Just waiting for the merge.

Test failing for something unrelated to this PR?

The test is actually failing for the correct reason. There are unexpected can messaged.
If you change selfdrive/test/process_replay/ref_commit with b33259e2dae9852c58b0507dbb98fb01d8b884b8 it will pass again.

The test is actually failing for the correct reason. There are unexpected can messages being sent out.
If you change selfdrive/test/process_replay/ref_commit with b33259e2dae9852c58b0507dbb98fb01d8b884b8 it will pass again.

Done!


Document flannel usage

Add notes both for demo and devenv.

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Kinesis Video Stream hangs on getMedia request

Describe the bug
We are using cpp SDK to read video from Kinesis Video Streams, GetEndpoint is returning us a response without any issues, GetMedia endpoint however hangs forever, no response is received.
Expected Behavior
GetMedia should return a proper response, so we can read payload continuously
Current Behavior
GetMedia request is getting stuck
Reproduction Steps
Here is a sample code we try to consume the Kinesis Video Streams API:
  `Aws::KinesisVideo::Model::GetDataEndpointOutcome get_end_point(const std::string& stream_name, const
    Aws::Client::ClientConfiguration& client_config, std::string& client_name)
    {
        Aws::KinesisVideo::KinesisVideoClient client(aws_creds_, client_config);
        auto outcome = client.GetDataEndpoint(Aws::KinesisVideo::Model::GetDataEndpointRequest()
                                                           .WithStreamName(stream_name)
                                                           .WithAPIName(Aws::KinesisVideo::Model::APIName::GET_MEDIA));
        client_name = client.GetServiceClientName();
        if (!outcome.IsSuccess()) {
            std::cout << "Failed to get data endpoint: " << outcome.GetError().GetMessage() << std::endl;
        }
        client.OverrideEndpoint(outcome.GetResultWithOwnership().GetDataEndpoint());
        return outcome;
    }

    void read(const std::string& stream_name, const std::string& client_name)
    {
        Aws::Client::ClientConfiguration client_config;
        client_config.region = Aws::Region::US_EAST_2;
        std::string client_name_api;
        auto outcome = get_end_point(stream_name, client_config, client_name_api);
        client_config.endpointOverride = outcome.GetResult().GetDataEndpoint();
        Aws::KinesisVideoMedia::KinesisVideoMediaClient video_media_client(aws_creds_, client_config);
        video_media_client.EnableRequestProcessing();

        auto output = video_media_client.GetMedia(Aws::KinesisVideoMedia::Model::GetMediaRequest()
                                                          .WithStreamName(stream_name)
                                                          .WithStartSelector(Aws::KinesisVideoMedia::Model::StartSelector()
                                                                                .WithStartSelectorType(Aws::KinesisVideoMedia::Model::StartSelectorType::NOW)));
        if (!output.IsSuccess()) {
            std::cout << "Failed to retrieve data from Kinesis Video Stream: " << output.GetError().GetMessage() << std::endl;
        }

    }`

Code is being stuck on GetMedia call forever.
Possible Solution
No response
Additional Information/Context
No response
AWS CPP SDK version used
1.11.272
Compiler and Version used
gcc  version 11.4.0 / g++ version 11.4.0
Operating System and version
Ubuntu 22.04 Desktop

hey @intergavg I tried reproducing and I couldn't, i modified your test code a little bit
#include <aws/core/Aws.h>
#include <aws/kinesisvideo/KinesisVideoClient.h>
#include <aws/kinesisvideo/model/GetDataEndpointRequest.h>
#include <aws/kinesis-video-media/KinesisVideoMediaClient.h>
#include <aws/kinesis-video-media/model/GetMediaRequest.h>

using namespace Aws;
using namespace Aws::Utils;
using namespace Aws::KinesisVideo;
using namespace Aws::KinesisVideo::Model;
using namespace Aws::KinesisVideoMedia;
using namespace Aws::KinesisVideoMedia::Model;

const char* STREAM_NAME = "your-stream-name";

auto main() -> int {
  SDKOptions options;
  options.loggingOptions.logLevel = Logging::LogLevel::Trace;
  InitAPI(options); {
    const auto client = Aws::MakeUnique<KinesisVideoClient>("test");
    const auto outcome = client->GetDataEndpoint(GetDataEndpointRequest()
      .WithStreamName(STREAM_NAME)
      .WithAPIName(APIName::GET_MEDIA));
    assert(outcome.IsSuccess());
    const auto enpoint = outcome.GetResult().GetDataEndpoint();
    std::cout << "Found endpoint: " << enpoint << "\n";

    KinesisVideoMediaClientConfiguration configuration;
    configuration.endpointOverride = enpoint;
    const auto mediaClient = Aws::MakeUnique<KinesisVideoMediaClient>("test", configuration);
    const auto mediaOutcome = mediaClient->GetMedia(GetMediaRequest()
      .WithStreamName(STREAM_NAME)
      .WithStartSelector(StartSelector()
      .WithStartSelectorType(StartSelectorType::NOW)));
    assert(mediaOutcome.IsSuccess());
  }
  ShutdownAPI(options);
  return 0;
}

So looks like we might have a "works on my machine" issue here. one thing i want to rule out first and foremost is could you try setting you logs to trace i.e.
  SDKOptions options;
  options.loggingOptions.logLevel = Logging::LogLevel::Trace;
  InitAPI(options);

then in your workdir you should see a logfile aws_sdk_{DATE}.log
in that file near the end you should see a curl debug statement
[DEBUG] 2024-02-28 15:16:02.468 CURL [0x1d6f05c40] (HeaderOut) POST /getMedia HTTP/1.1
Host: REDACTED-HOST-PREFIX.kinesisvideo.us-east-1.amazonaws.com
Accept: */*
amz-sdk-invocation-id: DC534AB6-FB00-4F77-A07D-B389E81D3A1C
amz-sdk-request: attempt=1
authorization: AWS4-HMAC-SHA256 Credential=REDACTED/20240228/us-east-1/kinesisvideo/aws4_request, SignedHeaders=amz-sdk-invocation-id;amz-sdk-request;content-length;content-type;host;x-amz-api-version;x-amz-content-sha256;x-amz-date;x-amz-security-token, Signature=REDACTED
content-length: 84
content-type: application/json
user-agent: REDACTED
x-amz-api-version: 2017-09-30
x-amz-content-sha256: REDACTED
x-amz-date: 20240228T151602Z
x-amz-security-token: REDACTED

what im curious specifically about is if the Host: REDACTED-HOST-PREFIX.kinesisvideo.us-east-1.amazonaws.com header is reflective of what your override was set to or if it is something unexpected? gunna try creating a docker image so we can be on same page of "on my machine" for further debugging.
also give my test code a shot and see if it works for you. i know your code is likely a snippet of a larger piece of code but InitAPI and ShutdownAPI are requried for sdk usage. Just trying to rule out everything so let me know if the test code changes the outcome.

Alright created a docker file and example that cannot reproduce
Docker File:
FROM public.ecr.aws/amazonlinux/amazonlinux:2023

#Install g++
RUN yum groupinstall "Development Tools" -y

#Install required dependencies
RUN yum install -y curl-devel openssl-devel ninja-build cmake3 graphviz

# Clone repo
RUN git clone --depth 1 --recurse-submodules https://github.com/aws/aws-sdk-cpp && \
    cd aws-sdk-cpp && \
    mkdir build && \
    cd build && \
    cmake -DAUTORUN_UNIT_TESTS=OFF -DBUILD_ONLY="kinesisvideo;kinesis-video-media" .. && \
    cmake --build . && \
    cmake --install .

# Copy Code Over \
RUN mkdir sdk-example
COPY CMakeLists.txt sdk-example/CMakeLists.txt
COPY main.cpp sdk-example/main.cpp
RUN cd sdk-example && \
    mkdir build && \
    cd build && \
    cmake .. && \
    cmake --build .  I will also note that while developing the example i saw curl timeouts in the example in which i had to update with  config.connectTimeoutMs = 5000; which might also be your problem but would need to see your logs to be sure.
let me know if you see any errors in your logs, or if you can update the example to replicate your failure.

Thank you for making this sample, I tried this on my demo Ubuntu 20.04 PC, Docker version 25.0.3, build 4debf41
I got the following in the output:
sdk_usage_workspace: /sdk-example/main.cpp:31: int main(): Assertion outcome.IsSuccess()' failed.`

main.cpp:31

line 31 is the the endpoint call, which previously worked for you, there should be std::out right before the assertion about why the call failed. What does the std::out say why it failed? what do the logs say?


并发增加后，无法返回测试结果

100并发可以正常返回，200就会有这个错误，看请求都是正常的，但是压测结果无法返回
ERRO[0080] failed to handle the end-of-test summary error="GoError: failed to open connection, error: failed to verify client version: read tcp *****:22811->******:9669: i/o timeout \n\tat reflect.methodValueCall (native)\n\tat file:****/nebulagraph-bench/templates/k6_config.js.j2:12:27(59)\n"

graph 上有日志么？可以试一下 150 并发会不会有问题，然后看一下 graph 使用了多少的 cpu

@cooler810 hi, I have noticed that the issue you created hasn’t been updated for nearly a month, so I have to close it for now. If you have any new updates, you are welcome to reopen this issue anytime.
Thanks a lot for your contribution anyway 😊


Best approach for moving CRD from crds folder to templates folder?

Hello,
We are trying to move CRD from crds to templates for some reasons, so Helm could handle the CRD upgrade in the future (we have made sure this suits our use case). Our CRD in crds folder does not have any annotations nor labels.
Here is the approach that I am using but I am unsure if this the best way to do it. It would be great that you can take a look and this could serve as a guideline for the community if anybody encounter the same scenario.
Assuming version 1.0.0 is the version that CRD is in the crds folder, and 2.0.0 is the version that CRD is in the template folder.

In 2.0.0, use a pre-install hook to add the following annotations to the CRD that has been installed by 1.0.0:

  annotations:
    meta.helm.sh/release-name: {{ .Release.Name }}
    meta.helm.sh/release-namespace: {{ .Release.Namespace }}
  labels:
    app.kubernetes.io/managed-by: Helm

In 2.0.0, we move CRD from crds folder to templates folder and add the above annotations and labels in the yaml file.
Once 2.0.0 is installed, Helm should be able to handle upgrade afterward. i.e. 3.0.0 included a new version of the CRD.

Thanks in advance!

Unfortunately this does not work. Pre-install hook is executed after the template is rendered. Therefore, Helm still errors out with message:
Error: UPGRADE FAILED: rendered manifests contain a resource that already exists. Unable to continue with update: CustomResourceDefinition "..." in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; label validation error: missing key "app.kubernetes.io/managed-by": must be set to "Helm"; annotation validation error: missing key "meta.helm.sh/release-name": must be set to "..."; annotation validation error: missing key "meta.helm.sh/release-namespace": must be set to "..."

Are there any other workaround that could be integrated with Helm?

This is constituted as a breaking change in your chart, so you should be bumping your major version number (e.g. from 1.0.0 to 2.0.0) as there is no upgrade path between the two. Users will have to uninstall and re-install the chart.

But yes, your suggested guidelines looks correct.


PDF files not loading

Amazing script. Thank you for creating this.
One issue I'm trying to resolve is the PDF files are not loading from my Evernote export.  The files exist in resources but do not load in Obsidian.  I have too many of these manually updating each and every one won't be an option.
Any guidance on how to solve this issue?  I'm new to Obsidian so perhaps I'm missing something else.

I'd doublecheck the markdown and see if the path is corrupted.  For some reason when I did the export, a lot of the paths got changed to have the string resources (or similar) in front of them.
The good news is that if you remove the leading path and just have the bare filename, Obsidian is pretty good about finding that file if it's anywhere in your vault (especially if it has a unique name).
-- Doug


Build failure: legacyPackages.x86_64-linux.pkgsCross.aarch64-android.linuxHeaders

Steps To Reproduce
Steps to reproduce the behavior:
On 1945d198852b59e5d0dc1576a3af930f80454e37:
nix-repl> :b legacyPackages.x86_64-linux.pkgsCross.aarch64-android.linuxHeaders
error: Package ‘python-<IP_ADDRESS>’ in /nix/store/gsbfsbpqh5j7lmall7fwi8pxs9h682a1-source/pkgs/development/interpreters/python/cpython/2.7/default.nix:330 is marked as insecure, refusing to evaluate.

       Known issues:
        - Python 2.7 has reached its end of life after 2020-01-01. See https://www.python.org/doc/sunset-python-2/.

       You can install it anyway by allowing this package, using the
       following methods:

       a) To temporarily allow all insecure packages, you can use an environment
          variable for a single invocation of the nix tools:

            $ export NIXPKGS_ALLOW_INSECURE=1

        Note: For `nix shell`, `nix build`, `nix develop` or any other Nix 2.4+
        (Flake) command, `--impure` must be passed in order to read this
        environment variable.

       b) for `nixos-rebuild` you can add ‘python-<IP_ADDRESS>’ to
          `nixpkgs.config.permittedInsecurePackages` in the configuration.nix,
          like so:

            {
              nixpkgs.config.permittedInsecurePackages = [
                "python-<IP_ADDRESS>"
              ];
            }

       c) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
          ‘python-<IP_ADDRESS>’ to `permittedInsecurePackages` in
          ~/.config/nixpkgs/config.nix, like so:

            {
              permittedInsecurePackages = [
                "python-<IP_ADDRESS>"
              ];
            }

Additional context
This has many knock-on consequences .. maybe like all of legacyPackages.x86_64-linux.pkgsCross.aarch64-android fails to build (at least a few packages I looked at depend on linuxHeaders through a pretty common-looking stdenv path).
Notify maintainers
@s1341 @r-burns

Fixed in #215547


Memory leak in STAR WARS Jedi: Fallen Order

Hello,
I've encountered this memory leak while playing Star Wars after some time, which leads to a major slowdown and a crash (if you try really hard).
I've managed to replicate this issue by spamming Esc key to open and close menu, but I believe that the root cause might be somewhere else - I don't think I've opened menu this many times during normal gameplay.
In addition to required files, I've attached video of me spamming Esc key with full DXVK hud.

Software information

Game: STAR WARS Jedi: Fallen Order
Settings: Epic preset
Resolution: 2560x1440

System information

GPU: GTX 1080
Driver: 440.36.0
Wine version: 4.21
DXVK version: 1.4.6

Apitrace file(s)

starwarsjedifallenorder.trace

Log files

starwarsjedifallenorder_d3d11.log
starwarsjedifallenorder_dxgi.log

Notes

Apitrace includes crash, however log files had the game exit gracefully.
Video reproducing memory leak and an eventual crash.

Are you absolutely sure this isn't just a game bug? DXVK isn't known to leak memory.

It's a game bug.

(F for my poor escape key)

Thanks for checking.

Apitrace includes crash, however log files had the game exit gracefully.

FWIW the apitrace seems somewhat broken (not your fault); apitrace thinks it only contains one frame and lacks some references to DXGI objects -> I can't replay it. But yeah, this looks like it creates some framebuffers that it never frees afterwards.

apitrace thinks it only contains one frame and lacks some references to DXGI objects -> I can't replay it

Was dxgi.dll and dxgitrace.dll used?

Actually, now that Josh mentioned it, was the trace created on Proton? Proton uses Wine's DXGI now, so when recording a trace, you'll probably need to set the environment variable WINEDLLOVERRIDES=dxgi=n,b now.
Anyway, it's not important in this case since it seems to happen on Windows as well.

Ah, dang it!
Anyway, regarding the apitrace - I'm using proton. I had to enable apitrace by moving dlls, but I did not know about that override. Will do it the next time!
Since, it's a game bug, I'm closing the issue.


Proper PHP highlighting in readme

Highlights PHP code in example sections

Sorry for the long delay and thank you for this PR! This really enhances readability!

Well I can guess my PR has nothing to do with failed checks :)

No, but i wasn't able to merge it with the phone so i'll do this now!


502 BAD GATEWAY

Describe the bug
502 BAD GATEWAY is simply one problem
To Reproduce
click on anything clickable -- example:  https://www.commafeed.com/#/app/category/31724104
for the last week or so, the server becomes unavailable, sometimes for hours.  click on almost any page link and you either get a loading image that starts (and frequently stays) at 0%, or you get a bad gateway error
Expected behavior
click on link, go to a feed, or reload all feeds
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

commafeed.com (and anything underneath the site's home address)
latest chrome Browser
desktop

Additional context
for years, it's been stable.  in the last week, not so much.  seems like y'all have done a major UI change, and probably changed a bit of code underneath as well.  personally, if i had released this, i would have reverted days ago to find out why it's become so unstable.

Someone tried to upload an OPML file with 15k+ feeds and crashed the app. I'll add a check for this. There were indeed some instabilities last week with the deployment of CommaFeed 3.0 (https://github.com/Athou/commafeed/issues/1060#issuecomment-1565068616).
Note that I'm the sole developer of CommaFeed and www.commafeed.com is offered as a service for free, is costing me money and never was profitable. Comments like this make me question why I even bother.

apologies if my comment was poorly-worded.  perhaps you could use help with
testing the product before deploy?  might not catch every issue, but it
couldn't hurt.  i'm retired, and while i don't have a lot of free time, i
could spare a bit every now and then...  you can find me on linkedin as
"caren park", with over 40 years of experience in the computer field...
last positions included microsoft, chess4life, and precor...
On Tue, May 30, 2023 at 12:58 AM Jérémie Panzer @.***>
wrote:

Someone tried to upload an OPML file with 15k+ feeds and crashed the app.
I'll add a check for this. There were indeed some instabilities last week
with the deployment of CommaFeed 3.0 (#1060 (comment)
https://github.com/Athou/commafeed/issues/1060#issuecomment-1565068616).
Note that I'm the sole developer of CommaFeed and www.commafeed.com is
offered as a service for free, is costing me money and never was
profitable. Comments like this make me question why I even bother.
—
Reply to this email directly, view it on GitHub
https://github.com/Athou/commafeed/issues/1062#issuecomment-1567950471,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUHQF5LE3OZ6CW2UFWSFAXDXIWSEFANCNFSM6AAAAAAYTKSB4M
.
You are receiving this because you authored the thread.Message ID:
@.***>

It seems to be 502 now on commafeed.com. Can you please check it @Athou? Thanks!

With the server migration, this should not happen anymore, so I'm going to close this issue. If it happens again please open a new issue :)


Wrong Translation in notice_de.xml

Auftragsbekanntmachung – allgemeine Richtlinie, Sonderregelung
Auftragsbekanntmachung – allgemeine Richtlinie, Sonderregelung
The german translation for notice 21 is wrong.
The translation should be: "Auftragsbekanntmachung –Sektorenrichtlinie, Sonderregelung".

Thanks for spotting this - we will look into it asap.  It looks like the 2019 eForms regulation has this mistake.

The German translation will be corrected in the next amendment to the eForms regulation.
In the meantime we will also correct it in the next SDK release (and the label will be included in the patches to the previous SDK versions).
Thanks again for spotting this.

The corrected labels will be included in SDK 1.9.0.
See commit 292f3f2b.
So I'm closing this issue.


Implement legacyWatch parameter for ng serve

When using the command "ng serve" on a mounted fs (or in my case a vagrant shared folder), file changes aren't noticed.
Nodemon already solved this problem by adding a legacyWatch parameter:
https://github.com/remy/nodemon#application-isnt-restarting

We already have a poll option that we pass to webpack dev server. Try using ng set defaults.poll 100 (100 would be the interval, see here), and report back here if that doesn't help.
Closing this in the meantime. Cheers!

thx! It worked! You made my day!
$ ng set defaults.poll 100 && ng serve;


feat: add multiple theme support

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Track an ID per Sctp4j socket.

Use it, instead of memory pointer, for usrsctp pseudo-addresses.
Fixes external exposure of heap offset.

@ZeWaren : this fix will need the FreeBSD library recompiled as well.  Are you able to do that?


Buftabline in command bar?

I like the way bling/vim-bufferline displays buffer list in command bar but it lacks mappings options, make it hard to switch between buffers, any thought about it?

What options do you think should it have?
My thoughts are that it’s not this plugin that lacks convenient buffer switching, it’s Vim.
But it’s easy to make buffer switching more convenient in your ~/.vimrc… and leaving the choice to the user lets them pick the way they like. The simplest way (which is essentially what I use myself) is shown in the README. I’m not putting that bit in the plugin because then I’d have to add a setting to turn it off… and it’s so trivial it just doesn’t add enough value to justify that cost.
What’s not easy is getting a display of the buffer list. Hence this plugin. That’s its value proposition.
Adding more comprehensive buffer management might have value… but I’m not really the right person for that, since I don’t use any. And there are tons of plugins that address buffer navigation in some way – I don’t want to duplicate that functionality, and I especially don’t want to conflict with them. So I’d rather keep buftabline easy to combine with whatever buffer manager plugin you like.
 I’d like this plugin to be along the lines of You drop it into your Vim, It Just Works, and then you go on to learn about managing buffers in Vim… instead of learning buftabline.
Basically, I’d prefer it to be the missing buffer display built into Vim, as much as possible. (It kinda boggles the mind that Vim ships a built-in tab display but not a buffer display.)
Having said all that… I am open to considering suggestions contrary to that idea, even if I’m not likely to go that way. Particularly if you have an idea that would be hard to do as a separate plugin, I would be interested in hearing about it – though I’m trying hard to make sure such a thing cannot exist. 😊

Thanks for clarification, you did a great work with this plugin.


Update README.md

What:  added missing right curly brace in readme.md file

Why: to make the world a little bit better

How: forked the master branch and made the simple changes

Checklist:

[x] Documentation
[ ] Tests n/a
[x] Ready to be merged

[ ] Added myself to contributors table

Thanks for noticing!

@all-contributors add @adm1t for doc


[Table] Sticky header table with buttons/inputs

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
https://codesandbox.io/s/create-react-app-6v20t
Buttons and input are visible on top of the sticky header.
Expected Behavior 🤔
The buttons and input go below the sticky header.
Steps to Reproduce 🕹
Use a button or input field (I haven't checked anything else) in a table with sticky headers.
Context 🔦
I'm trying to make a table which has options within its rows.
Your Environment 🌎

Tech
Version

Material-UI
v4.4.0

React
v16.9.0

Browser
Any

@Studio384 Thanks for the report. It seems that the fix is simple, creating a stacking context address it:
diff --git a/packages/material-ui/src/TableCell/TableCell.js b/packages/material-ui/src/TableCell/TableCell.js
index a81b2417e..a034c45fe 100644
--- a/packages/material-ui/src/TableCell/TableCell.js
+++ b/packages/material-ui/src/TableCell/TableCell.js
@@ -101,6 +101,7 @@ export const styles = theme => ({
     position: 'sticky',
     top: 0,
     left: 0,
+    zIndex: 1,
     backgroundColor: theme.palette.background.default,
   },
 });

Thanks, Super easy fix, was obvious but didn't try until seeing this,
Why does this occur only on form control components and not on other components like ?

To be honest, I don't know why it happens with such component. Maybe we already create a stacking context for them?

Does anyone want to submit a pull request for the problem? :)

I can open a new issue for this, but I figure this is basically the same issue. Buttons work, but the checkboxes are still affected.
See: https://codesandbox.io/s/create-react-app-4stgk

@Lavoaster I can't reproduce it. What environment are you using? Do you have a screenshot,

@oliviertassinari Sorry, I just realised that the issue is a little bit different.
In this case if you try to check the box in the header, whilst a checkbox is beneath it, it checks the the one beneath the sticky header.
I'm on mac OS 10.15, with Chrome77.0.3865.120, I also tested this in safari and the same issue occurs.

I can reproduce it too. The z-index: 1 added in #17389 conflicts. What about increasing the z-index to 2 for the sticky header?

@oliviertassinari That makes it work for me.

@Lavoaster Do you want to submit a pull request? Or do you have a better fix in mind? :)


Not Searching Support

Device Info:

Device model: Google Pixel 3a XL
Screen size: 2,220 × 1,080
Android Version: Android 10
RAM: 4 GB

Downloaded app earlier today, been experimenting with it. I made a basic setup to farm aurora steel and it was working fine, but when I went to use it later in the day I noticed it was no longer scrolling through my friends list to find a suitable support (Caster, MLB Lunchtime Waver 10/10/10, Friends Only) as it had been previously. Instead it correctly navigates to the Caster lineup, but then just sits there for a few seconds (instead of slowly scrolling) until it hits the refresh button, then sits again, repeating a few times until it just gives up and goes to the fallback (set to manual).
Edit: I have no other recorders or overlays that would be interfering, that I'm aware of.

@welknair Seems like the template doesn't work if you're not logged in from the start.
new issue -> log in -> no template
log in -> new issue -> selection -> template


updated package.json

updated package minimist from 1.1.0 to 1.2.0
Npm link :  https://www.npmjs.com/package/minimist
Repository link : https://github.com/substack/minimist/releases

What is the point of this PR?


Open JS logo

Thanks for opening the PR!
I think it would be helpful to have the files added to this repo as individual files in a directory/folder instead of compressed as a .zip file. It would let folks link to them individual when needed.

Done!

This looks so good! Quick bit: is that a typo in the identity pdf? “Typface”

Thanks!  I am not quite sure what you are referring to.  Can you give me a
bit more information?
On Wed, Apr 17, 2019 at 7:16 PM Ryan Block<EMAIL_ADDRESS>wrote:

This looks so good! Quick bit: is that a typo in the identity pdf?
“Typface”
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openjs-foundation/bootstrap/pull/156#issuecomment-484329686,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKYQ3QIZ6VUCHE3V7YQMOJ3PQ7KWXANCNFSM4HGY4KLA
.

--
Eva Howe
This Dot Labs
<EMAIL_ADDRESS>

Thanks! I am not quite sure what you are referring to. Can you give me a bit more information?
…
On Wed, Apr 17, 2019 at 7:16 PM Ryan Block @.***> wrote: This looks so good! Quick bit: is that a typo in the identity pdf? “Typface” — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#156 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AKYQ3QIZ6VUCHE3V7YQMOJ3PQ7KWXANCNFSM4HGY4KLA .
-- Eva Howe This Dot Labs<EMAIL_ADDRESS>
I think they're referring to this in the PDF file in this PR:

I imagine Typface is supposed to say Typeface instead?

Yes, I suspect it is.  I am so sorry - I am afraid I don't have an editable copy of the PDF.

@Trott @evahowe yes, that's what I was referring to. Sorry that wasn't more specific, happened to notice it from my phone, that comment could have used more detail!

@ryanblock ok, that makes so much more sense now! :)  It is always hard to do things on your phone!

@evahowe I'm wondering about the context for this.  Are these being added so we can use them in the pages within github, so that people can use them in presntations or something else?

@mhdawson A bit of both.  @ladyleet wanted me to put them there so people can use them when they need them. :)

I will have the typo fixed. Thought it would be good to have the OpenJS Foundation logo and brand guidelines and all future project logos in one repo so that people always know where to find the latest and correct logos. This can be our source of truth for all logos.

Can this be merged or does it need to be discussed at a Monday meeting?

We've often got confirmation in the Monday meetings, but we should not always need to do that going forward.  Maybe we should discuss in the next meeting and see if there is consensus to moving to a 2 approvals (possibly 2 cpc member approval in the future) and then it can land?

I would add a 48-hour grace period for reviews, if not 72. Most of the issues are not urgent or blocking, so even waiting for some time should be ok.

+1
Folks who work on open source part-time or on personal time may not have daily time for review. 72 hours hours makes it more likely that they have the opportunity to engage constructively. fwiw, 72 hours hold time is used for governance-related issues and PRs in the Node.js project, and there haven't (yet) been objections to that process :)

+1 to 72 hours, possibly longer for PR's that affect governance.

I have an updated copy of the openjs brand guide that fixes the typo.  I wanted to upload it, but when I go to openjs-foundation/bootstrap I don't see the file "logos"  Where did it go?  Thanks!

To help me understand, do you mean that you're opening up the "bootstrap" directory on your machine and the "logos" directory is not in there?

I think I may have just had a brain fart, I went back to it again and I
think I did the right thing.  Deleted the brand guide with the typo and
uploaded the new one. Please let me know!
On Thu, Apr 25, 2019 at 3:11 PM Manil Chowdhury<EMAIL_ADDRESS>wrote:

To help me understand, do you mean that you're opening up the "bootstrap"
directory on your machine and the "logos" directory is not in there?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openjs-foundation/bootstrap/pull/156#issuecomment-486856149,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKYQ3QMTIUEZUF3MPFYQ6IDPSIUAHANCNFSM4HGY4KLA
.

--
Eva Howe
This Dot Labs
<EMAIL_ADDRESS>

So this can be merged now?

Thanks for PR @evahowe, and for the reviews folks. It's already been useful, folks have used the logo.

yay!  That makes me happy :)
On Mon, May 6, 2019 at 11:48 AM Manil Chowdhury<EMAIL_ADDRESS>wrote:

Thanks for PR @evahowe https://github.com/evahowe, and for the reviews
folks. It's already been useful, folks have used the logo.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openjs-foundation/bootstrap/pull/156#issuecomment-489731885,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKYQ3QNV5QDFB23KUUASVMTPUB4RDANCNFSM4HGY4KLA
.

--
Eva Howe
This Dot Labs
<EMAIL_ADDRESS>


[bugfix] first call failure problem when using nacos registery

What this PR does:

fix first call failure problem when using nacos registery

Codecov Report

Merging #380 (5377124) into develop-0.5.0 (44f09ad) will decrease coverage by 0.06%.
The diff coverage is 0.00%.

@@                Coverage Diff                @@
##           develop-0.5.0     #380      +/-   ##
=================================================
- Coverage          39.66%   39.59%   -0.07%
=================================================
  Files                 52       52
  Lines               3061     3066       +5
=================================================
  Hits                1214     1214
- Misses              1717     1722       +5
  Partials             130      130

Impacted Files
Coverage Δ

pkg/client/dubbo/dubbo.go
19.46% <0.00%> (-0.68%)
: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 44f09ad...5377124. Read the comment docs.


Support already factored H in DAQPProblem (#54)

include/types.h (DAQPProblem): Add is_factored field and document it.
src/utils.c (update_Rinv): Handle it: adapt the diagonal special case, and skip the factoring in the general case.
Fixes #54.

Thank you for this contribution! I think that the is_factored field might serve better in DAQPSettings rather than in the DAQPProblem. The main reason is that I want to keep the main API minimal, and adding the field is_factored would require the user to set it to zero in the "default" case  (while having it as zero in the default settings would keep it "invisible" to the average user)

adding the field is_factored would require the user to set it to zero in the "default" case

The compiler will actually automatically default the member at the end of the struct to 0 if it is not specified (if the initializer is "too short"). (It may or may not warn about that.) That is why I have added it to the end of the struct.
See, e.g., ISO C99 (9899:1999, or WG14/N1124 which is C99+TC1+TC2) 6.7.8§21, or the corresponding paragraph in ISO C90 6.5.7:

If there are fewer initializers in a brace-enclosed list than there are […] members
of an aggregate, […] the remainder of the aggregate shall be initialized implicitly
the same as objects that have static storage duration.

(The parts I cut out were added in C99 and are not relevant to the struct case.)
So, while we can move it to DAQPSettings, I am not yet convinced that that is an improvement. The flag says how a field in DAQPProblem must be interpreted. Even the dimension of the array H is different based on that flag. Having it in a different struct is just asking for mismatches.
Though in the end it is your code and your decision.


[HxInputBase] Allow for CustomValidation Context and Propety and Add LabelType into application-wide Defaults

Linked Issue
#872

https://github.com/havit/Havit.Blazor/issues/872

Input components (HxInputBase descendants) that do not support LabelType:

HxCheckbox, HxSwitch
HxCheckboxList, HxRadioButtonList
HxInputDateRange
HxInputRange
HxInputSelect - can we add support here?

...Adding LabelType to InputSettings will introduce this property to the settings/defaults of all inputs, which could lead to some confusion. I'm considering an alternative approach by adding a brand new ambient defaults such as HxSetup.Defaults.LabelType and HxSetup.Defaults.InputSize. I'll discuss this with my colleagues.

@mmonteagudo Thanks a lot for your contribution! It made me further develop the idea of "global defaults for all inputs," and I’ve added HxSetup.Defaults for these global settings, which currently allow setting LabelType and InputSize.
Apologies for not accepting the custom validation parameters idea. I’m concerned that this feature could introduce more issues than the value it adds. We might revisit this later, but for now, I’ve decided against it, as there are possible workarounds that can help you achieve the same goal without the proposed changes.

@mmonteagudo Thanks a lot for your contribution! It made me further develop the idea of "global defaults for all inputs," and I’ve added HxSetup.Defaults for these global settings, which currently allow setting LabelType and InputSize.
Apologies for not accepting the custom validation parameters idea. I’m concerned that this feature could introduce more issues than the value it adds. We might revisit this later, but for now, I’ve decided against it, as there are possible workarounds that can help you achieve the same goal without the proposed changes.

No worries, I understand.
That's an important feature for our implementation, so we are using it from our fork of Havit, so we just keep it up to date with latest updates and deploy a new package in our private package manager.
I do have a couple of customizations that works for our specific implementation, so I understand. :), Thanks for accepting the other change, that one was pretty important too.


Add Spanish u diéresis (ü)

Small PR to add the Spanish ü (known as "la diéresis", used in words like "bilingüe") to the Spanish unicode list. This can be achieved by also importing german.dtsi and using &de_ue, but this is more convenient and semantically appropriate for Spanish.

Looks good, thanks!


JSDom import causing error on Module initialization

I'm getting an error when I
 require 'zombie'

For version 0.10.1.  Seems to be related to the recent update of JSDOM (perhaps)
  TypeError: Cannot read property 'prototype' of undefined
      at Object.<anonymous> (/Users/phil/Dropbox/Code/JavaScript/open.js/open.core/node_modules/zombie/lib/zombie/jsdom_patches.js:4:17)

Seems to do it consistently across OSX and a couple of Ubuntu deployments I've tried it on.
The code the error message is referring to is:
core = require("jsdom").dom.level3.core;
.....
core.HTMLElement.prototype.__defineGetter__("offsetLeft", function() {
  return 0;
});

So core might have changed.

Fixed in master, and 0.10.2 coming out shortly.

You're awesome. Thanks assaf


annotation bundle is empty

The produced and released annotation artifact is empty (only MANIFEST.MF), so it's could not be used. Never version of maven-bundle-plugin has change some behaviour, so Private-Package header now is needed

R: @jbonofre


gRpc C#- Server with SslServerCredentials - Failed to bind port "<IP_ADDRESS>:50051"

I am getting the following error while trying to create gRpc server with SSL.
Unhandled Exception: System.IO.IOException: Failed to bind port "<IP_ADDRESS>:50051"
at Grpc.Core.Server.CheckPortsBoundSuccessfully() in T:\src\github\grpc\src\csharp\Grpc.Core\Server.cs:line 313
at Grpc.Core.Server.Start() in T:\src\github\grpc\src\csharp\Grpc.Core\Server.cs:line 155
gRpc version: 1.21.0
OS: Windows 10 1803
Dotnet Core 2.2
server core
          var server = new Server             {                 Services = {MessageBrokerV3Service.BindService(new MockBrokerV3ServiceImpl())},                 Ports = { new ServerPort("<IP_ADDRESS>", 50051, new SslServerCredentials(new []                 {                     new KeyCertificatePair(certificateChain, privateKey),                  })) }             };             server.Start();

My guess is that this doesn't seem to have to do anything with the fact you're using SSL. It just seems that the port is already bound by some other server. Have you tired binding another port?
You can get more logs by following https://github.com/grpc/grpc/blob/master/TROUBLESHOOTING.md

I'll reopen if the problem persists.


tcp input and kafka output processors

@vjeantet @AlexAkulov The code additions in this PR are not impacting test coverage according to this report, even though the report as a whole sees a minor decrease in coverage.

@vjeantet please merge


BlockNumber returned as TransactionIndex from getTransactionDetails()

While trying to get the transaction details for a given hash I noticed I'm getting the block number being reported for both the BlockNumber and TransactonHash vars
        tests.forEach { entry in
            let testProvider = Web3HttpProvider(URL(string: entry.url)!)!
            let node = web3(provider: testProvider)

            output.append("\n********************************\n")
            output.append("Test: \(entry.name)\n")

            do {
                let txn = try node.eth.getTransactionDetails(entry.id)
                output.append("Block Hash: \(txn.blockHash?.toHexString().addHexPrefix() ?? "nil")\n")
                output.append("Block Number: \(String(txn.blockNumber ?? "0"))\n")
                output.append("Transaction Index: \(String(txn.transactionIndex ?? "0"))\n")

            } catch {
                output.append("FAILED\n")
            }
            output.append("********************************")
        }

From this I'm getting the following when run on the Binance mainnet [also get the same behaviour on testnet] (unable to test on Ethereum due to Issue:463 )
********************************
Test: BSC - Good Transaction
Block Hash: 0xb72e...7764
Block Number: 15003247
Transaction Index: 15003247
********************************
********************************
Test: BSC - Failed Transaction
Block Hash: 0xb72e...7764
Block Number: 15003247
Transaction Index: 15003247
********************************

Looking into the source, I believe the following lines [195,196] in Web3+Structures.swift are the issue
        let transactionIndex = try decodeHexToBigUInt(container, key: .blockNumber, allowOptional: true)
        self.transactionIndex = transactionIndex

I imagine the key should be .transactionIndex


how can i increase gpu utilisation from 80% to 100%

currently, fluxgym is only using 80% of vram.

I believe the VRAM consumption is not a fixed number that has been defined somewhere and can be adjusted precisely.
Rather, the developer has summarized some common training settings, parameters and workarounds in profiles, which then result in approximately the specified VRAM consumption (see app.py,  line 400...).
I don't know whether the combination of these command line parameters and the resulting VRAM consumption is already documented somewhere or whether the developer came up with these profiles himself, but in any case it works very well (for me). I am also very grateful to the developer that he has massively lowered the entry barrier for beginners like me because of these profiles. I think the fact that you can switch so easily between the different profiles is one reason why fluxgym has been so well received and is so beginner-friendly.
From a purely technical point of view, I would say that moving even closer to 100% VRAM consumption could be problematic. This is because there is an increased risk that the VRAM will suddenly no longer be sufficient due to a combination of other parameters. The result would be that the training is aborted in the middle or does not start at all.

When using the 12GB version and playing around with batch workers ( works best for me with around 42 ) then it uses around 7.6 GB of VRAM (of which 0.3GB are shared)


Refreshing installation instructions

Added conjure-up. Updated displays and juju versions to current versions.

This change is

/assign @bradtopol

Thanks for the help @bradtopol. I have updated the anchors.

/approve
:wave: @bradtopol take a look at the changes - if you're good, please add an /lgtm to the PR and it'll merge

Great job!
/lgtm

Thanks for fixing the merge conflicts!
/lgtm