[QUESTION: How can I use models with relationships?

When I try to use a model with relationship, an error occurs:
Traceback (most recent call last):
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/uvicorn/protocols/http/httptools_impl.py", line 375, in run_asgi
    result = await app(self.scope, self.receive, self.send)
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/uvicorn/middleware/proxy_headers.py", line 37, in __call__
    await self.app(scope, receive, send)
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/starlette/applications.py", line 133, in __call__
    await self.error_middleware(scope, receive, send)
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/starlette/middleware/errors.py", line 122, in __call__
    raise exc from None
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/starlette/middleware/errors.py", line 100, in __call__
    await self.app(scope, receive, _send)
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/starlette/middleware/base.py", line 25, in __call__
    response = await self.dispatch_func(request, self.call_next)
  File "/home/mrd/Projects/itg/orion/feature_vstack/middelware.py", line 10, in LoggerMiddleware
    response = await next_call(request)
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/starlette/middleware/base.py", line 45, in call_next
    task.result()
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/starlette/middleware/base.py", line 38, in coro
    await self.app(scope, receive, send)
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/starlette/exceptions.py", line 73, in __call__
    raise exc from None
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/starlette/exceptions.py", line 62, in __call__
    await self.app(scope, receive, sender)
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/starlette/routing.py", line 585, in __call__
    await route(scope, receive, send)
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/starlette/routing.py", line 207, in __call__
    await self.app(scope, receive, send)
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/starlette/routing.py", line 40, in app
    response = await func(request)
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/fastapi/routing.py", line 122, in app
    skip_defaults=response_model_skip_defaults,
  File "/home/mrd/Projects/itg/orion/.env/lib/python3.7/site-packages/fastapi/routing.py", line 54, in serialize_response
    raise ValidationError(errors)
pydantic.error_wrappers.ValidationError: 1 validation error
response -> interfaces -> 0
  value is not a valid dict (type=type_error.dict)

interfaces is simple relationship field.

Need more details.

I saw a very similar looking issue on FastAPI yesterday; try adding
class Model(BaseModel):
    ...
    # add this:
    class Config:
        orm_mode = True

to whatever model your endpoint is returning a list of. (I'm not exactly sure why this works, but I think it has to do with FastAPI internals.)

closing this as not enough detail and no response.


Sort Order

Ive used the QueryBuilder to order the results like so:
    $listProducts = $query->select('*')
                ->from('table')
                ->range('0-0')
                ->order('color')
                ->execute()
                ->getResult();

Which gives me the first result ordering by 'color' descending. How can I do the same thing but ascending?
Thanks!

Hello sharkstate,
By default, the "order" method shows the data in ascending order. Anyway, you can make your ordering explicit.
    $listProducts = $query->select('*')
                ->from('table')
                ->range('0-0')
                ->order('color.asc') //or "color.desc"
                ->execute()
                ->getResult();

Hope this helps!


update the global variables using scss

add color variables
add font variables
add text variables

Descriptions

Link to UI design (if possible):

Provide screenshots or videos that the function is working.
Provide screenshots of no warning, or error in the console.

[x] All unit tests passing and coverage of testing is more than 90% unless special needs.
[x] No merge conflict with the main branch, make sure your code is up to date with main.
[x] Check PR title is aligned with your ticket.
[x] Check your app will run without crashes.
[x] Check if there are hard-coded values.
[x] No //eslint-disable-next-line no-unused-vars, unless otherwise specified.
[x] Check you are using camelCase && PascalCase correctly.
[x] Remove unused comments.

update the font variables
- can use font a new class name which is the same as the documents
- please use 'npm i sass' to add the support of sass

plz update & merge main branch

Is it only me? I have this error when I try to open a dev server.

Without this module Nextjs does not allow to use scss/sass directly(must compile to css first)please use 'npm i sass' to add the support of sass

So my question is, why not provide a modified package.json with this sass package installed? So that everyone does not need to install this package individually.

I'm worried that some of the modules I learned and tried earlier might contaminate the current PACKAGE.JSON files.

I don't get it. So you want to let everyone install this dependency individually rather than globally install it once?
You have something using the sass package for sure. Using this sass function without the related library provided is nonsense.

I will mark that, and there is a conflicting issue when I pull the JSON file, please help me to fix it.


[DOC] Document when to upgrade OpenSearch Dashboards node on a rolling upgrade

What do you want to do?
The rolling upgrade documentation does not clearly mention when to upgrade the OpenSearch Dashboards node during the process. I'm not sure what is recommended, but I went through the code and will highlight a few areas.
There is a notion of compatibility between OS nodes and OSD and you can find details in the following fn https://github.com/opensearch-project/OpenSearch-Dashboards/blob/main/src/core/ser[…]check/opensearch_opensearch_dashboards_version_compatability.ts:

Reject mismatching major versions
OSD minor version cannot be ahead of OS version

Assuming the nodes meet compatibility requirements, then OSD will collect a list of warningNodes and log out the nodes where the OS version and OSD version do not match.
https://github.com/opensearch-project/OpenSearch-Dashboards/blob/main/src/core/server/opensearch/version_check/ensure_opensearch_version.ts#L189C3-L196
At first glance it looks like for a patch release it can be at any time during the rolling upgrade, but it may different for minor and major upgrades and have to occur after every node has been upgraded.

[X] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Hi Jeff, is this something you can take a look at? Thanks.


Not working

I tried it today with Chrome (last version) / Windows 11 and it stays forever on blinking "ChassistantGPT is ready for your voice commands".
Ctrl+shift+E does nothing.
Does it still work to you?

"Ready for voice conmands" signals that you gave ChassistantGPT microphone permissions, so it's listening for the voice trigger (and the prompt itself). It should always be blinking.
Ctrl+shift+E should trigger the prompt. Does the voice trigger (e.g., "Hey girl") work for you?
It definitely works on Mac. I'll test it on my Windows tomorrow.

"Hey girl" does nothing, nor does any other custom words I tried

Works for me. Does restarting Chrome help? What other extensions do you have installed?

Solved!
It turned out that the cause was external to your app: my LG Gram notebook has a preinstalled "protection" software that silently mutes mic and webcam until turned "off".


Windows Portable App

Fixes https://github.com/feross/webtorrent-desktop/issues/358
Background

What is a portable app?
Portable Audacity

Changes

A new *-win.zip output file is now produced, which just contains the electron-packaged windows app, with an added empty "Portable Settings" folder.
At startup, when a "Portable Settings" folder is found next to the WebTorrent.exe file, then the app is in "Portable Mode" which entails:

Electron "userData" folder is set to the "Portable Settings" folder
The config.json file is saved in the "Portable Settings" folder
Torrent posters and torrent files are saved in the "Portable Settings" folder
The default download folder is set to the "Portable Settings" folder

Am I forgetting anything?
PortableApps.com Platform
I started a discussion to get WebTorrent Desktop included in the PortableApps.com bundle.

LGTM


Add option to insert scripts before  instead of  for faster rendering

I've added an insert_js_last option that ensures all scripts are added after all content, just before the  tag for faster rendering, since rendering halts for inline scripts.
Practicing being opinionated I've made this option true by default, it shouldn't cause backwards compatibility problems and it will result in (slightly) faster render times for the average user.
I've updated the specs as well.

Hi James,
I don't think this option should be enabled as default, it isn't correct. I know it may give you slightly better load times but being a default doesn't seems nice to me, specially because people are not using it like that right now.
Can you disable this as default and explain how to use this in the README file before merge this?
Thanks.

Will do.

Great, just merged.


Tlaplus > Java: Options ignored by "Evaluate selected expression"

Let A be a module that EXTENDS module B that happens to be in a library L.  Library L is added to TLC's classpath in Tlaplus > Java: Options.  Ordinary model-checking works fine.  However, selecting and evaluating a (constant) expressions fails with:
Error evaluating expression:
Cannot find source file for module B imported in module A.
Parsing or semantic analysis failed.

@lemmy, could you, please, reproduce the error again and check the TLC output channel to see the full command line. Are there any problems with it? Could it be that we have the same case here as in #157 -- switching between Win and WSL2?

This appears to be a classpath issue. With Java: Options set to -classpath CommunityModules.jar, evaluation fails whereas it succeeds with a full qualified path -classpath /home/markus/blockingqueue/CommunityModules.jar.
Fails:
/usr/lib/jvm/java-11-openjdk-amd64//bin/java -classpath CommunityModules.jar:/home/markus/.vscode-server/extensions/alygin.vscode-tlaplus-1.5.0/tools/tla2tools.jar -XX:+UseParallelGC -Dtlc2.TLC.ide=vscode tlc2.TLC t1592236841015.tla -tool -modelcheck -coverage 0 -config t1592236841015.cfg -deadlock -workers 4 -dump dot BlockingQueue

TLC2 Version 2.15 of Day Month 20?? (rev: eb3ff99)
Running breadth-first search Model-Checking with fp 63 and seed<PHONE_NUMBER>921528276 with 4 workers on 8 cores with 17148MB heap and 64MB offheap memory [pid: 307] (Linux 4.19.84-microsoft-standard amd64, Ubuntu 11.0.7 x86_64, MSBDiskFPSet, DiskStateQueue).
Starting SANY...
Parsing file /tmp/vscode-tlaplus-1592236841011/t1592236841015.tla
Parsing file /tmp/vscode-tlaplus-1592236841011/BlockingQueue.tla
Parsing file /tmp/TLC.tla
Parsing file /tmp/Naturals.tla
Parsing file /tmp/Sequences.tla
Parsing file /tmp/FiniteSets.tla
Fatal errors while parsing TLA+ spec in file t1592236841015
tla2sany.semantic.AbortException
*** Abort messages: 1
Unknown location
Cannot find source file for module SequencesExt imported in module BlockingQueue.
SANY finished.
Starting... (2020-06-15 09:00:41)
Parsing or semantic analysis failed.
Finished in 380ms at (2020-06-15 09:00:41)

Works:
/usr/lib/jvm/java-11-openjdk-amd64//bin/java -classpath /home/markus/blockingqueue/CommunityModules.jar:/home/markus/.vscode-server/extensions/alygin.vscode-tlaplus-1.5.0/tools/tla2tools.jar -XX:+UseParallelGC -Dtlc2.TLC.ide=vscode tlc2.TLC t1592237137714.tla -tool -modelcheck -coverage 0 -config t1592237137714.cfg -deadlock -workers 4 -dump dot BlockingQueue

TLC2 Version 2.15 of Day Month 20?? (rev: eb3ff99)
Running breadth-first search Model-Checking with fp 74 and seed -8785635609764670148 with 4 workers on 8 cores with 17148MB heap and 64MB offheap memory [pid: 449] (Linux 4.19.84-microsoft-standard amd64, Ubuntu 11.0.7 x86_64, MSBDiskFPSet, DiskStateQueue).
Starting SANY...
Parsing file /tmp/vscode-tlaplus-1592237137711/t1592237137714.tla
Parsing file /tmp/vscode-tlaplus-1592237137711/BlockingQueue.tla
Parsing file /tmp/TLC.tla
Parsing file /tmp/Naturals.tla
Parsing file /tmp/Sequences.tla
Parsing file /tmp/FiniteSets.tla
Parsing file /tmp/SequencesExt.tla
Parsing file /tmp/vscode-tlaplus-1592237137711/TLAPS.tla
Parsing file /tmp/FiniteSetsExt.tla
Parsing file /tmp/Functions.tla
Semantic processing of module Naturals
Semantic processing of module Sequences
Semantic processing of module FiniteSets
Semantic processing of module FiniteSetsExt
Semantic processing of module Functions
Semantic processing of module SequencesExt
Semantic processing of module TLAPS
Semantic processing of module BlockingQueue
Semantic processing of module TLC
Semantic processing of module t1592237137714
SANY finished.
Starting... (2020-06-15 09:05:38)
com.fasterxml.jackson dependencies of Json overrides not found, Json module won't work unless the libraries in the lib/ folder of the CommunityModules have been added to the classpath of TLC.
<<"$!@$!@$!@$!@$!", TRUE>>
Computing initial states...
Finished computing initial states: 0 distinct states generated at 2020-06-15 09:05:38.
Model checking completed. No error has been found.
  Estimates of the probability that TLC did not check all reachable states
  because two distinct states had the same fingerprint:
  calculated (optimistic):  val = 0.0
The coverage statistics at 2020-06-15 09:05:38
<Init_1592237137711 line 7, col 1 to line 7, col 18 of module t1592237137714>: 0:0
<Next_1592237137711 line 8, col 1 to line 8, col 18 of module t1592237137714>: 0:0
  line 8, col 32 to line 8, col 65 of module t1592237137714: 0
End of statistics.
Progress(1) at 2020-06-15 09:05:38: 0 states generated (0 s/min), 0 distinct states found (0 ds/min), 0 states left on queue.
0 states generated, 0 distinct states found, 0 states left on queue.
The depth of the complete state graph search is 1.
Finished in 660ms at (2020-06-15 09:05:38)

Model-checking with a relative path:
/usr/lib/jvm/java-11-openjdk-amd64//bin/java -classpath CommunityModules.jar:/home/markus/.vscode-server/extensions/alygin.vscode-tlaplus-1.5.0/tools/tla2tools.jar -XX:+UseParallelGC -Dtlc2.TLC.ide=vscode tlc2.TLC BlockingQueue.tla -tool -modelcheck -coverage 0 -config BlockingQueue.cfg -deadlock -workers 4 -dump dot BlockingQueue

TLC2 Version 2.15 of Day Month 20?? (rev: eb3ff99)
Running breadth-first search Model-Checking with fp 58 and seed<PHONE_NUMBER>791825383 with 4 workers on 8 cores with 17148MB heap and 64MB offheap memory [pid: 540] (Linux 4.19.84-microsoft-standard amd64, Ubuntu 11.0.7 x86_64, MSBDiskFPSet, DiskStateQueue).
Starting SANY...
Parsing file /home/markus/blockingqueue/BlockingQueue.tla
Parsing file /tmp/Naturals.tla
Parsing file /tmp/Sequences.tla
Parsing file /tmp/FiniteSets.tla
Parsing file /tmp/SequencesExt.tla
Parsing file /home/markus/blockingqueue/TLAPS.tla
Parsing file /tmp/FiniteSetsExt.tla
Parsing file /tmp/Functions.tla
Semantic processing of module Naturals
Semantic processing of module Sequences
Semantic processing of module FiniteSets
Semantic processing of module FiniteSetsExt
Semantic processing of module Functions
Semantic processing of module SequencesExt
Semantic processing of module TLAPS
Semantic processing of module BlockingQueue
SANY finished.
Starting... (2020-06-15 09:13:34)
com.fasterxml.jackson dependencies of Json overrides not found, Json module won't work unless the libraries in the lib/ folder of the CommunityModules have been added to the classpath of TLC.
Implied-temporal checking--satisfiability problem has 1 branches.
Computing initial states...
Finished computing initial states: 1 distinct state generated at 2020-06-15 09:13:34.
Progress(16) at 2020-06-15 09:13:35: 12,313 states generated, 1,600 distinct states found, 0 states left on queue.
Checking temporal properties for the complete state space with 1600 total distinct states at (2020-06-15 09:13:35)
Temporal properties were violated.
The following behavior constitutes a counter-example:
1: <Initial predicate>
/\ buffer = <<>>
/\ waitSet = {}
2: <Put line 34, col 1 to line 39, col 16 of module BlockingQueue>
/\ buffer = <<p4>>
/\ waitSet = {}
3: <Put line 34, col 1 to line 39, col 16 of module BlockingQueue>
/\ buffer = <<p4, p4>>
/\ waitSet = {}
4: <Put line 34, col 1 to line 39, col 16 of module BlockingQueue>
/\ buffer = <<p4, p4, p3>>
/\ waitSet = {}
5: <Put line 34, col 1 to line 39, col 16 of module BlockingQueue>
/\ buffer = <<p4, p4, p3>>
/\ waitSet = {p1}
6: <Put line 34, col 1 to line 39, col 16 of module BlockingQueue>
/\ buffer = <<p4, p4, p3>>
/\ waitSet = {p1, p2}
7: <Put line 34, col 1 to line 39, col 16 of module BlockingQueue>
/\ buffer = <<p4, p4, p3>>
/\ waitSet = {p1, p2, p4}
8: <Put line 34, col 1 to line 39, col 16 of module BlockingQueue>
/\ buffer = <<p4, p4, p3>>
/\ waitSet = {p1, p2, p3, p4}
9: <Get line 42, col 1 to line 47, col 16 of module BlockingQueue>
/\ buffer = <<p4, p3>>
/\ waitSet = {p1, p2, p4}
10: <Get line 42, col 1 to line 47, col 16 of module BlockingQueue>
/\ buffer = <<p3>>
/\ waitSet = {p1, p2}
11: <Get line 42, col 1 to line 47, col 16 of module BlockingQueue>
/\ buffer = <<>>
/\ waitSet = {p1}
12: <Put line 34, col 1 to line 39, col 16 of module BlockingQueue>
/\ buffer = <<p4>>
/\ waitSet = {p1}
13: <Put line 34, col 1 to line 39, col 16 of module BlockingQueue>
/\ buffer = <<p4, p4>>
/\ waitSet = {p1}
5: Back to state: <Put line 34, col 1 to line 39, col 16 of module BlockingQueue>
Finished checking temporal properties in 00s at 2020-06-15 09:13:35
Progress(16) at 2020-06-15 09:13:35: 12,313 states generated (568,729 s/min), 1,600 distinct states found (73,903 ds/min), 0 states left on queue.
12313 states generated, 1600 distinct states found, 0 states left on queue.
Finished in 1310ms at (2020-06-15 09:13:35)

By the way, you could filter the coverage parameter to improve performance of constant expression evaluation.  Likewise, using https://github.com/tlaplus/tlaplus/blob/master/tlatools/org.lamport.tlatools/src/tlc2/tool/fp/NoopFPSet.java might shave off a few milliseconds.

Thank you! The cause of the problem is clear now. Expression evaluation is executed in a temporary directory, not in the original one. That's why a relative path has no effect there. I'll fix that.


feat: wrapper modal

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
this pr introduces a wrapper modal, which will act as a base for every other modals. includes functionality to close the modal on a click outside the modal or an ESC key press.

Why was this change needed? (You can also link to an open issue here)
fixes: #1231

Other information:

@ManishMadan2882 should I try adding something to prevent the modal from closing while training any upload?

@RohittCodes Yes, we should avoid closing of the modal while upload/train is in process, good catch!


Update tutorial-custom-images.md

It would be better to set image location to westeurope, as the source VM's location is westeurope in sample script of VM creation

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@pngfei : Thanks for your contribution to the Azure documentation! The author, @cynthn, has been notified to review your proposed change.

#please-close @pngfei thank you for the suggestion! We are discussing the best option for location to use across all of the docs for consistency. Once we decide, we will bulk update everything.


Fixes #83

Working in the airport...

I think you might have not wanted to check in test.png?

Where did that come from ?
one sec

we are expecting this to break the test. merging.


f****

What you were trying to do
What actually happened
How to reproduce

Step 1
Step 2
Step 3

Log details and game version
Computer details

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/35681322-f?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github).

@ahmed992293: hi there! If you're trying to report an issue, can you fill in the report template with some details on the problem?


Show a Dialog after each link is created

After each link is created, right now it only creates a corresponding card related to it. Instead of this, it should show a Dialog with the created link, a button to copy the link, different ways to share the URL and a button to generate a QR code and share it as a jpg

hey @monizb
Looking forward to contribute on this one.

Hey @paultech4u please go ahead :)

