"Python3.6 cannot be opened" on Insiders

Environment data
VS Code version:  Version 1.20.0-insider (1.20.0-insider)
Python Extension version:  0.9.1
Python Version:   3.6 in my venv, installed through homebrew
OS and version:   MacOS 10.13.2
Actual behavior
"python3.6 cannot be opened because of a problem.". I appear to be receiving this error in the last couple weeks. Pylint still seems to work, though. I am using a virtualenv in vscode, so maybe this is referencing a non-venv python?
Expected  behavior
I don't receive multiple error messages about python3.6 launching.
Steps to reproduce:

Open VSCode.

Logs
Output from Python output panel
##########Linting Output - pylint##########
Using config file /Users/dfee/code/native/.pylintrc

--------------------------------------------------------------------
Your code has been rated at 10.00/10 (previous run: 10.00/10, +0.00)

Output from Console window (Help->Developer Tools menu)
/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:2693 [Extension Host] debugger listening on port 9333
/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:2691 [Extension Host] debugger inspector at %cDebugger listening on port 9333.
Warning: This is an experimental feature and could change at any time.
To start debugging, open the following URL in Chrome:
    chrome-devtools://devtools/bundled/inspector.html?experiments=true&v8only=true&ws=<IP_ADDRESS>:9333/c66d619f-b86d-4fe2-80d6-36ef4f63b38a

/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:1312  INFO SearchService#search rg --files --hidden --case-sensitive -g '/tsconfig.json' -g '!**/node_modules/**' -g '!**/.git' -g '!**/.svn' -g '!**/.hg' -g '!**/CVS' -g '!**/.DS_Store' -g '!**/.cache' -g '!**/.eggs' -g '!**/*.egg-info' -g '!**/*.pyc' -g '!**/__pycache__' -g '!**/coverage' --no-ignore --follow -- '.'
 - cwd: /Users/dfee/code/native
 - Sibling clauses: {}

console.ts:123 [Extension Host] (node:11197) DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead. (at process.on (internal/process/warning.js:21:17))

Output from MacOS Error Console
Process:               python3.6 [9682]
Path:                  /Users/USER/*/python3.6
Identifier:            python3.6
Version:               ???
Code Type:             X86-64 (Native)
Parent Process:        Code - Insiders Helper [9600]
Responsible:           python3.6 [9682]
User ID:               501

Date/Time:             2018-01-23 10:54:10.573 -0800
OS Version:            Mac OS X 10.13.2 (17C205)
Report Version:        12
Anonymous UUID:        DC05A83E-CBDC-142B-BB04-36D21E83A6AF

Sleep/Wake UUID:       D4F5DC0F-AF1F-4AE4-87D7-6B8DDF774450

Time Awake Since Boot: 200000 seconds
Time Since Wake:       1100 seconds

System Integrity Protection: enabled

Crashed Thread:        0

Exception Type:        EXC_CRASH (SIGABRT)
Exception Codes:       0x0000000000000000, 0x0000000000000000
Exception Note:        EXC_CORPSE_NOTIFY

Termination Reason:    DYLD, [0x1] Library missing

Application Specific Information:
dyld: launch, loading dependent libraries

Dyld Error Message:
  Library not loaded: @executable_path/../.Python
  Referenced from: /Users/USER/*/python
  Reason: image not found

