Release timeline/branch

Do you have a timeframe you want to do the release in? I am currently running the latest master (as of like an hour ago) against the unit tests I have at work just to make sure nothing crazy comes up that the library's tests didnt catch, if you plan on releasing soon I can make sure we are on the release or master branch on our pre-prod system and report any strangeness.
The only thing I have noticed so far: running migrations appears to be a bit slower than the version we were using that was a month or two old. Runing a trace, it appears to be spending a lot of time determining if a migration should be run on a db, more time than it used to. Not sure if this is a repeatable behavior though, I will make some new junk migrations and see how they go.

If I can release by next week that would be great. What's the best timeline for you as far as testing it out and being reasonably confident?
Hmm. And this is on the same version of Django? Interesting. If this is true and we can compare to another code base where it's not taking as long, we can try to deduce it from the trace of both the old and the new?

I am not 100% certain its the allow_migrate, that just seems to be called a lot in the trace. I don't have a trace from the version that was running faster for me. I will investigate some more and maybe checkout one of the old versions of the branch I was using.

Timeline for us is almost immediate, we have 98%+ unit test coverage and are averaging over 20 executions per line during unit tests for all our sharded model code. I will run our end to end tests as well just to be ultra safe, but I can let you know of anything breaking by tomorrow evening. The e2e tests take a while since we do billing stuff in them.

Brillaint, if I was to upgrade our app it would also be same day. If you can run your tests for tomorrow evening and give me a :+1 then I'll run my app through it tomorrow as well and release it tomorrow night/ the next morning or asap if something comes up.

That sounds great. Will let you know, running our unit tests now and will kick off e2e tests in the morning (they take hours)

adding a few small fixes, nothing that should break it: https://github.com/JBKahn/django-sharding/pull/41 . Mostly linting.

Our test suite is passing.

Our test suite is passing as well, but I can confirm the migration stuff is definitely slower than it was. Our testing suite between the two versions runs for ~ 10 minutes longer with no other code changes, and one of my co-workers has complained about it this morning a few times already since they had new migrations to deal with.

I am betting this new code in the router is the culprit, inspecting the stack in python is super slow, and when we are doing migrations stuff, we are applying it to 512 shards + our main default database, so this is being executed 512 times more often than it usually does.
# New versions of Django use the router to make migrations with no hints.....
        making_migrations = any(['django/core/management/commands/makemigrations.py' in i[1] for i in inspect.stack()])
        if making_migrations:
            return True

I need a way to detect it was making migrations because they do not pass the model name as a hint.

I decided to open a ticket on https://code.djangoproject.com/ticket/27200#ticket

Interesting. Thanks for opening a ticket. I will take a look as well to see if there is something we can do that's more efficient. It is killer right now: making a migration for a single field on a single model takes like 12 minutes!

I'm not sure what to do and why they created a API like that for allow_migrate.

So... looking at the django source, I think you can just return True if the app has at least 1 sharded model in it and the model_name hint is not passed.
Even in dynamic usage, ALL django calls to the method have the model_name except during make_migrations and during any user defined RunPython or RunSql custom migration commands. We can easily document that you must pass a model_name if you use one of those migration functions.

So I think the check can be changed to something like:
if not hints.get('model_name') and ...code for checking the app has at least 1 sharded model here... :
    return True

It looks like all the makemigrations commands are looking for is a hard stop, a reason from the router that the migrations should not even be considered. For example, if there is a read-only model in the app, you would not want django to generate the migrations, the user should define them on their own (ideally, the user would put read-only models into their own app, but I think people not doing that are the reason for this check).
I would almost say to just return True if the model_name isn't defined no matter what. Thats basically the old behavior, correct? Since the method was never called, it was always just doing the migration?

