🐛 Popup Camera still show after unmounted Camera screen

What were you trying to do?
I want to use front camera device to take a picture
Reproduceable Code
No response
What happened instead?
After leave camera screen, I still see popup camera still here. It couldn't hide unless I close app.
Relevant log output
No response
Device
VSmart Active 3
VisionCamera Version
2.10.0
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

Not enough reproduction steps for me.

Sorry about missing reproduce code cos I think it's just simple code , I created this issue cos I see my popup camera in keep showing after I close screen having your  component:

Camera is working fine, issue is just popup camera didn't hide after unmount

What is popup camera?

That's what I'm refering.

Lol, I don't have a device like this to test. Maybe there is an additional step involved in dismissing this again? Does it also happen if you set active to false or completely unmount the camera but stay on the same screen?

Yes I tried to set active to false and keep staying on same screen but the popup camera is still not hiden

I have same issue

Hey! I just found out that I really forgot to close and dispose the locked Camera resources, so I just fixed that in this PR: https://github.com/mrousavy/react-native-vision-camera/pull/2174
Now the Camera fully & synchronously closes all resources (CameraSession, CameraDevice, OpenGL context, Video & Photo outputs, Photo Synchronizer) once the view gets removed from the React view hierarchy ("unmounted"), and things like Flash, Torch, NFC, and other Camera components should be working again.
There is still a small issue that causes once Camera component to turn into a blackscreen when navigating back and forth between two Camera components, that's a pretty rare edge case but I will still try to fix that soon when I have some free time.
If you appreciate my time, expertise and dedication to this project, pleas 💖 consider sponsoring me on GitHub 💖 to support the development of this project.


Add the salt-overlay directory to the GCE master-pd and reserve the master's IP

...upon creation to make it easier to replace the master later.
This pulls out the parts of PR #3174 that don't break anything and will make upgrading existing clusters in the future less painful. #3174 will still need more effort to make work, unfortunately.

I've also added /etc/salt/, as /etc/salt/pki/ contains the dynamically generated RSA keys used by salt. Without saving it, the salt minions on the nodes don't trust a replacement master.

LGTM. Will merge on Travis.

Travis is being dumb, and won't show anything useful here. Rolling the dice.


Provide automated canary deployment and analysis support when applying changes to running aerospike cluster

We are using netflix kayenta as service to automatically analyze canary deployments
When making changes to running aerospike cluster we want to avoid downtime if changes are breaking cluster

https://www.infoq.com/presentations/canary-analysis-deployment-pattern/
https://github.com/spinnaker/kayenta
https://www.youtube.com/watch?v=3xQm7CsR0ww

So, MVP here - when applying changes operator should support canary deployment and we should have ability to programmatically continue or rollback deployment via deployment pipeline based on data from canary analysis which pipeline will fetch from kayenta

https://github.com/helm/helm/issues/6572

We will try to get a roadmap for this feature.


[BUG] Dead Link in Docs

Describe the bug
There is a dead link in the Quick Start section of the docs. It is the Supported models page.

Choose a model from the list of models available on Ollama. Two recommended models to get started are codellama:7b-instruct for chat and codellama:7b-code for fill-in-middle. See the Supported models page for more options.

Expected behavior
The correct link should be this.

Thanks, feel free to make a PR :D


Add sample_rate and update version of otel collector

Adds sample_rate to the collector configuration and bumps the release to the latest.

Thanks for the merge!


Bump parallelio, eccodes, py-eccodes versions

Description
Update eccodes to 2.27.0 and py-eccodes to 1.4.2 as requested by @gthompsnJCSDA. Also bump parallelio to 2.5.7.
Tested to work on Orion with intel and on macOS with apple-clang, in addition to the CI tests (which test building only).
Waiting on https://github.com/NOAA-EMC/spack/pull/170
Note. Unit test failure https://github.com/NOAA-EMC/spack-stack/actions/runs/3047150784/jobs/4910815663 will be fixed in one of the next spack (submodule) PRs (https://github.com/NOAA-EMC/spack/pull/171)

Looks good to go as long as the only test failure is the one unit test noted in the description. Thanks!

Thanks! Yes, the unit test is the only failure and will be fixed with the next spack PR.


How-to article about AWS Timestream - The new Time Series database from AWS to work with IoT products.

Hi, let me know if this addition makes sense for you:
https://itnext.io/how-to-begin-with-the-amazon-timestream-in-5-simple-steps-19c129040d9c
This time series is awesome and help a lot who is new in this field, I wrote an article to help who is starting on this world, also who would like to test this new AWS managed time-series database
AWS Timestream is a time-series database to collect IoT data over-time.

sorry, in current design of this repository, I would like to collection a series of some topics but not just a alone article.


Translate "How to interpret phylogenetic trees" into Indonesian

Hi @kumincir & @sentausa,
I've really appreciated your efficiency and hard work on situation reports week after week. I'd really love to translate How to interpret phylogenetic trees into Indonesian as I think this could help make these reports even more accessible.
I've created a branch trees_id to translate this on. The file to translate is here: https://github.com/nextstrain/translations/blob/trees_id/narratives/translations_trees-background_id.md
The link to preview is here: https://nextstrain.org/community/narratives/nextstrain/translations@trees_id/trees-background/id
Extra care should be taken with this translation as it's quite technical and will be referenced in every future report.
Thanks so much!
Cassia

Hi @cassiawag,
It's a really great and timely idea, as there are now 3 sequences from Indonesia on Nextstrain, and people (at least the scientists, and one non-scientist who tagged both @kumincir & me on Facebook) there started to check them out. So yeah, I think it'll be great if we translate it.

@cassiawag I just saw a typo in the original text. In slide 4 (How does the "diversity" panel relate to the tree?), second paragraph, last sentence, the word "intimitely" should be "intimately".

Thanks @sentausa; I'll get that typo fixed. Glad to hear you're on the translation!

Finished translating, waiting for @kumincir for review

Ready for publication :)

Thanks so much @sentausa & @kumincir! I've published it here: https://nextstrain.org/narratives/trees-background/id
I was also excited to see the Indonesian sequences on Nextstrain.


Mapillary Coverage for OSM Contributions- Collaboration?

Hi All,
Is anyone interested in coordinating for Mapillary Contributions to get a resource on the ground in terms of imagery that we can use for OSM Contributions?
Just curious if anyone is interested. I have found it helpful for making contributions quickly with a good deal of leg work done for me.

Hmm.  One issue is getting more mapillary images.  A separate issue is #9, using the imagery to get work done.

I edited the issue to be more specific to imagery collection.

We are getting a rig from Mapbox courtesy of @1ec5 .


Errors from splunk-fluentd-k8s-logs container

With query: index=eks_logs sourcetype="kube:container:splunk-fluentd-k8s-logs"
I am frequently seeing the following error in the logs:
[warn]: #0 failed to flush the buffer. retry_time=0 next_retry_seconds=2018-12-03 17:42:26 +0000 chunk="57c21abb713ca798a76f45298b755dcb" error_class=SocketError error="Failed to open TCP connection to ..com:443 (getaddrinfo: Name or service not known)"
I'm using port 443 as my Splunk hosts are sitting behind an F5 vip (vip:443 -> splunk hosts:8088)
Everything else (logging, metrics, objects) seems to be working......so what do these errors mean?  Can they be ignored?
Splunk 7.1.3

Edit:  I purposely omitted by FQDN above for security.  It is correct in the actual output.  It seems the DNS settings are unknown inside the container.

Hey @dwayne-lee!
What does a timechart of those errors look like split by host, say over a 7 day period in Splunk?
Would be interesting to understand the frequency of the errors.
Also can you tell us more about your kubernets deploy? Version, which DNS provider, etc?
Thanks!

ALL Hosts:      72,744 events	(11/29/18 8:00:00.000 AM to 12/6/18 8:32:31.000 AM)
A Single Host:  18,029 events	(11/29/18 8:00:00.000 AM to 12/6/18 8:36:54.000 AM)
[root@kube-master ~]# kubectl describe node kube-master
Name:               kube-master
Roles:              master
Labels:             beta.kubernetes.io/arch=amd64
beta.kubernetes.io/os=linux
kubernetes.io/hostname=kube-master
node-role.kubernetes.io/master=
Annotations:        flannel.alpha.coreos.com/backend-data: {"VtepMAC":"3a:52:61:aa:73:88"}
flannel.alpha.coreos.com/backend-type: vxlan
flannel.alpha.coreos.com/kube-subnet-manager: true
flannel.alpha.coreos.com/public-ip: <IP_ADDRESS>
kubeadm.alpha.kubernetes.io/cri-socket: /var/run/dockershim.sock
node.alpha.kubernetes.io/ttl: 0
volumes.kubernetes.io/controller-managed-attach-detach: true
CreationTimestamp:  Tue, 13 Nov 2018 19:40:17 +0000
Taints:             node-role.kubernetes.io/master:NoSchedule
Unschedulable:      false
Conditions:
Type             Status  LastHeartbeatTime                 LastTransitionTime                Reason                       Message

OutOfDisk        False   Thu, 06 Dec 2018 13:43:59 +0000   Tue, 13 Nov 2018 19:40:08 +0000   KubeletHasSufficientDisk     kubelet has sufficient disk space available
MemoryPressure   False   Thu, 06 Dec 2018 13:43:59 +0000   Tue, 13 Nov 2018 19:40:08 +0000   KubeletHasSufficientMemory   kubelet has sufficient memory available
DiskPressure     False   Thu, 06 Dec 2018 13:43:59 +0000   Tue, 13 Nov 2018 19:40:08 +0000   KubeletHasNoDiskPressure     kubelet has no disk pressure
PIDPressure      False   Thu, 06 Dec 2018 13:43:59 +0000   Tue, 13 Nov 2018 19:40:08 +0000   KubeletHasSufficientPID      kubelet has sufficient PID available
Ready            True    Thu, 06 Dec 2018 13:43:59 +0000   Tue, 13 Nov 2018 19:40:47 +0000   KubeletReady                 kubelet is posting ready status
Addresses:
InternalIP:  <IP_ADDRESS>
Hostname:    kube-master
Capacity:
cpu:                2
ephemeral-storage:  8181Mi
hugepages-2Mi:      0
memory:             3879568Ki
pods:               110
Allocatable:
cpu:                2
ephemeral-storage:<PHONE_NUMBER>
hugepages-2Mi:      0
memory:             3777168Ki
pods:               110
System Info:
Machine ID:                 b30d0f2110ac3807b210c19ede3ce88f
System UUID:                EC219C1E-B496-9EBA-B585-3F6D4697A113
Boot ID:                    51214336-cbca-4a97-843e-d4789ea81ae3
Kernel Version:             3.10.0-862.3.2.el7.x86_64
OS Image:                   CentOS Linux 7 (Core)
Operating System:           linux
Architecture:               amd64
Container Runtime Version:  docker://18.3.1
Kubelet Version:            v1.12.2
Kube-Proxy Version:         v1.12.2
PodCIDR:                     <IP_ADDRESS>/24
Non-terminated Pods:         (7 in total)
Namespace                  Name                                   CPU Requests  CPU Limits  Memory Requests  Memory Limits