Binary Images:
       0x10c478000 -        0x10c479fff +python (0) <CD809507-2F54-3EE5-AF72-E9E2B654BA17> /Users/USER/*/python
       0x1134b2000 -        0x1134fc98f  dyld (519.2.2) <6695F30B-4E88-3C0B-9867-7D738C44A3E6> /usr/lib/dyld
    0x7fff469cc000 -     0x7fff46e65fff  com.apple.CoreFoundation (6.9 - 1450.16) <DD253147-5720-3B3B-B222-77FD1419C400> /System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation

Model: MacBookPro6,2, BootROM MBP61.005A.B00, 2 processors, Intel Core i5, 2.4 GHz, 8 GB, SMC 1.58f17
Graphics: Intel HD Graphics, Intel HD Graphics, Built-In
Graphics: NVIDIA GeForce GT 330M, NVIDIA GeForce GT 330M, PCIe, 256 MB
Memory Module: BANK 0/DIMM0, 4 GB, DDR3, 1067 MHz, 0x859B, 0x435434473353313036374D2E433136460000
Memory Module: BANK 1/DIMM0, 4 GB, DDR3, 1067 MHz, 0x859B, 0x435434473353313036374D2E433136460000
AirPort: spairport_wireless_card_type_airport_extreme (0x14E4, 0x93), Broadcom BCM43xx 1.0 (<IP_ADDRESS>.30)
Bluetooth: Version 6.0.2f2, 3 services, 27 devices, 1 incoming serial ports
Network Service: Wi-Fi, AirPort, en1
Serial ATA Device: M4-CT256M4SSD2, 256.06 GB
Serial ATA Device: MATSHITADVD-R   UJ-898
USB Device: USB 2.0 Bus
USB Device: Hub
USB Device: IR Receiver
USB Device: Built-in iSight
USB Device: USB 2.0 Bus
USB Device: Hub
USB Device: Internal Memory Card Reader
USB Device: Apple Internal Keyboard / Trackpad
USB Device: BRCM2070 Hub
USB Device: Bluetooth USB Host Controller
Thunderbolt Bus:

I installed Python 3.6 through homebrew.
Versions:
$ python --version
Python 3.6.4
$ which python
/usr/bin/python

$ python3 --version
Python 3.6.4
$ which python3
/usr/local/bin/python3

I just opened the regular VSCode and it didn't crash. So I installed the update. Still, no crash. Then, I reloaded VSCode after it updated the Python extension and I got the message about Python3.6 quitting unexpectedly again.
I don't have a PYTHONPATH env var set.
Here is my sys.path after opening it:
+>>> pprint.pprint(sys.path)
['',
 '/Library/Python/2.7/site-packages/jsonschema-2.5.1-py2.7.egg',
 '/Library/Python/2.7/site-packages/functools32-3.2.3_2-py2.7.egg',
 '/Library/Python/2.7/site-packages/docker_compose-1.6.0dev-py2.7.egg',
 '/Library/Python/2.7/site-packages',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python27.zip',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plat-darwin',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plat-mac',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plat-mac/lib-scriptpackages',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/lib-tk',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/lib-old',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/lib-dynload',
 '/Library/Python/2.7/site-packages',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/PyObjC']

This also seems to have no effect:

I think you may have mis-pasted something as you said /usr/bin/python was Python 3.6.4 but that output for sys.path is for Python 2.7. In the newest version of the release we actually don't recommend people use the macOS-system Python. So I would select /usr/local/bin/python3 as your Python.

Yes – I must've messed something up when determining the Python version. Here we go –
$ python
Python 2.7.10 (default, Jul 15 2017, 17:16:57)
[GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-900.0.31)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
+>>> import sys, pprint
+>>> pprint.pprint(sys.path)
['',
 '/Library/Python/2.7/site-packages/jsonschema-2.5.1-py2.7.egg',
 '/Library/Python/2.7/site-packages/functools32-3.2.3_2-py2.7.egg',
 '/Library/Python/2.7/site-packages/docker_compose-1.6.0dev-py2.7.egg',
 '/Library/Python/2.7/site-packages',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python27.zip',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plat-darwin',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plat-mac',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plat-mac/lib-scriptpackages',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/lib-tk',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/lib-old',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/lib-dynload',
 '/Library/Python/2.7/site-packages',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python',
 '/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/PyObjC']
+>>>

As for your remark about using the macOS-system python … I don't understand what you mean. The crash I'm getting is for Python3.6 – I don't think macOS ships with Python3, so I'm unclear on what your recommendation is.

@brettcannon is there anything other information I can provide?

Sorry, your previous message came while I was on vacation.
My comment about the system Python is your screenshot shows you specified /usr/bin/python which is the system Python that comes on your Mac and is Python 2. We do not recommend people use it directly, so you should specify /usr/local/bin/python3 to use the Homebrew copy of Python. And have you tried uninstalling and re-installing your Python installation from Homebrew?

I'm unclear what you mean when you say I shouldn't use Python 2 directly. I'm not using it directly.
I launch VSCode, open a .py file, and get an alert that Python 3.6 crashed.
Python 3.6 works fine on my Mac – I develop with it constantly. In fact, after VSCode crashes a Python3 interpreter a couple times, I get no more messages about it, linting works fine, etc.

https://github.com/Microsoft/vscode-python/issues/616#issuecomment-362487878 is a screenshot of what looks like a settings.json file with your Python path explicitly set to Python 2.
And have you tried turning off all other extensions except the Python extension? And you are running the latest 2018.1.0 release of the extension?

@brettcannon I had set it to that (explicitly) because you explicitly instructed me to do so https://github.com/Microsoft/vscode-python/issues/616#issuecomment-360627891
Indeed, it is currently set to

And to be clear, a user should not need to configure a python path at all.
If VSCode crashes a python process a handful of times upon launch when the python plugin is installed, VSCode should probably consider bundling a python binary, or supporting the natively installed python.
However, setting "python.pythonPath": "python3.6" has no impact on the crashes, and having an explicitly set "python.pythonPath": "/usr/bin/python" (i.e. MacOS's python2.7) also has no impact on a crashing python3.6. This leads me to believe that configuration value is not involved in this problem.

Please set python.pythonPath to a full path and not python3.6 (this has been known to cause issues).
I realize this is frustrating, but I'm unfortunately running out of ideas on how to help solve this as no one else is reporting this issue (including me on my own Mac w/ Python 3.6 installed via Homebrew). And shipping Python in the extension is not currently an option as that gets rather complicated due to compilation flags, etc. that are system-specific. And supporting the Python install on Mac has caused us nothing but grief because Apple's build of Python is non-standard (we do support system Python installs on Linux).
Otherwise I'm curious to know what happens if you turn off all other extensions and if you re-install Python since we are not running any custom extension modules in the extension which would lead to such a failure.

I set the pythonPath to an abspath and still got the error.

I guess I'll need to find some time to perform a re-installation of python3.

At this point by best guess is your Python install is somehow corrupted and we're triggering the corruption by running some code on our end. Let us know if a re-install solves it.

Because it has been more than a month since we requested more info, I'm closing this issue. If you are able to provide the info later then please do so and we can re-open the issue.


dynamically routing pagination on page '/votelog'.

related to #86

@karnpapon Thanks for your PR. Will look into this soon.

Thanks a lot @karnpapon


Index refactoring

Main changes:

Abstracted how we generate a Lucene document for indexing from a source document into the notion of a generator.
Cleaned up code on multithreaded indexing into a new class called MultithreadedIndexer. The previous implementation used ThreadPoolExecutor in a janky way - cleaned all of that up.
Now we pass around IndexArgs to the various classes that need configuring - simplifies.

Tested and it works fine.
Only one concern: do you think it is better to put all the Generators to another folder?

Addressed - moved to io.anserini.index.generator package.


Automatically deploy semapps.org and dev.semapps.org on Github pages

See how it is done for https://github.com/assemblee-virtuelle/semantic-bus-documentation

Unfortunately Github pages only allow a single deployment / domain name.


Remove element from builtins

Removes 'Element' from the built-in types and adopts existing unit tests.
A Test is added to verify that Element can be part of a model hierarchy.

Closes #30.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Frank Panno seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Before we can consider to merge this PR or work with it in any way we'd need you to sign the CLI. Please adjust your commits to use an email address you signed the CLA with, if you signed it already.

Won't fix, cf. https://github.com/bpmn-io/moddle/issues/30#issuecomment-828357210.


using css in stylish

Hi Dylan,
today i tried setting my startpage to use wal colors via stylish in Firefox. The method described in the wiki does not work. Stylish expects a Hex value and gives a warning when i try to use var ( --background)
Can you point me in the right direction?

Can you send me a copy of your stylesheet?

Hi, I am having the same issue. In fact most web pages default to white if I try to import the colors.css file from pywal. Any advice?

Have you tried asking in the discord channels? maybe the community might be
able to help you out.
-- att. Fernando Vasquez --
On Thu, Jul 5, 2018 at 11:23 AM, Cimaje Horace<EMAIL_ADDRESS>wrote:

Hi, I am having the same issue. In fact most web pages default to white if
I try to import the colors.css file from pywal. Any advice?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dylanaraps/pywal/issues/171#issuecomment-402794746,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKojSdHgYTv999f6BL-3jEO1yyJFpp2tks5uDkuBgaJpZM4RuvP_
.

how was it fixed?


move HTML automatic closing tags immediately after placing

I love that brackets automatically creates closing tags in html when you close the opening tag; however, I sometimes need to add tags to existing content. When this happens, the auto-close tag does not wrap around the content as needed. I would love to be able to hold ctrl and right arrow to move the closing tag immediately when it is placed.

@AbleLincoln We have this item in the backlog. Please upvote this here @TomMalbran Are you working on this PR to modify it?

@AbleLincoln Meanwhile extension Brackets Surround may be useful for you.

@prafulVaishnav I haven't touched that PR in a really long time. But I am guessing that it might still work after fixing the merge issues. But I don't know how it might behave with multiple cursor. But that PR works only with html, and I thought that the team wanted something less language specific.


abolished cytokinesis with stable actinomyosin ring

could this be a related synonym for
abolished actomyosin contractile ring contraction

sure, there's more flexibility with related synonyms than exact ones
done in
edit file: 5db139c03ed753121711c62c481a369bf4b04bb7
release: 9431b22e28d71d174a057ee0fac5591aac32b6ad


Fix broken getPrFileNames() function

Background

The current getPrFileNames() function doesn't work.
https://github.com/:owner/:repo/pull/:number.diff doesn't accept Authorization token.
Closes #19

Problem Solving

https://stackoverflow.com/questions/35471316/using-curl-how-do-i-download-a-diff-file-from-a-private-repository-on-github
Use https://api.github.com/repos/:owner/:repo/pulls/:number API
Tested with my project. This works well.

Known Issues

github.com/waigani/diffparser package cannot parse correctly with renamed diff.

@srvaroa Could you add a label hacktoberfest-accepted to this PR for Hacktoberfest again? 😆


Add second commit on wrong branch

In addition to the reverted-merge kata, I have also created a second kata that explores how you can move a commit that has accidentally landed on the wrong branch to the correct branch.
The existing kata has its value in that it shows how to deal with putting a commit in between two others by rebasing (at least that is a solution that I found). However I find that there is at least a second situation where you have accidentally committed on the wrong branch and where you cannot easily use rebasing, but can use stash and reset instead.
I'd value any comments that you have on this exercise and be glad if you'd accept my contribution.
Perhaps you also have a better name than putting a "2" at the end...

Hi,
Thank you so much for your contribution, it is always good to see multiple perspectives on similar tasks!
I've rephrased one of your useful commands, to clarify that reset does not actually remove a commit from the repository.


[1.1.1] Test genpkey app for EC keygen with various args

This is the equivalent of #12080 for the 1.1.1 branch.
The difference here, and the reason why this should be evaluated on its own and is already out of WIP is that in 1.1.1 these new tests are already passing, while #12080 is held back until the issue they highlight is fixed.
Notice also that this PR already adds new aliases and makes testing on SM2 dependent on enabling the sm2 feature during config.
Credits
Thanks to @bbbrumley and the rest of the team behind the OpenSSL Triggerflow CI (paper) that detected and reported the original issue that led to the development of these new tests.
Checklist

[x] tests are added or updated
[x] add more curve aliases

Can I get also another @openssl/otc confirmation that it is fine in this case to go ahead and include these new tests in 1.1.1 even if they are not included in master yet?
As stated in the description, the reason for the exemption from the normal procedure of "first in master, then backport" is that master is currently broken w.r.t. the aspect of CLI usage tested by the new tests, so #12080 will stay in WIP until we decide how to fix master.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

So, we discussed #12080 in a conf call: for now the new tests that are failing are going to be annotated with a TODO label, and a separate issue will be opened to address the issue before the final release of 3.0.0.
The TODO annotation is not useful in 1.1.1, so that change won't be included in this PR, which effectively becomes independent from #12080 for now.
So this is now ready to merge!


pkg-config .pc files for boost?

Hey guys,
Not sure if this is the correct issue tracker but perhaps someone who has
some project-decision-making process may read this.
Would it be possible to provide a boost .pc file, to query the
current boost version? In particular the game wesnoth needs
a specific boost version and I tend to compile the latest boost
and then I have to compile an older one for wesnoth. I can
determine the version of boost installed in other ways, sure
(read the .h file for instance or whether it was .hccp I forgot)
but a .pc file would be so much cleaner.
Hopefully someone reads this but thanks for reading anyway
and feel free to close this issue at any moment in time.

Probably a better place to mention this is the boost-developers' mailing list<EMAIL_ADDRESS>


Updated More Lenses to latest version

Adds Naturalist lens, which is similar to #562, except it does not calculate if park can be built if a tile is bought or improvement is removed.
Combined City Overlap lenses to one.
Adds side panels to Resource Lens and City Overlap Lens.
Fixed Prereq techs and civics not being checked in builder lens in some instances.

I'm going to wait on @pspjuth's PR to develop a bit before merging the update. Ideally I'd like to avoid creating extra work for everyone involved

I closed this and pushing the latest version of ML (with the new version of Naturalist lens) in the summer-2017 branch.


Fixing dynamic transaction check failure.

Proposed changes
Availing revert to None as a prior strategy to avoid failure of dynamic transaction check at web3._utils.transactions.fill_transaction_defaults.
Fixes
#28
Types of changes
What types of changes does your code introduce to agents-aea?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the develop branch (left side). Also you should start your branch off our develop.
[ ] Lint and unit tests pass locally with my changes and CI passes too
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that code coverage does not decrease.
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #29 (819806b) into main (1a45da1) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #29   +/-   ##
=======================================
  Coverage   98.20%   98.21%
=======================================
  Files         313      313
  Lines       24875    24880    +5
=======================================
+ Hits        24429    24435    +6
+ Misses        446      445    -1

Flag
Coverage Δ

unittests
98.21% <100.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ackages/fetchai/protocols/contract_api/__init__.py
100.00% <ø> (ø)

...ckages/fetchai/protocols/contract_api/dialogues.py
100.00% <ø> (ø)

packages/fetchai/protocols/contract_api/message.py
100.00% <ø> (ø)

...es/fetchai/protocols/contract_api/serialization.py
100.00% <ø> (ø)

packages/fetchai/protocols/default/__init__.py
100.00% <ø> (ø)

packages/fetchai/protocols/default/dialogues.py
100.00% <ø> (ø)

packages/fetchai/protocols/default/message.py
100.00% <ø> (ø)

...ackages/fetchai/protocols/default/serialization.py
100.00% <ø> (ø)

packages/fetchai/protocols/fipa/__init__.py
100.00% <ø> (ø)

packages/fetchai/protocols/fipa/dialogues.py
100.00% <ø> (ø)

... and 27 more

Continue to review full report at Codecov.

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


MHD_058: add support for Binary or Bundle to support FHIR-Documents better

MHD_058: The profile requires that the document submitted is encoded in a FHIR Binary. Is there interest in also allowing a Bundle of type Document? This would be useful when publishing FHIR-Documents. The FHIR-Document would still need to be seralized into a Bundle of type Document, but that Bundle would not need to be further encoded into a Binary (e.g. base64 encoding). Note that the mime-type in this case would be forced to be the same mime-type as the ITI-65 Bundle, where a Document Source wants to encode ITI-65 in a mime-type that is different than the document, the Binary methodology would need to be used.

a Document Responder that is not MHDS, would need to address what happens with Document Bundles. would that be deterministic?
Would all Responders need to support Document Bundles?
Would there be any limits?

Or a Document Responder that is grouped with XDR/XCDR. Good point. Adds more work for the responder for minimal benefit. Let's discuss.

Leave as an Open issue
Look at Retrieve to see if text could be added explaining likely Retrieve behavior when the URL is a FHIR-Document and fhir mime types are in the GET request. Could it, Should it, WILL it, return the Document Bundle?


How reasonable would it be to have Identity a be Similar to a?

From further investigations regarding issues raised in #24, I found that a possible solution to all my concerns would be the ability to have an instance Similar (Identity a) a. That would mean that I could recover the older flexibility of over2 by writing unpack . over2. However, as I don't know the details of the implementation, I'm not sure if this is feasible or if it would break some other assumptions.

No, I don't think that can work very well. Using overlapping instances will definitely cause trouble for typed holes (even worse error messages! Yay!). Using incoherent instances will work okay in many cases, but I'm not sure how reliable it is and it limits some other potential future changes. Additionally, ad hoc instances here seem likely to confuse a lot of people. I'm not a fan. I'd much rather add new combinators to get the power you need, and resurrect the old API.

Yeah, in retrospect, this doesn't even address the problem I thought it did. Thank you for clarifying though - it'll help me come up with a good motivating example explaining what I need more clearly.


Inverse path properties

Provide a configurable means to specify inverse paths to related objects when rendering.

Can you give an example in RDF?
Do you mean something like this: if I have a Feature in a Feature Collection, but only FC -> feature specified with rdfs:member in the data, I could specify an inverse like ex:memberOf. But how would I know to build that inverse property out?
If you just make up an inverse now like ex:memberOf and add it to data, it will show up in object rendering as a property.
We could implement a set of common inverses, like the above for GeoSPARQL.

Well a classic example for be to find observations on a featureOfInterest using the SOSA model.  Not a fan of calculating every possible inverse - as many models do not define suitable inverse properties.

would be better to have configurable rather than "common" inverses in the design.  And probably active vs lazy options for navigation and rendering.

This could be done by extending the profiles to allow "sh:construct" - prez would then append this SPARQL CONSTRUCT query to the default SPARQL query for an object.
For example if the standard object query is along the lines of:
PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>

CONSTRUCT {?s ?p ?o .
  ?o ?p2 ?o2 .
  ?o2 ?p3 ?o3 }
WHERE {
  BIND(<feature_uri> AS ?s)
  ?s !rdfs:member ?o ;
      ?p ?o
  OPTIONAL {
    FILTER(ISBLANK(?o))
    ?o ?p2 ?o2 ;
       OPTIONAL {
      FILTER(ISBLANK(?o2))
      ?o2 ?p3 ?o3 ;
          }
  }
}

Additions to the query, for example for inverse paths, of the form
CONSTRUCT {?s ?p ?o}
WHERE {
?s ?p ?o ;
     sosa:hasFeatureOfInterest <feature_uri>}

Can be merged (in Prez, using Python) with UNION, e.g.:
PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>

CONSTRUCT {?s ?p ?o .
  ?o ?p2 ?o2 .
  ?o2 ?p3 ?o3 }
WHERE
{
 {
  BIND(<feature_uri> AS ?s)
  ?s !rdfs:member ?o ;
      ?p ?o
  OPTIONAL {
    FILTER(ISBLANK(?o))
    ?o ?p2 ?o2 ;
       OPTIONAL {
      FILTER(ISBLANK(?o2))
      ?o2 ?p3 ?o3 ;
          }
   }
 }
UNION
 {
  BIND(<feature_uri> AS ?s)
  ?s ?p ?o ;
       sosa:hasFeatureOfInterest <feature_uri>
 }
}

To be clear the sh:construct would be added to the profiles such as this example: https://github.com/surroundaustralia/ga-spaceprez-profile/blob/main/ga-profile.ttl

Not sure where exactly in the profile the `sh:construct` would go, perhaps under a node shape. The SHACL spec might give guidance here

thats my thinking too - property paths with inverse are declarative - turning these into constructs ought to be implementation.
This is important because we can query declarative paths - e.g. to find all outputs that include a particular path content -whereas looking inside SPARQL is nastier.  Also, decomposing SPARQL to take advantage of cached object details would be hard.

OK so the profile would have something like "sh:inversePath", which could get translated in to a CONSTRUCT by Prez?

Implemented, see https://github.com/RDFLib/prez/blob/546e1f2184506a80f53741831b36afe4e22da28b/prez/services/sparql_new.py#L19 and https://github.com/RDFLib/prez/blob/546e1f2184506a80f53741831b36afe4e22da28b/prez/services/sparql_new.py#L360


require admin privileges if the application is running from a regular user

We have written the needed data into your clipboard because it was too large to send. Please paste.

https://prnt.sc/qahbtw

https://prnt.sc/qahbya
https://prnt.sc/qahc37
https://prnt.sc/qahc6l
https://prnt.sc/qahcac
etc...

ошибка 5: Отказано в доступе => error 5: Access denied

https://prnt.sc/qaha38
https://prnt.sc/qaha87
https://prnt.sc/qahabh
https://prnt.sc/qahaea
https://prnt.sc/qahahd
https://prnt.sc/qahajw
https://prnt.sc/qahanl
https://prnt.sc/qahar4
etc...
VS Code version: Code 1.40.2 (f359dd69833dd8800b54d458f6d37ab7c78df520, 2019-11-25T14:54:45.096Z)
OS version: Windows_NT x64 10.0.17763

Please provide reproducible steps and explanations in English.


Team 4 projectiles

changes Include:
projectile factory
attack component
projectile shoot action
disposal
tests
Issues:
closes #33
closes #34

Please fix the merge conficts


How to change from sessionStorage to localStorage on compiled version (dist)

Basically what the title says: how can I change from sessionStorage to localStorage on the compiled version? Because I followed and tried with userStore: new WebStorageStateStore({ store: window.localStorage }) as explained in the wiki but thats obviously for the ES6 version and doesn't work on the compiled version I need to use (there's no WebStorageStateStore).
Or perhaps I'm just overseeing something.

Have you tried using the following: userStore: new Oidc.WebStorageStateStore({ store: window.localStorage }). If you are using the files from the dist folder (UMD/AMD way), this is the way to go

I just tried and it doesn't even pick up the current user; as if not logged in. But the IdentityServer shows me as logged in. After I commented out the line, it picked up the user again as logged in.

I made this work.
I had to add the options configuration in every constructor for the different callbacks used on the redirect pages.
So, for example, in the signinRedirectCallback I wrote it like this:
new Oidc.UserManager({ userStore: new Oidc.WebStorageStateStore({ store: window.localStorage }) }).signinRedirectCallback().then(function () {
            window.location.replace("./");
        });

And now this works without problem.
One question left is: why using sessionStorage by default instead of localStorage? I found it more useful the later because it keeps the data in between tabs and windows...

I think this is because sessionStorage is persistant as long as the browser window exists. Once you close it, this is cleared. In most cases this is the life time of your user sesssion. In most cases you want the user to relogin after he has closed the browser.

Yeah, what I think is odd is that lots of users might want to open a link on another tab or just another window and it looks kind of nasty that he/she has to login again just because of that. Also, it renders useless the "Remember me" option as it will never "remember" you...
Anyway, that's just my point of view as another use case for devs to take into consideration :)

All set on this issue?

Yeah, I left it configured as I explained above. Only doubt would be for you to confirm if everything is correct (like having to configure the option in every redirect page callback or if I'm overseeing something). Other than that I'm done. Thanks.

Yes, if you're changing the storage config then on the callback page you'd need to make the same config change.

Yeah I agree that it's weird that localStorage isn't default. Multi-tab usage is super common nowadays.

I avoided it because it stored PII to disk. I had customers who could not allow that, so I made session storage the default. Also, when you use a shared store, you then run into problems where multiple tabs might think they need to renew the token, so they're all competing to update the same thing. I didn't try to solve that, as it felt like it would be complicated.

Ok it's fine not being default, but I really think it should be a well supported config option. The current default behaviour I feel like is very surprising for new users of the library.

you then run into problems where multiple tabs might think they need to renew the token, so they're all competing to update the same thing

Yeah I don't know what's the cleanest way to do this, but I think figuring this out for the user should be one of the central things that this module does.


GeoLitePrivate2-City.mmdb doesn't exist

to use the geoip enrichment, you need to files, specifically
          database => "/mnt/s3fs_geoip/GeoLite2-City.mmdb"
          database => "/mnt/s3fs_geoip/GeoLitePrivate2-City.mmdb"

unfortunately seems like GeoLitePrivate2-City.mmdb doesn't exist anywhere in the internet and maxmind only provides

GeoLite2-ASN.mmdb
GeoLite2-City.mmdb
GeoLite2-Country.mmdb

i'd expect that either more information on where to find GeoLitePrivate2-City.mmdb is added to the documentation or the enrichment pipeline is updated to function without that file

Hi,
maybe i wasn't clear.. when you subscribe and download the geoip databases, you can download only

GeoLite2-ASN.mmdb
GeoLite2-City.mmdb
GeoLite2-Country.mmdb

the DB that the enrichment pipeline requires, specifically GeoLitePrivate2-City.mmdb is not provided by maxmind.

Yeah the files aren't present in the repo. It is also mentioned here that you need to add them yourself https://github.com/Cargill/OpenSIEM-Logstash-Parsing/tree/1.0/build_scripts#getting-started
The private geoip file is what we created for us with the internal network information we have to be used inside Cargill. Distributing that would be sensitive to Cargill and useless for others.  You can use the maxmind writer to create a private geoip file yourself.


Need clarification on known errors page for SQL VA

Need clarification on known errors page:
Switching to express configuration failed due to a database policy error. Database policies aren't visible in the Azure portal for Defender for SQL vulnerability assessment, so we check for them during the validation stage of switching to express configuration.
Solution: Disable all database policies for the relevant server and then try to switch to express configuration again.
Consider using the provided PowerShell script for assistance.

What are database policies if we can't see them in the Azure Portal?
Why did/do they exist?
How do I remove them manually? Do I have to use the script?

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

ID: abac8313-5b62-7a01-748b-8db14e5b1a3f
Version Independent ID: 93192a83-6999-4c64-a9e3-2700edf3dd41
Content: Manage vulnerability findings in your Azure SQL databases - Microsoft Defender for Cloud
Content Source: articles/defender-for-cloud/sql-azure-vulnerability-assessment-manage.md
Service: defender-for-cloud
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

@applefacts
Thanks for your feedback! We will investigate and update as appropriate.

@applefacts
I'm going to assign this to the document author so they can take a look at it accordingly

Hey there,
Thanks for the feedback!
Database policies are a legacy feature supported in classic configuration that allow you to set a specific storage account/report recipients list per database within an Azure SQL Server.
This feature was removed from the UI, but still supported in classic configuration API/PS for backwards compatibility, but is no longer supported in express configuration.
The script provided in the documentation also looks for existing database policies and removes them before migrating the server to express configuration.
Best regards,
Catalin

#please-close


Add connection Reconnect method

Currently if a connection goes offline, there is no way to re-estalbish this entire connection because the Connect method will see the existing net.Conn object in c.conn on these lines and use the existing (and closed) connection.

I will include unit tests if this approach is approved

Alternative approach would be to nil out the c.conn, I felt this was a bit safer as it didnt impact existing code

closing it for now. if we want to return to it, we can re-open PR


Support updating a login credential

Task/Issue URL: https://app.asana.com/0/0/1202487666612311/f
Description
Support updating a saved credential workflow
Steps to test this PR

[x] Visit https://trello.com/login (having not previously saved any credentials for it)
[x] Attempt login. Verify you are prompted to save credentials.
[x] Choose to save them.
[x] Reload page, and verify you can autofill with the saved credentials from before
[x] Reload again, and this time change the password before logging in
[x] Verify you are prompted to update the login, not create a new one
[x] Choose to update
[x] Visit the credential management screen to verify the password was updated successfully
[x] Revisit that login page. This time, enter a new username and attempt a login
[x] Verify you are prompted to save a new login (not to update)

While testing the changes on this PR, I noticed that:

When I have saved the credential and loaded the page, no key icon is visible BUT the area where they key icon used to be seems to work since when you click it, it launches the bottom sheet.
When I enter a site with credentials saved, clicking on the login text area launches the bottom sheet. I think it used to only launch when clicking the key icon.


crash from starting with empty database

This is worrying. I cleaned up my python environment and am loading a new copy of the ca data into a fresh and empty database and this happened:
Loading data files
 Loading CvrSoCd
Traceback (most recent call last):
  File "manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 385, in execute_from_command_line
    utility.execute()
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 377, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/django/core/management/base.py", line 288, in run_from_argv
    self.execute(*args, **options.__dict__)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/django/core/management/base.py", line 338, in execute
    output = self.handle(*args, **options)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/calaccess_raw/management/commands/downloadcalaccessrawdata.py", line 121, in handle
    self.load()
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/calaccess_raw/management/commands/downloadcalaccessrawdata.py", line 251, in load
    call_command("loadcalaccessrawfile", model.__name__)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 115, in call_command
    return klass.execute(*args, **defaults)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/django/core/management/base.py", line 338, in execute
    output = self.handle(*args, **options)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/django/core/management/base.py", line 503, in handle
    label_output = self.handle_label(label, **options)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/calaccess_raw/management/commands/loadcalaccessrawfile.py", line 22, in handle_label
    self.load(label)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/calaccess_raw/management/commands/loadcalaccessrawfile.py", line 36, in load
    self.load_mysql(model, csv_path)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/calaccess_raw/management/commands/loadcalaccessrawfile.py", line 54, in load_mysql
    self.cursor.execute('TRUNCATE TABLE %s' % model._meta.db_table)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/django/db/backends/utils.py", line 81, in execute
    return super(CursorDebugWrapper, self).execute(sql, params)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/django/db/backends/utils.py", line 65, in execute
    return self.cursor.execute(sql, params)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/django/db/utils.py", line 94, in __exit__
    six.reraise(dj_exc_type, dj_exc_value, traceback)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/django/db/backends/utils.py", line 63, in execute
    return self.cursor.execute(sql)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/django/db/backends/mysql/base.py", line 129, in execute
    return self.cursor.execute(query, args)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/MySQLdb/cursors.py", line 205, in execute
    self.errorhandler(self, exc, value)
  File "/home/ray/Projects/OpenCA/ccdcEnv/local/lib/python2.7/site-packages/MySQLdb/connections.py", line 36, in defaulterrorhandler
    raise errorclass, errorvalue
django.db.utils.ProgrammingError: (1146, "Table 'calaccess_raw.CVR_SO_CD' doesn't exist")

If the table does not exist, it obviously does not need to be emptied. I hope this was not something I introduced. :-(

I then created the tables myself and got this:
$ python manage.py downloadcalaccessrawdata --skip-download --skip-unzip --skip-prep --skip-clear --skip-clean
Loading data files
 Loading CvrSoCd
  Table Record count doesn't match CSV. Table: 1	CSV: 13959
 Loading Cvr2SoCd
  Table Record count doesn't match CSV. Table: 1	CSV: 34739
 Loading CvrCampaignDisclosureCd
  Table Record count doesn't match CSV. Table: 1	CSV: 332697
 Loading Cvr2CampaignDisclosureCd
  Table Record count doesn't match CSV. Table: 1	CSV: 93646
 Loading RcptCd
  Table Record count doesn't match CSV. Table: 1	CSV: 8817934
 Loading Cvr3VerificationInfoCd
  Table Record count doesn't match CSV. Table: 1	CSV: 212737
 Loading LoanCd
  Table Record count doesn't match CSV. Table: 1	CSV: 66298
 Loading S401Cd
  Table Record count doesn't match CSV. Table: 1	CSV: 40845

A single row in each table? Not helpful....

I have re-created my "ccdc" project, the django project I created that uses the raw-data and browser project. We will see if that helps.

Interesting. Well, my newly created project did not have the error from the truncate command, but it did get only 1 row per table. I think I can manually load each csv file. We will see.

Not sure what my issue was here. Cleaned things up and am getting past this.


Hash not applied to CSS variables when using negative syntax

Description
When I use a negative token value such as marginLeft: '-sm', along with hash: true set in config, I expect the class names produced by panda cssgen to be hashed. Instead, the non-hashed css variable is used.
// panda.config.ts
export default defineConfig({
  theme: {
    tokens: {
      spacing: {
        sm: { value: '1rem' },
        md: { value: '2rem' },
      },
    },
  },
  hash: true,
}

// CSS
const styles = css({
  padding: 'sm',
  marginLeft: '-sm',
});

// styled-system/styles.css
.fzBPom {
    padding: var(--jolVMp)
}

.dVvtuO {
    margin-left: calc(var(--spacing-sm) * -1) // This should referencing var(--jolVMp)
}

Link to Reproduction
https://stackblitz.com/edit/vitejs-vite-tp9rs7?file=styled-system%2Fstyles.css
Steps to reproduce

Go to https://stackblitz.com/edit/vitejs-vite-tp9rs7?file=styled-system%2Fstyles.css
Let npm install and panda codgen && panda cssgen run
Open the styled-system/styles.css file
See the CSS generated

JS Framework
No response
Panda CSS Version
0.13.1
Browser
No response
Operating System

[X] macOS
[ ] Windows
[ ] Linux

Additional Information
No response

Good catch @jonambas,
I just pushed a fix for this. I'll release an update shortly.
If the issue persists after upgrading, I'll re-open it.


Store appserver URL with each event

(get originator/referrer URL)

Implemented here > https://github.com/alexbfree/Serengeti/commit/12e8ee4465c71b2deb5d87ef6ff780dae1fc858f
Not yet modularised into Geordi component

Now supported in geordi-client here > https://github.com/zooniverse/geordi-client/commit/3f708bf4b68169c82a4e6ae473b3d9c3c90b8304#diff-2fbc5085a7bbe6fb696a1a3b75a6f420R105


UCC operator creation on demand

Summary
Change UCC such that operators are built on demand rather than being built only when the circuit is built. This means now, that as soon as UCC has a valid configuration, a call to get operators or num_qubits will return actual values rather than None and 0 respectively. The later behavior is problematic with VQE when used more dynamically (i.e. beyond just passing in a fully configured UCC to VQE constructor). See discussion in #570.
Operators will still be built, if they were not already, when the circuit is built, but if they were built earlier it will use these. Any built operators are invalidated if UCC config is changed as before. The config checking was improved to include None checks since UCC starts out with None for num particles and spin orbitals and config check would fail previously if it gets to the point of building and either was still None.
Why it works via the VQE constructor today happens to be a side effect of a logging statement that draws the circuit and hence builds it, see https://github.com/Qiskit/qiskit-terra/blob/c9fc6d29b4c81b06d1c154690e3d2e64749e9870/qiskit/algorithms/minimum_eigen_solvers/vqe.py#L164
The updated UCC now works whether that side-effect exists or not.
Also fixes a minor bullet list formatting in the docstring that needed a new line preceding the first item in order not to be wrapped up into the paragraph text where it did not end up as a list.
Details and comments
I should have noted, that UVCC ought to have a similar alteration. That is not included in this PR at this point.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

22 of 32 (68.75%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.04%) to 83.184%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_nature/circuit/library/ansatzes/ucc.py
22
32
68.75%

Files with Coverage Reduction
New Missed Lines
%

qiskit_nature/circuit/library/ansatzes/ucc.py
2
89.24%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
8840

Relevant Lines:
10627

💛  - Coveralls

Could we backport this PR to 0.3? Since Qiskit/qiskit-terra#7216 merged, the UCC circuits are not correctly built but this PR fixes the issue 🙂
The problem is that the Terra PR referenced above changes the check whether a blueprint circuit is built from self._data is None to using a dedicated variable self._is_built. The self._data attribute is now never None anymore and UCC (without this PR here) fails to be built.

Sounds like a reasonable request :+1: @woodsp-ibm what do you think about backporting this PR as well as #615 ?
If we decide to do this, can we still trigger the mergify backport by adding the respective label? (@manoelmarques ?)

@mrossinek UCC is commonly enough used that its probably best, as was requested, to have this backported as a bug fix to stable ahead of the upcoming release. And we may as well do UVCC one too at the same time since its the same fix though its probably not as widely used.


Is there a way to configure the locale language of the SOAPFaults?

It seems that the language setting of the messages depends on the settings of the OS. I already configured the :
quarkus.default-locale=en_US
but that does not change the language of the soap fault messages. Is there any way to change this to be not dependent on the locale settings of the OS?

Interesting question. Where would the alt-locale messages come from? CXF delivers only messages in English. Does perhaps your service have bundles for other locales?
For the mechanism to pick some non-default locale, CXF seems to always take Locale.getDefault(), so you would have to pass -Duser.language=hu when starting your app in JVM mode and set quarkus.default-locale=hu at build time for native mode. That would only hold for CXF's own messages and you would have to provide the bundles for your locale yourself.
If you ask about the language of the messages thrown by your own service, then I think you are on your own. Neither JAX-WS nor CXF prescribe you anything. You can pick whatever message loading mechanism you want and you can make it sensitive to whatever system property or SOAP/HTTP message header you like.
Disclaimer: I never solved this in practice, so I can be wrong.

Thank you @ppalaga for the quick answer, I will look into this, then I get back to you when I have new updates. :)

@ppalaga This problem popped up when my coworker ran the test on his computer and the assertion of the error messages appeared in different language, because his OS configured with different languages. Sidenote: We have already configured the quarkus locale settings, tried with jvm arguments etc.
I am not sure what else we can be done to change the locale for the CXF. :)

I could perhaps help more if I knew which messages you speak about. Could you cite one? Which jar or Java class produces it?

Caused by: jakarta.xml.bind.UnmarshalException
 - with linked exception:
[org.xml.sax.SAXParseException; lineNumber: 0; columnNumber: 0; cvc-complex-type.2.4.a: Invalid content was found starting with element 'invalidElement'. One of '{expectedElement}' is expected.]

That's one half of the requested info. The other half would be where that exception gets instantiated. It can be seen from the stack trace.

Closing because of missing information. Please feel free to reopen providing it.


How to modify nodal temperatures in a RTH file with PyAnsys?

How to modify or change the nodal temperatures in a RTH file with PyAnsys? I want to display the modified temperature distribution of the same structure using Pyansys or ansys-dpf-post. So, is it possible?

Hi @itanfeng I would not recommend modifying a result file. Instead it would be better to create a custom data/plot of your modified temperature, by using dpf-core and creating a new field with your modified data, and plot this field.

Hi @pmaroneh Thanks! It's a good idea to create a new one which I didn't think of. As I'm new to and not familiar with pyansys, could you please provide some details or tutorial links about creating a new field and mapping the modified nodal temperatures/displacements on the same structure?

@itanfeng There isn't specific examples for thermal results but it will work the same as any other type of results. There are some examples that can help in the PyDPF-Core doc, for example these ones:
https://dpf.docs.pyansys.com/version/stable/examples/05-file-IO/04-basic-load-file.html#sphx-glr-examples-05-file-io-04-basic-load-file-py
https://dpf.docs.pyansys.com/version/stable/examples/06-plotting/02-solution_combination.html#sphx-glr-examples-06-plotting-02-solution-combination-py
Feel free to also have a look at the Ansys Developer Forum.

@pmaroneh Thanks a lot for your help! I will check them carefully.


Write Tests for the New API

Any changes made to the indexer should come with tests written as soon as #982 merges.
Is your feature request related to a problem? Please describe.
The current codebase lacks comprehensive test coverage, particularly for the new API endpoints and core functionality. While there is a basic test structure in place (as seen in src/tests.rs), it only tests the index route. This makes it difficult to ensure reliability and catch potential regressions when making changes.
Reference:
#[test]
fn test_index() {
    use rocket::local::blocking::Client;

    // Construct a client to use for dispatching requests.
    let client = Client::tracked(super::rocket()).expect("valid `Rocket`");

    // Dispatch a request to 'GET /' and validate the response.
    let response = client.get("/").dispatch();
    assert_eq!(response.into_string().unwrap(), "Welcome from fly.io!!!!!");
}

Describe the solution you'd like
We need to implement a comprehensive test suite that covers:
API Endpoints Testing:

Proposal endpoints (reference: src/entrypoints/proposal/mod.rs)
RFP endpoints (reference: src/entrypoints/rfp/mod.rs)
Search functionality for both proposals and RFPs

Examples on endpoints we need to test:

Search Proposals
Get Proposals
Get Proposal Snapshots
Search RFPs
Get RFPs
Get RFP Snapshots

Database Operations:

Test all DB operations in src/db/mod.rs
Ensure proper handling of proposal and RFP snapshots
Verify pagination and filtering logic

External Service Integration:

Mock and test NearBlocks API client functionality
Test RPC service interactions
Verify proper error handling for external service failures

Background Services:

Test the API background service functionality
Verify proper cache invalidation
Test the update mechanism for keeping data current

The tests should use the existing test infrastructure shown in the CI workflow.
Describe alternatives you've considered

Integration Tests Only: We could focus solely on integration tests that verify the entire system works together, but this would make it harder to isolate and fix issues.
Manual Testing: Continue with manual testing, but this isn't scalable and doesn't provide the confidence needed for continuous deployment.
Property-Based Testing: Implement property-based testing for complex data transformations, though this would require additional setup and complexity.

Additional context

The project already has SQLx set up for database testing.
The CI pipeline is configured to run tests in a PostgreSQL environment.
We should consider using test fixtures for common data scenarios.
Mock external services (NearBlocks API, RPC) to ensure tests are reliable and fast.

Key files to consider when implementing tests:

src/tests.rs
src/db/mod.rs
src/entrypoints/proposal/mod.rs
src/entrypoints/rfp/mod.rs

@petersalomonsen @Tguntenaar as discussed on our call, can you please split this task up in the way that makes sense? Thanks.

@Tguntenaar I have added two more test tasks #1018 and #1019. Do we need more at this point?
Also I see part of it is covered in https://github.com/NEAR-DevHub/neardevhub-api-rs/pull/8/files. Should we finalize this and continue with the rest in a separate PR?

Hi @petersalomonsen, I think it is best to create seperate a PR for other tests. Easier to review


Multiple lockfiles

We have both package-lock.json and yarn.lock lockfiles.
This is a recipe for disaster as it can lead contributors to get out of sync depending on which package manager they use.
At this point, the lockfiles themselves are even out of sync.
Which lockfile do we use, going forward?

We will go with npm. Thanks.


Support OpenDML

Currently only classic AVI files are supported.  OpenDML provides a much richer source of metadata.  Further, some OpenDML files may not play or may only play partially.

This has been implemented in the Media3Avi repo.


NIFI-8752: Automatic diagnostic at NiFi restart/stop

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
https://issues.apache.org/jira/browse/NIFI-8752
This PR allows NiFi to run diagnostics when the appropriate NiFi script is called with stop parameter.
The following details can be configured via bootstrap.conf.

allowed
diagnostic directory path
file count limit
directory size limit
verbose

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

RAT check failures need fixing in the builds (interstingly the Windows build seems to have passed, but it failed on Mac & Ubuntu)

In case it doesn't show up - see my reply to my original comment

Latest changes LGTM :+1:
(unfortunately I'm not able to merge, so hopefully someone else involved in the MR will give final approval and comment/merge)

Since there is no option to run nifi with start and stop arguments on Windows I relocated the implementation to the NiFi shutdownhook. Though it seems the NiFi server port has already been closed by the time the diagnostic runs. Will update as soon as I get more info about this.

LGTM+1.
Merged into main.


Please add a README.md

Please add a README.md ty ty

Missing parts list too. BTW, thanks for this awesome project. Really excited to start on it.

A bill of materials would be greatly appreciated. Ta

Awesome project!
A parts list would be a very great addition, also maybe releasing the apps (software) as open source so others can contribute?
Thanks!

all of these are just .STL files so just the 3D printable parts. It doesn't include the other electronic hardware.

I'd just be happy knowing what size the servos are. 😁

I think this project might have died before it took off. Just radio silence. Too bad, I kinda thought it was promising.

README and LICENSE files are coming, but so many people are already building this hexapod and didn't need those to get started, and unfortunately I have other priorities.
On a separate note, comments like this are one of the main reasons that I am not open-sourcing my code.

I think this project might have died before it took off. Just radio silence. Too bad, I kinda thought it was promising.

README and LICENSE files are coming, but so many people are already building this hexapod and didn't need those to get started, and unfortunately I have other priorities.
On a separate note, comments like this are one of the main reasons that I am not open-sourcing my code.

We do appreciate all your work on the project. It's exciting!
I can imagine many people have been able to make their own with little instruction. Some of us are more noobs and might need a bit more direction.
I'll get there eventually, but I'm really looking forward to getting specs and parts lists to help. 😁

I understand, and appreciate your interest in this project as well. Then I would suggest for you to hold off a month or two, until we resolve all the issues and refine the hard edges while some others finish building their hexapods. Because in the current format you would need to be able to pick your electronics and finish the wiring on your own, and I don't have a detailed step by step assembly instructions ready yet.

Just join their discord. Lots of people are ahead of building the pod

Go to www.makeyourpet.com and click on the Discord button to join. Thank you.

I also added some of the useful links to the README file. I was under the impression that all of the folks here know about my YouTube channel and my Discord server. Now your comments make sense. I hope you build and enjoy the hexapod.

And just added a fan-made wiring diagram and two separate fan-made parts lists.

This is all looking so good now! Exciting!
Thank you so much for all the updates.


Convert HTML into Markdown

Would it be possible to implement something like markdown::from_html()?

Anything is possible if you spend the time making it.
You need to port:

https://github.com/syntax-tree/hast-util-from-html (complex, this needs an HTML parser)
https://github.com/syntax-tree/hast-util-to-mdast
https://github.com/syntax-tree/mdast-util-to-markdown

Thanks!


Run test until failure

I just saw this feature on IntelliJ. I think, it would be quite useful for flakey tests.
https://twitter.com/chrisjenx/status/1232067112727769088

It's an interesting idea though several things spring to mind:-

This plug-in is not concerned with whether the tests pass or fail, just executing the correct runner command in the strategy
To determine if a failure or pass occurred you'd need to use the the MakeGreen strategy and have configured a proper errorformat for the runner
I feel like this might be a feature of the actual runner itself, say akin to test watching

My gut feeling is this is beyond the remit of this plug-in but something you could build on top of it. @janko any further thoughts?

Those are all valid points. Just shared it to start a conversation. :)
That being said, the first thing that come up to my mind was implementing an option for runner commands. However, what I didn't know was this plug-in not being aware of the outcome of the test's result.

Am closing this for now but feel free to reopen.


CNDB-12277: Upgrade to jvector 3.0.5

What is the issue
Upgrade JVector to 3.0.5 to fix
fix http://github.com/riptano/cndb/issues/12277
Porting of https://github.com/datastax/cassandra/pull/1478 to the release branch
Checklist before you submit for review

[ ] Make sure there is a PR in the CNDB project updating the Converged Cassandra version
[ ] Use NoSpamLogger for log lines that may appear frequently in the logs
[ ] Verify test results on Butler
[ ] Test coverage for new/modified code is > 80%
[ ] Proper code formatting
[ ] Proper title for each commit staring with the project-issue number, like CNDB-1234
[ ] Each commit has a meaningful description
[ ] Each commit is not very long and contains related changes
[ ] Renames, moves and reformatting are in distinct commits

(closing as the commit is already committed to the release branch)
thank you @k-rus


JDK11/openj9 releases fail extended.functional on AIX https://ci.adoptopenjdk.net/label/aix/

Please set the title to indicate the test name and machine name where known.
To make it easy for the infrastructure team to repeat and diagnose, please
answer the following questions:

test suite/name? TARGET: extended.functional
Is there an existing issue elsewhere covering this?
Which machine(s) does it work on?
Which machine(s) does it fail on? Fails on all AIX systems
Do you have a link to a Grinder re-run if the test with the failure?
https://ci.adoptopenjdk.net/job/Grinder/parambuild/?TARGET=extended.functional&CUSTOM_TARGET=&SDK_RESOURCE=releases&DEBUG_IMAGES_REQUIRED=true&JDK_BRANCH=&TEST_FLAG=&UPSTREAM_TEST_JOB_NAME=&TEST_IMAGES_REQUIRED=true&DOCKER_REQUIRED=false&EXTRA_DOCKER_ARGS=&OPENJ9_SYSTEMTEST_OWNER_BRANCH=eclipse%3Amaster&PLATFORM=ppc64_aix&PERSONAL_BUILD=false&ADOPTOPENJDK_REPO=https%3A%2F%2Fgithub.com%2FAdoptOpenJDK%2Fopenjdk-tests.git&UPSTREAM_TEST_JOB_NUMBER=&DOCKERIMAGE_TAG=&JDK_IMPL=openj9&LABEL=test-ibm-aix71-ppc64-2&EXTRA_OPTIONS=&CUSTOMIZED_SDK_URL=&AUTO_DETECT=true&ADOPTOPENJDK_BRANCH=master&ADOPTOPENJDK_SYSTEMTEST_OWNER_BRANCH=AdoptOpenJDK%3Amaster&ARCHIVE_TEST_RESULTS=false&NUM_MACHINES=&TRSS_URL=https%3A%2F%2Ftrss.adoptopenjdk.net&KEEP_WORKSPACE=false&BUILD_LIST=functional&JDK_VERSION=11&UPSTREAM_JOB_NUMBER=&ITERATIONS=1&STF_OWNER_BRANCH=AdoptOpenJDK%3Amaster&TIME_LIMIT=10&JDK_REPO=&OPENJ9_BRANCH=master&OPENJ9_SHA=&RELEASE_TAG=&OPENJ9_REPO=https%3A%2F%2Fgithub.com%2Feclipse%2Fopenj9.git&UPSTREAM_JOB_NAME=&JVM_OPTIONS=&PARALLEL=None
Any other details:

If the sdk_resource is changed to nightly - the tests proceed much further (maybe PASS, but too earely to say)
A week ago (roughly) when nightly was used as source the build failed immediately, rather than after 2 minutes (so that is a great improvement)
In short, this is an issue with the 'releases' - ALL AIX systems fail at the same point.
e.g.:

00:01:46.978       [echo] Ant version is Apache Ant(TM) version 1.9.9 compiled on February 2 2017
00:01:46.978       [echo] ============COMPILER SETTINGS============
00:01:46.978       [echo] ===fork:                         yes
00:01:46.978       [echo] ===executable:                   /home/jenkins/workspace/Grinder/openjdkbinary/j2sdk-image/bin/javac
00:01:46.978       [echo] ===debug:                        on
00:01:46.978       [echo] ===destdir:                      /home/jenkins/workspace/Grinder/openjdk-tests/TKG/../../jvmtest/functional/DDR_Test
00:01:46.978      [javac] Compiling 56 source files to /home/jenkins/workspace/Grinder/openjdk-tests/functional/DDR_Test/bin
00:01:49.468      [javac] /home/jenkins/workspace/Grinder/openjdk-tests/functional/DDR_Test/src/j9vm/test/ddrext/AutoRun.java:60: error: package com.ibm.j9ddr.tools.ddrinteractive does not exist
00:01:49.468      [javac] import com.ibm.j9ddr.tools.ddrinteractive.Context;
00:01:49.468      [javac]                                          ^
00:01:49.468      [javac] /home/jenkins/workspace/Grinder/openjdk-tests/functional/DDR_Test/src/j9vm/test/ddrext/AutoRun.java:310: error: cannot find symbol
00:01:49.468      [javac] 	public static void runTest(Context ctx, PrintStream out, String testCaseList) {
00:01:49.468      [javac] 	                           ^
00:01:49.468      [javac]   symbol:   class Context
00:01:49.468      [javac]   location: class AutoRun
00:01:49.468      [javac] /home/jenkins/workspace/Grinder/openjdk-tests/functional/DDR_Test/src/j9vm/test/ddrext/DDRExtTesterBase.java:39: error: package com.ibm.j9ddr.tools.ddrinteractive does not exist
00:01:49.468      [javac] import com.ibm.j9ddr.tools.ddrinteractive.CommandUtils;
00:01:49.468      [javac]                                          ^
00:01:49.468      [javac] /home/jenkins/workspace/Grinder/openjdk-tests/functional/DDR_Test/src/j9vm/test/ddrext/DDRExtTesterBase.java:40: error: package com.ibm.j9ddr.tools.ddrinteractive does not exist

@aixtools I think this is due to a missmatch of the jdk and test tools, see: https://github.com/AdoptOpenJDK/openjdk-tests/issues/2346
jdk-11 January release will need openjdk-tests and TKG from January, not the latest as changes occurred late january.

I set OPENJ9_BRANCH=openj9-0.24.0 in https://ci.adoptopenjdk.net/job/Grinder/8150/console which matched the release version being selected and it passed the build section OK. It didn't get as far as running any tests, but that explains the original issue described here, therefore closing.


UCF101: site down

Hi,
Anyone can provide a temporary link for UCF101.rar ? Or send to me (i can host the file) because the official link are down.
Thanks

Site up again....


Alternate checkboxes

First of all, thank you for making this theme! <3
One thing I really miss though from the minimal theme is Alternate Checkboxes. Is there any way something like these could be implemented in the Catppuccin theme? It's a shame these have to be implemented theme by theme.

Thanks! And thanks for the suggestion. Let me look into this further and I’ll get back to you with an update. On Nov 18, 2024, at 12:47 AM, Aleksandra Glesaaen Ødegård @.***> wrote:
First of all, thank you for making this theme! <3
One thing I really miss though from the minimal theme is Alternate Checkboxes. Is there any way something like these could be implemented in the Catppuccin theme? It's a shame these have to be implemented theme by theme.
image.png (view on web)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

+1 on this the theme is really nice but i also really like the extra functionality from more checkbox options.

On it! I’ve had some irl work come up but I’ll try to get this implemented sometime over the next few days. On Nov 22, 2024, at 6:15 PM, Garbosz @.***> wrote:
+1 on this the theme is really nice but i also really like the extra functionality from more checkbox options.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

This should be implemented in #94. If some functionality from Minimal is missing, please reopen let me know. And thank you to Kepano for his hard work in the community. <3


Action Column Empty

Hello, I have the SQL example working. I see the full table and it interacts correctly with the SQL database. However, the Action cell is blank for each record/row. I'm expecting to see a way to delete the record from the database. Is there anything missing from the setup that would trigger something appearing in that cell that would call the delete.php file?

Update: I can see the trash can using Firefox 49.0.2 but not using IE 11.  What do I need to change so that the trash can shows in IE 11.

Came down to document mode.


fix(loading) complete renaming by correctly moving files into subdirs

according to the prophecy of: http://guides.rubygems.org/name-your-gem/
maybe i should've read this earlier ;)

Very good!


Choose an approach for coherent path handling

What's the problem this feature will solve?
pip-tools currently handles paths in general a bit weakly:

using string parsing and manipulation, achieving incorrect results depending on the path names, potentially overwriting unrelated user files
producing different compilation output depending on the form given for equivalent relative paths (./requirements.in vs requirements.in)
naively forwarding relative paths (relative to a reqsin file) into the output file annotations, which may be in a different folder
not yet annotating setup.py sources with relative paths if appropriate
occasionally issues are raised about transformation between absolute and relative paths for various purposes
sometimes flops when trying to write to un-writable cache paths

We need to reliably inspect and transform paths, absolute and relative, for issues present and future.
Describe the solution you'd like
I'd like to build a consensus on a high level approach to more robust and consistent path handling (which modules to use and how it will affect the packaging of pip-tools itself), and then use that approach to fix up and improve path operations.
Here's an example problem in the current code:
$ echo '# real important data' > some.txt
$ mkdir some.folder
$ echo 'requests' > some.folder/reqsin
$ pip-compile --no-header some.folder/reqsin
$ ls some.folder
reqsin
$ cat some.txt
certifi==2019.11.28       # via requests
chardet==3.0.4            # via requests
idna==2.8                 # via requests
requests==2.22.0          # via -r some.folder/reqsin (line 1)
urllib3==1.25.8           # via requests

Similarly, if some.txt content is uncommented, pip-compile fails because it tries to parse that for existing requirements.
See also: #1061 #204 #966 #616 #395 #1084 and discussion at #1058
Alternatives and Considerations

We're still supporting python2
pathlib (stdlib) + pathlib2 (pypi or vendored)
better use of os.path
plumbum (which could also supplant click)

Some things I like about plumbum's path objects, relative to pathlib:

subclass str
always absolute, with a different type RelativePath (glorified list of path parts) which can be generated from a path's .relative_to function
because of the above, there's none of this pathlib nonsense: p.parent == p
the 'absolute' paths do not auto-resolve symlinks like pathlib does -- AFAIK it's impossible to get pathlib to resolve relative to absolute paths without also resolving symlinks
no divergent handling for python2
can potentially replace click, meaning that we could keep the dep count equal, while using pathlib2 would increase that number

I'm in favor of pathlib (stdlib) + pathlib2 (pypi).

@atugushev alright, can we make that pathlib + os.path and pathlib2 + os.path, to avoid pathlib's forced symlink resolution?
I still don't prefer pathlib here for the above reasons (especially that this will have us using 2 PyPI modules and 5 different stdlib modules where 1 PyPI module could do, and will create extra python2-specific baggage, and pathlib2 will never get any updates/fixes), but am happy to implement it this way, as improving path handling from our current state is important to me.

@AndydeCleyre

I still don't prefer pathlib here for the above reasons (especially that this will have us using 2 PyPI modules and 5 different stdlib modules where 1 PyPI module could do, and will create extra python2-specific baggage, and pathlib2 will never get any updates/fixes

Yeah, that's unfortunate, but hopefully, pip (i.e. pip-tools) drops py2 support someday.
BTW, could you elaborate on better use of os.path alternative?

@AndydeCleyre
Offtopic: why don't you want to join the jazzband team to share our maintenance burden (especially with your awesome activity on the project)? :)

BTW, could you elaborate on better use of os.path alternative?

Probably we can get much safer and more reliable behavior than we have without using anything other than os.path, even though more convenient options are now available. Looking at the example problem in this issue description, we might handle it like:
diff --git a/piptools/scripts/compile.py b/piptools/scripts/compile.py
index 0572cd3..670c215 100755
--- a/piptools/scripts/compile.py
+++ b/piptools/scripts/compile.py
@@ -235,7 +235,7 @@ def cli(
             )
         # Otherwise derive the output file from the source file
         else:
-            base_name = src_files[0].rsplit(".", 1)[0]
+            base_name = os.path.splitext(src_files[0])[0]
             file_name = base_name + ".txt"

         output_file = click.open_file(file_name, "w+b", atomic=True, lazy=True)

I may be more tempted to go an all-os.path route here if feasible, since pathlib doesn't handle symlinks sufficiently IMO.

I may be more tempted to go an all-os.path route here if feasible, since pathlib doesn't handle symlinks sufficiently IMO.

Oh, I see. That's great if os.path would be enough.


The docxorcist: scariest doc PR ever

Got JavaScript docs to look half-decent, made Python docs more self-similar, switched to "read the docs" theme

commented the bleep out of the .thrift files
improved the script that generates .rst docs for Sphinx from that
considered factoring out exceptions and using Thrift "include", but this created all sorts of other issues, so backed out of that (some minor changes to setup.py as a result, nothing truly different)
Generated docs pushed to the Data API page on github.io

For next PR, will get the Perl docs in good shape too. Most of the way there, just need to refactor the generator module to create Perl syntax examples and signatures in the API ref. Java would be the same idea, so not too far from autodoc. for all the clients (off the comments in the Thrift), except Python of course which is "special".

Current coverage is 80.06%

Merging #185 into develop will not affect coverage as of 0d2ba44

@@            develop    #185   diff @@
=======================================
  Files            21      21
  Stmts          2994    2994
  Branches          0       0
  Methods           0       0
=======================================
  Hit            2397    2397
  Partial           0       0
  Missed          597     597

Review entire Coverage Diff as of 0d2ba44

Powered by Codecov. Updated on successful CI builds.

:imp: :feelsgood:
We still need to reconcile the javascript code with the docs, because we have some inconsistencies there.

Did you mean to disable redis in deployment.cfg?

Oh, no, that was the by-product of some redis question earlier. I can put that back. Good catch.

re: the javascript code, I agree but to be clear I think the code should be modified to be like the docs, not the other way around (mostly get_foo() instead of plain foo()). Did you have anything else in mind?

That was what I was referring to, updating the js code to be consistent with the docs and the other api code.  That isn't in this repo actually, just noting that we need to fix that.


Fixe: Events API null total

Fixes https://github.com/FrigadeHQ/trench/issues/13
Earlier the total was returning a hardcoded null value, I changed it to get count on the where clause
const total = await this.clickhouse.queryResults(totalQuery, workspace.databaseName)
const totalCount = total?.[0]?.count || null

@christianmat I am not sure if I am supposed to write tests for it. Anyways do let me know what you think about it.

Hey @tuminzee, thanks for this PR. Please see the guide on contributing re: tests: https://docs.trench.dev/contributing
One thing we need to fix here is that the total is not meant to simply equal the length of the array, it's supposed to be the total amount of records matching the query string. It is meant to be used for pagination so you know how many results you have. The real solution should be to do a separate query to clickhouse in parallel that counts the number of rows that matches the query.

@christianmat I have a made a separate query for calculating the total. It is not a length fn.
I will check the contribution guidelines

Please add e2e tests as well that test that this field works. See other e2e tests for examples.

Yes I will do that

waitForQueryResults function should return the complete res.body instead of just res.body.results since we need to test other response properties like total
Questions for review:

Should I update all affected tests to include total property validation, or should we keep this as a separate enhancement?
If no results are found the query would always return a timeout error, shouldn't it just return a plain response?

waitForQueryResults function should return the complete res.body instead of just res.body.results since we need to test other response properties like total
Questions for review:

Should I update all affected tests to include total property validation, or should we keep this as a separate enhancement?
If no results are found the query would always return a timeout error, shouldn't it just return a plain response?

Yes, please update all the existing tests to account for the new total behavior.
Not sure what you mean. It currently returns an empty array if no results are returned, it doesn't throw an error (see test screenshot below)

I have updated the tests to include the total property.
Regarding the timeout error: I apologize for not clarifying earlier—this issue occurs only in the e2e tests. waitForQueryResults: It currently throws an error if no results are found. Should this be the expected behavior, or should we modify it to return an empty array instead? Let me know your preference, and I can adjust accordingly. I have added a test to demonstrate this here


backend: compress and check size of PDF when uploaded

Currently, the pdf is uploaded without checking its size and no compression is performed, introduce validation and compression.

@namantiwari2002 you can work on this


mauerbac/prune

Approved in phab. Will merge & release on Monday with @mikeng13.

build

Coverage increased (+0.001%) to 99.798% when pulling 9bc657795e55c232f75f96d842837bd65359719a on mauerbac/prune into 6c29f5b3acaf563ff9f6faf3042882bcf5370fe0 on master.


Add customization on save data and files WIP

WIP : fix checkForMissingFiles
Add full customization of request options, and customization of data parsing.

@jacquarg is this PR still work in progress?

@nono
Yes, actually I don't manage to make it work, because of some troubles creating the files ( cf https://github.com/cozy/cozy-client-js/issues/137 ). I will have a look right now.

@jacquarg I think this PR is no longer wanted. But if I'm wrong, feel free to reopen it!


Fix typo in README.md

Fix typo in README.md . The action name is send-google-chat-webhook and not sent-google-chat-webhook

Somehow github still says that a "review is required" even after you approved the change @alex-blueguardianco


can't insert new event

System Information:

Version: 2.6
Language: it
Timezone Offset: +01:00
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/48.0.2564.116 Safari/537.36

What steps will reproduce the problem?

press plus
write a description
press "save"

What is the expected result?
What happens instead?
Can't insert a new event
Screenshot URL:
Logs:

What happens instead? Can you share a screenshot? I’m afraid there is not enough information right now for us to diagnose the problem fully.


Add iOS popover (Resolves #2 Resolves #3)

Add iOS popover. Resolves #2 Resolves #3
This PR adds IosPopoverMenu  and IosToolbar  widgets with demos.

Can you add a short video that shows the menu in various configurations, as well as show what it looks like to scroll the buttons when they don't fit?

@matthew-carroll
https://user-images.githubusercontent.com/7597082/192404972-af252225-0514-49e0-b6f0-1063dc16974c.mp4

@angelosilvestre can you add an ability to the demo that allows the user to click and drag around the screen, and cause the arrow to point towards that offset? And then can you post another video showing that the arrow moves as expected for both styles of menu?

@matthew-carroll
Popover menu
https://user-images.githubusercontent.com/7597082/193695908-b9f60a41-7250-47e2-818e-ed24169068fb.mp4
Toolbar
https://user-images.githubusercontent.com/7597082/193695939-dcc7cfe7-a9e0-4aad-beae-99f9bef9b13e.mp4

It looks like this calculation is still only leaving space for an arrow on one side. Am I missing anything?

@matthew-carroll I think this was the outdated code.

@matthew-carroll Changed child constraints to respect screen bounds.


For git add force update support!

GitConfigurationSourceBuilder.setForceUpdate(true);

now when sone one force pull to remote git repository,cfg4j-git reload() method will throw exception:"Checkout conflict with files" and "HEAD is detached"


Calendar header handling animation controller incorrectly

Hello,
It seems that _CalendarHeaderState causes the following exception:
flutter error: This widget has been unmounted, so the State no longer has a context (and should be considered defunct)
The fix:
1:
Move initialization of _controller to initState():
_controller=AnimationController(duration: _kExpand, vsync: this);
2: Move everything in consctructor to initState() (after setting the _controller):
_monthIndex = monthIndexFromTime(clock.now());
_easeInAnimation =
CurvedAnimation(parent: _controller, curve: Curves.easeIn);
_iconTurns = Tween(begin: 0.0, end: 0.5).animate(_easeInAnimation);
HTH :)

It works, thanks a log.

Thanks a lot, it's work


Int64 - Int64(Field.one).neg() just wraps around the max Field value

Description
When calling .neg() on Int64(Field.one), it returns<PHONE_NUMBER>9329048855892746252171976963363056481941560715954676764349967630336.
Is this correct behavior?
import { Int64, Field, isReady, shutdown } from 'snarkyjs';
await isReady;

const x = new Int64(Field.one).neg();
console.log(x.value.toString()); //<PHONE_NUMBER>9329048855892746252171976963363056481941560715954676764349967630336

await shutdown();

This was correct in the old representation, however, it's outdated, the PR that I merged today changed the Int64 representation from one to two field elements, and the failing tests are all fixed (in this case, I changed the test to check that toString gives '-1')


chore(main): release 0.5.4

:robot: I have created a release beep boop
0.5.4 (2024-07-24)
Bug Fixes

docker versions (6c90f74)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/socialgene/sgnf/releases/tag/v0.5.4 :sunflower:


formating switch is now broken

In version 2.5.3 the switch is formated in a wrong way
before

now

close duplicate.


Prefix hostname to restic copy output

Output of restic version
16.0

What should restic do differently? Which functionality do you think we should add?
The restic copy feature at present only displays the snapshot date & path.
Prefixing the hostname to the path would be great.

What are you trying to do? What problem would this solve?
By prefixing the hostname to the path would help identify which host snapshot is being copied.
When there are multiple snapshot from different hosts having the same source path and even more if it was scheduled at the same time.

Did restic help you today? Did it make you happy in any way?
Oh I am happy....

https://forum.restic.net/t/restic-copy-should-also-show-hostname/6748

https://forum.restic.net/t/restic-copy-should-also-show-hostname/6748


add runtime error handling line to config_list

Why are these changes needed?
There's a few questions about why config_list is a list and how it works.
While this contribution doesn't address how config_list actually works, more details and links are added in relevant places in the documentation to give more clarity.

Related issue number
#749 Why is config list a list?

Checks

[x] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

@sonichi alright, committed the change. =)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f812a23) 27.76% compared to head (da8ac58) 27.56%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #779      +/-   ##
==========================================
- Coverage   27.76%   27.56%   -0.21%
==========================================
  Files          27       27
  Lines        3493     3493
  Branches      791      791
==========================================
- Hits          970      963       -7
- Misses       2452     2458       +6
- Partials       71       72       +1

Flag
Coverage Δ

unittests
27.56% <ø> (-0.15%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

There is a conflict with the main. Could you resolve it?

@sonichi Hi! Sorry I just saw this. Not sure if I did it correctly, can you please check?

I still see a conflict.

The mentioned issue has been addressed in another PR. Thank you for the PR, @javadrip


Don't redundently document directory configuration in plugins

See original issue on GitLab
In GitLab by [Gitlab user @phildawson] on Oct 16, 2018, 10:30
Background
A left over item following the merge of !776
This item describes Technical debt.
See https://gitlab.com/BuildStream/buildstream/merge_requests/776#note_107972976 for the original comment.
Task description
Change all of the individual source plugins to no longer redundantly document the directory configuration; instead they should all have a mention that the reader should refer to the built-in functionality section of the Source docs for common configurations.
The same should go for the BuildElement implementations in plugins/elements/.

[ ] Sources
[ ] BuildElements

Acceptance Criteria
Source and build elements no longer redundantly document the directory configuration.

In GitLab by [Gitlab user @phildawson] on Oct 16, 2018, 13:32
changed the description

In GitLab by [Gitlab user @phildawson] on Oct 16, 2018, 13:38
mentioned in merge request !776

In GitLab by [Gitlab user @willsalmon] on Oct 22, 2018, 10:43
changed the description

In GitLab by [Gitlab user @cs-shadow] on Oct 24, 2018, 22:49
changed the description

In GitLab by [Gitlab user @cs-shadow] on Oct 24, 2018, 22:49
Edited description to say "sources plugins" instead of "source elements".

In GitLab by [Gitlab user @cs-shadow] on Oct 24, 2018, 23:44
mentioned in commit 3e984ca6f0d07b6df4dbd7f5de0f7d62412b5412

In GitLab by [Gitlab user @cs-shadow] on Oct 24, 2018, 23:44
mentioned in commit 97d0cd7150422466ae5787db60ca2e49a47cbbbf

In GitLab by [Gitlab user @cs-shadow] on Oct 24, 2018, 23:49
mentioned in merge request !896

In GitLab by [Gitlab user @cs-shadow] on Oct 24, 2018, 23:51
assigned to [Gitlab user @cs-shadow]

In GitLab by [Gitlab user @cs-shadow] on Oct 25, 2018, 16:25
mentioned in commit fb25141ac265fc10f97a7e09670f3e17b1470fbe

In GitLab by [Gitlab user @cs-shadow] on Oct 25, 2018, 16:25
mentioned in commit c477dcfd27b70ef1aee68aecb88de2c538921454

In GitLab by [Gitlab user @cs-shadow] on Oct 25, 2018, 16:25
mentioned in commit 5ea98f7734a6d1f6d3128c4e59c2b29c5d106e2c

In GitLab by [Gitlab user @cs-shadow] on Oct 25, 2018, 16:25
mentioned in commit 23140b6b41718db884f16b967ac63af35431f85f

In GitLab by [Gitlab user @cs-shadow] on Oct 25, 2018, 16:26
mentioned in commit db2a676e292f2970c4d03661c5ed7db63717f744

In GitLab by [Gitlab user @cs-shadow] on Oct 25, 2018, 17:59
closed via merge request !896

In GitLab by [Gitlab user @cs-shadow] on Oct 25, 2018, 17:59
mentioned in commit 356d84cdce6c44516d09ab561e10545434e4a609


feat(frontend): Grouping util for token list

Motivation
We want to change the way we group tokens, to have more than two elements in the group, and to try and avoid repetitions/glitches when the order of the token list is not the usual one.
Plus, the
Definitions
Main Token
The "original" token of a certain group or of a certain other token; for example BTC is the "main token" of ckBTC, or ETH is the "main token" of SepoliaETH. Typically, a token and its "main token" should be 1:1.
Secondary token
A token that has a "main token", even if not present in the list. It is just to indicate that it should always be grouped if the latter is present.
Implementation
The function loops through the tokens list and groups them according to a prop key that links a token to its "main token". For example, it could be twinToken for ck-tokens. But the logic can be extended to other props, if necessary.
The tokens with no "main token" will still be included in a group, but it will be a single-element group, where the the token itself takes the place of the "main token". That, in general, makes sense for:

tokens that are not "secondary tokens" of a "main token";
tokens that have a "main token" but it is not present in the list.

The returned list respects the sorting order of the initial tokens list, meaning that the group is created at each position of the first encountered token of the group. So, independently of being a "main token" or a "secondary token", the group will replace the first token of the group in the list. That is useful if a "secondary token" is before the "main token" in the list; for example, if the list is sorted by balance.
Tests
I created a few tests mocking some twin tokens and specific scenarios: different sorting, multiple "secondary tokens", missing "main token".

[OISY-164]

@peterpeterparker I think ready for another review

@peterpeterparker ready for another review

Superseded by:


In-Context Progress

User should see in the course outline what assignments have been completed. Completed subsections and modules should be highlighted in green. Also, the “Completed” mark should be displayed after  task has been completed.

@volodymyr-chekyrta can you please confirm if this is already implemented in the app or not?
I've tested it and a tick icon appears on some content while on others it does not.


Generate error unions even when neverThrowErrors: false

Even when we're throwing errors, it can be helpful to have an error visitor that you can use the catch block. This way, you can still have some type safety that you're handling all the errors you're supposed to

closing as stale


Adapt Vlasov and reg test input file to use new parallelism input

instead of .low_inp, .has_low_inp, and .use_gpu.
Tested by running rt_vlasov_weibel in all sorts of serial and parallel CPU/GPU combinations, always producing the same final field. Also did make regression to ensure all reg tests compile.
My only doubt is about is with regards to the tests where we separately pass use_gpu to gkyl_wv_euler_new. Are we handling this correctly? in this commit/PR we only modified the Vlasov app, not the wv_euler objects.

So long as the wv_euler object is still taking the use_gpu flag like so:
struct gkyl_wv_eqn *euler = gkyl_wv_euler_new(ctx.gas_gamma, app_args.use_gpu);
everything is fine. This initialization is Vlasov following the design of moments in that certain systems instantiate their equation object outside of the app, and the app acquires a pointer to that equation object internally.


How do I suppress the logs?

The logs, whilst handy, are getting really annoying in between stuff I am trying to debug. Is there a way I can turn this off?

Not sure if you fixed this. But googling it the angular log.debug can be disabled using:         $logProvider.debugEnabled(false);
in your angular.config i think (inject logProvider)


Cannot apply unknown props to html elements.

In React 15.1.0 I am getting this warning in the console:
Warning: Unknown props `search`, `caseSensitive`, `matchElement`, `matchClass`, `matchStyle` on <span> tag. Remove these props from the element. For details, see https://fb.me/react-unknown-prop
This occurs from applying non standard props to an HTML element. i.e. <span search='string'>.
Thanks for your work on this project. 🙇

Hey @canfie1d https://github.com/helior/react-highlighter/pull/30 should resolve your issue.

Thanks!


Not working

I run the docker command and it doenst work in port 5901 and 6901, nothing happens.
On the other hand it doesn t keep running

It is working


calling against ambiguous nucleotides in the reference

Hello,
I have not been able to find the answer to this:
I have a consensus sequence which contains N in some positions.
Those positions are variant positions in some cases which is why they are ambigous in the consensus, but I am not able to get variants back from freebayes from where the reference fasta file has an N.
Is this possible?  I mapped the reads with bwa and created the bam with samtools 0.1.19 or samtools 0.1.18.
I would like to see called as a SNP any nucleotides that map to the reference N, thus:
seqname    POS   ID   REF   ALT  QUAL ...
seqname    1        .      N      A      ..

We could change freebayes to allow this. However, it's not really designed
to do this.
If you want to solve this quickly, I'd suggest picking a majority allele in
your consensus. Ns cause many problems in alignment and variant calling.
The systems aren't necessarily designed to handle this use case.
This kind of problem is precisely why I am working on
https://github.com/vgteam/vg.
On Thu, Sep 15, 2016 at 5:33 PM LCrossman<EMAIL_ADDRESS>wrote:

Hello,
I have not been able to find the answer to this:
I have a consensus sequence which contains N in some positions.
Those positions are variant positions in some cases which is why they are
ambigous in the consensus, but I am not able to get variants back from
freebayes from where the reference fasta file has an N.
Is this possible? I mapped the reads with bwa and created the bam with
samtools 0.1.19 or samtools 0.1.18.
I would like to see called as a SNP any nucleotides that map to the
reference N, thus:
seqname POS ID REF ALT QUAL ...
seqname 1 . N A ..
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ekg/freebayes/issues/320, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAI4EceIjkHusuNWip7hmG2AL2BaT-MYks5qqWUtgaJpZM4J-AsQ
.

Hi, thanks, samtools does call against the N but I definitely want to use freebayes so I'll alter consensus which was called by default biopython.
Cheers,
Lisa

We should be able to fix this. There is a check which skips N bases in the
reference that can be disabled in a line of code.
Allowing calls against them could introduce a number of issues. For
instance many aligners like to stick alignments randomly against runs of Ns.
Reiterating, it is exactly this kind of issue that we are working on with
vg. In this case you actually have a good model for which bases could be at
the Ns. Why throw that away during the analysis? It is technically possible
to do your analysis with vg. However I won't suggest it now unless you are
excited by the possibility because documentation is lacking due to the
rapid pace of development.
On Thu, Sep 15, 2016, 22:22 LCrossman<EMAIL_ADDRESS>wrote:

Hi, thanks, samtools does call against the N but I definitely want to use
freebayes so I'll alter consensus which was called by default biopython.
Cheers,
Lisa
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ekg/freebayes/issues/320#issuecomment-247443064, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAI4EQhRJIg7Hcu8WG9ooao4Iidtsoobks5qqaj4gaJpZM4J-AsQ
.