Also, another option is to write our own makemigrations command that all it does is set an environment variable we can check, then calls super, then unsets the variable. Then any makemigrations would be caught and we could handle them.
class Command(MakemigrationCommand):
    os.environ["MAKING_MIGRATIONS] = "true"
    return_val = super(....)
    del os.environ["MAKING_MIGRATIONS"]
    return return_val

So... looking at the django source, I think you can just return True if the app has at least 1 sharded model in it and the model_name hint is not passed.

Even in dynamic usage, ALL django calls to the method have the model_name except during makemigrations and during any user defined RunPython or RunSql custom migration commands. We can easily document that you must pass a model_name if you use one of those migration functions.

Agreed and I did write that. However I don't want to return True in any case when I'm not sure. if anything I should return None. However, the issue is that when you make a data migration/RunPython command without it...I don't want it to run the migration successfully with no warning. How would you even know that you'd forgotten or where to look. I'm leaning toward overriding makemigrations for now.

I have one more PR to put in this evening that I would like to get put into the release, will open it up after I get some time to write unit tests.

Cool. :+1:
On Thu, Sep 8, 2016, 7:09 PM Titus Peterson<EMAIL_ADDRESS>wrote:

I have one more PR to put in this evening that I would like to get put
into the release, will open it up after I get some time to write unit tests.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/JBKahn/django-sharding/issues/40#issuecomment-245770286,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACiBvQefibMs-cwEki-gRS27eqZZVryAks5qoJXBgaJpZM4J3fpt
.

Alright, all the tests are passing on my project but I didn't get to this two days ago and I'm taking a few days off work. The day I get in I'm going to push it to our CI server and release it if I don't find anything.

:tada:

Released.


Reduce long methods

/src/objects/zcl_abapgit_object_tabl.clas.abap[349, 3] - Reduce method length, 126 statements
/src/objects/zcl_abapgit_object_tran.clas.abap[63, 3]  - Reduce method length, 166 statements
/src/objects/zcl_abapgit_object_tran.clas.abap[344, 3] - Reduce method length, 115 statements
/src/ui/zcl_abapgit_gui_router.clas.abap[178, 3]       - Reduce method length, 165 statements

zcl_abapgit_gui_router.clas.abap after https://github.com/larshp/abapGit/pull/2126 is merged


Update Configuration.php for deprecation fix SF5.0

Hi @nicolasricci , could you merge it? thanks

Any news about it ? I need it too !

@remigarcia @tifabien Sorry guys, not been using this bundle for a while and I don't really have time to maintain it at the minute. Would any (or both) of you be interested to be added to the repository to maintain it?

Sonata admin officially migrated to runroom-packages/sortable-behavior-bundle, try it instead.
https://symfony.com/bundles/SonataAdminBundle/current/cookbook/recipe_sortable_listing.html


The utils.uid is used as a nonce; it should be secure.

% grep -r uid lib
lib/middleware/csrf.js:    var token = req.session._csrf || (req.session._csrf = utils.uid(24));
lib/middleware/session.js:    req.sessionID = utils.uid(24);

even node's docs mention this as "pseudo random" so im not sure it would really make much of a difference

In my reading, Crypto.randomBytes goes to OpenSSL’s rand_bytes. It is still pseudo-random, but it is cryptographically strong, which is the important bit.

+1

thanks for the improvement!


Exclude Ironmen accounts from herb box notifications

This will do as the title states.
Fixes issue: #3307

#3307


[docs] URL Structure section is unreadable

I'm submitting a
(Check one with "x" from given options)

[x] Bug report
[ ] Feature request

Current behavior

Links with the issue:
https://github.com/Cloud-CV/EvalAI-ngx/blob/master/.github/CODE_STRUCTURE.md#url-structure
The content under sections Static Pages and Dynamic Pages is quite difficult to read.
Expected behavior

I think the data is supposed to be tables but doesn't appear so due to a minor typo.
Related screenshots

Other information

This might help: https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet#tables
P.S. I would like to make a quick fix for this.

@nikochiko Can i Work on this?

Yes of course! You did nice work on the PR!
Please reopen it! :+1:


Automator: update istio/api@master dependency in istio/istio@master

Generated by Automator - 2020-03-07T00:47:42+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
a92d03e886e73a91f35bd53e125f0a97658b40a5
link
/test gencheck_istio

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

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
875b848612a1e1593865088468d5693b107c88a9
link
/test gencheck_istio

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

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
f0adfdd3fe54bc64a6cd4375dd903b74f184c451
link
/test gencheck_istio

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

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
7b6eb6ae37e46f39546f9d433bdb56ef46105ec2
link
/test gencheck_istio

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


feat(ui): add opening date to channel details

Description:
Add channel opening date to channel details form (if it is available)

How Has This Been Tested?
Manually

Screenshots (if appropriate):

Types of changes:
Enhancement

Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage decreased (-0.03%) to 21.086% when pulling 892bc98321fab91f129aa0ed9c34b88cd3ad32c5 on korhaliv:feat/channel-opening-timestamp into 9ca7ae26be031c4e89f946ab67a04e910416c83f on LN-Zap:next.


chore(CI): enable SC and CAI APIs in cft-project-manager

Needed for https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/pull/2350

applied


[HOLD for payment 2023-06-07] [$1000] Bank Account - User goes to 'Connect manually' flow after 'Start Over' offline

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open App, create/open workspace and start to add bank account in workspace
Exits in the middle of the flow (after page 'Company Information' or 'Personal Information')
Return to the flow again > Settings > Workspaces > Connect bank account
Go offline
Select Start Over on the page 'Almost done!'
User get page page 'You appear to be offline'
Go Online

Expected Result:
The flow of adding bank Account is reset. User is on the start 'Connect bank account' page.
Actual Result:
The flow of adding Bank Account is reset. User goes to goes to 'Connect manually' flow.
Workaround:
Uknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: yes
Reproducible in production?: yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/236586987-b2257397-b798-4d2f-8d92-f17e3be4c0de.mp4

Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01aeed1cce3edf8b8d
Upwork Job ID:<PHONE_NUMBER>396211200
Last Price Increase: 2023-05-18

I'm dealing with a few things and I need to re-assign this. The auto-assigner hath chosen you @arielgreen - I'm sorry about the delay on this, but I appreciate you stepping in if you're able. <3

Happy to take this on @greg-schroeder ! Working on it tomorrow morning

@mananjadhav thoughts on the above proposal?

thanks for the proposals. I haven't had the chance to implement them but I think both of them can solve the problem. We agree the fetchData basically doesn't reset the flow. I think we should use @HezekielT's proposal because it looks straight forward. I also think it's good we're avoiding unnecessarily API call.
@hoangzinh I don't think it's a good idea to add an overhead of copying the step data just to paste it to failureData.
@flodnv All yours ^

Thanks for the review @mananjadhav. I actually think the UX in @HezekielT's proposal makes more sense than in @hoangzinh's proposal. Go for it @HezekielT

@HezekielT @mananjadhav please be sure to submit an application on the Upwork job here.

Offers sent in Upwork.

Reassigning, OOO tomorrow/Monday.

I have issued payment. @mananjadhav, do you believe the steps in the OP are sufficient for the regression test, or do we need something else?

I think it sounds good 👍

@mananjadhav Hmm, I thought the PR was merged in 7 days. Am I wrong? I'm happy to issue a bonus, I just thought it wasn't eligible.

@mananjadhav Hmm, I thought the PR was merged in 7 days. Am I wrong? I'm happy to issue a bonus, I just thought it wasn't eligible.

@johncschuster Because the checklist was completed within 3 days. Generally we focus on when the checklist is completed without further changes.

Got it. Thank you for clearing that up! I'll get those bonuses taken care of. While I'm working on that, can you finish out the BZ checklist above, @mananjadhav?
Thank you!

Thanks @johncschuster.
About the checklist yeah, I couldn't trace the offending PR. I saw this PR added the network offline check to fetch the data on switching the network. But I couldn't pinpoint if we did support offline behavior that time. The PR seems to be more than a year old.

Ok! Bonuses have been issued 🙌
Thanks for calling that out, @mananjadhav!
I'll get a regression test GH going shortly.

Regression test GH is live! I think we're good to close this one. 👍


Add a configuration option to control mock delay behavior

Currently, mock responses will be delayed when the time property in a HAR is set. This can be useful to simulate realistic API latency. We mainly use this library for automated tests however, and these delays could add a significant amount of time to the test run. I'd like to propose adding an option to control how responses are delayed:
responseDelay: real | none
Real: Default value. Responses will be delayed based on the time property in the HAR, if set.
None: Responses will not be delayed
This option could easily be a boolean but I was thinking that making it a string would help with backwards compatibility if there ever needs to be additional options (implicit delay for example). We'd only use none at the moment, however. Also I'm not married to the names in here if you'd prefer calling it something else
Curious what you think! Happy to submit a PR with the change if it sounds ok

Sorry, for the late response! I think it's a great idea 🤗

Don't know if we also want to accept a function that would allow to add a random delay on top of what is defined in the time-property? E.g. function (timeDelay: number) { return timeDelay * 2 }

Sorry for my late reply!
That's an interesting idea, are you thinking that the function could replace the real/none options? Or would it be a third option in addition to the other 2? Using a function is generic enough that you could just have it always return 0 to never sleep and if someone needs to do something fancy, it'd also support that. But at the same time, it might be easier to just set responseDelay to an explicit setting if you don't need the advanced functionality
Maybe the function could also take the request URL as an additional parameter in case someone wants to customize the behavior per route? Or do you think that's too much

Yeah, think that would be really cool. Maybe could pass it like some sort of context containing the requestInfo (e.g. path, method, etc). Only not sure if we only want to make it a function and then have a default implementation. What would you prefer?

Hmm might be easier to understand how to use it if there's non-function options, especially since I bet those would be the more common choices. What do you think about:
responseDelay: 'real' | 'none' | (context, timeDelay) => int

Sounds great!


Countries endpoint

make an endpoint called  /countries
create a country - POSt /countries
get all countries - GET /countries
get one country - GET /country/sl

nice


refactor(reports): refactoring reports

SUMMARY

This changes the reports feature to be a more compact and object oriented component. Before we had parts of alerts and reports residing in the headers of Dashboard and Explore respectively. We have moved this all to be inside two components, HeaderReportActions and the ReportModal, making the code more legible and easier to work with.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Dashboard Flow:
https://user-images.githubusercontent.com/48933336/150829066-a6518953-6aa4-43d1-b8dd-599e93a11a42.mov
Chart Flow:
https://user-images.githubusercontent.com/48933336/150829186-240e76a7-eccf-415f-810e-be4e36ff8ff5.mov
Error Message Mapping:
https://user-images.githubusercontent.com/48933336/152575928-8f806ef0-f164-44dc-8f15-b624272bc435.mov
TESTING INSTRUCTIONS

ADDITIONAL
INFORMATION

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

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

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

/testenv up FEATURE_ALERT_REPORTS=True

@AAfghahi this looks great. I'm trying not to get greedy with too many changes at once. I think we can merge this and maybe clean up a little bit on the final merge. The only thing that seems like perhaps a mistake that we may want to address in this PR is the comment here: return currentReport?.error;


Odm command redundancies

Description

Removed the redundancies on the ODM docker command line and updated documentation
Types of changes

Changes to Documentation
Adjusted the parameters in the ODM workflow recipe to remove redundant parameters
Checklist:

[x] I have read the documentation on CONTRIBUTING
[ ] My updates require a change to the documentation
[x] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] All new and existing tests passed

Time frame for completing Pull Request review (from creation date):

[x] As soon as possible
[ ] In the next two weeks
[ ] In the next month

Focus of Pull Request:

Reviewers should focus on the following aspects of this pull request. All comments are welcome regardless of which items are checked off.

[x] Inform others of new features and changes
[ ] There are no errors or inconsistencies
[ ] Everything appears clear and easy to understand
[ ] Suggest improvements
[ ] Identify areas where functionality and/or documentation is missing

Hints to reviewers:

@julianpistorius Please review README and recipe
@dlebauer Please review README
@jvanderleeuw Please review
Optional: issue links this Pull Request resolves

Nice! Yeah this solves two problems in one go. 💪


Add trace_methods option

Defines a syntax for matching methods which should be traced
closes elastic/apm-agent-java#347

Codecov Report

Merging #398 into master will increase coverage by 0.27%.
The diff coverage is 75.72%.

@@             Coverage Diff             @@
##             master    #398      +/-   ##
===========================================
+ Coverage     72.03%   72.3%   +0.27%
+ Complexity     1315    1303      -12
===========================================
  Files           141     143       +2
  Lines          5142    4962     -180
  Branches        532     505      -27
===========================================
- Hits           3704    3588     -116
+ Misses         1212    1148      -64
  Partials        226     226

Impacted Files
Coverage Δ
Complexity Δ

...tic/apm/agent/configuration/CoreConfiguration.java
95.52% <100%> (+0.24%)
16 <1> (+1)
:arrow_up:

...ic/apm/agent/bci/methodmatching/MethodMatcher.java
100% <100%> (ø)
17 <17> (?)

...apm/agent/bci/bytebuddy/CustomElementMatchers.java
92.85% <100%> (+0.85%)
7 <1> (+1)
:arrow_up:

...bci/methodmatching/TraceMethodInstrumentation.java
48.57% <48.57%> (ø)
8 <8> (?)

...ing/configuration/MethodMatcherValueConverter.java
60% <60%> (ø)
2 <2> (?)

...java/co/elastic/apm/agent/bci/ElasticApmAgent.java
73.04% <70%> (+0.37%)
22 <3> (+1)
:arrow_up:

.../co/elastic/apm/agent/matcher/WildcardMatcher.java
95.19% <80%> (-0.69%)
30 <2> (+3)

.../agent/plugin/api/AbstractSpanInstrumentation.java
53.19% <0%> (-2.06%)
3% <0%> (-4%)

...bci/bytebuddy/FailSafeDeclaredMethodsCompiler.java

... and 9 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 863822f...e4293fc. Read the comment docs.


Update for sz-sdk-python-abstract to sz-sdk-python name change

See https://github.com/senzing-garage/sz-sdk-python/issues/137

Pull requests:

https://github.com/senzing-garage/sz-sdk-python-core/pull/137


FEAT Have model cards

Thought it would be good to have model cards in HfPusher. We can firstly discuss here, implement and later open a PR to TFA.
I took a look at training script and thought it would be good to include:
(to-do)

