Deprecated features used - chef 12.7.2

Deprecated features used!
Default value false is invalid for property enable_utf8 of resource mysql_service. In Chef 13 this will  become an error: Option enable_utf8 must be a kind of [String]!  You passed false.. at 1 location:

/var/chef/cache/cookbooks/mysql/libraries/resource_mysql_service.rb:26:in `class:MysqlService'

Looking at libraries/resource_mysql_service.rb it seems that ':enable_utf8' should be kind of '[TrueClass, FalseClass]' rather than 'String':
attribute :enable_utf8, :kind_of => [TrueClass, FalseClass], :default => false
The definition of 'parsed_enable_utf8' (in the same file) should also be changed:
def parsed_enable_utf8
return enable_utf8 unless enable_utf8.nil?
end

Looks like this is no longer relevant, the enable_utf8 setting no longer exists AFAICS?

I'm pretty sure this is no longer the case.


grabber 4.6 new release issue with dll

What steps will reproduce the problem?
... when i open the programm
What is the expected behavior? What do you get instead?
... i get a window that says im missing MSVCP140.dll
What version of the program are you using? On what operating system?
...windows 7
Please provide any additional information below
...

Thanks for the report, I uploaded a fix. You can get it from the same release:
https://github.com/Bionus/imgbrd-grabber/releases/tag/v4.6.0
This can also be fixed by installing the VS redistribuable 2015 manually:
https://www.microsoft.com/en-us/download/details.aspx?id=48145


MINOR: Fix NPE when Connect offset contains non-primitive type

When storing a non-primitive type in a Connect offset, the following NullPointerException will occur:
07:18:23.702 [pool-3-thread-1] ERROR o.a.k.c.storage.OffsetStorageWriter - CRITICAL: Failed to serialize offset data, making it impossible to commit offsets under namespace tenant-db-bootstrap-source. This likely won't recover unless the unserializable partition or offset information is overwritten.
07:18:23.702 [pool-3-thread-1] ERROR o.a.k.c.storage.OffsetStorageWriter - Cause of serialization failure:
java.lang.NullPointerException: null
	at org.apache.kafka.connect.storage.OffsetUtils.validateFormat(OffsetUtils.java:51)
	at org.apache.kafka.connect.storage.OffsetStorageWriter.doFlush(OffsetStorageWriter.java:143)
	at org.apache.kafka.connect.runtime.WorkerSourceTask.commitOffsets(WorkerSourceTask.java:319)
... snip ...

The attached patch fixes the specific case where OffsetUtils.validateFormat is attempting to provide a useful error message, but fails to because the schemaType method could return null.
This contribution is my original work and I license the work to the project under the project's open source license.

LGTM. Test timeout seems unrelated. Thank you for contributing!


install fails on resigned 'trans-org' enterprise app - archived-expanded-entitlements.xcent not updated

Firstly - thanks for provise.  Been using it happily for a bit when resigning apps from within an organization.
Having some trouble trying to resign an app between organizations.
Process goes like this:

Compile an IPA with an explicit bundle id and matching MPP, with push notifications enabled and configured, and signed with an 'in-house' (enterprise) key.  This IPA installs correctly and recieves push notifications as expected.

Call resign using a certificate and MPP from another developer org enrolled in the apple enterprise dev program while changing bundle id to match new MPP.  Completes without error.

Attempt OTA installation of the new IPA.  This fails with:

Mar 25 12:03:33 Michaels-iPad itunesstored[2512] : 0x20bf000 __MobileInstallationInstallForLaunchServices_block_invoke240: Returned error Error Domain=MIInstallerErrorDomain Code=63 "Application is missing the application-identifier entitlement." UserInfo=0x145c0c60 {LegacyErrorString=ApplicationVerificationFailed, FunctionName=-[MIInstallableBundle performVerificationWithError:], NSLocalizedDescription=Application is missing the application-identifier entitlement., SourceFileLine=473}

In the resigned IPA, the correct NEW app id and bundle id appear in embedded.mobileprovision file under 'Entitlements' -> 'application-identifier'.  The file 'archived-expanded-entitlements.xcent' in the new IPA still contains references to the old app id and bundle id, though.   I >think< this is why the device declines to install the IPA.

@astletron Can you confirm this happens in the latest 0.0.3 version? And also, what XCode version are you using to build it?
I never experience such error but to be honest, I was never able to test between 2 enterprise accounts (I don't have access to one :disappointed: ).
If I could guess I would say that I have to add --entitlements as an extra parameter to the codesign command

Problem was not related to archived-expanded-entitlements.xcent.
I created an entitlements.xml file by hand and added --entitlementsFile to my use of 'resign ipa' and the resultant .ipa installed and worked just fine.
Would be nice to automate updating entitlements.  Some of those values are specific to the new app id and bundle id.

Oh, glad to hear it is working properly.
I will close this for now, then.

@astletron quick question: do you have push notifications enabled for this specific app you are resigning? are they working properly after resigning?


Embeds app inserts unnecessary padding.

(I'm novice of OSS and English. So, this sentences may be appropriate.)
Issue Summary
Template of Embeds block has a bad attribute 'padding-bottom'.
If inserting some embeds block, big padding is inserted too.
Steps to Reproduce

Start a new project with wagtail start myproject
Edit home/models.py as follows

from django.db import models

from wagtail.core.models import Page
from wagtail.core.fields import StreamField
from wagtail.admin.edit_handlers import StreamFieldPanel
from wagtail.embeds.blocks import EmbedBlock

class HomePage(Page):
    body = StreamField([
        ('embed', EmbedBlock()),
    ])

    content_panels = Page.content_panels + [
        StreamFieldPanel('body'),
    ]

and edit home/templates/home/home_page.html as follows
{% extends "base.html" %}

{% load wagtailcore_tags %}

{% block body_class %}template-homepage{% endblock %}

{% block content %}
    {% include_block page.body %}
{% endblock %}

cd myproject ./manage.py migrate ./manage.py createsuperuser ./manage.py runserver
access http://<IP_ADDRESS>:8000/admin/pages/
add Homepage, and add two embeds to Homepage. url example: https://www.youtube.com/watch?v=0oYrRCxo30k
preview the page
there is a lot of padding

Any other relevant information. For example, why do you consider this a bug and what did you expect to happen instead?
this code is involved.
https://github.com/wagtail/wagtail/blob/f4cc454e13c23161490b995da970cd2f33ec92fa/wagtail/embeds/templates/wagtailembeds/embed_frontend.html#L1
6 years ago, the attribute 'pudding-bottom' was added.
But, I don't have idea what the code intend.
https://github.com/wagtail/wagtail/commit/17aa3fc89e5587032dbe7f688269915266998d73

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

Technical details

Python version: Python 3.7.0
Django version: Version: 2.2.6
Wagtail version: Version: 2.6.2

Hi @evistream - please see #1495.


forge test fail on tests that run create2

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (63ed109 2022-08-09T00:20:50.211015Z)
What command(s) is the bug in?
forge test
Operating System
macOS (Apple Silicon)
Describe the bug
The same test script has been running fine in CI until the CI picked up a new version of foundry last Wednesday that broke tests that cover code with create2. The code after create2 are not run and the test shows a large gas usage.

Can you please give an example?

Hey @gakonst, I did some digging, our test broke 6 days ago. I'm pretty sure it's related to this commit where an empty account gets etched some bytecode. https://github.com/foundry-rs/foundry/commit/3df9536132bd213b5de3a6ebab0825302061b61e.
In my test, I'm mocking a function call on an address that will be the result of create2.

With a mockCall, the test reverts at create2
Without the mockCall the test goes past create2 and breaks because the mock is missing.

Maybe I shouldn't be mocking on that create2 contract address?

Yeah, you shouldn't be mocking an empty account if you plan to deploy on top of it, however I'd like to see an example of your test to see if it might make sense? If so we should extend vm.mockCall with a flag to skip the etching

yeah, it was my unit test where the function being tested will lazy deploy a contract and call a function on it. I think it kinda make sense either way. I just stopped mocking it and rely on the contract being actually deployed.


server: do not check zone imagestores for directdownload template delete

Description
This PR fixes the error while deleting a direct-download template when there is no secondary storage added (probably only in Edge zones).
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan package

@shwstppr a [SF] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@weizhouapache cc @DaanHoogland this can be included in 4.18.1 milestone. I wrongly targetted it to main branch previously

@weizhouapache cc @DaanHoogland this can be included in 4.18.1 milestone. I wrongly targetted it to main branch previously

thanks @shwstppr
moved to <IP_ADDRESS> milestone

Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6320

@blueorangutan test keepEnv

@DaanHoogland a [SF] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-6830)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 45677 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7607-t6830-kvm-centos7.zip
Smoke tests completed. 106 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_create_pvlan_network
Error
0.08
test_pvlan.py

test_01_migrate_VM_and_root_volume
Error
79.93
test_vm_life_cycle.py

test_02_migrate_VM_with_two_data_disks
Error
52.61
test_vm_life_cycle.py

tested this:

registered two templates, one direct download and one downloaded
disabled the zone
removed the secondary storage
enabled the zone
tried to delete the downloaded template, this failed (PASS)
tried to the delete the direct download template, this succeeded (PASS)

@blueorangutan package

@shwstppr a [SF] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6335

@blueorangutan test

[SF] Trillian test result (tid-6882)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 40884 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7607-t6882-kvm-centos7.zip
Smoke tests completed. 99 look OK, 9 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

ContextSuite context=TestCpuCapServiceOfferings>:setup
Error
0.00
test_service_offerings.py

ContextSuite context=TestServiceOfferings>:setup
Error
0.25
test_service_offerings.py

ContextSuite context=TestVMWareStoragePolicies>:setup
Error
0.00
test_storage_policy.py

test_01_migrate_VM_and_root_volume
Error
75.52
test_vm_life_cycle.py

test_02_migrate_VM_with_two_data_disks
Error
53.36
test_vm_life_cycle.py

test_10_attachAndDetach_iso
Error
0.07
test_vm_life_cycle.py

test_06_download_detached_volume
Error
306.72
test_volumes.py

test_13_migrate_volume_and_change_offering
Error
120.86
test_volumes.py

ContextSuite context=TestIpv6Vpc>:setup
Error
0.00
test_vpc_ipv6.py

ContextSuite context=TestVPCRedundancy>:setup
Error
0.00
test_vpc_redundant.py

ContextSuite context=TestVPCNics>:setup
Error
0.00
test_vpc_router_nics.py

ContextSuite context=TestRVPCSite2SiteVpn>:setup
Error
0.00
test_vpc_vpn.py

ContextSuite context=TestVPCSite2SiteVPNMultipleOptions>:setup
Error
0.00
test_vpc_vpn.py

ContextSuite context=TestVpcRemoteAccessVpn>:setup
Error
0.00
test_vpc_vpn.py

ContextSuite context=TestVpcSite2SiteVpn>:setup
Error
0.00
test_vpc_vpn.py

test_disable_oobm_ha_state_ineligible
Error
0.33
test_hostha_kvm.py

@blueorangutan package

Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6393

@blueorangutan test

@DaanHoogland a [SF] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-6951)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 49578 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr7607-t6951-kvm-centos7.zip
Smoke tests completed. 107 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_01_migrate_VM_and_root_volume
Error
76.50
test_vm_life_cycle.py

test_02_migrate_VM_with_two_data_disks
Error
50.29
test_vm_life_cycle.py


Add support for energy transfer contract measurement

The following commands now work with energy transfer contracts:

get_daily_measurements_json
get_monthly_measurements_json

Should make https://github.com/carohauta/oma-helen-ha-integration/issues/6 work for measurements

Thank you for your contribution and proactivity!
I tested it out and everything seems to be working 👍 (even though I still do not have Helen as my transfer company)
Could you change the PR to point into the dev branch?
Thanks again!


All "Narrow the Scope" entries coming back as "Not Valid

Description
When trying to Narrow down the scope to reduce cardinality all entries are coming back as "Not Valid". I tried both "basic" and advanced NRQL with the same result.
Steps to Reproduce
Using Transaction eventtype. To limit scope I used where request.uri like abc. I also used various attributes, operators and values. All come back as "Not Valid"

Closing this for now. Re-open if you're able to describe this further.


⚠ Bofrost.fr has degraded performance

In 7b14381, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1708 ms

Resolved: Bofrost.fr performance has improved in ac885f6 after 20 minutes.


VM: Restore checking fingerprints of recognized methods

VM used to check source fingerprints of 'recognized' methods in core libraries.
Condition in Function::CheckSourceFingerprint includes kernel_offset() <= 0 which is false since Dart 2. We need to restore these checks.
/cc @aartbik @mraleph @a-siva @rmacnak-google

I believe this (temporarily) invalidates the comment here:
https://github.com/dart-lang/sdk/blob/c5ea3e85a097a52a063e4d8c061ad3b51299b15a/runtime/vm/compiler/recognized_methods_list.h#L12-L13
Since the code to generate the new fingerprints has been disabled:
https://github.com/dart-lang/sdk/blob/c5ea3e85a097a52a063e4d8c061ad3b51299b15a/runtime/vm/object.cc#L9165-L9170
Unless there's some other mechanism I'm unaware of we should probably amend the first comment to clarify expectations.

Fingerprints checking was disabled once again when we had 2 versions of core libraries (sdk and sdk_nnbd).
After SDK unforking we have only one version of core libraries, so we should re-enable fingerprints checking by removing this #if and updating all fingerprints.


Article - Develop Spotify Player web extension

https://www.dzungnguyen.dev/blog/develop-spotify-player-web-extension


d3 library exception with latest fusebox

fuse >= next.400 has a problem with d3 library (d3-selection event)
vendor.js:18817 Uncaught TypeError: Cannot read property 'ctrlKey' of null
    at SVGSVGElement.defaultFilter (vendor.js:18817)
    at SVGSVGElement.wheeled (vendor.js:18948)
  at SVGSVGElement.<anonymous> (vendor.js:19567)

I'd tried google recommendations but since it works on next.6 I believe this is fuse's issue
there is repo with bug reproduced: react-example

Please help debug it. I've replaced the contents of node_modules/d3-selection/src/selection/on.js @62 with the transpiled version of ts 3.9.2 and got the same result. It must be something else.
Also, I noticed a very old version of typescript 3.1.0. That might have made it compile differently

I believe it's some smelly events realisation on d3
they do modification of exported event
// node_modules/d3-selection/src/selection/on.js @45
exports.event = null;
// ... SKIP ....
function contextListener(listener, index, group) {
  return function (event1) {
    var event0 = exports.event; //HERE exports modified
    exports.event = event1;
    try {
      listener.call(this, this.__data__, index, group);
    } finally {
      exports.event = event0;
    }
  };
}

imported event doesn't exists here
// node_modules/d3-zoom/src/zoom.js @26
26: function(__fusereq, exports, module){
exports.__esModule = true;
var d3_dispatch_1 = __fusereq(65);
var d3_drag_1 = __fusereq(66);
var d3_interpolate_1 = __fusereq(67);
var d3_selection_1 = __fusereq(22);
var d3_transition_1 = __fusereq(68);
var constant_js_1 = __fusereq(69);
var constant_js_1d = __fuse.dt(constant_js_1);
var event_js_1 = __fusereq(70);
var event_js_1d = __fuse.dt(event_js_1);
var transform_js_1 = __fusereq(27);
var noevent_js_1 = __fusereq(71);
var noevent_js_1d = __fuse.dt(noevent_js_1);
function defaultFilter() {
  return !d3_selection_1.event.ctrlKey && !d3_selection_1.event.button; //HERE no event exported....
}

Also found similar problem here: https://github.com/d3/d3/issues/2733#issuecomment-190743489
But any of suggested work-arounds doesn't helps

Quick work-around to make d3 happy:
                'd3-selection': 'd3-selection/dist/d3-selection.js',
            }


test: add test for onsignalingstatechange

Codecov Report

Merging #33 into master will increase coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
+ Coverage   91.91%   92.04%   +0.12%
==========================================
  Files           1        1
  Lines         779      779
==========================================
+ Hits          716      717       +1
+ Misses         63       62       -1

Impacted Files
Coverage Δ

rtcpeerconnection.js
92.04% <0%> (+0.12%)
:arrow_up:

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 89adb96...7fdf202. Read the comment docs.


Fix for the reStructuredText preview in vscode

Added cleansing of an environment variable when calling the drawio application

The variable is automatically set when run in certain vscode extension contexts
The environment variable is evaluated by electron and prevents the normal application entry point

This fix is necessary to utilize the reStructuredText (sphinx) preview, directly in vscode

Resolved the review findings as suggested

Hi
This seems fine.
For some reasons the tests seem to be broken - they seem to be fine on master, so not sure what's up with that.
I'm currently in final exam period (well, whenever it happens after COVID dies down here again) so I'm busy studying and don't have time to try figure that out, sorry.
If you'd like to have a look and see what's going wrong I can merge a fix but until then I'd prefer not to merge failing tests to master (or release).
Sorry about that

Understandable. Could you please rerun the tests on the master?
Either my test environment has some issues or the muster is currently not passing the tests either. I'll try to take a look on the weekend.

Can't seem to figure out how to rerun it through GitHub actions.
My guess would probably be something has broken on master, maybe sphinx upgraded? Not sure exactly.

I finally came around to take a look at it...
First I updated the drawio version as the old ones are no longer available on GitHub. This caused me to change the default drawio path (changed for a while now in the drawio.deb package) and only use the old path as a fall back (could not try it with the old versions though).
The failing test cases can be narrowed down due to changes of drawio and sphinx in their latest versions compared to the expected result in the test cases.
Test cases page-index and figure:

Dimensions are off (often only by one pixel)
You reported the ticket https://github.com/jgraph/drawio-desktop/issues/254 which was "fixed" by adjusting their test cases in the came way as I had to do

Test case figure:

Seems as if Sphinx does generate something else in the latest version than what's specified in the test case. Expected is a div with class == "figure" but its actually a figure tag that gets generated in the HTML

Test case imgconverter:

From my understanding Sphinx will try to choose the "best" output of the images. Which does seem to be SVG for images displayed on webpages (which makes sens to keep them as vector graphics rather than converting them to PNG/ raster graphics).

Please have a look at my elaborations and changes. Please, correct me if I'm wrong or you disagree.

Apparently we do have very different development styles but I tried to change my commits according to your suggestions.
We need to create a new issue for the 'imgconverter' test case, which is skipped for now.
When not adapting the path for the deprecated 'drawio' directive, a whole bunch of test cases fail (due to not finding the draw.io binary). I removed the test steps and test cases. Meaning, that the deprecated directive can also only be used with older versions of the draw.io package. We might need to emphasize this rather breaking change a bit more. What do you think?

Uh, sorry about that. I didn't do most of the working creating the deprecated version so I had forgotten that there were tests for the deprecated version.
I'm okay with changing the path if it is necessary for the tests to fail (I'd rather keep the deprecated version working if possible). If you'd like to undo 58e6296 and just put the fix for the path into the deprecated file that'd be great :)
Also if you can just do a once over your commits for spelling, and I'll be happy to merge this.

I think you have some commits with different authors (Milicaus vs josh?) Might want to fix that.
Also if you could merge the "fixed review" commit back into the first commit for fixing the preview.

Added the changes in the deprecated file and reverted the removal of test cases/ test steps :heavy_check_mark:
I'm sorry for the bad English in the previous commit message. Should be fixed now.
Thank you for taking the time, being so patient and responding so quickly.

You're welcome :)
Just the last couple of things I mentioned just before: https://github.com/Modelmat/sphinxcontrib-drawio/pull/62#issuecomment-901026260
And then I need to remember how to publish a new release to pypi for you :)

I was working on multiple machines which are apparently set up differently. Now everything should be addressed

... And it's released. Thanks for the contribution!

Thank you very much. Tested it on my Linux and Windows machine and both are working great :+1:
Thank you for the great extension


WH Loss Term

Unless I am reading the code incorrectly, it looks like the box WH loss term is defined as:
sum[ ( truth_wh - pred_wh) ^2 ]
whereas the yolo_v1 paper defines the loss as
sum [  (sqrt(truth_wh) -  sqrt(pred_wh) ) ^2]
[Note added sqrt]
Am I missing something in the code or is this an oversight? As mentioned in the paper this helps to prevent large boxes from dominating the loss over small boxes.

Thanks for the reference.
I just dug through the git repository (Hadn't really looked at any of the source code up until now!) and it looks like this loss term has changed quite a bit since YoloV1, but hasn't been mentioned in any of the papers.
As far as I can tell, it looks like the current incarnation is effectively using  something like:
(Log(truth) - Log(pred) )^2
If you go into yolo_layer.c line 93-107 there is a function called delta_yolo_box.
The tw and th get passed into a log.  And they take the raw pred_w and pred_h (before they have gone through an exponential).  Kinda weird and not super readable but I think that's whats happening here.  I haven't compiled this code or looked at it in debugger so it's hard to really be sure that's what's happening.

@stevethatsmyname Yes, you are right about this. I should have taken the log before computing the loss. I do get it right in keras-yolo3.

Cool, good to know I'm not crazy 😀 . I did a test where I trained on my dataset where i compared the end accuracy (mAP) obtained with all 3 cost terms (one with sqrt, one without sqrt, and one with log) and the log one did slightly better but only by about 1% mAP.


Funky Boy[HACTOBERFEST 2023]

Describe the project you want to add with tech stack

Funky boy CSS Animation
Expected behavior

Hello Sir,
A boy is playing with a ball using different skills of kicking, volleying the ball in different ways. I have made it using HTML & CSS.
Screenshots (optional)

https://github.com/pranjay-poddar/Dev-Geeks/assets/114172928/01b7a945-3416-495f-b93f-e47f676ad50f
Additional context (optional )

Please assign me this issue under Hactoberfest 2023

@debrajrout Please assign me this issue under Hactoberfest 2023


fix: add immage flag for . image convention

Add -i|--image flag to allow for specifying the image to override when the image property in a score file has a value .
Description

Added a new Flag to all commands
Added a backwards compatible implementation to override .containers.{containerId}.image where the value is .

What does this PR do?
It is a convention when working with score that setting the image to . means: "the image in the current context". Usually, this means the image that is associated with the score file. The CI pipeline will know the generated image to be inserted.
Today, there are a number of workarounds involving --property overrides (which require knowledge of the container IDs) or even sed scripts such as:
sed -i -E "s/^(\\s+image:)\\s+.*\$/\\1 '${IMAGE_NAME}'/" score.yaml

This change will make updated in the current image much simpler:
score-humanitec delta -i my-image:my-tag --org....

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New chore (expected functionality to be implemented)

Checklist:

[ ] My change requires a change to the documentation.

[ ] I have updated the documentation accordingly.

[x] I've signed off with an email address that matches the commit author.

Change looks good, just a duplicated typo in the flag description.

Fixed the duplicated text and rebased to main.


k8s: taints: import package

import the taints package, used just once in the e2e serial suite, to remove the last dependency from k8s.io/kubernetes
this PR won't actually remove that dep, but other PRs in the pipeline will.
no intended changes in behavior

/cherry-pick release-4.14

/test security

thanks for the PR.
/lgtm
/approve


non authenticated /invite_users should return 401 regardless of method - Story Id: 119381417

Link: https://www.pivotaltracker.com/story/show/119381417
Labels:
Requester: Filip Hanik
Estimate: 0.0

currently you get 405 when doing a GET /invite_users
That means there is no denyAll statement on the security.
curl -v -k https://login.identity.cf-app.com/invite_users

Link: https://www.pivotaltracker.com/n/projects/997278/stories/119381417/comments133743193
User: Filip Hanik

@stummidi
Comment Id: 133743193


QEI: Unable to (safely) define encoder pins as pull-up

I have this bit of code, which attempts to enable pull-up resistors on PB6 and PB7, and then construct a quadrature encoder interface on top of those pins using TIM4:
        let enc_a = gpiob.pb6.into_pull_up_input(&mut gpiob.crl);
        let enc_b = gpiob.pb7.into_pull_up_input(&mut gpiob.crl);
        let encoder = Timer::tim4(dp.TIM4, &clocks, &mut rcc.apb1).qei(
            (enc_a, enc_b),
            &mut afio.mapr,
            QeiOptions::default(),
        );

This fails because apparently the qei() method expects the pins to be floating instead:
error[E0277]: the trait bound `PB6<Input<PullUp>>: stm32f1xx_hal::gpio::Mode<Input<Floating>>` is not satisfied
  --> src\main.rs:95:68
   |
95 |         let encoder = Timer::tim4(dp.TIM4, &clocks, &mut rcc.apb1).qei(
   |                                                                    ^^^ the trait `stm32f1xx_hal::gpio::Mode<Input<Floating>>` is not implemented for `PB6<Input<PullUp>>`
   |
   = help: the following implementations were found:
             <PB6<MODE> as stm32f1xx_hal::gpio::Mode<MODE>>
   = note: required because of the requirements on the impl of `stm32f1xx_hal::pwm_input::Pins<Tim4NoRemap>` for `(PB6<Input<PullUp>>, PB7<Input<PullUp>>)`

error[E0277]: the trait bound `PB7<Input<PullUp>>: stm32f1xx_hal::gpio::Mode<Input<Floating>>` is not satisfied
  --> src\main.rs:95:68
   |
95 |         let encoder = Timer::tim4(dp.TIM4, &clocks, &mut rcc.apb1).qei(
   |                                                                    ^^^ the trait `stm32f1xx_hal::gpio::Mode<Input<Floating>>` is not implemented for `PB7<Input<PullUp>>`
   |
   = help: the following implementations were found:
             <PB7<MODE> as stm32f1xx_hal::gpio::Mode<MODE>>
   = note: required because of the requirements on the impl of `stm32f1xx_hal::pwm_input::Pins<Tim4NoRemap>` for `(PB6<Input<PullUp>>, PB7<Input<PullUp>>)`

Is there a way to do this safely and within the confines of the library? Note that this configuration seems to be valid as I have tested it using a C program generated in STM32CubeIDE and the behaviour is as I expect it.

I'm guessing the unsafe workaround looks like this:
        // let enc_a = gpiob.pb6.into_pull_up_input(&mut gpiob.crl);
        unsafe { PB6::<Input<PullUp>>::set_mode(&mut gpiob.crl) };
        // let enc_b = gpiob.pb7.into_pull_up_input(&mut gpiob.crl);
        unsafe { PB7::<Input<PullUp>>::set_mode(&mut gpiob.crl) };

Seems like a bug in pwm_input.rs, which implements Pins<REMAP> only for those GPIOs whose Mode = Input<Floating>. I guess it could work to replace the Floating by two PullState1/2 type parameters, so it accepts just any Input<Whatever>.
Do you see any problems with that?

Sorry. I've not seen this issue. What all allowed pin modes? Are they same for PwmPin?

I am gonna look into it and will open a PR this weekend if I find the time

From reading the reference manual, I cannot see a reason why any pullup/down/floating restrictions should be made on any "alternate function" input pin (which aren't really in any "alternate function" mode on this chip).
Also, I could not see any reason why any peripheral that wants to drive an Alternate pin would make any restrictions on the pin being Push-Pull or OpenDrain.
For this cause, I would assume that Input<Mode> is valid for any Mode. PR incoming.

I've created the PRs #357, #359 and #358, which incrementally lift the Input<...> and Alternate<...> restrictions for most peripherals. (I did not touch USB, and I2C remains OpenDrain-only).
Please prefer to review and merge #358.


Opencast upgrade multipart/form-data parsing lib (re: idleTimeout exceptions)

Describe the bug
When I upload really big files from a bad network, I frequently get an idleTimeout exception.
** Workaround **
I am increasing my idleTimeout param by enabling "jetty-opencast.xml" custom (increased) idle timeout param.
** Long Term Alternative **
However, I notice that I can avoid trouble shooting, and creating workarounds, for the specific commons-fileupload "idleTimeout" exception by migrating the Opencast ingest-service's use of "org.apache.commons.fileupload.servlet.ServletFileUpload" to the native multi-part parse support built into javax.servlet-api 3.1.0 (there since Opencast v3x).
A complication is to also migrate the use of "org.apache.commons.io.input.ProxyInputStream" for file upload progress (JMX tracking). The Servlet 3.1.0's @WebListener annotation might help here?
https://github.com/opencast/opencast/commit/0ada84a552db5d1251d0dc2e14ba893bf5ded80f
Migration is recommended by several community support threads
https://stackoverflow.com/questions/51446413/jetty-server-9-4-10-timeout-exception-idle-timeout-expired
https://www.eclipse.org/lists/jetty-users/msg08629.html

I'm going to start work on this, part time. Not sure how long that might take. This is not priority work for our site unless the idleTimeout extension doesn't stop upload fails or something else goes wrong with that lib.
I added this issue to Opencast repo in case anyone else is already thinking about this and/or has started work.


update gf256.h

update fg256.h from  https://github.com/catid/gf256/pull/6

Hi @yselkowitz
Can you update the fg256.h from the PR above?


Initialize recipes for the /pr action variant

Summary
Expand the number of example workflows for the /pr variant of the action by adding a "Recipes" section to its README.md. It currently provides two examples, one to mimic Dependabot updates and one to minimize noise.

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

Comparison is base (092ba9d) 98.57% compared to head (e0f6617) 98.57%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #180   +/-   ##
=======================================
  Coverage   98.57%   98.57%
=======================================
  Files           4        4
  Lines         140      140
=======================================
  Hits          138      138
  Misses          2        2

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


CP4D Authenticator for service token api

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

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


Updates to accessions.tsv.gz (173518 new, 873 removed, 168 updated) seqs downloaded 2022-10-07

Thanks!  🙂

Thank you Angie! 😁 Hope you are having a nice start to autumn! 🍂

Wow, that was fast!  😄  Autumn is kind of a relative thing in the non-mountainous parts of California, but very nice!

Haha, I guess that's true! Well, I'm sure you can still find fun pumpkins to decorate with! Here in Switzerland the leaves are turning and looking lovely!


Is this library still maintained?

Hi folks,
great work so far! However just wanted to ask if this library is still being maintained or if you need some help?

Hi!
Yes, it is maintained. We don't have any new features planned as of today, but we do address bugs as they come.
Thanks!


improve accessibility of the editor by output more infos over UIA

Hey Guys,
I don't know whether I am here on the right place, but I will try this.
The Editor component of VS has some accessibility problems, because some infos are not communicated over the UIA interface or are wrong.

The Linenumbers are always characters as TextUnit, that is nice, but if you enable "show line numbers" in Visual studio, this numbers will always take in the selectedTextRange in the UIA-Framework, if you go on the begin of a line.
the Tags like ErrorTag and other classifiers are not exposed to the UIA interface, so no screenreader can get infos about that.
Something seems wrong with the TextRange and Move, if you enable the Line numbers, the indexes are not right or the textrange gives strange values back.

The Issues could be resolved with a custom UIA pattern and some fixes, let's try to find a solution for this problems because this would give the accessibility of the whole VS a very push forward.
all the best,
Christopher

+@DanteGagne our accessibility expert


Fix userupdates status parsing

List entries, with any status except Plan to Watch/Read, when list entry has zero progress (0 watched episodes/0 read chapters) and entry doesn't have total episodes/chapters, in most cases it means that entry is ongoing, MAL displays like Status - e.g. Watching - instead of Watching or Watching -/?.
This PR simply replaces  - with empty string which later will be trimmed in such type of list entries statuses. It doesn't affect entries with On-Hold status since they don't have space before dash, and no other status on MAL have dashes in it.
Example in my profile https://myanimelist.net/profile/N0D4N

Plan To Watch entries are displayed with just status.
Reading with non zero progress but unknown total chapters is displayed as Reading 3/?
Dropped with zero progress and known total chapters is displayed as Dropped -/58
Completed with zero progress and zero total chapters is displayed as Completed -

That was quick, thank you!


Improve the readme a little

Spell APNs the same way throughout the readme
Fix grammar as best as I could
Fix minor typos in the examples

Tnx!


MINOR: A few small cleanups from KAFKA-6299

Use local fields in AdminClientRunnable to avoid the need to pass the collections as method parameters and fix warnings about invalid javadoc tags. Also use the internals package name for AdminMetadataManager for consistency with consumer.internals and producer.internals
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@hachikuji when merging, can you please mention AdminClient in the commit title?


config: remove allowUndeclaredRTE property from checkstyle JavadocMethod

https://checkstyle.org/releasenotes.html#Release_8.28
https://github.com/checkstyle/checkstyle/issues/7329
checkstyle project need this update to continue to keep Orekit in checkstyle's CI verification process.

Manually merged into the develop branch.
Thanks for the fix!


Solidity 0.5.x

Is it possible to check vulnerabilities in contract build with Solidity 0.5x?

We are working on the bytecode level not at source code level. This should work as long as your compiler produces eligible bytecode.

Hi does it produce python or any other form of language output after decompile like panoramic easy to understand code I mean?


[BACKLOG-5408] - tests for kettle5-log4j-plugin

@mdamour1976 please find someone who can review this.
I marked some classes as deprecated in separate commit - they have negative effect on code coverage but I see no reason to waste time on them.

@brosander @mattyb149

For all the added "deprecated" annotations, we need to indicate why they're deprecated and what should be used instead:
http://wiki.pentaho.com/display/PEOpen/Deprecation+Guidelines

Excellent comment Matt - thanks!

@AndreyBurikhin seeing 2 failures when running here, looks like it could be timezone related
formatLogMessage_with_add_time
formatNonLogMessage_with_add_time

@brosander fix timezone issue

@mattyb149  I removed commit with "deprecated" annotations and created this case http://jira.pentaho.com/browse/BACKLOG-5447.

Changed files:
plugins/kettle5-log4j-plugin/ivy.xml
plugins/kettle5-log4j-plugin/src/org/pentaho/di/core/logging/log4j/Log4jLogging.java
plugins/kettle5-log4j-plugin/test-src/.gitignore
plugins/kettle5-log4j-plugin/test-src/org/pentaho/di/core/logging/Log4jAppenderTest.java
plugins/kettle5-log4j-plugin/test-src/org/pentaho/di/core/logging/Log4jFileAppenderTest.java
plugins/kettle5-log4j-plugin/test-src/org/pentaho/di/core/logging/Log4jKettleLayoutTest.java
plugins/kettle5-log4j-plugin/test-src/org/pentaho/di/core/logging/Log4jPipedAppenderTest.java
plugins/kettle5-log4j-plugin/test-src/org/pentaho/di/core/logging/log4j/Log4jLoggingTest.java
plugins/kettle5-log4j-plugin/test/.gitignore
Checkstyle violations:
Errors:File plugins/kettle5-log4j-plugin/src/org/pentaho/di/core/logging/log4j/Log4jLogging.java1. Message: Line has trailing spaces. at Line: 38 Column: None2. Message: Line has trailing spaces. at Line: 40 Column: None3.


Support for lcd

Could you maybe support a lcd too and in that case disable the TTS. So your code would cover all the possibilities :)

Sorry for the late answer.
https://github.com/9and3r/mopidy-touchscreen could be what you want.


How to get test predictions (and other non-scalar metrics)?

❓ Questions and Help
What is your question?
If I have a trained model and I want to test it using Trainer.test(), how do I get the actual predictions of the model on the test set?
I tried to log the predictions and writing a Callback to get the logs at test end, but it seems like I can only log scalar Tensors in the dictionary returned by my model's test_end().

Do need also to measure model performances? if not you can just get predictions, see
https://pytorch-lightning.readthedocs.io/en/latest/introduction_guide.html#predicting

It would be nice to not have to call the model separately since then I would have to write my own testing loop on the test set which kinda defeats the purpose of Pytorch lightning

@KeAWang
If I understand correctly, you want to log ALL predictions on your test set. This should be possible:

Implement test_dataloader, test_step and test_end
In test_step you can't return a log dict, that's normal. But you can call self.logger.experiment.add_image or whatever method you want to use to log your predictions (depends on the logger class).

Is this what you need?

Right but I want to be able to get the predictions as a local variable that I can continue to process in my jupyter notebook. I'm not sure how to extract it from a tensorboard logger. Would I have to write my own logger?

That's not the same as logging, that was not clear in your original question.
You will have to collect your predictions in test_step in a variable like self.predictions which is a list or something. Then after you call trainer.test() you can access model.predictions in your notebook.
What do you think?

I was wondering if there was something else that fit within the framework. But that would work. Thank you!

If you want direct access, it can be done directly using the forward pass as follows. (The way it is done in PyTorch.)
model  = YourLightningModule.load_from_checkpoint(r"path/to/checkout.ckpt")
model.to(device)
data.to(device)
with torch.no_grad():
    out = model(data)

You can loop over a dataloader if you want.

If you want direct access, it can be done directly using the forward pass as follows. (The way it is done in PyTorch.)
model  = YourLightningModule.load_from_checkpoint(r"path/to/checkout.ckpt")
model.to(device)
data.to(device)
with torch.no_grad():
    out = model(data)

You can loop over a dataloader if you want.

That's not the same as logging, that was not clear in your original question.
You will have to collect your predictions in test_step in a variable like self.predictions which is a list or something. Then after you call trainer.test() you can access model.predictions in your notebook.
What do you think?

i tried its empty

Can anyone make it clear about the  self.trainer.results. This is useful in the following scenario.

Test dataset: all label is set to zero
The predictions (in test dataset ) have to align with the metadata
All x, y, and metadata are processed in Dataset together

In this scenario, we just call test_epoch_end, it will return the prediction and aligned metadata


Development/server override

Successfully tested locally and on the pi.

Needs some rework and Wissem is not available for some time


System unavailable: infra-ibmcloud-vagrant-* not booting Vagrant VMs as intended

Link to any log file showing the problem:
https://ci.adoptopenjdk.net/job/VagrantPlaybookCheck/OS=CentOS6,label=vagrant/981/
https://ci.adoptopenjdk.net/job/VagrantPlaybookCheck/OS=CentOS6,label=vagrant/982/

Please describe the issue:

On vagrant up, the following error message occurs:
15:41:15 Command: ["hostonlyif", "create"]
15:41:15
15:41:15 Stderr: 0%...
15:41:15 Progress state: NS_ERROR_FAILURE
15:41:15 VBoxManage: error: Failed to create the host-only adapter
15:41:15 VBoxManage: error: VBoxNetAdpCtl: Error while adding new interface: failed to open /dev/vboxnetctl: No such file or directory
15:41:15 VBoxManage: error: Details: code NS_ERROR_FAILURE (0x80004005), component HostNetworkInterfaceWrap, interface IHostNetworkInterface

I have confirmed that /dev/vboxnetctl isn't there on the machine, and I was able to recreate the issue. This may or may not be related to the changes I made to my job that clears out the hostonlyifs of the machine recently (ref: #1766 ). The recorded last time a machine was booted properly was on the 24th of December (see: https://ci.adoptopenjdk.net/job/VagrantPlaybookCheck/979/ ).
A quick google says that this is an issue with the virtualbox service however, there apparently isn't one on the machine (though according to systemctl -a, there are several services prefixed with vbox- . I'll go through and see if there's any services that aren't running that should be, and see if any of those work. If all else fails, apparently re-installation of Virtualbox will fix the issue (though, this will be my final resort, as it can often be a pain).

Restarting the machine fixed the issue! :+1:
(note for if anyone (i.e. future me) has to restart that machine in the future, the way to start all the separate Jenkins agents for each vagrant* user is  nohup ./startjenkins.sh & )
Closing issue

Restarting the machine fixed the issue! :+1:
(note for if anyone (i.e. future me) has to restart that machine in the future, the way to start all the separate Jenkins agents for each vagrant* user is  nohup ./startjenkins.sh & )
Closing issue

Issue has re-occured (see: https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1814#issuecomment-756723043 ). I feel a more permanent fix is required, instead of just restarting the machine.
I'm going to try reinstalling virtualbox-6.0, as a starting point.

Issue has re-occured (see: https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1814#issuecomment-756723043 ). I feel a more permanent fix is required, instead of just restarting the machine.
I'm going to try reinstalling virtualbox-6.0, as a starting point.

Note: to uninstall virtualbox-6.0, all VBox processes need to be stopped - meaning VBoxHeadless, VBoxSVC\ and VBoxNetDHCPin theps aux` output.

