docker swarm does not update images when it is a mode global

docker version
Client:
 Version:      1.12.2
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   bb80604
 Built:        Tue Oct 11 18:19:35 2016
 OS/Arch:      linux/amd64
Cannot connect to the Docker daemon. Is the docker daemon running on this host?
afuentes@chimera-swarm-dev-db-01:~$ sudo docker version
Client:
 Version:      1.12.2
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   bb80604
 Built:        Tue Oct 11 18:19:35 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.2
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   bb80604
 Built:        Tue Oct 11 18:19:35 2016
 OS/Arch:      linux/amd64

I have declared a service with mode global and with a constraint to run it only in two servers (my cluster runs in 3 servers). When I try to update as following:
sudo docker service update --image $IMAGE:$TAG $SERVICE

The service is not updated.
If I do it with in "replicas" mode it works perfectly.
Thank you in advance.

I can confirm this is an issue on 1.12.0-1.12.3 and looks to be fixed in https://github.com/docker/swarmkit/pull/1570 which is planned for 1.13

Just to confirm; if you update the service, are you changing the actual name/tag of the image, or is the name/tag of the image exactly the same as before?

Changing tag. The global service will create fine with constraints in place, but when running update, it won't actually change the image to the new tag. If constraints aren't used with initial create, it works as expected.

/cc @aaronlehmann

@BretFisher: I think you're right that docker/swarmkit#1570 fixes this. The problem in 1.12.x is that Docker tries to update the tasks one at a time, but because of the constraints, some of the tasks can never start, and this blocks the update.
I'll close this since the issue is believed to be fixed on the master branch.