[ ]  A template in case user feels like filling in later
[ ] Hyperparameters
[ ] Metrics
[ ] Plot of the model
[ ] A prepared text we can modify according to model configs (if user doesn't want to fill in later) (like @sayakpaul did with MAXIM models)
We can also push training data in a dataset repository and link it to the model.

We can also push tensorboard logs BTW if you'd like to add it to training script. You can see how we host them here: https://huggingface.co/keras-io/lowlight-enhance-mirnet/tensorboard
WDYT?

@merveenoyan thanks for initiating the conversation. Your proposal sounds solid to me.
Just for our context, https://huggingface.co/chansung/segmentation-training-pipeline/ - this is how the current repository looks like for the pushed model. The model is automatically pushed from a TFX pipeline (refer here and here).
Would be really good to include model cards here in the Hub model repository as huggingface_hub already provides a couple of neat utilities to do that.
It's also possible to customize the model card w.r.t the components mentioned by Merve (hyperparameters, metrics, dataset, etc.). A concrete example is here.
I believe the model card utilities should be implemented as a part of HFPusher().

@sayakpaul can you assign this to me? I'll create a tracker for model card parts above so we can handle it one step at a time and come up with something very fast.

@deep-diver could invite Merve to the repo? I will assign right away after it.

@sayakpaul
I just invited her as a collaborator to this repository!

About this example of using TensorBoard(https://huggingface.co/keras-io/lowlight-enhance-mirnet/tensorboard), can we automatically generate Model card based on the TensorBoard logs?

also, it is worth noting how TFX official ModelCardGenerator works and what it generates.

https://github.com/digits/tfx-addons/blob/add-model-card-component/examples/model_card_generator/MLMD_Model_Card_Toolkit_Demo.ipynb

@deep-diver normally the tensorboard logs need to be pushed. the logs in model card are regardless of tensorboard logs but rather depend on model history, so if we keep history we can write metrics to card 🙂 it gets a bit exhausting though when there are too many epochs so we should have it under a toggle IMO.
@deep-diver I was in the team building the model cards on Hub and I did look into the model cards toolkit itself and I find it a bit overkill for generating a simple markdown file. I think the model card they create includes nice evaluations for fairness but is too overwhelming, the model cards should be minimal-er compared to that one IMO. WDYT? @sayakpaul

We can respect the formats. I.e., if we're pushing to Hugging Face Hub, then I guess it's best to follow what's typically followed / suggested for the Hub model repositories as they've already gone through rigorous validation.
For Hub model cards here, I think it's best to combine the core eval metrics and whatever we get from history.

There are some things to note about TensorBoard.

The logs of TensorBoard does not represent the final performance. It comes with many logs including lots of experiments.

I think a model card should contain only the final performance measurement on a separate dataset.
in TFX, the model is trained through the normal training process in Trainer component, and it produces ModelRun artifact which points to where the TensorBoard logs are if we have specified the TensorBoard callback.
in this case, we could pass ModelRun artifact to the HFPusher component so that HFPusher could build a model card based on TensorBoard logs.

However, in TFX, the model is evaluated on Evaluator component whose sole purpose is to evaluate the trained model. It evaluates a model through many different eyes. For instance, how well does the model do on different classes instead of overall(average) performance?

TFX standard ModelCardGenerator creates a Model Card based on the output of Evaluator component. Like you said, it seems like overkill, but we could think it like we can take almost every reports based on our interests (may just need to filter). It also generates any types of output based on Jinja template engine such as Markdown or HTML, and we could even build our own template instead of the defaults ones.

I am just sharing what I know, so we could discuss and decide!

@merveenoyan, @sayakpaul
I got an idea for initial implementation.

first read TensorBoard logs [reference]
second create model card based one this using huggingface_hub utility package [reference]

I guess this is a good starting point, and we can research more about TensorFlow official model card toolkit and Evaluator component later. What do you guys think?

@deep-diver there's a very lightweight dependency called tabulate which we can use to turn TB logs in tables and host in model card. We can also keep it as CSV file separately if you wish.
Evaluator sounds good to me.


Update mitsi_lookup.py to include 0x00 for DIR

Updated DIR to include a 0x00 lookup for systems without vane.
Without this value MQTT doesn't connect/update as its got a None value in a field.

Thanks :)


Upgrade paperclip to version 0.4.1

Coverage increased (+0.02%) to 89.47% when pulling 5d321d3f67d01572b50bb30b7ea7d02ef62de4e2 on youtube into de00e6f779fe724f6234280362ee56b43c8adfe2 on develop.


Improve Update/Patch of DockerMachine

/kind feature
Describe the solution you'd like
We are using .Update to update DockerMachines and we should be using patch. One for the object itself and one for the status.
Anything else you would like to add:
Take a look here for inspiration:
https://github.com/kubernetes-sigs/cluster-api/blob/62582da077611649856cb2a88ec73c957e8569f5/pkg/controller/cluster/cluster_controller.go#L116-L138

/kind cleanup
/priority important-soon

Can you add information on why a Patch is needed for future reference.

Update can fail if the resource version changes. Patch (especially when leveraging the controller-runtime tooling) will only apply a diff and is less likely to conflict.

/assign
/lifecycle active


daemon: Reserve Geneve tunnel port if enabled

This is a follow-up suggested by Julian in a previous PR: https://github.com/cilium/cilium/pull/32128#discussion_r1591927219
Fixes: 11fe7cc144aa ("cilium-cni: Reserve ports that can conflict with transparent DNS proxy")
Note: This fix has already been applied to the backported version of the above commit.

/test

/ci-eks


Basic logging system

Hey,
Kadira Debug is really impressive and helps a lot to debug Meteor apps.
But to replace the good old-known Developer Tools, I miss a basic logging system to debug easier states from variables (or whatever). I played around a bit with the package and found a way to insert custom events and format them with HTML, but that solution is not so good and a real logging integration would still improve Kadira.
A simple implementation of a logging system might include multiple states (such as information, warning and error) and should not take too much time to develop. I hope you like my idea and I am sorry for my bad English :/

Okay. That means, you seems to like to have a custom logging functionality to Kadira Debug. We can simply track console log. But I prefer to our own API.
What do you think?

BTW: Kadira Debug is not to replace Chrome Dev Tools or similar. But another tool to add into your existing Dev Tools.

Yes, I would also prefer a custom API. You mentioned it should not replace the Dev Tools, that's a good point. - But I think Kadira Debug actually implements the most needed tools and it would be great to have to use only one tool in the most time.

Okay. Great. I'll add a logging API.
Right now, I'm planning for the launch and so, this will get bit delayed (end of this week like)

Ok that's great. Just take as much time as needed.


app server error and stop call any api

  flutter: Server Error: SocketException: Failed to create server socket (OS Error: Too many open files, errno = 24), address = localhost, port = 0 #0      _NativeSocket.bind (dart:io-patch/socket_patch.dart:1008:7)
<asynchronous suspension>
flutter: Server Error: SocketException: Failed to create server socket (OS Error: Too many open files, errno = 24), address = localhost, port = 0 #0      _NativeSocket.bind (dart:io-patch/socket_patch.dart:1008:7)
<asynchronous suspension>
flutter: Server Error: SocketException: Failed to create server socket (OS Error: Too many open files, errno = 24), address = localhost, port = 0 #0      _NativeSocket.bind (dart:io-patch/socket_patch.dart:1008:7)
<asynchronous suspension>
flutter: Server Error: SocketException: Failed host lookup: 'localhost' (OS Error: nodename nor servname provided, or not known, errno = 8) #0      _NativeSocket.lookup.<anonymous closure> (dart:io-patch/socket_patch.dart:511:9)
#1      _rootRunUnary (dart:async/zone.dart:1406:47)
<asynchronous suspension>
#2      _NativeSocket._resolveHost (dart:io-patch/socket_patch.dart:978:20)
<asynchronous suspension>
#3      _NativeSocket.bind (dart:io-patch/socket_patch.dart:992:21)
<asynchronous suspension>
flutter: Server Error: SocketException: Failed host lookup: 'localhost' (OS Error: nodename nor servname provided, or not known, errno = 8) #0      _NativeSocket.lookup.<anonymous closure> (dart:io-patch/socket_patch.dart:511:9)
#1      _rootRunUnary (dart:async/zone.dart:1406:47)
<asynchronous suspension>

[✓] Flutter (Channel stable, 3.7.6, on macOS 13.2.1 22D68 darwin-arm64, locale
    en-EG)
    • Flutter version 3.7.6 on channel stable at /Users/pc/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 12cb4eb7a0 (8 days ago), 2023-03-01 10:29:26 -0800
    • Engine revision ada363ee93
    • Dart version 2.19.3
    • DevTools version 2.20.1

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/pc/Library/Android/sdk
    • Platform android-33, build-tools 32.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      11.0.15+0-b2043.56-8887301)