Note: to uninstall virtualbox-6.0, all VBox processes need to be stopped - meaning VBoxHeadless, VBoxSVC\ and VBoxNetDHCPin theps aux` output.

I've managed to boot a Vagrant VM locally, successfully. Running VPC - I expect there may need to be some additional setup after reinstalling ansible and Virtualbox-6.0.

I've managed to boot a Vagrant VM locally, successfully. Running VPC - I expect there may need to be some additional setup after reinstalling ansible and Virtualbox-6.0.

Looks okay actually! Closing issue :D

Looks okay actually! Closing issue :D


Tests for custom assertions

Hello,
maybe I missed it or this is not the right place to ask my question.
Can someone give me a hint how unit tests for custom assertions could be written?
Thanks in advance for your time!
Greetings
Lars

I suggest to look at the documentation: http://joel-costigliola.github.io/assertj/assertj-core-custom-assertions.html and the assertj-examples tests.
Hope it helps

@ltennstedt  comment or ask the mailing list if you need more help


Request Headers (Authorization)

Discussed in https://github.com/GraphQL-Portal/graphql-portal/discussions/517

Originally posted by rmolinet September  4, 2021
Seems like th Graphql Portal Gateway remove the Authorization request header, the API return UNAUTHENTICATED error everytime, please can anybody help how config the Gateway to allow Authorization header. Thanks.

Hi, @Erislandys! You have to add the operation/schema header to you source.
Click at "edit data-source" -> "Data Connector Configuration" -> and add Schema/Operation Headers like bellow.

It will map gateway request header "auth-header" to source request header "authorization".
You can rename it and use any gateway header you want.
Open gateway playground and send any request with header { "auth-header": "my_dynamic_token"}, source will get request with {"authorization": "Bearer my_dynamic_token"} header.
Thanks!

@msadovyi  thank you very much, it works,


While running yarn start got 'tabler-react' does not contain an export named 'Media'. Error

@Raghavsalotra sorry to hear that!  Can you do me a favor and give me the following info so I can help debug:

Output of git log so I can see what commit you are on.  Alternatively, run git pull and try again with the latest updates.
Did you run yarn install in both the root directory of the repository and in the examples directory?
Any other info you could provide on how to reproduce would be really helpful - I just tried a fresh install and couldn't reproduce the issue

@AaronCoplan

Please find below output of git log:
`commit bae1dbb5141dcb8694d191b3a82aa621e99d808c
Merge: 8ce202c b0bea1a
Author: Aaron Coplan<EMAIL_ADDRESS>Date:   Tue May 8 04:27:32 2018 -0400
Merge pull request #132 from tabler/missing-component-documentation-pages
Missing component documentation pages