kube-system                etcd-kube-master                       0 (0%)        0 (0%)      0 (0%)           0 (0%)
kube-system                kube-apiserver-kube-master             250m (12%)    0 (0%)      0 (0%)           0 (0%)
kube-system                kube-controller-manager-kube-master    200m (10%)    0 (0%)      0 (0%)           0 (0%)
kube-system                kube-flannel-ds-amd64-wjctz            100m (5%)     100m (5%)   50Mi (1%)        50Mi (1%)
kube-system                kube-proxy-4wrrf                       0 (0%)        0 (0%)      0 (0%)           0 (0%)
kube-system                kube-scheduler-kube-master             100m (5%)     0 (0%)      0 (0%)           0 (0%)
splunk-connect-k8s         ql-splunk-kubernetes-logging-kvmvt     100m (5%)     0 (0%)      200Mi (5%)       0 (0%)
Allocated resources:
(Total limits may be over 100 percent, i.e., overcommitted.)
Resource  Requests    Limits

cpu       750m (37%)  100m (5%)
memory    250Mi (6%)  50Mi (1%)
Events:

To me, it appears you are having some DNS challenges...I find it strange you say it is working though...are all your expected hosts showing up in Splunk?
Do you have pod with nslookup in it? The logging pod does not have it installed, though if this is a test enviro, it can be installed by hand be exec'ing into the pod, or dropping utility pod in the namespace.
Can I assume this is a test environment?? You have named the index eks_logs but you don't appear to be using eks....

Any updates on this?

Hey @dwayne-lee - will close issue. Please re-open with any updates or if you continue to have issues.


Describe how to use docker-compose with react SPA created by Vite

Hi, I found it useful to configure Docker support for React + .NET Core but there is no information how to use it with docker-compose. I think it could be helpful for anyone who already usese docker-compose. I mean mostly, how to configure Debug (e.g. start multiple projects: docker-compose and Vite) and Release configurations (e.g just docker-compose).

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

ID: 1abf884f-eb39-1007-25a6-d049b87504a3
Version Independent ID: 33636e79-d3e1-8c24-f3a5-0c3c79e2a488
Content: Build React app with Visual Studio Container Tools
Content Source: docs/containers/container-tools-react.md
Service: visual-studio-windows
Sub-service: container-tools
GitHub Login: @ghogen
Microsoft Alias: ghogen

@mikolajpiecuch
This is a potentially interesting scenario for a blog post, but it's not currently considered a core scenario for our documentation. Let us know if you get this working, and consider sharing your solutions and posting a link to your work.