hey,i would  like to contribute to this issue

Hey @abhinavkrin right now this issue has been assigned, do you want to look into other open issues?

@paultech4u are you still on this?

Yes,  I am almost done.

@paultech4u  no issues :)

No issues @monizb

@monizb the different way to share should I  include social media sharing also

@paultech4u yes please, also the QR code


Implement class_getitem for failing types

Test failure might be related to https://github.com/RustPython/RustPython/issues/3934.


Is this repo necessary?

ref: https://github.com/kubernetes/community/issues/1721
Seems like it hasn't been touched in a while, and mostly refers to docs that are elsewhere.
I'm not sure there's much here worth moving to kubernetes-retired but we could certainly go that route if necessary.

/cc @mbohlool @brendandburns

@lavalamp there is this:
https://kubernetes.io/docs/reference/using-api/client-libraries/#officially-supported-kubernetes-client-libraries
which must be in the docs repo somewhere...
That list is more up to date anyway :)

Deleting this repo is fine with me. @mbohlool any thoughts?

I am fine with deleting this repo. We need to make sure all references to the docs in this repo are fixed.

spiffxp@spiffxp-macbookpro:kubernetes-client$ ag clients.md
spiffxp@spiffxp-macbookpro:kubernetes-client$ ag design-docs
python/devel/submodules.md
2:To comply with [client library structure requirement](https://github.com/kubernetes-client/community/blob/master/design-docs/clients-library-structure.md),

go-base/README.md
5:For more information refer to [clients-library-structure](https://github.com/kubernetes-client/community/blob/master/design-docs/clients-library-structure.md).

python-base/README.md
8:For more information refer to [clients-library-structure](https://github.com/kubernetes-client/community/blob/master/design-docs/clients-library-structure.md).

I have PR'ed out all the relevant links I could find.  Just to be sure, I'm going to move this to kubernetes-retired for a bit.  Can you confirm this covers everything?


Capacitive measurement

Hi,
I bought one chirp, and make another to test diferences.. I read from https://wemakethings.net/chirp/ that "chirp uses capacitive sensing to measure the moisture. 1MHz square wave (system clock is used for that) is output from the chip through a resistor into a big pad that together with the surrounding ground plane forms a parasitic capacitor. "
What i'm see in this code is just a diference between two readings. Can you explain how do you read the capacitance?
uint16_t getCapacitance() {
    capMeasurementInProgress = 1;
    uint16_t caph = adcReadChannel(CHANNEL_CAPACITANCE_HIGH);
    uint16_t capl = adcReadChannel(CHANNEL_CAPACITANCE_LOW);
    capMeasurementInProgress = 0;
    return 1023 - (caph - capl);
}

Thanks

I am reading the capacitance the same way as described, but the idea is extended a bit further. I am measuring both peak and dip levels of the resulting triangle wave, capacitance estimate is a difference of those two numbers. Note, I don't return a true capacitance value in Farrads, just an arbitrary number that changes with changing capacitance.

How do you know in microcontroller if we are on peak or dip level?


Not clicking all fields after update

Addon is no longer clicking all entered fields in the configuration after the latest update. Some fields simply remain unchecked.

Hi @Malvenue ,
We have fixed those changes and release another update with v3.2.1 can you please update and let us know if you still face same issue


AUR Arch linux packages

This is very interesting project, congratulations to the devs!
I have created 2 AUR packages for it using the dockerfiles as reference, fetching the patches from this repo.
They will install the curl-impersonate-chrome and curl-impersonate-firefox commands. I haven't added the scripts to add the browser headers though since they didn't matter much to me.
Maybe you would be interested on adding this to your readme?
https://aur.archlinux.org/packages/curl-impersonate-chrome
https://aur.archlinux.org/packages/curl-impersonate-firefox

This looks great, I can definitely add it to the README later.
About the wrapper scripts - their importance also comes from settings the correct --ciphers flag. It's different across browsers and even across versions (see the Firefox 95 vs the Firefox 91 scripts) and without it the TLS signature of curl wouldn't be identical to the browser. So I think it might be worth including them as well.

Alright,  I included the scripts.


Model migration problem

Hello,author,
Your work is great.Can I use the model trained on these data sets to process the salt body data?For example, SEG salt
Thank you very much for your attention

Hello, thanks for your interest. Unfortunately, the pre trained model cannot be directly applied to salt models because we did not include any salt body in our dataset. I would suggest to use other dataset that contains salt body structures to retrain, at least fine-tune or transfer learning, with our networks.

Thank you very much for your reply.Is there open source real data available to test the effectiveness of models trained in openfwi?

Sorry, but I do not know any open source dataset that is designed for salt body imaging. You may rescale SEG salt for a preliminary test with our model. However, for fine tuning the model, I would suggest you build a few samples, for example 1000, and start from our pretrained models. For retraining, you would probably need a larger dataset designed for the task.

You may find some open source data on SEG open data platform or the EDX platform of DOE: https://edx.netl.doe.gov/
Remember you will need to scale the data to our model capacity.

Thank you very much for your reply. Wish you good health and success in your work.
Thank you again.


In TLAB allocation sizes rounded up

I know there are a handful of prior issues related to allocation size handling, so forgive me if this is a duplicate of an existing question.
Currently, the object sampler records the object size as max(objectSize, HeapSamplingInterval): https://github.com/async-profiler/async-profiler/blob/ddc280dfc4a690cfcca6d19ec9d45706a75dbe1a/src/objectSampler.cpp#L146. i.e. if an object is smaller than the heap sampling interval, its size is rounded up to the heap sampling interval.
The alloc tracer seems to always report the actual object size: https://github.com/async-profiler/async-profiler/blob/ddc280dfc4a690cfcca6d19ec9d45706a75dbe1a/src/allocTracer.cpp#L61.
Both allocation tracers seem to rely on the same principle of emitting an event once a byte threshold has been crossed. So the difference in behavior here strikes me as odd.
In the case of the object sampler, I'm also questioning why the object size is rounded up. This results in over-reporting object sizes for small objects. An object that is 100 bytes is reported as the same size as an object of 100 kb. It normalizes all small objects to the same size. In flamegraph visualizations, this often results in object counts and size visualizations converging in shape, giving false impressions on the relationships between object allocation volume and their sizes.
I understand that during post processing of the raw events that one often weights each sample by the interval between samples. For object allocation sampling, the weighting would be the heap sampling interval. But typically this weighting is done post facto. In the case of CPU samples there are no weights in the JFR events and weighting is inferred from the sampling interval. So I'm confused why the JFR events emitted by the object sampler are throwing away the actual object size.
If the object sampler size rounding is intentional, would you consider a CLI argument to make the behavior configurable so users wanting to preserve the raw object sizes in the JFR recordings could do so?

Thanks for the explanation. I think this makes sense to me now.
And on further scrutiny, some code outside of async-profiler was reading the allocation and tlab size fields inconsistently, adding to my confusion.
It also didn't help that async-profiler's internal JFR types refer to ObjectAllocationInNewTLAB's fields as _instance_size and _total_size instead of the JFR metadata defined allocationSize and tlabSize. But I understand OpenJDK's terminology may have changed over the years creating this drift. 🤷♂
I'm still trying to make sense around whether reporting the sampling interval as the TLAB size is appropriate and what effects that may have on random JFR readers. But I can figure this out on my own by reading source code.
Thanks again.


Add build instructions for Windows

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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

Project coverage is 40.72%. Comparing base (a279f54) to head (9e40b80).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #49      +/-   ##
==========================================
- Coverage   40.79%   40.72%   -0.07%
==========================================
  Files          15       15
  Lines        8541     8540       -1
==========================================
- Hits         3484     3478       -6
- Misses       5057     5062       +5

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


Should not try to upgrade EF projects

If EF project contains Migrations, it will be broken after upgrade.
Detection could be done using presense of "Migrations" folder in project

An example csproj would be really helpful.


Add option to expand user path

Description
Add option to expand user path from ~ to /Users/foobar
Option is a Bool under Dir category and is named SPACESHIP_DIR_EXPAND_USER_PATH

Thanks for the PR. But I don't think this option is required now.


Fix package comment in component template

What does this PR do?

Fix the package comment in the component template.
Motivation

Not have linter errors just when using the template.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Adapt to upstream Spring Security 5.2 changes

Depends on spring-cloud/spring-cloud-dataflow#3572
Depends on spring-cloud/spring-cloud-common-security-config#57

Duplicates https://github.com/spring-cloud-task-app-starters/composed-task-runner/issues/102.


Fix 'pip install .'

Fix error :

flask-appbuilder 1.10.0 has requirement Flask-SQLAlchemy==2.1,
but you'll have flask-sqlalchemy 2.3.2 which is incompatible.
botocore 1.10.5 has requirement python-dateutil<2.7.0,>=2.1, but you'll
have python-dateutil 2.7.2 which is incompatible.

@mistercrunch isn't the fix to resolve the version specified in the requirements.txt file? I'm also interested in learning why this hasn't been an issue for Travis CI.

Probably because of https://github.com/apache/incubator-superset/blob/master/tox.ini#L29, if you install the requirements.txt first and then run pip install . then it works.
For me if I try a simple pip install . from an empty virtualenv I get the error above. Note that setuptools is not aware of requirements.txt, only of what's in setup.py

@mistercrunch I'll try to triage the problem seeing as I introduced the regression. If we remove the dependency we'll also need to update requirements.txt.

@mistercrunch I tested the following in Python 2.7 and 3.6 and wasn't able to reproduce your issue:
virtualenv env
source env/bin/activate
pip install -r requirements.txt
pip install .

@john-bodley but the point is that it doesn't work if you don't pip install -r requirements.txt.
When you pip install superset from pypi, it only knows about what's in setup.py, and it doesn't know about the requirements.txt file (it's not even in the package). I'm pretty sure if I was to publish what is in master (before this PR) to pypi, it would break pip install superset.

@mistercrunch sorry from the title it mentioned pip install . and thus I thought it was a local install. Note it seems like we don't use flask-sqlalchemy, i.e, the following produces no hits.
 grep -R flask_sqlalchemy *

I think your change is fine if you also remove the line from requirements.txt.

Yes what I was trying to get to is that pip install . is a local equivalent of how a pip install superset will behave once uploaded to PyPI and that it was broken.
flask-sqlalchemy is a FAB dep that somehow crawled its way into our setup.py probably at a point in time where it needed to be pinned and FAB had not pinned it yet. Now I'm unclear on whether it belongs in requirements.txt

Per my last comment, I recommend removing the line from requirements.txt as it’s unused by Superset directly.


Fix: Remove unnecessary async state machine generation

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes #9871

Validation
How did you test these changes?

[*] Built and ran the app

Screenshots (optional)
Add screenshots here.

That should be it then.

@hez2010 Is everything ok now?


[QST]some question about nsight system

What is your question?

my test data is gzip compress ，picture shows 2 stream executed on gpu , as you see,  ptds is on , and my first 2 question is :

as red block, there are 2 times memory copy from host to device ,between is inflate_kernel i guess is decompression file , what's differences of 2 times memory copy ? and at the same time ,the first copy is pageable ,and the second is pinned , and after the second copy , there are gpuBuildStringDictionaryIndex and gpuDecodePageData and I guess this is decode parquet file ，and why the second copy size every time is 8bytes ,and looks like is Inefficiency .
as yellow block, there is no gpu action , and now Where is the bottleneck ,Why does this happen and how to optimize it

my last question is  my nsight system looks like abnormal ，java tag is empty ,as shown in the figure below and what i expect is operators nvtx  like " buffer file split " ,"Parquet readBatch  Acquire GPU "

my nsys trace way is
first , i stop a worker process of node A, and modify spark-class file ,replaced "exec "${CMD[@]}" with "exec nsys profile --trace-fork-before-exec=true --trace=cuda,openacc,nvtx "${CMD[@]}"
second , i restart a worker process by "./start-slave.sh spark://master:7077"
finally, i run spark-shell and run a query,
after that , i go to node A ,and kill the woker process which is doesn't launched by "nsys profile -"

Thanks for the traces, @chenrui17!
For the first question, this appears to be within libcudf's Parquet decoder, as all the behavior in question occurs within that read_parquet sample within libcudf.  If you have a small repro case, it would be great to file a question in the https://github.com/rapidsai/cudf project, or if you attach it here I can do the same.
If you have configured a sufficient pinned memory pool for the SQL plugin, the buffer being handed to the libcudf parquet decoder will be pinned memory.  Assuming you have configured enough pinned memory to hold the parquet data across the concurrent tasks, pageable transfers may be from CPU buffers computed within the libcudf call.
For the second question about missing Java traces, cudf Java does not generate NVTX traces by default and will only do so if the system property ai.rapids.cudf.nvtx.enabled=true.  The NVTX profiling guide details how to setup this property and how to setup profiling in general.  It sounds like your general approach to profiling works, and I suspect the system property isn't being set which would explain how you're getting libcudf traces but not Java traces.


CURLINFO_TOTAL_TIME

Thanks!

@zachborboa - I hate to do this on a merged PR, but whereas the total time is not something that will always be accessed, and this happens in the exec call, it seems not great to add the (minimal, to be sure, but still present) overhead of calling that function for something that may not be used in 90% of use-cases.  For instance, I will nearly never use that information in my usage, but this will always be called...
In a case like #314, this would be moot, and an interesting thing to have.  But otherwise, it seems unnecessary...

@thebeline You raise a good point. Do you think Curl::totalTime would be best removed and do which of the following:

Left as is and call as needed; e.g. $totalTime = curl_getinfo($curl->curl, CURLINFO_TOTAL_TIME);
Implemented as Curl::getInfo so $totalTime = $curl->getInfo(CURLINFO_TOTAL_TIME);
Implemented as Curl::getTotalTime so $totalTime = $curl->getTotalTime();

I'm leaning towards implementing Curl::getInfo as this will also allow retrieving other information listed on https://secure.php.net/manual/en/function.curl-getinfo.php
$totalTime = $curl->getInfo(CURLINFO_TOTAL_TIME);
$uploadSpeed =  = $curl->getInfo(CURLINFO_SPEED_UPLOAD);
$downloadSpeed =  = $curl->getInfo(CURLINFO_SPEED_DOWNLOAD);

Personally?  I like my deferred properties PR, and we could add all of those to the deferred properties list, so:
$total_time = $curl->total_time;
$upload_speed = $curl->speed_upload;
// etc.

This will cache the results, too, referencing $curl->total_time; multiple times just pulls the property on subsequent calls.
That said, sure, I mean, implementing getInfo() would be good, it could even be used with deferred properties.  (I'm partial, I know)


[stable/graylog] multiple tcp input with NodePort

Describe the bug
We can not define multiple tcp inputs with NodePort. Only one NodePort is possible because the template takes the NodePort-value outside of the Loop: https://github.com/helm/charts/blob/master/stable/graylog/templates/tcp-service.yaml#L27
What the template suggests:
input:
    tcp:
      service:
        type: NodePort
        nodePort: 31488
      ports:
        - name: gelf
          port: 31488
        - name: json
           port: 31489

What we want to define:
input:
    tcp:
      service:
        type: NodePort
      ports:
        - name: gelf
          port: 31488
          nodePort: 31488
        - name: json
          port: 31489
          nodePort: 31489

Version of Helm and Kubernetes:
Helm: 2.13.1
Kubernetes: 1.14.3
Which chart:
stable/graylog
What happened:
We can not define multiple NodePort Inputs for Graylog (GELF and JSON) for different sources of Logfiles.
What you expected to happen:
We want to define it via HELM-values.yaml
How to reproduce it (as minimally and precisely as possible):
Put the snippet above into the values.yaml and deploy a graylog instance in K8s. You can not define multiple NodePort TCP inputs.
Anything else we need to know:
The Fix is very simple for my purpose:
In https://github.com/helm/charts/blob/master/stable/graylog/templates/tcp-service.yaml#L27
spec:
  ports:
  {{- range .Values.graylog.input.tcp.ports }}
    - name: {{ .name }}
      port: {{ .port }}
      protocol: TCP
      targetPort: {{ .port }}
      nodePort:  {{ .nodePort }}
  {{- end }}

But i don't know weather it works for different kind of input definitions like LoadBalancer, etc ...

I can confirm this bug.
The fix suggested is working fine too

any alternative? I dont have the possiblity to modify the helm chart in our usage..

any alternative? I dont have the possiblity to modify the helm chart in our usage..  @Andrioshe
Why can't you?
You can always do like this:
helm fetch stable/graylog
untar the file and just add this line after targetPort: {{ .port }} (line 23) in graylog/templates/tcp-service.yaml
       nodePort:  {{ .nodePort }}

When installing the helm, use something like
helm upgrade --install graylog --namespace graylog -f values.yaml path_to_extracted_graylog_helm

I am limited to the helm charts provided by the catalog in rancher. I dont have the permissions to modify these.


Updated Fleet Engine package names

Fleet Engine Java package name
We’re making a name change to the Java packages in the Fleet Engine gRPC client libraries. The package names changed from "google.maps.fleetengine." to "com.google.maps.fleetengine.".  We’re making this change to bring the packages aligned with standard Google package names to allow us to deploy these libraries through standard Maven distribution mechanisms.

This has been obsoleted by: https://github.com/googlemaps/last-mile-fleet-solution-samples/pull/216


Multiple Heatpumps commands not send

** Versions **

Home Assistant version: 2024.10.3
This integration version (for instance 1.1.0): 1.4.1

Hi,
I have added a second heatpump by changing the second topic prefix to panasonic_heat_pump2.
It´s working i got all entities. But now i have the problem that sometimes commands are not beeing send.
For example:
Right now i set Force DHW Mode. The switch jumps up. But after refresh its back down and state is not changed to dhw only.
Maybe 1h later i set Force DHW Mode and it works immediately.
Same think for other parameter like Aquarea Mode or heat request shift.
Sometimes the command works in real time and sometimes its not working at all.
Any idea where the problem could be?
Thanks a lot

Hello, thanks for reporting this.
It's hard to know what could be the issue. Ideally you would look at a MQTT software such as https://mqtt-explorer.com/ to see "live" the effect of changing a switch such as Force DHW Mode.
In theory, you should see immediately an MQTT message being sent on the right topic. If you don't, the bug is in this integration.
If you do see the message however, it's possible there is a conflict between heishamon and HA view.
Could you try to reproduce the issue while observing MQTT messages? It will help to see how we could fix this.

Hi,
Thanks a lot. I checked it now. All commands are visible in mqtt explorer. I checked the Heisha monitor and i see, that i got an reconnect to MQTT every 10 seconds - WIFI is on 100%.
What could be the problem here?

To be honest I don’t know.
I think you can try to make sure you run with the latest version of heishamon and then report the issue on Heishamon repository (or the community slack).
From the point of view of this integration, I don’t think we can do better!


Crashing in +[DDLog isRegisteredClass] in ARC+Simulator

Hi,
It SEEMS like the method +[DDLog isRegisteredClass] will crash a project when using it with ARC + Simulator.
Environment:
Lion 10.7.3
iOS 5 SDK
Xcode 4.2.1
iPhone 5.0 Simulator
Currently, the method contains a macro #if check that reads:
if TARGET_OS_IPHONE (near line 307)
For running in the Simulator, it needs to read:
if TARGET_OS_IPHONE && !TARGET_IPHONE_SIMULATOR
such that the non-iOS code gets executed.
If the code is allowed to stand, it will trip up on the following classes:
"ARCLite"
"__NSGenericDeallocHandler"
"NSZombie"
"Object"
"__IncompleteProtocol"
"NSLeafProxy"
"Protocol"
"NSMessageBuilder"
which will throw a does not implement "-methodSignatureForSelector" exception on NSInvocation.
Thanks.

Thanks for the bug report.

This issue is not entirely fixed it is still crashing on iPad version 5.0.1 over "Object"

With iOS 17 the non-IOS code in the Simulator part does have some weird issues!
UserDefaults stop working!
But with iOS 17, the normal iOS Code just works fine, even in the simulator.
So the current version does not work on iOS 17 simulators (or at least UserDefaults do not work after calling)


Don't dereference null pointer if elements don't exist

StageTestbed.ts tried to use playButton, statusElement, and infoElement without checking whether they existed. This change fixes that.
Fixes #276

Looks good, thanks for the fix!

Thanks, btw where is the documentation source? I could submit a PR that would fix that up to add the testbed elements. I also noticed a few errors in the example in https://piqnt.com/planck.js/docs/rendering that I would like to fix.

@jes Thanks for making updates, I added used manula docs to /docs folder in this repo.


Replacement hook name to deprecated hook warnings

[Warning] AutoDoorCloser plugin is using deprecated hook: CanOpenDoor. Please encourage the plugin's author to update to the new hook: CanUseDoor. Deprecated hook will be removed on May 17th, 2015.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/11794037-replacement-hook-name-to-deprecated-hook-warnings?utm_campaign=plugin&utm_content=tracker%2F6855523&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F6855523&utm_medium=issues&utm_source=github).

Completed by @Mughisi, pushing soon.


Eel on Phone

I have ready project and exe file on PC, but i want convert this project on my phone
How i can that do?

No sorry, I don't think this would be possible.

No sorry, I don't think this would be possible.


Add vcpkg cache secrets for C++

This file also needs to change to include the variable group -- https://github.com/Azure/azure-sdk-for-cpp/blob/main/eng/pipelines/prepare-pipelines.yml#L9

Don't use the Test KV, use the EngSys KV.

/check-enforcer reset

/check-enforcer evaluate

/check-enforcer override


(Feature) #1448 user should see privacy terms in same screen

Description
Merge privacy policy ant terms menu items into one.
About #1448
How Has This Been Tested?
Open side menu - see changes
Checklist:

[x] PR title matches follow: (Feature|Bug|Chore) Task Name
[x] My code follows the style guidelines of this project
[x] I have followed all the instructions described in the initial task (check Definitions of Done)
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have added reference to a related issue in the repository
[x] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[ ] I have added screenshots related to my pull request ( for frontend tasks)
[x] I have pasted a gif showing the feature.
[x] @mentions of the person or team responsible for reviewing proposed changes

Video: https://drive.google.com/file/d/1LmlLSrv3p7lVgz7CkgKAW5xUzp6YhGK_/view

@yaroslav-fedyshyn-nordwhale change to the links in #1449

@yaroslav-fedyshyn-nordwhale change to the links in #1449

done


Fix the package versions in the CI-generated wheels

The wheels generated by cibuildwheel in our CI pipeline currently have the wrong pyodbc package version id.s  For example, the file "pyodbc-4.0.dev0-cp310-cp310-win_amd64.whl".  "4.0.dev0" is incorrect and should be something like "4.0.35".  The wheels are given the id "4.0.dev0" because setup.py is unable to find any tags in the available git repo.  When building pyodbc locally on laptops, setup.py usually figures out the version from the git tags in the local repo.  setup.py is unable to figure this out in the CI pipeline because Github Actions does not include the tags when it fetches the repo.  It uses a command similar to:
/usr/bin/git -c protocol.version=2 fetch --no-tags --prune --progress --no-recurse-submodules --depth=1 origin +cafa37e846e09b36ebf821ac69b6f7d35f423baa:refs/remotes/origin/fix-labels-on-cicd-wheels

Note the options --no-tags and --depth=1.
To get around this, I have introduced a constant to setup.py called VERSION, currently of value "4.0.35".  When setup.py is called within the CI pipelines, setup.py will detect this and use the VERSION constant.  This constant will naturally have to be manually maintained but that shouldn't be too onerous.
Whilst here, I have added a basic pyproject.toml file to help pyodbc be more PEP517-compliant.  We should probably make more use of pyproject.toml in the future, transferring the static elements of setup.py into it at some point.
Also, a tweak to .gitignore.

Just to emphasize, once this PR is merged, we should be able to let cibuildwheel in Github Actions build all our wheels for us going forward, for uploading to PyPi.  cibuildwheel has the big advantage of being able to build wheels for a wide range of operating systems, and any future ones too.
cibuildwheel won't build Python 2.7 wheels anymore but we can get AppVeyor to generate one last set of 2.7 Windows wheels if needed (by setting APVYR_GENERATE_WHEELS to "true" in the AppVeyor portal).  As I mentioned before, maintaining support for Python 2.7 is becoming a bit problematic.  Not many other Python packages, utilities, and applications support 2.7 anymore.  It might be time to drop 2.7 and move on to version 5 of pyodbc.

FYI, I've uploaded a deployment based on this branch to the Test PyPi index, here:
https://test.pypi.org/project/pyodbc-keef-v1/
The installed package name is the same, i.e. "pyodbc", it's just a different catalog entry in the Python Package Index.
Feel free to try installing the wheels at your leisure:
pip install -i https://test.pypi.org/simple/ pyodbc-keef-v1
Note, the linux wheels do not include any .so files anymore.

Thanks, Gord.  I too have checked the install works from the Test PyPi on Windows 64-bit (Python 2.7, 3.6-3.11), Linux Ubuntu 20.04 (Python 3.10), and Mac (not ARM) (Python 3.8).  I'll merge this PR now to get it in the master branch, but if we need to make changes subject to review by Kleehammer, that will be easy enough.  It'll be great to drop Python 2.7 at some point but it'll be even better to fix the Linux wheels through version 4.0.35 in the meantime.

Further to this PR, here are my thoughts on how a new release should be deployed.  Happy to hear any feedback on this.  Maybe add this to the Wiki?

The package version for the new release should already be defined in setup.py (see the VERSION constant).  For example "5.2.0".  If that value is not correct, change it and commit.

Choose the commit that will be used for the release.  This will almost always be the head of the master branch, but it can be any commit.

In your local pyodbc repository, create an annotated git tag for the chosen commit, as follows:
(If you are not already on the commit git checkout <commit hash>)
git tag -a <package version>
You will be prompted to enter a message for the tag.  Give a short description of the release, a full description will be added to the Github release notes later.
Push this tag to Github.

Delete all the files in your top-level "dist" directory (create the "dist" directory if necessary).

From the Github Actions page, find the commit in the "Ubuntu build" workflow and download the two zipped artifacts "sdist" and "wheels" into your "dist" directory.  These artifacts cover all the files for Python 3.
https://github.com/mkleehammer/pyodbc/actions/workflows/ubuntu_build.yml

From the AppVeyor History page, find the commit and download the two artifacts for Python 2.7 (both 32-bit and 64-bit) from the build page into your "dist" directory.
https://ci.appveyor.com/project/mkleehammer/pyodbc/history
(If there are no artifacts, go to the Environment page in Settings, and set the environment variable "APVYR_GENERATE_WHEELS" to "true", then re-build the commit from the build page)

In your "dist" directory, unzip the two zip files "sdist" and "wheels" to retrieve the sdist/wheel files within them.  Delete the zip files.  Don't unzip the AppVeyor .tar.gz files.

From the command prompt, navigate to the top-level pyodbc directory, and then upload the contents of the "dist" directory to the Test PyPi index:
twine upload -r testpypi dist/*
Check https://test.pypi.org/project/pyodbc/ to make sure the files were uploaded successfully.

If that is successful, upload the contents of the "dist" directory to the real PyPi index.
Bear in mind, you get only one shot at this!  You can't replace files in PyPi, so make sure you get this right first time:
twine upload dist/*
Check https://pypi.org/project/pyodbc/ to make sure the files were uploaded successfully.

Create release notes on the Github portal, based on the git tag created earlier.  This will be the main source of information about the release so make it reasonably comprehensive.
https://github.com/mkleehammer/pyodbc/releases

Finally, increment the VERSION constant in setup.py to the next release, and commit.  For example, from "5.2.0" to "5.3.0".  And that's it!


iOS Support

Description
Would be cool to have the wallet on iOS.

Thanks for your suggestion!
Unfortunately iOS support is not planned.
Here are the reasons why:

BitBanana is a native Android app. This means the App has to be rewritten from scratch in order to work for iOS.
Additionally supporting iOS would take a huge amount of time. Time that would be missing to do the Android implementation right. I rather focus on one platform and do it in a proper way than having a half decent thing on multiple platforms.
I am not familiar with the apple ecosystem and I don't have any Apple deviced needed for depoying and testing.

In short, I am not going to do it, sorry. But BitBanana is open source under MIT License. Whoever wants to create something similar for iOS is welcome to do so.
If you are looking for node management on iOS, try Zeus.
I am keeping this issue open for visibility.

I am keeping this issue open for visibility reason.


Highlighted text in Peek Declaration list is the same color as the background

Right-click on a symbol and select Peek Declaration. The highlighted text in the list of results has no contrast against the background. (Peek Declaration may be specific to the C/C++ extension when Intellisense is configured for the project. I can provide whatever you need if you don't have code where this shows up.)
Night Owl:

Dark+ (Default):

@lauragift21 fixed this! Thank you Gift.


fix(dashboard): label colors included in explore url

SUMMARY
Due to changes from https://github.com/apache/superset/pull/16444, label_colors from dashboard were being included in chart's URL when user visited explore from dashboard. This PR partially reverts the changes from https://github.com/apache/superset/pull/16444, while still preserving the perf improvement introduced in that PR
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before - example link of Proportion of Monthly Revenue by Product Line chart from Sales Dashboard (2142 characters):
http://localhost:9000/superset/explore/?form_data={"viz_type"%3A"area"%2C"datasource"%3A"20__table"%2C"slice_id"%3A96%2C"url_params"%3A{}%2C"time_range_endpoints"%3A["inclusive"%2C"exclusive"]%2C"granularity_sqla"%3A"order_date"%2C"time_grain_sqla"%3A"P1M"%2C"time_range"%3A"No+filter"%2C"metrics"%3A[{"aggregate"%3A"SUM"%2C"column"%3A{"column_name"%3A"sales"%2C"description"%3Anull%2C"expression"%3Anull%2C"filterable"%3Atrue%2C"groupby"%3Atrue%2C"id"%3A917%2C"is_dttm"%3Afalse%2C"optionName"%3A"_col_Sales"%2C"python_date_format"%3Anull%2C"type"%3A"DOUBLE+PRECISION"%2C"verbose_name"%3Anull}%2C"expressionType"%3A"SIMPLE"%2C"hasCustomLabel"%3Afalse%2C"isNew"%3Afalse%2C"label"%3A"(Sales)"%2C"optionName"%3A"metric_3is69ofceho_6d0ezok7ry6"%2C"sqlExpression"%3Anull}]%2C"adhoc_filters"%3A[]%2C"groupby"%3A["product_line"]%2C"limit"%3A100%2C"order_desc"%3Atrue%2C"contribution"%3Atrue%2C"row_limit"%3Anull%2C"show_brush"%3A"auto"%2C"show_legend"%3Atrue%2C"line_interpolation"%3A"linear"%2C"stacked_style"%3A"stack"%2C"color_scheme"%3A"bnbColors"%2C"label_colors"%3A{"SUM(Sales)"%3A"%23ff5a5f"%2C"Medium"%3A"%237b0051"%2C"Small"%3A"%23007A87"%2C"Large"%3A"%2300d1c1"%2C"SUM(SALES)"%3A"%238ce071"%2C"Classic+Cars"%3A"%23ffb400"%2C"Vintage+Cars"%3A"%23b4a76c"%2C"Motorcycles"%3A"%23ff8083"%2C"Trucks+and+Buses"%3A"%23cc0086"%2C"Planes"%3A"%2300a1b3"%2C"Ships"%3A"%2300ffeb"%2C"Trains"%3A"%23bbedab"}%2C"rich_tooltip"%3Atrue%2C"bottom_margin"%3A"auto"%2C"x_ticks_layout"%3A"auto"%2C"x_axis_format"%3A"smart_date"%2C"y_axis_format"%3A"SMART_NUMBER"%2C"y_axis_bounds"%3A[null%2Cnull]%2C"rolling_type"%3A"None"%2C"comparison_type"%3A"values"%2C"annotation_layers"%3A[]%2C"extra_form_data"%3A{}}
After (1709 characters):
http://localhost:9000/superset/explore/?form_data={"viz_type"%3A"area"%2C"datasource"%3A"20__table"%2C"slice_id"%3A96%2C"url_params"%3A{}%2C"time_range_endpoints"%3A["inclusive"%2C"exclusive"]%2C"granularity_sqla"%3A"order_date"%2C"time_grain_sqla"%3A"P1M"%2C"time_range"%3A"No+filter"%2C"metrics"%3A[{"aggregate"%3A"SUM"%2C"column"%3A{"column_name"%3A"sales"%2C"description"%3Anull%2C"expression"%3Anull%2C"filterable"%3Atrue%2C"groupby"%3Atrue%2C"id"%3A917%2C"is_dttm"%3Afalse%2C"optionName"%3A"_col_Sales"%2C"python_date_format"%3Anull%2C"type"%3A"DOUBLE+PRECISION"%2C"verbose_name"%3Anull}%2C"expressionType"%3A"SIMPLE"%2C"hasCustomLabel"%3Afalse%2C"isNew"%3Afalse%2C"label"%3A"(Sales)"%2C"optionName"%3A"metric_3is69ofceho_6d0ezok7ry6"%2C"sqlExpression"%3Anull}]%2C"adhoc_filters"%3A[]%2C"groupby"%3A["product_line"]%2C"limit"%3A100%2C"order_desc"%3Atrue%2C"contribution"%3Atrue%2C"row_limit"%3Anull%2C"show_brush"%3A"auto"%2C"show_legend"%3Atrue%2C"line_interpolation"%3A"linear"%2C"stacked_style"%3A"stack"%2C"color_scheme"%3A"bnbColors"%2C"label_colors"%3A{}%2C"rich_tooltip"%3Atrue%2C"bottom_margin"%3A"auto"%2C"x_ticks_layout"%3A"auto"%2C"x_axis_format"%3A"smart_date"%2C"y_axis_format"%3A"SMART_NUMBER"%2C"y_axis_bounds"%3A[null%2Cnull]%2C"rolling_type"%3A"None"%2C"comparison_type"%3A"values"%2C"annotation_layers"%3A[]%2C"extra_form_data"%3A{}}
TESTING INSTRUCTIONS

Go to a dashboard
Click "View chart in Explore"
Verify that label colors are not included in url

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

CC @junlincc @serenajiang

The issue for "explore URL" is not really caused by #16444. This line
 <a href={this.props.exploreUrl} rel="noopener noreferrer">

will generate a GET request when user click on it, but browser (and many companies using nginx) has limitation for length of GET request. In airbnb we had a lot of charts with very long parameters, these charts couldn't be open by GET request. #16444 added label colors into url makes things worse, but it is no the root cause.
Note, originally, there was an onClick event handler for this View chart in explore menu item, but it was lost since #11554. In the event handler, it generate a POST request to open the chart since GET won't work for long url.
Ideally we should avoid open chart using GET request.


update HTML & CSS

need to clean up and update both HTML and CSS code for upgrading accessibility on webpage

Code for both HTML and CSS has been cleaned up to show semantic elements as well as overall structure management. Changed layout of benefits section to read from top down horizontally to help with accessibility as well.


Having the thermostat information

Hello,
User of your node, I also need to get my Netatmo thermostat information.
It seems to be possible to do it with the node package of karbassi, but not with your node-red package...
Thanks in advance !

Glad you are getting value from it.   I myself don't have a thermostat to test with.   Feel free to fork and play.  It's really simple code and not much of it.
Cloning one of the existing nodes and hacking it for the thermostat API should be relatively easy.
Sent from my iPhone

On Jan 11, 2017, at 3:58 PM, Nekloth<EMAIL_ADDRESS>wrote:
Hello,
User of your node, I also need to get my Netatmo thermostat information.
It seems to be possible to do it with the node package of karbassi, but not with your node-red package...
Thanks in advance !
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Arrgg... I'm not good at coding... I may try to fork, but I don't even know where to start to code it (html/js/npm...)
Ok, anyway, thank you ! :)

try https://nodered.org/docs/creating-nodes/
Thats the guide I followed.
Try the example they use first to get the idea.
And have fun!!
Regards,
Sam
Sam S. Adams, Cognitive Eldercare
IBM Master Inventor, IBM Academy of Technology
IBM Distinguished Engineer, IBM Research
Mobile:<PHONE_NUMBER>, email<EMAIL_ADDRESS>Assistant: Jill Taylor/Austin/Contr/IBM<EMAIL_ADDRESS>720-430-9048
<<Hebrews 11:6, Proverbs 3:5-6, Romans 1:16-17, 1 Corinthians 1:10>>
From:	Nekloth<EMAIL_ADDRESS>To:	ssadams11/node-red-contrib-netatmo
<EMAIL_ADDRESS>Cc:	Sam Adams/Raleigh/IBM@IBMUS, Comment
<EMAIL_ADDRESS>Date:	01/12/2017 04:07 AM
Subject:	Re: [ssadams11/node-red-contrib-netatmo] Having the thermostat
information (#3)
Arrgg... I'm not good at coding... I may try to fork, but I don't even know
where to start to code it (html/js/npm...)
Ok, anyway, thank you ! :)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

OK, I'll try.
Thank you.

Final question: do you suggest an evolution of your package or a new package dedicated for the Thermostat
(first answer will lead me to fin another npm node-red package name...)

You could build a separate package for thermostat only,
node-red-contrib-netatmo-thermostat.  That might be easier.  Use any of my
code you want.
Then I could reintegrate you work later into a more complete package.
Or you could just fork and enhance my package, save it as
node-red-contrib-netatmo-nek or something so you can use it directly, then
I could jsut accpet your enhacnements and republish a new version of the
whole package.
Regards,
Sam
Sam S. Adams, Cognitive Eldercare
IBM Master Inventor, IBM Academy of Technology
IBM Distinguished Engineer, IBM Research
Mobile:<PHONE_NUMBER>, email<EMAIL_ADDRESS>Assistant: Jill Taylor/Austin/Contr/IBM<EMAIL_ADDRESS>720-430-9048
<<Hebrews 11:6, Proverbs 3:5-6, Romans 1:16-17, 1 Corinthians 1:10>>
From:	Nekloth<EMAIL_ADDRESS>To:	ssadams11/node-red-contrib-netatmo
<EMAIL_ADDRESS>Cc:	Sam Adams/Raleigh/IBM@IBMUS, Comment
<EMAIL_ADDRESS>Date:	01/13/2017 06:57 AM
Subject:	Re: [ssadams11/node-red-contrib-netatmo] Having the thermostat
information (#3)
Final question: do you suggest an evolution of your package or a new
package dedicated for the Thermostat
(first answer will lead me to fin another npm node-red package name...)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Thank you for your answer.

I have a first working version of thermostat packages but there are side effects. I'll try to see how to solve them (but it is maybe on @karbassi project ... or Netatmo side)

No problem.  I've only done one fork-pull myself.
We all learning!  ;-)
Regards,
Sam
Sam S. Adams, Cognitive Eldercare
IBM Master Inventor, IBM Academy of Technology
IBM Distinguished Engineer, IBM Research
Mobile:<PHONE_NUMBER>, email<EMAIL_ADDRESS>Assistant: Jill Taylor/Austin/Contr/IBM<EMAIL_ADDRESS>720-430-9048
<<Hebrews 11:6, Proverbs 3:5-6, Romans 1:16-17, 1 Corinthians 1:10>>
From:	Nekloth<EMAIL_ADDRESS>To:	ssadams11/node-red-contrib-netatmo
<EMAIL_ADDRESS>Cc:	Sam Adams/Raleigh/IBM@IBMUS, Comment
<EMAIL_ADDRESS>Date:	01/13/2017 08:10 AM
Subject:	Re: [ssadams11/node-red-contrib-netatmo] Having the thermostat
information (#3)
Thank you for your answer.

I have a first working version of thermostat packages but there are side
effects. I'll try to see how to solve them (but it is maybe on @karbassi
project ... or Netatmo side)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Yep !
Thank you

Hello,
Just to let you know I publish a node dedicated for the Netatmo thermostat:
https://www.npmjs.com/package/node-red-contrib-netatmo-thermostat
&
https://github.com/nekloth/node-red-contrib-netatmo
Thank you for your help !
Nekloth

Done somewhere else ;-)


"-f" flag not recognized as alternative to "--features"

Prerequisites
Please put an X between the brackets as you perform the following steps:

[X] Check that your issue is not already filed:
https://github.com/leanprover/lean4/issues
[X] Reduce the issue to a minimal, self-contained, reproducible test case.
Avoid dependencies to Mathlib or Batteries.
[X] Test your test case against the latest nightly release, for example on
https://live.lean-lang.org/#project=lean-nightly
(You can also use the settings there to switch to “Lean nightly”)

Description
lean -h states that lean -f may be used as shorthand for lean --features.
  -f, --features         display features compiler provides (eg. LLVM support)

while lean --features works as intended, -f is not recognized as a valid flag.
Steps to Reproduce
elan run nightly lean -f

Expected behavior:
Identical output to lean --features
Actual behavior:
/home/james/.elan/toolchains/nightly/bin/lean: invalid option -- 'f'
Unknown command line option
Lean (version 4.12.0-nightly-2024-10-12, x86_64-unknown-linux-gnu, commit a3bc4d235961, Release)
Miscellaneous:
...

Versions
Output of elan run nightly lean -v
Lean (version 4.12.0-nightly-2024-10-12, x86_64-unknown-linux-gnu, commit a3bc4d235961, Release)
OS Version: Ubuntu 22.04.3 LTS
Impact
Minimal to no impact, just an inconsistency between information in the help command and actual behavior.

Thanks for raising this. Care to make a PR that adjusts the help string to reality? Probably the easiest course of action. Ideally before any bike shedding about the CLI interface ensues :-)


Update ActivationRuleGroupTest.java

modify a piece of code in whenNoSelectedRule_thenNothingShouldHappen unitCase .
the rule2 xor rule3  should be set to rule4

Coverage remained the same at 89.594% when pulling f0d06564a7d958a23cf7a96d160fbb9747f75e94 on zhhaojie:patch-1 into f6a8b9a6faad63e8e1f86f5936d34040301dee3c on j-easy:master.


Check for newlines before and after headers

This builds off https://github.com/hackedu/markcop/pull/25 and adds another function

Waiting on #25 to be merged. I commented over there with some issues I ran into with it.

Does #31 replace this?


[Event Hubs] Samples links

Summary
The focus of these changes is to cross-link the repository samples in the doc comments for each client and options type, to aid discoverability.

API change check
API changes are not detected in this pull request.


fix(index): add missing components to build

This adds components to the index of all packages which will make them build since
packages/index.ts is our entry point of typescript compiler.

@Poltergeist , build is failing.


Median filter error: Failed to execute 'importScripts' on 'WorkerGlobalScope': The script at 'http://localhost:9999/itk/Pipelines/itkfilteringWasm.js' failed to load.

ParaViewGlance: master
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/111.0.1661.54
Median filter error: Failed to execute 'importScripts' on 'WorkerGlobalScope': The script at 'http://localhost:9999/itk/Pipelines/itkfilteringWasm.js' failed to load.

I'm guessing this is due to build artifacts being incorrectly referenced. I can look into this.

After I cloned the project to local, I modified the config code,
{
from:path.join(paths.itk_path,'web-build'),
to:path.join('itk', 'Pipelines'),
// to: '[name][ext]',
},
which did not report error failed to load, but then reported error Median filter error: undefined

I'm guessing this is due to build artifacts being incorrectly referenced. I can look into this.

After I cloned the project to local, I modified the config code,
{
from:path.join(paths.itk_path,'web-build'),
to:path.join('itk', 'Pipelines'),
// to: '[name][ext]',
},
which did not report error failed to load, but then reported error Median filter error: undefined
How do I fix it

This should be fixed with #478 and #479.


chore(algorithm): mark reverse as return scope

Parameters with indirections make reverse template @system.
Signed-off-by: Luís Ferreira<EMAIL_ADDRESS>
I tested on the latest DMD on run.dlang.io and can't reproduce, so I assume the error is only on master? Otherwise, this needs a regression issue and should target stable.
CC @dkorpel , as I think you are the most experienced with return scope and you might have some context.

Weird, can't reproduce on master alone. But I can reproduce in the same file with a unittest.

I prefer not to explicitly mark generic range functions, since it forces the range primitives to be compatible with scope or it makes the code @system. On the other hand, scope inference is pretty bad, and sometimes it is done to work around that.

I tested on the latest DMD on run.dlang.io and can't reproduce, so I assume the error is only on master?

run.dlang.io is pretty unreliable with regards to using the latest compiler versions, it sometimes runs out of space and then just doesn't update anymore.

Weird, can't reproduce on master alone. But I can reproduce in the same file with a unittest.

It might be solved on master by https://github.com/dlang/dmd/pull/13993 Can you share the code?

I prefer not to explicitly mark generic range functions, since it forces the range primitives to be compatible with scope or it makes the code @system. On the other hand, scope inference is pretty bad, and sometimes it is done to work around that.

I tested on the latest DMD on run.dlang.io and can't reproduce, so I assume the error is only on master?

run.dlang.io is pretty unreliable with regards to using the latest compiler versions, it sometimes runs out of space and then just doesn't update anymore.

Weird, can't reproduce on master alone. But I can reproduce in the same file with a unittest.

It might be solved on master by dlang/dmd#13993 Can you share the code?

Yes, I did an attempt to add reverse for static arrays. See bellow:
T[S] reverse(T, size_t S)(auto ref T[S] arr)
{
    cast(void).reverse(arr[]);
    return arr;
}

@safe unittest
{
    import std.array : staticArray;
    auto sarr = [1,2,3].staticArray;
    sarr.reverse();

    assert(sarr == [3,2,1]);
    assert(reverse([1,2,3].staticArray) == [3,2,1]);

    static immutable carr = "hello\U00010143\u0100\U00010143";
    char[carr.length] arr = carr;
    assert(arr.reverse() == "\U00010143\u0100\U00010143olleh");
}

I also dislike this idea for the same reasons Dennis does. If we bump into a considerable number of cases where the inference fails but this makes it work, then this is good idea IMO. Otherwise I'd rather skip it.

I also dislike this idea for the same reasons Dennis does. If we bump into a considerable number of cases where the inference fails but this makes it work, then this is good idea IMO. Otherwise I'd rather skip it.

Yeah, I understand the rationale and makes sense to me. I can't find a reduced test case to file an issue here.


Multiple Collections?

Hello, thank you for the awesome package!
What would be the optimum way to deal with multiple collections? For example I need to calculate some response over collections A & B I currently do it in "before: {}". Is that the best way? Somehow it doesn't feel very clean. I understand the package ties itself to a single collection by default?
Best,
Kris

You are welcome, Kris. Your understanding is correct.
This package is designed to handle simple RESTful API with ONE collection. But we provide the before function hook to let developers handle request themselves.
I suggest you separate your database query/operation in a different module. So that the before functions can only handle request related operations, i.e. validation, authentication, prepare response status code, etc. And dispatch to the real query to your query module. It leads to cleaner and testable code.
You can refer my implementation: https://github.com/ops-class/test161-web-ui/blob/master/app/src/server/restful/score.js#L10-L26
It is basically doing the same thing you talked about.


[WIP] Implement handling of virtual sites for off-center charges/etc.

This will likely take a while to finish, but I'm starting work on implementing virtual sites for off-center charges (and LJ parameters if desired). So far this just adds a draft spec of virtual sites into forcefield format document.
Done so far:

[x] Update spec draft to explain final spec
[x] Add diagrams of virtual sites

To do:

[ ] Write class/generators
[ ] Update XML parser
[ ] Test on TIP4P, TIP5P water (energy validations should be possible)
[ ] Write tests
...

cc: #41

@jchodera can you by any chance provide a pointer to docs or example code showing how to PUT something at a virtual site? I have a test notebook where I'm trying to set up TIP4P to make sure I know how to do it, and I've defined the appropriate virtual site, but I cant figure out (so far) how to put a charge there... Basic, but not obvious in the docs or code I've found.

Here's a gist illustrating how to do this:
https://gist.github.com/anonymous/be8693af41fe070638a27a834cfcb7e9

Awesome, that looks like it will get me going. Thanks so much!

@jchodera - from your GIST, I'm confused about part of it -- I think I'm missing something basic:
# Add a virtual site to the System
# This appends one more particle to the System
virtual_site_index = system.addParticle(0.0 * unit.amu) # mass can be zero
virtual_site = openmm.OutOfPlaneSite(0, 1, 2, 1.0, 1.0, 0.0)
system.setVirtualSite(virtual_site_index, virtual_site)

# Add another site to the Lennard-Jones force for this virtual site
force.addParticle(0*charge, sigma, epsilon)

How does the force.addParticle here know that this refers to the virtual site? None of the arguments have anything to do with the virtual site. Is there an order-of-operations effect here where, if you add a particle to a force after creating a virtual site, the particle is assumed to be the virtual site? I haven't been able to find where this is documented, so I'm having trouble figuring out why THIS force is different from the ones you create earlier in your GIST which are "normal" particles. (Obviously this one is also massless, but I can't see how it is connected to the virtual site.)
What am I missing? Or is your code missing a line which connects the force/particle to the virtual site?

How does the force.addParticle here know that this refers to the virtual site? None of the arguments have anything to do with the virtual site. Is there an order-of-operations effect here where, if you add a particle to a force after creating a virtual site, the particle is assumed to be the virtual site? I haven't been able to find where this is documented, so I'm having trouble figuring out why THIS force is different from the ones you create earlier in your GIST which are "normal" particles. (Obviously this one is also massless, but I can't see how it is connected to the virtual site.)

The force.addParticle() call adds a new charge/LJ site to the end of the current list of sites. force.addParticle() actually returns the index of the particle that was just added, which should (in this example) match the virtual_site_index. You will want to be careful that these match up---an assert to make sure they are equal couldn't hurt.
# Add a virtual site to the System
virtual_site_index = system.addParticle(0.0 * unit.amu) # mass can be zero
nonbonded_site_index = force.addParticle(0*charge, sigma, epsilon)
assert virtual_site_index == nonbonded_site_index
virtual_site = openmm.OutOfPlaneSite(0, 1, 2, 1.0, 1.0, 0.0)
system.setVirtualSite(virtual_site_index, virtual_site)

@jchodera - now that I have figured out how to work with virtual sites, I need some help with an architectural issue. Specifically, how do we want to deal with the modified Topology and coordinates we will need for working with virtual sites?
Right now, application of a SMIRNOFF consists of basically one step after loading the forcefield: We have an initial OpenMM Topology (and, currently, molecules corresponding to a list of molecules in the system), we we provide these to forcefield.createSystem to create and return the relevant OpenMM system, which one can then simulate using the original Topology and coordinates.
When we add virtual sites, the system returned will need to contain additional particles beyond what would be represented in an INITIAL OpenMM Topology such as one read from a PDB file. This will be useless without a corresponding Topology, so how should we make application of a SMIRNOFF specifying virtual sites easy for the user/how should we handle this?  Naively it seems like we have two main options:

Modify createSystem so it also returns a Topology corresponding to the contents of the new system if has added any virtual sites/particles, or
Add a pre-processing step before createSystem for SMIRNOFFs using virtual sites, which would provide or create a new Topology and coordinates which have the virtual sites added; this would then be used by createSystem.

I think I am leaning towards option 2, since it doesn't break our existing approach for current SMIRNOFFs and simply means application of virtual-site containing SMIRNOFFs is slightly more complex.
It might work something like this, for a SMIRNOFF containing virtual sites, assuming we already have a topology and OEMols of the components:
ff = ForceField( 'vsite_forcefield.ffxml')
extended_topology, extended_positions = ff.extendSystem( topology, positions)
system = ff.createSystem( extended_topology, molecules)

What I'm imagining is that extended_topology would contain both the original (chemical) Topology as well as the extended Topology containing virtual sites (as well as information mapping atoms between the two), as the SMIRKS matching functionality in ForceField will need access to the original topology/molecules in order to do substructure searches, but it will also need access to the new topology information/mappings in order to know what particle numbers in the new system the relevant parameters should be assigned to. createSystem would raise an exception if it's applied to a regular OpenMM Topology rather than an extended topology containing this additional information, IF the SMIRNOFF in question specifies use of virtual sites.
extended_positions is a hair tricky; probably it would just be positions including the positions of the virtual sites. However, it would probably be good if we had an easy way for users to recover just the positions of their "chemical" atoms later from the full position array without too much hassle, such as if they want to convert back into OEMol or RDKit molecules from our extended_topology. Perhaps we would address this by providing utility functionality which takes an extended_topology and extended_positions and converts back into a standard OpenMM topology and positions.
Thoughts?

The second approach is how OpenMM handles this. See Modeller.addExtraParticles(), though I realize we don't actually document this in the User Guide in Model Building and Editing. Maybe raise an OpenMM issue that documentation of this feature is missing from the User Guide?
This is also likely the best approach for us, but let me think about this for a few hours. We've been aiming to make SMIRNOFF a true drop-in replacement for OpenMM's app.ForceField, but I wonder if that's really the best goal, and whether we might try to strive for simplicity instead even if it means evolving the API a bit.
For example instead of
ff = ForceField( 'vsite_forcefield.ffxml')
extended_topology, extended_positions = ff.extendSystem( topology, positions)
system = ff.createSystem( extended_topology, molecules)

we might want to just evolve the API to aim for
forcefield = ForceField( 'vsite_forcefield.ffxml')
[system, new_topology] = forcefield.createSystem(topology)

where I think our real goals are to

eliminate the need for molecules by using a Topology representation that also contains the necessary information about bonds or molecular identities
automatically add new virtual sites or Drude particles that are then represented in new_topology
allow automated charging, which may mean we want to add an optional or required positions argument to createSystem if there's any situation in which an initial set of positions (instead of automatically-generated conformers) might be wanted

This might also be a good time to create our own Topology object with the features you describe attached to it. We discussed some of the other features we would like at our Tuesday meeting. Maybe we should even open a new issue to discuss what features we might want, and what we can implement now vs punt to later.

I agree that this is definitely time to make our own Topology. I am OK with changing the API (and this might also alleviate some confusion wherein people think that, since the API is so similar, they should be able to use our FFXML files with Modeller and other aspects of OpenMM) though, so let me know what you think after you think about it for a few hours.

A few notes, since I am working on this now:

I think 3fad is equivalent to MonovalentLonePair, if outOfPlaneAngle == 0.0

Doing a centroid vsite was easy enough with openmm, so I have plugged it in (but haven't thought about the spec). It could be possible to mass weight to get a COM, see below. Currently I have named it VirtualSiteCentroidType (maybe e.g. type="centroid" in the current spec)

I think I was reading this on the openmm github repo, but it could be useful to have both an absolute charge and a chargeincrement simultaneously. The vsite charge could be the sum of the two, and we could perhaps limit the absolute charge parameter to be integer values to maintain sanity. This will add flexibility and generality to their definition. In lieu of this, a chargescale rather than a chargeincrement could be interesting in terms of parameter fitting.

I found it easy enough to define a BondCharge vsite as being centered on an atom, and pointing towards the centroid of the remaining vsite atoms. Positive distances would point away from the centroid, relative to the first atom. The spec is currently equivalent when the centroid is one atom.

To enable mass weighting, I am assuming a mass boolean tag could be set per xml vsite element, which means in the internal representation, each vsite will know whether it should mass-weight (where applicable).

tagging @dmobley, @leeping

@trevorgokey that sounds fine to me, I think. What's the argument to having the absolute charge/what use case?
Please tag me in Slack if you need my eyes on a GitHub issue; when I fall behind on notifications the GitHub ones are always the first, so I'll get to your issue faster if you point me to it in Slack.

I'm closing this PR, since #548 is taking up the mantle. Let's make sure to have further discussions over there, so the history remains linear.


Add support for using chalk-animation and ora or nanospinner

I don't know if "add support" is the correct phrase to use.
But these three libraries provide interesting functionality that might be interesting to have.
chalk-animation - enables animations in the terminal.

ora or nanospinner - Adds cool spinner controls indicating success, or failure.

Hi @leopinnheiro
regarding chalk, colors or other styling libraries: they can be used within the format option (see examples)
animation libraries are quite difficult to integrated, because cli-progress requires full control of the terminal (redraw). therefore i didn't see any possibilities to integrate them

Hi there, sorry to reopen this issue, but it's kinda the same topic. I've been trying to apply a custom style to the progress bar using gradient-string and it's been quite difficult. So far I've accomplished to style my own personal variables, but the bar itself has been quite a challenge.
You said other styling libraries can be used, any tips for how to do this or this particularly library isn't fully compatible? I've seen a couple examples here and there using chalk, but the colors are very limited to me.
Thank you.


add cat --csv-no-header option

pqrs cat --csv-no-header parquetFile will output a csv without the header line

It looks like my editor put in a few markdown syntax changes by accident. I can revert those if necessary.

ok. I removed those markdown changes

Thanks a lot! I will release a new version with this change shortly.

Awesome! Thanks for the quick turn around.

@manojkarthick Will you be able to deploy a new release soon? I would love to reference your version to get this feature rather than building my own. Thanks in advance!

@manojkarthick Will you be able to deploy a new release soon? I would love to reference your version to get this feature rather than building my own. Thanks in advance!

Sorry for the delay @jeffbski-rga - the Release workflow is broken and I've had time to fix it yet. I've published it on Cargo now, not sure that's how you plan to reference it


Develop merged

Respect the coding style!
if( ! uspp_.is_initialized ) -> if (!uspp_.is_initialized)
intent is 4 spaces, no tabs!


Update setup.sh

Installs some dependencies that might be missing

Hi Braalfa, I think this is unnecessary as mysql-server is a meta-package which depends on the newest version of mysql-server (currently 5.5), which itself depends on the newest version of mysql-server-core. Same with mysql-client. So the meta-package way is more version independent. And, installing mysql-server AND mysql-server-5.5 may lead to package collisions as soon as there is a new mysql version available.


Missing theme from Terminal

I am trying to apply the same "theme" that I use in Terminal, but it does not appear?
Is it possible to use snazzy in my case?

Where did you find the option to customize the theme?

@giovapanasiti which part are you referring to?

Within Atom I can't find where to sed the terminal colors. I did it via CSS and .bash_profile but I would love to change the appearence through atom

@mikeerickson I would too like to know about where you found the theme menu for the terminal; currently my font is huge and it's just black and white with windows powershell message. Would like to change it but not as clever as @giovapanasiti to change it myself

@SoulSkirx I will get you this information in the morning.

@SoulSkrix sorry for the name typo.  Replying on phone and it doesn't have name auto complete 😊

@SoulSkrix OK, I am  ready to assist :-)

Select Atom -> Preferences

Select Settings -> Install (I just do this to easily find out if a package is installed)

Select platformio-ide-terminal to access plugin settings

Select Settings

Scroll down to Style section

See Theme option to select desired terminal theme

@SoulSkrix
Here is what it looks like when I activate terminal ( ctrl-` ) or in my case, I use ( shift-cmd-t ) as I have ( ctrl-` ) configured to toggle iTerm

I had this problem. What I did wrong was that I had missed the enum in platformio-ide-terminal.coffee that you must add your theme to. I used pull request #42 for reference.

Please retest again with platformio-atom-ide-terminal 2.9.0 is using node-pty-prebuilt instead unmaintained and outdated pty.js


AppBar, suggestions not showing

When inside an AppBar the suggestions not showing!

i'm talking about CupertinoNavigationBar, material one its working

@itisnajim Please share screen recording of issue and code snippets.

@itisnajim Closing this issue for now as there is no significant details. You can reopen at anytime.


OS 2.1: line numering settings are lost if German menus are used

use German menus in OS 2.1dev
go to the motion settings
enable the line numbering
change to another menu, e.g. motions
switch back to the motion settings

result: the line numbering setting is lost
This only happens with German menu language, not with English, not with Spanish.
This sounds very similar to bug #2550 but this is a new behavior compared to the time when I reported #2550.

This bug is fixed in today's master


New Features

[ ] Text to speech
[ ] Slide Show
[ ] Account System
[ ] Color Blind
[ ] Store Tab

where is your wire frame?
what do you mean by text to speech?
note an accoutn system and store tab, are things that require a database (of the accounts and items for sale respectively) so we will not be able to add those on this site.  Is there anything you want to replace those with?

Perfect!


The conda virtual environment kernel that vscode remotely connects to the server cannot run. The cell prompts that ipykernel is needed

<!   -- Please search existing issues to avoid creating duplicates.    -->
Environment data

VS Code version: 1.89.1
Jupyter Extension version (available under the Extensions sidebar): v2024.4.0
Python Extension version (available under the Extensions sidebar): v2024.6.0
OS (Windows | Mac | Linux distro) and version: Linux version 6.5.0-35 22.04.1-Ubuntu
Python and/or Anaconda version: 3.7.12
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: Remote

Expected behaviour
The.ipynb file is ready to run
Actual behaviour
The.ipynb file cannot be run
Tip: Running a cell with "XXX (Python 3.7.12)" requires the ipykernel package.
Install "ipykernel" into your Python environment by running the following command.
Run the conda install -n selfocc ipykernel --update-deps --force-reinstall command.
However, I have an ipykernel package in envs
pip install -U ipykernel -i https://pypi.tuna.tsinghua.edu.cn/simple/
Looking in indexes: https://pypi.tuna.tsinghua.edu.cn/simple/
Requirement already satisfied: ipykernel in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (6.16.2)
Requirement already satisfied: debugpy>=1.0 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipykernel) (1.6.3)
Requirement already satisfied: ipython>=7.23.1 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipykernel) (7.33.0)
Requirement already satisfied: jupyter-client>=6.1.12 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipykernel) (7.4.9)
Requirement already satisfied: matplotlib-inline>=0.1 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipykernel) (0.1.7)
Requirement already satisfied: nest-asyncio in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipykernel) (1.6.0)
Requirement already satisfied: packaging in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipykernel) (23.2)
Requirement already satisfied: psutil in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipykernel) (5.9.3)
Requirement already satisfied: pyzmq>=17 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipykernel) (24.0.1)
Requirement already satisfied: tornado>=6.1 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipykernel) (6.2)
Requirement already satisfied: traitlets>=5.1.0 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipykernel) (5.9.0)
Requirement already satisfied: setuptools>=18.5 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipython>=7.23.1->ipykernel) (59.5.0)
Requirement already satisfied: jedi>=0.16 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipython>=7.23.1->ipykernel) (0.19.1)
Requirement already satisfied: decorator in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipython>=7.23.1->ipykernel) (5.1.1)
Requirement already satisfied: pickleshare in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipython>=7.23.1->ipykernel) (0.7.5)
Requirement already satisfied: prompt-toolkit!=3.0.0,!=3.0.1,<3.1.0,>=2.0.0 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipython>=7.23.1->ipykernel) (3.0.42)
Requirement already satisfied: pygments in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipython>=7.23.1->ipykernel) (2.17.2)
Requirement already satisfied: backcall in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipython>=7.23.1->ipykernel) (0.2.0)
Requirement already satisfied: pexpect>4.3 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from ipython>=7.23.1->ipykernel) (4.9.0)
Requirement already satisfied: entrypoints in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from jupyter-client>=6.1.12->ipykernel) (0.4)
Requirement already satisfied: jupyter-core>=4.9.2 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from jupyter-client>=6.1.12->ipykernel) (4.12.0)
Requirement already satisfied: python-dateutil>=2.8.2 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from jupyter-client>=6.1.12->ipykernel) (2.9.0)
Requirement already satisfied: parso<0.9.0,>=0.8.3 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from jedi>=0.16->ipython>=7.23.1->ipykernel) (0.8.4)
Requirement already satisfied: ptyprocess>=0.5 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from pexpect>4.3->ipython>=7.23.1->ipykernel) (0.7.0)
Requirement already satisfied: wcwidth in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from prompt-toolkit!=3.0.0,!=3.0.1,<3.1.0,>=2.0.0->ipython>=7.23.1->ipykernel) (0.2.10)
Requirement already satisfied: six>=1.5 in /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages (from python-dateutil>=2.8.2->jupyter-client>=6.1.12->ipykernel) (1.16.0)
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

XXX

<!   --
Note: If you think a GIF of what is happening would be helpful, consider tools like ht

use Jupyter: Select Interpreter to Start Jupyter Server
error : Running a cell with "XXX (Python 3.7.16)" requires the kernelspec package.

ImportError: /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages/zmq/backend/cython/../../../../.././libstdc++.so.6: version `GLIBCXX_3.4.29' not found (required by /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages/zmq/backend/cython/../../../../../libzmq.so.5)

thank you for filing this issue and sorry you are running into this issue.

GLIBCXX_3.4.29' not found (required by /home/a/anaconda3/envs/selfocc/lib/python3.7/site-packages/zmq/backend/cython/../../../../../libzmq.so.5)

This is a problem with the installation of the ipykernel package and related dependencies.
Unfortunately this isn't something we control, I would suggest trying to install this using conda
Using pip to install packages into conda will lead to issues, if you prefer using pip, then I would suggest using virtual envs.
In my experience, when using conda envs, you should use conda to install the packages.
Finally python 3.7 is no longer supported, I would suggest upgrading to a more recent version.
See supported versions here https://devguide.python.org/versions/


fix(Form): your description

Steps to Reproduce

Do something
Do something else.

Expected
The Form should do this
Result
The Form does not do this
Testcase
If the docs show the issue, use: http://react.semantic-ui.com/collections/form
Otherwise, fork this to get started: http://codepen.io/levithomason/pen/ZpBaJX

This appears to be the default issue template that is generated when clicking the bug icon an example in the docs.  Closing for housekeeping.


Windows Phone SL 8.1 certification failing

With the latest nuget package 0.11.0 the Windows Phone Silverlight 8.1 certification is failing saying it contains "Debug" version of the sharpcompress,dll. Is it possible to update the nuget package with the "Release" version anytime soon please?
Also it seems that the zip directory listing does not obey the character encoding in the names... how to use UTF8 so that all entry names appear correct?

I think the issue is too trivial for the developer and hence not fixed yet. Need to find some other library now.

The opposite actually.  I don't have time.  You can fix it yourself or use another library.  No need to snooty to free and open source projects.


Unsaved comments are not persisted

Open a PR overview
Start typing a comment
Oh I forgot something, let me look at that file in another editor
Come back to the editor that has the PR overview

All of my drafted comment is gone!!!

Absolutely need this.

Drafts should be persisted automatically
If not #1, then warn user that comment will be lost.

I would go further and say without this, the main benefit of this extension; being able to truly explore the codebase while reviewing, is completely undermined. At least on GH I can scroll between PR files while writing a comment, with this extension I can't even look at a single other file!

@rebornix has fixed this: https://github.com/Microsoft/vscode/pull/61757
The fix was in the core of VSCode, so this should work in versions 1.29.0 and up