Decide how we'll implement the article feed

In the spec there's a feed endpoint made up of articles written by people you follow.
Problem: The people you follow is not necessarily a small number.
I see two ways to go here:

We assume there is an actual limit (say, 500) to the people you can follow, and we use the 500 ids in a SQL query.
We materialize a feed per user via listeners. New article published? Let's push it into the feeds of all users. I heard this is how facebook does it.
We take a hybrid / heuristic approach. If a user follows < LIMIT people, we calculate it. Otherwise we materialize it.

Since the purpose of this example is to show how we would do it in a real scenario, I would actually consider just (2) or (3), but let me know what you think!

I will go with (2). I do not see the benefits of (3) over (2), and it is a more complex solution.

Fair enough, go ahead ;)


Fix calendar background color

Fixes https://github.com/pyxelr/Dark_Google_Calendar/issues/4.
Removes a custom background-color property for the calendar background.
With the custom background-color:

Without the custom background-color (the change included in this PR):

Thanks for reporting and fixing this issue.
I have merged your pull request.


Row highlights on the control

I liked the idea of highlighting the deprecated controls but there are two type of highlights. One is gray and another is red. What is the difference between the two.
Can you add another column or have some UI where you can show the reason why those components are getting highlighted?

Deployment: Online
DB Version: <IP_ADDRESS>49
Connection Controls Version: 1.2021.1.39
XrmToolBox Version: 1.2020.11.42
Tool Version: <IP_ADDRESS>

The red indicates deprecated ones that are in use, while grey is deprecated but not in use. If you hover your mouse over the row the deprecation details should show as a tooltip.

The red indicates deprecated ones that are in use, while grey is deprecated but not in use. If you hover your mouse over the row the deprecation details should show as a tooltip.


Cognyte Luminar Initial Commit

Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

After making the above changes, please work on the following changes
We need to package the solution with version 3.0.0, please go through the readme file
https://github.com/Azure/Azure-Sentinel/tree/master/Tools/Create-Azure-Sentinel-Solution/V3
Also do add release notes, please follow this link -https://github.com/Azure/Azure-Sentinel/blob/master/Solutions/ReleaseNotesGuidance.md

Hi @RamboV,
We look forward to hearing from you once your updates are complete. Thanks

Hi @RamboV, would appreciate your update on this PR.

Hello @v-rbajaj , please review

Hi @RamboV, Thanks for making these changes, we will review your PR by 20 Sep 2023

After making the above changes, please work on the following changes
We need to package the solution with version 3.0.0, please go through the readme file https://github.com/Azure/Azure-Sentinel/tree/master/Tools/Create-Azure-Sentinel-Solution/V3
Also do add release notes, please follow this link -https://github.com/Azure/Azure-Sentinel/blob/master/Solutions/ReleaseNotesGuidance.md

Hi @RamboV, please add release notes as mentioned earlier also please fix the ARM TTK validation failure..


Feat/add phase 2

This PR implements the following features:

Refactored sidh, imbolc & beltaine components into PhaseCard component
Develops the phase type & introduces phase entity objects
adds phase 2 tasks
Refactored Task page & added time and timezone
addedtask query propertyto allow the opening and closing of task accordions on the task page

:tada: This PR is included in version 1.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Doesn't compile

I am likely missing something:
$ rustup install nightly
info: syncing channel updates for 'nightly-x86_64-unknown-linux-gnu'

  nightly-x86_64-unknown-linux-gnu unchanged - rustc 1.54.0-nightly (676ee1472 2021-05-06)

$ cargo +nightly install --path .
  Installing frawk v0.4.0 (/home/bo/work/frawk)
    Updating crates.io index
   Compiling frawk v0.4.0 (/home/bo/work/frawk)
error[E0554]: `#![feature]` may not be used on the stable release channel
 --> src/main.rs:2:35
  |
2 | #![cfg_attr(feature = "unstable", feature(core_intrinsics))]
  |                                   ^^^^^^^^^^^^^^^^^^^^^^^^

error[E0554]: `#![feature]` may not be used on the stable release channel
 --> src/main.rs:3:35
  |
3 | #![cfg_attr(feature = "unstable", feature(test))]
  |                                   ^^^^^^^^^^^^^

error[E0554]: `#![feature]` may not be used on the stable release channel
 --> src/main.rs:4:35
  |
4 | #![cfg_attr(feature = "unstable", feature(write_all_vectored))]
  |                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^

error: aborting due to 3 previous errors

For more information about this error, try `rustc --explain E0554`.
error: failed to compile `frawk v0.4.0 (/home/bo/work/frawk)`, intermediate artifacts can be found at `/home/bo/work/frawk/target`

Caused by:
  could not compile `frawk`

To learn more, run the command again with --verbose.

OK, plan B:
cargo install frawk

error[E0554]: `#![feature]` may not be used on the stable release channel
 --> /home/bo/.cargo/registry/src/github.com-1ecc6299db9ec823/frawk-0.4.0/src/main.rs:2:35
  |
2 | #![cfg_attr(feature = "unstable", feature(core_intrinsics))]
  |                                   ^^^^^^^^^^^^^^^^^^^^^^^^

error[E0554]: `#![feature]` may not be used on the stable release channel
 --> /home/bo/.cargo/registry/src/github.com-1ecc6299db9ec823/frawk-0.4.0/src/main.rs:3:35
  |
3 | #![cfg_attr(feature = "unstable", feature(test))]
  |                                   ^^^^^^^^^^^^^

error[E0554]: `#![feature]` may not be used on the stable release channel
 --> /home/bo/.cargo/registry/src/github.com-1ecc6299db9ec823/frawk-0.4.0/src/main.rs:4:35
  |
4 | #![cfg_attr(feature = "unstable", feature(write_all_vectored))]
  |                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^

error: aborting due to 3 previous errors

For more information about this error, try `rustc --explain E0554`.
error: failed to compile `frawk v0.4.0`, intermediate artifacts can be found at `/tmp/cargo-installuy6UkM`

Caused by:
  could not compile `frawk`

To learn more, run the command again with --verbose.

I guess, it is not finished yet...

Closing this  due to trouble reproducing this error message (and lack of activity over last few months). Please file a follow-up if you're still having issues though.


feat(task): Add API events for drawtask plugin

Add api.tasks.on.drawBegin, api.tasks.on.draw and
api.tasks.on.drawEnd events for the drawtask plugin.

I added this feature because in my project I need to remove tasks with zero length (tasks with equal from and to moments). At first I tried to listen to the resizeEnd event, but it turns out that that event is not fired when the mouse has moved exactly enough to pass the threshold.
I figured the drawtask plugin could use its own events just like the movable plugin.

Thank you ! I'll give it a try a soon as possible.

@Cerzus Thx for the PR. It works great.


Added a link in wiki.python.org

Hi. Just a quick note. I've added a link to your curated list in the Python.org wiki:
https://wiki.python.org/moin/Decorators
(Might copy a handful over later..)

👍


Using createGenerateClassName with AutoComplete causes style miss-match across dev-prod environments

AutoComplete styling uses CSS attribute selectors, per example: .MuiAutocomplete-inputRoot[class*="MuiOutlinedInput-root"] .MuiAutocomplete-endAdornment
If using createGenerateClassName with disableGlobal, this part [class*="MuiOutlinedInput-root"] will NOT be converted, resulting that this rule will look something like: .jss-1[class*="MuiOutlinedInput-root"] .jss-2 and as you can imagine, these styles will not be applied if using this function, resulting that development and production build will have different styles and CSS rules.
Example generateClassName:
const generateClassName = createGenerateClassName({
  productionPrefix: "jss",
  disableGlobal: true
});

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

Current Behavior 😯

The attribute selector parameter (*=MuiOutlinedInput-root) is not converted alongside the other class names, resulting in a miss-match between the rule and the element classes.
Expected Behavior 🤔

The value for the selector should also be converted to the respective class name that matches what is expected.
Steps to Reproduce 🕹

Development version: https://codesandbox.io/s/material-demo-9hfi2
Note: Inspect elements and check the style being applied to MuiAutocomplete-endAdornment
Production version: https://csb-9hfi2-5hq37dorc.now.sh/
Note: Inspect elements and check the style being applied to jss12 (equivalent to MuiAutocomplete-endAdornment).
You will note that the right padding is missing. If you edit the DOM content and add this class MuiAutocomplete-endAdornment to the element that has these classes  jss66 jss51 jss9 jss76 jss67 jss71 jss56 (div which is the parent of jss12 | MuiAutocomplete-endAdornment) you will notice that the padding will appear as now the rule .jss7.jss8 .jss9[class*="MuiOutlinedInput-root"] is being applied to the end adornment.
Steps:

Define generateClassName configuration and enable disableGlobal
Use AutoComplete component
Deploy to "Production"
Check Development and Production environments, the styles are different between them.

Context 🔦

It's impacting the production build of our app which is relying on Material UI, In the development environment it looks good but once we properly build it to release the styles get messed.
Also, it's good to mention that this issue is happening on all other places that have this same approach, this element is just one example but inside AutoComplete there are a lot of other rules using this, not sure if any other component also follows this, if so, probably it's happening there as well.
Your Environment 🌎

Tech
Version

Material-UI
v4.9.11

React
v16.3.1

Browser
Chrome 81.0.4044.113

What's your use case for disableGlobal?

I don't want to display any actual class name, only the generated codes, if I don't use it, the actual MUI classes will be shown.

@reenan Ok so you can stick to disableGlobal: false for now. We will likely prevent developer to easily disable the global class names in v5.

In this case, I won't be able to replace the default MUI class names with the generated ones?

Correct

OK, understood. Thank you for the clarification.
Is there any release date/period already planned/estimated for v5?

@reenan Can't tell could be stable in 2021.

One thing which confused me that I had to use serverGenerateClassName during ssr:
 const sheets = new ServerStyleSheets({
    serverGenerateClassName: generateClassName
  })

I am not sure if this is related to your problem but maybe worth a try.

@jantimon serverGenerateClassName is supposed to be private, we don't document it.

Okay good to know - for me it did not work without it for SSR.
As you can see it will create a new instance of generateClassName and will directly pass it to StylesProvider:
https://github.com/mui-org/material-ui/blob/89687f38cae750650555772ba4d821c9084d8dfc/packages/material-ui-styles/src/ServerStyleSheets/ServerStyleSheets.js#L15-L22