feat(#345): Add unit test for XmlAnnotations

In this PR I added unit tests for XmlAnnotations and refactored BytecodeAnnotations usage.
Closes: #345.
History:

feat(#345): remove several redundand methods
feat(#345): fix all the unit tests
feat(#345): fix all the unit tests
feat(#345): add unit test for XmlAnnotations
feat(#345): fix all the code offences

PR-Codex overview
The focus of this PR is to refactor the handling of annotations in various classes by introducing the BytecodeAnnotations class.
Detailed summary

Refactored classes to use BytecodeAnnotations for annotations handling
Added BytecodeAnnotations class for better annotation management
Replaced list of annotations with BytecodeAnnotations object in multiple classes

The following files were skipped due to too many changes: src/main/java/org/eolang/jeo/representation/bytecode/BytecodeMethodParameter.java, src/test/java/org/eolang/jeo/representation/xmir/XmlAnnotationsTest.java, src/main/java/org/eolang/jeo/representation/bytecode/BytecodeMethod.java

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 28min)

@volodya-lombrozo Thank you for your contribution! We appreciate your effort, but there are some areas for improvement. Your submission earned +4 points as a base reward, but we had to deduct points due to the following: -8 for exceeding the recommended hits-of-code (260 ≥ 200), -16 for lacking a code review, and -4 for having only 2 comments. However, we've adjusted the total to ensure you receive at least the minimum reward. Your final point allocation is +4. We encourage you to focus on quality in future contributions. Keep up the good work! Your current balance stands at +93.


Added launch files for different situations.

The ones for neato and alameda launch the navigation and perception nodes.

Updated the launch files.


RTM is deprecated: not_allowed_token_type

Slack is in progress of deprecating RTM protocol in favor of events (which require the bot to roll a whole http server).
To reproduce:

Create new app on https://api.slack.com/apps?new_app=1
Add your oauth scopes: https://api.slack.com/apps/AAAAAAA/oauth?
Run the example code:

slacker.Error: not_allowed_token_type

To fix:

Create app through classic app endpoint: https://api.slack.com/apps?new_classic_app=1
Set "bot" oauth scope.
Add legacy bot user (App Home) section

However this endpoint is deprecated and seems like it will be disabled in favor of the god-awful events api that requires a full web server running waiting for events.
Additionally classic apps are not allowed on slack's app directory.K

Hi, I hope this helps you:
I was hoping for slackbot to change with Slack's new events method. I also asked https://github.com/lins05/slackbot/issues/209
The problem is the way slackbot connects to slack.com. I attempted to recode slackbot to work with rtm.connect() as oppose to rtm.start() which is what slackbot uses right now. Its been a good learning experience, so I'm leaving slackbot behind and creating a custom app as it seems its the future of slack.
As a quick solution I moved to errbot https://github.com/errbotio/errbot I found it easier to quickly remake the plugins to work with errbot which supports the rtm.connect() method. Here you can get a legacy token from Slack.
https://api.slack.com/custom-integrations/legacy-tokens
I also am a big fan of slackbot, I hope the developers wish to continue working on it.

@a0149659 Could you elaborate more what you mean? slackbot works fine as long as you establish a legacy app.
The problem I raised in this issue that slack is slackbot raises a vague error, instead it should be capable of understanding that it's connecting to a non-legacy app and can't function.
I don't think supporting events is a good idea. Running a bot as web service is just an absolutely awful idea no matter how you look at it and Slack will most likely not get rid of legacy apps anytime soon.

I apologize, I was talking about Slack Enterprise and we found the issue is due to rtm.start vs rtm.connect. I assumed due to the error mentioned and the solution, that you we're facing the same issue. Slackbot doesn't work with Slack Enterprise as is with rtm.start() however you're correct it works fine with Workspaces.

I just tried this bot, and got the same error with a new user bot token.
└─[$] <git:(develop) > python run.py
Traceback (most recent call last):
  File "run.py", line 23, in <module>
    main()
  File "run.py", line 19, in main
    bot = Bot()
  File "/Users/i530566/devel/slack/slackbot/slackbot/bot.py", line 20, in __init__
    self._client = SlackClient(
  File "/Users/i530566/devel/slack/slackbot/slackbot/slackclient.py", line 43, in __init__
    self.rtm_connect()
  File "/Users/i530566/devel/slack/slackbot/slackbot/slackclient.py", line 46, in rtm_connect
    reply = self.webapi.rtm.start(**(self.rtm_start_args or {})).body
  File "/Users/i530566/devel/slack/slackbot/.venv/lib/python3.8/site-packages/slacker/__init__.py", line 818, in start
    return self.get('rtm.start',
  File "/Users/i530566/devel/slack/slackbot/.venv/lib/python3.8/site-packages/slacker/__init__.py", line 118, in get
    return self._request(
  File "/Users/i530566/devel/slack/slackbot/.venv/lib/python3.8/site-packages/slacker/__init__.py", line 102, in _request
    raise Error(response.error)
slacker.Error: not_allowed_token_type


Stockfish crashes on invalid position instead of error mesasge

When stockfish is given a position where both kings are in check, such as 1q2nB2/pP1k2KP/NN1Q1qP1/8/1P1p4/4p1br/3R4/6n1/ w, stockfish will simply crash. Is there a way to instead of crashing throwing an error message, as then it would be possible to use a try: except: statement to filter out these positions?

I would also like to have an error message for invalid FENs that can be caught.
I think a change to the _read_line-function like this would work:
def _read_line(self) -> str:
    if not self._stockfish.stdout:
        raise BrokenPipeError()
    if self._stockfish.poll() is not None:
        raise AttributeError(
            "Stockfish has crashed. This was probably caused by an invalid FEN."
        )
    return self._stockfish.stdout.readline().strip()

What do you think @zhelyabuzhsky? I could add that, some checks with invalid FENs and an entry in the README and then open a PR if you want this feature to be included.

@kieferro I think that would be good to include. An invalid FEN can cause SF to either crash, or return (none) for the best move (in both cases, after giving it the "go" command). It looks like your code handles the first case. I'm writing some stuff which could help with that, and also detect the second case.
Would u like me to make a PR to your repo? We could work on your branch together and then when done you can make the PR to the official repo here.

@johndoknjas Okay, I've already included my code and a unittest on my fork. Feel free to take care of the second case. As long as both cases return the same kind of error, everything should be fine.
Do you want me to make the change to the README as well? I would just add a section and write that when an invalid FEN is entered, a ValueError with the text "Stockfish has crashed. This was probably caused by an invalid FEN." gets raised and maybe explain how to catch it. Does this fit with what you are planning on doing?

@kieferro I think your error message makes sense, since the process just stopping has a good chance of being caused by a bad FEN. Although in the readme, you could say that it's not guaranteed invalid FENs will cause a crash. The stuff I'm doing more or less tries to deal with these kinds of FENs. So I think what you have fits with my code, and updating the readme is fine. My PR will involve some stuff other than fen validation, so I'll probably wait until yours is merged into master first. As far as error messages go, right now my plan for my function is to just return a bool instead, and then say in the readme that the user can call this function to test their FEN. For your code though, returning an error should be done, since generally if read_line is called, errors aren't the expected behaviour.

@johndoknjas Yes, that makes sense. Thank you


design and implement website describing capa and showing the available rules

We'd like to have a homepage for capa that describes the tool, exposes the current rule set, and provides other features. We can deploy the website via Github Actions to Github Pages, which would result in the url: https://mandiant.github.io/capa
### Tasks
- [ ] website: mockup layout
- [ ] website: pick tech stack
- [ ] website: configure GH Actions to build and deploy website
- [ ] website: implement home page
- [ ] website: implement rule view page
- [ ] website: implement rule list page
- [ ] website: implement search rules functionality
- [ ] website: demonstrate capa output via xterm.js
- [ ] website: use pyodide to run capa in the browser
- [ ] website: show example results for each rule

Specifications by @deeyasingh:
Homepage:
The homepage features a clean and modern design with the capa project logo and a brief introduction to Capa's capabilities.
It includes a prominent call-to-action button inviting users to explore the rule library.
Rule Browsing:
The rule browsing section provides a grid or list view of available rules.
Each rule card displays the rule's name, a short description, and metadata such as author and date of creation.
Users can click on a rule card to view the rule details.
Rule Details:
The rule details page presents comprehensive information about a specific rule.
The top section includes the rule's name, description, and metadata.
The middle section provides a more detailed explanation of the rule's purpose, behavior, and detection logic.
The bottom section may include example code snippets or screenshots illustrating the rule's application.
Search and Filter:
The website includes a search bar at the top for users to search for rules by keywords or metadata.
Users can also use filters to narrow down the rule list based on specific criteria such as rule type or category (If necessary)
Interactive Demo:
The website offers an interactive demo where users can upload a file or paste its content for analysis.
Upon analysis, the demo displays the matched rules and their results in a user-friendly format.
Navigation and Layout:
The website has a fixed navigation bar at the top, allowing easy access to different sections of the site.
The layout is responsive, ensuring the website adapts to different screen sizes and devices.
Consistent styling, typography, and color schemes are used throughout the website for a cohesive and professional look.


Allow login with username OR mail

I want to allow users to login using their username or their email.
I set the filter (|(uid=%s)(mail=%s)), but i have the error:

PHP Warning:  sprintf(): Too few arguments in /var/www/wordpress/wp-content/plugins/authldap/ldap.php on line 249

Use str_replace instead of sprintf fix the issue. But I'm not a php dev, there is probably a more elegant way to do it.

Thank you very much, and sorry for the noise. I should have read the readme before creating this PR


Html not updating for Objects (valueConverter)

Hi,
I am currently trying to filter an array with a value converter. This is going quite well, when passing just a string or a boolean to the value converter. When I try to pass an object, the value converter is excecuted on start. But if a value changes inside the object, nothing happens.
Here is my code:
module.js
export class ModuleName {
  items = [
    {name: 'Item 1', flag: true},
    {name: 'Item 2', flag: false}
  ];
  filterObject = {
    flag: false
  };
}

value-converter/filter.js
export class FilterValueConverter {
  toView(array, filterObject) {
    return array.filter((item) => {
      for (let key in filterObject) {
        if (item[key] !== filterObject[key]) {
          return false;
        }
      }
      return true;
    });
  }
}

module.html
<input type="checkbox" checked.bind="filterObject.flag" />

  ${item.name}

When clicking the checkbox, the value converter should filter again. But nothing happens.
I think it should work like this, but maybe I am doing something wrong?
P.S.: Hope you can understand my english ;) and this is the correct place to post this issue.

@Webviech you have to load the value converter before using it.  You can do that in two ways that I know of:

If you're going to use it all over the place, make it a global resource.   Using the skeleton navigation example, you can configure it in the animation-main.js:

  ...
  aurelia.use
    .standardConfiguration()
    .developmentLogging()
    .plugin('aurelia-animator-css');

  // load the global resource
  aurelia.globalizeResources('./value-converter/filter')
  ...

<require> it directly in your module.html ( using this method will only enable the converter for the current module )

<require from="./value-converter/filter"></require>

...

<input type="checkbox" checked.bind="filterObject.flag" />

  ${item.name}

When Aurelia encounters this binding expression: repeat.for="item of items | filter:filterObject" it sets up an array observer for the view model's items property.  If items are added/removed from the array (via push/pop/splice/etc).  Aurelia also sets up a property observer for the view model's items property which tracks scenarios where the items property is assigned a new array instance or null, etc.
None of these observers track property changes to the objects in the array, which is why you're not seeing the repeat binding re-evaluate when you change the flag property.
We'll be able to handle these kinds of scenarios with binding-behaviors in a future release of aurelia-binding.

-- removed my comment because it wasn't valid -- thanks @jdanyow ...

@jdanyow Thanks for your answer. Actually I don't want to change an object in the item array. I want to change a property in the filterObject. Aurelia observes the change for the property in the object, but not for the object itself.
I just found a workaround:
Mehod in the class which is excecuted on change (checkbox):
  applyFilter() {
    let filterObject = this.filterObject;
    this.filterObject = {};
    this.filterObject = filterObject;
  }

<input type="checkbox" checked.bind="filterObject.flag" change.trigger="applyFilter()" />

Not pretty, but it works :)
Or is there a better solution?

Ah- I see- yes, here's a sneaky workaround...
use the filterObject.flag as a converter parameter (your converter doesn't need to use it).  Aurelia will start observing this expression and update the repeat when it changes:
<input type="checkbox" checked.bind="filterObject.flag" />

  ${item.name}

I have the same problem with ValueConverter. I download JSON data from web and insert it to the object. ValueConverter run only once at page load.
View:

Object:
this.Data = {
         "URL": {"url" : "url" },
         "Raw": {
            "Generated": null,
            "Consumed": null,
            "Imprted": null,
            "Exported": null
         },
         "Parsed": [ ]
      };

ValueConverter:
export class toJSONValueConverter
{
   toView(object)
   {
      if(typeof object !== "object") return object;
      return JSON.stringify(object, null, 2);
   }
}

RawData getter:
get rawData()
   {
      return this.Data.Raw;
   }

We'll be able to handle these kinds of scenarios with binding-behaviors in a future release of aurelia-binding.

I can't figure out how to get it to work with binding behaviors, could you guys give me any pointers on this? I'm in a similar situation as described above, I have an object like this
count = { "min": <val>, "max": <val> }

and I want to update text every time min or max changes. It doesn't work with CountConstraintsValueConverter:

Count: ${metadata.constraints.count | countConstraints}

I'd rather avoid workarounds if it can be achieved with a proper method.

@gronostajo, per @jdanyow's suggestion, have you tried the following as a workaround?

Count: ${metadata.constraints.count | countConstraints:metadata.constraints.count.max:metadata.constraints.count.min}

While I can see this as a problem for a large number of properties, I think it would work for your case here...


DOC: Include super().init() for subclassing

Codecov Report

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

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
- Coverage   99.56%   98.04%   -1.53%
==========================================
  Files           3        2       -1
  Lines         465      461       -4
==========================================
- Hits          463      452      -11
- Misses          2        9       +7

Impacted Files
Coverage Δ

scikeras/_utils.py
93.47% <0.00%> (-5.44%)
:arrow_down:

scikeras/wrappers.py
99.18% <0.00%> (-0.55%)
:arrow_down:

scikeras/__init__.py

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 1c045dc...8a7a507. Read the comment docs.


Typo in lesson 1 sketchnote: "Thing" instead of "Think"

IoT-For-Beginners/sketchnotes/lesson-1.jpg
Typo in lesson 1 sketchnote: It says "Thing of it" instead of "Think of it" (top-middle box). Normally the intended meaning would be obvious, but given the context it should probably be changed :)
Great sketches, btw!!

@nitya

the typo still exist. can you please assign it to me? (alongwith telling me how would I edit the image. OR share the original file (on whatever software it was drawn/build))

Just seeing this now - I can send a PR!


Setup the protection rules

Require a code review for PRs.
Require approvals
Dismiss stale pull request approvals when new commits are pushed
Require approval of the most recent reviewable push
Require conversation resolution before merging
Do not allow bypassing the above settings

I have already added the protection rule to our repository. Don't worry about it anymore; you can refresh your browser, and you'll see that the notification is not showing anymore.

Dear @WuorBhang I have double checked the rules, yet I can't find any applied rules.

If you click on branch and rules under the branch, then rulesets, you will see that I have created two rules

We will talk about this in our next meeting.


Update models.py

This will fix KeyError on entry['comments']['data'] when using a sandboxed app

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at<EMAIL_ADDRESS>Thanks!

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

when will this be merged

Looking forward to have it merged.

The whole object_from_dictionary method relies too much on the dictionary structure. If one of those keys is not present, it raises exceptions.


Update CONTRIBUTING.md

Restore the steps for importing Gradle into Eclipse.
Make gradlew invocation agnostic
Signed-off-by: zosrothko<EMAIL_ADDRESS>Context
Importing Gradle into Eclipse requires a JDK 8 at least.
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commmits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@oehme I think you get this one as you'd intentionally removed this but I forget why.

I removed this, because we don't want to troubleshoot Eclipse import problems for this build. All committers use IntelliJ and we encourage all contributors to do so as well. I'm not terribly attached to this opinion though. I'll leave the decision to you @eriwen

@oehme @eriwen  You want to promote IntelliJ! No problem with... but let the contributor that have chosen Eclipse since years run as smooth as possible when importing Gradle into Eclipse. By the way, Gradle itself is promoting Eclipse Buildship...

There is a difference between what we support for Gradle builds in general and what we decided for our build. We don't promote IntelliJ over Eclipse in general, it's just the only tool that can handle our code base. Eclipse's Groovy support can't handle our Spock tests unfortunately.

@eriwen For you information, the JDK 8 is mandatory for importing Gradle into Eclipse even if it is not mandate by Gradle, and as @oehme stated it, "Eclipse's Groovy support can't handle our Spock tests". But even with this unsupported Groovy support, all Gradle Java classes were compiling fine which made Eclipse a pretty IDE for debugging Gradle itself or developer's private plugins as mine. Note that I used the preterit because since Gradle 4.3, one is facing to this EJC  issue....

@eriwen Changes done. I restore the invokation of Gradle to follow the Unix way..

@zosrothko I approve of the changes, and I see that your commit is in fact signed off so I'm merging. Thank you!


plugins/languages/treesitter: change default value for parserInstallDir

Currently, the nixGrammars = true method for installing parsers is broken.
Fortunately, nixvim also supports the legacy way of installing parsers.
The problem is that treesitter tries to put the parsers in its package directory which is not writable when using NixOS.
This can be overcome by setting a custom directory using the parserInstallDir option.
This PR proposes to change the default value for this option when nixGrammar is set to false to $XDG_DATA_HOME/nvim/treesitter.
Hence, when a NixOS user wants to use the legacy way of installing parsers, he only has to set nixGrammar to false.

Maybe this is not a good solution.
Here are the other possibilities:

We leave null as the default value, but we add a warning/entry in the documentation: "If you are using NixOS and if you set nixGrammars to false, then, you have to set the option parserInstallDir to some custom directory such as $XDG_DATA_HOME/nvim/treesitter."
We set the default of the parserInstallDir option to $XDG_DATA_HOME/nvim/treesitter independantly of whether or not nixGrammars is true or false.

Only concern I have is that XDG_DATA_HOME is not defined for macOS (and in some cases might not even be defined in Linux?) so some fallback might be nice

Yes you are right. I thought about this, but I do not think that there is an easy way of implementing a fallback :/

I'll say we leave it like this at the moment then, the option is there to change it anyway and it's just a best-effort default. Will just change the documentation to have that in mind


SAK-50711 Calendar json console error on startup

@hornersa i noticed 3 GET requests on startup. once i got rid of the three identical requests, the error went away

I'm going to let @hornersa test on Monday before merging

I've tested @ottenhoff 's patch this morning and it works-- it passes the test plan.


extend extension apis

this are currently limited to what you can do with the c api. see this feature request for more info https://github.com/duckdb/duckdb/discussions/11988

Unary function support was added in https://github.com/duckdb/duckdb/pull/11786


[DOCS] Add link to MIG metadata configuration spreadsheet to official docs

If applicable, link to the documentation page relevant to your issue below.
https://islandora.github.io/documentation/user-documentation/metadata/

If this is a request for new documentation, where in the existing documentation does this documentation belong?
N/A

Describe your documentation request.
The MIG group created a spreadsheet with the details of the metadata configuration of the “starter site” version of Islandora 2. We think that we should link to this spreadsheet from the official docs.

Which version of islandora are you using?
Islandora 2

What user roles is this documentation intended for?

Collections Manager, Developer, Systems Administrator

What implementations of islandora does this documentation request apply to?
"Starter site"

Any related open or closed issues to this bug report?
Screenshots:

Lina and I created this issue, and we are still not sure where exactly in the page https://islandora.github.io/documentation/user-documentation/metadata/ we should add a link to the spreadsheet. Perhaps, we should create a new metadata related page to link for to the spreadsheet.
I can envision adding a section called the same thing as the name of the file: "Islandora Starter Site Metadata Configuration"
Then we could add one or two sentences explaining what the spreadsheet is. For example...
### Islandora Starter Site Metadata Configuration

This spreadsheet contains the details of the various fields, and taxonomy vocabularies, and relators configured for the starter site.

This spreadsheet was created and maintained by the Islandora [Metadata Interest Group (MIG)](https://github.com/islandora-interest-groups/Islandora-Metadata-Interest-Group).

What do people think?

addressed by https://github.com/Islandora/documentation/pull/2295


⚠ Bofrost.de has degraded performance

In 0dda09f, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1148 ms

Resolved: Bofrost.de performance has improved in 680f19d after 11 minutes.


Validation model spec

Blog App: validations and model specs
In this branch, I implemented the following:

Use validations for models.
Write unit tests for models.

@fobadara I am going to do the changes but at same time I am giving a 1 start review. Because as you can see there, it states that the live view is coming soon and that feature was suggested to me by another reviewer.

@fobadara I am going to do the changes but at same time I am giving a 1 start review. Because as you can see there, it states that the live view is coming soon and that feature was suggested to me by another reviewer.

I understand but reviewers are not allowed to review a project if one of the criteria for invalidating a project is detected.  You can delete that section in your README for this submission and add it back once you have deployed the project

@rloterh thank you for making my read me more professional


the demo "starwars" doesn't work because websocket_client

../../../../flutter/.pub-cache/hosted/pub.dartlang.org/graphql-5.0.1/lib/src/links/websocket_link/websocket_client.dart:208:9: Error: The method 'unawaited' isn't defined for the class 'SocketClient'.

'SocketClient' is from 'package:graphql/src/links/websocket_link/websocket_client.dart' ('../../../../flutter/.pub-cache/hosted/pub.dartlang.org/graphql-5.0.1/lib/src/links/websocket_link/websocket_client.dart').
Try correcting the name to the name of an existing method, or defining a method named 'unawaited'.
unawaited(_closeSocketChannel());
^^^^^^^^^

reviewing this error, I think it would be good to add more examples using this packages I have a while to use it, I have some on my github, I have examples using artemis to generate the static classes in dart based on my graphql shema, and after that I have those classes I do the magic with this package 😎
https://github.com/manuelduarte077/pokemon_graphql

The problem is that you are using a public API and the public API died somehow, like in the case of the prev example!
The optimal idea is to provide an example with the API, and these API are used inside the test too.
And all the tests are possible to reproduce with a docker image not only in the CI, because the CI dies somehow (e.g travis)
Working on this idea https://github.com/zino-hofmann/graphql-flutter/issues/1162#issuecomment-1182946910
In addition, closing this issue because there is already a discussion https://github.com/zino-hofmann/graphql-flutter/discussions/1165
P.S: You can add your message here :)

The problem is that you are using a public API and the public API died somehow, like in the case of the prev example!
The optimal idea is to provide an example with the API, and these API are used inside the test too.
And all the tests are possible to reproduce with a docker image not only in the CI, because the CI dies somehow (e.g travis)
Working on this idea #1162 (comment)
In addition, closing this issue because there is already a discussion #1165
P.S: You can add your message here :)

Thank you very much for your reply


Remote leak fix

Makes remotes self-freeing.
I needed to add new functionality for https://libgit2.github.com/libgit2/#HEAD/group/remote/git_remote_get_refspec, because the returned refspec belongs to the remote (and gets freed with it), but there is no way to duplicate the refspec (it is an opaque struct).  AFAICT this requires an ownership mechanism, allowing the refspec to keep a handle on its owner remote so it doesn't get freed.
So I extended shouldDuplicate into ownedByThis: when ownedByThis is turned on we duplicate if the type has a dupFunction, otherwise we keep a handle on the owner.

Thanks @srajko!


CustomerSubscriptionWebhook: let sync_customer fetch the live customer

    if cu is None:
        cu = customer.stripe_customer

The sync method is doing this already.  I don't see this accomplishing anything.  I guess it cleans up the API a bit.

Yes, it is just a minor cleanup and centralization of responsibilities.


UI Hints for Model Behaviours

Spin-off from https://github.com/tarasglek/chatcraft.org/pull/664#issuecomment-2242881213
Not all models support the same features.  For example, some can handle images or audio, some can do function calling, etc.  However, you have to know about this to use it.
We should come up with a way to hint to users what a model can do.  Maybe in the list of models we add icons?  Maybe once you've selected a model, we update the prompt area with some icons to indicate what can or can't be done?
Needs thought.

@humphd I think at some point we started showing only chat models in ask menu list. Is this still relevant?
https://github.com/tarasglek/chatcraft.org/blob/26e39eab4bc40db0e40051f7eb7edcd85735a952/src/components/PromptForm/PromptSendButton.tsx#L324-L328

Maybe not according to how we define a chat model now.  But I want to make sure we filter out the models that won't work for chat completions (OpenAI has some pre-chat-completion stuff that's useless).
Maybe we should make a list that blacklists models, or maybe I'm overthinking and we should show them all and just error when stuff fails.

I made this explicit cos before we only did if openai & .includes("gpt"). I needed this to not show whisper on groq


Changed a way to check that user with given e-mail is in iTunes Testers.

I not an ruby master, but without this change, version from HEAD don't create new users. As far I understand, we are checking that user with given email is present in iTunes Connect Testers, but instead use params[:email] or simply email we are using config[:email] which is empty and this return us account owner account?

Yes, there's definitely something wrong in HEAD. It will not create new users properly. Applying your pull request fixed it for me. Thanks.

Looks good, thanks :+1:


New Network school buses aren't showing up

I decided to have a little play around with the api (which should show the same thing that the waka app does, correct me if I'm wrong here), and I had a search for a school bus (the 510 bus specifically using https://getwaka.com/docs/index.html#api-Lines-GetLine). I just got []. I tried a few others and got the same result. When I then had a look at https://getwaka.com/docs/index.html#api-Lines-GetLines, I found the 515 bus, which should be a school bus after the New Network changes, but it still has the old route: "515": [             [                 "Britomart",                 "Otahuhu Station",                 "Ruawai Rd and Panama Rd"             ]         ],
What's wrong here?

We don't support school buses, mostly because they used to be super weird with horrible numbers - we explicitly filter them all out. Maybe we can add them in, if that would be useful to anyone?

that's odd, I remember getting this over the at one Because it supported school buses XD

that's odd, I remember getting this over the at one Because it supported school buses XD


JSON Output for Documentation Data

Fixes #2772

CI wants you to run crystal tool format ;)

I think that this is getting really close to merging. I'll keep tweaking with it a little, but please tell me if you need/want me to change anything.

This can be closed. #4746 has been merged with JSON output.


Webpack error

What kind of change does this PR introduce?
Refactor which introduces new feature.
Did you add tests for your changes?
Yes. Three additional tests have been added.
If relevant, link to documentation update:
Currently no PR for documentation exists. Upon review, node api documentation regarding error handling could be altered to show simplified logging approach:
if (err) {
  return console.error(err);
}

Summary
Currently the process to log errors is unnecessarily tedious:
  if (err) {
    console.error(err.stack || err);
    if (err.details) {
      console.error(err.details);
    }
    return;
  }

Behind the scenes console.error utilizes util.inspect to determine how to transform the passed arguments into a string. If the passed argument contains an inspect method, util.inspect will respect it. By having all Error classes inherit from a base WebpackError class we can add this method in a central location (with ability to override) and reduce error logging complexity:
  if (err) {
    return console.log(err);
  }

Does this PR introduce a breaking change?
No
Other information
N/A

Rebased and updated to include EntryModuleNotFoundError added in #4544

@sokra - Appears to be some sort of hangup with codecov. Not sure what service is supposed to "report" the status to it.

Thanks


Server generate converters for input/output/error structs to Client structs

As started to discuss here - https://github.com/awslabs/smithy-rs/issues/903#issuecomment-995223758
This is desired to implement a server that forwards requests as a client to another server.
In my use case this is needed for S3 API, see https://github.com/s3d-rs/s3d.
If you can direct me where in the code to add it, I can start a PR.

I am doing something similar and I found a very ugly workaround for the time being, until we do not have a real conversion system.
Client and server structures are equal right now, they are just coming from a different namespace.
let server_type = unsafe { mem::transmute<my_client::Type, my_server::Type>(data) };

That said, I agree with you, we should have a conversion system.

Thanks @crisidev! that's a useful hack to have for now!!

 :)

Hey @crisidev
A quick update, I generated converters in my own build.rs for now, you can see it here:
https://github.com/s3d-rs/s3d/blob/201aa13fca8ba1ff95a6d356cbd24d67968c4536/build.rs#L165-L166
https://github.com/s3d-rs/s3d/blob/201aa13fca8ba1ff95a6d356cbd24d67968c4536/build.rs#L211
https://github.com/s3d-rs/s3d/blob/201aa13fca8ba1ff95a6d356cbd24d67968c4536/build.rs#L318-L322
However while this works fine for now, it requires quite a few conditions in order to handle the differences in the output of the smithy codegen - for example the location of the generated shape struct (input/output/model), and the type of fields (option, string, vec, etc) which might be a pain to maintain in a separate codebase from smithy-rs.
I think we it should be possible to generate these converters inside smithy-rs server-codegen by generating it as macros, so that the types of the client structures are not required to be known on compile time of the server crate, as suggested in the discussion here - https://github.com/awslabs/smithy-rs/issues/1159#issuecomment-1034262563.

I wanted also to show you the call to these converters (from a macro in a server router) -
https://github.com/s3d-rs/s3d/blob/201aa13fca8ba1ff95a6d356cbd24d67968c4536/src/s3_server.rs#L40-L63
What do you think on the server code in general?
I wasn't sure if you had a similar/different view of how this would work in practice.
Tagging @crisidev and @david-perez, and anyone who's interested in rust servers to give feedback (missing comments for sure...) thanks!


Changed CliCommand to implement Supplier instead of Callable<…

…String>, so that it doesn't throw Exceptions in its signature. Cleaned up CmdParser.doLine().

Doing this leaves the main thread open to potentially being crashed by an exception in the terminal CLI. Needs fixing.

Main thread should no longer crash if CLI crashes, and GUI input should not get stuck inactive if the same occurs.


URL rule for /groups/{GroupId}/tags/{TagValue} should specify allowed methods

https://github.com/scitran/core/blob/master/api/api.py#L155
I believe the allowed methods should be ["GET", "PUT", "DELETE"], notice POST is not included

I think the same is true for the /{ContainerName}/{ContainerId}/tags/{TagValue} route:
https://github.com/scitran/core/blob/master/api/api.py#L158


Data Location

Reason
We need to inform the potential customers so they can understand how to integrate with existing datasets.
Description
A Text description of the data storage location
Success Criteria

[ ] Text description complete and approved

Additional Information

AWS

Amazon Elastic Compute Cloud
AWS Lambda
Amazon Simple Storage Service
Elastic Block Store
Amazon Virtual Private Cloud
Amazon Route 53

AZURE

Azure Kubernetes
Azure SQL
Azure Machine Learning
Azure Backup
Azure Cosmos DP
Azure Active Directory

Google

Google Compute Engine
Google Kubernetes Engine
Google Cloud Spanner
Google Cloud Virtual Network

IBM

IBM Cloud Code Engine
IBM Hyper Protect Virtual Servers
IBM Cloud Functions
IBM WebSphere Application Servers
IBM Power Systems Virtual Servers

Oracle

Oracle Cloud Infrastructure
Oracle Big Data Cloud
Oracle Database Cloud Service
Oracle Autonomous Database

Databricks
Snowflake
DigitalOcean
ZStack
Tencent
OVH Cloud
Lumen
SkyTap

The dateset is hosted on AWS on an S3..........
All files (data and metadata)for the lake can be found in the S3 bucket
s3://mytiki-cleanroom-sample with AWS Glueas the metadata catalog. - Expand


How to obtain certificates

I have moved this section after the schedule to not break the flow of the page.

Shouldn't we remove the questions? I am not sure what the conclusion was but my recollection was that we wanted more lightweight and only figure out whether active participation was there. Another problem with the questions is that they are only about week 1.


Disable Save and Save as when last tab is closed.

Also allow Close to close the app when there are no tabs.
Parts of issue #117.

This does not work for me. I am able to click save after the last tab is closed and it just throws an assertion error.

Thanks for catching that. It should work now when clicking on the close button with the mouse.

nice, that works!


user_doc: download backup

also changed backup name from “xxx-backup-tool” to “xxx-backup-sidecar”.
This fixes #669. Partly solves #667

Looks awesome. Let's clean this up and get it merged.

lgtm

Fixed.
Merging this to continue for #673


add required NeoVim version in the README

Closes #7

I'll resolve this PR now as there is no further feedback. I'm optimistic that it solves the linked issue.
If not, @DanielBarton446 please re-open the issue.


feat: this PR adds back the environment var support

Hi,
This PR adds back the support for the environment variables for:
Required

client_id (String) Client ID for Port-labs (environment: PORT_CLIENT_ID)
secret (String, Sensitive) Client Secret for Port-labs (environment: PORT_CLIENT_SECRET)

Optional

base_url (String) Base URL for Port-labs (environment: PORT_BASE_URL)

Happy to get your feedback.

Continue here: https://github.com/port-labs/terraform-provider-port-labs/pull/60/


Feature/85/use fetch auth header wrapper

Description
Please see the description add to the README.md on this branch.
Fixes #50
Fixes #85
How to test?
Check the Usage section added in the PR
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] This PR is ready to be merged and not breaking any other functionality

tests are failing jest cannot handle async/wait (generators) probably you need to import babel-polyfill in the test setup files


Document Update

[Enter feedback here]
It would be helpful if this article included a note about setting up Mutating Admissions Webhook on the AKS cluster from (https://azure.github.io/azure-workload-identity/docs/installation/mutating-admission-webhook.html).  If someone follows this article without setting up Mutating Admissions Webhook the the quick-start pod will fail to start.

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

ID: efde8151-ca72-0ce1-271a-57a14a8c7ccd
Version Independent ID: f37efaf0-6573-6813-a3e4-3065e44666e4
Content: Tutorial - Use a workload identity with an application on Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/learn/tutorial-kubernetes-workload-identity.md
Service: azure-kubernetes-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@cliffjohns-progenity
Thanks for your feedback! We will investigate and update as appropriate.

@cliffjohns-progenity
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@cliffjohns-progenity - Thanks for your feedback. I'll bring this up with the feature PM and investigate further given there hasn't been an issue completing this tutorial as defined.

@cliffjohns-progenity - I reviewed the article and under step 2 in the section - Create Kubernetes service account, the webhook is established and in the pod spec when you use - azure.workload.identity/use:true, label, pods are mutated by the azure-workload-identity mutating admission webhook to inject the Azure specific environment variables and the projected service account token volume. If the tutorial steps are followed, the quick-pod does start and function as expected.
I'm going to proceed with closing this issue as a result of determining there aren't any actions that need to be taken with the article. Thanks.
#please-close


Gens better doc handling for what is used for generation

implements better mechanisms for deciding what goes into generation
adds a token count to datasource blocks

(happy to deploy core once this is merged :+1: )


Better regex

Hi,
https://github.com/tivie/php-git-log-parser/blob/master/src/Parser.php#L175
^\[(\S*?)\](.*)
Without ?, key become subject][xxx

Hey
Can you please provide an example that isn't working?

Commit subject is [Fix #1, #2] My message.
With your actual regex; subject key in array become subject][Fix ...

Ran into the same issue @Biromain 's regex fixed the issue.

Should be fixed now


PackageModel: add struct BuildFlags, reduce reliance on TSC

Motivation:
We're relying on BuildFlags from TSC for passing flags in some places, but there are also cases where flags are passed as separate properties, such as extraCFlags, extraCXXFlags etc.  BuildFlags is quite useful, as it allows grouping of these properties instead of passing them around separately. In most if not all cases they need to be passed all at once anyway. This apparently already let to issues, where Destination didn't have its own linker flags property, also see https://github.com/apple/swift-package-manager/pull/5793. We also had issues with inconsistent naming of these properties. Unifying everything under BuildFlags could prevent such issues from happening.
Modifications:
Adding BuildFlags to PackageModel, mostly a copy of the declaration in TSC, but with a cleaned up initializer arguments naming. Modified uses of build tools flags to use this new struct instead of passing arounds properties separately.
Result:
Reduced reliance on TSC, unified use of BuildFlags across the codebase.

@swift-ci please smoke test

@swift-ci please smoke test

https://github.com/apple/sourcekit-lsp/pull/659
@swift-ci please smoke test


fix: always enable soft-reset on 700+ series controllers

Experience has shown that the problems caused by soft reset are limited to 500 series and older controllers (shutting down, reconnecting with a different port, etc...)
On 700+ series, soft-resetting is however required in lots of cases. Here, having soft-reset disabled by the users rather causes problems.
This PR changes how we handle the enableSoftReset driver option. On 700+ series, this is now ignored and soft-reset is always performed, regardless of user preference. On 500 series and older, we still respect the setting, unless a stick is known not to support soft reset.

/cc @robertsLando
When this lands, please update the config option description to mention that this only affects 500 series and older.

@zwave-js-bot automerge

@AlCalzone done


update CHANGELOG.md, close #11167

What is changed and how it works?
Update CHANGELOG.md to 20220105;
Close #11167
What's Changed:

PR to update tikv/CHANGELOG.md:
PR update content since tikv-server v5.0.1(20210423);
PR update content refers to "https://github.com/tikv/tikv/releases"

Check List
Tests - Preview works good.
No code
Release note
None

/merge


update to 2.1.1

I just released 2.1.1, and I need a bit of a break from CI services 😉 , so I will contribute by posting this issue.

:smile: I think the conda-forge automation did almost all of the hard work! #6

Sorry, I'm a bit behind with the patches. Now we should focus on 2.1.2 directly.

I finally merged the changes, so closing this. If we find problems, let's open a new issue.


Using on a mobile device

Hello
How are you?
Thanks for contributing to this project.
I want to customize PIDNet  network in order to use it on a mobile device.
I think that it is difficult to run even PIDNet-S model on a mobile device.
Is there any solution to reduce PIDNet network size?
Thanks

Thanks. For mobile application, I would recommend to replace the traditional convolution by depthwise separable convolution or you can try PP-Liteseg, which is the state-of-art ultra-light model.

Thanks for your reply.

thank you @XuJiacong ,
Any idea about mobile speed, if I use for example Iphone 12?


isTraining block training

A model should be called to train from multiple threads.

This should be done; waiting for external confirmation.


Python example of means in accuracy utility

Once

#1922

is merged, we can make sure it's demonstrated somewhere in the python examples.

Sorry. This is already covered in the PR stack.


Spark connector version for Spark 3.2.x

Hi, we are currently on spark 3.2.x and using Spark connector 1.2.0.  On project  version matrix , I see 1.2.0 is recommended for Spark 3.1.x and 1.3.0 is recommended for Spark 3.3.x.  Please share your thoughts on which version should be used for 3.2.x
thanks

Please use 1.2.0.


Much higher mapping time during test time.

Hey Authors,
Firstly, thanks for the ease of use of your code. Running it was almost seamless on multiple machines / datasets!
Although, after timing the code for the Apartment datataset, the mapping step seems to be considerably slower on my machine (100ms vs 500ms).
The CPU is an i9-10900X CPU @ 3.70GHz and GPU RTX 3090 .  (All 10 cores / 20 threads seem to be used during training/inference)
The paper claims to be using a 3.80GHz Intel i7-10700K CPU which only has 8 cores of an older generation.
I doubt the small difference in the clock frequecy can cause a difference of 5 fold. What do you guys think it could be?

Hi,
Thanks for your interest in our work!
The table 4 in our paper is using the same iteration and pixel samples as the table in iMAP paper. (since iMAP did not open source, we can only compare in that way) The goal is to show the benefit of our scene representation, not to report the runtime on each dataset. For different datasets, we use different iterations and pixel samples shown in implementation details. In our re-implementation of iMAP (iMAP*), we give much more iterations and pixel samples, to make a more than fair comparision.


Review all authorizations and migrate them to use action_policy

possibly update to handle permissions with pundit?

We have decided to go with action_policy instead of pundit for authorizations

Closed in #4865.

Closed in #4865.


Hi @gsv022 ,please kindly follow up the #734 after you asked question, thanks

Hi @gsv022 ,
We are not trying to fix the length, but we want to know the current length limit.
We are not facing any issue, but we would like to do some planning.
Based on your reply, can I assume the parameter-"AuthorizedCreatorUpns" can contain unlimited character? even 10,000 users or more?
Originally posted by @rex-ma in https://github.com/OfficeDev/microsoft-teams-apps-company-communicator/issues/734#issuecomment-1135715517

@rex-ma  Closing the duplicate issue and we will provide our response to #734


If upper, lower and compared versions are same it returns false

https://github.com/canonical/charmed-openstack-upgrader/blob/a4101a6972c1191628d95d1b24e2246a632054b5/cou/utils/openstack.py#L244
For example:
arr = [VersionRange("6.0.0", "6.0.0")]
if "6.0.0" in arr:
    print("I am in range")
else:
    print("I am not in range")

I'm not entirely sure if this is a bug or if it's a misusage of the VersionRange class. The upper it's not meant to be inclusive.
I think we should raise an error if someone tries to instantiate a VersionRange with the same value for upper and lower.
@aieri what is your point of view on this?

the idea was to model range(), and the described behavior is consistent:
In [4]: r = range(5,5)

In [5]: 5 in r
Out[5]: False

I'm open to raising an error if upper == lower if you prefer.


Explore API Gateway options

With the recent changes in architecture caused by Auth service and SpiceDB, there is a  need for API aggregation. A simple example is Domain removal in Auth svc, which should result in all the Things and Groups (and Users) being removed (from the domain). This causes messy and circular dependencies between Auth and other services. Extracting API aggregation and authorization to API GW would solve this problem. It would also simplify domain services. One more benefit is that it would remove the need for private API because internally, we can have all the data we need (say, when we need an email for users), and API GW will take care that those data are not leaking to the end-user.

This one might also be addressed by API GW: https://github.com/absmach/magistrala-old/pull/269.

Relevant document: https://docs.google.com/document/d/1tJ78iHx-_-W-GO5YFa8T1yd65Hu-AsxLmSDpwg5mBYw/edit#heading=h.nc9o9iq9wvl6.


Remote SSH stopped working in VS Code Insiders

Issue Type: Bug
Hi,
the Remote SSH connection stopped working yesterday. I haven't done any update or installed anything from the market place. I try to connect and it says "Could not establish connection to "xx.xx.xx.xxx": Connecting with SSH timed out".
I tried downgrading the Remote  - SSH extension version from 0.65.7 to 0.65.6 and it did not work. Also, I can confirm I am able to connect to the remote server from my computer, for I am using a regular windows cmd terminal. Also, I tried connecting with regular VS Code with the same 0.65.7 Remote - SSH version, and it worked.
Thanks.
VS Code version: Code - Insiders 1.61.0-insider (aa93eefe550167de3ca29aaa3d06f721a47594a3, 2021-09-09T05:15:19.597Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No
Fetching remote diagnostics for 'SSH' failed: Connecting with SSH timed out

Hi,
Thank you for reaching out to the VS Marketplace Team. Please note that remote connections have certain time limits within network connections. If you encounter any further issues, do not hesitate to contact us.
Regards,
VS Marketplace Team


Use zmq

To be deployed in tandem with https://github.com/anchor/ceilometer-publisher-zeromq/pull/18
Program now acts as a server, accepting requests over ZeroMQ. The requests contain the same JSON data as prior, and processing has not changed.

She look all the goods

Much cleaner. Nice work.


nix: Various improvements.

To fix flake setups in settings where the nixpkgs revision has not
been fetched yet.

There is another issue I've run into which I think could be a bug (?). nix build in this project gets:
cargo-bpf> building
cargo-bpf> cargo build $cargo_release -j "$NIX_BUILD_CORES" --message-format=$cargo_message_format
cargo-bpf>    Compiling getrandom v0.2.6
cargo-bpf>    Compiling cargo-bpf v2.3.0 (/build/d9gsyrv5np02k6103xap0hvf29m0ph39-source/cargo-bpf)
cargo-bpf>    Compiling redbpf-macros v2.3.0 (/build/d9gsyrv5np02k6103xap0hvf29m0ph39-source/redbpf-macros)
cargo-bpf>    Compiling bpf-sys v2.3.0 (/build/d9gsyrv5np02k6103xap0hvf29m0ph39-source/bpf-sys)
cargo-bpf>    Compiling uuid v0.8.2
cargo-bpf> error: failed to run custom build command for `bpf-sys v2.3.0 (/build/d9gsyrv5np02k6103xap0hvf29m0ph39-source/bpf-sys)`
cargo-bpf> Caused by:
cargo-bpf>   process didn't exit successfully: `/build/d9gsyrv5np02k6103xap0hvf29m0ph39-source/target/release/build/bpf-sys-9bbf30d6f29dde00/build-script-build` (exit status: 101)
cargo-bpf>   --- stderr
cargo-bpf>   bindings.h:1:10: fatal error: 'btf.h' file not found
cargo-bpf>   bindings.h:1:10: fatal error: 'btf.h' file not found, err: true
cargo-bpf>   thread 'main' panicked at 'Unable to generate bindings: ()', bpf-sys/build.rs:23:10
cargo-bpf>   note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
cargo-bpf> warning: build failed, waiting for other jobs to finish...
cargo-bpf> error: build failed
cargo-bpf> [naersk] cargo returned with exit code 101, exiting
error: builder for '/nix/store/n6rpv034w12q2zaqpbrb436fldfz1azr-cargo-bpf-3.0.0.drv' failed with exit code 101;
       last 10 log lines:
       > Caused by:
       >   process didn't exit successfully: `/build/d9gsyrv5np02k6103xap0hvf29m0ph39-source/target/release/build/bpf-sys-9bbf30d6f29dde00/build-script-build` (exit status: 101)
       >   --- stderr
       >   bindings.h:1:10: fatal error: 'btf.h' file not found
       >   bindings.h:1:10: fatal error: 'btf.h' file not found, err: true
       >   thread 'main' panicked at 'Unable to generate bindings: ()', bpf-sys/build.rs:23:10
       >   note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
       > warning: build failed, waiting for other jobs to finish...
       > error: build failed
       > [naersk] cargo returned with exit code 101, exiting

$ nix --version
nix (Nix) 2.5.1

Also:
$ nix build github:foniod/redbpf#defaultPackage.x86_64-linux
error: builder for '/nix/store/7q9jl4j6rc8a2l91pl9rqrqxbd6zx5mq-cargo-bpf-3.0.0.drv' failed with exit code 101;
       last 10 log lines:
       > Caused by:
       >   process didn't exit successfully: `/build/zphbmp0mmg457hnwij8iff3d93w26z4z-source/target/release/build/bpf-sys-c6e91f54233b0c77/build-script-build` (exit status: 101)
       >   --- stderr
       >   bindings.h:1:10: fatal error: 'btf.h' file not found
       >   bindings.h:1:10: fatal error: 'btf.h' file not found, err: true
       >   thread 'main' panicked at 'Unable to generate bindings: ()', bpf-sys/build.rs:23:10
       >   note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
       > warning: build failed, waiting for other jobs to finish...
       > error: build failed
       > [naersk] cargo returned with exit code 101, exiting
       For full logs, run 'nix log /nix/store/7q9jl4j6rc8a2l91pl9rqrqxbd6zx5mq-cargo-bpf-3.0.0.drv'.

I think the btf.h failure is https://github.com/nix-community/naersk/issues/230

@jsoo1 Yes, btf.h is due to submodule pain. I have not really found a way to resolve that one, therefore CI handles recursive checkout independently.

@jsoo1 In order to be able to merge this, I'm going to need you to adhere to our DCO policy. Please force-push signed-off commits to this branch, otherwise all this is ready to go. Thanks so much!

Ok, done. Thank you!

@jsoo1 your latest push seems to have removed the flake.nix changes, causing the nix builds to fail completely.

Oops, sorry!

(The sign-offs are for the same person, just work vs home person)


Add more comparisons (not, gt, lt).

@asolz this one had Python 3 issues too. Here's how you can test without installing a separate python 3 environment:
./test.sh -e py34


change(console): print sorted help (IDFGH-11382)

console commands may be registered in random order in application, for example each module registers its own commands.
the output of help is displayed to human, best to have consistent sorted output so that the implementation ordering will not affect the output and allow the user to see a list in some logic order.

sha=97251d324405e9296ccc753b93b3485e27e46656

Thanks, I can add unit test if required. I will work something out.

@0xjakob, I added a manual unit test that is per the way existing test works.

sha=07a9137fd769f0da2eda5936cc3e3db3232e8166

personally, I initialize my console commands to get the precise order I want.

@chipweinberger I'm sorry to hear that this change causes inconvenience for you. However, please keep in mind that before this change, the order of commands was implementation-specific, i.e., it was not defined by the API (see the documentation about command registration and the help command). If we would not be allowed to change implementation defined behavior, this would put us into a really difficult position, because then any change, including bugfixes, might be a breaking change. After all, now the order is defined. Even though it might not be what you are used to and even though you might need to change some tools, now you can rely on the order without worrying that another change or minor bugfix can easily change it again.

of course i understand your sentiment
but can you also have the old behavior as an option?
i.e. order depending on initialization order

@chipweinberger @alonbl After consultation with colleagues, we concluded that there is a point with registering commands in a certain order, e.g., when there are more important commands which should appear first. We would like to add a Kconfig option for switching between sorting in registration order and sorting in alphabetical order then. Would that be OK for both of you?

yes thats great =)

Hi,
If custom order strategy is important, I would have added a callback
registration to perform the ordering and order hint when registering.
This will make sure that the API supports any strategy, we can provide out
of the box text sort and hint sort callbacks.
The hint is important as most (I hope) developers do not implement
monolithic code as implied above, this enables each module to register its
own commands, and achieve a specific order (high priority and low
priority). If you accept this I will workout a patch.
I also think that adding a selector should also be done using an API and
not Kconfig, as there is already initialization and configuration using API
for this module.
Regards,
Message ID: @.***>

@alonbl This would work. However, the old API should still look and behave the same as before so users don't have a breaking change.
BTW., Kconfig options are not only used for pure configuration, but often also for reduction of code size. But so far, the code overhead seems negligible for the console component.


[SPARK-7174][Core] Move calling TaskScheduler.executorHeartbeatReceived to another thread

HeartbeatReceiver will call TaskScheduler.executorHeartbeatReceived, which is a blocking operation because TaskScheduler.executorHeartbeatReceived will call
    blockManagerMaster.driverEndpoint.askWithReply[Boolean](
      BlockManagerHeartbeat(blockManagerId), 600 seconds)

finally. Even if it asks from a local Actor, it may block the current Akka thread. E.g., the reply may be dispatched to the same thread of the ask operation. So the reply cannot be processed. An extreme case is setting the thread number of Akka dispatch thread pool to 1.
jstack log:
"sparkDriver-akka.actor.default-dispatcher-14" daemon prio=10 tid=0x00007f2a8c02d000 nid=0x725 waiting on condition [0x00007f2b1d6d0000]
   java.lang.Thread.State: TIMED_WAITING (parking)
	at sun.misc.Unsafe.park(Native Method)
	- parking to wait for  <0x00000006197a0868> (a scala.concurrent.impl.Promise$CompletionLatch)
	at java.util.concurrent.locks.LockSupport.parkNanos(LockSupport.java:226)
	at java.util.concurrent.locks.AbstractQueuedSynchronizer.doAcquireSharedNanos(AbstractQueuedSynchronizer.java:1033)
	at java.util.concurrent.locks.AbstractQueuedSynchronizer.tryAcquireSharedNanos(AbstractQueuedSynchronizer.java:1326)
	at scala.concurrent.impl.Promise$DefaultPromise.tryAwait(Promise.scala:208)
	at scala.concurrent.impl.Promise$DefaultPromise.ready(Promise.scala:218)
	at scala.concurrent.impl.Promise$DefaultPromise.result(Promise.scala:223)
	at scala.concurrent.Await$$anonfun$result$1.apply(package.scala:107)
	at akka.dispatch.MonitorableThreadFactory$AkkaForkJoinWorkerThread$$anon$3.block(ThreadPoolBuilder.scala:169)
	at scala.concurrent.forkjoin.ForkJoinPool.managedBlock(ForkJoinPool.java:3640)
	at akka.dispatch.MonitorableThreadFactory$AkkaForkJoinWorkerThread.blockOn(ThreadPoolBuilder.scala:167)
	at scala.concurrent.Await$.result(package.scala:107)
	at org.apache.spark.rpc.RpcEndpointRef.askWithReply(RpcEnv.scala:355)
	at org.apache.spark.scheduler.DAGScheduler.executorHeartbeatReceived(DAGScheduler.scala:169)
	at org.apache.spark.scheduler.TaskSchedulerImpl.executorHeartbeatReceived(TaskSchedulerImpl.scala:367)
	at org.apache.spark.HeartbeatReceiver$$anonfun$receiveAndReply$1.applyOrElse(HeartbeatReceiver.scala:103)
	at org.apache.spark.rpc.akka.AkkaRpcEnv.org$apache$spark$rpc$akka$AkkaRpcEnv$$processMessage(AkkaRpcEnv.scala:182)
	at org.apache.spark.rpc.akka.AkkaRpcEnv$$anonfun$actorRef$lzycompute$1$1$$anon$1$$anonfun$receiveWithLogging$1$$anonfun$applyOrElse$4.apply$mcV$sp(AkkaRpcEnv.scala:128)
	at org.apache.spark.rpc.akka.AkkaRpcEnv.org$apache$spark$rpc$akka$AkkaRpcEnv$$safelyCall(AkkaRpcEnv.scala:203)
	at org.apache.spark.rpc.akka.AkkaRpcEnv$$anonfun$actorRef$lzycompute$1$1$$anon$1$$anonfun$receiveWithLogging$1.applyOrElse(AkkaRpcEnv.scala:127)
	at scala.runtime.AbstractPartialFunction$mcVL$sp.apply$mcVL$sp(AbstractPartialFunction.scala:33)
	at scala.runtime.AbstractPartialFunction$mcVL$sp.apply(AbstractPartialFunction.scala:33)
	at scala.runtime.AbstractPartialFunction$mcVL$sp.apply(AbstractPartialFunction.scala:25)
	at org.apache.spark.util.ActorLogReceive$$anon$1.apply(ActorLogReceive.scala:59)
	at org.apache.spark.util.ActorLogReceive$$anon$1.apply(ActorLogReceive.scala:42)
	at scala.PartialFunction$class.applyOrElse(PartialFunction.scala:118)
	at org.apache.spark.util.ActorLogReceive$$anon$1.applyOrElse(ActorLogReceive.scala:42)
	at akka.actor.Actor$class.aroundReceive(Actor.scala:465)
	at org.apache.spark.rpc.akka.AkkaRpcEnv$$anonfun$actorRef$lzycompute$1$1$$anon$1.aroundReceive(AkkaRpcEnv.scala:94)
	at akka.actor.ActorCell.receiveMessage(ActorCell.scala:516)
	at akka.actor.ActorCell.invoke(ActorCell.scala:487)
	at akka.dispatch.Mailbox.processMailbox(Mailbox.scala:238)
	at akka.dispatch.Mailbox.run(Mailbox.scala:220)
	at akka.dispatch.ForkJoinExecutorConfigurator$AkkaForkJoinTask.exec(AbstractDispatcher.scala:393)
	at scala.concurrent.forkjoin.ForkJoinTask.doExec(ForkJoinTask.java:260)
	at scala.concurrent.forkjoin.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1339)
	at scala.concurrent.forkjoin.ForkJoinPool.runWorker(ForkJoinPool.java:1979)
	at scala.concurrent.forkjoin.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:107)

This PR moved this blocking operation to a separated thread.

cc @rxin @yhuai

Test build #30968 has started for   PR 5723 at commit 5b3b545.

Test build #31067 has started for   PR 5723 at commit 98bfe48.

Test build #31067 has finished for   PR 5723 at commit 98bfe48.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

LGTM.
We might want to instrument EventLoop to log warnings or errors if the rate of messages going in is much larger than the rate of processing.

I've merged this. Thanks.


628 triangle counting algorithm

Closes #189

This PR should be merged after #187

This PR depends on #188, #211 and #214

(note: rebased on latest develop)

Needs:

[ ] foldl matrix to value
[ ] select (triu/tril)
[ ] resolve the previous review comments

Maybe needed (unsure):

[ ] fold matrix to matrix

Preferred:

[ ] masked mxm


Log a warning for interfaces and namespaces missing [Exposed]

In https://github.com/heycam/webidl/pull/423 we've started making [Exposed] mandatory (to favor explicit over implicit and avoid bugs where interfaces aren't exposed).
It would help move this forward if a warning was printed to the console when a namespace, non-callback interface, or callback interface with constants is missing its [Exposed] extended attribute.
As a second step, this warning could be turned into a fatal error.

Also see https://github.com/plinss/widlparser/issues/27

This is done now, but you have to turn it on explicitly via Complain About: missing-exposed on. I need to do some refactoring of my metadata manager to enable it to be turned on by default, I think.


[Windows] - failed to add windows node in airgap setup

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):
In the airgap environment, try to add a Windows worker node to the custom cluster. I copy and paste the command from Rancher UI to the powerShell in the windows node, and here is the output:
PS C:\> docker images
REPOSITORY          TAG                 IMAGE ID            CREATED             SIZE
PS C:\>
PS C:\>
PS C:\> PowerShell -NoLogo -NonInteractive -Command "& {docker run -v c:/:c:/host ec2-13-56-165-173.us-west-1.compute.amazonaws.com:443/rancher/rancher-agent:v2.3.0-rc5 bootstrap --server https://jiaqi-ag-lb-e4f7e0724eaa18d0.elb.us-west-1.amazonaws.com --token gcjwzjcmt8bzn27tmxxk658dlx9hfwff4625clw85wrxl6jkkw765q --ca-checksum 78b36df18d1a6c24d5a85c8f616b36178ed6f5bbe376886b8a648d3ab8acbe6f --worker | iex}"
Unable to find image 'ec2-13-56-165-173.us-west-1.compute.amazonaws.com:443/rancher/rancher-agent:v2.3.0-rc5' locally
v2.3.0-rc5: Pulling from rancher/rancher-agent
3f81696d6765: Already exists
446dc6511ddc: Already exists
ffa747c13622: Already exists
b62f66ce2ff6: Already exists
6829b790015b: Already exists
1a5508aa7d2f: Already exists
96bfb26b3151: Already exists
bff9d71854d0: Already exists
352f0bc2c9ca: Already exists
c71c11ca485e: Already exists
df69c1d7f186: Already exists
67cf58d99f12: Already exists
9d65a1522e84: Already exists
da1fff6b47b4: Already exists
b7667626c68e: Already exists
89983e886988: Already exists
237778543dd4: Already exists
fffd633f9048: Already exists
ed4a098899e3: Already exists
5f910aa6cc76: Already exists
9fdd73c027b2: Already exists
8dba85e44c23: Already exists
00c27c1e790a: Already exists
641171d05d6c: Already exists
Digest: sha256:2023a4c9858258f5fa565f98b5d569d50c12c74a4b1e80ec6b0eb381a8679e11
Status: Downloaded newer image for ec2-13-56-165-173.us-west-1.compute.amazonaws.com:443/rancher/rancher-agent:v2.3.0-rc5
INFO: Detecting running permission ...
INFO: Detecting host CPU reservation ...
INFO: Detecting host RAM reservation ...
INFO: Detecting host DISK reservation ...
INFO: Detecting host system locale ...
INFO: Detecting host Docker name pipe existing ...
INFO: Detecting host Docker release version ...
INFO: Detecting host network interface ...
INFO: Configuring host Docker startup mode to automatical ...
INFO: Enabling host msiscsi service to support iscsi storage ...
WARN[2019-09-19T18:27:14Z] Service is not stopped, going to stop it
WARN[2019-09-19T18:27:17Z] Service is not stopped, going to stop it
FATA[2019-09-19T18:27:20Z] failed to register service: could not query status: The service has not been started.
Unable to find image 'rancher/rancher-agent:v2.3.0-rc5' locally
C:\Program Files\Docker\docker.exe: Error response from daemon: Get https://registry-1.docker.io/v2/: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers).
See 'C:\Program Files\Docker\docker.exe run --help'.
PS C:\> docker images
REPOSITORY                                                                    TAG                 IMAGE ID            CREATED             SIZE
ec2-13-56-165-173.us-west-1.compute.amazonaws.com:443/rancher/rancher-agent   v2.3.0-rc5          08436fc3bfaa        37 hours ago        561MB
PS C:\

I can fix the cannot stop service error by restarting docker daemon. But although the image ec2-13-56-165-173.us-west-1.compute.amazonaws.com:443/rancher/rancher-agent:v2.3.0-rc5 is pulled from my registry, docker still tries to start rancher/rancher-agent:v2.3.0-rc5.

https://github.com/rancher/rancher/blob/543d3822bdc80bf675a57c2a7ca047b79e738f4d/package/windows/bootstrap.ps1#L272
I think it's possible AGENT_IMAGE does not have the private registry prefix

Thanks @bmdepesa  to point out that we can set the Env variable in the docker command line. So I run
PowerShell -NoLogo -NonInteractive -Command "& {docker run -v c:/:c:/host -e AGENT_IMAGE=ec2-13-56-165-173.us-west-1.compute.amazonaws.com:443/rancher/rancher-agent:v2.3.0-rc5  ec2-13-56-165-173.us-west-1.compute.amazonaws.com:443/rancher/rancher-agent:v2.3.0-rc5 bootstrap --server https://jiaqi-ag-lb-e4f7e0724eaa18d0.elb.us-west-1.amazonaws.com --token gcjwzjcmt8bzn27tmxxk658dlx9hfwff4625clw85wrxl6jkkw765q --ca-checksum 78b36df18d1a6c24d5a85c8f616b36178ed6f5bbe376886b8a648d3ab8acbe6f --worker | iex}"

here is the output:
PS C:\> PowerShell -NoLogo -NonInteractive -Command "& {docker run -v c:/:c:/host -e AGENT_IMAGE=ec2-13-56-165-173.us-west-1.compute.amazonaws.com:443/rancher/rancher-agent:v2.3.0-rc5  ec2-13-56-165-173.us-west-1.compute.amazonaws.com:443/rancher/rancher-agent:v2.3.0-rc5 bootstrap --server https://jiaqi-ag-lb-e4f7e0724eaa18d0.elb.us-west-1.amazonaws.com --token gcjwzjcmt8bzn27tmxxk658dlx9hfwff4625clw85wrxl6jkkw765q --ca-checksum 78b36df18d1a6c24d5a85c8f616b36178ed6f5bbe376886b8a648d3ab8acbe6f --worker | iex}"
INFO: Detecting running permission ...
INFO: Detecting host CPU reservation ...
INFO: Detecting host RAM reservation ...
INFO: Detecting host DISK reservation ...
INFO: Detecting host system locale ...
INFO: Detecting host Docker name pipe existing ...
INFO: Detecting host Docker release version ...
INFO: Detecting host network interface ...
INFO: Configuring host Docker startup mode to automatical ...
INFO: Enabling host msiscsi service to support iscsi storage ...
WARN[2019-09-19T19:40:10Z] Service is not stopped, going to stop it
WARN[2019-09-19T19:40:14Z] Service is not stopped, going to stop it
FATA[2019-09-19T19:40:17Z] failed to register service: could not query status: The service has not been started.
b0abd8db4bc033350c7037be7ecb516d901ac890a6be835328a18b24f5763c2c
PS C:\> docker ps
CONTAINER ID        IMAGE                                                                                COMMAND                  CREATED             STATUS              PORTS               NAMES
b0abd8db4bc0        ec2-13-56-165-173.us-west-1.compute.amazonaws.com:443/rancher/rancher-agent:v2.3.0-rc5   "pwsh -NoLogo -NonIn…"   7 seconds ago       Up 5 seconds                            sad_austin

But the node is stuck in registering state in Rancher, and the log from the service-sidekick container in the node shows this
E0919 19:58:10.842316    5520 main.go:294] Error registering network: failed to acquire lease: node "ec2amaz-emeekib" not found
I0919 19:58:10.842316    5520 main.go:373] Stopping shutdownHandler...
INFO: Start flanneld with: --kubeconfig-file=c:\etc\kubernetes\ssl\kubecfg-kube-node.yaml --ip-masq --kube-subnet-mgr --iptables-forward-rules=false --iface=<IP_ADDRESS>
I0919 19:58:21.856651     172 main.go:457] Searching for interface using <IP_ADDRESS>
I0919 19:58:22.007554     172 main.go:534] Using interface with name Ethernet and address <IP_ADDRESS>
I0919 19:58:22.007554     172 main.go:551] Defaulting external address to interface address (<IP_ADDRESS>)
I0919 19:58:22.019599     172 kube.go:126] Waiting 10m0s for node controller to sync
I0919 19:58:22.019599     172 kube.go:309] Starting kube subnet manager
I0919 19:58:23.031476     172 kube.go:133] Node controller sync successful
I0919 19:58:23.031595     172 main.go:249] Created subnet manager: Kubernetes Subnet Manager - ec2amaz-emeekib
I0919 19:58:23.031595     172 main.go:252] Installing signal handlers
I0919 19:58:23.031595     172 main.go:393] Found network config - Backend type: vxlan
I0919 19:58:23.031595     172 vxlan_windows.go:116] VXLAN config: Name=vxlan0 VNI=4096 Port=4789 GBP=false DirectRouting=false
E0919 19:58:23.042664     172 main.go:294] Error registering network: failed to acquire lease: node "ec2amaz-emeekib" not found
I0919 19:58:23.042664     172 main.go:373] Stopping shutdownHandler...