[#16417] Remove unnecessary format functionality

Closes https://github.com/Leadfeeder/issue-tracker/issues/16417
It's not necessary to define a format for the value, as we can just JSON stringify / parse the value and it will work properly.
This PR removes the normalize from string util, and changes the logic so the tracked local storage class will stringify the value being set in browser local storage, then parse it when retrieving the value.
(It also adds the keyNameProperty feature which we are currently using in the monorepo package)
How to test
You can fetch and run the PR locally, then grabbing the tracked-local-storage service from ember inspector, and try manually setting / getting some local storage props of different types, and make sure everything is working properly.

Review @jbandura @tszymon

Review @jbandura @tszymon

Review @jbandura @Darren-LF

Review @jbandura @Darren-LF


How to extend popkorn to support memmove ?

Hi,
First I would like to thanks sharing for your awesome paper & work !
I read the last annex D, Case Study II and try to add support for memmove :
I added the following code to angr_analysis/angr_full_blown.py :
[redacted]
MEMMOVE = False

def check_imports(proj):
[redacted]
    mymemmove = proj.loader.find_symbol("memmove")
    if mymemmove:
        print("[+] Found memmove: ", hex(mymemmove.rebased_addr))

        MEMMOVE = True
        import_addr['memmove'] = mymemmove.rebased_addr

    else:
        print("memmove import not found!\n")

Then I run it on the Dell driver vulnerable to CVE-2021-21551 :
$ python3 angr_analysis/angr_full_blown.py datasets/my_dataset/dbutil_2_3.sys
Found WDM driver:  0x100068
Driver DEVICE_NAME:  \\\\.\\DBUtil_2_3
Looking for MmMapIoSpace, ZwOpenProcess, ZwMapViewOfSection and memmove Imports..
ZwOpenProcess import not found!
[+] Found MmapIoSpace:  0x100048
ZwMapViewOfSection import not found!
memmove import not found!
DriverObject @ 0x444f0000
[+] Finding the IOCTL Handler.
[redacted]

It didn't find memmove in the import so it fails to find the vuln.
I opened the driver in GHIDRA, indeed memmove is not imported.
Did I miss something ?
Have a nice day !

You are correct, memmove (or memcpy as Ghidra detects it) is not imported, but it is bundled in the application itself. So you would need a different kind of detection mechanism than looking at the import table.

ok !
What kind of detection mechanism did you use to manage to support this ?
thanks

I'm not a developer of this script, so I can't help you with that right now. I just thought to help you one step further in the right direction. Good luck!

I will have a look at this, thanks @zeze-zeze

@v1k1ngfr did you manage to find a solution? I'm now also running into this problem.


Japanese translation: the latter half of pages and rest

All fixed. Ready to merge.

@noroadsleft All fixed. Can you please merge this?

Sorry for the slow response. I was sick for a bit over a week and now I'm 1,200 km from home visiting family, so my internet usage has decreased.
Thanks!


How to use this with Chewie

How to Use Chewie with CachedVideoPlayerPlusController as a controller
or
How to convert CachedVideoPlayerPlusController to VideoPlayerPlusController ?

Currently it's not supported, but I'll see if I can change the code so that it extends video_player instead.

I am using video player with in Chewie in production, need this CachedVideoPlayer support soon

any update ?

any update ?

Not yet, would have to redo the package to extend the video_player plugin so it can be used with other packages.
If you are willing to work on it, PRs are always welcomed.


Slider dot in settings stays on screen after window is closed

Platform
Operating system and version: macOS Catalina 10.15.4 19E287 x86_64
Godot version: v3.2.1.stable.official
Device: MacBookAir8,1 with Intel i5-8210Y (4) @ 1.60GHz
GPU: Intel UHD Graphics 617
Steps to reproduce

Open any project
Open Project > Project Settings...
Find any setting with a slider (example: General > Display > Window > Width)
Hover the mouse cursor over the small circular slider dot
Press escape key to close Project Settings window
The dot stays onscreen

Hehe


What to do when memory usage is abnormally high?

You show a picture of having abnormally high memory usage. That is also the case for me lately. However, you do not provide a hint how to solve this.

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

ID: fc21902c-d522-3851-8587-177f01bc085c
Version Independent ID: 590bcc64-a86d-b61e-7d94-4e4322c4e71d
Content: How Microsoft Teams uses memory - Microsoft Teams
Content Source: Teams/teams-memory-usage-perf.md
Service: msteams
GitHub Login: @msdmaguire
Microsoft Alias: dmaguire

@bjschuit Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


pip10 is broken

Info:

Docker version ($ docker --version):
Laradock commit ($ git rev-parse HEAD):
System info (Mac, PC, Linux): Mac and Linux
System info disto/version:

Issue:

pip10 is broken
Expected behavior:

  Downloading https://files.pythonhosted.org/packages/0f/74/ecd13431bcc456ed390b44c8a6e917c1820365cbebcb6a8974d1cd045ab4/pip-10.0.1-py2.py3-none-any.whl (1.3MB)
Installing collected packages: pip
  Found existing installation: pip 8.1.1
    Not uninstalling pip at /usr/lib/python2.7/dist-packages, outside environment /usr
Successfully installed pip-10.0.1
Traceback (most recent call last):
  File "/usr/bin/pip", line 9, in <module>
    from pip import main
ImportError: cannot import name main
ERROR: Service 'workspace' failed to build: The command '/bin/sh -c if [ ${INSTALL_PYTHON} = true ]; then   apt-get -y install python python-pip python-dev build-essential    && pip install --upgrade pip    && pip install --upgrade virtualenv ;fi' returned a non-zero code: 1

Reproduce:

cp env-example .env
vi .env

and
INSTALL_PYTHON=true // default is false

and
docker-compose up -d workspace

Relevant Code:
From https://github.com/pypa/pip/issues/5240#issuecomment-383115342
workspace docker file
RUN if [ ${INSTALL_PYTHON} = true ]; then \
  apt-get -y install python python-pip python-dev build-essential  \
  && pip install --upgrade pip  \
  && pip install --upgrade virtualenv \
;fi

TO
RUN if [ ${INSTALL_PYTHON} = true ]; then \
  apt-get -y install python python-pip python-dev build-essential  \
  && pip install --upgrade pip==9.0.3  \
  && pip install --upgrade virtualenv \
;fi

I personally modify this way.
RUN if [ ${INSTALL_PYTHON} = true ]; then \
  apt-get -y install python python-pip python-dev build-essential  \
  && python -m pip install --disable-pip-version-check --user --upgrade pip  \
  && pip install --upgrade virtualenv \
;fi

ARG INSTALL_PYTHON=false

RUN if [ ${INSTALL_PYTHON} = true ]; then \
  apt-get -y install python python-pip python-dev build-essential  \
 # && pip install --upgrade pip  \
  && pip install --upgrade virtualenv \
;fi

commented out this line to have a temporary fix on this issue.

Here is the fix I implemented when getting build tests to work
ARG INSTALL_PYTHON=false

RUN if [ ${INSTALL_PYTHON} = true ]; then \
  apt-get -y install python python-pip python-dev build-essential  \
  && pip install --upgrade pip  \
  && hash -r pip \
  && pip install --upgrade virtualenv \
;fi

Added the has -r pip line which solves the build problem

I fix with
python -m pip ins......

Adding   apt-get -y update fixed the issue
`
ARG INSTALL_PYTHON=false
RUN if [ ${INSTALL_PYTHON} = true ]; then
apt-get -y update
&& apt-get -y install python python-pip python-dev build-essential
&& pip install --upgrade pip
&& hash -r pip
&& pip install --upgrade virtualenv
;fi`


Update main.py

https://pylint.pycqa.org/en/latest/user_guide/messages/convention/consider-using-dict-items.html
consider-using-dict-items / C0206
Message emitted:
Consider iterating with .items()
Description:
Emitted when iterating over the keys of a dictionary and accessing the value by index lookup. Both the key and value can be accessed by iterating using the .items() method of the dictionary instead.


Misc process improvements

Here are 3 improvements to our development process:
Use one development container for migrations and running the app
docker-compose-local.yml and docker-compose-local-migrate.yml define a "migrations" container that runs flask db upgrade and exits.  This can be folded into the "app" development container.  A separate container is unnecessary.
Requirements management
The repo contains requirements.txt and requirements-app.txt, among others.  The latter includes only top-level package requirements.  The process implemented by the previous team was to run the app locally using requirements-app.txt, run make freeze-requirements to create requirements.txt, and build the containers with requirements.txt.
Now we are using the containers directly for development, and this process is not appropriate.  The process should be:

Use requirements-app.txt with Dockerfile.local.
Periodically, and prior to any update to production, run docker exec -t ci_app_1 pip freeze > requirements.txt from the project root directory to update requirements.txt with the complete list of the latest dependencies.  Continue using requirements.txt with Dockerfile.

This will simplify dependency management, and it will address the security issue currently flagged by Dependabot.
Don't mandate rebuilding a Docker image to run unit tests locally
Every time I run unit tests locally, the Dockerfile.test image gets rebuilt, and this wastes considerable time.  Modify Dockerfile.test to mount the application directory as with Dockerfile.local.  Rebuilding Dockerfile.test will only be necessary when dependencies change.
Other work

Update README.md to reflect current processes.
Upgrade dependencies.

@mjones-oddball

Eliminating the "migrations" container didn't work as easily as I hoped, so I abandoned that effort for now.  I accomplished my other objectives.


Possible to get client  ipv4 address on HttpRequest or HttpResponse?

Hi,
I need to get the client ip address before the websocket connection and i have this:
  const app = uWS
  .SSLApp({ key_file_name: './privkey1.pem', cert_file_name: './cert1.pem' })
  .ws('/*', require('./socket/connect'))
  .any('/*', (res,req){
 res.cork(() => {
   res.write('
');
   res.write('Your proxied IP is: ' + Buffer.from(res.getProxiedRemoteAddressAsText()).toString());
   res.write('

');
   res.write('Your IP as seen by the origin server is: ' + Buffer.from(res.getRemoteAddressAsText()).toString());
   res.end('
');
  });
})
  .listen(443, (token) => {
   if (token) {
    console.log('Listening to port 443');
   } else {
    console.log('Failed to listen to port 443');
   }
  });

but I'm getting:
Your proxied IP is:
Your IP as seen by the origin server is: 0000:0000:0000:0000:0000:ffff:bc1b:f640

This seems my ipv6 address but if i check on https://www.whatismyip.com/ i get
 My Public IPv4 is: <IP_ADDRESS>
 My Public IPv6 is: Not Detected

Is is safe to use this address to identify connections coming from the same IP?
Is there a way to get the ipv4 address?
Thanks

You dont need the getProxiedRemoteAddress unless you have a reverse proxy set up, see here. The 0000:0000:0000:0000:0000:ffff:bc1b:f640 is the ip6 format which converts to/is the same thing as your ip4 <IP_ADDRESS>. All ip4 can convert to ip6 but not all ip6 can convert to ip4. So you should probably leave it in ip6 format, but you can convert to ip4 if needed https://stackoverflow.com/q/2786632

OK, Thanks


It's not clear what quota will be consumed

Does low-priority VMSS consume regular core quota, or does it consume separate "low-priority core" quota, like it is in Azure Batch?
What quota does one need to request from support?

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

ID: a1bd5076-fe52-cfca-4173-dec861167812
Version Independent ID: 0122fff1-79e0-446c-00a4-85248a21b6dc
Content: Create an Azure scale set that uses low-priority VMs (Preview)
Content Source: articles/virtual-machine-scale-sets/virtual-machine-scale-sets-use-low-priority.md
Service: virtual-machine-scale-sets
GitHub Login: @mmccrory
Microsoft Alias: memccror

@PoisonousJohn Thanks for the question! We are investigating and will update you shortly.

@PoisonousJohn the quota amount would be the same as you are still using the same amount of cores. The exception with low priority is that we can shutdown your VMs if something higher priority comes into play.
@mmccrory maybe something we need to add into this doc.

Reached out to PM owners and will update the docs as necessary.
#in-progress

@reassign @mayanknayar

@mayanknayar When the feedback has been addressed, you can close the issue by typing # please - close (just remove the extra spaces).

@cynthn - I sent you further connection details for low-priority. Please close the thread after discussion with the right owners.
#reassign @cynthn

#reassign @JagVeerappan

We are planning to refresh this document, and we will address this question in the refresh. #please - close.

#please-close


example code syntax error

[Enter feedback here]
There is an issue in the example code for adding the permissions for the client application and server application components to use the oAuth2 communication flow.
example code is:
az ad app permission add --id $clientApplicationId --api $serverApplicationId --api-permissions $oAuthPermissionId=Scope
but should be:
az ad app permission add --id $clientApplicationId --api $serverApplicationId --api-permissions ${oAuthPermissionId}=Scope

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

ID: bc86c3af-f6c7-4c48-e157-54abbc5958ff
Version Independent ID: 3611b478-6df6-f5cd-563d-2fff6cfec9ef
Content: Integrate Azure Active Directory with Azure Kubernetes Service
Content Source: articles/aks/azure-ad-integration-cli.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@AlexanderViken Thanks for the Feedback.
I have raised a PR for fixing this and changes will go live once the PR is approved.


CI ONLY Scala upgrades

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3899/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/5156/
Test FAILed.

Jenkins, retest this please

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3900/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/5157/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3901/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/5158/
Test PASSed.

Nightly: https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/

Nightly passed after one rerun. (neg/patmatexhaust-huge.scala was flaky)


Optimize performance by checking dirty/moved area.

Checking LastPresentTime is enough.

If only the pointer was updated (that is, the desktop image was not updated), the AccumulatedFrames, TotalMetadataBufferSize, and LastPresentTime members are set to zero.


Allow docker-gen to be used as a library

As discussed briefly at https://github.com/jwilder/docker-gen/pull/130 and https://github.com/jwilder/nginx-proxy/pull/254, it could be beneficial to restructure docker-gen to be usable both as a standalone binary and as an embedded library.
I'm opening this issue as a place to discuss the details of such a change.

+1 I would like to make a plugin to another go app so it can reconfigure itself.

I would love an api that notifies me anytime the set of running containers changes. Perhaps I can provide a callback that will receive []*RunningContainer anytime something new happens, or on an interval.

FYI, I've started some work on this change here: https://github.com/appropriate/docker-gen/tree/generator
It's currently not compiling due to being in an intermediate state. I still need to figure out how to deal with the GeneratorConfig struct and I haven't had time to work on this in the last week.


Feature: Upgrade to latest version of the pizza-cli

Currently we're using the beta branch. See https://github.com/open-sauced/pizza-action/blob/main/action.yml#L29C1-L32C18.
We should update to what we consider the latest stable release as some bug fixes/features are in flight at the moment.

@jpmcb, in action.yml, do you think it's reasonable to always have the latest version of the CLI set?
Currently it's @beta, but I was going to hard-code a specific release for the launch, see https://github.com/open-sauced/pizza-action/blob/main/action.yml#L35, but I'm thinking @latest seems fine.
The pizza-action is pretty much feature complete so just wondering if it makes sense to release everytime we have a CLI release.
Another option is we could default to @latest, but we could add another input to the action which could be cli-version which would default to latest. Something like this.
name: "OpenSauced Pizza Action"
description: "GitHub Action to run OpenSauced's Pizza CLI's various commands"
branding:
  icon: "package"
  color: "orange"
inputs:
  pizza-args:
    description: "OpenSauced Pizza CLI command to run"
    default: "generate codeowners ./ --tty-disable"
    required: true
  commit-and-pr:
    description: "Commit and push changes"
    default: "false"
    required: false
  pr-title:
    description: "Custom pull request title"
    default: "chore (automated): OpenSauced updates"
    required: false
+  cli-version:
+    description: "The pizza CLI version to use"
+    default: "latest"
+    required: false

runs:
  using: "composite"
  steps:
    - name: Checkout
      uses: actions/checkout@v3
      with:
        fetch-depth: 0

    - name: Setup Go
      uses: actions/setup-go@v4
      with:
        go-version: "1.22.5"

    - name: Install pizza CLI
      run: |
-        go install github.com/open-sauced/pizza-cli@beta
+        go install github.com/open-sauced/pizza-cli@${{ inputs.cli-version }}
      shell: bash

    - name: Check pizza CLI version
      run: pizza-cli version
      shell: bash

    - name: Run pizza CLI command
      run: |
        pwd
        ls -la
        pizza-cli ${{ inputs.pizza-args }}
      shell: bash
      working-directory: ${{ github.workspace }}

    - name: Make script executable
      run: chmod +x ${{ github.action_path }}/scripts/create-pr.sh
      shell: bash

    - name: Setup git config
      run: |
        git config user.name 'open-sauced[bot]'
        git config user.email<EMAIL_ADDRESS>      shell: bash
      working-directory: ${{ github.workspace }}

    - name: Create a PR with pizza CLI changes
      env:
        GH_TOKEN: ${{ github.token }}
      if: ${{ inputs.commit-and-pr == 'true' }}
      run: |
        export CUSTOM_PR_TITLE="${{ inputs.pr-title }}"
        ${{ github.action_path }}/scripts/create-pr.sh
      shell: bash
      working-directory: ${{ github.workspace }}

Another option is we could default to @latest, but we could add another input to the action which could be cli-version which would default to latest. Something like this.

^ This is the way. Technically the pizza-action and pizza-cli are completely seperate pieces of software and should not couple their version mechanics.
If we were going to just take latest, people would enevitably consume breaking changes. For example, say there's a breaking change in v1.2.3 of the pizza-cli and people continue to consume v7.8.9 of the pizza-action (and thus consume the latest which would bring in v1.2.3).
Decoupling the version of the configuration software with the version of the dependent software will be a net win.
So, yes, I'd say a  cli-version config makes alot of sense and default to latest is 👍🏼


Jump to Fold Text

I use folds pretty regularly in order to quiet some of the "visual noise" of the codebases I work in.  I'd like to be able to jump to the top of a collapsed fold in the same way that I can jump to expanded text.  For example, if I wanted to jump to the doThings function, I'd like to be able to type sdo and have the cursor jump to line 6.

Is this currently possible or would it be a new feature?
Thanks for such a useful plug-in, btw!

This is an obvious request, I've been wanting to implement this for ages, but seems impossible at the moment. Fold headers cover everything unconditionally, including extmarks, we cannot labels positions there. (Semi)related: https://github.com/neovim/neovim/issues/12649.

@ggandor Thanks for the context!  I figured it must be some sort of technical limitation since none of the other similar plugins (sneak.vim et al) have this feature either.  Crossing my fingers that support for this can be added soon!


rework yjs-store-db syncing

This PR is a major rework of the syncing logic among Yjs, Zustand store, and DB. The syncing logic will be detailed in the comment below.
Additional changes:

Remove addPod graphQL API. Use the updatePod API instead, which will create a pod if it does not exist.

This fixes the pod creation & updates race condition. We don't have to await the pod creation before updating it in the front-end canvas.
Additional note: the addPod Zustand store action will not accept ApolloClient anymore. Instead, you can set the pod.dirty=true flag to trigger the periodical updates to DB.

use @reactflow/node-resizer for scope resize.

The three data stores and their roles:

Yjs data: Yjs is the single source of truth in terms of canvas operations. It is in store.ydoc.nodesMap. This data is shared by all peers. The canvas is rendered from this data.
DB data: On canvas loading, if Yjs data is empty or inconsistent with the database, overwrite Yjs data with database data. That is, the database is the single source of truth in terms of eventual consistency.
Zustand store: The local Zustand store is used to store graph structure and metadata, as well as handle DB writes.

The files: the main logic is implemented in three files:

lib/nodes.tsx: this implements Yjs observer
components/Canvas.tsx: implements the Canvas React component.
lib/store/canvasSlices.tsx: actually implements all the operations (updateView, copy/paste/cut logic)

How the canvas nodes are rendered?
The ReactFlow canvas data is stored in store.nodes, and passed to <ReactFlow nodes={store.nodes}/>. Updating this state will trigger a canvas re-rendering. Only one action is allowed to actually update the nodes, the updateView function. Other actions are supposed to call it whenever a re-rendering is desired.
The updateView action computes the new nodes from the following data sources:

All nodes in Yjs nodesMap.
The pasting/cutting node. Nte: these temporary nodes are maintained outside nodesMap because they are local to the local user.

Additional logic is implemented in updateView to modify the nodes, e.g.,

add background color according to scope level;
set the className for cutting nodes and active scope.

Handling the changes made by remote peers
All changes made by remote peers are observed with useYjsObserver(). These observed changes (add,remove,update) will

Write to the local Zustand store without DB sync.
trigger updateView action to update the canvas.

Handling the changes made by the local user
Overall, the local user's changes will

write to nodesMap to sync with remote peer
write to zustand store, sync with DB (using the dirty=true flag), then call updateView to update canvas.

The specific logic for each Canvas operation:

Add node:

Update nodesMap to sync with the remote peer.
Do local updates (zustand store, db, updateView)

Move a node: onNodesChange will observe the change. It calls applyChange to compute new nodes, and do:

Set nodesMap to sync with the remote peer
Do local updates (zustand store, db, updateView)
Check if this is moved into scope. (handled by <ReactFlow onNodeDragStop={...}>)

Select nodes: onNodesChange observe the selection event, and record and update the local canvas.
Remove node:

nodesMap.delete to sync with peer
Do local updates (zustand store, db, updateView)

Paste

Begin (triggered by ctrl-v)

Create and set store.pastingNode. Set store.isPasting flag.

onMove: The "move" event listener will set store.pastingNode's position and update the view so that the pod will follow the cursor.
End:

Update nodesMap to sync with the remote peer
Do local updates (zustand store, db, updateView)
Check if dropped into a scope. If so, call store.moveIntoScope action to update parentNode and recompute all scopes' levels.

Cancel: clear the pastingNode and isPasting

Cut (triggered by clicking the cut button): similar to pasting.


syntactic_head_ID erroneously references a token in the previous sentence

Hi, thank you for making the great library!
When parsing long documents, the syntactic_head_ID will sometimes reference a token in the previous sentence. For example, in the parsing output in the attached file (dKDD.csv):
0	2	0	28	she	she	122	125	PRON	PRP	nsubj	29	O
0	2	1	29	's	be	126	128	AUX	VBZ	ROOT	29	O
0	2	2	30	not	not	129	132	PART	RB	neg	29	O
0	2	3	31	the	the	133	136	DET	DT	det	32	O
0	2	4	32	one	one	137	140	NOUN	NN	attr	29	O
0	2	5	33	to	to	141	143	PART	TO	aux	34	O
0	2	6	34	write	write	144	149	VERB	VB	relcl	32	O
0	2	7	35	.	.	150	151	PUNCT	.	punct	29	O
0	3	0	36	Yeah	yeah	152	156	INTJ	UH	intj	35	O
0	3	1	37	.	.	157	158	PUNCT	.	punct	36	O

The syntactic_head_ID of token 36 (in sentence 3) is token 35 (sentence 2), which doesn't seem to make sense.
The same happens with tokens 62, 68, 91, 202, 276, 327, 328, 344, 376, 378, 385, 387, 433, 434, 499, 503, 516, 550, 556, 557, 558, 566, 589, 725, 751, 755, 813, 818, 843, 845, 853, 876, 880, 1450, 1502, 1563, 1756, 1881, 1882, 1902, 1926, 1972, 1993, 2054, 2058, 2059, 2086, 2097, 2103, 2488, 2489, 2511.
Is there a way to fix this?
dKDD.csv
dKDD.txt

Thanks for the note! Yes that does seems weird -- if this is with a version <1.0.7, try upgrading and see if it still happens (I'm running 1.0.7 and the "big" model and not seeing that issue with dKDD.txt.)

Thanks for the note! Yes that does seems weird -- if this is with a version <1.0.7, try upgrading and see if it still happens (I'm running 1.0.7 and the "big" model and not seeing that issue with dKDD.txt.)

I tried and discovered that this bug only occurs when using en_core_web_lg as the parsing model (which I prefer as it appears to give more accurate results). Any idea why this is happening?

Which version of the en_core_web_lg model are you using @victoryhb?

@wjbmattingly I am using version 3.1.0


Add Logo on Navbar

Is your feature request related to a problem? Please describe.
I want to add a logo beside the BookFinder tag on the navbar.
Describe the solution you'd like
I want to add a logo beside the BookFinder tag on the navbar.
Describe alternatives you've considered
NA
Additional context
No response

I want to work on this issue under JWOC


1204 People, image, location

Checks

[x] Your YAML file has required fields as listed in Specs
[x] All checks have passed yaml-lint passed
[x] You agree to contribute your work under MIT license

this is not "people image locations"


python310Packages.channels: 3.0.5 -> 4.0.0, python310Packages.daphne: 3.0.2 -> 4.0.0

Description of changes
https://github.com/django/channels_redis/blob/main/CHANGELOG.txt
https://github.com/django/channels/blob/main/CHANGELOG.txt
https://github.com/django/daphne/blob/main/CHANGELOG.txt

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 197042 run on x86_64-linux 1

2 packages marked as broken and skipped:

paperless
paperless-ng

8 packages built:

baserow
paperless-ngx
python310Packages.channels
python310Packages.channels-redis
python310Packages.daphne
python39Packages.channels
python39Packages.channels-redis
python39Packages.daphne

Ok. Figured out that there is an error in the new versions and this is why the tests fail (as they should). Could you have a look at https://github.com/gador/nixpkgs/tree/pr-197042 and incorporate it in this PR?

Overriding is for sure the better day to deal with the issue.

Result of nixpkgs-review pr 197042 run on x86_64-linux 1

8 packages built:

baserow
paperless-ngx
python310Packages.channels
python310Packages.channels-redis
python310Packages.daphne
python39Packages.channels
python39Packages.channels-redis
python39Packages.daphne


update of RYDE logo.png image manual review requested

I'm submitting a ...
[ ] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary

[*]  Files changed

PR #22778
updated logo.png of RYDE image
thx

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

was added by @m33lucky-benny and needs approval by that account to do the change. added in this PR: https://github.com/solana-labs/token-list/pull/21715/files


Run code quality checks as a unit test

We could run the code quality checks as a unit test, to make sure code quality is always checked

Has this been done? Or been decided against?

Wasn't this addressed in the latest merge by @bouweandela ?

Yes, sorry. I was a bit confused by the fact the issue was not directly closed from the merge. But now I understand that this is just due to the technicality that the refactoring branch is (for clear reasons) not the default branch in this repository. As a side note: even then the automatic closing will work only if all issue numbers are prefixed indiviually with a keyword.
https://help.github.com/articles/closing-issues-using-keywords/


overlay-edits.awk should consider the 0/0 condition

at line 27 to 28:
if(FNR==2){z=int($9/$2)}
if(int($10/$3)!=z || z==0)
this may cause an Undesired error

Thank you, this check is now included.


some translated strings don't display in KUI Getting Start page

Describe the bug
Some strings in KUI getting start page are translated, but they display as English when change the locale to non-English.
Steps to reproduce the behavior

Open KUI
run help
Please refer to the attached screen capture.
my mail is<EMAIL_ADDRESS>
Expected behavior
The strings should display translated languages.
Screenshots
If applicable, add screenshots to help explain your problem.
Code snippets
If applicable, add code samples to help explain your problem.
System

Kui version:1.0.12
Operating system:Mac OS

duplicate of #2669


npm run build takes 1-2 hours to build

First off, thank you for everything.
I have a fairly big react app (200K locs , css excluded), and recently it takes me hours to npm run build (It used to be like 9min which was aweful as well but i could still live with it). Some change in the last few weeks, which we haven't been able to identify what because our package.json still looks fine, has increased the build time from 10m to 1-2hours, sometimes 3 on my laptop. And the build process usually runs out of memory and dies on my colleagues' machines.
We use react-router and react-loadable to split  the code.
When I examine the build folder, i still have like
main.ad2ec57c.js 3.5M
main.ad2ec57c.js.map : 21M
and lots of other chunks of size 5-100K.
Can you pls point me some direction on how to go about solving this?

Can you disable sourcemaps?
Put
GENERATE_SOURCEMAP=false

in your .env file and try again.
Also it would help to have more info about react-scripts version, etc.

Thanks @gaearon for the really quick reply
I'm running Ubuntu 16 LTS and have
"react": "^16.3.2",
"react-scripts": "^1.1.4",
node v8.1.2
npm 6.0.1
And here is my complete package.js : https://gist.github.com/clu3/d008f57f77dbff284427aa4666fb3b58
Im trying to re-build with no source map. It still looks like it's taking time. So i'll go ahead and post my comment here, hoping i'll get something figured out earlier.


yjaf-gateway access

Use this issue template if you do not feel comfortable raising a PR with the required changes to files in the terraform/ directory.
Please supply all the requested information, so that your request can be processed as quickly as possible.
Which MoJ repository/s are you requesting access to (e.g. money-to-prisoners)?
yjaf-gateway
What is the GitHub username of the person who needs access?
ttipler
What is their full name?
Thomas Tipler
What is their email address?
<EMAIL_ADDRESS>What company/organisation do they belong to (e.g. "Department for Education")?
NEC (YJAF)
What level of access should they have?

[ ] Read
[ ] Triage
[ ] Write
[ ] Maintain
[x] Admin

Why is this access being requested?
Currently devops - Missing access to this one repo for work on circleci pipeline
Which MoJ team/person is responsible for this request? This must be a ".justice.gov.uk" or ".digital.justice.gov.uk" email address.
This should be an email address<EMAIL_ADDRESS>Team email addresses are preferred, rather than individual email addresses, to make it easier to follow-up at a later date.
<EMAIL_ADDRESS>Review date
Date after which this collaborator's access should be reviewed. This should be a date in the format: 2021-01-15, no more than one year into the future from today.
06/09/2024

Access already given. Resent invite link.


Complete draft

Add /complete draft endpoint
Sends the service id to the API to mark the service as "completed"
and redirects to the draft services page.
Shows "service was completed" message on the services page.
Split service name and description into separate sections
Since services without names are not shown in the service list
saving the service name should be as simple as possible and
shouldn't require a value for the service summary.
Hide "return to service" links for drafts without a name

Drafts without a name won't appear in the drafts list, so it shouldn't
be possible to input any other data until the draft name is set.
Hiding the links for unnamed drafts means there's no way to get to
the draft sections page before entering the service name.
Don't allow updates to non-editable sections
Section editable flag affects whether section "Edit" link is visible.
But it was still possible to open the section edit page for non-editable
sections and the changed data was validated and saved as service update.
This aborts any edit attempts to readonly sections with a 404.
Show complete and draft submission lists and counts
Splits draft services based on status into complete and incomplete
drafts.
Hide unnamed services from the draft services list
The need for non-stored "New service" name for unnamed services
pops up in different contexts (message notifications, button labels,
copied service names etc.), so in order to avoid these edge cases
we've decided to pretend that unnamed services weren't saved in the
first place.
Service name is the section right after service lot, so we've removed
any links to the draft overview page for unnamed services and split
the description into a separate section to make this step easier.
This way if the user decides not to enter a service name the service
is never displayed in the drafts list and there's no way to get to
any other edit page to enter other service data (apart from manually
entering the URL).
Unnamed drafts can be cleaned up when the framework goes live.
Reverse draft lists so that newer drafts are always first
Since API returns drafts order by asc(id), reversing the list shows
newly created or copied drafts at the top of the list.
Add "Move to complete" button and questions count to the draft page
Uses content questions data to count required questions that don't
have a value (this matches the check for "Answer required" in the
section fields).
If all required questions are answered a "Move to complete" button
is displayed.
Add last edit information to the draft service page …
Shows time and user name of the last update using the auditEvents
data sent by the API endpoint.

Show unanswered question counts on the drafts list page

Need to speak to @ralph-hawkins about this, but I think we need a line under the complete button that says something like "You can still edit services once they're complete".
Otherwise we'll probably have a situation where everyone waits until the last minute to complete their services, just in case they want to make changes.

:+1:


Formatting of python code fencing for SidecarEvaluatorModelExport

In the tensorflow.org the documentation page of tf.keras.callbacks.SidecarEvaluatorModelExport showing content of raw HTML pages. The reason may be that the Python code fence was not closed causing the page view breaking. Hence correcting the mistake.
May Fixes TF ticket #61375

Done the changes requested. Thank you!

Looks like it's failing lint check, please run sh shell/lint.sh and remove the whitespace from the newline.

Hi @SuryanarayanaY Can you please check @grasskin's comments ? Thank you!

Hi @SuryanarayanaY Any update on this PR? Please. Thank you!

Opened a new PR#723.


This line leads to customer confusion...

https://github.com/microsoft/azure-pipelines-tasks/blob/db1a26a76be78d6a4d97144ebd861d08ae8a8c3e/Tasks/AzureResourceGroupDeploymentV2/task.json#L176
Please do consider renaming the label to 'Complete (will destroy existing resources)' to allow the customers to be aware of the potential damage it may cause to existing resources.
Best Regards!

Hi @ernani https://docs.microsoft.com/en-us/azure/azure-resource-manager/deployment-modes describes more about it and link to this is already given in the task.json

I would rename it just in case as some developers are experimenting with DevOps before knowing the impact of both modes and may cause a unrecoverable harm (that recently happened in a real customer case scenario).


MSSQL Statistics collection

This approach is too slow in practice. I will open-source the better approach once it's working.


Update zio, zio-test, zio-test-sbt to 1.0.0-RC19-1

Updates

dev.zio:zio
dev.zio:zio-test
dev.zio:zio-test-sbt

from 1.0.0-RC18-2 to 1.0.0-RC19-1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

labels: library-update, semver-pre-release

Codecov Report

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

@@             Coverage Diff             @@
##            master      #79      +/-   ##
===========================================
- Coverage   100.00%   97.56%   -2.44%
===========================================
  Files            1        1
  Lines           41       41
===========================================
- Hits            41       40       -1
- Misses           0        1       +1

Impacted Files
Coverage Δ

...src/main/scala/com/vladkopanev/zio/saga/Saga.scala
97.56% <0.00%> (-2.44%)
:arrow_down:

Continue to review full report at Codecov.

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


Testing the new weights reveals issues: they fail to recognize general objects

"Is this training pipeline fine-tuning pretrained SwinT weights or training from scratch? Testing the new weights reveals issues: they fail to recognize general objects and have incomplete detection on familiar frames.

@Pratik-Ghute pretrain or finetune are OK. Using finetune can save some costs, but if you can obtain more and higher quality data, you can train the model from scratch for higher performance.

I wanted to know what this repo is doing finetuning or training from scratch ?

The models we release (1.8M and COCO) are finetuning on GDINO-T
Some models we not released training from scratch

ok and if I train my model using this command:
!python main.py
--output_dir "/content/"
-c /content/Open-GroundingDino/config/cfg_odvg.py
--datasets /content/Open-GroundingDino/config/datasets_od_example.json
--pretrain_model_path /content/Open-GroundingDino/weights/groundingdino_swint_ogc.pth
--options text_encoder_type=/content/Open-GroundingDino/bert_weights/
    It must be finetuning on groundingdino_swint_ogc right?

No, you can use any model, it doesn’t matter fine-tuning or pre-training

No, you can use any model(GDINO pretrain, DINO pretrain, swin pretrain, random init, etc. ), it doesn’t matter fine-tuning or pre-training.

ok thanks alot ,
"Requesting clarification on the multiple weight files generated. Is 'checkpoint_best_regular.pth' the weight file for the best epoch with the highest accuracy?
Additionally, in the 'eval' folder, what are the purposes of 'latest.pth' and '000.pth' weight files?"

@Pratik-Ghute

Is 'checkpoint_best_regular.pth' the weight file for the best epoch with the highest accuracy? yes
you can just ignore these files, it dumps some eval metrics for the latest evaluation.

>>> x = torch.load("000.pth")
>>> x.keys()
dict_keys(['params', 'counts', 'date', 'precision', 'recall', 'scores'])

@Pratik-Ghute pretrain or finetune are OK. Using finetune can save some costs, but if you can obtain more and higher quality data, you can train the model from scratch for higher performance.

For fintune: choose a reasonable steps(epochs) to get good downstream task performance but avoid overfitting (depends on dataset itself)

For generalization(very important): you can use model ensemble tech such as WISE-FT to balacne the weights of finetune model and pretrain model.

OWL-V2  (which is the strongest OVD model now) is also used this model ensemble trick.

will model ensemble tech launch two or more models everytime I inference or take predictions.

as for me, I do that offline and get a new model by ensemble pre-train and fine-tuned model. so nothing changed in inference stage


Access  TField->OldValue via PSQLQuery

Hellow.
When I try to access  TField->OldValue from PSQLQuery it alway undefind
TField->NewValue always have actual data - on Open() its from db, after edited.
CachedUpdates = true

I find the proble. It depend of FieldName case.
in db "username", in TField->FieldName='USERNAME'.
PSQLQuery1->FieldByName("USERNAME")->OldValue   ---  undefined
when TField->FieldName='username'.
PSQLQuery1->FieldByName("USERNAME")->OldValue  filled whith correct value.


Demotivator-Generator | Aseev Grigory

Reviewer: @SpirinEgor

@SpirinEgor, corrected all comments, please check in your free time.


Put experimental label on KodaDot -- Beta label

Most marketplaces puts them selves in the political position they are in beta, however, we are quite rolling revenue, traction userbase, with adding extra new features on daily basis could bring us more armour in argumentation and withstand any incoming drama (double spends hahahaa, pls resolve it yuri)

could be even in form of shield gun lmao

We already have


RPC failed during git clone due to large repo size

While attempting to clone the repository signature-base, in rare cases the following error can be encountered:
[-] Captured error when running system level command, log output:
Cloning into 'signature-base'...
error: RPC failed; curl 92 HTTP/2 stream 0 was not closed cleanly: CANCEL (err 8)
error: 2974 bytes of body are still expected
fetch-pack: unexpected disconnect while reading sideband packet
fatal: early EOF
fatal: fetch-pack: invalid index-pack output

Issue Details:
This error seems to occur due to the size of the repository, which can cause issues during the cloning process, especially over slower networks or when using HTTP/2. Sometimes, large repositories lead to incomplete downloads or fetch errors, as seen above.
Temporary Workaround:
There is a workaround in place using --depth 1 to minimize the data cloned, which may help avoid such issues. However, a permanent fix to ensure reliable cloning is still being evaluated.
Next Steps:
This issue will be looked into at a later stage to explore a more robust solution. This GitHub issue has been opened to make users aware of the problem and track any updates or fixes related to this.

Additional Information:
This error is commonly associated with network or buffer size limitations during the cloning process of large repositories.
Users encountering this issue can try the workaround with --depth 1 or adjust git settings like http.postBuffer.

A minor fix will be in place for this issue, hence I shall be closing it today. This will not be treated with as much urgency due to the rare state of this issue, which currently has not happened again based on some fixed I have done, to confirm, I shall be doing a check and a potential retry mechanism for the following.

Added an if statement to catch this issue which informs the user about the problem. This is a very minor fix however as mentioned the chances are very low to encounter this problem due to new enhancements are in place.


Latest Docker Container Untagged

The latest tag of the official Docker image contains an unreleased version of powerdns-admin, which appears to be 32983635c6bc51d2ca74496aa9cb0cf834fe2563. If it's considered stable, could you tag a new release on GitHub? Thank you!

A new image has been built and released lately.
https://hub.docker.com/r/ngoduykhanh/powerdns-admin

Hi, thanks for that! However, my issue is about the opposite: The version isn't tagged here on GitHub. I maintain the powerdns-admin package on Nixpkgs and would be great if there is some official indication that the version is stable and ready.

Thank you for the explanation,
Regarding the releases, @ngoduykhanh should tag a new version.
Thanks for the feedback :)


Refactored matplotlib axis setting for easier reusage

Not ready yet but this PR begins to refactor some of the code in setting up the axes limits, labels, ticks, aspect etc. to make reusing this code easier in custom extensions with different types of axes.

All these changes look good.
@philippjfr  That said, can we agree to have a freeze on the plotting code once this is merged?
I really feel we need a chance to document as much of the API (including internal API) that you consider stable. This will take the pressure off you and allow other people to start contributing by writing their own plotting classes. I don't think you should be the sole maintainer of the plotting module forever!
In other words, I think all these refactors have pushed your current bus factor a bit too high. :-)

I would also like to link to PR #438. That seems the right place to start thinking about the API in order to finally document it properly.

Yes, there will be a freeze on the plotting code until we have sufficiently documented the current implementation. Both the matplotlib and bokeh element plots should now be pretty stable and after documenting both we can consider how to improve other aspects of the plotting code including improved layouts, unifying some of the options across backends and then finally document how to create an entire backend.

The current test failures are as follows and are all expected:

Columnar Data:

15/16/22: Axis label changes

Composing_Data:

0/3: GridSpace gridlines reenabled

Containers:

9: GridSpace gridlines reenabled

Elements:

8/10: Axis label changes
11/21: Tiny axis frame change

The axis label change is to provide more consistent handling for axes representing more than one dimension (e.g. on Bars and ErrorBars types). The GridSpace line change reverts an earlier change where I'd accidentally removed gridlines completely when the axis itself is hidden. The axis frame change is down to adjoined plots, where the edge where the adjoined plot attaches was automatically added even if not specified. I'll rebuild the data soon and then this PR is ready to review and merge.

Tests passing, ready to review.

I've had a look through and I am happy to merge if you are happy with it. My only other suggestion is to run flakes on your branch because there is quite a bit changed and flakes might help you catch bugs.

Good plan, I'll apply your has_axes suggestion and run a few linters over it.

Made that fix and ran pyflakes, apart from superfluous imports there were a few issues that were introduced in a previous PR I think.

Ok..the tests are passing. Merging!


SHACL: dash:hasValueIn duplicates the sh:focusNode value for sh:value in constraint validation reports

When using the new extended dash:hasValueIn constraint component, validation reports duplicate the sh:focusNode value in the sh:value field.
Example SHACL shape:
@prefix sh: <http://www.w3.org/ns/shacl#> .
@prefix dash: <http://datashapes.org/dash#> .
@prefix rsx: <http://rdf4j.org/shacl-extensions#> .
@prefix vocsh: <http://example.org/shape/> .
@prefix voc: <https://shacl.co/vocabulary/> .
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .

vocsh:SimpleObject
	a sh:NodeShape ;
	rsx:targetShape [ a sh:Shape ; sh:property[ sh:path rdf:type ; sh:hasValue voc:SimpleObject ;  ] ];
;
	sh:property [
		sh:path voc:valIn ;
		sh:maxCount 1 ;
		dash:hasValueIn ("blue" "green" "pink") ;
		sh:datatype xsd:string ;
	] .

Example SPARQL:
base <https://shacl.co/resource/>
prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
prefix voc: <https://shacl.co/vocabulary/>
INSERT {
    <https://shacl.co/vocabulary/123> voc:valIn "pin";
           rdf:type <https://shacl.co/vocabulary/SimpleObject>.
}
WHERE {}

The SPARQL produces the following report:
_:node1efmbis8tx33004 a sh:ValidationResult;
  sh:focusNode <https://shacl.co/vocabulary/123>;
  sh:resultPath <https://shacl.co/vocabulary/valIn>;
  sh:sourceConstraintComponent <http://datashapes.org/dash#HasValueInConstraintComponent>;
  sh:sourceShape _:node125;
  sh:value <https://shacl.co/vocabulary/123> .

Expected:
_:node1efmbis8tx33004 a sh:ValidationResult;
  sh:focusNode <https://shacl.co/vocabulary/123>;
  sh:resultPath <https://shacl.co/vocabulary/valIn>;
  sh:sourceConstraintComponent <http://datashapes.org/dash#HasValueInConstraintComponent>;
  sh:sourceShape _:node125;
  sh:value "pin" .

This was observed while running RDF4J 3.3.0

After reading the spec a bit closer I think the correct approach is to omit sh:value for sh:hasValue and for dash:hasValueIn.
If there should be a value for sh:value then it should be mentioned in the spec:
https://www.w3.org/TR/shacl/#HasValueConstraintComponent
I really wish that the shacl document could have a nice little table showing which contraint components can be used together and which produce a sh:value.


Cu 2u55f5t eosdac fix remove candidate action

fix unsigned integer wraparound below zero cause by bug that allowed candidates to be remove more than once

Task linked: CU-2u55f5t eosdac: Fix removeCandidate action


CIF-1425 - The code handling search filters should be covered by unit…

… tests

extended unit tests

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes and the overall coverage did not decrease.
[x] All unit tests pass on CircleCi.
[x] I ran all tests locally and they pass.

Adds code coverage for https://github.com/adobe/aem-core-cif-components/pull/287


Set ComputeCount: 0 if no baremetalset created

The DefaultCount for the Compute is 1, set ComputeCount to 0 in
the rendered deploy environment file if no Compute baremetalset
got created that we can deploy a controlplane only overcloud.

/lgtm


IDFA dialog not working on real devices

Hi all, the prompt is working fine on simulator, however it doesn't shown on real device.
I tried to submit to App Store Connect and I got rejected with following message:
'We're looking forward to completing the review of your app, but we need more information to continue. Specifically, we noticed that your app uses the AppTrackingTransparency framework, but we haven't been able to locate the App Tracking Transparency permission request.'
No error shown on xcode console, any Idea on how to fix this? Thanks!

Found a discussion on apple developer forum, seems related to iOS14.3.. ( mine is too)
[https://developer.apple.com/forums/thread/671623]


Cancel filter button doesn't hide when value is ' '

missing check string empty
colFilter.term !== undefined && colFilter.term != null

should be
colFilter.term !== undefined && colFilter.term != null && colFilter.term !== ''

Not sure what issue this creates. I'm assuming you mean the select dropdown and not the string filter.
It seems to work fine here http://ui-grid.info/docs/#/tutorial/103_filtering

@JLLeitschuh I'm using UI-grid with angular 1.4.0

Ui-Grid is currently not working with angular 1.4 #3658
We plan to support it soon.

It was fixed  in v3.0.0-rc.22.
https://github.com/angular-ui/ng-grid/commit/22e6acd81ece42f1133712199edc0142df920ed1


Ensure ca certs are up to date

Otherwise Letsencrypt (and other new CAs) certs do not validate.

Thanks :)


Settings Metro App is opening as Control Panel in Windows 10

Describe the bug
When opening Settings (either through the Programs menu or the Desktop Search), instead of the expected Metro App, you see the older-style Control Panel. Its shortcut is displayed as the Metro App shortcut.
My Hypothesis is that the Settings App shortcut has been incorrectly mapped to Control Panel and is using the wrong icon,even though Control Panel itself is already mapped.
TL, DR: What you expect to be the settings app based on the shortcut's name and icon is actually just regular Control Panel.
Steps to reproduce

Apply Cairo settings as per screenshots and restart (pay particular attention to "Enable Desktop" being turned on).
Try to open Settings via the Programs menu or the Desktop search
Both appear as Control Panel.

Expected behavior
I expect that the correct operation would be:

Settings shortcut > Settings App
Control Panel Shortcut > Control panel

System information:

Windows version: Windows 10 Pro, 1909
Cairo version: 0.3.6842.0 - Pre-release
32-bit or 64-bit: 64
In Cairo Settings > Advanced, is Cairo set as shell?: Yes.

Additional context
I have a dual-monitor setup.
Screenshots
Settings:

Results:

This is occurring due to #365, so closing this as a duplicate.


Fix matplolib.use('WXAgg') warnings/errors

This small pull request is only to fix annoying warnings one gets if import larch is done within Ipython shell started as ipython --gui=qt5 (in view of more integration with SILX).

@maurov  That seems OK to me.   The initialization of the matplotlib backend is pretty annoying and we'd like to be able to support multiple GUI backends.


Gmail timed out connection error and No results found for known gmail, yahoo and other email providers

I am using pycharm to run the script but it doesn't work.
1st issue:
Known email addresses are not found. e.g.jsmith and johnsmith bring back zero results to almost all providers (but for sure not gmail and yahoo) which is impossible.
Sometimes running the script gives me results to these usernames for only some of the email providers (not gmail and yahoo), but then running it again doesn't show again these results.
2nd issue:
Gmail, Yandex and mailDe always throw connection errors or timed out connection. This happen even if I use VPN services to change my IP. I have also tried changing timeout= parameter to 20,30,60 in the "server = aiosmtplib.SMTP(timeout=10, validate_certs=False)" line of code250 function but the issue persists. I get various messages, different each time like:
a) Error while checking gmail: Timed out connecting to alt3.gmail-smtp-in.l.google.com. on port 25
b) Error while checking gmail: 421 Cannot connect to SMTP server <IP_ADDRESS> (<IP_ADDRESS>:25), connect error 10060
c) Error while checking gmail: Timed out waiting for server ready message
I checked powershell with "Test-NetConnection alt4.gmail-smtp-in.l.google.com -Port 25" and connection was successful as reply message was "TcpTestSucceeded : True"
My pycharm environment details are:
PyCharm 2023.1 (Community Edition)
Runtime version: 17.0.6+10-b829.5 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Python version: 3.9.1
pip freeze gives me the following:
aiohttp==3.8.4
aiosignal==1.3.1
aiosmtplib==2.0.1
appdirs==1.4.4
async-timeout==4.0.2
attrs==22.2.0
beautifulsoup4==4.12.0
bs4==0.0.1
certifi==2022.12.7
charset-normalizer==3.1.0
colorama==0.4.6
cssselect==1.2.0
dnspython==2.3.0
fake-useragent==1.1.3
frozenlist==1.3.3
idna==3.4
importlib-metadata==6.1.0
importlib-resources==5.12.0
lxml==4.9.2
multidict==6.0.4
parse==1.19.0
pyee==8.2.2
pyppeteer==1.0.2
pyquery==2.0.0
requests==2.28.2
requests-html==0.10.0
sd-aiosmtplib==0.0.2
soupsieve==2.4
tqdm==4.65.0
urllib3==1.26.15
w3lib==2.1.1
websockets==10.4
yarl==1.8.2
zipp==3.15.0
The results are after the change from "server = aiosmtplib.esmtp.ESMTP(timeout=10)", which threw error: "Error while checking gmail: module ‘aiosmtplib.esmtp’ has no attribute ‘ESMTP’" to server = aiosmtplib.SMTP(timeout=10, validate_certs=False)

Thank you for the issue. I tried to reproduce with your versions of libs but had no luck.
I added a new option timeout, try to increase it and reproduce it, let's check.
Also, I have some thoughts about paralleling checks, cause now they are running simultaneously, and it could be a reason for handshake timeout (-p mailde -p yandex -p gmail running successfully).
Maybe I'll add something like a parallel executor with a limited queue as in Maigret, I should experiment a little bit to understand will it be helpful.

@WhiteHatInspector, please, check mailcat version from this branch: https://github.com/sharsil/mailcat/tree/async-refactor

Finally fixed in #43.


Bizarre enhacement

if you host it at a VPS
can you mirror it (hidden service)
you just need
1 download tor (at VPS)
2 add these lines to /etc/tor/torrc
HiddenServiceDir /var/lib/tor/hidden_service/
HiddenServicePort 80 <IP_ADDRESS>:80
3 restart tor "sysstemctl restart tor"
4 restart apache "systemctl restart httpd"
5 copy /var/lib/tor/hidden_service/hostname
6 publick this (its your link)

Interesting


Click element to keep its tooltip on the screen, click away to dismiss

Demo at https://wbazant.github.io/react-tooltip-demo/
In our project we want the user to:

Hover over a table column, show tooltip
Click column, keep the tooltip on the screen
Access link inside tooltip or
Click outside tooltip to dismiss

Don't merge this PR as it is because I probably didn't think about everything, also it's not very elegant because half the implementation is outside (see demo code - keeping the state of frozen outside, toggle frozen when clicking the component, and on click outside tooltip also toggle frozen).
This solution will work for our project and I want to show it to you, and maybe inspire you to come up with a better way to add this functionality :)