Here is a small demo:
https://codesandbox.io/s/youthful-davinci-1umic?file=/src/App.tsx
As you can see ServerStyleSheets({generateClassName: /* ... */}) will not work.
Is there any other api to use generateClassName during ssr?

This was fixed in v5. We removed the notion of createGenerateClassName and used hard coded global class names instead


Built artefacts have wrong names

looks like a copy-paste error somewhere in the build resulting in the github releases all being named "de-markiewb-netbeans-plugins-hints-x.y.z" instead of "de-markiewb-netbeans-plugins-copyfqn-x.y.z"

Thanks for noticing! I fixed the issue in the pom.xml and uploaded a renamed version.


[WIP] Salt AG-Grid theme supports v29+ ag grid

Closes https://github.com/jpmorganchase/salt-ds/issues/2602

replaced by https://github.com/jpmorganchase/salt-ds/pull/2661


Basic info of mosques can not be edited on the staging site.

Issue in words : While we try to edit basic info of mosques in staging site, after changing the info of mosque when the "Update mosque info" button  is clicked message appears " Mosque's basic info has been updated", but the update doesn't reflect as expected, it remains same as before.
Steps to reproduce the issue :
1: Logged in to staging site.
2. Clicked on Dashboard
3. Clicked on the hamburger bar.
4. Clicked on the manage mosque
5. Selected random mosque.
6. Clicked "Edit Basic Info".
7. Edited name of the mosque.
8. "Update mosque info" button appeared on the right bottom corner, clicked the button.
9. " Mosque's basic info has been updated" message appeared on the left bottom corner.
10. Clicked the "View mosque" button to verify the change, but the update doesn't reflect as expected, it remains same as before.

Clicked the "View mosque" button to verify the change, but the update doesn't reflect as expected, it remains same as before.

This could be due to caching. We cache two levels

5 mins in redis
up to 24 hours in CloudFront

So to verify, you can login to API, go to same mosque's page and see if the updated value is there.
I think we should communicate to user that changes will take a while to reflect on the site.

Test case Title:
Basic info on mosques can not be edited on the staging site.
Test Steps:

Navigate to https://staging-api.prayersconnect.com/mosques/5040429/edit.
** Please note mosque ID 5040429 is a test mosque.
Observe that the "Edit Mosque" page is displayed with fields for editing mosque details.
Modify the mosque name to "Test Mosque By Amranul on 03/28/2023"
Modify the Name in the Native Language to "Bangla".
Modify the Other Names to "Test name".
Modify the Contact email to "Test#test.com
Modify the Website to "test.com"
Modify the Phone to "1234567890"
Modify the Imam email to<EMAIL_ADDRESS>Modify Imam's phone to "+1234567890"
Observe that the changes are reflected in the appropriate fields.
Click on the "Save" button.
Observe that a message is displayed confirming that the changes have been saved.
Refresh the page.
Observe that the modified mosque details are retained.


Install on Windows without Docker?

Hi there, excellent looking application. I've been looking for something like this for a long time.
Currently, I am unable to install Docker on my computer, however I do have Sonarr, Radarr, etc etc running on here via their installers. Is there a way I can install this on my Windows machine without Docker?
I have Python, MongoDB, NodeJS, etc already installed, if that helps.

There's a windows .exe file on the Releases page. Combine that with a config file, and this should run just fine on Windows. Let me know if you have any specific questions I can help with!

Oh I see, is it titled amd64.exe.zip? I didn't recognize that as a windows release, most other GitHub releases I find are labeled as "Windows". I'll give that a try!

That is correct! It's compiled 64 bit for a 64 bit Windows OS. amd64 and x86_64 are the designators for that. Windows is just weird. :D

Relevant to this issue, would be great to have this implemented as a service/daemon like sonarr/radarr. Perhaps supporting windows services and also linux systemd etc. I'm running this on docker in Windows, previously I was using task scheduler to fire it off in the background.

