JSON serialization emitDefaults option

Similar to the existing JsonParsingContext for customizing deserialization of JSON via mergeFromProto3Json, this PR adds a new JsonSerializationContext for customizing the serialization of JSON via toProto3Json.
This addresses https://github.com/google/protobuf.dart/issues/585. Tests have been added for common uses cases to ensure that the proper default values are serialized for each use case.

@mraleph @kevmoo @nichite @sigurdm or @osa1 thank you for all of your contributions to this project.
Would one of you have a moment to take a look at this PR and provide some feedback?  Thank you in advance!

@osa1 Thank you for reviewing this PR.  Your feedback was very helpful.  I believe I have addressed all of the comments.  I also added additional tests that make it very clear what the expected behavior is.  Let me know if you have any additional questions or concerns when going through.

Thank you @osa1 for your analysis of this.  I agree this would be more readable.  Was trying not to change existing functionality too much, but it definitely is more readable, and will ensure all use cases are covered as you mentioned.  I augmented the tests to help ensure uses cases are covered as well.
I have a question for you about this part:

(Note that the !skipField part above does not exist in the current version, which I think is a bug. The spec says "field is omitted" which I think means the JSON object field, and not the field value)

You are correct.  This is referring to the fact that the JSON object field should be omitted, which I agree is a bug in the current version.  What are your thoughts about how we address this?  My only concern if we fix this in the current version is it may break clients who are expecting it to behave as it does now, which is not necessarily according to the spec.
If we feel good about the overall approach you have outlined here, let me know the answer to the above and I can take a pass at implementing it in this way.
Thanks for your assistance on this!

You are correct. This is referring to the fact that the JSON object field should be omitted, which I agree is a bug in the current version. What are your thoughts about how we address this? My only concern if we fix this in the current version is it may break clients who are expecting it to behave as it does now, which is not necessarily according to the spec.

We discussed this with @sigurdm. Since this is a bug we should fix it. The question is whether we bump major version number or minor, but we don't have to decide this right now, before merging this PR. So we should fix it and merge this PR.

We discussed this with @sigurdm. Since this is a bug we should fix it. The question is whether we bump major version number or minor, but we don't have to decide this right now, before merging this PR. So we should fix it and merge this PR.

Excellent.  Glad there is a consensus on fixing the existing bug.  I'll work on implementing the suggested changes and it ready for review (hopefully today).

Excellent. Glad there is a consensus on fixing the existing bug. I'll work on implementing the suggested changes and get it ready for review (hopefully today).

@osa1 This is now ready for your review

Could you take a look at those failures? I think for some of them we just need to update the expected outputs. I didn't check all though.

The unit tests within proto3_json_test.dart have been fixed.

I wonder if we should incorporate this code in this PR and simplify the large if-then-else chain in this PR. We could add it in another PR and update the code in binary encoders and decoders, and then use it in this PR. Any thoughts on this @sigurdm?

@osa1 I like this idea.  I was also not liking the if/else, but wasn't sure how to incorporate a switch with the existing code base.  That would be great to use the code you mentioned above in this PR.

@osa1 I believe I have addressed all of your comments.  The major outstanding issue is the if/else vs. switch.  I personally like the switch much better, but we'd just need your other code merged in first I believe.  Lemme know.
Thanks for all of the great feedback and insightful review.

Re: updated tests, I found it a bit strange that we test proto2 files with proto3 serialization. Is this expected @sigurdm? JSON serialization implemented in proto3_json.dart is for https://developers.google.com/protocol-buffers/docs/proto3#json as far as I understand, and proto2 doesn't specify a JSON encoding. That doesn't mean we can't implement JSON encoding for proto2, but we will need to figure out the interaction with optional and hasX methods (presence) ourselves instead of following a spec.

I found it a bit strange that we test proto2 files with proto3 serialization. Is this expected @sigurdm?

I've always assumed that proto2-fields serialize more or less like their proto3 counterparts. Especially since you can have a proto2 message type as a field of a proto3 message. It would be problematic to deny serializing it. (That is, I look at the proto3-json serialization format as independent of the proto3-protobuf language definition)...
I don't know of any cases where this causes trouble. But I might be overlooking something

The major outstanding issue is the if/else vs. switch. I personally like the switch much better, but we'd just need your other code merged in first I believe

I submitted a PR for this: #605. Currently protoc_plugin fails when building test protos so it needs a bit more debugging.

@osa1 Looks like there has been great progress on #605.  Let me know if there is anything I can do to help.

#659 shows an actual problem (found in downstream code, caused breakage) with value is List etc. checks. We should remove them and use field info types.

@osa1 I want to get your thoughts on this PR.  Is it valuable, should I bring it up to date with master and get this across the finish line?  For my purposes, it may not be needed anymore, but it may be valuable to others in the future?

@jmartin127 this fixes a bug in the library so we definitely want to merge it, whether it's a blocker for someone today or not.
You did most of the work already (thanks!), we just need to simplify it a little bit and remove the is checks and use the type information from field infos.  I can finish this in a separate PR (created from your branch) and merge it, as I've been doing recently with some of the other, older PRs.


On roll refactor

#56
This combines all rolling logic into a single "rollDice" final step that is a renamed/modified copy of "vampireRoll"
there is now a logic to check whether or not hunger dice should be used in the roll. If useHunger is set to "false" is sets hunger dice to 0 for the roll.

omg thank you lol I'll have to review it more closely but I've always been unhappy with my mess of roll logic

conflicts with the dev branch :(


Better support for LLVM IR

In the new UI, the "hack" for -emit-llvm to show IR shows up shortcomings...
https://godbolt.org/g/1cAQxA
Might be nice to either detect llvm output and colourise it better...or better yet...support llvm IR properly.

Taking this out of the milestone: I think we can live without it. It's an enhancement for later.

FYI the above link is broken -- is that a bug?

whoops, I broke the cardinal rule of linking to the beta site and expecting it to be valid long term!
https://godbolt.org/g/MwHd7E is a non beta lonk

Somewhat duplicated by #241

Comment from 241: "The Monaco syntax highlighting library is pretty straightforward (https://microsoft.github.io/monaco-editor/monarch.html)", combining both issues here

We should consider making -emit-llvm a first-class operation (a button maybe?)

would be nice to be able to write IR as well as compile into it

As a workaround IR can be written with -x llvm-ir (or similar, I forget the exact syntax...but yes!) Adding IR as a language is something we will have as part #277 (hopefully!)

And CFG view for LLVM IR?

@kula85 An LLVM IR view is added by #1022 and #992. For syntax highlighting see #1114

Can this be closed? :-)
The IR pane is live on godbolt.org


Remove need to compile signal.wasm

Uses static hosted signal.wasm file to transform input into signal topic. Keys to filter for are passed via environment variables using the --var argument of the rpk transform deploy command.
Currently still testing.

Unfortunately, the maximum size of environment variables in Redpanda transforms is currently limited to 2048 characters/bytes, which would only allow us to configure a baseline of at most 55 keys. As this is not enough for our use case, I will close this pull request for now.

Unfortunately, the maximum size of environment variables in Redpanda transforms is currently limited to 2048 characters/bytes, which would only allow us to configure a baseline of at most 55 keys. As this is not enough for our use case, I will close this pull request for now.


Differentiate between empty structures and components that can take on any value

Changes
Currently, the SDK doesn't distinguish between empty structures and components that can take any value. As a result, the SDK uses type any for fields that should really have a type of struct{} or *struct{} (or an equivalent type). This PR updates the OpenAPI generator code to distinguish between these two cases.
Additionally, we update the generation of SDK structures to include empty types. This includes some types that are not currently used in the SDK, mainly empty responses because the generated API calls for methods with an empty response just omits the response return value entirely. Later, we may want to include these by default; even though they are empty, it means that adding anything to an empty response is a breaking change for SDK users.
Tests

[ ] make test passing
[ ] make fmt applied
[ ] relevant integration tests applied

Codecov Report
Attention: 21 lines in your changes are missing coverage. Please review.

Comparison is base (99aeb90) 18.69% compared to head (c420dd0) 18.67%.

Files
Patch %
Lines

service/sql/model.go
0.00%
15 Missing :warning:

openapi/code/package.go
40.00%
2 Missing and 1 partial :warning:

openapi/model.go
0.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #821      +/-   ##
==========================================
- Coverage   18.69%   18.67%   -0.02%
==========================================
  Files         117      117
  Lines       21653    21671      +18
==========================================
+ Hits         4047     4048       +1
- Misses      17355    17372      +17
  Partials      251      251

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


LNbits thinks that wrapped invoice is internal invoice

Description of the situation:

There is a campaign to fund Vexl on Geyser: https://geyser.fund/project/vexl
Geyser takes invoice from an LNbits server
Geyser wraps the invoice into their invoice and adds their fee (4%)
When one tries to pay the invoice from the same LNbits server, the server thinks the invoice is their own (internal) - this is a BUG
There is a check in LNbits that checks whether amounts for internal invoices matches (for both sender and receiver)
This check fails, because geyser added fee in step 3, resulting in "Invoice invalid" error which is confusing to say it politely

https://github.com/lnbits/lnbits/blob/3a64cf5245f251c168d5563352d0c8fc25702995/lnbits/core/services.py#L261-L265
When I remove the check, the payment goes through.
We should:
a) fix the wording to be less confusing
b) fix the issue and treat such invoices as not internal since they are not

This problem still persists, i cannot pay a Geyser Invoice from LN Bits

When one tries to pay the invoice from the same LNbits server, the server thinks the invoice is their own (internal) - this is a BUG

This is not a bug, from the description I understand that that is actually an internal invoice

We should:
a) fix the wording to be less confusing
b) fix the issue and treat such invoices as not internal since they are not

a) I agree
b) even if the invoice is wrapped it is generated by the same LNbits instance (if I understand correctly)
The scenario that the above check guards against is:

user generates invoice for 100 sats on Wallet A
user modifies the invoice and changes the amount to 90 sats
user pays modified invoice (90 sats) from Wallet B
the original invoice from Wallet A is marked as paid
now Wallet A has additional 100 sats but Walet B only -90 sats

Easier alternatives (than wrapping) are possible.
For example the Split Payments extension can be used: https://github.com/lnbits/splitpayments
The extension can be configured to move X% of each incoming payment to Wallet P into Wallet Q.

Suggestion:
Use the Split Payments extension (on Geyser side): https://github.com/lnbits/splitpayments
The extension can be configured to move X% of each incoming payment to Wallet P into Wallet Q.

This is not a bug, from the description I understand that that is actually an internal invoice