That's pretty useful I think. I was planning to add a feature so that user can move mouse to the tooltip itself(current solution is awful).
And yeah, clicking to keep the tooltip is also a good idea, I can try to implement this later, let's keep this PR open for the moment, thanks for your explain and demo : )

Can someone try to correct these conflicts? @wbazant @alfonsomunozpomer

@thedaviddias thank you for your interest in this PR! :) I no longer use the component and don't want to commit time to test the change, rerun a build, and in general do a proper job.
Feel free to hm, fix the change and PR against my branch, I will merge, and merge here. Or squash my commits, add the change yourself, and tell me to close the PR.

I agree these are better, thank you for reviewing! If you have push access
to the branch do change it and merge. I only have my small internet device
for the next month or so, and no longer depend on the component.
W dniu niedziela, 13 grudnia 2020 Reinaldo Simoes<EMAIL_ADDRESS>napisał(a):

@reinaldosimoes commented on this pull request.
In src/index.js
https://github.com/wwayne/react-tooltip/pull/215#discussion_r542150330:

 originTooltip: originTooltip,

 isMultiline: isMultiline,

Just a small tweak, everything else looks good
⬇ Suggested change

 originTooltip: originTooltip,

 isMultiline: isMultiline,

 originTooltip,

 isMultiline,

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wwayne/react-tooltip/pull/215#pullrequestreview-551084302,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEBQVAGO5VSGFDIXL2IURGTSUWZONANCNFSM4CTVAMQQ
.