I have no idea how to build an installer or setup a service in windows. :(

I have no idea how to build an installer or setup a service in windows. :(

Check out https://github.com/mh-cbon/go-msi for building msi packages, and there are examples for writing a windows service here
There is also https://github.com/judwhite/go-svc for writing services. I personally haven't done this, but I think that these options are pretty baseline for what people do.

Appreciate the links. I still want to look into this more, but haven't had time. Leaving this issue open so I don't lose this context.

Having trouble with how to install this on windows, took a look at the links but cant figure out...any help?

Copy the executable and the conf to C:\ProgramData\unpackerr and run it from there (to simplify your life)
Open the conf and edit it accordingly with your api keys, etc

what should be the extension of the conf file? or should I just name it CONF?

Thanks! I'll be trying it now

A few more questions and then I think it will be all cleared up:
Suppose both Sonarr and Radarr download to D:\sonarr_radarr_down, and, lets say, the Sonarr folder is D:\TV. The files downloaded via Sonarr (using Deluge) are being stored in D:\TV\show_name\Season_No\, under which each episode has a folder that contains the rared files.
I have set my Sonarr and Radarr download directories as D:\sonarr_radarr_down, but I also want to be able to delete the extracted files from the extracted folder after importing is finished but keep it in the Sonarr media library. Do I need to edit the bottom-most section of the conf file?

No, do not use the Folder feature for arr apps. The only thing that can be difficult to configure is the path, but on a single system (no Docker) it's much easier. Just run it, watch the log/output for a bit and see if it has the right paths in the messages. If not, quit it, fix the paths, and try again. It overall pretty easy. It will deleted the extracted files after sonarr/radarr import them.


Could apheleia work as indent-line-function?

I don't know if that is feasible, but apheleia, more than the other formatting packages (namely, reformatter and format-all) seems like the best contender for working with indent-line-function. It'd be great if we could that because not only are we working with the default Emacs system for formatting code while you type, but it would avoid some users to have to bind the formatting command to a key, instead relying on the TAB key for it (using electric-indent-mode, RET would also indent in most cases).
It might be utterly impractical, given that most formatters actually error out on unfinished forms. As a naive implementation, I tried defining, using reformatter, which gives me a cljstyle-format-region, for example, the following:
(defun cljstyle-format-line () (cljstyle-format-region (bol) (eol)))

Where bol and eol are just simple functions that return the point in the beginning and end of line, respectively. Setting that as the indent-line-function will pretty much always fail, since it will most like catch some unclosed paren and cljstyle uses the parsing functionality of Clojure's reader in its code.
Of course that would require the long apheleia-format-region (#11), which I am willing to take a stab (even if my proposition isn't possible), just for the sake of dropping reformatter for apheleia.

I don't have any objection to exploring something like this, although there would definitely be drawbacks (e.g. a lot of extra CPU load from running an external program every time electric indent triggers)! Probably won't implement it myself though.


Add first exercise

Wow nice! @inyono  @thellmund
I'll tweak with the styles and add the font.


[PE-D][Tester C] Not much use of visuals in UG

I really like that the user guide is pretty clear in the description of the commands, but there could be more use of images in the UG to make it easier to understand what is happening at a glance.

Labels: severity.VeryLow type.DocumentationBug
original: matochichap/ped#4

Add image for the more complicated commands only.


Feature Request: Add support for action post-processing (Delayed docker pause)

Currently, an activation is complete when a function returns a response (Or it's promise is fulfilled).
This architecture prevents doing any post-processing within the lifecycle of the action.
It would be great to be able to provide a response back to the action caller and be able to do some post processing before the activation is considered complete and the container is paused.
Example use cases:

An IoT device interacts with a function and needs a fast response. After the response is sent back to the device, the action needs to send data to an external data warehouse/Analytics system. We want to be able to send the response back to the device before sending the analytics data out. Currently this is possible only by creating two separate actions which means using two containers instead of one.

Integration with Microsoft Bot Framework (https://dev.botframework.com/). The bot framework decouples bot message receive from reply. When the bot message is received, the framework needs to send an immediate acknowledgement to the chatbot and then do some post processing before sending one or more replies through the Microsoft bot backend.

Trying to integrate the bot framework with the current architecture fails because the docker is paused before the post processing is complete.
Possible solutions:
Allow an action to declare when it's considered complete, the options could be:

When function returns (default)
When Node JS event loop is empty (Or the equivalent for other programming languages)
Using a specific hook (Which could be made available through https://github.com/apache/incubator-openwhisk-client-js and other clients) which will single back to Openwhisk that the action is complete.

This is an interesting use-case. I'm wondering if invoking actions with a non-blocking request can be used to handle these examples? This would allow the receiving action to respond immediately, whilst triggering background work. This will mean a second action is used but the total execution time (and cost) should be the same as the first action finishes immediately.

That's exactly what I did - Broke my action into two actions, one doing the pre-processing up to the response, and another doing the post-processing.
It ends up utilizing more containers, as well as make the development process much more complex than it would have been otherwise, so the ability to accomplish post-processing in a single action would be very helpful.


Add Part B to the PDF output

Tasks:

[x] Part B Handover details
[x] Part B Record of handover
[x] Part B Release at court and receipt of property
[x] Part B Record of checks and significant events x 3

This will need some alterations to the specs so that the suite can finish faster. Currently it runs in 14s on master, this branch pushes that to 1m5s. I don't think we need to render the whole document out in each spec.


Error while loading shared libraries: libfuse.so.2

IBM Kubernetes Cluster
Kubernetes Version: 1.29.9 (Node with Ubuntu 24)
ibm-object-storage-plugin: 2.2.32
Since updating our Kubernetes Nodes to Ubuntu 24 we get the following error in our container:
MountVolume.SetUp failed for volume "pvc-..." :
mount command failed, status: Failure, reason:
Error mounting volume: s3fs mount failed: s3fs: error while loading shared libraries: libfuse.so.2:
cannot open shared object file: No such file or directory

The container stays in pending mode. The PVC however is bound.
Example PVC:
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: test-pvc
  labels:
    billingType: hourly
    region: eu-de
  annotations:
    ibm.io/add-mount-param: complement_stat,compat_dir
    ibm.io/auto-create-bucket: 'false'
    ibm.io/auto-delete-bucket: 'false'
    ibm.io/bucket: BUCKET_NAME
    ibm.io/endpoint: https://s3.direct.eu-de.cloud-object-storage.appdomain.cloud
    ibm.io/secret-name: s3-access-secret
spec:
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 100Gi
  storageClassName: ibmc-s3fs-smart-perf-regional

Problem still persists.
Also tested on a Cluster Node with Kubernetes version 1.31.1_1527 with Ubuntu 24.
PVC YAML:
kind: PersistentVolumeClaim
apiVersion: v1
metadata:
  name: test-pvc
  annotations:
    ibm.io/add-mount-param: complement_stat,compat_dir
    ibm.io/auto-create-bucket: 'false'
    ibm.io/auto-delete-bucket: 'false'
    ibm.io/bucket: BUCKET_NAME
    ibm.io/endpoint: https://s3.direct.eu-de.cloud-object-storage.appdomain.cloud
    ibm.io/secret-name: SECRET_NAME
spec:
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 10Gi
  storageClassName: ibmc-s3fs-smart-perf-regional

Status is bound and PersistentVolume is created, but the Pod will not start:
apiVersion: v1
kind: Pod
metadata:
  name: pvc-test
spec:
  volumes:
    - name: data
      persistentVolumeClaim:
        claimName: test-pvc
  containers:
    - name: pvc-size-test
      image: public.ecr.aws/docker/library/busybox:stable
      command: ['/bin/sh', '-c']
      args: ['while true; do du -sh /data; sleep 10;done']
      volumeMounts:
        - mountPath: '/data'
          name: data
      resources: {}

This still leads to the error FailedMount and a pod stuck in ContainerCreating:
MountVolume.SetUp failed for volume "pvc-3a8a9b35-75a5-4289-8aaa-fcd38143bcb3" : mount command failed, status:
Failure, reason: Error mounting volume: s3fs mount failed: s3fs: error while loading shared libraries:
libfuse.so.2: cannot open shared object file: No such file or directory

Just switching to Ubuntu 20 as the base OS for the Nodes resolves this error.
Ubuntu 20 is deprecated and will soon stop to be be supported by IBM Kubernetes Clusters, we need a solution for this!


Cannot read property 'classList' of undefined

https://github.com/ripjar/material-datetime-picker/blob/476b08bd6b629828c6a5f9909d94d8d7a4e73f0d/lib/js/index.js#L411
Throws error :

Cannot read property 'classList' of undefined

If the options.default value for minutes can not be divided into 5
If picker is integrated in existing project. I think there must be a way to be intercepted and not as is currently throwing error.

Also experiencing this issue. I see https://github.com/ripjar/material-datetime-picker/issues/153 was closed and should have fixed this, so maybe it has re-emerged.
Any ideas @kayleighbray?


fix docs inconsistency in default db/schema name and password

Updated osx/ubuntu guides to match quickstart guide and scripts

securitymonkeydb -> secmonkey
securitymonkeypass -> securitymonkeypassword

Coverage remained the same at 63.617% when pulling 015c7e806995d66e1f12de763aeab2a648f78954 on frohoff:doc-fixes into b807cf6b2f0f476d63003581c06a865010ec9fe7 on Netflix:develop.

LGTM


Announce of Framaboard (Kanboard-inside)

Hello everybody,
first of all, thank you all for your work on Kanboard, it is a great tool :).
I'm not sure I should open a ticket for that but since I can't find any mailing-list, I do that here.
Just to let you know that Framasoft (a French association, well-known for its "de-google-ify Internet" campaign http://degooglisons-internet.org/) is about to release a service based on Kanboard, known by the name of Framaboard (https://framaboard.org/).
We are still working on it before the official release on Wednesday but I think its better to ping you first (Frédéric was already aware about Framaboard, thank you again for your reactivity!).
The official repository (README is in French) is on our Gitlab: https://git.framasoft.org/marien.fressinaud/Framaboard
So what's next? Well, we have already some feedback thanks to our "Colibris" (i.e. Framasoft community) and I will probably open some tickets in the next few days. Note that the biggest complaint is about UX (User eXperience) and we, at Framasoft, would really appreciate to help you to improve that.
Best,
Marien

cool :) I will post a news on the website

:+1:

Good luck!
Which aspects of the user experience is your community unhappy with?

@phillc73
We're unhappy with nothing in kanboard, but we wanted to provide easy "register your own kanboard" easyly.
E.g: by registering (totally free and with strong ethical charter) on framaboard, you can access to phillc73.framaboard.org (a standard kanboard installation) in seconds. No server or skills needed.
The development of this functionnality is of course libre software and developped by @marienfressinaud : https://git.framasoft.org/marien.fressinaud/Framaboard/
@fguillot : please note that Framaboard is out! http://framablog.org/2015/10/07/framaboard-les-libristes-ont-reinvente-le-tableau-blanc-mais-en-mieux/ :)

@pyg77 That's a great idea! Especially for those without the technical skills to host their own Kanboard.

Just posted a news: http://kanboard.net/news/framaboard

Thanks! :)


Quick Start example on .retrieve_citation() not working

I use a  working Scopus key in variable KEY which is enabled for the Citation Overview API.  Trying to reproduce the Quick Start example I get the following error message:
>>> scopus = Scopus(KEY)
>>> pub_citations = scopus.retrieve_citation('84905286162')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/pyscopus/scopus.py", line 306, in retrieve_citation
    soup = bs(urlopen(citation_url).read(), 'lxml')
  File "/usr/lib/python2.7/urllib2.py", line 127, in urlopen
    return _opener.open(url, data, timeout)
  File "/usr/lib/python2.7/urllib2.py", line 410, in open
    response = meth(req, response)
  File "/usr/lib/python2.7/urllib2.py", line 523, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python2.7/urllib2.py", line 448, in error
    return self._call_chain(*args)
  File "/usr/lib/python2.7/urllib2.py", line 382, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 531, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)
urllib2.HTTPError: HTTP Error 404: Not Found

Thank you for bringing this up.
As stated in the quick start page,

Please note that if you are searching the citation record for only one publication, you still need to put it in a list or numpy array.

So please try to put the scopus id in a list like this: ['84905286162'] and see if that works. My api keys are expired now and cannot test this feature at this moment.

Ah yes, that's how it works. I was reading it but then felt to simply repeat the example. You should maybe update the quick start page.

fixed quick start page


nodes on mainnet are not loading

maybe we should index healthy nodes (responsive ones) on the grid proxy

https://github.com/threefoldtech/tfgrid-sdk-go/issues/603
the proxy will provide another query param to help achieving that too

work is done from the proxy side & deployed now on devnet.
changes:

boolean field healthy is added on the /nodes and /nodes/<id> endpoint responses
boolean query healthy is added on the node filters

healthy=true indicates that the node responded to an rmb call in the last 5 minutes

should this query should be added to swagger ?

the endpoint is working fine but i think it should be added to swagger too

I opened this issue in tfgrid-go

okay we shall close this for now since there is another issue for the swagger
testlodge
https://app.testlodge.com/a/26076/projects/40893/suites/211680?expand_section=354530#case_4189153


[question] [v.0.13.1] [android] getting IncompatibleClassChangeError when trying to bind .aar in C# project

Describe the bug
Getting Java.Lang.IncompatibleClassChangeError: no non-static method "Lcom/cossacklabs/themis/SecureCellSeal;.decrypt([B[B)[B" in Release configuration in C# android project.
When decrypting "obfuscated" string constant on app start.
Any ideas?  Have you seen anything like this in some java or kotlin android project?
To Reproduce
On app start I try to decrypt an "obfuscated" string constant
_secureCell = SecureCell.SealWithKey(masterKeyData);
_secureCell.Decrypt(cipherTextBytes, context);

Getting an error in Release configuration:
Java.Lang.IncompatibleClassChangeError: no non-static method "Lcom/cossacklabs/themis/SecureCellSeal;.decrypt([B[B)[B"
[orion.mobile]   at Java.Interop.JniEnvironment+InstanceMethods.GetMethodID (Java.Interop.JniObjectReference type, System.String name, System.String signature) [0x0005b] in <42d2b7086f0a46efb99253c5db1ecca9>:0
[orion.mobile]   at Android.Runtime.JNIEnv.GetMethodID (System.IntPtr kls, System.String name, System.String signature) [0x00007] in <3080427739614e60a939a88bf3f838d5>:0
[orion.mobile]   at Com.Cossacklabs.Themis.SecureCell+ISealInvoker.Decrypt (System.Byte[] p0, System.Byte[] p1) [0x00017] in <cd618986d1ce4194b63cdd3366dad291>:0
[orion.mobile]   at Themis.Droid.CellSealDroid.UnwrapData (Themis.ISecureCellData cipherTextData, System.Byte[] context) [0x0007e] in <a492e7118e094c3296442a386fe5d80e>:0
[orion.mobile]    --- End of inner exception stack trace ---

Expected behavior
N/A - this issue is a question
Environment (please complete the following information):

OS: Android 10, build 00WW_2_250
Hardware: Nokia 7.2
Themis version: 0.13.1
Installation way:

[x] via package manager
[ ] built from source

Additional context
Sorry for asking in a wrong place if I'm violating any of your policies with this ticket.
I've spent a while debugging it and am a bit desperate at the moment.
I know you do not support that C# and Xamarin.Forms but filing this question just in case you've seen a similar issue in some java or kotlin android project.
Unable to share a sample project
since that does not reproduce on https://github.com/dodikk/themis-xamarin-prototype/tree/bugfix/v0.13.2/droid-strip-symbols
Only in a project under NDA, unfortunately.

I've checked the data I'm getting the failure on. It has been encrypted with wasm-themis CLI tools. Also I can decrypt the
data collected from my app's exception (again, with wasm-themis CLI tools)
The same app code and bindings work in debug configuration
apk seems to have SecureCellandSecureCellSeal class symbols (checked via "profile apk" UI in android studio)

Hm, that doesn't look right.
@dodikk thank you for notifying, although we don't support C# at the moment, but let's try to find a workaround. Do I understand correctly, that the sample project  doesn't have the same issue?
Right now, I do not have meaningful suggestions yet, but I do have a set of experiments that might help to locate the issue.

Quick ideas to try on the "real" project

Does the issue on real project happens on some particular Android emulator or device? Can you try different device / emulator? Please try 64-bit emulator if you're using x32.

Does the issue on real project happens only for Android 10? Can you try different Android version?

Can you please try older versions of Themis, like 0.13.0 or 0.12.2 to see if the issue still persists on your main project? Themis 0.13.0 has introduces some changes in API that might be causing this issue (but pure guess). Please install 0.12.2 using maven { url "https://dl.bintray.com/cossacklabs/maven/" }, as it's not available in JCenter.

Can you please specify how exactly do you install Themis in your real project? I see that you're installing Themis using maven for your sample project.

Please try using Themis 0.13.1 from JCenter directly (just indicate jcenter(), don't indicate maven { url "https://dl.bintray.com/cossacklabs/maven/" }) to see if this will have any changes. We haven't published officially that Themis supports JCenter yet, so please refer to the non-published docs if you have any questions.

More experiments trying to find the difference between sample project and real project

Can you please list all the differences between the sample project and the real: the way of Themis installation, usage of same/different Themis APIs, usage of same/different Themis versions, usage of same/different gradle versions, etc?

Can you please use the exact data (encrypted message) that breaks real project to see if it breaks a sample project?

Do I understand correctly, that the sample project doesn't have the same issue?

Right.
Same device, same thems.aar binary ==> different results for some reason.

Please try using Themis 0.13.1 from JCenter directly (just indicate jcenter(), don't indicate maven

Actually, I've installed the package like this.
curl -L --output themis-0.13.1.aar https://bintray.com/cossacklabs/maven/download_file?file_path=com%2Fcossacklabs%2Fcom%2Fthemis%2F0.13.1%2Fthemis-0.13.1.aar

P.S.  I had some difficulties integrating maven/gradle into C# project build, so I took that shortcut.
It seems to have been working fine for v0.12.0

Can you please specify how exactly do you install Themis in your real project? I see that you're installing Themis using maven for your sample project.

That sample project might have some failed experiments "leftovers".
So, as shown in the comment above,
I'm using curl to download themis binary

Can you please use the exact data (encrypted message) that breaks real project to see if it breaks a sample project?

Will come back with this a bit later today.
As well as with "another device" and "another droid version" checking.

@vixentael thanks for blazing fast reply. I appreciate a lot.

Java.Lang.IncompatibleClassChangeError: no non-static method "Lcom/cossacklabs/themis/SecureCellSeal;.decrypt([B[B)[B"

Hm... Can this be because SecureCellSeal is not public? It's a package-private class implementing a public SecureCell.Seal interface, and the user code is expected to use the interface. SecureCell#SealWithKey returns this interface. It seems like C# interop tries to be smart and devirtualize the call but that doesn't work quite well.
Another idea is that the method name may be not correct. JNI has some peculiar rules, especially when it comes to overloads. It seems correct – decrypt method of com.cossacklabs.themis.SecureCellSeal which takes two []byte arrays and returns one – but there's also another decrypt method which may throw a spanner in the works. But again, the signature and the method name seem to be correct.

Hm... Can this be because SecureCellSeal is not public?

Thanks @ilammy . I've thought about that. But I did not have enough time to fork themis and rebuild it with the mentioned changes just yet. Will try that when all other "low hanging fruit" ideas prove being unrelated to the issue.

Can you please use the exact data (encrypted message) that breaks real project to see if it breaks a sample project?

That same data from the "real project" does not break my "sample project".
Checked on the very same nokia device.

Can you please list all the differences between the sample project and the real:

[x] the way of Themis installation ==> same.
Just downloaded that aar with curl and generated xamarin bindings on top of it. No significant diff between the correspondingcsproj files (C# projects) discovered.
[x]  usage of same/different Themis API ==> same.
The C# code of binding/wrapper projects is identical (for "sample" and "real" projects). That code is available in dodikk/themis-xamarin-prototype repository.
[x]  usage of same/different Themis versions ==> same
My "real project" bindings are based on the very same aar binary as the "sample project".
[-] usage of same/different gradle versions, etc? ==> N/A.
In my case that was just curl. Copying the same binary on the file system does not help either.

/cc @vixentael

@vixentael @ilammy can that be caused by mismatch of the android studio versions?
Which one did you use to produce that themis-0.13.1.aar artefact?
Might updating to the most recent toolchain help mitigating the issue?

Can you try different device / emulator? Please try 64-bit emulator if you're using x32.

Same results on x64 - Nexus 5 - Android Pie emulator (both for the "real project" and for the prototype; on CipherText data from "real app")
/cc @vixentael

can that be caused by mismatch of the android studio versions?

Well, everything is possible, but I don't think that the toolchain versions matter very much here. The issue seems to be on the Java side of things, not in the native interop, so NDK should not affect it. Java/Android alone have a very good story with compatibility (I'm looking at you, Apple), so I doubt that Java compiler is doing something weird here.

Which one did you use to produce that themis-0.13.1.aar artefact?

We use a standalone toolchain to build it, not Android Studio. It's available in cossacklabs/android-build Docker image. It is somewhat outdated, but that's a result of brittle equilibrium of bugs and incompatibilities that is able produce results. We try updating it from time to time, but it's not kept on the bleeding edge.

Well, everything is possible, but I don't think that the toolchain versions matter very much here.
We use a standalone toolchain to build it, not Android Studio.

Ok. Then I'll try other things before applying your toolchain to my app's build environment.
Thanks for providing context, @ilammy

I'm trying the guides below to check the "troubles from C# world"

https://gist.github.com/JonDouglas/dda6d8ace7d071b0e8cb
https://www.jon-douglas.com/2017/04/13/linker-bitdiffer/

But no success fo far.
Things start to break when C# apk size optimizer (which they call "linker" for some reason) is set to more aggressive mode (None ==> Sdk Assemblies Only modes mentioned in ).
So the high-level hypothesis "the apk size optimizer is stripping too much" looks like the best candidate. On the other hand, the symbols mentioned in the exception are there in the apk
P.S. This comment is not a question but rather an update on "what I've discovered so far"

thank you @dodikk, we're watching your progress! Linker's optimization is a good guess, do you think it's possible to disable it / change optimization level?

@vixentael sure thing it is possible. But I cannot use None since that would bloat the apk size with the entire mono runtime. Which is highly unwanted (ok for debug and in-house; but bad for release and playmarket).
So I need a more fine-tuned fix and can't solve the issue that easily.

Also, xamarin allows specifying manually "which symbols should stay" (see the link below). But I'm viewing that as "weapon of last resort" since that approach would be rather time-consuming (just like writing a makefile without those cmake or autotools helpers). So I'm hoping to find an easier fix and still "on a quest" to find one...
https://docs.microsoft.com/en-us/xamarin/cross-platform/deploy-test/linker

Solved
What had to be done:

Add proguard.cfg file to the build
Add instructions to "ask" proguard to not remove themis classes

-keep class com.cossacklabs.themis.**
-keep class com.cossacklabs.themis.** {*;}

/cc @vixentael @ilammy

Do you accept pull requests to your documentation?
@vixentael , could you please share the URL of the corresponding repo?
As far as I've understood so far... Native android devs might run into such issue as well. So documenting this pitfall (in the Installing JavaThemis for Android development) guide might make sense.
Unless either

you've done so already but I have not noticed that hint
maven or gradle deals with proguard automatically by default
Having the solution documented in this ticket as is good enough

Do you accept pull requests to your documentation?
@vixentael , could you please share the URL of the corresponding repo?

Sure we do. The docs live here: https://github.com/cossacklabs/product-docs
Though, I'm not sure if this is an issue strictly related to Themis. Android Studio sets up ProGuard in Android projects by default, but the issue seem to be generic enough to happen with any other library if that's ProGuard 'optimizing away' parts that it does not deem essential when in fact they are.
I think, if this issue affects multiple projects then it's worth documenting to caution developers. Otherwise, I don't think that Themis docs should bloat into a list of trivia and howtos on configuring (effectively) third-party components of the build. I guess, if this issue pops up for someone else they can be directed here at first.
@vixentael, what's your take?

I'd mention this on Installation page, maybe like a separate paragraph "Configuring ProGuard rules" under "Installing stable version with Gradle".
I totally agree with @ilammy that we shouldn't put all the tips&tricks into docs, but this is a tricky issue which might take some time to troubleshoot, so having it mentioned somewhere with link to this issue makes sense to me.
@dodikk can you add these proguard rules to your sample project? As an illustration.

can you add these proguard rules to your sample project? As an illustration.

@vixentael sure. Of course I'll do that.
Also I was planning to document this stuff in the sample's readme.

Amazing, really waiting forward!

Amazing, really waiting forward!

@vixentael , I have updated develop branch of the demo project. Both the project and readme.
https://github.com/dodikk/themis-xamarin-prototype/tree/develop
Will update the default branch of demo repository later, after more testing.

Add info to docs here https://github.com/cossacklabs/product-docs/pull/28


visualization of multiply  indented code

It is not an issue, but rather a suggestion. I have large piece of code with several indentation levels..It is quite tedious and error prone to trace the indentation, while the consequences of mistakes can be quite serious.  I think it would be very convenient to have an option in the editor to highlight the regions with different indentation  with different colors.. It can be either automatic (based on the indentation) or manual (by user specified levels of cells, for example)

This was sort of fixed in Spyder 4 because in that version we added indent guides to the editor.
You can activate them by going to the menu
Source > Indent guides

I think it would be very convenient to have an option in the editor to highlight the regions with different indentation with different colors

Sorry, we don't have plans to implement something like that.


issue of IDASH PRIVACY & SECURITY WORKSHOP 2022

Hi,
I'd like to know that will the code of IDASH PRIVACY & SECURITY WORKSHOP 2022 - secure genome analysis competition be public?
And how could I find the video of the presentation? Looks like Indiana University account is need to access the video?
BR

Hi @macknight, this question is unrelated to Lattigo. Please contact Tune Insight directly https://tuneinsight.com/.


MapGuard dereferences to a dangling pointer

Hello fellow Rustacean,
we (Rust group @sslab-gatech) are scanning Rust code on crates.io for potential memory safety and soundness bugs and found an issue in this crate which allows safe Rust code to exhibit an undefined behavior.
Issue Description
https://github.com/vorner/arc-swap/blob/b5ec44cd5d9b4b4a491cf85c1cbb2b1d5cdae061/src/access.rs#L279-L293
https://github.com/vorner/arc-swap/blob/b5ec44cd5d9b4b4a491cf85c1cbb2b1d5cdae061/src/access.rs#L230-L242
As noted in the comment, unsafe code in MapGuard expects the underlying guard type to dereferences to the same value even when the guard object is moved. However, Map uses Access as a trait bound which does not guarantee this property. As a result, Map accesses a dangling pointer when it is used with an Access implementation that does not dereferences to the same value when moved.
https://github.com/vorner/arc-swap/blob/b5ec44cd5d9b4b4a491cf85c1cbb2b1d5cdae061/src/access.rs#L295-L322
Constant seems to be the only type in this crate that implements Access in this way, but there can be other user types that implements Access on their own.
Reproduction
Below is an example program that segfaults, written only with safe APIs of arc-swap.
Show Detail

#![forbid(unsafe_code)]

use arc_swap::access::Map;
use arc_swap::access::{Access, Constant};

static CORRECT_ADDR: &str = "I'm pointing to the correct location!";

#[derive(Clone)]
struct MemoryChecker {
    // should be always CORRECT_ADDR
    message: &'static str,
}

impl MemoryChecker {
    pub fn new() -> Self {
        MemoryChecker {
            message: CORRECT_ADDR,
        }
    }

    pub fn validate(&self) {
        println!(
            "Pointing to {:?}, len {}",
            self.message as *const _,
            self.message.len()
        );
        println!("Message: {}", self.message);
    }
}

fn overwrite() {
    let a = 123;
    let b = 456;
    println!("Overwriting stack content {} {}", a, b);
}

fn main() {
    let constant = Constant(MemoryChecker::new());
    constant.0.validate();

    // Create a map with identity mapping
    let map = Map::new(constant, |checker: &MemoryChecker| checker);

    // After calling this, `value` pointer of `MapGuard` points to a dangling stack address
    let loaded = map.load();

    overwrite();

    loaded.validate();
}

Output:
Pointing to 0x55dcc9269000, len 37
Message: I'm pointing to the correct location!
Overwriting stack content 123 456
Pointing to 0x55dcc9267530, len<PHONE_NUMBER>73556

Terminated with signal 11 (SIGSEGV)

Tested Environment

Crate: arc-swap
Version: 1.0.0
OS: Ubuntu 20.04.1 LTS
Rustc version: rustc 1.48.0 (7eac88abb 2020-11-16)

Hello
It's a great discovery, though disturbing this has slipped through. I'll have to think how to plug this problem in a way that doesn't disrupt users too much, but I'll have a look at it today or tomorrow.
Thank you for finding it.

Fix released as 1.1.0 (by eliminating all unsafe code in that file).
It is technically a breaking change, but the chance of actually breaking code is low and the chance people will migrate from the broken version are higher this way, and even rustc makes an exception for soundness issues in the stability guarantees. It seems less bad to release as part of the 1. version.

Added a backport, released as 0.4.8, for all the reverse dependencies that didn't migrate to the 1. version yet.

Thank you for the quick fix!

CVE-2020-35711 was assigned to this issue.


Fix failing GPU test on single GPU host (kvstore) #10977

Description
See title
Fixes #10977
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@mxnet-label-bot [pr-awaiting-merge]

@mxnet-label-bot [pr-awaiting-testing]

@marcoabreu seems the test depends on this hardcoding when checking the numerical results.

@eric-haibin-lin


add another unknownRevision error to detect

The following error also caused blame not working:

git rev-parse --symbolic-full-name my_branch@{u}
fatal: upstream branch 'refs/heads/my_branch' not stored as a remote-tracking branch

Git version: 2.17.1

Tested locally and it works, with this change the inline blame and comparing with previous revision works as expected.
But I wonder if it's possible to skip resolving the remote tracking branch? Is it possible to add an option to disable this behavior?

Thank you for your contribution! 👍
As for skipping, I use the information about the remote branch to enable/disable other GitLens features (and it is only done once), so I don't see a compelling reason to have an option to disable it.


Login Review Ticket

CB Create Task completed and documented on GH Pages - Prep to submit to CB on or before May, 2nd. (2 Points - Individual)
CB Create Task Coding requirements documented
Runtime of Code(with comments) on Replit/Intellij
Accounts and Logins Challenges/Hacks ( 3 Points - Pair/Trio)
Start or add to existing project: init.py, main.py, requirements.txt and cruddy blueprint directory are things you will need to add to your existing project. Additionally, you should start with some of the content in templates and static folders. Find a pair and work on this together. Ask questions.
Analysis: the csp.nighthawkcodingsociety.com implementation is for educational purposes. It has some obvious flaws in its security. Discuss some of the flaws that you see in its security as implemented.
There is no password to access any of the information, essentially making all of the information public.
Link to Forked Repo
Hack 1 Add Phone Number to Sign Up / Authorization screen
Hack 2 Add logout to CRUD screen
Hack 3 Add login_required to another portion of the project
Extra Credit: A quality design would have login and logout on Navbar with user information. Be sure to have login protecting something on your site, however, do NOT make entire site under login_required.

5/5
-0.5 because login required tag added to separate page takes you back to crud page when you click login button
+0.5 for extra credit hack added login/logout to navbar
All hacks documented nicely and all code is working and running smoothly. All hacks completed and create task documentation  was very good and thorough.
CB Create Task Coding requirements documented
Runtime of Code(with comments) on Replit/Intellij
Accounts and Logins Challenges/Hacks ( 3 Points - Pair/Trio)
Start or add to existing project: init.py, main.py, requirements.txt and cruddy blueprint directory are things you will need to add to your existing project. Additionally, you should start with some of the content in templates and static folders. Find a pair and work on this together. Ask questions.
Analysis: the csp.nighthawkcodingsociety.com implementation is for educational purposes. It has some obvious flaws in its security. Discuss some of the flaws that you see in its security as implemented.
There is no password to access any of the information, essentially making all of the information public.
Link to Forked Repo
Hack 1 Add Phone Number to Sign Up / Authorization screen
Hack 2 Add logout to CRUD screen
Hack 3 Add login_required to another portion of the project
Extra Credit: A quality design would have login and logout on Navbar with user information. Be sure to have login protecting something on your site, however, do NOT make entire site under login_required.


Added jeffs ashr fix, added combinational memory tests, unknown bit now has definition

Just removed the cout statements.
This PR does not handle high impedance values, but the CGRA high impedance values are removed by tristate pre-processing anyway.


PAMI FIFO Size bug for large messages

Original issue: https://charm.cs.illinois.edu/redmine/issues/39

Connected to rendezvous issues, but can occur independently.
Workaround is to increase environment variable,
for many small msgs:
for large msgs:
Add text to our error output describing env variables.
Add ENV to release notes.

Original date: 2013-02-13 17:29:54

Descheduling from 6.5 release because the pamilrts build seems to be sufficiently stable

Original author: Nikhil Jain
Original date: 2013-02-25 21:51:56

Added the warning to src code. Added the note in release notes.


Extend ci checks

It would be nice somehow checking

superfluous todo entries (pylint)
something else that pylint provides (basic pep8)
That translations are complete (harder to implement I believe)

Regarding the translation issue, running pybabel extract via make extract_messages and checking for a diff should be perfectly sufficient: If there is a diff, something must have been forgotten.
It's a low hanging fruit and something often forget.


Librdkafka cppkafka

Automatic installation librdkafka/cppkafka

Thank you for your pull request. However, we encourage you to host your own plugin. You can follow the pattern set by #5026.
We would be happy to accept a pull request to update: https://github.com/EOSIO/eos/blob/master/plugins/COMMUNITY.md
with a link to your plugin.


Instagram and Following

Some Instagram required following to see content, how i can login my account to download it ? because i dont see any option to use instagram account

This repo is no longer maintained, all work has moved to https://github.com/ripmeapp/ripme


cni: update README

Signed-off-by: Xiang Dai<EMAIL_ADDRESS>Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[X] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[X] Does not have any changes that may affect Istio users.

Hi @daixiang0. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Some links become 404, no sure right point, delete them and wait for review input.

/ok-to-test

@daixiang0: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
3c62bc6b4d00899e016466eda36e494a0949fa03
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-notes_istio


When I delete options from the DialogNode with Backspace, getting errors when I click on the remaining option boxes

The errors don't crash anything and are easily fixed by remembering to not click on any remaining option edit boxes and just click off the options into empty Edit Grid space. Once bugged can be fixed by closing and reopening the tree. But this may indicate a problem that could cause further problems so should be fixed at some point.
scene/main/node.cpp:1529 - Index p_index = 5 is out of bounds ((int)data.children_cache.size() - data.internal_children_front_count_cache - data.internal_children_back_count_cache = 5).
 res://addons/dialogue_nodes/nodes/option.gd:34 - Invalid get index 'selected' (on base: 'Nil').
  res://addons/dialogue_nodes/nodes/dialogueNode.gd:72 - Invalid type in function 'set_condition' in base 'HBoxContainer (option.gd)'. Cannot convert argument 1 from Nil to Dictionary.
  Cannot set slot with index (-1) lesser than zero.

To illustrate how to reproduce these buggos, I present this unlisted YouTube video-o-o. Because I worked all day and can't really describe with my fingers very well right now... https://youtu.be/Auvjf3CkccY

Yep, this is a known issue mentioned in the Readme file


questions regarding the wiki page "Using the Code Base"

on the wiki page "Using the Code Base"..

why are we doing an "istioctl install" before building and pushing the images?
doesn't the "istioctl install" command require the flags --set hub= and --set tag= to actually fetch the images that one built and pushed?

also it would be nice to have instructions that work on apple silicon, if it's not too much trouble.  thanks!

I agree the ordering as specified in your #1 is wrong. Also, missing the --set's.
I updated the wiki page for those items, also adding an item on the env variables (copied from the Linux page, wasn't on the Mac page).
As for working on Apple silicon, we would need someone to provide that data. I only have a Mac Intel at the moment. I would expect that most of the commands would actually be the same. For example, I use Docker Desktop and if I use its Kubernetes environment, I can do a make docker and then run istioctl install and it installs Istio. Since the K8s env can see the built images, I don't need to push them. I would expect the same to work on Apple silicon, but unfortunately, I can't verify. You would need to be on a very recent commit as the ARM support was only very recently added.

Since I can't do anymore with the wiki, I will close this issue. If you have some additional thoughts/feedback for running on Apple silicon, please add it here and reopen the issue.

super. thanks!  i'll run through the updated instructions and provide feedback if necessary.

this command looks incorrect:
go run ./istioctl/cmd/istioctl install uninstall --purge

shouldn't it read:
go run ./istioctl/cmd/istioctl x uninstall --purge

also the sentence "you can deleted you installed istio.." -> "you can delete your installed istio.."


validate array

This will help to validate an array and in some cases validate array keys values. @Domeshow wait for your feed back

Thank you bro. I’ll check this as possible thanks
Sent from my iPhone

On Nov 21, 2021, at 8:30 PM, ibrahim @.***> wrote:

This will help to validate an array and in some cases validate array keys values. @Domeshow wait for your feed back
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Now resolved


Custom cluster S3 restore empty zip file

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Add a Custom Cluster to a 2.2.8 rancher installation
Add a etc/controller/worker node
Configure S3 backup and do a snapshot now
Verify successfully S3 backup
Do a Restore snapshot

Result:
Rancher will notify:
Failed to download etcd snapshot from s3, exit code [1]: time="2019-08-22T12:55:03Z" level=fatal msg="Unable to decompress [/backup/c-szfbb-ms-m8r6l_2019-08-22T12:36:49Z.zip] to [/backup/c-szfbb-ms-m8r6l_2019-08-22T12:36:49Z]: zip: not a valid zip file"

Other details that may be helpful:
On the node, the backup zip file is epmty
Environment information

Rancher 2.2.8
Rancher HA (3 nodes)
Single node Custom cluster with S3 backup

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Custom
Machine type (cloud/VM/metal) and specifications (CPU/memory): CoreOS/Ubuntu (both haveing same issue)
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.9", GitCommit:"3e4f6a92de5f259ef313ad876bb008897f6a98f0", GitTreeState:"clean", BuildDate:"2019-08-05T09:22:00Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.6", GitCommit:"96fac5cd13a5dc064f7d9f4f23030a6aeface6cc", GitTreeState:"clean", BuildDate:"2019-08-19T11:05:16Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client: Docker Engine - Community
 Version:           19.03.1
 API version:       1.40
 Go version:        go1.12.5
 Git commit:        74b1e89
 Built:             Thu Jul 25 21:21:05 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.1
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.12.5
  Git commit:       74b1e89
  Built:            Thu Jul 25 21:19:41 2019
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.2.6
  GitCommit:        894b81a4b802e4eb2a91d1ce216b8817763c29fb
 runc:
  Version:          1.0.0-rc8
  GitCommit:        425e105d5a03fabd737a126ad93d62a9eeede87f
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

Closing this issue as it was a result of using Hitachi S3 backend. The http.defaulttransport does not work, only minio.defaulttransport


First Name and Last Name, We are able to fill any data in Registration form.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
2.0.0
Issue Description
We are able to Create a new customer with filling any data in the First Name and Last Name in Registration Form.
Preconditions
Need to Create a new user from store front.
Steps to reproduce

Go to Store Front.
Click on Sign Up
Fill the First Name and Last Name Information.
Sample Text - test123243<EMAIL_ADDRESS>Create User

Expected Result
It should not take any unrequired data in store front registration form.
Actual Result
Please check the video for reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/df480bd7-fd32-4b50-b760-b6ea64840873
Screenshots
No response
Additional context
No response

Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.


SCHEDULE-STATUS should not contain an explanatory text

I was looking at the IMipPlugin sabredav provides and was wondering about the scheduleStatus values being set in the Itip\Message. Reading through http://tools.ietf.org/html/rfc6638#section-7.3 it looks like there is no room for explanatory text and it should instead be a list of statcodes.
I haven't checked what scheduling emails actually look like so I may be wrong here, but I think its a bug.
Another trivial issue you might want to just piggyback here, $server is defined as a class variable, but the initialize function does not set it.

Thanks!
I wonder where I got the reason phrase from... I couldn't just have made that up I feel :P

Re-open the issue because it is partly fixed.

@Hywan what did I miss?

@evert The description of this PR raises 2 issues:

scheduleStatus does not contain the right information,
an unsed variable.

#588 addressed 2. So 1 is still opened. Nop?

You may have missed b16493c =)

Going to assume that @Hywan overlooked that commit. Reopen and elaborate if I was wrong.

@evert Yup, I missed b16493c, because you didn't make a PR :-D!


Refactor smoke tests

Current coverage is 81.54%

Merging #430 into master will increase coverage by +0.27% as of ff5a1ac

@@            master    #430   diff @@
======================================
  Files          139     139
  Stmts         3130    3089    -41
  Branches         0       0
  Methods
======================================
- Hit           2544    2519    -25
  Partial          0       0
+ Missed         586     570    -16

Review entire Coverage Diff as of ff5a1ac

Powered by Codecov. Updated on successful CI builds.


Submodule/update-ui

Please answer these questions before submitting pull request

Why submit this pull request?

[x] Bug fix

Related issues

https://github.com/apache/incubator-skywalking-ui/pull/126

Coverage remained the same at 19.44% when pulling 00229e4fdb6fba15f6002add1d2fbe5b443d5f54 on hanahmily:submodule/update-ui into 9b6f87237a78933c99c0af99162d9ba17d1a3adc on apache:master.


[release-1.24] Preload iptable_filter/ip6table_filter

Proposed Changes
Preload iptable_filter/ip6table_filter
ServiceLB now requires this module, but it will not get autoloaded by the kubelet if the host is using nftables.
Types of Changes
bugfix
Verification
See linked issue
Testing

Linked Issues

https://github.com/k3s-io/k3s/issues/6644

User-Facing Change

NONE

Further Comments

arm64 flaked; merging
[STABLE-SERVER] E1214 00:25:42.095617      79 server.go:213] "Failed to create listener for podResources endpoint" err="failed to move temporary file to addr \"/var/lib/kubelet/pod-resources/kubelet.sock\": rename /var/lib/kubelet/pod-resources/1147896728 /var/lib/kubelet/pod-resources/kubelet.sock: invalid cross-device link"


add german translation to readme

@tentakel This is awesome! Thanks.
Ping @gretel and @orangecms! :)

sorry i am germish and live in germomoland.. could offer to translate into therefore though 🚮

@gretel kein problem! 🇩🇪


Code lens not working with suggested FSharp.fsacRuntime:netcore setting

Ionide-fsharp 3.3 / Windows 10 x64 / .Net Core 2.2.102

If I switch to the default "net" it works as usual. Things I've tried to fix the problem after reading other similar issues:

Installed MS Build Tools with F# support
Updated VS Code
Removed .foders from solution path
Re-installed .Net SDK

@Krzysztof-Cieslak thanks, this solutions seems to work so far although it appears I have to do it once per .fs file, but that's ok because eventually all files will be fixed and I guess I could just delete obj for the entire projects


How to save the host dns ip addr?

Resolve hosts to ip addr ,Want  to save the ip addr, Because Resolve spends many times.
I want to replace the default dial func,  but ,how to get the tcpAddr?
Some one help to give some detail code?
d := net.Dialer{LocalAddr: tcpAddr}
proxy.Tr.Dial = d.Dial

Thanks very much!

ip := net.ParseIP("<IP_ADDRESS>").To4()
tcpAddr := &net.TCPAddr{IP: net.IPv4(ip[0], ip[1], ip[2], ip[3]), Port: 0}
d := net.Dialer{LocalAddr: tcpAddr}
proxy.Tr.Dial = d.Dial


Upload v0.2.0 to Maven Central

Tracking bug to make sure we follow through once I get permission or chat with someone who does. Release bundle is attached.
release_java_v0.2.0.zip

@loosebazooka Would you be able to upload this? This is the recently released 0.2 cut (changelog, main changes were to require inclusion proofs and a few more rekor response fields). I don't have the credentials

will do.

FYI for anyone that comes upon this issue in the future -- the linked release bundle in the first comment is not what we upload. It's a single file in the generated release directory: https://github.com/sigstore/protobuf-specs/tree/main/java#releasing

anyway, the release has been done, I'll close this once it starts showing up on maven central

Also there's a release checklist bug template for tracking releases :D

Ah, to confirm, it's just the JAR, not the zip?

Ah, to confirm, it's just the JAR, not the zip?
yeah something-something-bundle.jar, I'm not actually sure where this zip came from?

anyway, it's up on maven central now: https://repo1.maven.org/maven2/dev/sigstore/protobuf-specs/0.2.0/

this issues can be closed (I do not have permissions)

IIRC sign_and_bundle_release.sh produced something and I zipped it up, but I don't recall.


Connect listener to multiple routing rules

In this document it is said to repeat the creation of a routing rule for 'Video' on the 'Configuration' tab. However, when creating a new routing rule we cannot select the listener anymore as it is already in use by another rule.
How to resolve this?

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

ID: 7c5c94a5-2548-b3a2-3a43-d52bc0892227
Version Independent ID: 01e9c2bc-3265-447c-5d0f-488039c60e87
Content: Tutorial: Create an application gateway with URL path-based routing rules using Azure portal
Content Source: articles/application-gateway/create-url-route-portal.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@DiSae , the tutorial is correct.
In the Configuration tab section, if you check the steps from 7 to 14, you will see that it do not ask you to save/add the whole routing rule before adding the Video path.
It says : Select Add to save the path rule and return to the Add a routing rule tab. Repeat to add another rule for Video.
And lastly select Add to add the routing rule and return to the Configuration tab.
So at step 12, you only add the image path and then click on "Add multiple targets to create a path-based rule" again on the same routing rule and add Video path and once both paths are added, you should click Add at the bottom of the "Add a routing rule". Not before that.

Hope this clarifies.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Ticket/1.6.x/9482 virtual solaris vm

Added correct fact virtual and is_virtual for Solaris VM

Your patch has random whitespace damage.


Fetch initial weighted pool data on-chain

This PR adds an additional pool initializer based on on-chain token data, using only pool addresses, factory and type from the subgraph. This allows the pool initialization to be done without having to index historical events since the dawn of time for networks without archive node support (which causes Balancer pools to not have all the required token data).
This currently doesn't quite work and requires a change to the Balancer subgraph balancer-labs/balancer-subgraph-v2#91
Test Plan
Added unit test for new pool initializer. Waiting to merge pending required subgraph change. We should then be able to trade with old pools on Rinkeby without event indexing.

Balancer subgraph has been updated and now returns the data we want!


make: *** [all] Error 2

General Question
incubator-pegasus-2.5
centos7
gcc --version
gcc (GCC) 13.3.0
gcc --version
gcc (GCC) 13.3.0
源码编译报错如下，请求帮助
[ 80%] Performing install step for 'boost'
[ 80%] Completed 'boost'
[ 80%] Built target boost
make: *** [all] Error 2

First, follow this link to build the environment. If the build is successful, proceed to build incubator-pegasus-2.5 using CentOS 7.

@yokie121 Has this issue been resolved?


[test-framework] fix port forwarder

It was not correctly initializing the output buffer, so the result
began with a bunch of 0's, which cause the post-processing to fail.

It was not correctly initializing the output buffer, so the result
began with a bunch of 0's

Can this happen? I remembered I tested the buffer thing before submitted the change, and did not see this.
But agree that it should be better to limit the length.

@lichuqiang I was definitely seeing this happen.  It's not about limiting the length .... the problem is that the port forwarder does an append operation, so whatever the size of the array is, it was putting the content at the end.  So with an array of length 4096, I'd see 4096 zeros followed by the actual text.

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS [geeknoid]

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

New changes are detected. LGTM label has been removed.


RangeError: Maximum call stack size exceeded

When running on macos or ubuntu we get a RangeError: Maximum call stack size exceeded. On windows it seems to be fine.
We use following definition:
  - name: Test Report
    uses: dorny/test-reporter@v1
    # TODO: apparently the reporter only works on windows
    if: (success() || failure()) && matrix.os == 'windows-latest'
    with:
      name: "Java Tests (OS: ${{ matrix.os }}, Java: ${{ matrix.java }})"
      path: '**/(surefire|failsafe)-reports/TEST-*.xml'
      reporter: java-junit

Is it the glob pattern which throws that error or is it that the reporter just can't parse those outputs?
https://github.com/apache/plc4x/actions/runs/5154522011

Hey, just found this issue and I am also experiencing this with a recent increase in the size of the junit .xml I'm generating inside a GH action workflow.
The previous report suggested that perhaps:

Is it the glob pattern which throws that error or is it that the reporter just can't parse those outputs?

However, in my case the number of generated junit XML files is constant and only their contents/size has changed.
If there is a code area that I can be pointed to I could try and fix this myself...

We hit the same issue with our test suite (over 20,000 tests, run in a matrix that generates 8 jobs) when publishing.
To workaround this issue, we use the following step in our workflow .yml file:
[...]
jobs:
  Report:
    runs-on: ubuntu-latest
    steps:
      - name: 'Add default settings to node calls'
        shell: bash
        run: |
          #
          # The later used dorny/test-reporter@v1 action can throw the following exception when enough tests have been
          # executed:
          #     RangeError: Maximum call stack size exceeded
          #
          # We explicitly increase the max. stack size here to workaround this issue.
          #

          which node
          node --version

          mv /usr/local/bin/node /usr/local/bin/node_org
          echo '#!/bin/bash' >> /usr/local/bin/node
          echo '/usr/local/bin/node_org --stack-size=8192 $@' >> /usr/local/bin/node
          cat /usr/local/bin/node
          chmod +x /usr/local/bin/node
      - name: 'Publish Test Report'
        uses: dorny/test-reporter@v1
[...]


feat: add codeql workflow for code analysis 🔍✨

Close #864

Your review is appreciated @panwar8279 🙌🏻


refactor docs regarding local installation

closes: #946
related: #751
Signed-off-by: Thiago Pagotto<EMAIL_ADDRESS>

I also removed GitLab Runner as it's not used on local, unless I'm mistaken. If there are other tools that should be removed for local, feel free to do so.


Expose generic conditions for use in snippets

Hello,
I'd like to make a snippet with an insert node that defaults to env.LS_SELECT_RAW when luasnip has a selection 'in store', or something else if not.
Grepping the docs shows there is a require"luasnip.extras.conditions.show".has_selected_text 'function' I could use to check if there is a selection available, but it feels weird to use something from a module mentioning show (as if it was specific to show conditions). I would prefer importing something from luasnip.extras.conditions (generic condition objects), or have a handy simple function (not a condition object) for use in snippets.
Could also be used in statusline segment to put an indicator when there is a luasnip text selection available

When implementing these condition 'functions'/objects, we started out with luasnip.extras.conditions. The issue is that currently we have two different places where it makes sense to use these conditions, show_condition and condition (relevant for if the snippet can be expanded), which do not share the same API. Therefore we split up the module in conditions.expand and conditions.show to reflect that difference (... .expand contains everything from ... .show as the API is a superset of the show_condition API).
So the naming most probably won't change for this.
Tbh I'm still not sure what exactly your issue is (is it just the naming/organization in modules?).
In effect the condition objects from luasnip.extras.conditions.{show,expand} are just simple tables {func = function(...) ... end} (with a special metatable attached). So if you really need to, you could try accessing the underlying function via require"luasnip.extras.conditions.show".has_selected_text.func.
But be aware that the contents of that table is kinda an internal representation and might change in the future (although currently I see no reason why it should).

I'd like to make a snippet with an insert node that defaults to env.LS_SELECT_RAW when luasnip has a selection 'in store', or something else if not.

Sorry, completely forgot to answer here, with all the other stuff going on :sweat_smile:
You can use a dynamicNode and check if the select-variables contain text. For example:
d(1, function(_, parent)
	if #parent.snippet.env.LS_SELECT_DEDENT ~= 0 then
		return sn(nil, {
			t(parent.snippet.env.LS_SELECT_DEDENT)
		})
	else
		return sn(nil, { i(1) })
	end
end

Tbh I'm still not sure what exactly your issue is

I think @bew just thought that the condition-function is the only way to check whether there is a selection

note: I actually noticed that using has_selected_text from the extra show conditions module always returns false when inside a snippet for some reason.
If I instead check that snip.env.LS_SELECT_RAW isn't empty before using it that works correctly.
I think having a nice function would be better though, so my snippet code don't need to depend on what the default / empty value of LS_SELECT_RAW is when there are no selection.

note: I actually noticed that using has_selected_text from the extra show conditions module always returns false when inside a snippet for some reason.

Oh, I'll check that out

I think having a nice function would be better though, so my snippet code don't need to depend on what the default / empty value of LS_SELECT_RAW is when there are no selection.

True, hmm..
I think we should add that, but I'm not sure where :D
We don't yet have a module with functions for use inside of dynamic/functionNode, and I'm not sure where to put it/what to call it.
extras/helpers, maybe

d(1, function(_, parent)
	if #parent.snippet.env.LS_SELECT_DEDENT ~= 0 then
		return sn(nil, {
			t(parent.snippet.env.LS_SELECT_DEDENT)
		})
	else
		return sn(nil, { i(1) })
	end
end

This workaround has the flaw that the inserted LS_SELECT_DEDENT won't be selected at the end.
I am looking for something in the sense of "i(1, LS_SELECT_DEDENT)" but the above solution is basically

if LS_SELECT_DEDENT exists then t(LS_SELECT_DEDENT)
else i(1)

which is something different.
I'm migrating from UltiSnips and basically half my snippets use ${1:$VISUAL}}, which is a problem for me.

fyi: With the rework of how selection/restore works from #1013, the way to check for has_stored_selection is now:
function has_stored_seleciton()
  local success, _ =  	local ok, val = pcall(vim.api.nvim_buf_get_var, 0, "LUASNIP_SELECT_RAW")
  return success
end

Actually I've been working with this recently as well. With this comment you made me remember/reread this issue again.
Currently I'm using this function
local function visual_selection(_, parent, _, uarg)
	if #parent.snippet.env.LS_SELECT_RAW > 0 then
		return sn(nil, {t(parent.snippet.env.LS_SELECT_RAW)})
	else
		return sn(nil, {i(unpack(uarg.i))})
	end
end

for example as follows
		stop = isn(2, {d(1, visual_selection, nil, {user_args={{i={1}}}})}, "$PARENT_INDENT\t"),

while it might be worth adding some common function for dynamic nodes as some sort of utils library, I'm not sure if we can generalize them enough (the function above for example also is just for a specific usecase even though one can adjust the insertNode arbitrarily) so that it makes sense to ship those functions to every user. Also if we manage to generalize this, it might be not really easier to use compared to writing own functions for dynamicNodes (after all writing a custom function is peak generalization)

Actually the code I posted above is wrong 👀😬.
Since the luasnip impl will retrieve the selection from buffer vars and immediately after clear those variable, it's no longer possible to check for that buffer variable during the snippet execution (e.g. in a dynamic node).
I think the only reliable way right now is still to check:
    if snip.env.LS_SELECT_RAW and #snip.env.LS_SELECT_RAW > 0 then
      -- ...
    end


Update readme.md

I had copy/pasted the example grammar and was getting tons of errors -- I thought it was my code but it turns out the grammar was using outdated syntax for saving data. Updating that here with something from the tutorials that works in both the browser and Node.js libs.
I also added a link to @v21's Node lib. I have a pending pull request over there that would make it very clear how to install the library via npm.

It's worth noting that I've not updated the npm library to new Tracery yet - that is not quite trivial to do, but is on my list (as then I can upgrade CBDQ)

If there's anything I can do to make porting eaiser, too, lmk. I don't know
what the additional constraints are, but if I know I can make sure I
support them going forward.
On Nov 30, 2015 4:04 PM, "George Buckenham"<EMAIL_ADDRESS>wrote:

It's worth noting that I've not updated the npm library to new Tracery yet

that is not quite trivial to do, but is on my list (as then I can upgrade
CBDQ)

—
Reply to this email directly or view it on GitHub
https://github.com/galaxykate/tracery/pull/8#issuecomment-160654963.

Opened up a new issue to chat about this on: https://github.com/galaxykate/tracery/issues/9


Service token resource can either be a Database or Organization

I'm working on adding support to the CLI for adding organization level accesses.
When this was originally built, we only had database level accesses. All the code assumes a Resource is a database.
This allows a Resource to be either an Organization or a Database. + Custom Unmarshal function for handling.

Closing this, prefer: https://github.com/planetscale/planetscale-go/pull/189


Added European Tracks

This PR adds six tracks.
Track limits (left and right) have been constructed with Google Earth and exported to kml files (included). Both kml are post processed using fastest-lap Python command:
fastest_lap.circuit_preprocessor(options)
to produce a single .xml file included.
Tracks:

Hungaroring
Jarama
Pau ville (city track)
Vallelunga
Varano
Zolder

Hello!
Could you also attach this notebook applied to the tracks you sent? I want to set this as a requisite to collaborators uploading their tracks, but I have forgot to add it to Contributing.md. If it is very inconvenient given the amount of tracks you have submitted, can you at least do it for pau_ville, vallelunga, and zolder? They seem to have large errors, though I am aware that the error measure is wrong for some tracks and the issue might be on my side.
Thank you very much for your very nice contribution, Pierre

I don't have Jupyter running but I can find all the i_start and i_end for every corner and add all the plot commands in a circuit_preprocessor.ipynb. Would that be good?
You say the tracks have errors, what kind of error? What should I look for when plotting the tracks?

In each of the XML files produced there's a header with this information
    <header>
        <track_length units="m">3995.6606035603131</track_length>
        <L2_error_left>3.746581096896739</L2_error_left>
        <L2_error_right>1.9467386462668475</L2_error_right>
        <max_error_left>84.101605320233034</max_error_left>
        <max_error_right>61.549736514958695</max_error_right>
    </header>

This error is the distance between the approximation of the track through polynomials (what circuit_preprocessor creates) and the real KML files. I am aware that with certain geometries, the computation of the error is incorrect, which is probably the case. I will open an issue about this. This is why I want to see the jupyter notebooks. In these notebooks, I plot the KML paths against the circuit_preprocessor approximation to see if they match properly.
If you cannot install Jupyter notebook, it is good enough if you can give me the i_start and i_end in notebooks and then I can run them myself, as you mention. Please call these notebooks as the name of the track (e.g. jarama.ipynb).
Thanks a lot!
Juan.

Yes, the correct syntax is
circuit = fastest_lap.load_track("../database/tracks/zolder/zolder.xml", track)

I saw that load_track returns this value but in order to generate all the plots, I don't only need:
s = fastest_lap.load_track("../database/tracks/zolder/zolder.xml", track)
But also:
x = fastest_lap.download_vector("track/centerline/x")
y = fastest_lap.download_vector("track/centerline/y")
x_left = fastest_lap.download_vector("track/left/x")
y_left = fastest_lap.download_vector("track/left/y")
x_right = fastest_lap.download_vector("track/right/x")
y_right = fastest_lap.download_vector("track/right/y")
x_left_measured = fastest_lap.download_vector("track/left_measured/x")
y_left_measured = fastest_lap.download_vector("track/left_measured/y")
x_right_measured = fastest_lap.download_vector("track/right_measured/x")
y_right_measured = fastest_lap.download_vector("track/right_measured/y")
kappa = fastest_lap.download_vector("track/kappa")
nl = fastest_lap.download_vector("track/nl")
nr = fastest_lap.download_vector("track/nr")
How do I get those?

Sorry again. I have opened an issue on this matter too.

No worries, I understand that everything is still under construction.
I'm happy to help improve the tool

New commit with the Notebooks for all tracks + Sachsenring.
Let me know if it's all okay.

Amazing! Thank you very much Pierre


updated queries.js

working on updating queries.js to allow data for all pages, not just home page fridge list

Got a working auth login. Was able to login to a seeded profile. Successful redirect after login to home page which rendered the fridge list.


Partners page

Here is a first stab at the partners page. I have several questions/comments:
According to @joshfraser, this page will replace  /build-on-origin.

Should the URL path change to something like /partners, or keep it the same for SEO continuity?
Does there need to be a nav link for this page?
I tried to create this branch from develop, but some past hotfixes to master were necessary to have in this branch. Should I be branching from develop and how should I deal with differences between develop and master when, for example, style.css is different in all three branches?
10538a55a21a98587cdaa99fb8b6ce489ab742b3 updates FontAwesome to a newer version, which seems to have been necessary to get some icons that I needed. Is that alright? I believe that nothing is broken as a result. But should that update have been its own commit?
I noticed that the form styles on the existing build-on-origin page (and my update) do not match the email form on the landing page nor Aure's designs. I can update those in a future commit.
The existing build-on-origin had no rtl language support. I have not yet added that, but will in a future commit.
Is there a good way for me to test locally to see that the form has not been broken?
I believe that there are no translations for this page unless I failed to connect them correctly.
I think I saw some mention of a possible future design/development environment to run alongside the staging environment. It might be good to have something like that if there is not already so that I can get real-world feedback from Aure before you merge and deploy to staging.

I tried to create this branch from develop, but some past hotfixes to master were necessary to have in this branch. Should I be branching from develop and how should I deal with differences between develop and master when, for example, style.css is different in all three branches?

My bad on develop falling behind--now updated. I'd actually say we should develop this in a topic/partners-page branch. Also fine to just point us to branches in your fork in mean time. Sound good?

10538a5 updates FontAwesome to a newer version, which seems to have been necessary to get some icons that I needed. Is that alright? I believe that nothing is broken as a result. But should that update have been its own commit?

New versions always good. I'll check it from a security standpoint is all.

I noticed that the form styles on the existing build-on-origin page (and my update) do not match the email form on the landing page nor Aure's designs. I can update those in a future commit.

Awesome.

The existing build-on-origin had no rtl language support. I have not yet added that, but will in a future commit.

Good eye.

Is there a good way for me to test locally to see that the form has not been broken?

Not at this time; no way to test the emails actually send. This may be a good time for us to set up a testing version of sendgrid. I hope this isn't a blocker right away. We can always test on staging.

I believe that there are no translations for this page unless I failed to connect them correctly.

Nope. Just be sure all English is wrapped in gettext() so that babel-extract can find them. translations/README.md has docs on how to do the extraction and update the messages.pot file. I can walk you through that, and even getting that up into Google's translation tool to get first-pass machine translation. (I'd love to have someone else able to do that..it's kind of tedius. ;) )
Then we'll have to ping the gracious people in slack #translation channel to help with any new text. :)

I think I saw some mention of a possible future design/development environment to run alongside the staging environment. It might be good to have something like that if there is not already so that I can get real-world feedback from Aure before you merge and deploy to staging.

For sure. But really, the best (and fastest) thing would be for you to create your own free Heroku instance and deploy there. The free app is more than enough for testing. It will give you a crazy looking domain with herokuapp in it, but that's fine for testing and designer feedback. I can walk you through that too.
You'll need to

Install heroku command line tools: https://devcenter.heroku.com/articles/heroku-cli
Add heroku as remote: https://devcenter.heroku.com/articles/git
Push your dev branch as to master on heroku (it only deploys from master): https://stackoverflow.com/a/8123465/59913

So great to have you helping out!

@wanderingstan great, that all makes sense. Is it appropriate for me to...
a) close this PR
b) delete my branch and create a topic branch from develop
c) create a new PR when everything is ready to go
I generally know how to git but have never actually collaborated on a team with multiple branches and forks. 🙉

Totally.
You could also just rename your branch to topic/partners-page or issue/79_partners_page or similar, and then do PR to develop when its in good shape. (I.e. doesn't matter what you name things in your fork.)
We're still figuring out our git flow here as well!

We are abandoning the social icons on the partners page, so I will rollback the FontAwesome update. I don't think that it makes sense to update it on this branch and unnecessarily risk some consequences.

Closing in favor of #85


Add manifest to the current chunk

Correctly add the emitted manifest file to the current chunk. This, for example, fixes Webpack statistics where the manifest wouldn't be there in assetsByChunkName otherwise.

Thanks :)

https://github.com/soundcloud/chunk-manifest-webpack-plugin/issues/47#issuecomment-323390951 - is it caused by this patch?


Change GitHub Logo to White when shifting to dark-mode.

Change GitHub Logo (in join community) to White when shifting to dark-mode as it is not properly visible.
@DeveloperAshish8 I would like to work on it.
Please assign the issue to me.

Update:
@DeveloperAshish8 I have worked on the issue and its ready, if you assign me this issue i can quickly make a PR.

Please share some screenshots for better understanding.

Before:

After:

It will change according to light/dark mode

Hey @DeveloperAshish8 any update? I posted the screenshots.

@DeveloperAshish8 ??????

Ok create a PR for the same @Vedant-Manjrekar

Ok create a PR for the same @Vedant-Manjrekar

@DeveloperAshish8 Done 👍🏽


Thanks a lot, but could you please check the spelling before publishing?

seperates read and update operations
the flexability created by

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

ID: 0c0ec92e-c237-f3a6-1835-f2078486dcb3
Version Independent ID: f597c9a2-f18f-3722-57f9-acd535472b0b
Content: CQRS pattern - Azure Architecture Center
Content Source: docs/patterns/cqrs.md
Service: architecture-center
Sub-service: cloud-fundamentals
GitHub Login: @dragon119
Microsoft Alias: pnp

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#


A question above negative loss calculation

Hi, Thanks for this awesome work, I'm trying to train it on my own dataset, so I am exploring how to organize my data. I notice that for unmatched images, you calculate their matching loss by rolling a batch of images(https://github.com/ignacio-rocco/ncnet/blob/master/train.py#L137):
    batch['source_image']=batch['source_image'][np.roll(np.arange(b),-1),:] # roll

I don't quite get why moving each sample one position forward creates unmatched pairs, how do you organize your data? Also, is the set information in the csv files useful during training? Thanks!

I also have this question. I didn't see how rolling by one position forward guarantee the images are from different sets which is how negative is defined. By visualizing the sets numbers inside a batch, if seems true that mosts pairs will go to different sets if you roll the order by 1 forward. But still if the several consecutive pairs have the same set number. Do you simply ignored this case?
I am looking forward to hearing from your reply!!! Thanks!