I think you misunderstood the issue.

Easier alternatives (than wrapping) are possible. Use the Split Payments extension (on Geyser side):

I don't think we are in a position to dictate how people should build their infrastructure 😄

The scenario that the above check guards against is:

I think there should be a better way how to protect against this scenario


[Question] Is it possible to freely scale? example: 2.5x, 3.0x, 4.1x etc.

Hi nagadomi
I've a question about scaling. But before I begin: I finally resolved the CUDA issues on my machine and I can finally use your version and it's so much faster :) | this was my issue i opened long time ago #77
I used this alternative before, you suggested long time ago -> tanakamura's waifu2x c++/opencv
In this version you could specify any scaling factor you wanted, as long as it was above 1.0, like 1.1, 1.3, 2.0, 2.42, 4.32 and so on...
The scaling works normally, without any quality regressions or anything.
In your version I noticed that this is not possible and I just get a file not found exception, because it wants to open a model named after the scaling factor :(
The question I have now: Does your version explicitly needs a model for every scaling factor or is there a workaround.
If I need extra models, how exactly I can train one? What pictures are the best to use for model training? I really miss some factors I use on a regular basis, which are: 2.5, 3.0, 4.0, 1.5, 1.7
Regards

In fact, waifu2x supports only 2x scaling.
optional scaling factor in the other software is implemented by the following steps.

Repeatedly applying 2x model until sufficient size is reached
Downscaling the image to fit a specified size with a generic image resizer (e.g bilinear, bicubic)

Currently, waifu2x.lua is not supported this feature.

I see. In this case the best solution for me would be to write a wrapper script around waifu2x with the help of imagemagick or something to implement optimal scaling factors. I will look into this in my free time, i really miss the possibility to freely scale.
Anyways, thanks for the info :)


Typo in documentation

https://github.com/mmattel/EET-Solmate/blame/9b06b314cc40cdc5add6d6fc7d2f90b383f29789/docs/prep-ha.md#L71
Following the instructions, I ran into this error wondering what I did wrong:
python: can't open file '/config/shell/solmate/check_reqirements.py': [Errno 2] No such file or directory
looks like there is a typo in the documentation
python check_reqirements.py -> should be python check_requirements.py
PS: thanks for the hard work you've put into this!

Nice catch 🤦♂ thanks for highlighting 😄 Will fix asap.
For curiosity, mind to tell how you installed it, on an own server or inside HAOS?

You are wellcome ;)
While your're at it, there one more thing I noticed:
After adding the shell command in the configuration.yaml file, HA needs to be restarted for the command to be available. Reloading the config doesn't seem to be enough.
I installed it inside HAOS (Homeassistant yellow)
It worked yesterday, but during the night (after about 2h) it stopped sending data to HA. I guess I will open a new issue for this.

Thanks, good to know.
I am working on an update and I have an idea for how to solve it. The fencing of HAOS is a nightmare...

Might have been my fault. I think I hat multiple instances running or something, the values were updated every second and the values were flickering.
Maybe related to issue #40 ?

Multiple instances is the thing I am looking for. I am preparing a code change that should prevent that. And this is only for HAOS. Running the script on a normal server is so much easier...