Hi guys, can someone take a look on that PR, please?

FYI all, I'm gonna unsubscribe myself from this thread now. Leaving it to @danielbarion to take care of any development that needs to happen to get this feature in

Closing due to V5


mui-datatables v3.2.0

Branch for the 3.2.0 release of mui-datatables. It contains the following:

Added setFilterChipProps option - An option to set props on the filter chips (https://github.com/gregnb/mui-datatables/pull/1387)
Refactored out unsafe_componentWillMount - Should now allow the table to run in strict mode without warnings (https://github.com/gregnb/mui-datatables/pull/1388, https://github.com/gregnb/mui-datatables/issues/1261)
Added jumpToPage option - This option, if true, adds a "Jump to Page" dropdown to the table footer that allows users to navigate to a specific page. (https://github.com/gregnb/mui-datatables/pull/1398)
Added TableViewCol and Checkbox to components property. This allows custom versions to be inputted. For example, using a checkbox with a different color. The large-data-set example has been updated to show custom versions of these components in action. (https://github.com/gregnb/mui-datatables/pull/1399, https://github.com/gregnb/mui-datatables/issues/1250#issuecomment-655604028, https://github.com/gregnb/mui-datatables/issues/1255, https://github.com/gregnb/mui-datatables/issues/1249)
Several components refactored into hooks (https://github.com/gregnb/mui-datatables/pull/1351, https://github.com/gregnb/mui-datatables/pull/1349, https://github.com/gregnb/mui-datatables/pull/1347)

Currently a work in progress, will probably be released next week.

Coverage increased (+1.0%) to 72.104% when pulling 7192a9389d7b531c88f35f4f573a9a19ff90764b on v3_2_0 into efd5f1e228d993e9341b3fd9092613140a0c77f6 on master.


Conversions between different genome versions

As we could see from the data hub, mm9 has much more epi-data than mm10. Isn't it possible to automatically convert mm9 to mm10 in datahub(e.g., using liftover)?

For now, there are several ways of changing references:

Re-align everything in the new reference.
Using liftOver on the raw reads and generate bigWig files again. Quality may not be as good and I'm not very sure if using liftOver for all reads has less computational burden than the previous one.

On the other hand, ENCODE is in the process of transiting from old reference genomes into the new ones (namely hg19 to hg38, and mm9 to mm10). We are discussing about how to synchronize with them (maybe in several stages or in one single batch once they are done).


Performance(?): Compatibility with other mods appears to register items even if said mods are not present.

Custom modpack (doubt it's relevant). Forgero 0.10.2. minecraft 1.19.2.
My server log displays those massages even though mythicmetals and wired redstone are not present:
[03:08:17] [main/INFO]: Registered wired-redstone-materials
[03:08:17] [main/INFO]: Registered mythicmetals-materials
Effect on load times is probably minimal, but bundled with other optimization issues it might become relevant.

It's actually only registering the pack as present, but it should also say that the packs are disabled cause the mods are not present. It never actually processes the resources into items. Although I should probably just remove all mentions of disabled packs, and keep it at a Debug log level.
As for the performance, I recently switched to async loading of the forgero packs in the Forge branch, so when it gets merged, all pack loading will be done asynchronously and the resource loading process will only be restricted by the pack that takes the longest to load, as opposed to loading each pack in sequence.
Still I believe I could do some more optimization on the disabled packs to reduce load times.


Addressing issue #30 about NIO FileSystem lifecycle

Issue #, if available:
#30
Description of changes:
As per the discussion on #30 we want to implement NIO FileSystem semantic, which is lead by the methods newFileSystem() and getFileSystem(); the former shall return a new instance or throw an exception, while the latter should return an existing instance. This means that S3FileSystem instances shall be cached (as opposed to the current implementation that caches AWSClients).
Another change introduced and discussed in other thread is the use of S3ClientProvider at S3FileSystem level in order to better decouple file system logic from the aws client creation logic (discussed in #50). This obsoletes IMHO the methods in S3FileSystemProvider that receives a AWSAsyncClient, which I would remove in future version to keep S3FileSystemProvider cleaner and simpler.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
PS: as often gradle complains, this time about javadoc I think :( it builds locally with both gradle and maven ...

Hi @markjschreiber, do you mind looking at this one? It's probably the last one with remarkable architectural changes to the current codebase. After that we should be moving smoothly with the S3X provider.

thanks! I am going to work on it in the next days

@markjschreiber I should be done with your comments, thanks a lot for your detailed review. I left open just one comment that was not clear to me. Let me know if I can do anything else.

Hi, thanks for this update. I will be out of the office for the next week and with very limited connectivity. I will attempt to provide reviews as I am able.

Hi, thanks for this update. I will be out of the office for the next week and with very limited connectivity. I will attempt to provide reviews as I am able.

have fun!

Hi, thanks for this update. I will be out of the office for the next week and with very limited connectivity. I will attempt to provide reviews as I am able.

have fun!

Hey @markjschreiber , I hope you had a good time! can we resume this work? The PR should be ready to be merged IMHO, but I am indeed available for updates.

I should have done with everything. let me know.

sorry, I closed it by mistake...

Re-opening

I just did some quick checks with the applications in examples. None of them currently work so something has broken in this PR for (at least) S3 native buckets/ URIs.

uh... unexpected : ) can you give me a bucket I can try with?

do you get the NPE: Exception in thread "main" java.lang.NullPointerException: Cannot invoke "software.amazon.nio.spi.s3.S3ClientProvider.generateAsyncClient(String) ?

yes, there is a bug here I believe. I am working on it

Yes, I get a null pointer exception for main
The other two example apps give "no such filesystem" errors.  For testing purposes any of the buckets listed in the "registry of open data" are usually OK. I tend to use s3://broad-references/ or some text file therein.

ok, I fixed the issue. I introduced couple of regressions not captured by unit tests. I tested it with sample applications and it works now.

ok, I fixed the issue. I introduced couple of regressions not captured by unit tests. I tested it with sample applications and it works now.

Nice. If you haven't already can you open an issue(s) that outline the code parts/ scenarios where we need to add unit tests to spot similar issues in the future? It would be better to rely on automated testing rather than manually running the example apps.

ok, I fixed the issue. I introduced couple of regressions not captured by unit tests. I tested it with sample applications and it works now.

Nice. If you haven't already can you open an issue(s) that outline the code parts/ scenarios where we need to add unit tests to spot similar issues in the future? It would be better to rely on automated testing rather than manually running the example apps.

#165


fix multicaching when result in not cacheable

Closes #105
If you isolate the new unit test and run it against current master, you'll see the test fail via timeout.

Coverage increased (+0.008%) to 99.278% when pulling 0d93b9321c7897b36d7f1cbdca07e6d5f06f0c1e on gswalden:fix-multicache-issue into 053296d8481224a8a73413e28d67dd415e275816 on BryanDonovan:master.

Coverage increased (+0.008%) to 99.278% when pulling 95dfd810b8a378855e6a9a4ad0e9b459cf04a8d1 on gswalden:fix-multicache-issue into 053296d8481224a8a73413e28d67dd415e275816 on BryanDonovan:master.

Looks like the following test still breaks, closing till fixed.
it("allows repeated calls for uncacheable value at same time", function (done) {
    function getUndefined(name, cb) {
        multiCache.wrap(key, function(cacheCb) {
            process.nextTick(function() {
                cacheCb(null, undefined);
            });
        }, cb);
    }

    var i = 0;
    function cb() {
        if (++i === 2) done();
    }

    var name = 'oof';

    getUndefined(name, function(err, val) {
        checkErr(err);
        assert.ok(val === undefined);
        cb();
    });
    getUndefined(name, function(err, val) {
        checkErr(err);
        assert.ok(val === undefined);
        cb();
    });
});

Okay, simpler code changes and issue fixed.

Coverage remained the same at 99.27% when pulling a7180b5592746330e45fdcbb52532fd2f8332304 on gswalden:fix-multicache-issue into 053296d8481224a8a73413e28d67dd415e275816 on BryanDonovan:master.

Just noticed that exact same fix I made two years ago never made it into the release although it is marked as merged: https://github.com/BryanDonovan/node-cache-manager/pull/48

@gswalden - sorry, I didn't notice your bug or fix.  I was out of town that weekend and I guess I skimmed over my emails too quickly.  I'll take a look ASAP.
@theorm - I think that fix was in the lib/caching.js file, not in the muliticaching.js file. (https://github.com/BryanDonovan/node-cache-manager/pull/48/commits/e9c4f0af2a7e1f14eeb91691a8dcbb757f03f662).

This is fixed in 2.6.0
CC @yoavbaruch, @rubenstolk


value of providers.kubernetesingress.ingressendpoint.publishedservice not correct when overriding namespace

Welcome!

[x] Yes, I've searched similar issues on GitHub and didn't find any.
[x] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What version of the Traefik's Helm Chart are you using?
7.7.11
What version of Traefik are you using?
v3.2.2
What did you do?
While configuring a local environment, I created a monolithic helm chart that imports and configures dependencies where traefik is one.
I set the namespaceOverride config to traefik. All resources deployed to the traefik namespace except for value of the --providers.kubernetesingress.ingressendpoint.publishedservice arg where it was default/local-traefik where default is the helm installation namespace and not the one for traefik.
What did you see instead?
I expected for the value of --providers.kubernetesingress.ingressendpoint.publishedservice to have the namespace I specified traefik/local-traefik
What is your environment & configuration?
I ran a Minikube and deployed helm chart with the following:
# file: Chart.yaml

#...

dependencies:
  - name: argo-cd
    repository: https://argoproj.github.io/argo-helm
    version: 7.7.11
    alias: argo-cd
    condition: argo-cd.deploy
#...

# file: Values.yaml
#...

traefik:
  namespaceOverride: traefik
  deploy: true
  ports:
    traefik:
      expose:
        default: true
  service:
    type: NodePort

#...

Then I ran: helm install local . -n default
Additional Information
I believe the issue is due to line 102 in `_helpers.tpl` where instead of being:

{{- $defServiceName := printf "%s/%s" .Release.Namespace (include "traefik.fullname" .) -}}

it should be:

{{- $defServiceName := printf "%s/%s" (include "traefik.namespace" .) (include "traefik.fullname" .) -}}

Thanks for this detailed bug report.
It was quick to open a PR.


Fixed a small typo in Lesson02-GitHub-Repo-Search/T02.01-Exercise-Cre…

…ateLayout MainActivity

Thanks! Pull requests don't really work in this repo because of some Git magic we're doing, but I've made the fix here: https://github.com/udacity/ud851-Exercises/commit/b86923c9da54358ca7c3c144e9524808860c7cba


Test framework

Test framwork for SRP, includes some testing for low end.
See document on how to use here:
https://docs.google.com/a/unity3d.com/document/d/18elRgdQF1w0b5mnzZ_S-OLADBl1oZ1YLXEe5cFu6sm0/edit?usp=sharing

Nice work. Easy to use :). Will convert all our test to it next week.


Questions: Material transition animations in Pre-Lollipop versions (Min-API level14)??

Which all  new material transition animations are possible in fragment transition  for API-level 14 to latest.
@lgvalle

Most of these animations can't be run on pre-lollipop devices, sorry.
See #7 for more info


Fix sysconfig template when public_ip is defined

No need to double quote the IP (esp. since the generated string will be part
of an other double quoted string). Also, prevent a line break. Without those
changes, we end up generating something like that:
FLANNEL_OPTIONS=" -alsologtostderr=true -ip-masq=false -public-ip="<IP_ADDRESS>"
 -subnet-dir=/run/flannel/networks -subnet-file=/run/flannel/subnet.env"

which prevent flanneld from starting.

Thank you @bpineau


Cannot import the dashboards

Hi,
Thanks a lot for this great repository :)
I have added the data source into Grafana, it seems to work. Prometheus and cadvisor seem to be working as well.
However, I am having a hard time importing the graphs. There are not imported directly. And if I try to import them by copying the JSON into the GUI, I get an empty dashboard (not even empty graphs, just empty page, no panels).
Is there something specific to do to import the dashboards?
Thanks

I found how to import them manually :)
Just need to change the first line and the last line of the JSON file :)

I found how to import them manually :)
Just need to change the first line and the last line of the JSON file :)

What did you need to change on the first and last line @wichtounet ?

I just figured it out. Remove the {"dashboard": and the last bracket  }


MAINT: Release 0.18

I am hoping that we can push out a release this week. I know it's a bit short notice, but we are already overdue for our ~6 mo. schedule, and there aren't that many issues/PRs marked for 0.18 anyway. I've gone through and marked issues/PRs that seem doable this week / appropriate for 0.18:
https://github.com/mne-tools/mne-python/milestone/14
Everyone feel free to look at the list of open issues/PRs and let me know if there are others you think should be included. Once there are no remaining issues in the list, I'll push out a release.

Can I volunteer for the releasing process described in https://github.com/mne-tools/mne-python/wiki/How-to-make-a-release (@sik promised to help me)

That would be great

We'll be releasing tomorrow morning (Paris time). cc: @GuillaumeFavelier

Actually I have a pretty busy schedule today so I won't be available much but I cleared my weekend, is it still okay for you?

Sure I've no trouble we'll find a nice coffee shop and spent the Saturday there. RELEASING MNE :D
Is everybody ok with it, or shall we do it today? @agramfort

It might be better to do it today

then either you or I can do it

let's see what the CI in master says

Yeah once https://travis-ci.org/mne-tools/mne-python/builds/533853924 and https://circleci.com/gh/mne-tools/mne-python/13179 come back happy, feel free to push out 0.18

I'm available to work on it now, what do you say @massich ?

Just pushed https://github.com/mne-tools/mne-python/commit/3b75010ffdffabf1146f7ca7700e51f93bba3ac6 to fix https://circleci.com/gh/mne-tools/mne-python/13179, now we need to wait 1.5 hours for https://circleci.com/gh/mne-tools/mne-python/13181 to complete

https://circleci.com/gh/mne-tools/mne-python/13186 is going, but it's probably better to wait until #6335 builds fully, gets merged, then builds fully on master.
@GuillaumeFavelier you might end up getting your wish to work on a Saturday to do the release :)

I'm going through the authors list and having troubles with my laptop so tomorrow would be perfect :D

@jeythekey we don't have your name to update the mailing for the release. Can you provide us your full name and desired email to ament that? thx.

here's the mne v0.17 release note

RFC for 0.18 release note is in #6338

We can't release without #6340

I do not understand why stable and dev websites render like this:

7ae4626 should fix it. I thought I had pushed that already in maint/0.18

now it should be good! @larsoner you can upload martions.

This version is live on PyPi!

for completness:
/tmp
(release-mne-018) ❯ pip search mne
mne (0.18.0)              - MNE python project for MEG and EEG data analysis.
mne-bids (0.2)            - Experimental code for BIDS using MNE.
mne-hcp (0.1.dev12)       - MNE HCP project for accessing the human connectome MEG data in Python.
mnespikes (0.1a)          - A simple package for representing neural spike data and converting to MNE
okiti-cnt-to-cnt (0.0.3)  - Create mne readable cnt files from OKITI created ones
fieldtrip2mne (0.0.7)     - Convert MEG and EEG brain scan data from FieldTrip toolbox in Matlab to
                            MNE toolbox in python.

/tmp
(release-mne-018) ❯ pip install mne
Collecting mne
  Downloading https://files.pythonhosted.org/packages/c0/96/a968294f3f21a7e028869bfd8e78f5c5a812232c6ab84091ec2ce578e68b/mne-0.18.0.tar.gz (6.3MB)
     |████████████████████████████████| 6.3MB 2.0MB/s
Collecting numpy>=1.11.3 (from mne)
  Using cached https://files.pythonhosted.org/packages/bb/76/24e9f32c78e6f6fb26cf2596b428f393bf015b63459468119f282f70a7fd/numpy-1.16.3-cp37-cp37m-manylinux1_x86_64.whl
Collecting scipy>=0.17.1 (from mne)
  Using cached https://files.pythonhosted.org/packages/5d/bd/c0feba81fb60e231cf40fc8a322ed5873c90ef7711795508692b1481a4ae/scipy-1.3.0-cp37-cp37m-manylinux1_x86_64.whl
Building wheels for collected packages: mne
  Building wheel for mne (setup.py) ... done
  Stored in directory: /home/sik/.cache/pip/wheels/49/3b/23/b10a8d761c86821e19511afaba5e266f77c66b19af4c7cffb0
Successfully built mne
Installing collected packages: numpy, scipy, mne
Successfully installed mne-0.18.0 numpy-1.16.3 scipy-1.3.0

/tmp 34s
(release-mne-018) ❯ python -c "import mne; print(mne.__file__); mne.sys_info()"
/home/sik/miniconda3/envs/release-mne-018/lib/python3.7/site-packages/mne/__init__.py
Platform:      Linux-4.15.0-46-generic-x86_64-with-debian-buster-sid
Python:        3.7.3 | packaged by conda-forge | (default, Mar 27 2019, 23:01:00)  [GCC 7.3.0]
Executable:    /home/sik/miniconda3/envs/release-mne-018/bin/python
CPU:           x86_64: 4 cores
Memory:        Unavailable (requires "psutil" package)
mne:           0.18.0
numpy:         1.16.3 {blas=openblas, lapack=openblas}
scipy:         1.3.0
matplotlib:    Not found

sklearn:       Not found
nibabel:       Not found
mayavi:        Not found
cupy:          Not found
pandas:        Not found
dipy:          Not found

The version 0.18 is live on PyPi!

We can't release without #6340

I believe we would like to get this #6340 merged in before the release is out. does pypi allow for re-releases or was this a test run?

@teonbrooks do you mean #6342 ?
The idea was to release a release candidate to wait for #6342 . But we messed it up. But don't worry, we can do another one quick and it will be the same.

ah, ok, gotcha! cool, thanks for the clarification :)