commit 8ce202cd24764c9e724f7f48a213f8c462bb6d90
Merge: 421ffa3 732817d
Author: Aaron Coplan<EMAIL_ADDRESS>Date:   Tue May 8 04:24:15 2018 -0400
Merge pull request #133 from tabler/react-styleguidist

Auto-generating documentation powered by react-styleguidist`

Yes I did Yarn install in both root as well as example dir.
I just cloned the project and followed instruction in Readme.md .

Perhaps only run yarn start in the example dir, don't run it in the root dir in a separate terminal window.  Instead, run yarn build in the root dir, then once that finishes run yarn start in the example dir.  Let me know if that helps!

@AaronCoplan  That did not help.
PFA of error i am getting.

I guess, @Raghavsalotra  tried to import into his own project. NPM module does not contain Media component

Thanks @burakakyol you might be right there. @Raghavsalotra I've just published v0.0.4 to npm, please can you give that a try?

Thanks for help team, It worked. :)


Fix for #198 start-polaris throwing error

I was unable to re-create the issue by running Start-Polaris and Stop-Polaris consecutively but if you happened to run Start-Polaris twice you were out of luck until restarting PowerShell as we were just creating a new instance of HttpListener right over the top of an already listening HttpListener and would throw an error.
@harikanth64 - If possible I'd love it if you could pull this code down and see if it resolves your issue before I merge it in.

@TylerLeonhardt - Got confirmation from @Harikanth64 this resolves this issue. When you get a chance could you give me a code review?


Technical Indicator RSI

What bug or feature you wants to report.
can we add technical indicators too ?

As long as it satisfies financial needs you are much welcome! Go ahead @programerr01 :)

Is it done @programerr01 ?

Yes, i have done it but i am thinking about not adding it to the project since it wouldn't fit very well with financial calculations.

Ok. Then if you are not merging then let me know I'll close the issue.


[c#] Fix cs.sln load in Visual Studio 2015/2017

The project type GUID was still for a C++ project. It has been
switched to a C# project.

This caused a platform change from Win32 to Any CPU

The cloning example was missing a solution-level dependency on
Bond.IO.dll. One has been added.

FYI: @hcoona, I had to adjust the cs.sln to get it to load again in Visual Studio.

Thank you, I didn't notice that yet, and I didn't know the GUID would identify the project type.


how to align components in multiple line as arguments?

Hi, could anyone help me with this case?  I've tried a couple of rules but couldn't find out how to change
  replaceBodyHTMLString(renderToString(<Provider store={store}>
    <StyleSheetManager sheet={sheet.instance}>
      {bodyComponent}
    </StyleSheetManager>
  </Provider>))

to this
  replaceBodyHTMLString(
    renderToString(
      <Provider store={store}>
        <StyleSheetManager sheet={sheet.instance}>
          {bodyComponent}
        </StyleSheetManager>
      </Provider>
    )
  )

eslintrc
{
  "env": {
    "browser": true,
    "es6": true,
    "node": true
  },
  "extends": "eslint:recommended",
  "parserOptions": {
    "ecmaFeatures": {
      "experimentalObjectRestSpread": true,
      "jsx": true
    },
    "sourceType": "module"
  },
  "plugins": ["react"],
  "rules": {
    "indent": [2, 2],
    "linebreak-style": [2, "unix"],
    "quotes": [2, "double", { "allowTemplateLiterals": true }],
    "semi": [2, "never"],
    "brace-style": [2, "1tbs", { "allowSingleLine": true }],
    "curly": [2, "all"],
    "no-console": [2, { "allow": ["warn", "error", "log"] }],
    "no-unused-vars": 0,
    "function-paren-newline": ["error", "multiline"],
    "comma-dangle": [2, "only-multiline"]
  }
}

That isn't a jsx concern nor a react one; you'd use an eslint rule to govern that - but I don't think one exists (or if it does, I'm not sure which it'd be).
Either way, the jsx-indent rule would help once the jsx was moved onto its own line.

@ljharb Thanks for the help, jsx-indent definitely fills part of the job, I'll dig more with that and eslint

@jomasti Thanks for the close lookup, that was me frustratingly trying out splitting multi-arguments for the case I am asking, did not entirely understand the compatibilities. 🤐

Ok, I found the solution, and FWIW, this is the final eslintrc.json
{
  "env": {
    "browser": true,
    "es6": true,
    "node": true
  },
  "extends": "eslint:recommended",
  "parserOptions": {
    "ecmaFeatures": {
      "experimentalObjectRestSpread": true,
      "jsx": true
    },
    "sourceType": "module"
  },
  "plugins": ["react"],
  "rules": {
    "indent": [2, 2],
    "linebreak-style": [2, "unix"],
    "quotes": [2, "double", { "allowTemplateLiterals": true }],
    "semi": [2, "never"],
    "brace-style": [2, "1tbs", { "allowSingleLine": true }],
    "curly": [2, "all"],
    "no-console": [2, { "allow": ["warn", "error", "log"] }],
    "no-unused-vars": 0,
    "comma-dangle": [2, "only-multiline"],
    "react/jsx-indent": [2, 2]
  }
}

Note that it won't trigged warning or error if having the format like I did from the original issue.


[Bug]: Correctly handle Int64_MAX as a key in ordered list state

What happened?
This is a follow-up issue from comment https://github.com/apache/beam/pull/30317#discussion_r1638540287.
The ordered list implementations or RunnerV2 and direct runner are impacted.
Issue Priority
Priority: 2 (default / most bugs should be filed as P2)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue


cmd/go: get / mod: "module source tree too big" error

What version of Go are you using (go version)?

go version go1.15 darwin/amd64

Does this issue reproduce with the latest release?
yes.
What operating system and processor architecture are you using (go env)?
go env Output
GO111MODULE="auto"
GOARCH="amd64"
GOBIN="/Users/ikawaha/go/bin"
GOCACHE="/Users/ikawaha/Library/Caches/go-build"
GOENV="/Users/ikawaha/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/ikawaha/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/ikawaha/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/ikawaha/sdk/go1.15"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/ikawaha/sdk/go1.15/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/ikawaha/work/zzz/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/z5/_pm6q9c9263cv0cjx1b039h80000gn/T/go-build087421608=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
An error occurred when I tried to run the following program.
$ go run main.go
go: finding module for package github.com/ikawaha/kagome-dict-ipa-neologd
go: downloading github.com/ikawaha/kagome/v2 v2.0.4
go: downloading github.com/ikawaha/kagome-dict-ipa-neologd v0.0.1
main.go:6:2: create zip: module source tree too large (max size is 524288000 bytes)

main.go
package main

import (
        "fmt"

         dict "github.com/ikawaha/kagome-dict-ipa-neologd"
        "github.com/ikawaha/kagome/v2/tokenizer"
)

func main() {
        t, err := tokenizer.New(dict.Dict())
        if err != nil {
                panic(err)
        }
        // wakati
        fmt.Println("---wakati---")
        seg := t.Wakati("すもももももももものうち")
        fmt.Println(seg)
}

Kagome is a self-contained morphological analyzer that embeds dictionaries into binaries. I built a dictionary package for kagome using a named entity dictionary called Neologd, but got an error when I tried to get it.
The repository contains only the content of the dictionary as converted by go-bindata.
kagome-dict-ipa-neologd
$ du -h -d 1
600M	./internal
288M	./.git
 24K	./tool
889M	.

This error can be reproduced by go get github.com/ikawaha/kagome-dict-ipa-neologd or by running the above sample program.
What did you expect to see?
Successfully download the repository containing the large assets.
What did you see instead?
create zip: module source tree too large (max size is 524288000 bytes)

Duplicate of #29210 and previously #25470, which raised the limit from 100MB to 500MB.
In practice, it's useful have this limit, since it prevents a lot of accidental abuse for proxy operators and for users. Most people who see this error have checked in large resources that aren't needed to build packages in their module, so the workaround is to move those resources into a directory with a go.mod file at the root (excluding them from the parent module) or to delete them from tagged commits.
This may be the first module we've seen that actually has more than 500MB of source code though. Still, the size limit is important, and other workarounds should be exhausted before increasing it.

Can the data be compressed or encoded in a more compact format within the source code?
Can this package be broken up into package in smaller modules?
Does the data need to be embedded into packages, or can it be fetched separately?

Thanks for the advice. I was able to get around the limitations by breaking the package into smaller modules.


Add SHEM-based neutron energy group structures

When using DRAGON to generate multigroup cross sections, one should use neutron energy group structures that snap to the SHEM361 energy structure because that's the "fine group structure" for open source nuclear data for DRAGON.

There should be an energy spectrum that's analogous the ARMI33, but snapped to the SHEM361 rather than the ANL2081
Having the SHEM361 structure will be good for modeling thermal spectrum reactors

Closed with #238


Safetynet fails

Hi, I wanna say that Safetynet fails all the time even with the SU hide (Magisk)
I've done the following steps:

Switch to Lineage for MicroG
SUHide all of the MicroG apps
Register my device
Login to Google Play
MOVED DROIDGUARD HELPER TO /system/priv-app AND SETTED RIGHT PERMISSIONS
I guess is the server problem or something, I got Response payload validation failed (According to Safety net test)

Duplicate.
See #526

@ale5000-git could you close this issue? It's about the old bug fixed by https://github.com/microg/android_packages_apps_RemoteDroidGuard/pull/19


⚠ Processors has degraded performance

In 530591b, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 172 ms

Resolved: Processors performance has improved in f6d7d0d.


.github: add workflow for rejecting PRs that introduce whitespace errors.

This PR adds a workflow to check and reject PRs that introduce whitespace errors. The method used to check for whitespace errors is a minimalistic git diff-index --check against the base ref of the PR.

Okay, I ditched all the git hooks, their installation and related additions, leaving only a minimal whitespace error check in the workflows.


Less versions coupling in .hbs templates

Simpler way of fetching current version of the project: project.currentVersion rather than project.versions.0.version meaning less coupling and in a sence logic in the .hbs templates.
Also replaced and removed the version property in locale/en/index.md as it should be read from the versions configuration rather than a locale property.

We still need the entire list of versions for the download page.

It's still there as before (project.versions), but the current / latest version is alot easier to get a hold of.

Ah, sorry. I overlooked the line in the diff.

Np! I'm playing with the download matrix, this cleans up alot of verbosity in that markup too.


On Decision Page for Edit Request, CPU for test-namespace is showing requested quota in current quota

Description:
There is a bug in representation of the Edit Request. In Summary it is shown correctly, but in decision page it displays requested quota for Test as current quota, thus not showing the changes:

@funtigr I don't think it is a bug as the decision is already made. I would hide current xxx texts if the decision is made.

At this screenshot decision is not yet made.
It is the decision page for the Admin.

At this screenshot decision is not yet made.
It is the decision page for the Admin.

Is it edit request, if the request is still active how come the inputs are grayed out for admin?

Sorry, my bad.
Decision page shows everything right. And yes, we don't need Current Quota on other than Decision page.


atlas update

changelog
-remove christmas
-skub

i will do apc after server crashes for 2nd time while trolling saltern :trollface:

ok


Relationship between subscriptions and orders

It's not really an issue, but I'm struggling with getting all the orders associated with a specific subscription.
At this moment I only manage to get the orders through the $subscription->owner->orders relationship, but then I also get the orders of any other subscription from the same owner.
As far as I can see there is no $subscription->orders relation available. How do I achieve this? Or do I misinterpret the package?
Regards.

Probably the best way is via the order items. These contain links to the subscription, the order and the owner

So each processed single order item relates to a single order?

One order can have multiple items

So each processed single order item relates to a single order?

Correct!


Setup windows builders to build ruby

There's 2 changes here:
First, we need to get the build toolchain into the path. These changes are in _compile.rb.
Second, I've removed the devkit mingw from the path because:

Devkit installs a thing which does this. It runs that whenever
rubygems needs to gem install something that has native extensions.
The issue with having this in the path is if you did something like
install a build toolchain that came before the devkit mingw path.
In this case, gem installing will not correctly use the devkit toolchain
because it only checks that Devkit/mingw is in the path, meaning the
toolchain we put in front of it will be used instead.

This relies on https://github.com/chef-cookbooks/build-essential/pull/80 to function correctly

We might want to add a constraint in the metadata.rb like this:
depends 'build-essential', '>= TBD'

Obviously we'll need to make that change after a new version of build-essential has been released.

@schisamo we can definitely do that. This cookbook will continue to work how it currently does for the most part even without getting the latest version of build-essential. It will only break if you're using the DevKit mingw to build things other than gems.

@chef-cookbooks/omnibus-maintainers

Pending the pinning LGTM :+1:

:+1:

:+1:


Documentation update: small fix to code example in working with vectors doc

Description
Very small fix to the doc examples to make the code valid (mostly so lazy people like me can copy and past the code...)
Closes #<issue_number>
Type of change

[x] Documentation update

How Has This Been Tested
Checklist

[ ] I added relevant documentation
[ ] follows the style guidelines of this project
[ ] I did a self-review of my code
[ ] I made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I filled out the contributor form (see text above)
[ ] I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

Thanks @davanstrien


Is that possible add cookie as source for extract from request.

I want to use Extractible to extract data from cookie.
But I found cookie is not a valid source for extract.
Is that possible to add it?
Thanks!

I'll add it in future.

https://github.com/salvo-rs/salvo/pull/105


[V] Adjust interface for Regex Validation

Adjust interface to display the regex in the input

Validation implemented


type_registry update

Hi!
I tried to work with a Crust network with the help of py-substrate-interface. They have a type_registry template for Crust network.
When I use the template and try to fetch chain state market->filesV2, which should return a FileInfoV2 object, I get a NotImplementedError: Decoder class for "FileInfoV2<AccountId, BalanceOf>" not found which seems logic since there is no FileInfoV2 object in the template, just the FileInfo one.
Okay, there is a way to modify the template. I found this  description and tried to recreate it in the JSON format:
CRUST_TYPE_REGISTRY = {
    "types": {
        "FileInfoV2": {
            "type": "struct",
            "type_mapping": [
                ["file_size", "u64"],
                ["spower", "u64"],
                ["expired_at", "BlockNumber"],
                ["calculated_at", "BlockNumber"],
                ["amount", "Balance"],
                ["prepaid", "Balance"],
                ["reported_replica_count", "u32"],
                ["remaining_paid_count", "u32"],
                ["replicas", "BTreeMap<AccountId, Replica<AccountId>>"]
            ]
        },
        "Replica": {
            "type": "struct",
            "type_mapping": [
                ["who", "AccountId"],
                ["valid_at", "BlockNumber"],
                ["anchor", "SworkerAnchor"],
                ["is_reported", "Bool"],
                ["created_at", "Option<BlockNumber>"]
            ]
        },
    }
}

This results in scalecodec.exceptions.InvalidScaleTypeValueException: Invalid value for datatype "bool" error for the same chain state query. Seems like is_reported returns something but bool.
How can I perform this call?
p.s. other functions work fine: I use market->placeStorageOrder extrinsic and system->account chain state query.

Hi!
I tried to work with a Crust network with the help of py-substrate-interface. They have a type_registry template for Crust network.
When I use the template and try to fetch chain state market->filesV2, which should return a FileInfoV2 object, I get a NotImplementedError: Decoder class for "FileInfoV2<AccountId, BalanceOf>" not found which seems logic since there is no FileInfoV2 object in the template, just the FileInfo one.
Okay, there is a way to modify the template. I found this description and tried to recreate it in the JSON format:
CRUST_TYPE_REGISTRY = {
    "types": {
        "FileInfoV2": {
            "type": "struct",
            "type_mapping": [
                ["file_size", "u64"],
                ["spower", "u64"],
                ["expired_at", "BlockNumber"],
                ["calculated_at", "BlockNumber"],
                ["amount", "Balance"],
                ["prepaid", "Balance"],
                ["reported_replica_count", "u32"],
                ["remaining_paid_count", "u32"],
                ["replicas", "BTreeMap<AccountId, Replica<AccountId>>"]
            ]
        },
        "Replica": {
            "type": "struct",
            "type_mapping": [
                ["who", "AccountId"],
                ["valid_at", "BlockNumber"],
                ["anchor", "SworkerAnchor"],
                ["is_reported", "Bool"],
                ["created_at", "Option<BlockNumber>"]
            ]
        },
    }
}

This results in scalecodec.exceptions.InvalidScaleTypeValueException: Invalid value for datatype "bool" error for the same chain state query. Seems like is_reported returns something but bool.
How can I perform this call?
p.s. other functions work fine: I use market->placeStorageOrder extrinsic and system->account chain state query.

Thanks for your feedback @PaTara43 ! @mmyyrroonn Would you mind take a look at this

Sorry for not closing the issue, this was solved in Discord
Types also updated for the template, though not released yet
https://github.com/polkascan/py-scale-codec/commit/6c897a0f09d3fadcb84eae393d98a831cf403b50 1.0.39 here
https://github.com/polkascan/py-substrate-interface/commit/5b0a0027df6c8b9c6f37d5346d803846594f110b 1.0.37 here


CORS issue setting up Amplify hosting with GitHub in ca-central-1

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
No response
Region
ca-central-1
Amplify Hosting feature
Not applicable
Describe the bug
On 2 separate accounts we are unable to create a new Amplify hosted app in ca-central-1 with GitHub repo.  On the first step we select GitHub and click "Continue".  The screen refreshes back to the "From your existing code" options but gets stuck there, errors showing in console indicate some kind of CORS problem (see attached).
Again, we tried this on 2 completely separate AWS accounts with the same results.  It worked when we switched to us-east-1 and created a new app with GitHub linkage there, so it seems to be specifically with the Canadian region (which we need to use here for compliance reasons).

Expected behavior
We expect this first step of the process to complete successfully and continue on to the next step of selecting the specific repository within the GitHub account for the app build + deployment steps, etc.
Reproduction steps

Inside ca-central-1 region, click "New app -> host web app" and then select "GitHub" and "Continue"
The authorization screen for aws-amplify will display, confirm by clicking the green button.
You will be directed back to the previous screen where GitHub was previously selected, and the process just dies there.  CORS errors showing in the console.

Build Settings
No response
Additional information
No response

This appears to have now been resolved.


”Ozun”, ”Bicfalău”, ”Lisnău”, ”Lisnău-Vale” are listed in the drop-down menu

Description:
When user writes ”Ozun”, ”Bicfalău”, ”Lisnău”, ”Lisnău-Vale” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Ozun"

Expected results
The location appears in the drop-down menu as "Sat Ozun, COVASNA (Ozun)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Bicfalău"

Expected results
The location appears in the drop-down menu as "Sat Bicfalău, COVASNA (Ozun)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Lisnău"

Expected results
The location appears in the drop-down menu as "Sat Lisnău, COVASNA (Ozun)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Lisnău-Vale"

Expected results
The location appears in the drop-down menu as "Sat Lisnău-Vale, COVASNA (Ozun)".


appId // make sure this matches with your Server NgModule

Don't quite get this.
https://github.com/MarkPieszak/aspnetcore-angular2-universal/blob/82afbb5781165081e6167e4679a89ed1caeba8b6/ClientApp/app/app.module.ts#L52
What should it match?
The selector of the Server NgModule bootstrapped component?

This is just used to match up the IDs created during the server-render with the client-rendered version so that styles get reused/etc, otherwise you'd get a flicker from it.

But where in the Server NgModule is this set? Can't find it anywhere 😕
Is it this one?
https://github.com/MarkPieszak/aspnetcore-angular2-universal/blob/82afbb5781165081e6167e4679a89ed1caeba8b6/ClientApp/app/app.module.server.ts#L19


pass option all the way to the backend

Hi @john7doe , thanks for submitting the PR ! We've set up a process on how to contribute to the CLI repo, and now, PR submitters should include a description of the change they would like to include in the changelog. Each time a PR is merged and the next version of the CLI is released, the first paragraph in the PR description will be copied into the changelog. Could you include a description of the change in the PR description?


fix: add overflow-y-auto and no-scrollbar to project list container

Pull Request Summary

Added overflow-y-auto and no-scrollbar to the div element to get the desired functionality
fixes: https://github.com/aadeshkulkarni/first-issues/issues/8

https://github.com/aadeshkulkarni/first-issues/assets/22127725/345ae10c-536f-4b60-83da-1254e7ba820f

This is already fixed.


log certificate version on deploy

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

You may need to update some unit tests.


Remove redundant properties from sonar-project configuration

The commit removes superfluous properties from 'sonar-project.properties'. The dimensions 'sonar.sources' and 'sonar.tests' were deleted to improve the configuration management of the project, focusing on the essential parameters only. New configuration should streamline the project configuration and management process.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.745%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3151

Relevant Lines:
3257

💛  - Coveralls


failed to instrument with error: 'import' and 'export' may appear only with 'sourceType: module'

I'm trying to run coverage through this flow: TypeScript -> ES6 -> Buble -> ES5. All tests pass but I get these errors and Unknown coverage. What could be the cause?
Expected Behavior
Should run coverage
Observed Behavior
Doesn't run coverage, fails with the above error
Bonus Points! Code (or Repository) that Reproduces Issue
Forensic Information
Operating System: windows 10
Environment Information: information about your project's environment, see instructions below:
node v6.7.0
npm v3.10.3

I'm also seeing this issue with Typescript. How did you solve this?

@LukeSheard Sourcemaps were not built correctly, resulting in an attempt to parse the import statements.


Integrate asset-sync w/ azure-sdk-for-go

Right now, the start on record/playback POSTS to the playback/start or record/start with a json body that looks like:
{
  "x-recording-file": "<relative path from repo root to test file>"
}

The adjustment to the framework is the following.

Starting from the test path, begin traversing upwards along the directories. Checking each level for a assets.json file.
If an assets.json file is found. Save that path.
During creation of the POST to start, add the found assets.json path as an argument, either relative or absolute.
If the repo root is reached without finding an assets.json, abandon trying to find one. Don't send the x-recording-assets-file key in the POST body.

Example modified post
// relative
{
  "x-recording-file": "<relative path from repo root to test file>"
  "x-recording-assets-file": "sdk/aztables/assets.json"
}
// absolute
{
  "x-recording-file": "<relative path from repo root to test file>"
  "x-recording-assets-file": "C:/repo/sdk-for-go/sdk/aztables/assets.json"
}

@scbedd I think we may need to be more flexible regarding relative vs. absolute paths. Relative paths work for a containerized proxy, but absolute paths work for a local process proxy. From the perspective of the test framework, we don't really know which mode the proxy may be running in. I'm musing over a few options:

Add a new proxy route that returns some information about the proxy host (e.g. container vs. not)
Add a new proxy route that returns information about which kind of path is desired (absolute vs. relative)
Support posting both absolute and relative paths and letting the proxy try both
Try posting an absolute path from the test framework, if there's a not found failure, try posting a relative path.


Lab 8 Level 3

Hello, I am currently struggling with takeSword and how to define it. How should calling the method affect the state change in the Sword class? Also, how do I make the lambda expression only print one statement and not multiple at once, even thought the tick method will call the takeSword method multiple times (depending on how many objects there are in the room)?

i think it may help for you to define a helper function for takeSword which takes in a room and outputs a new room (which contains the updated state of the sword). You can update the state of the sword to identify that it has already been taken, then add the updated sword to the new room you are returning and also make use of System.out.println() to print the statements

Hey I think you can try to store a boolean in ur sword class to say whether its taken or not. As for the tick method calling takeSword multiple times, I don't think that's true(?) it doesn't call the takeSword method repeatedly, but only once for each time its mentioned in the jshell input code

hi, I would say it would be helpful to have if else statements in your takeSword to keep track of which statement is printed wrt the conditions in the room

Hi, you can add a boolean method inside your Sword class to return the status of the sword (whether it is taken or not).
i have a similar idea to @rexlionz , where my takeSword have multiple if/else statements to check for the different conditions and print out the correct message.
Hope this helps

Hey @jonyeokj
I actually got stuck at level 3 as well, you might want to take a look at this thread. It contains quite alot of info and is a good place to start looking if youre still stuck!
https://github.com/nus-cs2030/2122-s2/issues/202
Hope it helps!

Hiii i think can add a boolean isTaken() into Sword class to check if the sword is taken or not.
There are many conditions so it would be good to draw out a tree diagram to have a clearer understanding.
The conditions include:
if there is no sword ...
if there is sword -> if sword is taken
if there is sword -> if sword is not taken

Yeap i also agree with nanfang02 because thats what i did and it works! Do check that the item is a sword before using the isTaken() method too!

Hi Jon, i think this can be attributed to Since events are associated with customers and the shop associated with the servers, a particular instance of the simulation can be completely defined by the priority queue of events, as well as the status of the shop. Write the class Simulate2 with a constructor that takes in the number of servers, together with a List of customer arrival times, and constructs a priority queue of EventStub events using the arrival time as the event time, and the status of the shop.
Include a run method within the Simulator2 class that takes no arguments, such that invoking the method will cause the simulation to start by having the events in the priority queue polled one after another until the queue is empty. The run method should return a string.
Hope this helps!


Chapter 8 404 error on automatic route, 500 error on manual route.

Using the GitHub code, after Listing 8.5, I tried URL http://localhost:8072/organization-service/v1/organization/958aa1bf-18dc405c-b84a-b69f04d98d4f with Postman and got 404 returned:
{
"timestamp": "2020-09-13T17:52:13.413+0000",
"path": "/organization-service/v1/organization/958aa1bf-18dc405c-b84a-b69f04d98d4f",
"status": 404,
"error": "Not Found",
"message": null,
"requestId": "4b1374e2-6"
}
Is the above error caused by the GitHub code supporting automatic and manual routing?  When I changed the URL to use manual routing http://localhost:8072/organization/v1/organization/958aa1bf-18dc405c-b84a-b69f04d98d4f, a 500 error was returned:
{
"timestamp": "2020-09-13T17:55:50.256+0000",
"status": 500,
"error": "Internal Server Error",
"message": "could not extract ResultSet; SQL [n/a]; nested exception is org.hibernate.exception.SQLGrammarException: could not extract ResultSet",
"path": "/v1/organization/958aa1bf-18dc405c-b84a-b69f04d98d4f"
}
The latter error is caused by, "org.postgresql.util.PSQLException: ERROR: relation "organizations" does not exist"

There may be a couple of issues; one having to do with manual routing (no mapping error) and the other having to do with the organizations table (organizations does not exist).  I checked the repository code and the table in the annotation is organizations.  I followed the chapter 8 .md mvn clean and docker UP using docker/docker-compose.yml instructions.
licensingservice_1     | 2020-09-13 20:12:20.196  WARN 1 --- [nio-8080-exec-1] o.s.web.servlet.PageNotFound             : No mapping for GET /organization/v1/organization/958aa1bf-18dc405c-b84a-b69f04d98d4f
gatewayserver_1        | 2020-09-13 20:12:30.680 DEBUG 1 --- [or-http-epoll-3] c.o.gateway.filters.TrackingFilter       : tmx-correlation-id found in tracking filter: TEST-CORRELATION-ID.
organizationservice_1  | 2020-09-13 20:12:31.083  INFO 1 --- [nio-8081-exec-1] o.a.c.c.C.[Tomcat].[localhost].[/]       : Initializing Spring DispatcherServlet 'dispatcherServlet'
organizationservice_1  | 2020-09-13 20:12:31.083  INFO 1 --- [nio-8081-exec-1] o.s.web.servlet.DispatcherServlet        : Initializing Servlet 'dispatcherServlet'
organizationservice_1  | 2020-09-13 20:12:31.101  INFO 1 --- [nio-8081-exec-1] o.s.web.servlet.DispatcherServlet        : Completed initialization in 18 ms
organizationservice_1  | Hibernate: select organizati0_.organization_id as organiza1_0_0_, organizati0_.contact_email as contact_2_0_0_, organizati0_.contact_name as contact_3_0_0_, organizati0_.contact_phone as contact_4_0_0_, organizati0_.name as name5_0_0_ from organizations organizati0_ where organizati0_.organization_id=?
database_1             | 2020-09-13 20:12:31.223 UTC [54] ERROR:  relation "organizations" does not exist at character 230
database_1             | 2020-09-13 20:12:31.223 UTC [54] STATEMENT:  select organizati0_.organization_id as organiza1_0_0_, organizati0_.contact_email as contact_2_0_0_, organizati0_.contact_name as contact_3_0_0_, organizati0_.contact_phone as contact_4_0_0_, organizati0_.name as name5_0_0_ from organizations organizati0_ where organizati0_.organization_id=$1
organizationservice_1  | 2020-09-13 20:12:31.229  WARN 1 --- [nio-8081-exec-1] o.h.engine.jdbc.spi.SqlExceptionHelper   : SQL Error: 0, SQLState: 42P01
organizationservice_1  | 2020-09-13 20:12:31.229 ERROR 1 --- [nio-8081-exec-1] o.h.engine.jdbc.spi.SqlExceptionHelper   : ERROR: relation "organizations" does not exist
organizationservice_1  |   Position: 230
organizationservice_1  | 2020-09-13 20:12:31.242  INFO 1 --- [nio-8081-exec-1] o.h.e.internal.DefaultLoadEventListener  : HHH000327: Error performing load command
organizationservice_1  |
organizationservice_1  | org.hibernate.exception.SQLGrammarException: could not extract ResultSet

Please verify that the organization and license table are created in the PostgreSQL database. In order to use the services you have to create the database tables, you can find the script in the docker's folder.

I didn't have any problems with those tables in previous chapters.  I thought the database.sql script was automatically processed.  Is there something different I need to do that I haven't done previously?
I was going to also mention that with respect to routing, I also used the URL in Figure 8.14, but also got the 404 error.  I know that's unrelated to the missing organizations table.  Thank you.

The URL can change depending on the organization and license ID records you have on the database. Remember, the URL is created by using http://localhost:8072/organization-service/v1/organization/${Organization_ID}
The organization_ID is the auto-generated ID of the record in the database.
The database.sql script is not automatically processed instead you have to run it the first time you execute the docker-compose command, in other words, the first time the database container is going to be created.

Hi Illary,
Here's the resolution and someone who knows more about postgresql than me can provide an explanation.  We're using the same postgresql Docker image so one would think things would be consistent.  I am running Docker on Windows 10 with the Windows Subsystem for Linux 2 enabled.  However, this problem wouldn't seem to be related to the host machine.  Either the postgresql image requires a compatible template or the database.sql script requires the LC_COLLATE and LC_CTYPE changes I I outlined below.  Of course, there's no need to separately create the database and the tables - I only did that for troubleshooting.
The database.sql script creates the database as follows:
CREATE DATABASE ostock_dev
WITH
OWNER = postgres
ENCODING = 'UTF8'
LC_COLLATE = 'en_US.UTF-8' <- change to 'en_US.utf8'
LC_CTYPE = 'en_US.UTF-8' <- change to 'en_US.utf8'
TABLESPACE = pg_default
CONNECTION LIMIT = -1;
The above changes allow the ostock_dev database to be created and the tables to be created in the right place.
The LC_COLLATE and LC_CTYPE values are considered to be incompatible as you can see in the next two create database commands (nice when error messages provide a working solution!):
C:\Users\ezazpi\Documents\IntelliJ\manning-smia\chapter9>echo create database ostock_dev with owner = postgres encoding = 'UTF8' LC_COLLATE = 'en_US.UTF-8' LC_CTYPE = 'en_US.UTF-8' TABLESPACE = pg_default CONNECTION LIMIT = -1; | docker exec -i 2352673e6083 psql -U postgres
ERROR:  new collation (en_US.UTF-8) is incompatible with the collation of the template database (en_US.utf8)HINT:  Use the same collation as in the template database, or use template0 as template.
C:\Users\ezazpi\Documents\IntelliJ\manning-smia\chapter9>echo create database ostock_dev with owner = postgres encoding = 'UTF8' LC_COLLATE = 'en_US.utf8' LC_CTYPE = 'en_US.UTF-8' TABLESPACE = pg_default CONNECTION LIMIT = -1; | docker exec -i 2352673e6083 psql -U postgres
ERROR:  new LC_CTYPE (en_US.UTF-8) is incompatible with the LC_CTYPE of the template database (en_US.utf8)HINT:  Use the same LC_CTYPE as in the template database, or use template0 as template.


Triggering CORS pre-flighting on firefox

The x-requested-with header set in https://github.com/jedmao/iso-http/blob/master/lib/browser/Http.ts#L18 causes firefox to perform a CORS request where it otherwise wouldn't need to.
This adds ~100ms to XHR requests for mobile users. Does it make sense for clients that want this behavior to specify it themselves rather than baking it into the library?

http://stackoverflow.com/questions/8163703/cross-domain-ajax-doesnt-send-x-requested-with-header explains what jQuery does in the same situation and why.


ARROW-3163: [Python] Add missing Cython dependency to source package

Codecov Report

Merging #2508 into master will increase coverage by 1.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2508      +/-   ##
==========================================
+ Coverage   86.28%   87.44%   +1.15%
==========================================
  Files         308      246      -62
  Lines       47090    43420    -3670
==========================================
- Hits        40631    37968    -2663
+ Misses       6387     5452     -935
+ Partials       72        0      -72

Impacted Files
Coverage Δ

rust/src/record_batch.rs

go/arrow/datatype_nested.go

rust/src/array.rs

go/arrow/math/uint64_amd64.go

go/arrow/internal/testing/tools/bool.go

go/arrow/internal/bitutil/bitutil.go

go/arrow/memory/memory_avx2_amd64.go

rust/src/error.rs

rust/src/lib.rs

go/arrow/math/uint64_avx2_amd64.go

... and 52 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 20c0405...07c580c. Read the comment docs.

Agreed with @xhochy. There doesn't seem to be a reason for this PR.

Thanks. I didn't know PEP 518.
I've added pyproject.toml and confirmed that it works well with the latest pip.
By the way, can we remove setup_requires from setup.py? Or should we keep it for old distutils (pip?)?