This getting off-topic, but have you thought about creating a Docker Image?
If you want, I could look into this.
PS: since cleaning up the /etc/crontabs/root file and switching to local connection yesterday, it's working fine!
Also I believe my account got locked due to too many API queries (can't log in with a browser any more).

have you thought about creating a Docker Image?

Technically, a docker container would ease things, but how you want to integrate an external container in HAOS?

I haven't yet tried running containers directly. But from what I have seen, the Add-ons from the HA/HACS store are containers, so there should be a way.
The documentation looks quite straight forward:
https://developers.home-assistant.io/docs/add-ons/tutorial
I will look into this matter after work, this is something I wanted to try for a while already.

@axcro7 mind to test PR #41 which improves the HAOS integration. See the comment for changes. There is btw no breaking change, just fixes and improvements.
I dont know how much you are fine with git, but in a nutshell, you drop any changes that are not automatically excluded like the config, then you checkout master and do a rebase and switch to the branch of 41. Finally you kill all solmate scripts and trigger the automation to start the crond-prepare script. After a while, check that it is running with ps and that there is only one instance of it.
Let me know your findings.

Update ist released: https://github.com/mmattel/EET-Solmate/releases/tag/v4.1.0 😃

@mmattel  sorry for not replying earlier :(
I did look into the container thing, but then failed at including python modules, due to my lac of experience with python..
Then life struck and I didn't get to it any more :/
Anyway, the whole thing is running stable since updating to v4.1.0
I will try to migrate to v6 soon.

Appreciate your feedback. Supporting my work is welcomed 😃
Release v6 will be released soon. I want to add one requested functionality with respect to logging and will tag afterwards to make it available.
As you are coming from v4, check the steps v4 -> v5 -> v6 for a smooth upgrade 😃

Thanks for the reminder, I will of course look into your documentation / release notes
Not sure if I can support with code, but I will buy you a coffee ☕👍


Game time in title bar

It is nice how the report button can show in-game-time or logged-in-time.  I find both time useful and would like to have both times displayed.  So it would be nice to be able to have a time displayed in the title bar, and then I could have the other time displayed on the report button.
For me I would have logged-in-time on the title bar and the in-game-time on the report button.

Displaying timestamp in titlebar is definitely not happening as that is really bad idea (updating title bar every second sounds very dumb).


RDF HDT

https://www.rdfhdt.org
@lbovet

Looks like something useful when datasets are huge (millions of triples). Is it reasonable to think about so high numbers at this stage?

Looks like something useful when datasets are huge (millions of triples). Is it reasonable to think about so high numbers at this stage?

No no, of course, this is not at all a priority, yet? 😂 Feel free to drag it "down"  on The Roadmap Plan!

https://rdf4j.org/javadoc/latest/org/eclipse/rdf4j/rio/hdt/HDTParser.html


Start to use engine changes in rpg demo for rendering and resolution

Codecov Report

Merging #123 (5a68734) into main (68266f3) will increase coverage by 10.95%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #123       +/-   ##
===========================================
+ Coverage   49.50%   60.45%   +10.94%
===========================================
  Files         105       14       -91
  Lines        3101      311     -2790
  Branches      320       28      -292
===========================================
- Hits         1535      188     -1347
+ Misses       1563      123     -1440
+ Partials        3        0        -3

Impacted Files
Coverage Δ

seagulls-engine/src/seagulls/rendering/__init__.py

...ne/src/seagulls/rendering/_renderable_component.py

seagulls-engine/src/seagulls/rendering/_sprite.py

...lls-rpg-demo/src/seagulls/rpg_demo/_cli_command.py

...ls-rpg-demo/src/seagulls/rpg_demo/_di_container.py

...-demo/src/seagulls/space_shooter/_score_overlay.py

...oter-demo/src/seagulls/space_shooter/_blue_ship.py

...ulls-engine/src/seagulls/app/_plugin_exceptions.py

...r-demo/src/seagulls/space_shooter/_ship_catalog.py

...o/src/seagulls/space_shooter/_game_over_overlay.py

... and 81 more

Continue to review full report at Codecov.

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


feat(web): reset container field buttons

Codecov Report
Base: 17.51% // Head: 17.51% // No change to project coverage :thumbsup:

Coverage data is based on head (0236171) compared to base (a72872b).
 Consider uploading reports for the commit 9cfe483 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #467   +/-   ##
========================================
  Coverage    17.51%   17.51%
========================================
  Files           57       57
  Lines         6105     6105
========================================
  Hits          1069     1069
  Misses        4954     4954
  Partials        82       82

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Testing revealed some problems with the config editors:

When disabling the Kubernetes filter, the group still shows but it's empty (I'm guessing it's due to the "extraLBannotations" thing?).
Args localization is wrong in the filter list.

Thanks for the testing, @robot9706 Awesome! I am going to fix them.


Cleanup Syncer to prepare for Syncer Virtual Workspace transformations

Summary
This PR fixes some corner cases or behaviors in the Syncer that would not be compatible with the Syncer Virtual Workspace transformations.

/approve

what are these corner cases? And why are they not compatible?

/retest-required

/retest

/retest


nix: Add nrf-command-line-tools derivation

nrf-command-line-tools (nrfjprog) are not available in nixpkgs so this patch adds a simpe derivation to the repository for easy installation.

No significant changes.

Insignifcant changes

metric
value

change

binary-size-nk3am
1208362
:white_circle:
+0 (+0.00%)

binary-size-nk3am-test
2000242
:white_circle:
+0 (+0.00%)

binary-size-nk3xn
434740
:white_circle:
+0 (+0.00%)

binary-size-nk3xn-test
588204
:white_circle:
+0 (+0.00%)

binary-size-nkpk
760594
:white_circle:
+0 (+0.00%)


Adding Michigan State Parks MIDNRReservations.com

Description
https://github.com/juftin/camply/issues/94

Are there GoingToCamp pages with more Captcha services than others? I tried testing this out with the following command and I get the following errors:
camply campgrounds --provider goingtocamp --rec-area 12

HTTPError: 403 Client Error: Forbidden for url: https://midnrreservations.com/api/resourcelocation/rootmaps

ConnectionError: Receiving bad data from GoingToCamp API: status_code: 403: <title>midnrreservations.com</title>
Please enable JS and disable any ad blocker

Hey @SoccerCore11 - I closed this issue inadvertently by merging the upstream branch. Would you re-open the PR against main?


Prompt for confirmation wherever appropriate

Subcommands like delete should default to prompting for confirmation from user.
We can add support for a global flag --force to skip all confirmations

This seems to be already resolved by #92 🤔


add meta info render, when config render:false, return content with m…

add meta info render, when config render:false, return content without markdown render.
this patch helps when we need to put some html page into wiki site.

Thanks @JamesPan . Could you please give this new feature pr to dev branch?
And this way may be more easier and clearer?
if meta.get('render', True):
    content = self._parse_markdown(markup_text)
else:
    content = markup_text

I just open a new pr in #48 , maybe this pr can be closed now.


Unsubscribe Accessibility from model in destructor. Fixes #109.

Accessibility subscribed to the model, but didn't unsubscribe in the destructor resulting in being called while not longer valid. Fixed #109

This is awesome. Thanks for both the find + fix!


[DOC]

Report incorrect documentation
Location of incorrect documentation
(https://docs.monai.io/projects/monai-deploy-app-sdk/en/stable/getting_started/tutorials/03_segmentation_app.html)
Describe the problems or issues found in the documentation
Missing several install steps.
Steps taken to verify documentation is incorrect
I had to add the following pip installations: scikit-image, numpy-stl and trimesh
Suggested fix for documentation
Add missing installation steps
Report needed documentation
Report needed documentation
Describe the documentation you'd like
Add the extra pip steps in the "Execute from Shell" section
Steps taken to search for needed documentation

Added a comment in the latest doc on these dependencies when an app uses the STL Conversion operator.


Confirming user email update sends user to an Error

Bug report
Describe the bug
When confirming an updated email via the link sent by Supabase, the user gets taken to a page which just contains a json object with the following message:
"message": "no Route matched with those values"

To Reproduce

Create an app that supports user registration via email & password with the js-lib.
Create a method to update the email, using the following code:

    const { user, error } = await supabase.auth.update({
      email: updatedEmail,
    });

Click link sent to user's old email
See error

Expected behavior
User email to be updated, then redirected to site.
System information

Version of supabase-js: 1.22.6

Additional context
Unsure how long links last before expiring, but for the time being this link allows you to view the error: https://udqdmhuoglnkrnpwdend.supabase.co/?token=-2VosgsXGFvwDzcL6BypZw&type=email_change&redirect_to=http://localhost:3000/

hi @joshcawthorne, it seems like this issue is due to an incorrectly formatted email link - but feel free to reach out to us at support if it's still an issue for you.


SVG export - Python 3 support

Hi,
I've found a possible error in rdkit python bindings (for python 3), when I want to draw mol to SVG - import from current folder is wrongly used in rdkit.sping SVG and PDF package. I also found loads of outdated unsupported stuff like cStringIO import, .has_key() dict method or using string module functions (lower, split etc.) instead of string object methods.
This should reproduce the error:
from rdkit import Chem
from rdkit.Chem import Draw

mol = Chem.MolFromSmiles("CCC")
Draw.MolToFile(mol, "/tmp/mol.svg")

Any news on this? I checked building 2015_03_01 and current HEAD against Python 3.4 and always get the bug, i.e. cannot export to ps or svg:
Python 3.4.2 (default, Oct  8 2014, 10:45:20)
[GCC 4.9.1] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from rdkit import Chem
>>> from rdkit.Chem import Draw
>>> mol = Chem.MolFromSmiles("CCCCO")
>>> Draw.MolToFile(mol, "test.ps")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/jranke/git/rdkit/rdkit/Chem/Draw/__init__.py", line 167, in MolToFile
    canvas = Canvas(size=size,name=fileName,imageType=imageType)
  File "/home/jranke/git/rdkit/rdkit/Chem/Draw/spingCanvas.py", line 31, in __init__
    from rdkit.sping.PS.pidPS import PSCanvas as _Canvas  #@UnresolvedImport
  File "/home/jranke/git/rdkit/rdkit/sping/PS/__init__.py", line 2, in <module>
    from pidPS import *
ImportError: No module named 'pidPS'
>>> Draw.MolToFile(mol, "test.svg")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/jranke/git/rdkit/rdkit/Chem/Draw/__init__.py", line 167, in MolToFile
    canvas = Canvas(size=size,name=fileName,imageType=imageType)
  File "/home/jranke/git/rdkit/rdkit/Chem/Draw/spingCanvas.py", line 33, in __init__
    from rdkit.sping.SVG.pidSVG import SVGCanvas as _Canvas
  File "/home/jranke/git/rdkit/rdkit/sping/SVG/__init__.py", line 3, in <module>
    from pidSVG import *
ImportError: No module named 'pidSVG'

Not a proper fix but maybe you can avoid the issue by activating the cairo
backend (e.g. no sping involved). You can do it by running cmake with the
-D RDK_BUILD_CAIRO_SUPPORT:BOOL=ON option.
On Fri, Oct 2, 2015 at 9:12 AM, Johannes Ranke<EMAIL_ADDRESS>wrote:

Any news on this? I checked building 2015_03_01 and current HEAD against
Python 3.4 and always get the bug, i.e. cannot export to ps or svg:
Python 3.4.2 (default, Oct  8 2014, 10:45:20)
[GCC 4.9.1] on linux
Type "help", "copyright", "credits" or "license" for more information.

from rdkit import Chem
from rdkit.Chem import Draw
mol = Chem.MolFromSmiles("CCCCO")
Draw.MolToFile(mol, "test.ps")
Traceback (most recent call last):
File "", line 1, in
File "/home/jranke/git/rdkit/rdkit/Chem/Draw/init.py", line 167, in MolToFile
canvas = Canvas(size=size,name=fileName,imageType=imageType)
File "/home/jranke/git/rdkit/rdkit/Chem/Draw/spingCanvas.py", line 31, in init
from rdkit.sping.PS.pidPS import PSCanvas as _Canvas  #@UnresolvedImport
File "/home/jranke/git/rdkit/rdkit/sping/PS/init.py", line 2, in
from pidPS import *
ImportError: No module named 'pidPS'
Draw.MolToFile(mol, "test.svg")
Traceback (most recent call last):
File "", line 1, in
File "/home/jranke/git/rdkit/rdkit/Chem/Draw/init.py", line 167, in MolToFile
canvas = Canvas(size=size,name=fileName,imageType=imageType)
File "/home/jranke/git/rdkit/rdkit/Chem/Draw/spingCanvas.py", line 33, in init
from rdkit.sping.SVG.pidSVG import SVGCanvas as _Canvas
File "/home/jranke/git/rdkit/rdkit/sping/SVG/init.py", line 3, in
from pidSVG import *
ImportError: No module named 'pidSVG'

—
Reply to this email directly or view it on GitHub
https://github.com/rdkit/rdkit/issues/398#issuecomment-144940730.

--
Gianluca Sforna
http://morefedora.blogspot.com
http://plus.google.com/+gianlucasforna - http://twitter.com/giallu

Thanks for the hint - but I get the exact same result when activating the cairo backend like this...

You can do either SVG or PNG (PNG only with the cairo backend) using the new MolDraw2D functionality.
There's some information here: http://rdkit.blogspot.com/2015/02/new-drawing-code.html

Thanks for the hint - however I am using postscript, because I can import this in R using grImport. Not a big deal at the moment, as using python 2.7 is fine for now.

@bp-kelley
I still get errors with current HEAD and python3.4:
Python 3.4.2 (default, Oct  8 2014, 10:45:20)
[GCC 4.9.1] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from rdkit import Chem
>>> from rdkit.Chem import Draw
>>> mol = Chem.MolFromSmiles("CCCCO")

For SVG:
>>> Draw.MolToFile(mol, "/tmp/mol.svg")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/jranke/git/rdkit/rdkit/Chem/Draw/__init__.py", line 170, in MolToFile
    drawer = MolDrawing(canvas=canvas, drawingOptions=options)
  File "/home/jranke/git/rdkit/rdkit/Chem/Draw/MolDrawing.py", line 105, in __init__
    fill=True, stroke=False)
  File "/home/jranke/git/rdkit/rdkit/Chem/Draw/spingCanvas.py", line 89, in addCanvasPolygon
    closed=1)
  File "/home/jranke/git/rdkit/rdkit/sping/SVG/pidSVG.py", line 317, in drawPolygon
    pointStr = _PointListToSVG(pointlist, dupFirst=closed)
  File "/home/jranke/git/rdkit/rdkit/sping/SVG/pidSVG.py", line 77, in _PointListToSVG
    for i in xrange(len(points)):
NameError: name 'xrange' is not defined

and for PostScript:
>>> Draw.MolToFile(mol, "/tmp/test.ps")
Traceback (most recent call last):
  File "/home/jranke/git/rdkit/rdkit/sping/PS/psmetrics.py", line 225, in stringwidth
    widths = _Widths[encoding][string.lower(font)]
AttributeError: 'module' object has no attribute 'lower'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/jranke/git/rdkit/rdkit/Chem/Draw/__init__.py", line 180, in MolToFile
    drawer.AddMol(mol, **kwargs)
  File "/home/jranke/git/rdkit/rdkit/Chem/Draw/MolDrawing.py", line 553, in AddMol
    labelSize = self._drawLabel(symbol, pos, baseOffset, font, color=color, orientation=orient)
  File "/home/jranke/git/rdkit/rdkit/Chem/Draw/MolDrawing.py", line 354, in _drawLabel
    labelSize = self.canvas.addCanvasText(label, (x1, y1, baseOffset), font, color, **kwargs)
  File "/home/jranke/git/rdkit/rdkit/Chem/Draw/spingCanvas.py", line 64, in addCanvasText
    txtWidth, txtHeight = self.canvas.stringBox(text, font)
  File "/home/jranke/git/rdkit/rdkit/sping/pid.py", line 288, in stringBox
    return self.stringWidth(s, font), self.fontHeight(font)
  File "/home/jranke/git/rdkit/rdkit/sping/PS/pidPS.py", line 494, in stringWidth
    self.fontMapEncoding["EncodingName"]) * font.size * 0.001
  File "/home/jranke/git/rdkit/rdkit/sping/PS/psmetrics.py", line 227, in stringwidth
    raise KeyError("Improper encoding {0} or font name {1}".format(encoding, font))
KeyError: 'Improper encoding Latin1Encoding or font name Helvetica'


A lane and its successor lane are sometimes not connected

When working with the file 4way_intersection.xodr I noticed that in some cases there are gaps between a lane and the next succeeding lane. This can be seen here:

According to the roadgraph, lines of the same color are in a predecessor/successor relationship. For the (global) LaneIds 35->3, 37->10, 46->29 and 47->50 there is a gap between the corresponding lines.
A test case for this error is implemented in #52
@klemense1

commented in https://github.com/bark-simulator/bark/issues/73
closing


Updating version to 1.0.0-alpha

🎯 Summary

A few bug fixes.
🔰 Checklist

[x] I have read and agree with the following checklist and am following the guidelines in our Code of Conduct document.

I have performed a self-review of my code.
I have commented my code, particularly in hard-to-understand areas.
I have made corresponding changes to the documentation where required.
I have tested my changes to the best of my ability.
I have consulted with the team if introducing a new dependency.
My changes generate no new warnings.

updates to readme and version number look good. do we want the updates for Wrapper.tsx in this PR?

If you think I should separate it I can

updates to readme and version number look good. do we want the updates for Wrapper.tsx in this PR?

If you think I should separate it I can

It doesnt really matter, just wanted to make sure it wasn't added in error :)


Scaffold settings ignore Layout if left blank

The scaffold settings indicate to leave the layout section blank if I want to use layout that's found in _viewstart, however, once the scaffold is executed the generated pages are created with a null layout:
@page
@model RazorPagesMovie.Pages.Movies.IndexModel