Thx everyone !!!


Code coverage for views.py

Describe the bug
The coverage of the file views.py is low 60%.
To Reproduce
Steps to reproduce the behavior:

coverage run manage.py test mainapp
coverage report

Expected behavior
Code coverage for the corresponding file should be atleast 90%
Screenshots

Additional context
To increase the coverage corresponding tests have to be written in tests.py.
To see the code which has not been covered by tests after running coverage report run coverage annotate and open the corresponding coverage file. For more details check out the docs.
If you are new to django testing check out this.
Definition of Done
Code coverage for views.py is atleast 90%

Can I work on this issue ?

Go for it @micksharma

Hey , I am participant in GSSOC'21. Can you please assign this to me?

@rk05-lenavo Checkout the starter guide

@Praful932 could you  please assign me this.

@micksharma Could you open a draft PR with your present progress, so we know you're going in the right direction

@micksharma no updates from your side.

Hey @maneprajakta . Can you please assign this issue to me. I am a GSSOC'21 participant

@Nidhi-2000 One issue at a time assigned you here #36.

Is anyone working on it? If not then please assign me

@hasanmdarif Go ahead :)
Do checkout the guidelines

@hasanmdarif Go ahead :)
Do checkout the guidelines

Thanks @Praful932

@hasanmdarif any updates?