So we have 2 issues here:

we need to fix the environment variable AGENT_IMAGE such that it can catch the private registry prefix
even we pass AGENT_IMAGE in the docker run command and workaround issue 1, and the node is added to the cluster (as shown in the UI ), the state is stuck in registering

In the windows node
cannot pull this image
ec2-13-56-165-173.us-west-1.compute.amazonaws.com:443/rancher/hyperkube:v1.15.3-rancher3
but can pull this one
c2-13-56-165-173.us-west-1.compute.amazonaws.com:443/rancher/hyperkube:v1.15.3-rancher3-windows-1903

For #1, I would try to find a way to catch the private registry image into AGENT_IMAGE, but it might be hard as Windows doesn't has a CGroup record like Linux: https://github.com/rancher/rancher/blob/7bf6786c2dab19d539b7e6afa34557f5d502cefd/package/share-root.sh#L5. so I prefer to your workaround at the moment.
For #2, you need to clean up the pre-adding host before re-add it, there is a cleanup script on c:\etc\rancher\cleanup.ps1.

After discussing with @loganhz , in order to solve https://github.com/rancher/rancher/issues/22966#issuecomment-533383440 #1, we will open a new PR to fix it: the UI would allow users to input the private registry address when adding host to the custom cluster. cc @n313893254 .