@{
    Layout = null;
}

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

ID: 6719f08e-3bd7-dc1a-71df-f2ef9fbca9d8
Version Independent ID: 7096fdb3-612e-9e00-bd0b-8ea4886a09ce
Content: Add a model to a Razor Pages app in ASP.NET Core
Content Source: aspnetcore/tutorials/razor-pages/model.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Follow the instructions and you get the right layout.

Try to scaffold via command line add -udl attribute


Extend available options

This request extends existing parameters to allow

separators of the decimal part of the number
brackets as negative markers
negativeOut to be controlled separately for left and right markers
I have kept these changes to work alongside the old options so this part is backward compatible

The other change that I have made is to expect the input decimal char to be ".".
This is on the basis that the input should be unformatted "computer" format.
I've also written lots of tests to both check some of the existing functionality and the new.
FYI My plan is to create another 'project' to use deconstruct-number-format  to pass parameters to this

Merged as 2bd079372a1880af7565e232b72a8582b25660f2 along with a change to how you override the units and separators.  N.B. this will be a breaking change so we will have to check for any places we currently call format-number to ensure that they work.

thanks.
I'll carry out the review


Removing the master-slave vocabulary

For diversity reasons, it would be nice to try to avoid 'master' and 'slave' terminology in this repository which can be associated to slavery. The master-slave terminology could be problematic for people in several countries which has the history of slavery like Romania, USA and many others. Thank you for considering the proposal. Let me know if any changes in the PR are needed, I would be happy to implement them.

@bopopescu  First of all, thanks for your advice
However, since we are using third-party services and need to follow third-party definitions, this cannot be changed for the time being. The details are as follows
https://www.alibabacloud.com/help/doc-detail/100625.htm?spm=a2c63.p38356.b99.65.500d4fe6SUfYXS


Attaching Debugger to Remote Docker Container Running Jupyter Server Hangs

From @cratter49 on November 3, 2017 19:54
Environment data
VS Code version:  1.18.0-insider
Python Extension version:  ptvsd (3.2.1)
Python Version: 3.4.3
OS and version: Mac OSX 10.11.6
The remote server is running a custom version of Jupyter that uses iPython on a Docker container on an image that is based on Ubuntu 14.04.
Actual behavior
Start remote program (startProgram-Docker.sh), start the debugger, and then the debugger infinitely hangs.
However, when I shutdown the remote server the following message pops up in an error dialog within VS Code: "Debug adapter process has terminated unexpectedly"...this must mean that the debugger becomes attached correct?
Expected  behavior
Start remote program (startProgram-Docker.sh), start the debugger, and then the debugger should hit a breakpoint I set in a Python script that exists in the remoteRoot and I know executes upon server startup.
Steps to reproduce:

Add the following launch configuration to launch.json:

        {
            "name": "Attach to DSWB Instace (Remote Debug)",
            "type": "python",
            "request": "attach",
            "localRoot": "${workspaceRoot}",
            "remoteRoot": "/home/ubuntu/notebook_server", // directories exist here which contain more directories and files in multiple languages, including Python, used to run the server
            "port": 8443, // VBox forwards this port so it can be accessed locally
            "secret": "my_secret",
            "host":"hostname.com" // hostname.com equates to the Docker machine's IP
        }

Execute the startProgram-Docker.sh mentioned above to start up the server remotely.

Launch the debugger

Nothing happens

Logs
Output from Python output panel
Nothing

Output from Console window (Help->Developer Tools menu)
Debug adapter process has terminated unexpectedly
e.doShow @ messageService.ts:126
e.show @ messageService.ts:105
_.onServerExit @ rawDebugSession.ts:535
(anonymous) @ rawDebugSession.ts:449
emitTwo @ events.js:111
emit @ events.js:194
__dirname.ChildProcess._handle.onexit @ internal/child_process.js:215

Copied from original issue: DonJayamanne/pythonVSCode#1369

From @cratter49 on November 9, 2017 1:27
@DonJayamanne?

Let me know how that goes. Also, debugging Jupiter kernels are not supported.

Please use ptvsd 3.0.0

@DonJayamanne
I was able to attach a debugger to a single Python script and have it work there after downgrading ptvsd remotely and locally but have not succeeded in attaching the debugger to the running server without errors. The notebook server is started via the jupyter notebook command line statement which starts the jupyter-notebook Python script containing the following code:
#!/usr/bin/python3
# EASY-INSTALL-ENTRY-SCRIPT: 'notebook','console_scripts','jupyter-notebook'
__requires__ = 'notebook'
import re
import sys
from pkg_resources import load_entry_point

if __name__ == '__main__':
    sys.argv[0] = re.sub(r'(-script\.pyw?|\.exe)?$', '', sys.argv[0])
    sys.exit(
        load_entry_point('notebook', 'console_scripts', 'jupyter-notebook')()
    )

I added the following line ptvsd.enable_attach("my_secret", address = ('<IP_ADDRESS>', 8443)) before the conditional in the script above and it seems the debugger attached itself to the server but encountered many errors according to the debug console. Here are some of the errors that appeared in the log:
KeyError, 'InteractiveShell'
KeyError, 'InteractiveShell'
KeyError, 'PlainTextFormatter'
KeyError, 'TerminalInteractiveShell'
ImportError, No module named 'urlparse'
ImportError, cannot import name 'unquote_plus'
ImportError, No module named 'urlparse'
ImportError, No module named 'urlparse'
ImportError, cannot import name 'unquote_plus'
KeyError, 'config'
KeyError, 'parent'
KeyError, 'argv'

The debugger would pause on an uncaught exception, in the Call Stack window all of the methods were visible but the sources were unknown, and each time I would resume or step forward a new exception would occur and a new error message like the ones above. All of the Local and Argument variables are visible in the Variables window.
I forgot to mention that the notebook server uses the Tornado web framework as well .
Also, it seems that some of the details from the setup guide in the pythonVSCode repo Wiki under the Remote Debugging section have been lost in transition to the official VSCode site. Because of this I have forgotten what the value of port should be in the launch.json. Currently it's pointing to the port that my notebook server is running on but is it supposed to be the port that the debugger runs on?
And why has this issue moved from to this forked repo's issues?

And why has this issue moved from to this forked repo's issues?

Simple, the extension is no longer maintained by me (not alone). It is now maintained by a team at Microsoft, a team that I am now a member of. We will continue supporting and extending this extension. But from the Microsoft repo.

Please could you confirm that attaching to a simple hello world program works, when it is running in a server. I.e. let's try to isolate the problem by removing notebook/jupyter completely from the picture.

@DonJayamanne Sorry, I wasn't clear enough in my last post. I was able to attach a debugger to a single Python script and have it work there meant I was able to attach the debugger to the simple hello world Python program within the Docker container.

At the moment debugging in notebook or jupyter isn't supported.
Hence closing The issue with a status of won't fix.


Feature/no maap cfg

This PR merged in Sujen's changes to remove the need to track maap.cfg file
We will need to deploy the changes to the devfiles when we merge this PR so that the MAAP_API_HOST variable is present. MAAP_API_HOST points to the ops api by default so it will need to be added to the devfiles for dit and uat

I just realized that I did not read the env value correctly. The API host is supposed to point to api..maap-project.org and not ade..maap-project.org. Sorry missed this.

@sujen1412 Created a PR to address this here: https://github.com/MAAP-Project/maap-workspaces/pull/126


Cache-Hit set to string, not boolean, as of v4.1.0

After a recent upgrade, our runners started behaving differently on cache miss.
As of 0c45773b623bea8c8e75f6c82b208c3cf94ea4f9, a step defined with if: steps.cache-release.outputs.cache-hit would be skipped if the "cache release" job missed the cache, as intended. As of 2cdf405574d6ef1f33a1d12acccd3ae82f47b3f2, the job now runs on cache miss, despite clearly missing the cache.
Here's a relevant snippet of the run with debug logging enabled
Cache not found for input keys: release-0e0ceb0ce2360a93ec77b898ac06177b7429666b
##[debug]Node Action run completed with exit code 0
##[debug]Save intra-action state CACHE_KEY = release-0e0ceb0ce2360a93ec77b898ac06177b7429666b
##[debug]Set output cache-hit = false
##[debug]Finishing: Cache release folder
<snip>
##[debug]Evaluating condition for step: 'Panic if build already exists'
##[debug]Evaluating: (success() && steps.cache-release.outputs.cache-hit)
##[debug]Evaluating And:
##[debug]..Evaluating success:
##[debug]..=> true
##[debug]..Evaluating Index:
##[debug]....Evaluating Index:
##[debug]......Evaluating Index:
##[debug]........Evaluating steps:
##[debug]........=> Object
##[debug]........Evaluating String:
##[debug]........=> 'cache-release'
##[debug]......=> Object
##[debug]......Evaluating String:
##[debug]......=> 'outputs'
##[debug]....=> Object
##[debug]....Evaluating String:
##[debug]....=> 'cache-hit'
##[debug]..=> 'false'
##[debug]=> 'false'
##[debug]Expanded: (true && 'false')
##[debug]Result: 'false'
##[debug]Starting: Panic if build already exists

I'm seeing the same bad behavior.
As a workaround, I currently added a string comparison to my workflows:
      if: ${{ (!steps.cache-restore.outputs.cache-hit || steps.cache-restore.outputs.cache-hit == 'false') }}

This is cause by #1404, which was a major breaking change.
It is not clear why the change was necessary.

This should be fixed in https://github.com/actions/cache/releases/tag/v4.1.1, the v4 tag will be updated soon.

@joshmgross,
Thank you.

@itchyny I agree that using cache-hit as-is is confusing and difficult.
Our priority is to maintain the existing behavior and avoid unnecessary customer friction.
The existing cache-hit output really has two purposes:

Indicates if a cache was restored ('' vs non-empty string)
Indicates if an exact cache match was found ('true' or 'false')

In the future, I'd like to break that up into two outputs so that we don't need this odd ternary string to represent all 3 cases. Ideally that would come with proper support in the Actions runner for typed outputs, but that's not currently on our roadmap.
Alternatively, we adopt the changes that were reverted and properly release that as a breaking change with a v5 of this action. Major versions are not something we do often - we end up with mixed adoption across major versions and either have to maintain multiple major versions or leave users on older major versions out of important security updates or bug fixes.
I'm going to call this closed for now, but I'm open to further feedback on how we can improve cache-hit. Feel free to open an issue or discussion if you'd like to discuss that further.