@Praful932 @maneprajakta I am willing to take this up if it is open :)

I am willing to take this up if it's still open :)

@noviicee okays go ahead.

@noviicee were you able to figure it out?

@noviicee any updates?

@noviicee any updates?

Unable to increase the coverage beyond 70%  :\

@noviicee We can help out, which functions are you facing issues for writing tests

@Praful932 @maneprajakta can i work on this please

@Praful932 @maneprajakta can i work on this please

Still working on it :)

Not prioritized right now


brownfield: Adding more brownfield tests

This PR adds more tests to pkg/brownfield
Redirects definitely needs more work. Would like to merge the fixtures PR (https://github.com/Azure/application-gateway-kubernetes-ingress/pull/476) to avoid merge conflicts (or semantic merge conflicts)

I think redirect tests can be improved.


error 400 undefined

when i try to configure the addon with the local authorization token, when i hit apply i got an error 400 undefined and can'T get access to things
edit:
i got this log:
2020-07-21 09:31:36.263 INFO   : followers: setting persistence file path failed, will try older method.
2020-07-21 09:31:36.265 INFO   : followers: Current working directory: /home/pi/mozilla-iot/gateway
2020-07-21 09:31:36.267 INFO   : followers: Could not load persistent data (if you just installed the add-on then this is normal)
2020-07-21 09:31:36.270 INFO   : followers: self.persistent_data is now: {'items': []}
2020-07-21 09:31:36.272 INFO   : followers: no user profile data
2020-07-21 09:31:36.275 INFO   : followers: self.manager_proxy = <gateway_addon.addon_manager_proxy.AddonManagerProxy object at 0x75fc9af0>
2020-07-21 09:31:36.277 INFO   : followers: Created new API HANDLER: followers
2020-07-21 09:31:36.280 INFO   : followers: 0.12.0
2020-07-21 09:31:36.282 INFO   : followers: Starting the internal clock
2020-07-21 09:31:36.285 INFO   : followers: Followers shutting down
2020-07-21 09:31:38.275 ERROR  : Failed to restart add-on followers
2020-07-21 09:31:38.283 ERROR  : Error: Checksum failed for file /home/pi/.mozilla-iot/addons/followers/pkg/pycache/init.cpython-37.pyc in add-on followers
at find.fileSync.forEach (/home/pi/mozilla-iot/gateway/build/webpack:/src/addon-utils.js:215:1)
at Array.forEach ()
at loadManifestJson (/home/pi/mozilla-iot/gateway/build/webpack:/src/addon-utils.js:197:1)
at Object.loadManifest (/home/pi/mozilla-iot/gateway/build/webpack:/src/addon-utils.js:303:1)
at AddonManager.loadAddon (/home/pi/mozilla-iot/gateway/build/webpack:/src/addon-manager.js:614:1)
at ./src/controllers/addons_controller.js.AddonsController.put (/home/pi/mozilla-iot/gateway/build/webpack:/src/controllers/addons_controller.js:114:1)

Thank you! It has been fixed.

i update the addon, reboot my gateway, and still the same issue:
2020-07-21 14:09:42.059 INFO   : followers: -Authorization token is present in the config data.
2020-07-21 14:09:42.062 INFO   : followers: -Debugging preference was in config: True
2020-07-21 14:09:42.065 INFO   : followers: setting persistence file path failed, will try older method.
2020-07-21 14:09:42.068 INFO   : followers: Current working directory: /home/pi/mozilla-iot/gateway
2020-07-21 14:09:42.070 INFO   : followers: Could not load persistent data (if you just installed the add-on then this is normal)
2020-07-21 14:09:42.073 INFO   : followers: self.persistent_data is now: {'items': []}
2020-07-21 14:09:42.076 INFO   : followers: no user profile data
2020-07-21 14:09:42.079 INFO   : followers: self.manager_proxy = <gateway_addon.addon_manager_proxy.AddonManagerProxy object at 0x75ff3b50>
2020-07-21 14:09:42.082 INFO   : followers: Created new API HANDLER: followers
2020-07-21 14:09:42.089 INFO   : followers: 0.12.0
2020-07-21 14:09:42.092 INFO   : followers: Starting the internal clock
2020-07-21 14:09:42.095 INFO   : followers: Followers shutting down
2020-07-21 14:09:43.342 ERROR  : Failed to restart add-on followers
2020-07-21 14:09:43.348 ERROR  : Error: Checksum failed for file /home/pi/.mozilla-iot/addons/followers/pkg/pycache/init.cpython-37.pyc in add-on followers
at find.fileSync.forEach (/home/pi/mozilla-iot/gateway/build/webpack:/src/addon-utils.js:215:1)
at Array.forEach ()
at loadManifestJson (/home/pi/mozilla-iot/gateway/build/webpack:/src/addon-utils.js:197:1)
at Object.loadManifest (/home/pi/mozilla-iot/gateway/build/webpack:/src/addon-utils.js:303:1)
at AddonManager.loadAddon (/home/pi/mozilla-iot/gateway/build/webpack:/src/addon-manager.js:614:1)
at ./src/controllers/addons_controller.js.AddonsController.put (/home/pi/mozilla-iot/gateway/build/webpack:/src/controllers/addons_controller.js:114:1)

Could you try deleting it first? It could be that an update doesn't remove the old file that is causing trouble (don't know).