The bug fix is validated with Rancher:v2.3.0-rc7 release
Steps:

Follow the instruction to prepare the private registry: https://github.com/rancher/docs/issues/1742#issuecomment-532553582
Follow the Rancher documentation to install Rancher in airgap HA mode
add a custom cluster with both Linux and Windows node

Result:

the Windows node is added to the cluster successfully


How to edit the config to use more than 2 GPU running SplaTAM?

I don't know how to edit the config.py to use more than 2 GPU running SplaTAM. Is SplaTAM supprots parallel running?

Hi, Thanks for your interest. The current SplaTAM code doesn't support multi-gpu. This might be possible by using gsplat (https://github.com/nerfstudio-project/gsplat) as the backend.


"Tunari" and "Dimieni" are listed in the drop-down menu

Description:
"Tunari" and "Dimieni" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Tunari" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Tunari, ILFOV (Tunari)" and "Comuna Tunari, ILFOV".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Dimieni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Dimieni, ILFOV (Tunari)".


Apply license headers to source files

Tracked at UW-Madison with MUMUP-2885.
First commit adds the maven license plugin and configures usage, including using Apereo's license header.
Second commit adds the license header to the files.

Contributor License Agreement adherence:

[X] This Contribution is under the terms of Individual Contributor License Agreements (and also Corporate Contributor License Agreements to the extent applicable) appearing in the Apereo CLA roster.

Closing because css linting error.  Guess I didn't test the linting like I thought I did.  Will re-open soon.


Intermittent failure in WebSocketBridgeTest.testWebSocketsWhenTCPDisabled

WebSocketBridgeTest.testWebSocketsWhenTCPDisabled fails intermittently with message Received a failed result Address already in use
See https://travis-ci.org/vert-x3/vertx-stomp/jobs/502071942

Closing as outdated


Compare 2 types of markers with diferent icons or colors

Hi guys.
How can I compare 2 types of markers? Example, I have a lot of bananas's and watermelons's makers around the world. I wanna compare the bananas x watermelons, suposing the banana marker is yellow and watermelon is red, visually I can get it.
It's possible?

You should be able to call getIcon() on the Marker instance and compare the results.


Fix typo with script name

The documentation was missing the letter a in the reference to the command that starts the mysql server

Thanks tim, I went ahead and fixed the filenames... rather than change docs