[✓] VS Code (version 1.76.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.60.0

[✓] Connected device (3 available)
    • iPhone 14 (mobile) • 496138E0-442C-4EEB-AF72-8FEC07738EE8 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-16-2 (simulator)
    • macOS (desktop)    • macos                                • darwin-arm64
      • macOS 13.2.1 22D68 darwin-arm64
    • Chrome (web)       • chrome                               • web-javascript
      • Google Chrome 110.0.5481.177

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

i faced this issue when i using webview_flutter_plus in ios after load many javascript code using webview_flutter_plus app stopped

flutter: Server Error: SocketException: Failed to create server socket (OS Error: Too many open files, errno = 24), address = localhost, port = 0 #0      _NativeSocket.bind (dart:io-patch/socket_patch.dart:1008:7)

flutter: Server Error: SocketException: Failed to create server socket (OS Error: Too many open files, errno = 24), address = localhost, port = 0 #0      _NativeSocket.bind (dart:io-patch/socket_patch.dart:1008:7)

flutter: Server Error: SocketException: Failed to create server socket (OS Error: Too many open files, errno = 24), address = localhost, port = 0 #0      _NativeSocket.bind (dart:io-patch/socket_patch.dart:1008:7)

flutter: Server Error: SocketException: Failed host lookup: 'localhost' (OS Error: nodename nor servname provided, or not known, errno = 8) #0      _NativeSocket.lookup. (dart:io-patch/socket_patch.dart:511:9)
#1      _rootRunUnary (dart:async/zone.dart:1406:47)

#2      _NativeSocket._resolveHost (dart:io-patch/socket_patch.dart:978:20)

#3      _NativeSocket.bind (dart:io-patch/socket_patch.dart:992:21)

flutter: Server Error: SocketException: Failed host lookup: 'localhost' (OS Error: nodename nor servname provided, or not known, errno = 8) #0      _NativeSocket.lookup. (dart:io-patch/socket_patch.dart:511:9)
#1      _rootRunUnary (dart:async/zone.dart:1406:47)

Hi @abdelrahman-abied, this is an issue with webview_flutter_plus. I see a similar report of your issue in https://github.com/shah-xad/webview_flutter_plus/issues/43
Kindly report this to the owner of the plugin.
Closing as this is not a flutter issue.
Thank you


SAM-3298: Fix for Safari issue for multiple input elements in calcula…

…ted questions and numeric fill in the blank.
In Safari (up to and incl. version 11), whenever a user clicks in any of multiple input elements in the two question types, the first input gets focus. Users are thus forced to tab through the inputs to answer questions.
The fix follows the changes committed in SAM-3131, with minimal adjustments for calculated questions and numeric fib types.

I have opted to submit a PR against master instead of 11.x. I hope this does not inconvenience the maintainers.


This is testing 1

I comments on diz thing.and include a double quote "

I has a single quote: '.


State of GUI Repo

What's the current state of the GUI Repository, I notice that the last commit was about a year ago, what's the current status? Has the gui been abandoned? I'd be interested what the original push was in creating the gui and if it's still desired.

@Smarticles101 That is an excellent question.
I think that this was an experiment from before we knew we were going to redesign Exercism.
The impetus, I think, was that the CLI was difficult for some people to use.
Given that:

we've redesigned the CLI
added the interactive walkthrough to help people install it
not seen any further commits on this for a year

I'm going to go ahead and archive it.
For anyone interested in potentially picking this up, please open an issue in http://github.com/exercism/exercism/issues to discuss, as this now needs to be considered from a product perspective.
Thanks, everyone!


Issue with brackets when highlighted.

Issue Summary
Hello. I'm experiencing an issue with curly brackets when highlighted, some parts aren't visible. When not highlighted, the brackets are rendered correctly. This is mostly visible on expressions with 4 rows. Here is the link to a test sandbox code .
Steps to Reproduce:
Check the MathML sample code here then highlight the equation.
Technical details:

MathJax Version: 2.7.9 (https://cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.9/MathJax.js?config=TeX-MML-AM_CHTML)
Client OS: Mac OS Monterey 12.2.1
Browser: Chrome (99.0.4844.51), safari

Supporting information:

Here is a test sandbox https://codesandbox.io/s/mathjax-v2-forked-kg3i22?file=/index.html:430-517

MathJax uses multi-character assemblies when creating brackets and other stretchy delimiters when the size is larger than any of the single-character sizes that it has available.  It looks like the selection background for these are overlapping each other and obscuring the other pieces.
I don't think there is much that can be done about this.  While selecting the expressions is possible, it is not something that MathJax strives to support, as copying and pasting a layout like this is not generally very effective (what one usually needs is the underlying TeX or MathML notation, not the HTML output).

@dpvc thanks for the response. we don't need this for copying and pasting, however, we need the highlighting feature. Our organization is none profit and this feature (highlight) is available to million users of our app. We are interested if you can solve this issue or at least put it on your radar. Thank you

we need the highlighting feature

Does that mean you need users to be able to drag-select the math, or that you will be highlighting the math programmatically yourselves?  If the user selects, is it important to be able select sub-expressions, or is selecting the whole expression sufficient?  Will the user be selecting the math as part of a larger text?
The underlying problem is that the characters used for the assembly are taken from the font that has the largest sizes of the delimiters, so it has an unusually large ascent and descent (i.e., it is tall and deep).  The selection highlighting sets the background for the height and depth of the font, not just the characters involved in the assembly.  That means the selection highlighting is much larger than needed, and since the extender is made by multiple overlapping characters, that makes for even more extensive overlapping.  In the case of the braces, the character involved is actually rather small, and so many copies are needed.  Even though the highlighting is partly transparent, when that many overlap, they add up to being more solid.
There are a couple of approaches that can help, but not completely resolve the issue.  One is to use CSS to clip the excess selection background when possible.  For example
<style>
span.mjx-delim-v,
span.mjx-delim-v > span:nth-child(1),
span.mjx-delim-v > span:nth-child(3),
span.mjx-delim-v > span:nth-child(5) {
  overflow: clip
}
</style>

will help, as it reduces the overlap.  Unfortunately, the extender itself can't be clipped in this way, as that will prevent the pieces of the assembly from connecting properly.  But this does improve the selection somewhat.
Another possibility is to reduce the opacity of the selection for the multi-character delimiters.  Unfortunately, that also requires setting the selection color, and since that varies from operating system to operating system, you probably need to set the selection background color for the entire page in order to have it be consistent.  Something like
<style>
</style>

OOPS, accidentally submitted before I finished the comment.  I have edited it, so see the edited comment for the rest of it.

Does that mean you need users to be able to drag-select the math, or that you will be highlighting the math programmatically yourselves? If the user selects, is it important to be able select sub-expressions, or is selecting the whole expression sufficient? Will the user be selecting the math as part of a larger text?

We highlighting the math as part of a larger text programmatically with a text to speech feature we have.
The hacks seems to work but not the ideal approach.

We highlighting the math as part of a larger text programmatically with a text to speech feature we have.

OK, thanks.  Is the math highlighted all at once, or do you step through the different parts of the math individually?
If it is all at once, perhaps using something like
.in-selection .mjx-math {
  background-color: Highlight;
  user-select: none;
}

with applying class="in-selection" to the span with id=MathJax-Element-x-Frame (where x is a number) when you add it to the selection would work out better for you.  That takes extra work during the selection process, but the result is perhaps better than regular selection for the math.

OK, thanks. Is the math highlighted all at once, or do you step through the different parts of the math individually?

It's highlighted all at once.
Will try your suggestion and revert. Thanks once again


AWS CIS 3.3 Ensure the S3 bucket used to store CloudTrail logs is not publicly accessible

Description
Adds a policy for AWS CIS 3.3 Ensure the S3 bucket used to store CloudTrail logs is not publicly accessible
Policy has been tested and works as expected.

          1 Message

:book:
Checking security/aws/log_ensure_cloudtrail_bucket_not_public/log_ensure_cloudtrail_bucket_not_public.pt

  Generated by :no_entry_sign: Danger


Support http(s) url for docker client (#13871)

This allows to use http or https for connecting client to host (using DOCKER_HOST or -H). This should fixes #13871, although we might want to implement the server part of this (beeing able to start the daemon with http(s) ).
$ docker -H http://<IP_ADDRESS>:80 info
$ docker -H http://<IP_ADDRESS>/some/path info
$ DOCKER_HOST="http://<IP_ADDRESS>:80" docker info
$ DOCKER_HOST="https://domain.tld/swarm" docker info

I'm not really happy about how I implemented this, but this will do fine to discuss the design first.
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>

I'll close this one as it's kinda a duplicate for #14304.


(#216) Move jenkins credential logic to helpers

Description Of Changes
This PR creates several new helper functions which cleans up the logic in the Jenkins setup step of the guide, and gives us greater control over what happens.
Motivation and Context
We encountered an edge case where the credential written to the README differed from the initial jenkins config file.
Testing
N/A at this time.
Operating Systems Testing
N/A at this time.
Change Types Made

[x] Bug fix (non-breaking change).
[ ] Feature / Enhancement (non-breaking change).
[ ] Breaking change (fix or feature that could cause existing functionality to change).
[ ] Documentation changes.
[x] PowerShell code changes.

Change Checklist

[ ] Requires a change to the documentation.
[ ] Documentation has been updated.
[ ] Tests to cover my changes, have been added.
[ ] All new and existing tests passed?
[ ] PowerShell code changes: PowerShell v2 compatibility checked?

Related Issue
Fixes #216

@JPRuskin it's impossible to say what happened, but moving these out to helpers will allow support to "fix it" relatively easily.


Setup Github API OAuth method to send requests(#1)

Why is this change necessary?
By setting OAuth we do not need to manage our authentication system. We can rely on OAuth to do the authentication.
How does this commit message address this issue?
I have generated clientId & clientSecret for GitPub. Using this we create an authorization with the scope- public_repo and call POST /authorizations. In response we get the oauth token. This token will serve as a credetial to make user related API requests.
What affects does this change have?
OAuth provides Gitpub application with secure delegated access. It works over HTTPS and authorizes applications and APIs with access tokens rather than credentials.

The checks fail because .pyc file is in .gitignore. So settings.pyc cannot be tracked. I will go ahead and add environment variables and give a commit soon.

@abiduzz420 - Even if they are in .gitignore, you can commit them by explicitly adding them. Btw this was just for your information, not saying you to include it :stuck_out_tongue_winking_eye:

Yeah I understand, but again I'll have to remove it. Hence I did not go and include it in tracked files :)

Cool, I will go through your PR and update you by tomorrow.

@Demfier I would like to know if we need to generate an auth token for every new user who uses the library ? Because to generate auth token for a user, he must enter his password. I'm not sure if we can get away with that ?
Yeah sure @Demfier I shall wait for feedback and modify changes accordingly.

Okay, so lets boil it down.
Firstly, oauth tokens are pretty important as they save a lot of time and give extra security. So every developer MUST have an oauth-token in the first place. Now lets look at what kind of situations we might run into:

Dev has an OAuth-token: This would be the so-called best-case scenario for us. So all the dev has to do is to setup the env variables and we would automatically detect it. No pain to the user of any sort.

Dev doesn't have an OAuth-token: Now 2 cases arise here. Either he can generate tokens from -
a) Github UI - So he will open his github account, generate OAuth-tokens and put them in an env variable. Then the rest goes as in 1.
b) GitPub - This would be the worst-case scenario for us. Here we WOULD have to take his username and password in order to generate tokens from him. After all, we are essentially trying to bring github functionalities in one library. It shouldn't be a problem because we are not storing those sensitive information in any of our personal database (and this library is open-source, just in case someone has any doubts). They are stored in the user's computer itself. Completely safe.
So now, if a dev trusts us he/she would simply follow 2b but if its otherwise, he/she would choose 2a.

As for us, it is our job to complete things from our end, rest we can leave upto the user.
Also, the tests are failing because dotenv isn't installed. You'll have to update .travis.yml to include its install command.
I hope I was able to answer your queries. Let me know if you face any more issues.
Cheers :smile: !

Thanks a lot @Demfier for the review. So

I will set generate auth token for dev which all the users can use in .env file
Allow user to set their own auth token in .env (with documentation)
I am not very proficient in python, so this project could be help me get a bit more experience :)
I would like to learn about continuous integration tools like TravisCI or CircleCI and how to setup in a project. I will hopefully learn in coming weeks.

Thanks @Demfier again! 😄

@Demfier Once we add OAuthToken of a user, we can fetch only his user's details. Since I'm using this end point GET /user. See
Is this working as expected or am I doing something wrong ? 🤔

@abiduzz420 - Sorry for the delayed response.
Firstly, I'll add a third kind of dev group to your knowledge:
3. Devs who don't wish to use OAuth - So all info for them would be fetched from public end-points.
Regarding your query:

This is working as expected. You can only access your details in github too right? And the end-point to access other users' details is /users/{username}. Basically, OAuth tokens are used for managing resources in your personal account. The way of retrieving rest of the public information remains the same.

Visit Github Marketplace to about continuous integration systems. Basically, they need a config file like .travis.yml where we put in all the commands to be tested in order and those commands are run over a server.

Coming to your PR, you should add:

Provision for optional passwords/oauth-tokens - Seems like everytime the dev makes a Profile object, he will have to put in the username and password. See, OAuth tokens are one-time things (unless the dev wishes to explicitly change it and why should he unless his account is compromised?). So after the 2 kind of devs have their token, it should be stored in the .env file, shifting them to the  category 1 developers. Now for these devs, we don't need the password anymore and should use OAuth-token automatically from the .env file. Remember to enumerate all the different possible situations while implementing.

Tests - Very very crucial. They will save us from a lot of trouble later. Since this is not a simple Doc change, you will need to modify the existing tests and add some of your own to ensure robustness of the functionality you are adding. Also, things like the typo that I figured out in your PR would've been very obvious if there was a test for it.

Thanks for your response @Demfier
Firstly I have removed the code where the user has to authorize every time he creates a Profile object. The user should generate an OAuth token and store it in .env file. Point number 1 is solved.
Coming to tests, yes it's true that tests are crucial. I shall write tests wherever necessary as a part of the PR. Need to ponder over what exactly should I test.

I shall learn about Travis CI. Thanks for pointing out to Github Marketplace.

Travis CI tests are failing because we have test user as Demfier. I will add my user name in place of it.

@abiduzz420 - You can modify the tests to incorporate the changes. See - there will different functions to test each of the cases (henceforth, will be referred as dev-paths)

Hi @Demfier Sorry I wasn't able to make much time due to my exams. I shall soon start working on the comments you have made


implement support for multiple plans with associated credentials

Given env variables definining multiple plans in format SERVICES_SID_PLAN_PID_NAME
SERVICES_API_DIRECTORY_NAME: MyService
SERVICES_API_DIRECTORY_PLAN_1_NAME: dev
SERVICES_API_DIRECTORY_PLAN_1_CREDENTIALS_URI: http://mydev-api.org
SERVICES_API_DIRECTORY_PLAN_1_CREDENTIALS_ACCESS_KEY: AZERT23456664DFDSFSDFDSF
SERVICES_API_DIRECTORY_PLAN_2_NAME: preprod
SERVICES_API_DIRECTORY_PLAN_2_CREDENTIALS_URI: http://mypreprod-api.org
SERVICES_API_DIRECTORY_PLAN_2_CREDENTIALS_ACCESS_KEY: AZERT23456664DFDSFSDFDSF
SERVICES_API_DIRECTORY_PLAN_3_NAME: prod
SERVICES_API_DIRECTORY_PLAN_3_CREDENTIALS_URI: http://myprod-api.org
SERVICES_API_DIRECTORY_PLAN_3_CREDENTIALS_ACCESS_KEY: AZERT23456664DFDSFSDFDSF

Then the marketplace will be registered with a service with 3 plans: dev, preprod, prod
Specific credentials for each plans are returned

fixed in commits:
bf32511ad84816cbebbd31e2b81e0f9051e331bf
49acd364cc6cf781d891f0bcb2ae24eab271688f
64677034283383b4013915f4e0f7a81ba9a42d0f
Note: both credentials definitions per service (ex. SERVICES_API_DIRECTORY_CREDENTIALS_URI) and per plan (ex. SERVICES_API_DIRECTORY_PLAN_1_CREDENTIALS_URI) is supported.
If there are conflicts keys defined both in service credentials and plan credentials, the value of plan credential will be taken.

Question: whether the support of updateable plan is necessary ?

@tao-xinxiu No I don't see use-cases that would require the ability for a static-creds service instance to be updated to switch between plans ( cf update-service -p NEW_PLAN). Let's defer this if that gets required by users.


Big refactor, API incompatible changes

Hey @jedbrown, we've been using git-fat at our company and we love it!
We've made a load of changes to suit our needs and some of those changes make it incompatible with yours.
I've also packaged it as a pypi package to make it easy to install (although it has no real usefulness as a module alone).
I don't expect you to merge it because of the API changes, but we wanted to let you know that we love what you've done and to give back what changes we made.

I see there was a release on PyPi all the way to 0.5.0 (https://github.com/cyaninc/git-fat/commit/a25555b1204db2a443542f1f825f4e3fc349ea01) and the network activity on this package is quite active.
There is a plan to consolidate and bring all the cool stuff into the fold with the blessing of @jedbrown or you guys have different plans?
Best and thanks for this neat module

Hi @mattions I've actually been lacking the time to properly maintain the cyaninc fork unfortunately.  I'm not sure what would have been needed to get it back into the mainline here, but I think it'll probably just sit the way it is.

At this point, the projects have diverged enough that it doesn't really make sense for them to keep using the same name.  @jedbrown - What do you think?

@ahmadia It appears that neither @abraithwaite nor myself have time to integrate the two forks in the near future. In that case, I think it's important to communicate to users that the two forks are not compatible. Names are the most common way to do that, but at this point would require a name change that may be even more confusing to users, thus I don't have a strong preference.


⚠ Bofrost.lu has degraded performance

In 7afcd08, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1679 ms

Resolved: Bofrost.lu performance has improved in cbbb92d after 7 minutes.


Server bound <IP_ADDRESS>:4499

Starting the server does not pull correct ip information. Just reads <IP_ADDRESS>

This is normal. If you enable AnnounceSelf in Settings.xml it will show your IP
Your public IP is x.x.x.x, announcing to master server...

Starting the server does not pull correct ip information. Just reads <IP_ADDRESS>

@oldnapalm was right, <IP_ADDRESS> means it's listening on all network interfaces (Wi-Fi, Ethernet...)
It will fetch your public IP from ipinfo.io if AnnounceSelf is set to true(but your server will not be reachable unless the port has been forwarded). If you want to know your private address, type ipconfig in command prompt to see it

Thanks everyone.  I did get it
working.
On Sun, Jul 24, 2022, 11:35 AM Sardelka9515 @.***>
wrote:

Starting the server does not pull correct ip information. Just reads
<IP_ADDRESS>
@oldnapalm https://github.com/oldnapalm was right, <IP_ADDRESS> means it's
listening on all network interfaces (Wi-Fi, Ethernet...)
It will fetch your public IP from ipinfo.io if AnnounceSelf is set to
true(but your server will not be reachable unless the port has been
forwarded). If you want to know your private address, type ipconfig in
command prompt to see it
—
Reply to this email directly, view it on GitHub
https://github.com/RAGECOOP/RAGECOOP-V/issues/30#issuecomment-1193341782,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE6MRSH7LMNWGBCZ2MBP6FTVVVPELANCNFSM54MRPBXQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fix padding for search results

See https://sphinx-readable-theme.readthedocs.org/en/latest/search.html?q=readable&check_keywords=yes&area=default

Thanks for the change and for using my theme. :)


update sample datasets for new column name

Title: Update sample datasets for new relation_to_reference_person column name
Description

Category: feature
JIRA issue: ???

RT requested that the column "relation_to_household_head" be changed to
"relation_to_reference_person". Rajan previously updated the PRL side of things
and re-ran the sim. This PR updates the sample data and related pseudopeople
code.
Testing
All tests pass

Closing b/c Rajan was already working on this.


Add a -stop-staking option to set-staking CLI

This PR adds a -stop-staking option to the client set-staking command. This allows users to turn off staking from the CLI.
I have been using this to control when block production happens when testing snapp commands, so that I don't need to wait for an empty block between executing commands.
Sample output of -help and invalid combinations:
$ _build/default/src/app/cli/src/coda.exe client set-staking -help
Start producing blocks

  coda.exe client set-staking

=== flags ===

  [-public-key PUBLICKEY]            Public key of account with which to produce
                                     blocks
  [-rest-server URI/LOCALHOST-PORT]  graphql rest server for daemon interaction
                                     (examples: 3085 or
                                     http://localhost:3085/graphql,
                                     /dns4/peer1-rising-phoenix.o1test.net:3085/graphql)
                                     (default: 3085 or
                                     http://localhost:3085/graphql)
  [-stop-staking]                    Stop producing blocks
  [-help]                            print this help text and exit
                                     (alias: -?)

$ _build/default/src/app/cli/src/coda.exe client set-staking
Error parsing command line.  Run with -help for usage information.
Must pass one of these: -public-key; -stop-staking
$ _build/default/src/app/cli/src/coda.exe client set-staking -public-key B62qrPN5Y5yq8kGE3FbVKbGTdTAJNdtNtB5sNVpxyRwWGcDEhpMzc8g -stop-staking
Error parsing command line.  Run with -help for usage information.
Cannot pass both -stop-staking and -public-key

Checklist:

[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them:

this sounds good to me, as long as we make it clear how to do so in the docs!

unset-staking ?

Suggestion: The set-snark-worker command disables snark working when no public key is passed to it. We probably should change that command or this to make the stopping feature consistent. (create an issue?)

I've updated this to implement Deepthi's proposal, for consistency with set-snark-worker; this now accepts 0, 1, or many --public-key flags, and running the command replaces the set of staking pks with exactly those passed to the command.
@lk86 does this seem okay to you, or should we still have a separate command?

There's clearly no demand for this, closing.


Is it possible to use event.preventDefault() when executing onSubmit?

When the onSubmit function is executed, the page reloads, preventing the API call within the onSubmit event.
I want to prevent the page from reloading when onSubmit is executed.
In a standard form's onSubmit, executing event.preventDefault() can prevent the page from reloading.
I would like to achieve the same behavior.
The current code is as follows:
<script lang="ts">
  const { form, errors, constraints, enhance } = superForm(dataForm, {
    resetForm: true,
    onSubmit: async ({ formData }) => {
      const entries = formData.entries();
      const data = Object.fromEntries(entries);
      const res = await fetch('/api/data', {
        method: 'POST',
	body: JSON.stringify(data)
      });
  ...
</script>

<form id="form" use:enhance>
...

Originally, I was using Form actions, but I wanted to enable prerender, so I switched to using API Routes instead.

You can deconstruct cancel and use that, though in your case it looks like your should use SPA mode instead: https://superforms.rocks/concepts/spa

@ciscoheat
Resolved! Thanks!!


Specified D part limits data extents

When calling a TS like //BRWM5/ELEV/01Oct1995/1Hour/DCP-REV/ the data is limited to Oct1995. Jython DBAPI would ignore the D part and return the full dataset.

Our next release should address and fix this issue.


Default anchor Point objects

I have added these "defines" to the Phaser namespace myself:
Phaser.ANCHOR_TOP_LEFT = new Phaser.Point(0, 0);
Phaser.ANCHOR_TOP = new Phaser.Point(0.5, 0);
Phaser.ANCHOR_TOP_RIGHT = new Phaser.Point(1, 0);
Phaser.ANCHOR_LEFT = new Phaser.Point(0, 0.5);
Phaser.ANCHOR_CENTER = new Phaser.Point(0.5, 0.5);
Phaser.ANCHOR_RIGHT = new Phaser.Point(1, 0.5);
Phaser.ANCHOR_BOTTOM_LEFT = new Phaser.Point(0, 1);
Phaser.ANCHOR_BOTTOM = new Phaser.Point(0.5, 1);
Phaser.ANCHOR_BOTTOM_RIGHT = new Phaser.Point(1, 1);

It seems to me that they would be helpful to everyone.

This feature request has been moved to the Phaser 3 feature list (so is being closed here)


Please add bluetooth support for android

Many people are using android phones to print receipt.

Hi @pjjy, hi @glennmichaelmejias
Does the library work well for you?
I've created a list of printers tested with this library.
May I ask you to add your printer model into the table?
It will help me to maintain the library and others to choose the right printer!
Thank you!


Set default cursor image in rootston

Test plan: setup two seats, and use this config:
[cursor:seat1]
default-image = crosshair

👍


#739 Choice-VR optional

Fixes #739
This PR is ready for review.
Risk
This PR makes no API changes.
Summary
This PR will allow app to create a choice without VrCommands.
CLA

[ ] I have signed the CLA

Codecov Report

Merging #750 into develop will decrease coverage by 0.06%.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             develop     #750      +/-   ##
=============================================
- Coverage      41.01%   40.95%   -0.07%
  Complexity      2643     2643
=============================================
  Files            362      362
  Lines          15853    15857       +4
  Branches        1695     1695
=============================================
- Hits            6502     6494       -8
- Misses          9044     9058      +14
+ Partials         307      305       -2

Impacted Files
Coverage Δ
Complexity Δ

...n/java/com/smartdevicelink/proxy/SdlProxyBase.java
5.35% <0%> (-0.01%)
6 <0> (ø)

...smartdevicelink/encoder/VirtualDisplayEncoder.java
24.71% <0%> (-4.6%)
8% <0%> (ø)

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 e0f940b...3b0e229. Read the comment docs.


Add tests for ensuring owner references on Machines targeted by the ControlPlaneMachineSet

This is currently based on #19 and as such, only the last 2 commits need to be reviewed for this PR.
When we have gathered information about the Machines, one part of the operation of the operator is to ensure the owner references on the Machines.
Since this can be done generically (only interacting with ObjectMeta) this part of the logic is in the core of the ControlPlaneMachineSet reconciler and not a part of the Machine provider itself.
I have added tests to define the logic of the ensureOwnerReferences function but the remaining logic needs to be implemented still (until the tests pass)

/retest-required

/retest-required

/override ci/prow/e2e-aws-disruptive
/override ci/prow/e2e-aws-serial
/override ci/prow/e2e-aws-upgrade
This isn't in payload yet and so these tests aren't actually testing anything


x/net/bpf: implement interface fmt.Stringer on BPF instructions

What version of Go are you using (go version)?
go version go1.7.4 linux/amd64
What operating system and processor architecture are you using (go env)?
linux, amd64
What did you do?
I would like to print bpf.Instructions in the assembler notation defined in https://www.kernel.org/doc/Documentation/networking/filter.txt.
What did you expect to see?
I suggest to implement fmt.Stringer an all types which implement bpf.Instruction.
For example the instructions
[]bpf.Instruction{
	bpf.LoadExtension{Num: bpf.ExtRand},
	bpf.JumpIf{Cond: bpf.JumpGreaterThan, Val: 4294967, SkipTrue: 1},
	bpf.RetConstant{Val: 1024},
	bpf.RetConstant{Val:0},
}

could be printed as
ld #rand
jgt #4294967, 1
ret #1024
ret #0

I have an implementation for the output in assembler notation (without fmt.Stringer): https://github.com/breml/bpfutils/blob/master/string.go#L26-L218 and I offer to provide a CL for the above suggestion, if this change is accepted.
What did you see instead?
Is not implemented.

/cc @danderson @mikioh

This seems fine, and a CL would be welcome as far as I'm concerned (though I don't have direct commit access, @mikioh has the final say there).
The only thing I'm worried about is what to do about RawInstruction. By design, RawInstruction is also an Instruction, so that programmers can specify custom opcodes for forks of the BPF machine. Those instructions might not match a known instruction of the Linux BPF implementation, so... What does String() return for that?
I suppose the simplest option is simply to not implement fmt.Stringer on RawInstruction. Does that seem reasonable to you?

It is not my intend to extend the existing bpf.Instruction interface but simply implement the fmt.Stringer interface on the types (like bpf.RetConstant) where it makes sense (respectively where the assembler instruction is defined).
So I fully agree with the suggestion by @danderson to not implement fmt.Stringer on RawInstruction.

I'll push the submit button once @danderson is happy.

@breml's plan sounds great to me. Is there a CL already? I'll happily review one.

@danderson there is not yet a CL. The implementation is straight forward. Currently I am looking for the best way to implement the tests.
One possibility would be to add a go style table driven test which includes for every bpf.Instruction the expected string.
An other possibility would be to use the already existing testdata and validate, that for all instructions in testdata/all_instructions.bpf the respective line in testdata/all_instructions.txt is generated. The downside of this approach is, the instructions in testdata/all_instructions.txt do need some preprocessing (remove all lines with comments, remove empty lines a and most importantly resolve the jump labels to the correct relativ jump distance).
@danderson what do you suggest?

@danderson Just uploaded a CL, which uses option 1 for the tests (see my previous comment).


Fix token generation for text > 0x10000

Also minor optimizations.
Python does not natively support Unicode surrogates, which are used when calculating the token for JS.
In addition, the loop was fixed to follow the value g instead of the static size, as the number of iterations is not necessarily the same as the length of the string.

Coverage decreased (-0.4%) to 98.381% when pulling aac44b619118b60a2a31fdace67633f3242ac6d3 on Roadcrosser:master into 48653332effe76d0a034a730abdba1e96cd56d8a on ssut:master.


Fix GO-2022-0586

Malicious HTTP responses can cause a number of misbehaviors,
including overwriting local files, resource exhaustion, and
panics. * Protocol switching, endless redirect, and
configuration bypass are possible through abuse of custom HTTP
response header processing. * Arbitrary host access is possible
through go-getter path traversal, symlink processing, and
command injection flaws. * Asymmetric resource exhaustion can
occur when go-getter processes malicious HTTP responses. * A
panic can be triggered when go-getter processed
password-protected ZIP files.
Found in<EMAIL_ADDRESS>Fixed in<EMAIL_ADDRESS>More info: https://pkg.go.dev/vuln/GO-2022-0586
Signed-off-by: Itxaka<EMAIL_ADDRESS>

@davidcassany just released govulncheck https://go.dev/blog/vuln :)


Opened is false Still sending massload ajax

Hi,
I am using jstree v3.3.6. Here is my  Code
loadData(configs){
	var _self = this;
  var timeout = null;
  var jstree = this.$selector.jstree({
    "expand_selected_onload": false,
    "plugins" : ["massload","state","checkbox"],
    'core' : {
      "themes" : {
        "dots" : false
      },
      'data' : {
        "url" : function (node) {
          var url = configs.dataUrl;
          if (node.id != "#"){
            url += "?node_id="+node.id;
          }
          return url;
        },
        "dataType" : "json"
      }
    },
    "checkbox" : {
    },
    "massload": {
      url: 'hierarchy/massload?model_name=Classification',
      "data": function (ids) {
        return { "ids": ids.join(',') };
      }
    }
  }).bind("loaded.jstree", function (event, data) {
  	var insertHTML = '<input type="checkbox" name="checkAll" id="checkAll" /><label for="checkAll" id="checkAllLabel">SELECT ALL</label>';
      $(this).prepend(insertHTML);
      _self.setSelectAllCheckboxforFlexAttribute();

   // this.jstree(true).check_node(nodeids);

	}).bind("open_node.jstree", function(event, data){
  	//_self.setSelectAllCheckboxforFlexAttribute();
 	});
}

What i am trying to achieve is, in my tree view if user clicks on parent node then all parent and its data must be selected and save to db then after when user open bootstrap modal same state of tree view should be loaded. I have try this to be working with tie_selection: false and manage checkbox state my own. But still it is sending xhr request if state is "checked: false" OR "selected: false" (according to tie_selection". Am i doing something wrong?

Sorry, but I do not understand the issue. If massload is configured it will be used. As for the custom logic you are trying to implement - I can not help you with that, all I can say is that the config seems fine.
Please clarify if I misunderstood and there is an issue with jstree itself and I will reopen the issue.


Clean latest api doc from PortMapping and outdated error check

Regarding the outdated error check, there's no docker.PortMapping
struct anymore and this is linked to something really old #1334
Somehow I missed these, my b
Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

LGTM

LGTM


Improved proposal template

Added breaking consensus instructions.
Restructured to include steps for execution
Improved links

ok, let's try this


Fill url when retrieving test cases

bors r+

Codecov Report

Merging #6 into master will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master      #6      +/-   ##
=========================================
- Coverage    0.63%   0.62%   -0.02%
=========================================
  Files          20      20
  Lines        2210    2246      +36
  Branches     1041    1072      +31
=========================================
  Hits           14      14
- Misses       2171    2207      +36
  Partials       25      25

Impacted Files
Coverage Δ

src/commands/open.rs
0.00% <0.00%> (ø)

src/commands/retrieve_testcases.rs
0.00% <0.00%> (ø)

src/project.rs
3.46% <0.00%> (ø)

src/web/retrieve_testcases.rs
0.00% <0.00%> (ø)

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 e4440bb...513e162. Read the comment docs.

bors r+


feat: add a tmp dir for adding dependency to reduce the impact of failed downloads

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[x] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

14 of 22 (63.64%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 44.308%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/client/client.go
14
22
63.64%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
2569

Relevant Lines:
5798

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

15 of 28 (53.57%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 44.28%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/client/client.go
15
28
53.57%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
2570

Relevant Lines:
5804

💛  - Coveralls


Immediate disconnected(7) after running /honeywell

As soon as I run honeywell it shows a disconnect mqtt message. I tried running with and without a username and password and it's the same problem. I've tested my mqtt broker with another client and there are no issues there.

What broker are you using?  I've almost exclusively used Mosquitto.  Does it support MQTT v3.1.1?

I'm using mosquitto as well.
https://www.home-assistant.io/addons/mosquitto/

I have this same problem.  Did you find a solution?

I never found a solution. How did you fix it?

Encountering the same problem.  Does anyone know how to resolve this?  Running Mosquitto w/ MQTT Ver. 3.1.1 and credentials are confirmed working with other applications.

I never found a solution. How did you fix it?

Figured it out!
If your MQTT Broker requires username/password authentication, add the reference line below to your mqtt_config.h file, in Line 2, of the mqtt.cpp file, then rebuild.
#include "mqtt_config.h"
Step #1 - nano mqtt.cpp
Step #2 - Add to Line 2, #include "mqtt_config.h"
Step #3 - Rebuild using the command, ./build.sh
You should then be able to immediately connect to your MQTT Broker with username/password authentication.

Do you need to do this even if using anonymous auth?

No.  This is only if your MQTT Broker requires username/password.


[PostgreSQL] Comment is lost when adding new column to a table

I found a bug: When I add a new column to an existing table and provide a comment (description) for this column then DBeaver does not save the comment nor does it generate any DDL for it. After adding a column with a comment and hitting "Save" I am presented with DDL like this:
ALTER TABLE public.my_table ADD col_name bool NOT NULL DEFAULT false ;
The comment DDL is missing and when I click "Persist" the column is added and the comment I have entered disappears from the view. In order to add the comment I have to enter it again to a column that already exists and only then I get the proper DDL like this:
COMMENT ON COLUMN public.my_table.col_name IS 'my comment' ;

Thanks for report

great!


Low sensitivity mode

Please add limitation notification by magnitude or seismic-intensity function.

🎉 https://github.com/Team-Fruit/EEWBot/releases/tag/2.0.8


Add a changelog entry

:notebook_with_decorative_cover: Description
Fixing an oversight of #1446.
:memo: Checklist

[x] All user-facing changes have changelog entries.

FYI I used the administrative override to merge this one as changes only to the changelog file apparently do not trigger all required checks. We should fix that separately.


CI: Tidy build output

By removing extra, unnecessary output that makes it
difficult to inspect the build log.

Before:
$ curl https://storage.googleapis.com/maistra-prow-testing/pr-logs/pull/maistra_proxy/68/proxy_2.0-unit/1333415764811583488/build-log.txt | wc -l
153071

After:
$ curl https://storage.googleapis.com/maistra-prow-testing/pr-logs/pull/maistra_proxy/69/proxy_2.0-unit/1333471381630750720/build-log.txt | wc -l
2999

Before:
$ curl https://storage.googleapis.com/maistra-prow-testing/pr-logs/pull/maistra_proxy/68/proxy_2.0-unit/1333415764811583488/build-log.txt | wc -l
153071

After:
$ curl https://storage.googleapis.com/maistra-prow-testing/pr-logs/pull/maistra_proxy/69/proxy_2.0-unit/1333471381630750720/build-log.txt | wc -l
2999

That comparison isn't really fair as this PR's build happened with a warm cache and no code changes

@dgn This has nothing to do with build time, but with the size of the build log. This PR removes clutter from logs that happen regardless of cache usage.

@dgn This has nothing to do with build time, but with the size of the build log. This PR removes clutter from logs that happen regardless of cache usage.

+1 but with a warm cache you won't get a lot of those log lines afaict

Nevermind you're right, still 153k lines on a warm cache

The amount of stuff being logged can be reduced (definitely the items in OUTPUT_TO_IGNORE) by removing sandbox_debug option, here: https://github.com/maistra/proxy/blob/maistra-2.0/maistra/bazelrc#L22.

@dmitri-d  Even without that flag I still see clutter in the logs. I'll open another PR so that we can see it in practice and compare.


patch: Updating screencaps in Account Management

Updated screencaps from the Access Tokens in Account Management.
Need to check the widths of the screencaps once on production with different browsers.

@mpous I have deployed your branch to staging. This way you can check your changes before they hit prod.

LGTM, merge this PR by issuing the I self-certify! command to approve the PR.
Reference: https://github.com/balena-io/docs/blob/master/CONTRIBUTING.md#balenistas-only-using-versionbot-to-manage-your-pull-request

@balena-ci I self-certify!

@mpous To get your branch updated with the master branch, one should rebase using versionbot. Having a merge commit when you update your branch using the GitHub UI often adds up to noise in the history and may have an undesirable effect. You can instead make use of Versionbot to rebase your branch. Check out the command @balena-ci rebase https://github.com/balena-io/docs/blob/master/CONTRIBUTING.md#balenistas-only-using-versionbot-to-manage-your-pull-request
Though it doesn't work when someone has already used the Github UI to update the PR.


bind tcp-server

tcp-server and tls-encrypted tcp-server should be able to bind to a specific subnet or network interface.

moved to https://github.com/kitsudaiki/Hanami-AI/issues/38


Add reference to One Observability workshop

Issue #, if available:
Description of changes:
Added references to our newly launched One Observability workshop. This will help users to know about the exhaustive set of AWS observability features using CloudWatch and X-Ray
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hello! I don't see a reciprocal link on your side. Did I miss that?

I can certainly add that to my workshop.

Just added it to my workshop. Should be deployed in a few minutes. You will see it here - https://observability.workshop.aws/containerinsights/eks.html


[camera_web] Update camera_web to support Dart's new async requestFullscreen API

No version change: No change to plugin functionality

fyi the publishable step won't pick up the change in a commit message without pushing to the branch

test-exempt: refactor with no semantic change

Please do add a changelog entry since you are changing code; it makes it easier to pinpoint issues if there's some unintended behavioral change.

Updated packages/camera/camera_web/CHANGELOG.md

/cc @bselwe FYI!


Typo "@redirected_publisher"→"@redirected_publisher"

Italic with escape characters
https://docs.microsoft.com/en-us/sql/relational-databases/system-stored-procedures/sp-validate-replica-hosts-as-publishers-transact-sql?view=sql-server-2017

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Correction confirmed. Thanks.


User config

Based off #5

In general this looks good to me.
My one suggestion would be to add more notices to pre-deinstall for removing directories. I see /var/run/<pkg_name> after removing the package. Maybe also the log file?


README: fix errors and add license link

Example for call method is moved to the right place

Thanks for your contribution


All examples should include pagination

To simplify usage, all provided code samples should include pagination (and error handling). Basically, the samples should demonstrate best practices.

Hi @data-henrik, thanks for your suggestions.   It turns out that we have recently delivered a new feature in the SDK generator (which is used to generate the code packaged in IBM Cloud SDKs) where we now have the ability to generate examples that do in fact demonstrate pagination for list-type operations that support it.   It will, however, take some time before the services are updated within the various SDKs such that their examples reflect this.   I'll leave this issue open for now...


Improve state checker in Mship Account

In gitlab by @A-Lawrence on Feb 27, 2016, 21:38
Mship Account state checking could be nicer.  It's not a high priority as we don't have the transferring status yet.

Refactor state checker
Extract to helper function
Take into account current state (i.e. transferring)

In gitlab by @A-Lawrence on Aug 20, 2016, 19:39
Removed ~98471 label

In gitlab by @A-Lawrence on Sep 1, 2016, 12:49
mentioned in commit 104440a7d8998e0ba2077db9c0b20d4cea9d7b2c

In gitlab by @A-Lawrence on Sep 2, 2016, 03:46
mentioned in commit 0cfa9d6c27628564489326746774db039ec4fbc7


docs: fix usage for module

Description
Fix the usage for module powervs-vpc-landing-zone
Release required?

[x] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline


duckdb-rs 0.10 is failing on macOS Venture (13.5.2) with code 14

Hello,  trying to compile v0.10.0 on my mac, I get a lot of errors from the c++ compiler (from xcode 14) about missing includes.
  cargo:warning=ToolExecError: Command env -u IPHONEOS_DEPLOYMENT_TARGET "/opt/homebrew/bin/sccache" "c++" "-O0" "-ffunction-sections" "-fdata-sections" "-fPIC" "-gdwarf-2" "-fno-omit-frame-pointer" "--target=arm64-apple-darwin" "-mmacosx-version-min=13.3" "-I" "duckdb" "-I" "duckdb/third_party/fmt/include" "-I" "duckdb/third_party/mbedtls/include" "-I" "duckdb/extension/parquet/include" "-I" "duckdb/third_party/tdigest" "-I" "duckdb/third_party/concurrentqueue" "-I" "duckdb/third_party/httplib" "-I" "duckdb/third_party/miniz" "-I" "duckdb/third_party/pcg" "-I" "duckdb/third_party/jaro_winkler/details" "-I" "duckdb/third_party/re2" "-I" "duckdb/third_party/parquet" "-I" "duckdb/third_party/hyperloglog" "-I" "duckdb/src/include" "-I" "duckdb/third_party/utf8proc/include" "-I" "duckdb/third_party/fastpforlib" "-I" "duckdb/third_party/zstd/include" "-I" "duckdb/third_party/thrift" "-I" "duckdb/third_party/snappy" "-I" "duckdb/third_party/mbedtls/library" "-I" "duckdb/third_party/mbedtls" "-I" "duckdb/third_party/jaro_winkler" "-I" "duckdb/third_party/fsst" "-I" "duckdb/third_party/libpg_query" "-I" "duckdb/third_party/skiplist" "-I" "duckdb/third_party/utf8proc" "-I" "duckdb/third_party/libpg_query/include" "-I" "duckdb/third_party/fast_float" "-std=c++11" "-stdlib=libc++" "-stdlib=libstdc++" "-w" "-DDUCKDB_EXTENSION_PARQUET_LINKED=1" "-DDUCKDB_EXTENSION_AUTOINSTALL_DEFAULT=1" "-DDUCKDB_EXTENSION_AUTOLOAD_DEFAULT=1" "-o" "/Users/roberto/Src/Rust/src/fetiche-rs/target/debug/build/libduckdb-sys-ea9d0cc43862ad60/out/239a61d5cb207136-ub_src_parser.o" "-c" "duckdb/ub_src_parser.cpp" with args "c++" did not execute successfully (status code exit status: 1).cargo:warning=In file included from duckdb/ub_src_core_functions_aggregate_nested.cpp:1:
  cargo:warning=In file included from duckdb/src/core_functions/aggregate/nested/list.cpp:1:
  cargo:warning=duckdb/src/include/duckdb/common/pair.hpp:11:10: fatal error: 'utility' file not found
  cargo:warning=#include <utility>
  cargo:warning=         ^~~~~~~~~
  cargo:warning=1 error generated.
  cargo:warning=In file included from duckdb/ub_src_common_enums.cpp:1:
  cargo:warning=In file included from duckdb/src/common/enums/catalog_type.cpp:1:
  cargo:warning=In file included from duckdb/src/include/duckdb/common/enums/catalog_type.hpp:11:
  cargo:warning=duckdb/src/include/duckdb/common/constants.hpp:11:10: fatal error: 'memory' file not found
  cargo:warning=#include <memory>
  cargo:warning=         ^~~~~~~~
  cargo:warning=1 error generated.

duckdb is installed through Homebrew.  Using rust 1.76/1.78-nightly.  These errors seem weird.

Not using xcode. For what it's worth, it works fine on macOS 14.3.1 with bundled feature for me.
cargo new test-duckdb
cd test-duckdb
cargo add duckdb --features bundled
cargo build --release

Finished building in 1m 27s:
   Finished release [optimized] target(s) in 1m 27s

I'm still running macOS Ventura aka 13.3.1 so I guess I need Sonoma.  This was not in the release notes…
bundled  makes no difference here.


make webphone as optional dependence in call module

Coverage remained the same at 37.489% when pulling a68aa5e11d9f771842653ca1144a4339dc9018b2 on embbnux:feature/call-dependences into 4177045dba29372f673e3afa4f37cdd1ab733024 on ringcentral:master.

Something went wrong. Investigate!


FAILURE: Build failed with an exception.

When I generate desktop project it works fine but when i generate android project it gives me this error
FAILURE: Build failed with an exception.

Where:
Build file 'D:\android_studio_project\test\build.gradle' line: 53

What went wrong:
A problem occurred evaluating root project 'test'.

Failed to apply plugin [id 'android']
Could not create an instance of type com.android.build.gradle.AppExtension_Decorated.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED
I can't understand what this problem is caused by?either it has some problem with my gradle version or some file is missing in my PC

You are running with the wrapper? Or you get this error in the setup tool?

@Tom-Ski I didn't get you. this error displayed when i click generate button on libgdx setup

With what settings?

I kept project name,package and game class to the default and provided correct path for android sdk and check only android platform without making any changes to advance option

Closed, there are no problems with freshly generated projects at the moment.


docs: add docs for SQL++ for Mobile

Included new documentation for:

Query Builder
SQL++ for MObile
SQL++ Server Differences
SQL++ Querybuilder Differences
Live Queries
Query Troubleshooting
Indexes

Looked through other documentation for syntax and formatting and tried to match the examples provided as much as possible.  Note that tables aren't going full width even though I updated the CSS for this.  Not sure why it's not going the full width of the container.

@biozal Thanks for all the help! Sorry that it took me a while to get to this PR. Maybe you have already seen that I have merged most of it in separate parts in the last few weeks. This PR is now just the SQL++ for Mobile part, which I finished last.


[Bug]: Noticed Minor Issue placed in styles.yml File🐞

Description
GSSOC'23 Contributor
Description

Minor Issue Existed in styles.yml in GitHub Issue Templates Folder.

Screenshots

Additional information
No response

Hi @apoorvaron . Please Review on this Issue. Thank You !

@NarendraKoya999 Assigned