deleted the add-on
rebooted the add-on
reinstalled the add-on
still same issue
2020-07-22 09:19:20.964 INFO   : Fetching add-on https://s3-us-west-2.amazonaws.com/mozilla-gateway-addons/followers-0.1.1.tgz as /tmp/VMNAci/followers.tar.gz
2020-07-22 09:19:21.641 INFO   : Expanding add-on /tmp/VMNAci/followers.tar.gz
2020-07-22 09:19:25.383 INFO   : Loading add-on: followers
2020-07-22 09:19:38.219 INFO   : Unloading API handler followers
2020-07-22 09:19:40.645 INFO   : followers: -Authorization token is present in the config data.
2020-07-22 09:19:40.648 INFO   : followers: -Debugging preference was in config: True
2020-07-22 09:19:40.651 INFO   : followers: setting persistence file path failed, will try older method.
2020-07-22 09:19:40.654 INFO   : followers: Current working directory: /home/pi/mozilla-iot/gateway
2020-07-22 09:19:40.657 INFO   : followers: Could not load persistent data (if you just installed the add-on then this is normal)
2020-07-22 09:19:40.660 INFO   : followers: self.persistent_data is now: {'items': []}
2020-07-22 09:19:40.677 INFO   : followers: no user profile data
2020-07-22 09:19:40.680 INFO   : followers: self.manager_proxy = <gateway_addon.addon_manager_proxy.AddonManagerProxy object at 0x75f82b50>
2020-07-22 09:19:40.682 INFO   : followers: Created new API HANDLER: followers
2020-07-22 09:19:40.685 INFO   : followers: 0.12.0
2020-07-22 09:19:40.687 INFO   : followers: Starting the internal clock
2020-07-22 09:19:40.689 INFO   : followers: Followers shutting down
2020-07-22 09:19:41.786 ERROR  : Failed to restart add-on followers
2020-07-22 09:19:41.793 ERROR  : Error: Checksum failed for file /home/pi/.mozilla-iot/addons/followers/pkg/pycache/init.cpython-37.pyc in add-on followers
at find.fileSync.forEach (/home/pi/mozilla-iot/gateway/build/webpack:/src/addon-utils.js:215:1)
at Array.forEach ()
at loadManifestJson (/home/pi/mozilla-iot/gateway/build/webpack:/src/addon-utils.js:197:1)
at Object.loadManifest (/home/pi/mozilla-iot/gateway/build/webpack:/src/addon-utils.js:303:1)
at AddonManager.loadAddon (/home/pi/mozilla-iot/gateway/build/webpack:/src/addon-manager.js:614:1)
at ./src/controllers/addons_controller.js.AddonsController.put (/home/pi/mozilla-iot/gateway/build/webpack:/src/controllers/addons_controller.js:114:1)