[SPARK-34003][SQL][FOLLOWUP] Avoid pushing modified Char/Varchar sort attributes into aggregate for existing ones

What changes were proposed in this pull request?
In https://github.com/apache/spark/commit/0f8e5dd445b03161a27893ba714db57919d8bcab, we partially fix the rule conflicts between PaddingAndLengthCheckForCharVarchar and ResolveAggregateFunctions, as error still exists in
sql like SELECT substr(v, 1, 2), sum(i) FROM t GROUP BY v ORDER BY substr(v, 1, 2)
[info]   Failed to analyze query: org.apache.spark.sql.AnalysisException: expression 'spark_catalog.default.t.`v`' is neither present in the group by, nor is it an aggregate function. Add to group by or wrap in first() (or first_value) if you don't care which value you get.;
[info]   Project [substr(v, 1, 2)#100, sum(i)#101L]
[info]   +- Sort [aggOrder#102 ASC NULLS FIRST], true
[info]      +- !Aggregate [v#106], [substr(v#106, 1, 2) AS substr(v, 1, 2)#100, sum(cast(i#98 as bigint)) AS sum(i)#101L, substr(v#103, 1, 2) AS aggOrder#102
[info]         +- SubqueryAlias spark_catalog.default.t
[info]            +- Project [if ((length(v#97) <= 3)) v#97 else if ((length(rtrim(v#97, None)) > 3)) cast(raise_error(concat(input string of length , cast(length(v#97) as string),  exceeds varchar type length limitation: 3)) as string) else rpad(rtrim(v#97, None), 3,  ) AS v#106, i#98]
[info]               +- Relation[v#97,i#98] parquet
[info]
[info]   Project [substr(v, 1, 2)#100, sum(i)#101L]
[info]   +- Sort [aggOrder#102 ASC NULLS FIRST], true
[info]      +- !Aggregate [v#106], [substr(v#106, 1, 2) AS substr(v, 1, 2)#100, sum(cast(i#98 as bigint)) AS sum(i)#101L, substr(v#103, 1, 2) AS aggOrder#102
[info]         +- SubqueryAlias spark_catalog.default.t
[info]            +- Project [if ((length(v#97) <= 3)) v#97 else if ((length(rtrim(v#97, None)) > 3)) cast(raise_error(concat(input string of length , cast(length(v#97) as string),  exceeds varchar type length limitation: 3)) as string) else rpad(rtrim(v#97, None), 3,  ) AS v#106, i#98]
[info]               +- Relation[v#97,i#98] parquet

We need to look recursively into children to find char/varchars.
In this PR,  we try to resolve the full attributes including the original Aggregate expressions and the candidates in SortOrder together, then use the new re-resolved Aggregate expressions to determine which candidate in the SortOrder shall be pushed. This can avoid mismatch for the same attributes w/o this change, as the expressions returned by executeSameContext will change when PaddingAndLengthCheckForCharVarchar takes effects. W/ this change, the expressions can be matched correctly.
For those unmatched, w need to look recursively into children to find char/varchars instead of the expression itself only.
Why are the changes needed?

bugfix
Does this PR introduce any user-facing change?

no
How was this patch tested?

add new tests

Test build #133921 has started for PR 31129 at commit 236bb18.

Test build #133921 has started for PR 31129 at commit 236bb18.

Test build #133926 has started for PR 31129 at commit fe93b53.

Test build #133926 has started for PR 31129 at commit fe93b53.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38508/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38508/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38508/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38508/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38513/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38513/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38508/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38508/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38513/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38513/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38513/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38513/

Test build #133921 has finished for PR 31129 at commit 236bb18.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #133921 has finished for PR 31129 at commit 236bb18.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133921/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133921/

Test build #133926 has finished for PR 31129 at commit fe93b53.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #133926 has finished for PR 31129 at commit fe93b53.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133926/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133926/

Test build #133952 has started for PR 31129 at commit 3e4ca3f.

Test build #133952 has started for PR 31129 at commit 3e4ca3f.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38539/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38539/

Test build #133952 has finished for PR 31129 at commit 3e4ca3f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #133952 has finished for PR 31129 at commit 3e4ca3f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133952/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133952/

thanks, merging to master/3.1!

thanks, merging to master/3.1!


Reference FS.File directly to an object

Dear community,
I am a beginner with meteor and angular meteor, but it has been a pleasure to work with so far. So maybe my following issue is an obvious one. When I try to reference a FS.File directly to a property of an object as I read in the API documentation here: https://github.com/CollectionFS/Meteor-CollectionFS, I get an error saying ""TypeError: Cannot convert undefined or null to object".
After installing the additional package cfs:ejson-file I tried something like:
vm.images.save(vm.newProfileImage).then(function (result) {
                    var imageFS = result[0]._id;
                    vm.contact.profileImage = imageFS;
                });

instead of saving only the ID of the FS.File (like in Tutorial)
vm.images.save(vm.newProfileImage).then(function (result) {
                    var imageFS = result[0]._id;
                    vm.contact.profileImage = imageFS._id;
                });

Am I missing something? I am using the newest angular 1.2.1 package.
Thanks, Joël

if your still using the autopublish package you should be able to do something like this
$scope.profileImage = scope.$meteorObject(Images, {_id:result[0]._id })

and in the template get the image with

Thanks for your answer, I thought there is another way to get the actual image without going through the images collection.

Thanks @Urigo
I tried it with the new API and it seems working but I get a "Maximum call stack size exceeded" error when I'm working with direct image references (e.g. specific image per model).
Now with new API saving an image reference directly works as expected:
var newImage = new FS.File(vm.newProfileImage);
            newImage.metadata = {owner: contactId};
            var imageRef = Images.insert(newImage, function(error, imageId) {
                if (error) {
                    logger.error(error.toString(), error, 'Error');
                } else {
                    logger.log('image inserted successful: ' + imageId);
                }
            });

vm.contact.profileImage = imageRef;

loading image url in view is working but I got the "Maximum call stack size exceeded", also with one time binding option.

or using one time binding

Furthermore within controller functions the same error happens when I call either vm.contact.profileImage.getFileRecord() or .url().

Also getting the maximum call stack size exceeded error.

I believe the issue is that we're calling a function inside of the template img.url() which gets called every time the digest loop runs through.
This is my rough workaround for the issue, inside of a helper.
var profileImages = ProfileImages.find().fetch()
this.imgs = profileImages.map(img => img.url())

This kind of works. It's not reactive or anything though.

Ok, so your are saying since version 1.3.2 you haven't got the issue anymore? I am using version 1.3.4 and still having the issue. My situation is probably slightly different to your as I am using it without a separate image collection.
Thanks for your workaround. I could workaround my situation as follows:
        vm.helpers({
            contact: function() {
                return Contacts.findOne({_id: $stateParams.contactId });
            },
            profileImageUrl: function() {
                var contact = Contacts.findOne({_id: $stateParams.contactId });
                return contact.profileImage.url();
            }
        });
```
instead of:
```
        vm.helpers({
            contact: function() {
                return Contacts.findOne({_id: $stateParams.contactId });
            }
        });

        var profileImageUrl = vm.contact.profileImage.url();
```

Woah, woah, woah, you can add file objects to a collection?
I'm calling the url method within the template.
Did the workaround work for you?
Yeah, I locked my version a 1.3.2 and I don't get the infinite watchers bug when calling .url() within a template. I would recommend adding a check for the subscription completion so that it doesn't throw a reference error.

Yes, indeed. You'll find the official documentation here: https://github.com/CollectionFS/Meteor-CollectionFS#storing-fsfile-references-in-your-objects. In order to get this to work you need to add an additional package: cfs:ejson-file
yeah for the sake of simplicity I did put the line directly under helpers here but yes you are right in my real controller I had put it after subscription is ready and vm.contact is fully loaded.
the workaround works great so far, for objects in collection without profileImage I modified it slightly:
            profileImageUrl: function() {
                var contact = Contacts.findOne({_id: $stateParams.contactId });
                return contact.profileImage ? contact.profileImage.url() : undefined;
            },

@cranesandcaff so everything works as you need? can I close this issue?
Thanks.

Good on my end.


NeoForge in Curse Forge modpacks

Enhancement Type
Improve an existing feature
Describe the enhancement
Some modpacks at Curse Forge are using NeoForge as their mod loader (for example ATM10).
Currently starting the modpack requires to start it with CF, after the mod has been downloaded, the type can be switched to NEOFORGE. However, it would be nice if CF had support for NeoForge as well :-)

Thanks for pointing this out. I had added NeoForge support for Modrinth modpacks, but apparently I didn't circle around and add it for CurseForge ones.

Do you have an example how to use it with CF/NEOFORGE :-)?

Referencing ATM10, as you found, would be the example. Making sure you've re-pulled the image it will be specified using https://docker-minecraft-server.readthedocs.io/en/latest/types-and-platforms/mod-platforms/auto-curseforge/

working docker compose for reference.
atm10:
    container_name: atm10
    image: itzg/minecraft-server
    environment:
      MEMORY: 8G
      EULA: true
      MOD_PLATFORM: AUTO_CURSEFORGE
      CF_API_KEY: ${CF_API_KEY}
      CF_SLUG: all-the-mods-10
      CF_FORCE_SYNCHRONIZE: "true"
      CF_EXCLUDE_MODS: | # remove problematic client side mods
        lighty
        986380
      TYPE: NEOFORGE
      VERSION: 1.21
      NEOFORGE_VERSION: 21.0.167
    tty: true
    stdin_open: true
    restart: no
    volumes:
      - dataatm:/data
volumes:
  dataatm:


Making the H2 Blue

Challenge Use CSS Selectors to Style Elements has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_2) AppleWebKit/602.3.12 (KHTML, like Gecko) Version/10.0.2 Safari/602.3.12.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

CatPhotoApp

Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching attack your ankles chase the red dot, hairball run catnip eat the grass sniff.

@MakeCodeOrDie Thank you for reporting this issue. Your code is incorrect. Please reread the last paragraph of the challenge for help.

Delete your h2 element's style attribute and instead create a CSS style element. Add the necessary CSS to turn all h2 elements blue.

This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "help" button on the challenge on Free Code Camp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification. Thank you, and happy coding!


Added > and ' to XML entity encoder.

What problem does it solve?

To generate translation for strings with source such as
<strike>%1$@</strike> • <color name='green50'>%2$@ for %3$#@nights@</color>
With out the additional entries, it was encoding to
<strike>%1$@</strike**>** • <color name='green50**'>**%2$@ por %3$#@nights@</color>
Which is ok for Android, but I wasn't sure if it was ok in stringsdict and just wanted to be sure


Contract test for http public api

[x] introduces http mock server in client
[x] both mock server and real server pass the same test
[x] client sdk uses mock server instead of request mocks

@electricmonk I think I’ve addressed all your comments, please take a look

@netoneko Please make sure to create tickets for the review comments you did not yet address


Proxy: update io_uring code

Builds on another PR so ignore the first two commits.
Updated the io_uring code to match the libevent code. Pushed a little further to add writes into uring, but performance is not improved over the libevent code.
Not giving up on it yet but it's good to periodically update the code or we'll never get there anyway.
Since the bulk of it is ifdef'd uring code that isn't stable anyway, all we need to review here is the simplification to the flush_pending_write code.

been benchmarking code on top of this for weeks. just fixed a bug with uring compilation that snuck in and merged.


Correctly handle "unique" params in filters

Currently, once a unique param is used in filter, the table is marked as no_filter, i.e., no more filter can be added to it. However, in some cases, this is not reasonable:

or operator: this should always be allowed even if unique param is used
not operator on unique param: unless the param is of type boolean or enum with only 2 possible values, this should turn the filter to be not unique

Also, if the filter value is not a constant

Also, if the operand is not "equal"


Allow change "__api_party" endpoint

Describe the feature
Allow change "__api_party" endpoint string
Additional information

[ ] Would you be willing to help implement this feature?
[ ] Can you think of other implementations of this feature?

Final checks

[X] Check existing issues.

May ask why do you intend to change this slug?

we want to obfuscate as much information as possible about the packages we work with


vMix integration not working

Hello, i can't make it work with vMix. TCP API should be available on port 8099, i can telnet into localhost port 8099 but the software it doesn't work. I have followed your tutorial strictly.
I noticed TCP API is used: please consider using HTTP API instead which seems to be more reliable. Other products like Universal Time Controller are using them successfully.
I have made it work using Putty as telnet on port 8099 and sending SetText command, so it should be someting wrong in the software

thanks for the ticket - this had been reported several times.
i'm investigating the source of the bug

@FabrizioBocci this shuold now be resolved in the released version

resolved

I try to get the VMix API working in the last version, but without any success. I download Scoreboard OCR and that works. With port localhost 8099 and right input en right names, do you have more complains with the last version?

Arie, are you sure you're using the correct port? Default port for the vMix-API is 8088?

Hi Rick, Yes i'm sure for Scoreboard info you need port 8099, look at the instructions and on Scoreboard OCR it works!
You use an other port for Scoreboard - VMix API?

https://github.com/locaal-ai/scoresight/blob/main/docs/vmix.md

i follow the document but without success, see my attached log files.
scoresight_2024-12-05_00-15-43.log
scoresight_std_2024-12-05_00-15-43.log

i change the port to 8088 and now it works!!

i change the port to 8088 and now it works!!

A great! Fijn Arie! 😀


WebUI unresponsive when session expires

Describe the bug
When using authentication on the CodeChecker server, webUI sessions will expire after a configurable amount of time.  If the session expires while a user still has the webUI open, it will become unresponsive instead of redirecting the user back to the login page.
CodeChecker version
6.8.1
To Reproduce

Start the CodeChecker server with authentication enabled (I used LDAP, but it's probably the same for everything)
open the WebUI and log in
leave the page to sit idle until the session expires
attempt to interact with the WebUI, especially opening a new run or report, etc.  Observe that nothing happens (or a loading icon will spin indefinitely).

Expected behavior
I'd expect the WebUI to prompt the user to log in again.
Desktop (please complete the following information)

OS: server running on Linux, browser running on Win 10.
Browser: Firefox
Version: 63.0.3

This has been fixed in #2057.


Align welcome view ids, add enum to schema documentation

fixes #91466
@isidorn This would need some adoption from debug extensions if any already contributes to the debug view.

Closing per https://github.com/microsoft/vscode/issues/91466#issuecomment-591843931


Many baseplate connections hang due to an upstream bug in Gevent

Upstream bug I filed: https://github.com/gevent/gevent/issues/1505
Gevent claims that [1]
``handle`` will be executed
        using :meth:`gevent.pool.Pool.spawn` only if the pool is not full.
        While it is full, no new connections are accepted

Unfortunately, the full check is stubbed out with "False" [2] such that the pool is never considered full.  I can't find anywhere in some subclass that this check might be overridden with a real check.  So this invariant appears broken upsteam.
gevent.StreamServer looks like it's thread-per-connection.  The default pool behavior when full is blocking, not rejecting.  We appear to use a default bounded pool.
The end result of this is that when max_connections is enabled in baseplate, we will accept a bunch of connections that we cannot service.  Those connections will simply hang.
A simple reproduction [5] on a brand-new baseplate-cookiecutter project confirms this.
I know max_connections is deprecated, in part because we are suspicious of it.  Perhaps understanding the root cause of our trouble is useful, and then we can retain the value of a properly implemented max_connections.
Possible solutions:

Set the thread pool behavior to error-on-full instead of block-on-full.  This is low effort and will prevent the hanging behavior.
Wait for upstream to fix.
Look at the circuit-breaking [3, 4] PRs that fix this by not using the threadpool to be the concurrency limiter.

[1] https://github.com/gevent/gevent/blob/master/src/gevent/baseserver.py#L64
[2] https://github.com/gevent/gevent/blob/master/src/gevent/baseserver.py#L245
[3] https://github.com/reddit/baseplate.py/pull/383
[4] https://github.com/reddit/baseplate.py/pull/384
[5] Set max_concurrency to something low like 5, modify the test-client to log on connect, then leave the connection open after the request for 30s, then do 10 requests in parallel.  You'll get 10 successful connects and only 5 requests serviced.

This should be fixed by the removal of max_concurrency in 1.2.


Custom Domain: gamedev.rs

Closes #233 ("Custom Domain")
Related to #586 ("Shorter URLs + split "posts" into news+blog")

It's required by https://docs.github.com/en/pages/configuring-a-custom-domain-for-your-github-pages-site/managing-a-custom-domain-for-your-github-pages-site


chai.spy.returns

Not a big deal but http://chaijs.com/plugins/chai-spies shows the readme on the master branch which includes the new chai.spy.return feature. This feature thats on master hasn't been tagged and released yet. So if you follow the instructions npm install chai-spies. The api for chai.spy.returns is missing.
You come to the gh repo and master has spy.returns. Should this be tagged and released?

Any news on this? Would love to use this functionality.

Sorry for the delay. I was considering cutting a release as a major version - but I think I might want to drop chai.spy().reset() and 1.0 onwards keeping everything out of the spy prototype. I'll work a bit on it in a few hours.

Any updates on this?

Update?  I need this to return true to a confirm dialog...

Any updates on this? We do need this feature and we can not access development/master version directly. May we know when will be the next version released?

@keithamus is it possible to make me contributor and provide all relevant rights? So, then I will work on new sandbox feature and rest stuff which we discussed about year ago?

@stalniy happy to make you a maintainer, but either way you are free to make PRs against the code to work on features. Our maintainers do not code directly into master, the only benefit of being maintainer is you can close issues and approve other user's PRs. Do you want me to make you a maintainer?

@keithamus sure, thanks! I just want to move stuff a bit forward and make chai spies more popular as it's really a shiny lib with the easiest/shortest implementation and powerful features

@keithamus @stalniy what's the recommended way to obtain this feature? I'm trying to move a large project from jasmine to mocha and the .returns functionality is key. Is there a tagged version of the module, or should I require on the repo itself?

We have few ideas of how to implement this in #38 but haven't agreed on the interface, because want leave all spies to be immutable.
There is a PR for Sanboxes feature which I plan to finish this weekend and then get back to spy.on
In case if you use Angular it shouldn't affect you because angular allows to recreate services before each test. So, you can do service.method = spy.returns(5).
As another temporary workaround you can do the same way or create a wrapper function to properly track and restore spies.

Any updates on this?

@stalniy @keithamus hey fellas, blocking the release of returns on several other issues that don't have any traction is brutal. We don't use angular, as as I laid out in #62, we're trying not to mix components for spies.
is there any possibility we can at least get an @beta tagged version of this module so we can start using that method? I am loathe to publish a derivative module of this from a fork, but we're in great need.

@shellscape For the existing returns feature you can simply monkey patch with a one-liner, it is just chai.spy.returns = (v) => chai.spy(() => v). Any other returns proposals such as #62 are an area of contention right now - I think it's important we retain immutability in spies to simplify testing.

@keithamus thanks for getting back to me on that. still bummed that the code in master is still blocked, but appreciative and hope you guys are able to move forward.

@shellscape to be honest me too, I'd like to use some of the features already in master, but I want to make sure we get the API right in a way that means we don't just ape other libs, and keep things clean and simple. FWIW you could, if you wanted you could always install chai from a specific commit ref, for example npm i chaijs/chai-spies#58f50ec3 will install a version of chai-spies based on 58f50ec3ccee4dfc7309e774a97a30a6f0cc6c6a which including chai.spy.returns()


Solve GLIBC_2.34 not found

Hi,
this PR solves the issue https://github.com/yuk7/ArchWSL/issues/280
As stated here https://bbs.archlinux.org/viewtopic.php?id=274705, executing sudo pacman -Syy ... before upgrading the whole system could lead to partial upgrades.
Thanks

Installing archlinux-keyring is not considered a partial upgrade, and this steps as documented in ArchWiki: https://wiki.archlinux.org/title/Pacman/Package_signing#Upgrade_system_regularly. See:

Upgrade system regularly
Upgrading the system regularly via pacman#Upgrading packages prevents most signing errors. If delay is unavoidable and system upgrade gets delayed for an extended period, manually sync the package database and upgrade the archlinux-keyring package before system upgrade:

# pacman -Sy archlinux-keyring && pacman -Su

This command is not considered a partial upgrade since it syncs the package database and upgrades the keyring package first. Both must be processed just before starting system upgrade to ensure signatures of all upgraded packages can be properly verified.

thank you @rffontenelle. Doesn't worth to add the sudo pacman -Syu command in the installation guide? Otherwise, if a user installs directly a package eg. curl without upgrading the system has that issue related to GLIBC_2.34

First, I am unable to reproduce this error. I just reinstalled my ArchWSL instance in WSL2 and followed every step in the How to Setup guide, and I was able to install archlinux-keyring without issue. I ran wsl.exe --unregister Arch and remove completely the folder where Arch.exe is in order to make sure I'd be register from scratch using the instructions in the setup guide.
Can you please try redoing it as well to confirm the issue that was presented to you?
Second, I have no doubt that running system upgrade (sudo pacman -Syu) is important, but not before installing the keyring. Installing the keyring before running system upgrade is exactly the point of the How to Setup's "Initialize keyring" section: without Arch's keyring properly set, some packages may fail to install if the developer/TU's keyring is not known to pacman. For example, see the first two messages in https://bbs.archlinux.org/viewtopic.php?id=191279

I am still able to reproduce the error. I run the following commands
>Arch.exe
[user@PC-NAME]$ sudo pacman-key --init

[user@PC-NAME]$ sudo pacman-key --populate

[user@PC-NAME]$ sudo pacman -Syy archlinux-keyring

[user@PC-NAME]$ sudo pacman -S curl

[user@PC-NAME]$ curl

Did you run these commands as well?

Had the same issue and pacman -Sy archlinux-keyring && pacman -Su after populate made all the difference for me.

@yuk7 @salvomcl I forgot to mention that you would run into keys not recognized or expired issue if you do pacman -Syu before pacman -Sy archlinux-keyring. So either swap the location of the two newly added lines or paste in the one line I mentioned above.


I chose "Allow" every time this popup appeared, and now my auto-format on save has stopped working!

This has also happened to me.
ESLINT output shows
[Info  - 6:08:48 PM] ESLint server is starting
[Info  - 6:08:49 PM] ESLint server running in node v12.14.1
[Info  - 6:08:49 PM] ESLint server is running.
[Info  - 6:08:50 PM] ESLint library loaded from: /Users/<USER>/Documents/<repo>/node_modules/eslint/lib/api.js

I chose "Allow" every time this popup appeared, and now my auto-format on save has stopped working!
Originally posted by @sarahsga in https://github.com/microsoft/vscode-eslint/issues/1012#issuecomment-659546167

@drew-royster can you share a repository I can clone that demos this. Are you seeing eslint errors in the problems view?

Apologies I believe that this was a mistake on my end.

No problem.


POST JSON with expectations with HTTP client of your choice

hi,
in the past I have set up expectations using the Java client but for more ad hoc, lightweight testing I don't want to be compiling Java classes. So I thought I'd try JavaScript, but there, too it's not a matter of firing up node and typing in any JS. You need the mockserver-client module, and to install it you need a package.json. And all that even though it says in the docs that "To specify a request matcher in JavaScript use JSON". Why can't I just POST some JSON to the mock-server using e.g. cURL?

You can't the clients are only provided to make things simpler.
I do plan on documenting the JSON Rest API to make this more straight forward, see #287.
I'm going to close this as a duplicate of #287.

ah, nice! I dug around in the code a bit and found the /expectation endpoint. So it's just a PUT to /expectation with the JSON that can be copied from the JS examples but without JSON.stringify. The "body" value is a string that contains a serialized JSON object with quotation marks escaped like so:
{
"httpRequest": {
"method": "GET",
"path": "/ts-json",
"queryStringParameters": [
]
},
"httpResponse": {
"statusCode": 200,
"body": "{"timestamp": "2010-01-01T00:00:00Z""
}
}
cool, thanks!


[system] Move stylesheet generator to extendTheme

A part of #40225
closes #40178
Summary

This PR moves the stylesheet generation to extendTheme for all packages (Material UI and next, Joy UI, Pigment CSS) via generateStyleSheets().
The generateCssVars is refactored to generateThemeVars to remove the duplicated logic across packages.
The switch between @media prefers-color-scheme, classes, or data attributes can be fully controlled by getSelector

Next step

Remove attribute, colorSchemeSelector props from CssVarsProvider (configured through theme instead)
Maybe renaming attribute to rootSelector.
Consider moving to class instead of data attribute #41068

[x] I have followed (at least) the PR section of the contributing guide.

I will move this to next branch instead.


Smoke test: install and enable vscode-smoketest-check extension

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=142153&view=logs&j=3792f238-f35e-5f82-0dbc-272432d9a0fb&t=07088cf7-fede-5f74-0d50-155edd53af7d&l=80
1) VSCode Smoke Tests (Electron)
       Extensions
         install and enable vscode-smoketest-check extension:
     Error: Timeout: get element 'div.extensions-viewlet[id="workbench.view.extensions"] .monaco-list-row[data-extension-id="ms-vscode.vscode-smoketest-check"]' after 20 seconds.
      at poll (/Users/runner/work/1/s/test/automation/out/code.js:178:19)
      at async Code.waitForElement (/Users/runner/work/1/s/test/automation/out/code.js:262:16)
      at async Extensions.searchForExtension (/Users/runner/work/1/s/test/automation/out/extensions.js:28:9)
      at async Extensions.installExtension (/Users/runner/work/1/s/test/automation/out/extensions.js:38:9)
      at async Context.<anonymous> (out/areas/extensions/extensions.test.js:19:13)

There seems to be network connectivity issue on the machine this was run.


⚠ Itaca Gallery has degraded performance

In 8d6e3f9, Itaca Gallery ($SITE_TCGLR) experienced degraded performance:

HTTP code: 200
Response time: 5157 ms

Resolved: Itaca Gallery performance has improved in 488c727 after 6 minutes.


Random to random results (5%)

I will store results for some time here, until we will resolve, if we need page for validation tests.
Polynomials, using only mutation (config.xml) resulted with 0.041289 (993 runs).

Polynomials, using mutation and crossover (with 20 individuals) both on polynomials and terms (config) resulted with 0.043 (1000 runs).
For polynomials, we are done, but we still want to check, if using more individuals is OK. So I'll run one more experiment, this time with 10 000 runs.

I expect, that results different from 5% were caused by polynomials count != 8*output_size. For this reason, I added another sanity check in configuration loading in commit https://github.com/crocs-muni/EACirc/commit/3f8c5c6eba467e8e7843f8c16ff80728c4fd6026.


adding the possibility to call others tasks from a task

This PR enable calling others tasks from a task within the same context.
orbit.yml:
tasks:
  - use: app
  - run:
      - {{ run "render" "upload" }}
      - echo "done!"

  - use: render
  - run:
      - cmd

  - use: upload
  - run:
      - cmd

Fixes #24

@j-vizcaino this solution is ok for your issue? 😄

Yes, this looks really good! Thank you very much!
I will give this a try and let you know if this works for my use-case.

Great! Thank you for the feedback :)


refactor(obstacle_velocity_limiter): rework parameters

Description
Implement the ROS Node configuration layout for the obstacle_velocity_limiter package.
Test performed

Build olcon build --symlink-install --cmake-args -DCMAKE_BUILD_TYPE=Release --packages-up-to obstacle_velocity_limiter
Launch ros2 launch obstacle_velocity_limiter  obstacle_velocity_limiter.launch.xml

Notes for reviewers
None
Interface Changes

Create schema.json files
Update README.md

Effects on system behavior
None
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed  the contribution guidelines
[x] The PR follows the pull request guidelines

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.
After all checkboxes are checked, anyone who has write access can merge the PR.

@Ariiees Could you update this branch and resolve conflicts so that we can merge this PR?


Feature request: dbt-osmosis yaml audit

I think it would be pretty useful to have a command to execute the audit standalone.
For example this line dbt-osmosis yaml audit -f development.test would have the following output:
             ✅ Audit Report
             -------------------------------

             Database: awsdatacatalog
             Schema: development
             Table: test

             Total Columns in Database: 3.0
             Total Documentation Coverage: 100.0%

             Action Log:
             Columns Added to dbt: 2
             Column Knowledge Inherited: 1
             Extra Columns Removed: 0

Then, one would be able to do whatever they want with it: post it as a comment in a PR or send it as a Slack message, etc

We have moved away from the standalone audits for now, the focus is narrow on purely automating most of the work. We may revisit at another point if someone takes another stab at the feature or dbt core implements it themselves


Integrate page object support

It seems that spectron does not handle page objects implicitly, so implementing like the example below will result in a browser is defined error.
Docs:
http://webdriver.io/guide/testrunner/pageobjects.html
Example from webdriver io:
https://github.com/webdriverio/webdriverio/tree/master/examples/pageobject

so implementing like the example below will result in a browser is defined error.

Have you tried replacing the browser calls with app.client calls instead? I think that should work.

app.client throws an app is not defined error. Here is a pr using pageobjects if you want to try out and double check. https://github.com/StephenDavidson/electron-spectron-example/pull/4

app.client throws an app is not defined error.

Yeah, app is not a global by default, you should pass in the value from your spec or set it as a global.
I commented on your pull request.

Works. The only issue I run into is that webdriverio implementation assumes sync tests, where it looks like spectron always runs async. Is there any option to force sync?

@StephenDavidson you can just use async/await to build the same PO as WebdriverIO in sync mode.


"Define Keybinding" should not be in F1

It does nothing atm and is confusing:

Define keybindings is not seen in the F1 list.. Looks like fixed.


Error on IE 10 when using non-string messages

On IE10, there is no ArrayBuffer.isView() method. As this is called when try to call the md5 function with a blob or anything else than a string, IE 10 reports error of isView not being defined.
As a workaround, a polyfill could be created to add isView() function to ArrayBuffer (should be pretty simple, I made it in several lines of code) or change the code to not call that method, at all.

I will check this. Thanks.

Added polyfill in v0.6.1. But I have no IE 10 to test this. Please try it. It should work. Thanks.

Apparently seems to work now. Anyway, you can grab IE 10 for testing purposes on a VM from Microsoft, here: https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/


Add peripheral file hosting and peripheral doc hosting to cloudbuild-deploy.yaml

Done! https://cyanobyte.dev/docs/reference/peripheral-docs/


PostGIS 2.2.2 released

http://postgis.net/2016/03/22/postgis-2.2.2/

Thanks! I saw the announcement, but Apt packages were not yet available
when I checked.

They appear to have been uploaded yesterday to the jessie-pgdg-testing repo, but they're not yet available in jessie-pgdg.

@ximex The updated 2.2.2 images are available on Docker Hub :+1:


Split repository to support Multiple languages

Split repository to support Multiple languages
First commit for C# in .Net Core 1.0 RTM

VS2015 update 3 + Net core SDK is mandatory to open in Visual Studio
Maybe working in VsCore (updated) + C# extension (not tested)

Thanks for the PR
Please move back .travis.yml file at the root of the repository and update inside this file the path to m

Thanks for the PR
Please move back .travis.yml file at the root of the repository and update inside this file the path to maven pom.xml files

Thanks for the PR !
Yes, moving back the main travis.yml and prefixing paths there should do the trick.

Closing ... Creating another repo ... Probably

Too bad ! I guess everything in a single repo would be simpler on the long run

I thought you wanted to create a new repo with AppVeyor etc ... inside "RxHandsOn" organisation last friday

Ok i did saw the previous msg

Did not see*

Ok this one is funny ;)

So the "Buffer size" was to 70 ... i'm just stupid. The Replay Buffer seems to works.
But stil there's an issue, there's 50% of chance to have a HUGE latency on connection.
Even a breakpoint on the first line of the Kestrel Middleware takes 3 to 10 sec to be catch, my guess is that there's really dirty lock/hang on my first attempt to use Kestrel for the Implementation of the SSE server.
I can't figure out yet why the first line of the Kestrel handler is having a bad time


After selecting 'Pet News,' it disappears from the nav bar, and similarly, 'About Us' disappears after selecting 'Donate.

Describe the bug
A clear and concise description of what the bug is.
Pet news isnt appearing in the nav bar and similarly about us is disappearing after selecting donate
To Reproduce
Steps to reproduce the behavior:

Go to 'https://akshitagupta15june.github.io/PetMe/index.html
Click on 'pet news', 'donate'
See the Nav bar'
See error

Expected behavior
A clear and concise description of what you expected to happen.
After selecting 'Pet News,' I expected it to remain visible in the nav bar, providing continued navigation access
After selecting 'donate,' I expected 'about us' to remain visible in the nav bar,because it is the required information,  providing continued navigation access
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [windows]
Browser [chrome]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [Version 120.0.6099.130 ]

Additional context
Add any other context about the problem here.

i kindly request u to assign me this issue under JWOC 2024

yes its not needed on those pages


chore: daddy script

the text will finall come to "#text" area, so here need to check parent again, to check if the parent of "#text" is textarea of input

And I found the node name all upcast.. so maybe upcast is needed

🙏 🙏 🙏


Fast Removal for .doc/.docx type not working

Fast Removal for word document throws error

After a numbers of tests, I can confirm that if your document doesn't have the valid structure required as a .dox or .doc file (possibly damaged or ancient version), it will trigger this error:
Error: "no relationship of type 'http://schemas.openxmlformats.org/officeDocument/2006/relationships/officeDocument' in collection"

Will possibly try another library other than python-docx in a feature update.


Odata Query() vs QueryAsync()

Hi,
I am using a mobile application and I need to filter pull using an Id this id does not belong to object so i can not use it directly, DomainManager.Query() method does not accept parameters, so I have tried to use QueryAsync() method that accepts other parameters apart from OData parameters
The problem is I have a GET like:
_https://subdomain.azurewebsites.net/tables/acmes?$filter=(updatedAt ge datetimeoffset'2018-09-30T17%3A53%3A05.6680000%2B00%3A00')&$orderby=updatedAt&$skip=0&$top=50&id=9c70ad2c-b159-4946-b9f6-ac6e494afc5d&__includeDeleted=true_
I have a controller with:

    [Route("")]
    [HttpGet]
    public IQueryable<AcmeDto> GetAllAcme(ODataQueryOptions query, string id)
    {
        return domain.QueryAsync(query,id);
    }

    [Route("")]
    [HttpGet]
    public IQueryable<AcmeDto> GetAllAcme()
    {
        return Query();
    }

When Query() is executed the value for this.Request.RequestUri is:
https: //demo.azurewebsites.net/tables/acmes?$filter=(UpdatedAt ge datetimeoffset'2018-09-27T16:20:27.5970000%2B00:00')&$orderby=UpdatedAt&$skip=0&$top=50&__includeDeleted=false HTTP/1.1&$select=Name,MimeType,Url,UserId,BlobGuid,DevicesId,Data,Id,Version,CreatedAt,UpdatedAt,Deleted
But when QueryAsync() is executed the value for this.Request.RequestUri is:
https: //demo.azurewebsites.net/tables/acmes?$filter=(updatedAt ge datetimeoffset'2018-09-30T17:53:05.6680000%2B00:00')&$orderby=updatedAt&$skip=0&$top=50&id=9c70ad2c-b159-4946-b9f6-ac6e494afc5d&__includeDeleted=true
One of the problems is I have an error like:
Microsoft.Data.OData.ODataException: 'Could not find a property named 'updatedAt' on type 'AcmeDto'.'
I suppose for Query() there is somekind of url preprocesssing or entity matching or any else, How can be it replicated in QueryAsync()?

Hey- I have a few questions to help understand the problem you are having.

What exactly is happening when you call Query()? Is that a wrapper method that you added to your controller? Or is it part of something you've imported that we aren't seeing? (In other words, what is the full qualification of that method?) - Alternatively, a call stack at the two query calls could prove helpful.

What storage is your mobile backend using? Azure Table Storage or SQL Database?

Is your schema case-sensitive? It appears that there is a capitalization difference between the updatedAt filters in the two URLs.

Thanks!

We haven't heard from you for a while so I'll go ahead and close this issue. Feel free to comment and we'll reopen.

Thanks for the questión seems the problem was solved using [EnableQuery] attribute on controller.

We are facing similar issue with our application , below is the stack trace
{
"message": "The query specified in the URI is not valid. Could not find a property named 'updatedAt' on type 'WebApplication1.DataObjects.TodoItem'.",
"exceptionMessage": "Could not find a property named 'updatedAt' on type 'WebApplication1.DataObjects.TodoItem'.",
"exceptionType": "Microsoft.Data.OData.ODataException",
"stackTrace": "   at Microsoft.Data.OData.Query.EndPathBinder.GeneratePropertyAccessQueryForOpenType(EndPathToken endPathToken, SingleValueNode parentNode)\r\n   at Microsoft.Data.OData.Query.EndPathBinder.BindEndPath(EndPathToken endPathToken, BindingState state)\r\n   at Microsoft.Data.OData.Query.MetadataBinder.BindEndPath(EndPathToken endPathToken)\r\n   at Microsoft.Data.OData.Query.MetadataBinder.Bind(QueryToken token)\r\n   at Microsoft.Data.OData.Query.BinaryOperatorBinder.GetOperandFromToken(BinaryOperatorKind operatorKind, QueryToken queryToken)\r\n   at Microsoft.Data.OData.Query.BinaryOperatorBinder.BindBinaryOperator(BinaryOperatorToken binaryOperatorToken)\r\n   at Microsoft.Data.OData.Query.MetadataBinder.BindBinaryOperator(BinaryOperatorToken binaryOperatorToken)\r\n   at Microsoft.Data.OData.Query.MetadataBinder.Bind(QueryToken token)\r\n   at Microsoft.Data.OData.Query.FilterBinder.BindFilter(QueryToken filter)\r\n   at Microsoft.Data.OData.Query.ODataUriParser.ParseFilterImplementation(String filter, IEdmType elementType, IEdmEntitySet entitySet)\r\n   at System.Web.Http.OData.Query.FilterQueryOption.get_FilterClause()\r\n   at System.Web.Http.OData.Query.Validators.FilterQueryValidator.Validate(FilterQueryOption filterQueryOption, ODataValidationSettings settings)\r\n   at System.Web.Http.OData.Query.FilterQueryOption.Validate(ODataValidationSettings validationSettings)\r\n   at System.Web.Http.OData.Query.Validators.ODataQueryValidator.Validate(ODataQueryOptions options, ODataValidationSettings validationSettings)\r\n   at System.Web.Http.OData.Query.ODataQueryOptions.Validate(ODataValidationSettings validationSettings)\r\n   at System.Web.Http.OData.EnableQueryAttribute.ValidateQuery(HttpRequestMessage request, ODataQueryOptions queryOptions)\r\n   at System.Web.Http.OData.EnableQueryAttribute.ExecuteQuery(Object response, HttpRequestMessage request, HttpActionDescriptor actionDescriptor)\r\n   at System.Web.Http.OData.EnableQueryAttribute.OnActionExecuted(HttpActionExecutedContext actionExecutedContext)"
}


How to reload configuration of nginx with winsw?

Thank for you reading this question.
I wrapped nginx as a service with winsw,but when I reload it ,cmd reminded me :
nginx: [error] OpenEvent("Global\ngx_reload_4308") failed (5: Access is denied)
What happend?What should I begin?

It seems to be an issue with Nginx permission management. Probably you could find a solution in this thread: https://forum.nginx.org/read.php?11,238513 (see the last post)

Yes,it's an issue with Windows Service permission management.
I fixed it by set the service login as Administrator.
Thank you.
:)

Yes,it's an issue with Windows Service permission management.
I fixed it by set the service login as Administrator.
Thank you.
:)

Thank you
This solution worked for me

Error:
nginx: [error] OpenEvent("Global\ngx_reload_21428") failed (2: The system cannot find the file specified)

Reason:
Due to a config error nginx exited.

Solution:
The nginx was not running, started it.


feat: impl cheatcode tracer

What :computer:

Adds cheatcode tracer

Why :hand:

downstream users of era-test-node (e.g foundry-zksync) can use cheatcodes to override

Evidence :camera:
Include screenshots, screen recordings, or console output here demonstrating that your changes work as intended

Notes :memo:

Allows some simple cheatcodes (deal(), setNonce()) to work
This design doesn't allow us to re-use the methods from RPC (e.g hardhat_impersonateAccount), some of these i'm not sure how to implement

Closing as this now exists in foundry-zksync repo.


ACT-658: Re-write programs listing, creation and deletion logic

What is the Purpose?
Re-implement Programs using Vue: Create modal
Re-implement Programs using Vue: Listing component
What was the approach?
Re-write programs listing, creation and deletion logic
Are there any concerns to addressed further before or after merging this PR?
State some additional info if any. For instance running install or setting some environment variable(s)
Mentions?
@Kimaiyo077
Issue(s) affected?
ACT-658
ACT-724

Codecov Report

Merging #345 into dev will decrease coverage by 0.02%.
The diff coverage is 23.33%.

@@            Coverage Diff             @@
##              dev     #345      +/-   ##
==========================================
- Coverage   50.18%   50.15%   -0.03%
==========================================
  Files          91       91
  Lines        8242     8252      +10
==========================================
+ Hits         4136     4139       +3
- Misses       4106     4113       +7

Impacted Files
Coverage Δ

workflow/urls.py
100.00% <ø> (ø)

workflow/views.py
30.24% <23.33%> (-0.01%)
:arrow_down:

Continue to review full report at Codecov.

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

@IvyMMutiso also in general I see you mentioned this PR is for Program CRUD but I see changes in project and stakeholder modal files. Apologies if I can not understanding it correctly but can you please clarify if this PR is addressing those as well? Then I can check accordingly

Am looking into the dependencies error