[IOTDB-3765] Reinforce DataNode startup process

The startup process of DataNode now only needs one RPC communication and we start the RPC service of DataNode in the end of startup process such that the client can get service as soon as connected.
The tests result are as following:

Start 3C3D:

Stop 2D:

Restart 2D:

Who will add show cluster command in cluster IT to make sure that the cluster is ready for testing.

IOTDB-3825


Bug fix conv function parser string failure return wrong result (#40530)

Proposed changes
Issue Number: close #39618
cherry-pick from master (#40530)

run buildall


Create separate input type for update operations

Input types are now left to the user as of 619d433677573c8b432613c02b27c480d8b7b4a9.


Plans for integrations with messenger apps?

The current SMS questionaire support seems suitable to be expanded to also work with messengers like Telegram.
Are there already any plans for such?
While SMS is still a good fallback, messengers are vastly more popular these days and probably also easier to set up.

hi @poVoq, I can confirm that we have no plans for non-SMS messenger integration, but I agree it's a useful feature idea. we'd welcome a well-documented pull request.


Correct way to add parameters in objects

Hello,
I am trying to add user defined parameters to the objects computed by the function segmentation_to_objects and I am doing it by iterating through the objects list and adding the parameter as obj.properties = {"seg_id": seg_id, "embedding": embeddings[embedding]} . If I print an object after this, I can see my parameter which got added, and it's also visible when I print a tracklet after computing the tracks with tracker.track(tracking_updates=TRACKING_UPDATES). My TRACKING_UPDATES list has the visual value in it. But even if I do not add any custom feature to my objects, the results are exactly the same as of when the custom features are added with the process specified above. So I am wondering if I am missing any step in between or do I have to keep something else is mine within this workflow. My dataset is Hela from CellTrackingChallenge. Posting the script output below:
Without adding custom parameters:
[INFO][2023/05/11 12:44:48 PM] Localizing objects from segmentation...
[INFO][2023/05/11 12:44:50 PM] Objects are of type: <class 'dict'>
[INFO][2023/05/11 12:44:50 PM] ...Found 1107 objects in 84 frames.
[INFO][2023/05/11 12:44:50 PM] Loaded btrack: E:\Pythoncodes\venv1\lib\site-packages\btrack\libs\libtracker.DLL
[INFO][2023/05/11 12:44:50 PM] btrack (v0.5.0) library imported
[INFO][2023/05/11 12:44:50 PM] Starting BayesianTracker session
[INFO][2023/05/11 12:44:50 PM] Loading configuration file: ./cell_config.json
[INFO][2023/05/11 12:44:50 PM] Objects are of type: <class 'list'>
[INFO][2023/05/11 12:44:50 PM] Starting tracking...
[INFO][2023/05/11 12:44:50 PM] Update using: ['VISUAL']
[INFO][2023/05/11 12:44:50 PM] Tracking objects in frames 0 to 84 (of 84)...
[INFO][2023/05/11 12:44:50 PM]  - Timing (Bayesian updates: 0.00ms, Linking: 0.00ms)
[INFO][2023/05/11 12:44:50 PM]  - Probabilities (Link: 0.00000, Lost: 0.00000)
[INFO][2023/05/11 12:44:50 PM] SUCCESS.
[INFO][2023/05/11 12:44:50 PM]  - Found 1024 tracks in 84 frames (in 0.0s)
[INFO][2023/05/11 12:44:50 PM]  - Inserted 50 dummy objects to fill tracking gaps
[INFO][2023/05/11 12:44:50 PM] Loading hypothesis model: cell_hypothesis
[INFO][2023/05/11 12:44:50 PM] Calculating hypotheses (relax: True)...
[INFO][2023/05/11 12:44:50 PM] Setting up constraints matrix for global optimisation...
[INFO][2023/05/11 12:44:50 PM] Using GLPK options: {'tm_lim': 60000}...
[INFO][2023/05/11 12:44:50 PM] Optimizing...
[INFO][2023/05/11 12:44:57 PM] Optimization complete. (Solution: optimal)
[INFO][2023/05/11 12:44:57 PM]  - Fates.FALSE_POSITIVE: 73 (of 1024)
[INFO][2023/05/11 12:44:57 PM]  - Fates.LINK: 858 (of 1799)
[INFO][2023/05/11 12:44:57 PM]  - Fates.DIVIDE: 34 (of 852)
[INFO][2023/05/11 12:44:57 PM]  - Fates.INITIALIZE_BORDER: 4 (of 43)
[INFO][2023/05/11 12:44:57 PM]  - Fates.INITIALIZE_FRONT: 11 (of 52)
[INFO][2023/05/11 12:44:57 PM]  - Fates.INITIALIZE_LAZY: 10 (of 929)
[INFO][2023/05/11 12:44:57 PM]  - Fates.TERMINATE_BORDER: 8 (of 42)
[INFO][2023/05/11 12:44:57 PM]  - Fates.TERMINATE_BACK: 46 (of 83)
[INFO][2023/05/11 12:44:57 PM]  - Fates.TERMINATE_LAZY: 5 (of 899)
[INFO][2023/05/11 12:44:57 PM]  - TOTAL: 5723 hypotheses
[INFO][2023/05/11 12:44:57 PM] Completed optimization with 166 tracks
[INFO][2023/05/11 12:44:57 PM] Writing out CSV files to: ../results_updated/1/results_v1.csv
[INFO][2023/05/11 12:44:58 PM] Ending BayesianTracker session
Adding custom parameters:
[INFO][2023/05/11 12:48:07 PM] Localizing objects from segmentation...
[INFO][2023/05/11 12:48:10 PM] Objects are of type: <class 'dict'>
[INFO][2023/05/11 12:48:10 PM] ...Found 1107 objects in 84 frames.
[INFO][2023/05/11 12:48:10 PM] Loaded btrack: E:\Pythoncodes\venv1\lib\site-packages\btrack\libs\libtracker.DLL
[INFO][2023/05/11 12:48:10 PM] btrack (v0.5.0) library imported
[INFO][2023/05/11 12:48:10 PM] Starting BayesianTracker session
[INFO][2023/05/11 12:48:10 PM] Loading configuration file: ./cell_config.json
[INFO][2023/05/11 12:48:10 PM] Objects are of type: <class 'list'>
[INFO][2023/05/11 12:48:10 PM] Starting tracking...
[INFO][2023/05/11 12:48:10 PM] Update using: ['VISUAL']
[INFO][2023/05/11 12:48:10 PM] Tracking objects in frames 0 to 84 (of 84)...
[INFO][2023/05/11 12:48:10 PM]  - Timing (Bayesian updates: 0.00ms, Linking: 0.00ms)
[INFO][2023/05/11 12:48:10 PM]  - Probabilities (Link: 0.00000, Lost: 0.00000)
[INFO][2023/05/11 12:48:10 PM] SUCCESS.
[INFO][2023/05/11 12:48:10 PM]  - Found 1024 tracks in 84 frames (in 0.0s)
[INFO][2023/05/11 12:48:10 PM]  - Inserted 50 dummy objects to fill tracking gaps
[INFO][2023/05/11 12:48:10 PM] Loading hypothesis model: cell_hypothesis
[INFO][2023/05/11 12:48:10 PM] Calculating hypotheses (relax: True)...
[INFO][2023/05/11 12:48:10 PM] Setting up constraints matrix for global optimisation...
[INFO][2023/05/11 12:48:10 PM] Using GLPK options: {'tm_lim': 60000}...
[INFO][2023/05/11 12:48:10 PM] Optimizing...
[INFO][2023/05/11 12:48:17 PM] Optimization complete. (Solution: optimal)
[INFO][2023/05/11 12:48:17 PM]  - Fates.FALSE_POSITIVE: 73 (of 1024)
[INFO][2023/05/11 12:48:17 PM]  - Fates.LINK: 858 (of 1799)
[INFO][2023/05/11 12:48:17 PM]  - Fates.DIVIDE: 34 (of 852)
[INFO][2023/05/11 12:48:17 PM]  - Fates.INITIALIZE_BORDER: 4 (of 43)
[INFO][2023/05/11 12:48:17 PM]  - Fates.INITIALIZE_FRONT: 11 (of 52)
[INFO][2023/05/11 12:48:17 PM]  - Fates.INITIALIZE_LAZY: 10 (of 929)
[INFO][2023/05/11 12:48:17 PM]  - Fates.TERMINATE_BORDER: 8 (of 42)
[INFO][2023/05/11 12:48:17 PM]  - Fates.TERMINATE_BACK: 46 (of 83)
[INFO][2023/05/11 12:48:17 PM]  - Fates.TERMINATE_LAZY: 5 (of 899)
[INFO][2023/05/11 12:48:17 PM]  - TOTAL: 5723 hypotheses
[INFO][2023/05/11 12:48:17 PM] Completed optimization with 166 tracks
[INFO][2023/05/11 12:48:17 PM] Writing out CSV files to: ../results_updated/1/results_v1.csv
[INFO][2023/05/11 12:48:19 PM] Ending BayesianTracker session

Hi @Sagnik700 - cool that you're trying this out. This feature is a bit experimental, to say the least. One immediate thought is that you need to tell the tracker which visual features to use, something like:
with btrack.BayesianTracker() as tracker:
    ...
    tracker.tracking_updates = ["MOTION", "VISUAL"]
    tracker.features = ["embedding",]

LMK if you still don't see any difference.
Also, I see that you're using v0.5.0, so it may be worth upgrading to the latest version, incase there was a bug in that earlier version.

Hey @quantumjot , thanks for the quick reply. I tried out what you suggested and also updated the version, but the issue still persists. Also, even if I put some fictitious string in the tracker.features like tracker.features = ["random_something",] it does not throw any error. I have putting this tracker.features line before tracker.track. Let me know if you can thing of some other step to understand more about this issue. Thanks in advance!!

Ah, it's critical that this line (tracker.features = ...) is before tracker.append(objects)

@quantumjot ok I get you. Now I can see some differences, thanks. But the number of tracks generated are a lot, so I think I have play around with the HypothesisModel parameters a little to find a good result. Is there anything else you would suggest?


chore: rename BIP32Params to BIP44Params

Description
Rename BIP32Params to BIP44Params
⚠ Impact on web, dedicated PR coming.
Notice
Before submitting a pull request, please make sure you have answered the following:

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Do all new and existing tests pass? Does the linter pass?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[x] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
closes #132

PR on web https://github.com/shapeshift/web/pull/567

@jdambron - thanks again for the PR!
I think there may be a few things left to refactor here.

lib/packages/chain-adapters/src/utils/bip32.ts file should be renamed
lib/packages/chain-adapters/src/bip32.test.ts file should be renamed
lib/packages/chain-adapters/src/utils/index.ts update the export for the new file name

@reidrankin - it looks like we need a similiar ticket in hdwallet-core for bip32ToAddressNList, can you confirm?

@0xean renaming of the files :heavy_check_mark:

We need to be careful here because this is a backwards incompatible change. The obvious solution would be to do a major version bump on the package.


Scrolling Organizations table stops suddenly on iOS 12 Safari

See https://twitter.com/felixkiss/status/1208808587054718977

Fixed: c322e813f09cc0a505b3c660b49fec345c0b4a39 👍


Add symfony dom crawler and browser kit to ignoreFiles for stack trace

Before:

After:

@nunomaduro is there any release planned? Is something blocking where I can maybe help?

cc @owenvoke


spawnObject: should always spawn an inspector

For example:
GtInspectorTool forObject: 42

should spawn an inspector on the tool object, not an inspector on 42.

The issue is here:
spawnObject: anObject
	(anObject isKindOf: GtPhlowAbstractTool)
		ifTrue: [ self spawnTool: anObject ]
		ifFalse: [ self spawnTool: (GtPhlowTool default object: anObject) ]

We should consider also cases in Phlow that might look like this:
gtMyViewFor: aView
  <gtView>
  ^ aView list
     items: [ self myItems ];
     send: [ :anItem | MyPhlowTool forItem: anItem ]

It works now:


Add Dodgecoin as a cryptocurrencies

Ref: https://www.coingecko.com/en/coins/dogecoin
This adds Dodgecoin as an option in the cryptocurrency widget.

@all-contributors please add @pinkahd to maintenance


Missing Documentation for Schema API

Schema API documentation page is 404
https://ibm.github.io/event-streams/schemas/schema-api-reference/

Hello @snaum, thank you so much for letting us know!
We will aim to fix this soon, in the meantime you can find the schema JavaDoc here: https://ibm.github.io/event-streams/schema-api/ which can be found in the APIs dropdown in the documentation.
You can also use our CLI provided with Event Streams to interact with schemas.

Thanks for reporting this issue, this is now fixed:
https://ibm.github.io/event-streams/schemas/overview/#serialization-and-deserialization


Migrate VAD, Speech Profile to GCP

Migrate VAD, Speech Profile to GCP, currently use Modal(modal.com)

VAD: https://github.com/BasedHardware/omi/blob/main/backend/modal/vad_modal.py
Speech Profile: https://github.com/BasedHardware/omi/blob/main/backend/modal/speech_profile_modal.py

following ticket #1244


closing anchor tag for"Group Resubscribe" is missing

Description of the change:
closing anchor tag symbol is missing
Reason for the change:
Previous
<a href="#groupresubscribe"Group Resubscribe
after
Group Resubscribe
Link to original source:

Signed-off-by: inyee786<EMAIL_ADDRESS>Closes #

Hello @inyee786,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Unable to decode StreamFlags: 3212032

I am not 100% sure what caused this panic, but potentially, it was a commit to a git repo that was being monitored for fs events. This is the backtrace.
thread '<unnamed>' panicked at 'Unable to decode StreamFlags: 3212032', ../src/libcore/option.rs:700
stack backtrace:
   1:        0x10896913e - std::sys::backtrace::tracing::imp::write::h29f5fdb9fc0a7395
   2:        0x10896ca2f - std::panicking::default_hook::_{{closure}}::h2cc84f0378700526
   3:        0x10896bd8b - std::panicking::default_hook::hbbe7fa36a995aca0
   4:        0x10896c397 - std::panicking::rust_panic_with_hook::h105c3d42fcd2fb5e
   5:        0x10896c254 - std::panicking::begin_panic::hbf62ea4a5ff3f9de
   6:        0x10896c162 - std::panicking::begin_panic_fmt::h20f5943904e5791d
   7:        0x10896c0c7 - rust_begin_unwind
   8:        0x108996c7d - core::panicking::panic_fmt::h19323e466869c656
   9:        0x108996ced - core::option::expect_failed::h5c893070d569702e
  10:        0x1088c7393 - _<core..option..Option<T>>::expect::h3fd640de5cdb8714
  11:        0x1088f9861 - notify::fsevent::callback::ha08d529e8690f9e3
  12:     0x7fff9bce2b0a - implementation_callback_rpc
  13:     0x7fff9bce06c9 - _Xcallback_rpc
  14:     0x7fff9bce079d - FSEventsD2F_server
  15:     0x7fff9bce5fd8 - FSEventsClientProcessMessageCallback
  16:     0x7fff9b34112b - __CFMachPortPerform
  17:     0x7fff9b341018 - __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE1_PERFORM_FUNCTION__
  18:     0x7fff9b340f88 - __CFRunLoopDoSource1
  19:     0x7fff9b3389ba - __CFRunLoopRun
  20:     0x7fff9b337ed7 - CFRunLoopRunSpecific
  21:     0x7fff9b3799b0 - CFRunLoopRun
  22:        0x1088fbe47 - notify::fsevent::FsEventWatcher::run::_{{closure}}::h7c9524f5a4947436
  23:        0x1088f859c - _<std..panic..AssertUnwindSafe<F> as core..ops..FnOnce<()>>::call_once::h826e04f275096133
  24:        0x1088fbb54 - std::panicking::try::_{{closure}}::_{{closure}}::hb09551845c8a0008
  25:        0x1088cbcbc - std::panicking::try::call::h47f6979155451676
  26:        0x10896d02a - __rust_maybe_catch_panic
  27:        0x1088fba6e - std::panicking::try::_{{closure}}::hc2071c10857d4c73
  28:        0x1088d09e5 - _<std..thread..local..LocalKey<T>>::with::h6c80b5d40b886fef
  29:        0x1088cbc0d - std::panicking::try::h9b03b8b74b847b38
  30:        0x1088cad50 - std::panic::catch_unwind::h14c9291bec764a56
  31:        0x1088fb7b2 - std::thread::Builder::spawn::_{{closure}}::heff91d698bfca38b
  32:        0x1088e9db0 - _<F as alloc..boxed..FnBox<A>>::call_box::h9a4ccf94f4cabe77
  33:        0x10896b405 - std::sys::thread::Thread::new::thread_start::h8f3bd45211e9f5ea
  34:     0x7fff9690399c - _pthread_body
  35:     0x7fff96903919 - _pthread_start

Ah, this is the same bug as before I believe. Need to update my version. Sorry for the spam :)


Update diagrams on lesson 5 module reuse

Update the slides that have the graphviz diagrams with PowerPoint diagrams with the new format.

Closing this for now. The first training was given, so let us re-evaluate whether this needs to be changed here or in the hands-on.


Don't crash when attempting scratch/simple build

Related to #429

We've talked about getting rid of build types before. I wonder if we should change Simple to Scratch?

@twaugh I'm working on that but it's not ready yet - I realized we probably need to distinguish build types at least for the purpose of input validation (e.g. prod build needs the sources command but simple build doesn't). Or should this be the responsibility of koji-containerbuild?
This patch is supposed to un-break simple builds in the meantime.


Move cleaning stale outputs to execution engine (take 2)

This is to move more logic into the execution engine that can be abstracted over different types of work.
Fixes https://github.com/gradle/gradle/issues/9113

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge please.

@bot-gradle cancel merge.

OK, I've already cancelled the pre-tested commit build.

@bot-gradle test this please.

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge please.

Your PR is queued. See the queue page for details.

@bot-gradle test and merge please.

OK, I've already triggered a build for you.


Can I rolate x label chart?

my x label is date so it's so long :(.
Thank!

sure, here is a demo http://plnkr.co/edit/j2KAcfI3Givae7ce4A2o?p=preview

thank @chaosmail. Sorry if I bothers you!

@truyen93 this doesn't bother me at all, you are always welcome to ask questions if you are stucked or if you found a bug :)
thanks for using n3-line-charts btw. people like you are the reason why we are working on this library

thank @chaosmail I seem that my chart was duplicate x label twice (it's date type).
Do you have any ideal?
my data:
$scope.dataTest = [{
"hits" : 41559,
"sum_unBlendedCost" : "868.8720307299947",
"avg_blendedRate" : "0.0542105187318246",
"key" :<PHONE_NUMBER>000,
"sum_blendedCost" : "859.3602748399655"
}, {
"hits" : 41974,
"sum_unBlendedCost" : "1005.5889742999409",
"avg_blendedRate" : "0.055651289718932094",
"key" :<PHONE_NUMBER>000,
"sum_blendedCost" : "976.9996922699304"
}, {
"hits" : 44481,
"sum_unBlendedCost" : "938.18124664995",
"avg_blendedRate" : "0.05294738550402792",
"key" :<PHONE_NUMBER>000,
"sum_blendedCost" : "907.1317417899413"
}];

ops. I fixed that by set ticks. It's ok if ticks not equal 6 :+1:

alright, here I put together some cool tick formatting and interval features http://plnkr.co/edit/Ujl3eqAkInFRwxxCUhHI?p=preview


⚠ Bofrost.be has degraded performance

In 6db426b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 960 ms

Resolved: Bofrost.be performance has improved in 465d3ea after 6 minutes.


Extension search by file extension (ext:) also matches against tags

Spin-off from https://github.com/microsoft/vscode/issues/139321#issuecomment-1004225117.

ext:production is converted this following tags filters: __ext_production, __ext_.production, production. MP is returning all extensions that contain at least one of these tags.

I suggest to audit if broadening the extension query to tags is more helpful (extensions that have defined the relevant tags but not file extension) or counter-productive (extensions are shown that have nothing to do with the searched file extension).
In case of the linked issue, I see it as more harmful. If there is no common cases where it helps to find the right extensions, I would recommend to make the query less broad by not including tags.

I would appreciate if there are any constructive recommendations or suggestions.

@digitarald do you have some suggestions on how to improve this? I do not unfortunetly.
We can also keep the discussion in #139321

@digitarald do you have some suggestions on how to improve this? I do not unfortunetly.

My recommendation would be to just search in the extension fields, but not including the tag in the search, so resolving ext:example is resolved into __ext_example, __ext_.example, but not adding ~example~ (using the format from the original comment https://github.com/microsoft/vscode/issues/139321#issuecomment-1004225117).
As I am not 100% familiar with the format of the marketplace search, did I capture the idea in a sensible format, @sandy081?

I am not sure what will we miss if example tag is excluded in this case. Will there be an extension with this tag that supports this language? Actually I do not know why this was like this as it was there from beginning even before I took over. It seems we need to study what happens if we avoid this tag

No action items I see from this item thus closing.
We can reopen in the future if we figure out what to do here


[Feature] A way for foreign citizens, vacinated in the home country (!=Germany) to be able to import vacinnation confirmation

User Story
A foreign citizen, living and working in Germany, that got vaccinated outside Germany, with one of the EMA-approved Covid-19 vaccines, should be able to import his/her vaccination certificate into the corona-warn-up.
Different countries already have a confirmation system, and QR codes generated as a validation of vaccination that took place in the country (Serbia is only one of numerous examples).
corona-warn-app should have at least a possibility to acknowledge the fact that not all vaccinations for persons living and working in Germany took place in Germany, and to have a way (either programmatic or procedural), to get those foreign vaccination certificates recognized.
We do that already normally by a house doctor for other vaccines, so the idea is that we don't discriminate people that got their shots, just because they got it somewhere else / outside the EU (quicker that they would have in EU/Germany itself)...
Some ideas to achieve this
Option 1 Let selected House Doctor (or Gesundheitsamt) have a possibility to see, verify and generate QR code for digital vaccination certificate in Germany
This may be a reasonable solution, as it will follow the same pattern we have for any other vaccines.
Of course, the verification person (House Dr) needs to have a way to actually confirm and generate the code for a person. Record will state who validated/generated the QR code (so they are of course responsible not to insert false data).
Technically, HA needs to have that possibility and a place where he/she will enter that data so that the QR code for a patient is generated.
Option 2 Let corona-warn-app support team to do verification
Same as option 1, just the checking of the vaccination state would be done by the support staff (questionable if this would be lawfully allowed).
Option 3 Integrate w/ other verification systems
This may not be realistic option, as lot of countries would have bodies (like local Ministry of Health) that is giving a confirmation upon scanning issued QR verification code. In this case corona-warn-app needs to make sure scanned QR code is indeed verified, which can be cumbersome on so many levels when integrating w/ other systems.

@anaperic
I would check with your local pharmacy next week if they can issue a digital certificate for vaccinations done outside Germany and outside the EU, like in Serbia.
There is an FAQ list on https://digitaler-impfnachweis-app.de/faq/ but it doesn't seem to cover this topic at the moment.
This is all ramping up at the moment, so some of it is still unclear.

I've brought this to then attention of the development team. This will have to be discussed with RKI. We'll investigate if this is possible to implement.

Corona-Warn-App Open Source Team

Here's a use case: I'm a German citizen but have lived in the USA for >10 years. I got fully vaccinated in the US over a month ago using the Pfizer/BioNTech vaccine. I often visit my family in Germany, and an traveling there today for the next three weeks. Yet all I have is a CDC paper record by a US healthy authority. Wouldn't it be great if someone recognized that and allowed me to digitize this certificate for use in Germany during my stay?

Just an update from my side. The variation of Option1 that I wrote in the ticket is already now working properly.
Instead of HA as a "verification point" it is the pharmacy. I managed today to get the EU Impfzertifikat in one of the pharmacies in town, and of course was able to import it to Corona-Warn App.
What would be great to find out is for which non-EU countries this process would work (and I am well aware this is more the question for RKI), but this kind of transparency would be good to have in the future.

@anaperic
Thanks for getting back to the community that you were successful!

I can contribute a contrary anecdata point: my sister-in-law from Hong Kong is visiting me in Germany, and I tried to get her a certificate at the pharmacy, they immediately said they cannot issue certificates for any foreign vaccination document.
After some research, EU regulation apparently says that it is in fact possible, but only if the person in question is not planning to return to the country they were vaccinated in: https://www.deutsche-apotheker-zeitung.de/news/artikel/2021/06/22/auch-impfungen-im-ausland-duerfen-jetzt-digitalisiert-werden
Additionally, there are a number of non-EU countries whose certificates are mutually accepted https://ec.europa.eu/info/live-work-travel-eu/coronavirus-response/safe-covid-19-vaccines-europeans/eu-digital-covid-certificate_de#anerkennung-von-covid-zertifikaten-aus-drittlndern-nicht-eu-lndern


Entered incorrect email and now can't fix it

Describe the bug
I entered my email incorrectly and now my newly-created account can't be verified and I can't add a new email either.
Expected behavior
The website says I can add a new address but no option is to be seen.
To Reproduce
Create an account with an invalid email address and try to add a functional email.
My Platform
Chrome, Windows 11
Additional context
Using a @users.noreply.github.com email address.

Please see https://pypi.org/help/#account-recovery.


Add new display language

Hi,
I would like to see Visual Studio Code in Persian language.
I want to translate it to Persian language.
Any idea how can I do this?
Thanks.

Besides the translation there is the fact that Persian is a RTL writing system.

I know, and I think it should add an option for RTL languages like Persian.

We don't have plan to enable additional language for VSCode in MLCP in near term.
New request to enable language in MLCP should be logged in Localization Community Repo

Close this issue as the reasons stated above.


Handling Collisions with Limited 3x3 FOV

Hello!
First off, thank you for your work on the SCRIMP paper. It presents an intriguing approach to the challenges problem.
One aspect I'm curious about is how SCRIMP handles potential collisions with a very restricted 3x3 FOV. To paint a picture, consider a scenario where two agents (agents A [moving right] and B [moving left]) are in a narrow corridor with just a single cell between them, such that they're outside each other's FOVs:
#####
.A.B.
#####

Given their limited visibility, how does the system ensure these agents avoid collisions in real-time? Does the global communication mechanism based on the modified transformer help in such close-quartered scenarios, or are there other components of the system that play a role in avoiding potential conflicts in such situations?

Hello!
I have not tried to visualize how the model works in such narrow corridor problem. But I think both the global communication and the tie-breaking strategy proposed in the paper can deal with this situation. In particular, before a collision occurs, the tie-breaking strategy will force only one agent to be allowed to perform its original action, while the other agent will perform a new valid action. In this example, if A and B will meet in the blank grid in the middle of the corridor, and assuming that B is always allowed to perform its original action (left), then the valid action for A will be left and stay. If A choose stay, then in the next step, the only valid action for A is left.  Eventually, B will walk through the corridor, while A will continue to walk back until it is completely out of the corridor. But that's just a perfect assumption; in an actual task, B might not always be the favoured agent, and then A and B might be stuck in the corridor until the end of the episode. So, my future work will be on how to choose the right agent and allow agents to perform more than just one action when breaking the breaking.

Thank you for your detailed response. I am particularly intrigued by the specifics of the communication and tie-breaking mechanisms in SCRIMP, especially within a decentralized framework. Could you provide more details on the type of information exchanged between agents during these phases? Additionally, is the communication process based on multiple rounds, or does it utilize a global information channel?
On another note, I recommend exploring lookahead techniques like Monte Carlo Tree Search  to further refine the tie-breaking strategy. Our recent paper https://arxiv.org/pdf/2312.15908.pdf delves into similar methodologies and might provide some valuable insights for your research.

Hello author,when installing modules according to txt in readme, networkx2.8.8 is inconsistent with python3.7. How can I solve it? Thank you


[flatiron] jitsu users missing help

Instead of getting general help, we should be seeing help specific to the "users" resource.
% jitsu users
info:   Welcome to Nodejitsu
info:   It worked if it ends with Nodejitsu ok
info:   Executing command users
help:             ___  __
help:       /  /  /   /_  /  /
help:    __/  /  /   __/ /__/
help:
help:   Flawless deployment of Node.js apps to the cloud
help:   open-source and fully customizable.
help:   https://github.com/nodejitsu/jitsu
help:
help:   Usage:
help:
help:     jitsu <resource> <action> <param1> <param2> ...
help:
help:   Common Commands:
help:
help:   To install a pre-built application
help:     jitsu install
help:
help:   To sign up for Nodejitsu
help:     jitsu signup
help:
help:   To log into Nodejitsu
help:     jitsu login
help:
help:   Deploys current path to Nodejitsu
help:     jitsu deploy
help:
help:   Creates a new application on Nodejitsu
help:     jitsu create
help:
help:   Lists all applications for the current user
help:     jitsu list
help:
help:   Additional Commands
help:     jitsu apps
help:     jitsu logs
help:     jitsu env
help:     jitsu conf
help:     jitsu users
help:     jitsu databases
help:     jitsu snapshots
help:     jitsu logout
help:
help:   Options:
help:     --version, -v    print jitsu version and exit                                               [string]
help:     --localconf      search for .jitsuconf file in ./ and then parent directories               [string]
help:     --jitsuconf, -j  specify file to load configuration from                                    [string]
help:     --noanalyze      skip require-analyzer: do not attempt to dynamicially detect dependencies  [boolean]
help:     --colors         --no-colors will disable output coloring                                   [boolean]  [default: true]

Duplicate of #157


Automatic persistence ?

Hi,
The question is simple : is there any way to persist an object into solr automatically ?
Something like annotations or even something in the configs files ?
And if not : are there any plan to make it ?

Closing this one after almost 11 years. Seems there is not much interest around adding this into the Bundle.


Can this be used without async/await?

I tried removing the async/await and got this error:
with websockets.connect('...') as websocket:
AttributeError: __exit__

@roman-ku since it is based on asyncio you can't use without async/await....
why are you trying to remove them ?

Yes, you can use websockets without async / await. You'll have to use @asyncio.coroutine and yield from instead. This is generally less convenient than async / await. The only reason I can think of for doing that is if you're stuck on Python 3.3 (which is uncommon, since major distros ship 3.4)
If your question really was "can I use websockets in a synchronously", as @qwexvf correctly points out, websockets isn't a good choice for you. In that case, I suggest using another library, as websockets is a very small library whose sole purpose is to provide a convenient asynchronous API.

@aaugustin @qwexvf
Yes my question was, can I use websockets synchronously. I need the response from the server before doing more stuff.
Alright thanks guys. I'll look for a different library.

Is there another library to use websockets synchronously?

I'm not super familiar with the landscape of other libraries.
Since this issue was filed, I started working on a solution to this problem, see #885.


Made Cartfile private to stop carthage build errors

Currently installing Bugsnag via Carthage leads to the dreaded “Parse error: expected submodule commit SHA in output of task (ls-tree -z 1.0.0 KSCrashServer) but encountered: ”
Pretty sure this is due to KSCrashReporter being included as a submodule as well as in the Cartfile. I’ve just made the Cartfile private so it’s not pulled in as a transitive dependency - this seems to work.

Hmm, I had to bump the IPHONEOS_DEPLOYMENT_TARGET setting otherwise iTunes upload fails. Looks like tests are failing now, will try to fix.

This still results in the error on my end

@Errortype520 until there’s a new release you’ll need to use github "samritchie/bugsnag-cocoa" "master"

Unfortunately, the deployment target cannot be changed as we support iOS 6.0+, looking into this issue though.

@kattrali the deployment target does not affect CocoaPods usage - 6.0 is still possible there. It will affect Carthage, but Carthage uses dynamic frameworks and needs iOS 8+ anyway.
It will also affect the downloadable framework, but again, this is dynamic. BTW I don’t think that framework is building properly at the moment, I got missing symbols errors when I tried to use it.

BTW I don’t think that framework is building properly at the moment, I got missing symbols errors when I tried to use it.

There are missing symbols when building the workspace? Can you give an example?

It will also affect the downloadable framework, but again, this is dynamic.

Yeah, this is the part I'm concerned about - those using the downloaded framework and building for 6.0 will see some strangeness.

There are missing symbols when building the workspace? Can you give an example?

No, the built framework attached to the release is missing architectures. If you download the Bugsnag iOS zip and run:
lipo -info Bugsnag.framework/Bugsnag

you get Architectures in the fat file: Bugsnag.framework/Bugsnag are: armv7 arm64 - ie you won’t be able to build an app for the simulator if you have this framework linked.

those using the downloaded framework and building for 6.0 will see some strangeness

You can’t use the downloaded framework and build for 6.0 - dynamic frameworks require iOS 8+ (technically you can, but you won’t be able to submit to the store). If you want to support iOS 6 without CocoaPods via a downloadable binary, you’ll need to add a static library target. This can have a separate  IPHONEOS_DEPLOYMENT_TARGET.

Will it be possible to get this PR merged, or are there other changes required? I’ve merged latest into my branch and it’s all working fin from the Carthage perspective.

I think this is good, thanks for the reminder and the patience @samritchie

I'll need to update the documentation for Carthage installation, though I haven't had much success with using Carthage in the past. I'll post an update when its live.


Add gosu install

Your docker images really helped me and I learn a lot. thx.
Wondering, I used sameersbn/docker-redis image.  Its start shell is using gosu.  But, its base image, this ubuntu image, is seemed by me not to install gosu.  And, the image pulling by docker pull sameersbn/ubuntu:14.04.20150603  included gosu.
Although maybe my less knowledge caused, I think a better way is to add run step installing gosu on Dockerfile.  And, if I miss, please tell me about the amazing happening.

@mizunashi-mana Please update the docker-redis image or source. The gosu command is not being used by my images anymore. I switched to using the start-stop-daemon command instead.

Oh, i see. Thx


pull request for e2e test at 12/06/2016 07:56:02

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add alias if select field from struct column

If select a field from a struct column, like
SELECT table.struct_column.field_name FROM table

we need to add an alias to make it as:
SELECT table.struct_column.field_name AS field_name FROM table

, which can avoid null pointer exception:
java.lang.NullPointerException
        at org.apache.calcite.sql.validate.SqlValidatorUtil.getTypeEntry(SqlValidatorUtil.java:978)
        at org.apache.calcite.prepare.CalciteCatalogReader.getNamedType(CalciteCatalogReader.java:174)
        at org.apache.calcite.sql.validate.SqlValidatorImpl.getValidatedNodeTypeIfKnown(SqlValidatorImpl.java:1609)
        ...

Tests:

unit test
test on affected production views by the above exception, they could be translated well with this patch
integration test on all the production views, no regression

I am not clear how/where the NPE is thrown, but it seems it would be better to handle the NPE where it occurs for this case (struct.field) rather than tweaking the input SQL (given the fact that it is a valid SQL).

Thank you @shenodaguirguis, the root cause is in calcite, and I opened a PR for that: https://github.com/linkedin/linkedin-calcite/pull/39

If the calcite PR address the issue, could we close this one? @ljfgem


fix: :bug: made bugFix branch an active branch insted of main

I've successfully fixed the bug! Now, the bugFix branch will be displayed as the active branch instead of main.
Before :

After:

Great thanks for fixing this bug! I'm sorry for everyone who thought it was possible in one step when two were required 😅
Thanks again @Theprathamshah

Welcome @pcottle !
It has been an absolute pleasure to contribute to this project. Thank you for providing us with such an incredible platform to learn Git. I truly appreciate the opportunity, and I look forward to continuing my contributions whenever I get the chance!


Network: Adds ovn.ovs_bridge setting to bridge networks to specify OVS uplink bridge name for OVN networks

This PR introduces the ovn.ovs_bridge bridge network setting. It is used by OVN networks that specify the bridge network as its parent network in order to create an OVS uplink bridge using the setting's name (and as a name prefix for the associated veth interfaces).
This is being introduced to work around an issue when joining an node to an existing cluster that has an OVN network, because previously the internal DB network row ID was used as part of the OVS bridge name. However this caused issues during the pre-join phase because potentially a different local network ID would be used to create the OVS bridge than is used in the rest of the cluster.
This removes the use of internal DB IDs for interface names and uses config setting instead, which will be used in the pre-join phase of network creation too.

@freeekanayaka this implements the change we discussed in Trello.

Testsuite passed


implement basic get_multi

This Draft PR contains a naive implementation of krate::get_multi() for answering some architectural questions.
Questions to answer with this implementation:

[ ] Is it enough to just return anyhow::Result<Vec<Krate>> when there is a failure with one krate getting fetched?
[ ] One Krate has an internal server error or error status code. Should I still return everything? anyhow::Result<Vec<Result<Krate>>>
[ ] I still would like to implement: threaded/paralell get_multi sync, single thread multi async. Should these all be parameters of the original get_multi() function, or separate functions, or separate cfg() features.

After reading through some external feedback, I think introducing the ability to access multiple krates violates the single purpose principals. I want to keep the lib lean and single purpose. Paralellism/Concurrency can be introduced on the consumer side.

Closes #7 #9


 replacement?

Less of an issue but more of a question as it is unclear how it works.
In the sample JSON config here is a comment as per below.
 // Views. First one be default. For fetchXML, have the required FETCHXML and add <MORECONDITIONS/> at the end of the filter which will be replaced dynamically
How does the  get replaced Dynamically? what replaces it and where does that get configured?
I'm thinking of scenarios where we would want to add a filter to the lookup based on another field or lookup field on the form like we would do with a prefilter or ootb filtering config. Is that was this is intended for and if so, how does it work?

Hi,
It is getting replaced as shown in the below screenshot.

Thanks and regards,
Shivakumar Paragi


Microsoft.Authorization/policyDefinitions

Describe the noise
Resource type (i.e. Microsoft.Storage/storageAccounts)
Microsoft.Authorization/policyDefinitions
apiVersion (i.e. 2019-04-01)
2019-01-01
Client (PowerShell, Azure CLI, or API)
PowerShell
Relevant ARM Template code (we only need the resource object for the above resourceType and apiVersion, but if it's easier you can include the entire template
Expected response (i.e. "I expected no noise since the template has not been modified since the resources were deployed)
No changes to parameter types values
Current (noisy) response (either include a screenshot of the what-if output, or copy/paste the text)
  ~ Microsoft.Authorization/policyDefinitions/add-vms-to-backup [2019-01-01]

    ~ properties.parameters.imageOffers.type:        "Array" => "array"
    ~ properties.parameters.imagePublishers.type:    "Array" => "array"
    ~ properties.parameters.imageSkus.type:          "Array" => "array"
    ~ properties.parameters.policyName.type:         "String" => "string"
    ~ properties.parameters.vaultLocation.type:      "String" => "string"
    ~ properties.parameters.vaultName.type:          "String" => "string"
    ~ properties.parameters.vaultResourceGroup.type: "String" => "string"

Additional context
Add any other context about the problem here.

Yeah, we need to mark these as case-insensitive.

Need to understand why these are not marked as enum values in swagger and why this wasn't automatically detected

[w26]

This should be resolved, let us know if you still see this noise


Set y-axis scale for different sample types

I'd like to plot bw files from my ATAC-Seq samples with the same y-axis scale for all samples alongside sashimi plots for my RNA-Seq samples with variable y-axis scales. Would that be possible, or has it not been worked into the current version?

Hi there,
I'm sorry for the late response, the same y-axis currently only worked globally, cannot set by groups. I'll try to work on that in late versions.
Best

Hi,
I update a new release v0.5.0, add --same-y-by-groups, please try with the latest version.


Rule updates 2019 06.v4

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flaky-test

If contributing rules or changes to rules, please make sure to uncomment the appropriate kind

/kind rule/update

/kind rule/create

Any specific area of the project related to this PR?

/area engine
/area rules
/area deployment
/area integrations
/area examples

What this PR does / why we need it:
Improve whitelisting to reduce false positive
Which issue(s) this PR fixes:

Fixes #

add placeholder for rules write below root/etc
add weaveworks/scope to whitelist in rule change thread namespace
add openshift image to whitelist

Special notes for your reviewer:
Does this PR introduce a user-facing change?:

1. add placeholder for rules write below root/etc
2. add `weaveworks/scope` to whitelist in rule change thread namespace
3. add openshift image to whitelist

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: kaizhe
If they are not already assigned, you can assign the PR to them by writing /assign @kaizhe in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Kaizhe: There is not a label identifying the kind of this PR.
Please specify it either using /kind <group> or manually from the side menu.
In case you do not know which kind this proposal is please mention the maintainers using @team/maintainers.

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.

LGTM label has been added.  Git tree hash: e35c75f5cf1415385c6c350c08f41b927c41f460

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS [fntlnz]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz, mstemm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS [fntlnz,mstemm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: fntlnz
If they are not already assigned, you can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: fntlnz
If they are not already assigned, you can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 1ab6c7e57501c7e05cfb1a66bb4b28aa0df1c7e3

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mstemm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS [mstemm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz, mstemm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS [fntlnz,mstemm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/needs-rebase

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: fntlnz
If they are not already assigned, you can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: f898ca5190efd5edc904a5e7b69890e13c8a16d1

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mstemm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS [mstemm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: fntlnz
If they are not already assigned, you can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 25f52fc66ed31665b34cd40bb4549b0a4dc75403

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mstemm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS [mstemm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mstemm can you please merge it for me?

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz, mstemm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

rules/OWNERS [fntlnz,mstemm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


cannot handle generics in with arrow functions

Describe the bug
Deno version: 1.7.0 (master branch)
Input Code
const a = <T>(value: T): any => {};

Expected Output
No panics.
Actual Output
thread 'tokio-runtime-worker' panicked at 'Expected text `=`, but found `>`', [REDACTED]/.cargo/registry/src/github.com-1ecc6299db9ec823/dprint-plugin-typescript-0.38.5/src/parsing/parser_types.rs:132:13

Problem here is that the span of the init expression doesn't include the type parameters in swc. Opened https://github.com/swc-project/swc/pull/1365
Just a note that this panic will only happen in debug mode to catch issues like this. Formats fine in release.

Problem here is that the span of the init expression doesn't include the type parameters in swc. Opened https://github.com/swc-project/swc/pull/1365
Just a note that this panic will only happen in debug mode to catch issues like this. Formats fine in release.


Kovan net will be deprecated, it is possible to move the testnet to goerli?

Thank you!

Support optimism goerli now. Please try again.


cleanup, update to use latest gems

fixes a bunch of deprecation warnings + upgrade to latest gems

Nice. Merging.


Laravel 5.4 compatible but error in 5.3

@franzose this trait Illuminate\Console\DetectsApplicationNamespace used in the recent released version 4.1.5 which does not available in laravel 5.3. Could you please take a look?

@duongtdvn You have to use L5.3 branch: https://github.com/franzose/ClosureTable/tree/L5.3. Master is always for the actual stable Laravel version, I'm sorry.

I see, it is because of the L5.3 branch also contain the 4.1.5 tagged in release and while including "franzose/closure-table": "4.*" in composer.json it will also upgrade to the latest release.
My suggest is to lock the 4.x version to support only laravel 5.3 and lower then create a new major version 5.x for supporting laravel >=5.4?

@duongtdvn I have removed tag v4.1.5 and changed the latest release to v5.0. Hope it'll solve problems.

Please remove v4.1.5 from Packagist
https://packagist.org/packages/franzose/closure-table#v4.1.5

@Chaddles23 Done!


inclusion in Gemfile impacts application boot-up time

I've just upgraded to Rails 3.1, and even though I don't have any .sass or .scss files yet, I notice that when I add gem 'sass' to my Gemfile, application boot-up time immediately takes about 1 second longer (timed via time rake route).  In comparison, most other well-behaving gems seem to only add around 0.1s.  Happy to provide further debug on request.

Without doing any profiling, my spidey sense says that is that Regex compilation at boot time is non-trivial and could be eliminated until the first RX constant is looked up.

Lazy-loading RX, if that is indeed the problem, wouldn't be the easiest thing to do. It's referenced at load time in a number of places.
Without further debugging information, though, I don't think there's anything we can act on here.


DEV 8775.1 Downgrade terraforn to v0.8.7 for u16

https://github.com/Shippable/pm/issues/8775

this is done closing


Statement of purpose

The example in the README does not make it clear what the intention of sciluigi is, nor what the benefits would be. I'm interested in this project but would need to know more about its intentions before contributing.

Hi @averagehat and thanks for reporting/asking! I will try to add a better description of that in the weekend!

@averagehat I have added some basic information about the library now! Hope it helps!
Please see also this collection of links, to materials covering our experiences which lead up to the decision to create SciLuigi: http://bionics.it/posts/our-experiences-using-spotifys-luigi-for-bioinformatics-workflows
I think especially see the talk I gave at eInfraMPS 2015, on pipelineing with Luigi:

Slides
Video

Feel free to report if there is any essential information missing!

Closing now. Feel free to re-open, if you think more information is needed.


🚧Add symbols and xml docs to nuget package

Description:
To allow this pull request to be merged, please make sure that the following items of this pull request are complete.
This pull request closes #1036
✅The pull request head branch must be a feature branch with the syntax 'feature/-sync-testing'.
✅The pull request base branch must be a 'main' or 'preview' branch..
✅The pull request head branch contains a valid issue number.
✅The pull request title matches the linked issue title exactly.
✅The pull request assignees match the assignees of the issue.
✅The pull request labels match the labels of the issue.
✅The pull request organizational projects match the organizational projects of the issue.
✅The pull requests milestone matches the milestone of the issue.

Additional Info:

Pull requests are automatically synced with the associated issue upon creation by one of the KinsonDigital workflow bots.
The associated issue is the issue number that is embedded in the pull request head branch.
The list above will be automatically updated as the pull request's various settings match or do not match the associated issue.
To manually sync the pull request to the issue, create a comment with the [run-sync] command.

Note You must be an admin member of the organization to use this command.

[x] Sync with the issue.  Use this to enable or disable this pull request from syncing with its associated issue.

Closed without changes.  The symbols and xml docs are added to the nuget package during the build process in Infrastructure


Fixed compile error on GCC 4.8

Fix for #207

Since the changeset is relatively small and it changes the code I authored, I'm happy to merge.
Thanks!


[v2] Router navigate issue

@nolimits4web
app.router.navigate('/');
it redirect us to http://example.com/#!/
Expected: http://example.com/

Same here

Well as F7 is using shebang it would be the expected although a bit silly for /

@ZanderBrown
Then I should use ~/ or what 😅


Unknown exporter "html_ch"

Maybe I'm forgetting something but it doesn't seem to find html_ch to convert with headings collapsed
!jupyter nbconvert --to html_ch Athena_test_sql.ipynb

Now that i have a closer look, I confused 2 different tools. Is there a way to export the html with the collapsible headings?


Add-Migration no longer working "No parameterless constructor was found on 'ApplicationDbContext'."

Attempting to run add-migration mig1 is failing.
Exception message:
No parameterless constructor was found on 'ApplicationDbContext'. Either add a parameterless constructor to 'ApplicationDbContext' or add an implementation of 'IDbContextFactory<ApplicationDbContext>' in the same assembly as 'ApplicationDbContext'.

Stack trace:
Finding DbContext classes...
Microsoft.EntityFrameworkCore.Design.OperationException: No parameterless constructor was found on 'ApplicationDbContext'. Either add a parameterless constructor to 'ApplicationDbContext' or add an implementation of 'IDbContextFactory<ApplicationDbContext>' in the same assembly as 'ApplicationDbContext'. ---> System.MissingMethodException: No parameterless constructor defined for this object.
   at System.RuntimeTypeHandle.CreateInstance(RuntimeType type, Boolean publicOnly, Boolean noCheck, Boolean& canBeCached, RuntimeMethodHandleInternal& ctor, Boolean& bNeedSecurityCheck)
   at System.RuntimeType.CreateInstanceSlow(Boolean publicOnly, Boolean skipCheckThis, Boolean fillCache, StackCrawlMark& stackMark)
   at System.RuntimeType.CreateInstanceDefaultCtor(Boolean publicOnly, Boolean skipCheckThis, Boolean fillCache, StackCrawlMark& stackMark)
   at System.Activator.CreateInstance(Type type, Boolean nonPublic)
   at System.Activator.CreateInstance(Type type)
   at Microsoft.EntityFrameworkCore.Design.Internal.DbContextOperations.<>c__DisplayClass12_3.<FindContextTypes>b__13()
   --- End of inner exception stack trace ---
   at Microsoft.EntityFrameworkCore.Design.Internal.DbContextOperations.<>c__DisplayClass12_3.<FindContextTypes>b__13()
   at Microsoft.EntityFrameworkCore.Design.Internal.DbContextOperations.CreateContext(Func`1 factory)
   at Microsoft.EntityFrameworkCore.Design.Internal.DbContextOperations.CreateContext(String contextType)
   at Microsoft.EntityFrameworkCore.Design.Internal.MigrationsOperations.AddMigration(String name, String outputDir, String contextType)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.AddMigrationImpl(String name, String outputDir, String contextType)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.AddMigration.<>c__DisplayClass0_1.<.ctor>b__0()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.<>c__DisplayClass3_0`1.<Execute>b__0()
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.Execute(Action action)

My program.cs has .UseStartup<StartupPortalBackend>()

And StartupPortalBackend has:

        public void ConfigureServices(IServiceCollection services)
        {
            // Add framework services.
            services.AddDbContext<ApplicationDbContext>(options =>
                options.UseSqlServer(Configuration.GetConnectionString("DefaultConnection")));
[snip]

And my ApplicationDbContext has:

    public class ApplicationDbContext : IdentityDbContext<ApplicationUser>
    {
        public ApplicationDbContext(DbContextOptions<ApplicationDbContext> options)
            : base(options)
        {
        }

        protected override void OnModelCreating(ModelBuilder builder)
        {
            base.OnModelCreating(builder);
[snip]

This looks pretty much the same as a default asp.net core with entityframework core identity project but for some reason I cannot add a migration whereas if I created a default project I can. This was originally created on a default project template but somewhere along the line simply stopped working - I have not had to migrate anything in a little while.
Further technical details
Microsoft.EntityFrameworkCore.SqlServer - 1.1.2
Microsoft.EntityFrameworkCore - 1.1.2
Operating system: Windows 10 latest updates
IDE: Visual Studio 2017 latest updates

I implemented IDbContextFactory<DataContext> and removed OnModelCreating to solve this issue:
public class DataContextFactory : IDbContextFactory<DataContext>
{
    public DataContext Create(DbContextFactoryOptions options)
    {
        // Used only for EF .NET Core CLI tools (update database/migrations etc.)
        var builder = new ConfigurationBuilder()
            .SetBasePath(Path.Combine(Directory.GetCurrentDirectory()))
            .AddJsonFile("appsettings.json", optional: false, reloadOnChange: true);

        var config = builder.Build();

        var optionsBuilder = new DbContextOptionsBuilder<DataContext>()
            .UseSqlServer(config.GetConnectionString("SearchDataContext"));

        return new DataContext(optionsBuilder.Options);
    }
}

See #8328 for additional context.

@bricelam - #8328 was fixed in preview1. Does it have same effect in 1.1.2 release too?

We only look in the class "Startup" for "ConfigureServices". Either rename the class as "Startup", or use the IDbContextFactory as you suggest.

@programulya sorry for my stupidity, but how would I then use this code? Like where would I put this code, and then how would I call it? Seriously sorry for this, but I'm really struggling here 😢.

Hi @robquinn, maybe it's already solved in new versions because the issue was in version netcoreapp1.1. Regarding that solution I created MigrationsHelper:
    public static class MigrationsHelper
    {
        public static void MigrateDatabase()
        {
            var dbFactory = new DataContextFactory();
            var dbContext = dbFactory.Create(new DbContextFactoryOptions());

            dbContext.Database.Migrate();
        }

        public static IConfigurationRoot GetConfigFile()
        {
            var environmentName = Environment.GetEnvironmentVariable("ASPNETCORE_ENVIRONMENT");

            var builder = new ConfigurationBuilder()
                .SetBasePath(Path.Combine(Directory.GetCurrentDirectory()))
                .AddJsonFile("appsettings.json", false, true)
                .AddJsonFile($"appsettings.{environmentName}.json", optional: true)
                .AddEnvironmentVariables();

            var config = builder.Build();

            return config;
        }
    }

and call it from Startup:
        public Startup(IHostingEnvironment env)
        {
            var builder = new ConfigurationBuilder()
                .SetBasePath(env.ContentRootPath)
                .AddJsonFile("appsettings.json", optional: false, reloadOnChange: true)
                .AddJsonFile($"appsettings.{env.EnvironmentName}.json", optional: true)
                .AddEnvironmentVariables();

            this.env = env;
            this.config = builder.Build();

            var logPath = this.config.GetSection("LogPath").Value;
            Log.Logger = new LoggerConfiguration()
                .Enrich.FromLogContext()
                .WriteTo.RollingFile(logPath, shared: true)
                .CreateLogger();

            MigrationsHelper.MigrateDatabase();
        }

Good luck ;)


Browsers and containers naming convention

I added IE and Edge to Selenoid and it is working great.
I just noticed something that I think should be improved the user experience in the future:
When looking at the available browsers I see:
0% | 0 | MicrosoftEdge
0% | 0 | android
0% | 0 | chrome
0% | 0 | firefox
0% | 0 | internet explorer
0% | 0 | opera

I think it will look better if it is like this:
0% | 0 | edge
0% | 0 | android
0% | 0 | chrome
0% | 0 | firefox
0% | 0 | ie
0% | 0 | opera

I know how picky are IE and Edge about calling them with the right name, but maybe an extra key on the browsers.json config can fix this.
    "internet explorer": {
        "display": "ie",  <=========// custom new key to display in the selenoid screen
        "default": "11.0",
        "versions": {
            "11.0": {
                "image": "windows:ie-11",
                "port": "4444",
                "path": "/",
                "privileged": true
            }
        }
    },

Again, this is just my opinion and an idea.
Thank you for the great job you guys have done.

@lanwen any simpler solutions?

whats the point of doing that?

The only point is to make happy all the QA, Developers and Testers with an OCD tendency, lol.

I mean what's the difference between original "MicrosoftEdge" and "edge"? Just want to understand the usecase - you want shorter names, all lowercase? Is that only for the stats?

This is just a user experience case. All you available browser/container that are not from Microsoft are in small letters and there comes Microsoft with its camel case for Edge and space in internet explorer. The main idea is to keep the name look about the same and also to give you the user the possibility of making them your own.

The issue I see is if you send the Edge 18 webdriver a session request with the browserName "edge" it will refuse it, and having the browser name be different in the UI may lead to confusion within your QA and Dev teams about what exactly the correct capabilities are.

I would leave as is as it could really confuse users. Let's leave things consistent.

I understand the possibility of confusions, but you also have a section where you display the basic setup of the capabilities where you can show the right name of the driver regardless of what the display name is.
You can also have this feature as a dark feature that only if use will overwrite the display otherwise it will behave as it currently does.


Update AppInsights Core to 3.0.9

It's only v4.1.0 and 4.0.3 that we need to update.

no need to update this version

But I see we don't currently have a branch for the 4.0.x range :-(
Lets wait until monday for this one, as this component is not used much at all.
And as v4.0.3 has only had 18 downloads in the last 7 days, this doesn't (seem) to be super critical
https://www.npmjs.com/package/@microsoft/applicationinsights-react-native?activeTab=versions

We will also need to include a publish tag for the release (so it doesn't become the "latest").


python tasks/run.py --config=egs/datasets/can/lm3d_radnerf_sr.yaml --exp_name=motion2video_nerf/may_head --reset

tensors used as indices must be long, byte or bool tensors

Hi, please check whether the CUDA extensions are installed correctly. For instance, whether the CUDA version and the python cudatoolkit version are the same.

thank to the great work, here is my env,which is testable in the radnerf.but i got the same issue in this project

same issue even though i changed my nvcc --version

Thanks for reporting the bug. cond_mask is a deprecated feature and we will delete it in our next commit.


[REQUEST] HAProxy Transparent Mode IPFW

This is an extract of a request by user @ROSU form this forum thread.

What I modified is basically as per the post I mentioned previously:
Create a new file /usr/local/etc/ipfw_custom.rules for the custom IPFW rules:
Code:
add 3000 fwd localhost tcp from ${server_ip} 80 to any in recv vmx1
Modify /usr/local/etc/rc.ipfw to include the custom rules
Code: [Select]
/sbin/ipfw -f /usr/local/etc/ipfw.rules
if [ -f /usr/local/etc/ipfw_custom.rules ]; then
/sbin/ipfw -f /usr/local/etc/ipfw_custom.rules
fi
Modify /usr/local/opnsense/service/templates/OPNsense/IPFW/rc.conf.d to enable IPFW
Code: [Select]
firewall_enable="{% if shapers or cp_zones %}YES{% else %}YES{% endif %}"
Then you can configure a backend in HAProxy via the Option pass-through section to use the client IP address as the source:
source <IP_ADDRESS> usesrc clientip
Just have to ensure that OPNsense is the default gateway for the servers you are load balancing. I believe there is an option called Transparent ClientIP on the pfSense HAProxy addin which will configure IPFW rules etc.

Is it possible to incorporate HAProxy Transparent Mode IPFW  thru the opnsense webgui?
Thanks!
Nuno

Hello? Is it possible?

Is it possible to incorporate HAProxy Transparent Mode IPFW thru the opnsense webgui?

Technically speaking it might be possible, but I'm unsure if it's possible to properly integrate it.
That being said, as the current maintainer of the HAProxy plugin I will not be able to work on this for several months. Contributions/patches are welcome, though.

@fichtner Please assign to me (and add the "help wanted" label).

Can't you just use the nat/forward options already in OPNsense (which uses pf), just like our proxy does?
(the link used to template the squid rules)
https://fw-ip/firewall_nat_edit.php?template=transparent_proxy
If there's anything missing there, we should be able to add it, but when possible we tend to use pf for our firewall and only ipfw for edge cases (like captive portal and traffic shaping).

@AdSchellevis  was that question for me? I would if i knew how :)
PS: thanks for all of your hard work!

@NunoHiggs yes, that question was indeed for you  :) I would start using the template rule which you can create using the above link and alter the parameters to match your situation.

@AdSchellevis I am going to give it a shot! Thanks

In many situations there are alternative ways available to get the client IP:

Use HAProxy in HTTP(S) mode and enable the X-Forwarded-For header
Use HAProxy's PROXY protocol if your application supports it (Dovecot, Postfix)

I'll close this issue now, but if someone wants to work on this we'll reopen it of course.


GitLab OAuth login is broken

Description
GitLab complains "The requested scope is invalid, unknown, or malformed."
Steps to reproduce

Create new application on GitLab
Configure GitLab OAuth for Portus
Try logging in via GitLab

Expected behavior: Proceed OAuth authentication process.
Actual behavior: The requested scope is invalid, unknown, or malformed.

Deployment information
GitLab version: 11.3.0-ce
Portus version: opensuse/portus:2.3.5

Oops, this is a configuration error. I need to grant the "api" scope for the application on GitLab. I think this requirement is not documented.
IMHO, I would expect using "openid" scope instead of the much more powerful "api" scope.


incorrect quoting for windows building in conda?

trying to build master in clean windows x64 miniconda environment, gcc itself is in windows path:
writing build\temp.win-amd64-2.7\Release\pandas\index.def
C:\Program Files\mingw-builds\x64-4.8.1-posix-seh-rev5\mingw64\bin\dllwrap.exe -DMS_WIN64 -mdll -static --entry _DllMain@12 --output-lib build\temp.win-amd64-2.7\Release\pandas\libindex.a --def build\temp.win-amd64-2.7\Release\pandas\index.def -s build\temp.win-amd64-2.7\Release\pandas\index.o build\temp.win-amd64-2.7\Release\pandas\src\datetime\np_datetime.o build\temp.win-amd64-2.7\Release\pandas\src\datetime\np_datetime_strings.o -LC:\Miniconda\envs\pandas_dev\libs -LC:\Miniconda\envs\pandas_dev\PCbuild\amd64 -lpython27 -lmsvcr90 -o c:\Dev\src\pandas\pandas\index.pyd
'gcc" "-Wl' is not recognized as an internal or external command,
operable program or batch file.
dllwrap: gcc exited with status 1
error: command 'C:\\Program Files\\mingw-builds\\x64-4.8.1-posix-seh-rev5\\mingw64\\bin\\dllwrap.exe' failed with exit status 1

was just going to post some conda builds for you.
use conda build conda.recipe --no-test to build it (then conda install install the binary)

make sure you have libpython installed, see here: http://pandas.pydata.org/pandas-docs/stable/contributing.html#creating-a-development-environment

thanks, having issues running it, should i raise with upstream?:
[pandas_dev] c:\Dev\src\pandas>conda build conda.recipe --no-test
Removing old build environment
Removing old work directory
BUILD START: pandas--nppy27_0
Fetching package metadata: ......
Solving package specifications: .
The following NEW packages will be INSTALLED:

    cython:     0.22.1-py27_0
    dateutil:   2.4.1-py27_0
    libpython:  1.0-py27_1
    mingw:      4.7-1
    numpy:      1.9.2-py27_0
    pip:        7.1.0-py27_0
    python:     2.7.10-0
    pytz:       2015.4-py27_0
    setuptools: 18.0.1-py27_0
    six:        1.9.0-py27_0

Linking packages ...
[      COMPLETE      ]|##################################################| 100%
Removing old work directory
Cloning into bare repository 'C:/Miniconda/conda-bld/git_cache/Dev_src_pandas'...
done.
checkout: u'f00e51cd4eb96537357b33932c9d78711ca4bdc9'
Cloning into 'C:\Miniconda\conda-bld\work'...
done.
Note: checking out 'f00e51cd4eb96537357b33932c9d78711ca4bdc9'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b new_branch_name

HEAD is now at f00e51c... Merge pull request #10743 from kawochen/TST-TestMsgpack-gen
==> git log -n1 <==

An unexpected error has occurred, please consider sending the
following traceback to the conda GitHub issue tracker at:

    https://github.com/conda/conda-build/issues

Include the output of the command 'conda info' in your report.

Traceback (most recent call last):
  File "C:\Miniconda\Scripts\conda-build-script.py", line 4, in <module>
    sys.exit(main())
  File "C:\Miniconda\lib\site-packages\conda_build\main_build.py", line 173, in main
    args_func(args, p)
  File "C:\Miniconda\lib\site-packages\conda_build\main_build.py", line 400, in args_func
    args.func(args, p)
  File "C:\Miniconda\lib\site-packages\conda_build\main_build.py", line 358, in execute
    channel_urls=channel_urls, override_channels=args.override_channels)
  File "C:\Miniconda\lib\site-packages\conda_build\build.py", line 354, in build
    source.provide(m.path, m.get_section('source'))
  File "C:\Miniconda\lib\site-packages\conda_build\source.py", line 253, in provide
    git_source(meta, recipe_dir)
  File "C:\Miniconda\lib\site-packages\conda_build\source.py", line 136, in git_source
    git_info()
  File "C:\Miniconda\lib\site-packages\conda_build\source.py", line 165, in git_info
    print(stdout + u'\n')
  File "C:\Miniconda\lib\encodings\cp850.py", line 12, in encode
    return codecs.charmap_encode(input,errors,encoding_map)
UnicodeEncodeError: 'charmap' codec can't encode character u'\u20ac' in position 291: character maps to <undefined>

and conda info:
[pandas_dev] c:\Dev\src\pandas>conda info
Current conda install:

             platform : win-64
        conda version : 3.15.1
  conda-build version : 1.16.0
       python version : 2.7.9.final.0
     requests version : 2.7.0
     root environment : C:\Miniconda  (writable)
  default environment : C:\Miniconda\envs\pandas_dev
     envs directories : C:\Miniconda\envs
        package cache : C:\Miniconda\pkgs
         channel URLs : https://repo.continuum.io/pkgs/free/win-64/
                        https://repo.continuum.io/pkgs/free/noarch/
                        https://repo.continuum.io/pkgs/pro/win-64/
                        https://repo.continuum.io/pkgs/pro/noarch/
          config file : None
    is foreign system : False

i was indeed following that,
[pandas_dev] c:\Dev\src\pandas>conda list
# packages in environment at C:\Miniconda\envs\pandas_dev:
#
cython                    0.22.1                   py27_0
jinja2                    2.8                      py27_0
libpython                 1.0                      py27_1
markupsafe                0.23                     py27_0
mingw                     4.7                           1
nose                      1.3.7                    py27_0
numpy                     1.9.2                    py27_0
pip                       7.1.0                    py27_0
python                    2.7.10                        0
python-dateutil           2.4.2                    py27_0
pytz                      2015.4                   py27_0
setuptools                18.0.1                   py27_0
six                       1.9.0                    py27_0

no, somehow unicode got committed in the log (from the last pr). going to fix. hold on a min.

I reverted then redid the offending commit. So you can give a try again (I will post some builds on binstar as well), my build process is not totally active yet (to make builds for all platforms on every commit).
also I do think this is an issue with the latest conda, very odd. You want to post a bug report to conda? thxs.

i put up the latest build, use the -c pandas to get it
[pandas] C:\Users\Jeff Reback\pandas>conda install pandas -c pandas --dry
Fetching package metadata: ......
Solving package specifications: .
Package plan for installation in environment C:\Miniconda\envs\pandas:

The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    pandas-v0.16.2             |       nppy27_286         5.1 MB

The following NEW packages will be INSTALLED:

    dateutil: 2.4.1-py27_0
    pandas:   v0.16.2-nppy27_286

Dry run: exiting

refreshed and retried, am back to my original error above - have raised the issue upstream.
trying to use your new build:
[pandas_snap] c:\Dev\src\pandas>conda list
# packages in environment at C:\Miniconda\envs\pandas_snap:
#
cython                    0.22.1                   py27_0
dateutil                  2.4.1                    py27_0
ipython                   3.2.1                    py27_0
libpython                 1.0                      py27_1
mingw                     4.7                           1
nose                      1.3.7                    py27_0
numpy                     1.9.2                    py27_0
pandas                    v0.16.2              nppy27_286
pip                       7.1.0                    py27_0
pyreadline                2.0                      py27_0
python                    2.7.10                        0
python-dateutil           2.4.2                    py27_0
pytz                      2015.4                   py27_0
setuptools                18.0.1                   py27_0
six                       1.9.0                    py27_0

and now get this:
In [1]: import pandas
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-1-d6ac987968b6> in <module>()
----> 1 import pandas

c:\Dev\src\pandas\pandas\__init__.py in <module>()
     11                       "pandas from the source directory, you may need to run "
     12                       "'python setup.py build_ext --inplace' to build the C "
---> 13                       "extensions first.".format(module))
     14
     15 from datetime import datetime

ImportError: C extension: hashtable not built. If you want to import pandas from the source directory, you may need to run 'python setup.py build_ext --inplace' to build the C extensions first.

you can't be in the actual build directory

ah, indeed, confirmed that https://github.com/pydata/pandas/issues/10732 is fixed with this build.
Now i need to distill the changes since 0.14.2 to consider backporting. Many thanks for the fix and support.

ok, gr8!


Add James Grunewald to yjaf repos with write access (dev)

Use this issue template if you do not feel comfortable raising a PR with the required changes to files in the terraform/ directory.
Please supply all the requested information, so that your request can be processed as quickly as possible.
Which MoJ repository are you requesting access to (e.g. money-to-prisoners)?
All yjaf-* repos excluding yjaf-devops-infrastructure
What is the GitHub username of the person who needs access?
JWGNPS
What is their full name?
James Grunewald
What is their email address?
<EMAIL_ADDRESS>What company/organisation do they belong to (e.g. "MadeTech", "Department for Education")?
NPS (northgate)
What level of access should they have?

[ ] Read
[ ] Triage
[X] Write
[ ] Maintain
[ ] Admin

Why is this access being requested?
New dev needs access to make app changes
Which MoJ team/person is responsible for this request?
Greg Whiting -<EMAIL_ADDRESS>- Devops for northgate
Review date
2021-12-31

yjaf-app-template
yjaf-assets
yjaf-auth
yjaf-bands
yjaf-bed-unlock
yjaf-case
yjaf-cmm
yjaf-conversions
yjaf-csppi
yjaf-dal
yjaf-documents
yjaf-gateway
yjaf-infra-aws-mgmt
yjaf-mule-hub-connector
yjaf-mule-mis-processor
yjaf-nginx-gateway
yjaf-nginx-pentaho
yjaf-placements
yjaf-pnomis
yjaf-refdata
yjaf-returns
yjaf-sentences
yjaf-transfers
yjaf-ui
yjaf-utilities
yjaf-views
yjaf-workflow
yjaf-yp

yjaf-app-template
yjaf-assets
yjaf-auth
yjaf-bands
yjaf-bed-unlock
yjaf-case
yjaf-cmm
yjaf-conversions
yjaf-csppi
yjaf-dal
yjaf-documents
yjaf-gateway
yjaf-infra-aws-mgmt
yjaf-mule-hub-connector
yjaf-mule-mis-processor
yjaf-nginx-gateway
yjaf-nginx-pentaho
yjaf-placements
yjaf-pnomis
yjaf-refdata
yjaf-returns
yjaf-sentences
yjaf-transfers
yjaf-ui
yjaf-utilities
yjaf-views
yjaf-workflow
yjaf-yp

Skipping yjaf-infra-aws-mgmt and yjaf-pnomis because no collaborators at all are currently defined on those repositories.

Skipping yjaf-infra-aws-mgmt and yjaf-pnomis because no collaborators at all are currently defined on those repositories.


Allow tokenizer inputs to be determined from model

closes https://github.com/knights-analytics/hugot/issues/41


Documentation

Write some documentation on how to actually use this plugin.

Notice: This plugin is no more maintained and it has been removed from Jetbrains plugins repository. There is a Markdown plugin officially supported by Jetbrains.


Feature/data preprocessing framework

What?

Adds the FusionDataPreprocessor interface with generic type
Adds interfaces extending FusionDataPreprocessor for each view model with this model as concrete type
Adds the Collection<FusionDataPreprocessor<T>>.preprocess extension function for applying multiple preprocessing steps in iteration order

Why?
This will allow for Fusion data to have preprocessing steps applied to it directly before populating the UI, allowing for features like:

Dynamic localisation of text
Styles which are automatically applied to views missing any declared style information

Additionally, with a proposed future feature (extensions on views), this will allow for far greater customisation of Fusion data

I am closing this PR without merging, as the direction of Fusion has changed significantly within the past couple of months.


Integration Tests Failing With 500 Error

In gitlab by @andytwf on Sep 3, 2016, 21:19
When running the integration tests, they fail out instantly with a 500 errors, caused by an undefined variable "errors".
From a bit of poking about, this appears to be the one on line 172 of layout.blade.php
Related to #550 and @A-Lawrence is looking into it.

In gitlab by @NFarrington on Sep 3, 2016, 22:04
This shouldn't be related to #550.
The middleware that ensures the errors variable exists has sometimes not run by the time the error page is displayed. The template needs to be modified to account for this (check if $errors exists).

In gitlab by @andytwf on Sep 3, 2016, 22:28
Ah ok - Ant asked for me to link the two for him :)

In gitlab by @NFarrington on Sep 7, 2016, 23:11
mentioned in commit c486229758af510a862b58b7f1f94095dca85b31

In gitlab by @NFarrington on Sep 7, 2016, 23:11
Status changed to closed by commit 297259ea1c92a83d5e46e0add736b1e37b43d116

In gitlab by @NFarrington on Sep 7, 2016, 23:14
The root issue here was that the baseUrl wasn't setting properly, causing the route to return a 404, fixed by 297259ea.
Display of the 404 error was then causing the issues with undefined variables, fixed by c4862297.


Added a really basic login page

Added an ejs test login page

This took the better part of two days to figure out due to very unhelpful error messages


Flaky service test: service/tcp_socket_service_test

Failed on the app_jit-vm release_x64 configuration today.

Added to status file in a424d65ecd9295279693ee53dddaeb331355c5c5.


Make new Postgres decoding public

Motivation
After a long time in the works, we finally want to make the new Postgres decoding public.
Changes

Add new protocol PostgresArrayDecodable that inherits from PostgresDecodable. A type that conforms to PostgresArrayDecodable can be decoded into an array of its own type from a Postgres array.
Make PostgresCell decode method public
Make PostgresRow decode methods public (we use @_alwaysEmitIntoClient here, to keep binary sizes small)
Mark a number of things @usableFromInlineable to allow better perf

Result
Faster code for users!

Codecov Report

Merging #244 (1e76c19) into main (c54c006) will increase coverage by 1.62%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main     #244      +/-   ##
==========================================
+ Coverage   39.57%   41.20%   +1.62%
==========================================
  Files         115      113       -2
  Lines        9479     8876     -603
==========================================
- Hits         3751     3657      -94
+ Misses       5728     5219     -509

Flag
Coverage Δ

unittests
41.20% <66.66%> (+1.62%)
:arrow_up:

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

Impacted Files
Coverage Δ

Sources/PostgresNIO/Data/PostgresRow.swift
50.00% <ø> (+1.02%)
:arrow_up:

...es/PostgresNIO/New/Data/Bool+PostgresCodable.swift
95.34% <ø> (+4.65%)
:arrow_up:

...s/PostgresNIO/New/Data/Bytes+PostgresCodable.swift
73.52% <ø> (ø)

...es/PostgresNIO/New/Data/Date+PostgresCodable.swift
89.28% <ø> (ø)

...PostgresNIO/New/Data/Decimal+PostgresCodable.swift
74.19% <ø> (+6.45%)
:arrow_up:

...s/PostgresNIO/New/Data/Float+PostgresCodable.swift
76.66% <ø> (+3.33%)
:arrow_up:

...ces/PostgresNIO/New/Data/Int+PostgresCodable.swift
28.75% <ø> (+3.26%)
:arrow_up:

.../PostgresNIO/New/Data/String+PostgresCodable.swift
100.00% <ø> (ø)

...es/PostgresNIO/New/Data/UUID+PostgresCodable.swift
98.30% <ø> (ø)

...s/PostgresNIO/New/Extensions/ByteBuffer+PSQL.swift
100.00% <ø> (ø)

... and 38 more


Discover better strategy to retries

To study https://github.com/GoogleCloudPlatform/google-cloud-ruby/blob/master/google-cloud-datastore/lib/google/cloud/datastore/v1/datastore_client_config.json

google-cloud gem uses google-gax (doc) to extend a "normal" stub.
Examples of definition and consume in datastore.


Setup CI

Before we start updating this package we should setup GitHub actions that run lint and tests

Minor gripe, if we adopt "standard" js style, can we leave semicolons in? I think it's a bit preposterous for a style standard to name itself "JavaScript Standard Style" and omit semicolons.

Yes, standard is available with semicolons, named semistandard we could use that as well.


Match conv2 weights stddev with cuda-convnet's layer def.

conv2 initW is 0.01 in cuda-convnet.
https://code.google.com/p/cuda-convnet/source/browse/trunk/example-layers/layers-conv-local-11pct.cfg
I change layer type from full connected to locally connected(just convolutional..).
Vanishing-Gradient problem is occurred at conv2.
Is this intended?

Can one of the admins verify this patch?

@tensorflow-jenkins test this please

@girving @vrv

I don't follow your message.  Are you saying that your new value helps after you change the network to a different topology?  What are the evaluation accuracies before and after your change?

Hi @girving,
conv2's parameter stddev is different from original cuda-convnet.
I think this may be a typo, because 1e-4 is same value with conv1 layer.
Currently parameter(1e-4) cause of vanishing-gradient at trying EXERCISE.

The original  cuda-convnet's conv2 initW is 0.01.
[conv2]
type=conv
inputs=rnorm1
filters=64
padding=2
stride=1
filterSize=5
channels=64
neuron=relu
initW=0.01
partialSum=8
sharedBiases=1

And cifar10.py's conv2 stddev is 1e-4.
  with tf.variable_scope('conv2') as scope:
    kernel = _variable_with_weight_decay('weights', shape=[5, 5, 64, 64],
                                         stddev=1e-4, wd=0.0)

I think this may be a typo, because 1e-4 is same value with conv1 layer.
TensorFlow document is questioning
EXERCISE: The model architecture in inference() differs slightly from the CIFAR-10 model specified in
cuda-convnet. In particular, the top layers of Alex's original model are locally connected and not fully
connected. Try editing the architecture to exactly reproduce the locally connected architecture in the top
layer.

I'm trying this exercise(unfortunately I cannot find model answer on web).
I tried replace local3 and local4 to convolutional layer. And if conv2's stddev was 1e-4, vanishing-gradient occurred at conv2 layer.

Here's my current inference code.
def inference(images):
  """Build the CIFAR-10 model.

  Args:
    images: Images returned from distorted_inputs() or inputs().

  Returns:
    Logits.
  """
  # We instantiate all variables using tf.get_variable() instead of
  # tf.Variable() in order to share variables across multiple GPU training runs.
  # If we only ran this model on a single GPU, we could simplify this function
  # by replacing all instances of tf.get_variable() with tf.Variable().
  #
  # conv1
  with tf.variable_scope('conv1') as scope:
    kernel = _variable_with_weight_decay('weights', shape=[5, 5, 3, 64],
                                         stddev=1e-4, wd=0.0)
    conv = tf.nn.conv2d(images, kernel, [1, 1, 1, 1], padding='SAME')
    biases = _variable_on_cpu('biases', [64], tf.constant_initializer(0.0))
    bias = tf.nn.bias_add(conv, biases)
    conv1 = tf.nn.relu(bias, name=scope.name)
    _activation_summary(conv1)

  # pool1
  pool1 = tf.nn.max_pool(conv1, ksize=[1, 3, 3, 1], strides=[1, 2, 2, 1],
                         padding='SAME', name='pool1')

  # norm1
  norm1 = tf.nn.lrn(pool1, 4, bias=1.0, alpha=0.001 / 9.0, beta=0.75,
                    name='norm1')

  # conv2
  with tf.variable_scope('conv2') as scope:
    kernel = _variable_with_weight_decay('weights', shape=[5, 5, 64, 64],
                                         stddev=1e-2, wd=0.0)
    conv = tf.nn.conv2d(norm1, kernel, [1, 1, 1, 1], padding='SAME')
    biases = _variable_on_cpu('biases', [64], tf.constant_initializer(0.1))
    bias = tf.nn.bias_add(conv, biases)
    conv2 = tf.nn.relu(bias, name=scope.name)
    _activation_summary(conv2)

  # norm2
  norm2 = tf.nn.lrn(conv2, 4, bias=1.0, alpha=0.001 / 9.0, beta=0.75,
                    name='norm2')
  # pool2
  pool2 = tf.nn.max_pool(norm2, ksize=[1, 3, 3, 1],
                         strides=[1, 2, 2, 1], padding='SAME', name='pool2')

  # local3
  with tf.variable_scope('local3') as scope:
    kernel = _variable_with_weight_decay('weights', shape=[3, 3, 64, 64],
                                         stddev=0.04, wd=0.04)
    conv = tf.nn.conv2d(pool2, kernel, [1, 1, 1, 1], padding='SAME')
    biases = tf.Variable(tf.constant(0.1, shape=[64]))
    bias = tf.nn.bias_add(conv, biases)
    local3 = tf.nn.relu(bias, name=scope.name)
    _activation_summary(local3)

  # local4
  with tf.variable_scope('local4') as scope:
    kernel = _variable_with_weight_decay('weights', shape=[3, 3, 64, 32],
                                         stddev=0.04, wd=0.04)
    conv = tf.nn.conv2d(local3, kernel, [1, 1, 1, 1], padding='SAME')
    biases = tf.Variable(tf.constant(0.1, shape=[32]))
    bias = tf.nn.bias_add(conv, biases)
    local4 = tf.nn.relu(bias, name=scope.name)
    _activation_summary(local4)

  # softmax, i.e. softmax(WX + b)
  with tf.variable_scope('softmax_linear') as scope:
    # Move everything into depth so we can perform a single matrix multiply.
    reshape = tf.reshape(local4, [FLAGS.batch_size, -1])
    dim = reshape.get_shape()[1].value

    weights = _variable_with_weight_decay('weights', [dim, NUM_CLASSES],
                                          stddev=1 / float(NUM_CLASSES), wd=0.0)
    biases = _variable_on_cpu('biases', [NUM_CLASSES],
                              tf.constant_initializer(0.0))
    softmax_linear = tf.nn.softmax(tf.nn.bias_add(tf.matmul(reshape, weights), biases), name=scope.name)

    _activation_summary(softmax_linear)

  return softmax_linear

If weight stddev was 1e-4 at conv2, evaluation accuracies is just 10%.
And tensorboard shows vanishing-gradient at conv2 layer.

@vincentvanhoucke: Can I get your eye here?  His change seems reasonable to me (I would think 1e-2 is a better std for both conv layers).

@keiji your solution doesn't exactly replicate Alex's model. You use convolutional layers for local3 and local4, whereas Alex used locally-connected layers that do not share their parameters across patches. My guess is that's possibly one reason why you see such a strong effect of initialization.
Come to think of it, I don't know how one would do that in TensorFlow. @girving do you know who wrote the tutorial and whether they had a specific trick in mind for locally connected, non-convolutional layers?

@vincentvanhoucke: To make sure I understand: you mean a convolutional layer but using separate filters for each output point?  You could do it with batch matmul and a bunch of reshaping / tiling logic, but it would be quite slow.  I think we'd need a custom op to it fast.  And actually even the reshaping / tiling logic may be out of reach in a performant way.

@girving yes, that's what the original Cifar10 model uses. We should remove that exercise if we don't know ourselves how to do it well at this point, unless the author of the tutorial has something up their sleeve I don't know.

@shlens: Looks like you wrote parts of the deep_cnn tutorial.  Do you know a trick for untied convolutions?

There is no trick. We envisioned the exercise to be difficult because a user would need to invoke several tiling, reshape and matmul operations. Most of the challenge would be in the construction of the tiling operation across the local3 and local4.

As far as I understand, TensorFlow doesn't have convenience operations for making locally-connected layer. We have to implement locally connected logic manually.
I'll retry exercise with this approach.

I'm going to close this PR. Please reopen if you feel strongly that the defaults should match cudaconvnet. As it stands, I'm not convinced it's an issue.


image display not center alignment

there is something with the code following:
// If we're zooming to fill then centralise
if (self.zoomScale != minScale) {
    // Centralise
    self.contentOffset = CGPointMake((imageSize.width * self.zoomScale - boundsSize.width) / 2.0,
                                     (imageSize.height * self.zoomScale - boundsSize.height) / 2.0);

}

the image display not center alignment, padding on the right bottom.

I also encountered this problem,how to solve it?

I'm also having this problem.

I'm also having this problem. Somebody have a solution?

this issue is a clone to issue #485 where method "" and method "" behaviour are related.
I find a fix seems to work on all iPhone screen size available in issue #485


router-level middleware becomes app-level if route is "/"

Hi,
I'm not sure if this is an expected behaviour, I found it at least misleading, and in case is expected
I'd suggest to make it clear in the docs.
Here a code example from the docs:

var router = express.Router();

router.use(function (req, res, next) {
  console.log('Time:', Date.now())
  next()
});

// if one uses the following the middleware will be run for any route which
// somehow surprises me...
app.use('/', router);

// if instead one does the following, the middleware is only attached to
// that route (as expected)
// app.use('/some_route', router);

Thanks in advance for the clarification,
cheers.

Hi,
Wow, that was fast reply, thanks!
the docs pages I'm talking about is here:
https://expressjs.com/en/guide/using-middleware.html#middleware.application
https://expressjs.com/en/guide/using-middleware.html#middleware.router
in my example I was expecting (or at least so I misunderstood) that:
app.use('/', router)
would run the middleware  only if one would ask for '/' and not also if one would ask for '/some_other_route'.
Anyway, does this mean that it is impossible to have a middleware that runs ONLY for a request to '/' (with any method)?

Anyway, does this mean that it is impossible to have a middleware that runs ONLY for a request to '/' (with any method)?

You'll want to use a method that is an exact-path-matching, like app.all. The app.use is prefix-matching, so if you app.use('/', foo) (the / is optional), then it matches all patches prefixed with that (which is basically every URL). Methods like app.all('/', foo) on the other hand are exact path matching, so it will run for all methods on exactly the path /.
I'm going to look over the website around this; if you spot anything as well countering what I said above, please feel free to click on "Edit this page on GitHub" at the bottom of any page to help fix it up! There is a lot of text on there, authored by multiple people and any one person can't really keep track of every word on the site, so things get messed up sometimes on accident.

thanks a lot!!
and by the way express is amazing!!

Coming from laravel, I find this behavior somewhat different. From the testing script I made below:
let express = require('express');
let app = express();

let middlewareForRoute1 = (req, res, next) => {
    if(req.url == "/route2"){
        console.log("MUST NOT EXECUTED");
    }
    next();
};

let route1 = express.Router();
route1.use(middlewareForRoute1);
route1.get('/route1', (req, res, next) => {
    res.send("1");
    res.end();
});

let route2 = express.Router();
route2.get('/route2', (req, res, next) => {
    res.send("2");
    res.end();
});

app.use('/', route1);
app.use('/', route2);

let port = 3000;
app.listen(port, function () {
    console.log("RUNNING");
});

It shows that while I apply the middleware explicitly on route1, it is also applied on route2. It doesn't applied on route2, when I put app.use('/', route2) above route1.
My expectation is middlewareForRoute1 is only applied for all routes defined in route1 object, and not applied into other express.Routes() object, based on documentation:

Router-level middleware works in the same way as application-level middleware, except it is bound to an instance of express.Router().