Hmm. That's odd. It's fixed here.

What version of the gateway are you using?
Could you try and delete the file? You can install the Seashell addon and then issue this command with it:
sudo rm -rf /home/pi/.mozilla-iot/addons/followers/pkg/pycache
(it removes the cache directory. 'rm' stands for 'remove', and the 'sudo' forces it to happen).
Then use the Power Settings addon to reboot the pi.

still got the same issue!
Le mer. 22 juill. 2020, à 13 h 41, flatsiedatsie<EMAIL_ADDRESS>a écrit :

Hmm. That's odd. It's fixed here.

What version of the gateway are you using?
Could you try and delete the file? You can install the Seashell
addon and then issue this command with it:
sudo rm -rf /home/pi/.mozilla-iot/addons/followers/pkg/pycache
(it removes the cache directory. 'rm' stands for 'remove', and the
'sudo' forces it to happen).
Then use the Power Settings addon to reboot the pi.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flatsiedatsie/followers-addon/issues/1#issuecomment-662590857,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSPAAATGAKACNXQF5F3UPTR44QC3ANCNFSM4PDTKLAQ
.

I just released a potential fix.
Do you still have the issue on version 0.1.2?

Working thank you!
Le mer. 22 juill. 2020 17 h 18, flatsiedatsie<EMAIL_ADDRESS>a
écrit :

I just released a potential fix.
Do you still have the issue on version 0.1.2?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flatsiedatsie/followers-addon/issues/1#issuecomment-662702015,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSPAAFNI7MQVDVTUR2TWA3R45JTDANCNFSM4PDTKLAQ
.


Function: allow imported IConstructs in attachPermissions

Currently, a list of constructs are supported to be handed into attachPermissions: https://docs.serverless-stack.com/util/Permissions#supported-constructs
Mostly sst constructs, but a handful of CDK ones.  For the CDK ones, only the actual construct is supported, not one that is "found" via a from method, i.e. fromTableArn.  This returns an ITable rather than a Table, and the attachPermissions throws a TypeScript error: Type 'ITable' is missing the following properties from type 'Construct': onValidate, onPrepare, onSynthesize, validate, and 2 more.ts(2740).
Would it be possible to support I interfaces?  My thought is "yes", since they also have the grant methods on them.  Currently a policy statement must be created from scratch.
Thank you!
Edit: I just tried it using a @ts-expect-error and it seems to work.

This is a good point, we should be looking for the interfaces and not the class. Will fix this

Also requested by Sam https://serverless-stack.slack.com/archives/C01JG3B20RY/p1638828949084500

Hey @dkershner6, this is now supported in v0.53.4


Upgrade Proxmox Provider

What steps did you take and what happened:
clusterctl upgrade plan
Error: failed to get configuration for the  with name in-cluster. Please check the provider name and/or add configuration for new providers using the .clusterctl config file
What did you expect to happen:
I would get list of clusterapi and proxmox provider versions.
Environment:

Cluster-api-provider-proxmox version: 0.3.0
Kubernetes version: (use kubectl version): 1.28.7
OS (e.g. from /etc/os-release):  Ubuntu 22

## -- Controller settings -- ##
PROXMOX_URL: "https://<IP_ADDRESS>:8006"                       # The Proxmox VE host
PROXMOX_TOKEN: "capmox@pve!sdf"                                # The Proxmox VE TokenID for authentication
PROXMOX_SECRET: "eea27f1f-de47-4ea1-ae6e-dfa"                                    # The secret associated with the TokenID

## -- Required workload cluster default settings -- ##
PROXMOX_SOURCENODE: "pve08"                                     # The node that hosts the VM template to be used to provision VMs
TEMPLATE_VMID: "1000"                                          # The template VM ID used for cloning VMs
 ##ALLOWED_NODES: "[pve1,pve2,pve3, ...]"                        # The Proxmox VE nodes used for VM deployments
ALLOWED_NODES: "[pve08]"                        # The Proxmox VE nodes used for VM deployments
VM_SSH_KEYS: "ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABgQCvEUpmAskfY54n6sy1ps00JnTbq9CZZmdnKtHhDZYj1QLlsyyijZuQv6tRl9QZo7ew/x1yKVEc12QhF+p5TDCKhX95DekpVNY0Mhn2bdW1bom4Dqbts6Hpu1rjfl3+safasdf/96n/ORRmodkyTHdeG2IxAXooxOVT8U4EyJvj3wtg0YO    C5Hv7uhYKgPzZEKD5zGXspqI4C0/59ujVJuIy3Fl9g+vIjCn32GhDyAvag13vTWZDF/Wn//8FsACG+N9OuN/Dpo0SjqmjgpBFS3Hn1eicvK/xNa/8e2c4pEb2KCi+0x/W9KYJW3lTwaNbIZ8ejXqcieMzd62geN8YnczDGDx/QxlmeTkLhFQlQBmaaG51lQRtxYNDpDbESVDAoe45qpA38GeQsFj8p18XGpLFHp3U6QiJFziqxwLy2C6jBAwNN4BMCSU8kPIvLsIs="               # The ssh authorized keys used to ssh to the machines.

## -- networking configuration-- ##
CONTROL_PLANE_ENDPOINT_IP: "<IP_ADDRESS>"                       # The IP that kube-vip is going to use as a control plane endpoint
NODE_IP_RANGES: "[<IP_ADDRESS>-<IP_ADDRESS>]"               # The IP ranges for Cluster nodes
GATEWAY: "<IP_ADDRESS>"                                         # The gateway for the machines network-config.
IP_PREFIX: "24"                                               # Subnet Mask in CIDR notation for your node IP ranges
DNS_SERVERS: "[<IP_ADDRESS>,<IP_ADDRESS>]"                              # The dns nameservers for the machines network-config.
BRIDGE: "vmbr0"                                               # The network bridge device for Proxmox VE VMs

## -- xl nodes -- ##
BOOT_VOLUME_DEVICE: "scsi0"                                   # The device used for the boot disk.
BOOT_VOLUME_SIZE: "100"                                       # The size of the boot disk in GB.
NUM_SOCKETS: "2"                                              # The number of sockets for the VMs.
NUM_CORES: "4"                                                # The number of cores for the VMs.
MEMORY_MIB: "8048"                                            # The memory size for the VMs.

EXP_CLUSTER_RESOURCE_SET: "true"                              # This enables the ClusterResourceSet feature that we are using to deploy CNI
providers:
  - name: in-cluster
    url: https://github.com/kubernetes-sigs/cluster-api-ipam-provider-in-cluster/releases/latest/ipam-components.yaml
###   url: https://github.com/kubernetes-sigs/cluster-api-ipam-provider-in-cluster/releases/v0.1.0/ipam-components.yaml
    type: IPAMProvider

Nevermind, I just had to upgrade clusterctl to the latest version.


Change CUDA version check for cudaSync from 10000 to 10010

When upgrading to CUDA 10.1, I've started to have the same problem fixed in #1514.
This is because the CUDA_VERSION wasn't change in gpu_data.cpp file.
This pull request simply upgrades version check from 10000 to 10010 (CUDA_VERSION in CUDA 10.1).

Sweet, thanks for the PR :)


Feature: UI - After saving a sniffer, minimize form fields

It will simplify the UI and will allow us to display other option.
Some ideas,:

a button to open the link in a new tab - opening the proxy is a recurring issue with first time users
a link to the service page - https://github.com/idodav/sharkio/issues/55

@idodav I'd like to take this issue :)

assigning this to you :) if you need any help feel free to ask

@hagarfisher Thank you very much, made us proud :)
I would love it if you could add a short video in this issue.
Showing a demo of what you implemented.

Thanks @idodav :) here's a short demo of the feature: