occasional crash in [RCTText drawRect]

The onLayout demo crashed in the iPhone 6 simulator using Xcode 6.3.1 with iOS 8.3. Before this I had cleaned DerivedData and run git clean -dfx && npm install in a checkout of react-native@41612f37b0c31320b025043df9e954df540db171.
cc @sahrens
May  8 14:31:31 mac.local UIExplorer[61720]: *** Terminating app due to uncaught exception 'NSRangeException', reason: '*** NSRunStorage, _NSBlockNumberForIndex(): index (72) beyond array bounds (23)'
    *** First throw call stack:
    (
        0   CoreFoundation                      0x000000010f731c65 __exceptionPreprocess + 165
        1   libobjc.A.dylib                     0x000000010eba3bb7 objc_exception_throw + 45
        2   CoreFoundation                      0x000000010f731b9d +[NSException raise:format:] + 205
        3   UIFoundation                        0x0000000117ac9adb _NSBlockNumberForIndex + 84
        4   UIFoundation                        0x0000000117ab6a1f -[NSLayoutManager _primitiveInvalidateDisplayForGlyphRange:] + 777
        5   UIFoundation                        0x0000000117ab8338 -[NSLayoutManager invalidateDisplayForCharacterRange:] + 450
        6   UIFoundation                        0x0000000117a8146a -[NSLayoutManager(NSPrivate) _invalidateDisplayIfNeeded] + 72
        7   UIFoundation                        0x0000000117aba83c -[NSLayoutManager textContainerForGlyphAtIndex:effectiveRange:] + 871
        8   UIFoundation                        0x0000000117abc6aa -[NSLayoutManager glyphRangeForTextContainer:] + 317
        9   UIExplorer                          0x000000010e56fbb7 -[RCTText drawRect:] + 231
        10  UIKit                               0x0000000111212612 -[UIView(CALayerDelegate) drawLayer:inContext:] + 495
        11  QuartzCore                          0x0000000110ed2a1a -[CALayer drawInContext:] + 119
        12  QuartzCore                          0x0000000110dcc93b CABackingStoreUpdate_ + 2793
        13  QuartzCore                          0x0000000110ed2923 ___ZN2CA5Layer8display_Ev_block_invoke + 59
        14  QuartzCore                          0x0000000110ed27ae _ZN2CA5Layer8display_Ev + 1478
        15  QuartzCore                          0x0000000110ec74c9 _ZN2CA5Layer17display_if_neededEPNS_11TransactionE + 301
        16  QuartzCore                          0x0000000110ec7551 _ZN2CA5Layer28layout_and_display_if_neededEPNS_11TransactionE + 35
        17  QuartzCore                          0x0000000110e33886 _ZN2CA7Context18commit_transactionEPNS_11TransactionE + 242
        18  QuartzCore                          0x0000000110e34a3a _ZN2CA11Transaction6commitEv + 462
        19  QuartzCore                          0x0000000110e350eb _ZN2CA11Transaction17observer_callbackEP19__CFRunLoopObservermPv + 89
        20  CoreFoundation                      0x000000010f664ca7 __CFRUNLOOP_IS_CALLING_OUT_TO_AN_OBSERVER_CALLBACK_FUNCTION__ + 23
        21  CoreFoundation                      0x000000010f664c00 __CFRunLoopDoObservers + 368
        22  CoreFoundation                      0x000000010f65aa33 __CFRunLoopRun + 1123
        23  CoreFoundation                      0x000000010f65a366 CFRunLoopRunSpecific + 470
        24  GraphicsServices                    0x000000011339ca3e GSEventRunModal + 161
        25  UIKit                               0x0000000111192900 UIApplicationMain + 1282
        26  UIExplorer                          0x000000010e4b5fff main + 111
        27  libdyld.dylib                       0x0000000112ca4145 start + 1
    )
May  8 14:31:55 mac com.apple.CoreSimulator.SimDevice.0AD0F9BD-A848-48A1-BFC6-BF8FABBF059A.launchd_sim[55138] (UIKitApplication:com.facebook.UIExplorer[0x89a0][61720]): Service exited due to signal: Killed: 9

Weird, I don't think I've ever seen a stack like that before.
Can you quantify "Occasionally"?

It happened the first time after cleaning my build artifacts and running the app, and then I wasn't able to repro after that. It might not be related to the onLayout demo (that [RCTText drawRect:] line is suspect) - that was just the demo that triggered the crash. If you guys have crash logs for the Groups app / Ads Manager is it possible to grep them for RCTText?

Saw this same stack on a friend's app. Repros very intermittently; no idea what the cause is.

@a2 based on RCTText.

happened with me today ! crashed when I opened the UIExplorer app.

This gist, which renders 100 <Text> nodes every frame, very reliably crashes for me:
https://gist.github.com/spicyj/22bdcb6b229cd9dcbd6c
Not exactly the same stack, but it may be the same root cause. Looks like a threading issue with the shared TextKit objects across threads – in every crash I've seen with this sample app, the main thread and shadow queue are both processing text views. AsyncDisplayKit puts a lock around all TextKit operations which might be appropriate here too. Although I'm not sure if adding a lock is all that's needed – it seems a little fishy that these objects are even shared between RCTShadowText and RCTText because it seems like one thread could get ahead of the other and we could be trying to process shadow text with a new string before the last string has been rendered on the main thread. I don't have a complete grasp of the concurrency involved here though so I could be off base.

Noticing a lot of these after the upgrade to 0.4.3; whereas before I only saw it occasionally, now it's crashing often.
The stack trace of the crash varies -- sometimes it's "!!! _NSGlyphTreeInsertGlyphs invalid char index", other times it's "_NSBlockNumberForIndex(): index (0) beyond array bounds (0) raised during typesetting layout manager"

Ping @a2, @nicklockwood, @tadeuzagallo

Not sure if this is related but have also discovered the following: EXC_BAD_ACCESS: RCTText line 90 - crashed with [_textStorage addLayoutManager:_layoutManager].

Stack trace ends at: [NSLayoutManager(NSPrivate) _recalculateUsageForTextContainerAtIndex]

Closing this because of https://github.com/facebook/react-native/commit/0689c0790edcc9cb8494e3f15300e2743556f442. Will reopen if the crash persists.

I can provide context on ASDK/ComponentKit TextKit locking strategies if this crops up.  Ping me, we have open radars with Apple.

I should mention that I'm still seeing this in v0.33.1. It's definitely rare, but doesn't seem to be entirely fixed, and is one of the leading exceptions I have right now. (Slightly more often than the RCTImageLoader exception)

Actually, I'm seeing a slightly different issue, in my case it's drawGlyphsForGlyphRange:atPoint: that is throwing the exception. I'm working on building a test project that reproduces it now.
Crashed: com.apple.main-thread
0  libobjc.A.dylib                0x18d8def70 objc_msgSend + 16
1  CoreText                       0x191b04bd4 CTFontDrawGlyphsAtPositions + 104
2  UIFoundation                   0x194c5d4c8 -[NSLayoutManager(OtherSupport) _showCGGlyphs:positions:count:font:matrix:attributes:inContext:] + 776
3  UIFoundation                   0x194ca81b0 -[NSLayoutManager showCGGlyphs:positions:count:font:matrix:attributes:inContext:] + 60
4  UIFoundation                   0x194c789b4 -[NSLayoutManager(NSPrivate) _drawGlyphsForGlyphRange:atPoint:] + 6016
5  MyApp                          0x100a076fc -[RCTText drawRect:] (RCTText.m:105)
6  UIKit                          0x194dcfa60 -[UIView(CALayerDelegate) drawLayer:inContext:] + 408
7  QuartzCore                     0x1921f559c -[CALayer drawInContext:] + 260
8  QuartzCore                     0x1920f6848 CABackingStoreUpdate_ + 2848
9  QuartzCore                     0x1921f516c ___ZN2CA5Layer8display_Ev_block_invoke + 52
10 QuartzCore                     0x1921f4e0c CA::Layer::display_() + 1524
11 QuartzCore                     0x1921ea314 CA::Layer::display_if_needed(CA::Transaction*) + 228
12 QuartzCore                     0x1921ea398 CA::Layer::layout_and_display_if_needed(CA::Transaction*) + 44
13 QuartzCore                     0x1921673e0 CA::Context::commit_transaction(CA::Transaction*) + 252
14 QuartzCore                     0x19218ea68 CA::Transaction::commit() + 512
15 QuartzCore                     0x19218f488 CA::Transaction::observer_callback(__CFRunLoopObserver*, unsigned long, void*) + 120
16 CoreFoundation                 0x18ee420c0 __CFRUNLOOP_IS_CALLING_OUT_TO_AN_OBSERVER_CALLBACK_FUNCTION__ + 32
17 CoreFoundation                 0x18ee3fcf0 __CFRunLoopDoObservers + 372
18 CoreFoundation                 0x18ee40180 __CFRunLoopRun + 1024
19 CoreFoundation                 0x18ed6e2b8 CFRunLoopRunSpecific + 444
20 GraphicsServices               0x190822198 GSEventRunModal + 180
21 UIKit                          0x194db57fc -[UIApplication _run] + 684
22 UIKit                          0x194db0534 UIApplicationMain + 208


blurred and jagged reconstruction result when using custom data

Applying Rome on custom data collected by ourselves, the road surface reconstruction looks abnormal. Just wonder what might be the possible reasons, and what potential steps I might follow to identify the root cause and fix the problems. Any tips/guidance are greatly appreciated, Thanks.
Below are the original undistorted raw image samples,

Below are the reconstruction results,

Sorry for late reply and there may be 3 reasons.

Most important one: poses accuracy.
Elevation estimation. You can check by comparing reconstructed points from traditional methods and your estimated road mesh.
Frame rate. If it is too low, it may cause such results below.

Hi @DRosemei , Thanks a lot for the input.

I used the outoput from a novtel pbox with RTK to derive camera poses. will double check relevant codes.
The distance between subsequent frames are mostly less than 1 meter, though some of the subsequent frames have larger distance, like 8 meters, will look into the issue.

Regarding reconstructing the scene with traditional methods, Which open source tools would you recommend for this purpose?

@LevinJ We all use colmap to reconstruct scenes as you can see here.

Yes, I guessed it would be colmap. Thanks for your confirmation,  @DRosemei


LT4 - continuity

(copying this from slack so you have it documented!)
I've found (or rather, a student has found!) an issue with LT4 in CheckIt. On the ones, like #14, where there is supposed to be a removable discontinuity, the function is actually continuous at that point. There is no hole because one side uses <= (or >=).
A fix -- on the removable ones make sure the "or equal to" does not come up. And then on all the others, maybe occasionally leave it off so students don't associate removable discontinuities as the only spot where they would say neither. Also update the instructions to say "The function h(x) is discontinuous at x=-3. Determine if h(x)
h(x) is left- or right-continuous (or perhaps neither) at x=-3 and then classify the type of discontinuity present at x=-3."

Not a huge fan of "left/right-continuous" as a concept. I also note that you don't define those ideas in https://stevenclontz.github.io/cal1-tbil-2021/LT4.html so should I just remove that part?
Would it make sense to always have part (b) have three cases, x<a, x=a, and x>a? Then we can make sure either x<a and x=a match, x=a and x>a match, or xa match, with the third always not matching, so it's not continuous. Then it's a jump discontinuity in the first two cases, and removable in the last.

Here's an implementation:

(adding back left/right continuity isn't too hard I guess)

I can go either way on left or right continuous. I just went with it and talked about it when we did a Check It problem in class.
I like the solution having the <, =, > in there. Makes it less obvious, and I don't think it's too much of a shift for my students since they have already started these.
Are the infinite discontinuities still getting generated? I wouldn't want to lose those.

Infinite discontinuity was lost (assuming it was there originally; I ended up rewriting). Activity 1.4.13
https://stevenclontz.github.io/cal1-tbil-2021/LT4.html#activity-continuity-piecewise didn't mention it so I didn't take care to make it a possibility. I'm not sure exactly how to generate that case without the issue of having the pattern of only showing fractions for infinite discontinuities.

Linking this in slack for other calc 1 people to weigh in!
Yeah. I think there is some disconnect through a lot of our writing and the CheckIt problems, not unique to this section. I don't know that I realized the infinite wasn't in the activity book, but they did do some in class when they worked on CheckIt problems.
In the other version, jump discontinuities and infinite both had one side as a fraction. See below. Just on the infinite the x-value was an asymptote and the other was not.

TO-DO items for LT4 seem to be

Rewording prompt a. to elicit explanation
Rewording prompt b. to elicit why the function is not cont in the sense of what condition for continuity fails and then classify the type of discontinuity
(already done?) Eliminate right and left-continuous language. I think that we get to the same concept with the rewording from item 2.

I have done the items above. @AbbyANoble check and close the issue if ok!


add verify on api body even no error

add verify on API body even no error, the body may empty and JSON.parse() will throw exception

The ci error reason:
The tests need real TestObject username and api_key, which is configured in the Travis ci config.
According to the document: https://docs.travis-ci.com/user/environment-variables/ variables configured in Travis config is unavailable to the PR from a forked repo.


Add file upload to project_questions

[x] Add file upload to project_questions
[x] Add uploaded files to project export
[x] Add uploaded files to views
[x] Add uploaded files to views export
[x] Add uploaded files to send_task

We will use the new dev branch.

We will use the new dev branch.


Support Canary Release

https://github.com/OpenFunction/OpenFunction/issues/487

Don‘ t merge now.

The e2e test has know issues which is still failing, it's ok not to fix it now.
But it's better to fix https://github.com/OpenFunction/OpenFunction/actions/runs/6109785188/job/16581498056?pr=490 maybe :) @Bryce-huang

@Bryce-huang Is this ready for review?

@Bryce-huang Is this ready for review?

Yes,I'm also doing tests.

@Bryce-huang Is this ready for review?

Yes,I'm also doing tests.

@wrongerror @wanjunlei @tpiperatgod @lizzzcai would you help to review this?

I would suggest the following changes:
@Bryce-huang @wrongerror @wanjunlei @tpiperatgod @lizzzcai what do you think?
apiVersion: core.openfunction.io/v1beta2
kind: Function
metadata:
  name: test-server
spec:
  rolloutStrategy:
    canary:
      steps:
      - weight: 20
        pause:
          duration: 60
      - weight: 80
         pause:
          duration: 120
  image: brycehuang/web-service-image:v1
  serving:
    template:
      containers:
      - imagePullPolicy: IfNotPresent
        name: function
  version: latest
  workloadRuntime: OCIContainer
status:
  # status.serving and status.revision should always be the stable serving and revision
  serving:
  revision:
  rollout:
    canary:
      status: # CanaryStatus
      canaryRevision:
	serving:
	revision

I would suggest the following changes: @Bryce-huang @wrongerror @wanjunlei @tpiperatgod @lizzzcai what do you think?
apiVersion: core.openfunction.io/v1beta2
kind: Function
metadata:
  name: test-server
spec:
  rolloutStrategy:
    canary:
      steps:
      - weight: 20
        pause:
          duration: 60
      - weight: 80
        pause:
          duration: 120
  image: brycehuang/web-service-image:v1
  serving:
    template:
      containers:
      - imagePullPolicy: IfNotPresent
        name: function
  version: latest
  workloadRuntime: OCIContainer
status:
  # status.serving and status.revision should always be the stable serving and revision
  # status.canary.serving and status.canary.revision should be the canary ones
  serving:
  revision:
  rollout:
    canary:
      status: # CanaryStatus
      serving:
      revision:

I would suggest the following changes: @Bryce-huang @wrongerror @wanjunlei @tpiperatgod @lizzzcai what do you think?
apiVersion: core.openfunction.io/v1beta2
kind: Function
metadata:
  name: test-server
spec:
  rolloutStrategy:
    canary:
      steps:
      - weight: 20
        pause:
          duration: 60
      - weight: 80
        pause:
          duration: 120
  image: brycehuang/web-service-image:v1
  serving:
    template:
      containers:
      - imagePullPolicy: IfNotPresent
        name: function
  version: latest
  workloadRuntime: OCIContainer
status:
  # status.serving and status.revision should always be the stable serving and revision
  # status.canary.serving and status.canary.revision should be the canary ones
  serving:
  revision:
  rollout:
    canary:
      status: # CanaryStatus
      serving:
      revision:

LGTM


Redirect the browser to the authorization server(keycloak 9.0.3) failed, getting invalid Parameter - redirect uri error

I tried setting up the  Documentation Requirements Lookup Service (DRLS) System in my Windows machine
On selecting the patient and their corresponding Device/Medication/Service requests from the available list in crd-request-generator and on submitting the request, the CRD server responses back a set of Cards.
And on clicking the cards, smart on fhir(DTR) redirection happens and then failed in loading keycloak authentication page, because of invalid parameter in the redirect uri.
Following is the Redirected URL generated from DTR:
http://localhost:8080/test-ehr/auth?response_type=code&client_id=app-login&scope=launch%20user%2FObservation.read%20user%2FPatient.read%20patient%2FObservation.read%20patient%2FPatient.read%20patient%2FCoverage.read%20patient%2FCondition.read%20user%2FPractitioner.read&redirect_uri=http%3A%2F%2Flocalhost%3A3005%2Findex&aud=http%3A%2F%2Flocalhost%3A8080%2Ftest-ehr%2Fr4&launch=338b46e3-b72f-4ca2-98f3-39e72dc6bc1e&state=36357282

I tried setting up Documentation Requirements Lookup Service locally all modules are up and running except redirection after keycloak authentication(i have given "*" redirect_uri in keycloak client app-login as its local environment)
http://localhost:8180/auth/realms/ClientFhirServer/protocol/openid-connect/auth?response_type=code&client_id=app-login&scope=launch+user%2FObservation.read+user%2FPatient.read+patient%2FObservation.read+patient%2FPatient.read+patient%2FCoverage.read+patient%2FCondition.read+user%2FPractitioner.read&redirect_uri=http%3A%2F%2F0%3A0%3A0%3A0%3A0%3A0%3A0%3A1%3A8080%2Ftest-ehr%2F_auth%2F2f7352e0-21e7-4985-841a-a33d4e8dbed7%3Fredirect_uri%3Dhttp%3A%2F%2Flocalhost%3A3005%2Findex&aud=http%3A%2F%2Flocalhost%3A8080%2Ftest-ehr%2Fr4%2F&state=27185624&launch=2f7352e0-21e7-4985-841a-a33d4e8dbed7
Attached SS for reference.

Only for local deployment you can provide '*' in app-login client.

.
it worked for me

Facing the same issue in windows environment.
This is the redirect url that got generated
http://localhost:8180/realms/ClientFhirServer/protocol/openid-connect/auth?response_type=code&client_id=app-login&scope=launch+user%2FObservation.read+user%2FPatient.read+patient%2FObservation.read+patient%2FPatient.read+patient%2FCoverage.read+patient%2FCondition.read+user%2FPractitioner.read&redirect_uri=0%3A0%3A0%3A0%3A0%3A0%3A0%3A1%3A8080%2Ftest-ehr%2F_auth%2F18350934-4b65-4821-bbc7-cbd80b805072%3Fredirect_uri%3Dhttp%3A%2F%2Flocalhost%3A3005%2Findex&aud=http%3A%2F%2Flocalhost%3A8080%2Ftest-ehr%2Fr4&state=17126996&launch=18350934-4b65-4821-bbc7-cbd80b805072
Seems some issue with the redirect url
edirect_uri=0%3A0%3A0%3A0%3A0%3A0%3A0%3A1%3A8080%2Ftest-ehr%2F_auth%2F18350934-4b65-4821-bbc7-
if i modify the protocol part and hit the url then i am able to see the login server. However i am seeing an empty screen after login

DTR url :
http://localhost:3005/index?state=193341&code=29e4818b-c1a8-4450-8617-7fdc41349dcb.4ce486bc-5dae-4228-9d6f-2c0089770f3b.5dd19d73-8f92-46a1-8751-c5aa3b5b5285

Facing this same issue still

Unable to completely reproduce this issue, but a fix was checked in to the Test EHR server ( https://github.com/HL7-DaVinci/test-ehr/ ) that should at least partially resolve an issue with the redirect back from the Keycloak server to the Test EHR server.
Feel free to reopen if necessary.


fix numbering on page "Using variants"

closes #4277

should I indent with 2 spaces or 4 spaces?

based on file "Testing.rst", there are 3 spaces!🤯
Well, I guess it's a compromise between 2 and 4

noticed that in that file there are 3 spaces everywhere, but here it's 2 spaces,
so changed to 2 spaces


Add username ssh api call

Added username and sshkey to create image api call based on RHELPLAN-90918
Note:
Sending properties data in the format requested in above rhelplan. Don't know if it needed to be in that format. (first letter uppercase)
@boaz0 Please review

Codecov Report

Merging #112 (ff1c254) into master (1d63482) will increase coverage by 0.16%.
The diff coverage is 0.00%.

:exclamation: Current head ff1c254 differs from pull request most recent head 9a89250. Consider uploading reports for the commit 9a89250 to get more accurate results

@@            Coverage Diff             @@
##           master     #112      +/-   ##
==========================================
+ Coverage   28.07%   28.23%   +0.16%
==========================================
  Files          70       70
  Lines        1211     1204       -7
  Branches      316      311       -5
==========================================
  Hits          340      340
+ Misses        802      796       -6
+ Partials       69       68       -1

Impacted Files
Coverage Δ

src/Routes/ImageManagerDetail/DetailsHeader.js
0.00% <0.00%> (ø)

src/Routes/ImageManagerDetail/ImageDetail.js
0.00% <ø> (ø)

src/api/index.js
53.33% <ø> (ø)

Continue to review full report at Codecov.

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

@resoluteCoder I would rather you to create a PR per change. It seems like you opened two PRs that overlaps.
I am closing #111 in favor of this.


Update library docs: adapter-transformers -> adapters

Updated docs page of adapter-transformers library to our newly released Adapters library. Tried to keep the general structure of the doc as is and only update code snippets, some explanation texts and necessary bits.
Todo:

Update code & text in "Use in Adapter Transformers" popup window on the hub
Update screenshots of hub page

cc @lenglaender @hSterz @TimoImhof

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Let me know whenever this PR is ready for review and I can take another look

@osanseviero Thanks! The only missing thing is to update the screenshots, otherwise it's ready for review. Should we update these ourselves and contribute to the HF dataset the old ones were located?

Yes, please, if you could open a PR to update the images that would be great!


feat: added support for pre-commit

Adds support for pre-commit as per discussion #1360
Relies on the docker image releases, & this referenced image tag shouble be incremented with each package release; i.e. updating the entry field in .pre-commit-hooks.yaml.
An example .pre-commit-config.yaml is provided in the changes to the README.

Yeah, great suggestion @knightdave & I agree. When I was looking into originally, I was focused on getting it running from source rather than a system executable from release, which I reckon is more common. I do like the idea of not requiring docker too.

Thanks for your contribution @MattTimms and for the insights @knightdave. Great team effort!


Fix missing pyimzml.ontology module in package

Ack, sorry, was not up to date on my emails when I merged this. Didn't see the parallel efforts. I'll revert.


UnmarshalJSON vs UnmarshalRQL

If I understand correctly, gorethink gets query result data from DB as a thinly wrapped JSON, json.Unmarshal's it into a map, then unmarshals the map into the user-provided object. I suspect you chose to copy the data one extra time (vs directly unmarshaling JSON to the user's object) to support user-provided struct tags + support for Rethingk-specific data types like time, binary & geo. You also support rethink-specific unmarshaling UnmarshalRQL.
Now, that represents somewhat of a problem. I want to add Rethink as another backend to a larger project. One of the data types I want to store is defined as an interface. It has the standard decoding interfaces defined: sql.Scanner, encoding.BinaryUnmarshaler, encoding.TextUnmarshaler, json.Unmarshaler. None of these work with gorethink. I have to require UnmarshalRQL at the top level even if the system uses a different DB backend. Alternatively, I can read the result into a map then unmarshal it manually into a user-provided object, copying it for the second time. Neither seems like a clean solution.
On a related note, gorethink ignores json.Marshaler if it's provided for a value (as opposite to a pointer). Why?

You have understood the unmarshalling code correctly, RethinkDBs pseudotypes mean that the second step is required. Unfortunately it is not currently possible to add support for Unmarshaler as there is no way to skip the initial JSON > map step. Well that is not entirely true if a json.Unmarshaler is detected the intermediate "map" representation of the result could be encoded as JSON and then passed to the  UnmarshalJSON function however in my opinion this is much less tidy than just using UnmarshalRQL.
I think for now just adding the UnmarshalRQL function is the best solution.

Thanks, that makes sense. Geometry is not a problem but binary & time are. Unless you are willing to expose custom binary & time types to the user, the way sql package exposes nullable types. Which may be an acceptable tradeoff because it would substantially improve driver's performance and simplify the code base.
P.S. My current interest is more academic because I circumvented the immediate problem.

In my opinion exposing custom types for the driver is even less flexible than having custom interfaces. Also while I understand that having an extra encoding step is unfortunate I think making the driver as easy to use as possible is more important for now.
For that reason I am going to close this issue for now unless anybody has any other ideas for how to add support for json.Marshaler/json.Unmarshaler. However thanks @or-else for asking a good question :)


Designating a module as library code

It is possible right now to call JsParse() or JsRun() using the flag JsParseScriptAttributeLibraryCode to designate the script as library code and make it invisible to the debugger.  It doesn't appear to be possible to do the same for ES modules.
Can this be done?

@fatcerberus thanks for reporting this.
@liminzhu @suwc this should be allowed, right?

+@agarwal-sandeep who is familiar with the debugging APIs.
Seems reasonable that we should allow host/debugger to ignore certain modules.

+1 we should allow.

In my case when debugging an ES module-based codebase, the debugger's first stop tends to be one of my runtime modules, rather than the game's root module.  So it'd be nice if I could step over those by default.

We have internal operations we use to do this for Intl.js library code. Maybe we can expose those as JSRT APIs?

We have internal operations we use to do this for Intl.js library code. Maybe we can expose those as JSRT APIs?

As noted above, you can already do this with normal scripts, using the parse flag JsParseScriptAttributeLibraryCode.  It's just that there's no equivalent option for ES modules.


Need Azure Search to remove data that is not in data-source

I have a file in blob storage folder/new/data.json.
It contains json array.
[
    {
        "name": "a",
        "data": {
            "1":"something1",
            "2":"something2"

        }
    },
    {
        "name": "b",
        "data": {
            "1":"something1",
            "2":"something2"
        }
    }
]

my datasource body :
{
    "name" : "datasource",
    "type" : "azureblob",
    "credentials" : { "connectionString" : "MyStorageConnStrning" },
    "container" : { "name" : "mycontaner", "query" : "folder/new" }
}

my index body:
{
    "name" : "index",
    "fields": [
       { "name": "id", "type": "Edm.String", "key": true, "searchable": false },
       { "name": "name", "type": "Edm.String", "searchable": true, "filterable": false, "sortable": true, "facetable": true},
       { "name": "data", "type": "Edm.String", "searchable": false}
    ]
}

indexer body:
{
    "name" : "indexer",
    "dataSourceName" : "datasource",
    "targetIndexName" : "index",
    "parameters" : { "configuration" : { "parsingMode" : "jsonArray" } }
}

when created i can search for a and b and get result.
now i have modified folder/new/data.json to
[
    {
        "name": "aa",
        "data": {
            "1":"something1",
            "2":"something2"

        }
    }
]

and run the indexer.
but I can still search b and get result.
as i get a auto generates unique id for each document, I can get the unique id for b and can make a single request to delete that document from the index.
Is there  no way that index automatically removes data that is not in the data-source?

Hello @nafisIslam Thank you for your feedback! This channel is for driving improvements towards MS Docs. In order to best address your question with the right team, could you provide us with the URL of the MS Doc that you were following, if any?


bunx shadcn-ui@latest init

file:///tmp/shadcn-ui@latest--bunx/node_modules/shadcn-ui/dist/index.js:67
Failed to load tsconfig.json. ${tsConfig.message ?? ""}.trim()
^
SyntaxError: Unexpected token '?'
at Loader.moduleStrategy (internal/modules/esm/translators.js:133:18)
at async link (internal/modules/esm/module_job.js:42:21)
error: "shadcn-ui" exited with code 1 (SIGHUP)

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Pretrained checkpoints?

Are there any pretrained checkpoints available for feature extraction? Or is there a way to return the intermediate feature maps generated for further downstream tasks?

Fixed by #234


Run Terminal as Admin is not working in the 1.8

If I hold shift on Taskbar or inside Windows explorer it does not run as admin anymore.
But it does show up a new menu item.

Hi @juliocoliveira
Change the value of the admin property to 1 for always run as administrator
/shell/imports/terminal.nss

I don't want to always run terminal as Admin, The shift key trick is removed? Disabled? Or is that just a bug?

Yes it's a bug. It was fixed immediately and will be released at the end of the week.
This is caused by separating the keyboard event to execute the command from the show menu event.

Hi @juliocoliveira
The update is available in portable versions

Hi @juliocoliveira The update is available in portable versions

How about the installer?

I updated and is working now. Still there is no text or icon indicating that it will run as Administrator as previous versions had, that is normal? By design? Or is another bug?

For me not working when installed via installer…
Did you install it via portable?
On Tue, 24 Jan 2023 at 14:53 Júlio C. Oliveira @.***>
wrote:

I updated and is working now. Still there is no text or icon indicating
that it will run as Administrator as previous versions had, that is normal?
By design? Or is another bug?
—
Reply to this email directly, view it on GitHub
https://github.com/moudey/Shell/issues/108#issuecomment-1401899619, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A4MXGIYELWY3RR4KRNP2ZPLWT7GCXANCNFSM6AAAAAAT6QU3I4
.
You are receiving this because you commented.Message ID:
@.***>

--
Sent from Yazan’s iPhone

For me not working when installed via installer… Did you install it via portable?
On Tue, 24 Jan 2023 at 14:53 Júlio C. Oliveira @.> wrote: I updated and is working now. Still there is no text or icon indicating that it will run as Administrator as previous versions had, that is normal? By design? Or is another bug? — Reply to this email directly, view it on GitHub <#108 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4MXGIYELWY3RR4KRNP2ZPLWT7GCXANCNFSM6AAAAAAT6QU3I4 . You are receiving this because you commented.Message ID: @.>
-- Sent from Yazan’s iPhone

@bigplayer-ai said portable, So...

@moudey said portable, so I guess installer is outdated  yet.

I've added to the terminal.nss, But it didn't showed a Text saying something like: "This  will run as administrator" as previous versions had, It only showed a dummy item:

Tip are disabled by default
set
{
    tip.enabled=true
}

As you can see I'm pressing shift and there is no text or tooltip saying that that window will be run as Administrator, This was removed by mistake or by design. I don't know, But would be cool to be back as a feature.

This is from the readme.me:

Would be cool to have this feature back.

add this code to terminal item
tip=["Note\nHold Shift key to run as administrator", tip.warning, 0.2]

I did that, and is working, Thanks, still I would prefer that this is enabled by default, When a new update comes out I'll have to do this all over again.

Is the installer updated yet?

Is the installer updated yet?

Soon...

@moudey did you update the installer?


Resources: Frontend

Learning HTML & CSS

You don't need to install the specific text editor each tutorial recommends. Just pick one editor that you like or is already using, and stick with it.

Static Tutorials

:green_heart: https://marksheet.io/ (this one I like very much)
:green_heart: https://www.internetingishard.com/ (also pretty good)
https://www.freecodecamp.org/learn/responsive-web-design/
Mozilla HTML Basics https://developer.mozilla.org/en-US/docs/Learn/Getting_started_with_the_web/HTML_basics
Mozilla CSS Basics https://developer.mozilla.org/en-US/docs/Learn/Getting_started_with_the_web/CSS_basics

Interactive Tutorials

https://www.learn-html.org/

Videos

HTML & CSS for beginners Series — Kevin Powell https://youtube.com/playlist?list=PL4-IK0AVhVjM0xE0K2uZRvsM7LkIhsPT-
HTML & CSS Crash Course Tutorial — The Net Ninja https://www.youtube.com/playlist?list=PL4cUxeGkcC9ivBf_eKCPIAYXWzLlPAm6G

Learning CSS Flexbox

Flexbox is the most important layout feature in CSS so really pay attention.
Also don't worry about CSS Grid. Anything that can be done with Grid can be done with Flexbox. We'll only be using Flexbox.

Static Tutorials

https://jstutorial.medium.com/flexbox-the-animated-tutorial-8075cbe4c1b2
https://www.freecodecamp.org/news/flexbox-the-ultimate-css-flex-cheatsheet/
:green_heart: https://css-tricks.com/snippets/css/a-guide-to-flexbox/ (my personal fav for continued reference)

Interactive Tutorials

:green_heart: https://flexboxfroggy.com/ (learn & play games at the same time!)

Videos

Learn Flexbox in 15 Minutes —  Web Dev Simplified https://youtu.be/fYq5PXgSsbE
:green_heart: What the Flexbox? Short video series. — Wes Bos https://youtube.com/playlist?list=PLu8EoSxDXHP7xj_y6NIAhy0wuCd4uVdid
Flexbox CSS In 20 Minutes — Traversy Media https://youtu.be/JJSoEo8JSnc?t=354

Learning JavaScript

If any of these ask you to install node or Node.js, DO NOT. You don't need it. Skip that part.

Static Tutorials

Mozilla JavaScript Basics https://developer.mozilla.org/en-US/docs/Learn/Getting_started_with_the_web/JavaScript_basics
https://javascript.info/

Interactive Tutorials

:green_heart: https://jgthms.com/javascript-in-14-minutes/ (my favorite JS tutorial)
https://www.learn-js.org/

Videos

JavaScript Tutorial for Beginners: Learn JavaScript in 1 Hour — Mosh https://www.youtube.com/watch?v=W6NZfCO5SIk
JavaScript Crash Course For Beginners — Traversy Media https://www.youtube.com/watch?v=hdI2bqOjy3c

Useful Resources

Start a simple HTTP server with Python3

Run the command below in a folder to start a simple HTTP server in that folder.
For Windows:

python3 -m http.server 80 --bind <IP_ADDRESS>

For MacOS and Linux:

sudo python3 -m http.server 80 --bind <IP_ADDRESS>

Advanced REST Client

Allows you to test and play with APIs and save your options
https://install.advancedrestclient.com/install

Front end recourses


Add an authentication

As headscale's admin console, I recommend that you add authentication before entering the console page, as this will make the service insecure if it is exposed to the public.

Any sensitive data is saved locally to the browser, so it does not expose sensitive information.
You can test by saving the key and opening the same site in incognito mode.
A proper OIDC based system would be better, but not feasible without a compatible backend.

I tested what you said and proved that you are absolutely right.


Can small object be detected？

Does anyone konw if this model can detect small object such as a cup？Because I find the objects in the sunrgb dataset are often big， such as bed，sofa， table.

I think that if you want to retrain the model to detect cup you should modify (lower) the radius of the set abstraction in models/backbone_module.py


Stop logging expected error values

Some HCS* functions return errors to signify a success with additional
information. In these cases we need to skip logging the error because the use
cases is expected and this isn't an error for the client who handles it.
 Shouldn't we just filter out expected errors but keep the spans themselves in place?

Each of those starts is a "sub-span" which logs the error again, and again, and again. They add no additional "new" context to a call because it was already logged as the entrypoint span in the shim. Its all just "excess"

Each of those starts is a "sub-span" which logs the error again, and again, and again. They add no additional "new" context to a call because it was already logged as the entrypoint span in the shim. Its all just "excess"

Okay, as long as we're keeping the top-level spans that sounds good.


Modeling late entries in CPH

When modeling late entries in a kmf, the survival function would be more 'pessimistic' about customer survival, which makes sense.
kmf.fit(data["Duration"], event_observed=data["Observed"], entry=data["W"], label='modeling late entries')

However, when modeling late entries in a cph, we see the opposite effect, being, getting a better survival function than when ignoring late entries.
cph.fit(data, 'Duration', 'Observed',entry_col='W')

You can also refer to my repo here: https://github.com/ah-sadek/CustomerAnalytics

It is worthy to note that you mentioned that there is a bug when calculating the survival function in the Cox model with late entries. Specifically, while the coefficient estimate calculations handle the late entries, the calculation of the survival function does not.

Thanks @ah-sadek!


How can I get the user alias who click the button in Teams Card

@XimenaWang You could fetch the users alias by using Bot context. Please let us know if you need more help.

@Gousia-MSFT - Hi, I am using the "Office 365 Connector Card" not Bot, Is there a way to get user information directly from Card ?


Stackoverflow Exception when using some controls in ItemsControl with a data-bound Items property

Describe the bug
When adding some controls (eg: ToggleSwitch) to a DataTemplate on ItemsControl it will cause a Stackoverflow Exception in a recursive measure call.
To Reproduce
Steps to reproduce the behavior:
Reproduced in full here: https://github.com/DevJohnC/ItemsControlStackOverflowReproduction

Create a new Avalonia project
Add an array or other list to the MainViewModel
Create an ItemsControl in MainView.axml and bind the Items property to the list you just created on MainViewModel
Create a DataTemplate for the bound items and add a ToggleSwitch
Run a debug session and observe the Stackoverflow exception

Expected behavior
Controls render as appropriate without causing a crash
Desktop (please complete the following information):

OS: Windows10, macOS 12.5 (Monterey)
Version: 0.10.18

Additional context
The issue isn't exclusive to ToggleSwitch; TextBlock is known to be exempt from the issue however.
Assigning a fixed size to either a container like Panel nor ToggleSwitch directly fixed the issue.

It's probably this: https://github.com/AvaloniaUI/Avalonia/issues/7985#issuecomment-1099696016.

@jp2masa yes. DataTemplates.DataTemplate without x:DataType in it will be applied to literally everything that doesn't have any other template. Which very likely can cause stack overflow.
While that's by design:

you can assign multiple templates inside of the DataTemplates and it will be inherited
you can create DataTemplate without DataType, so it can be applied on any object type
Together these features work badly. It was restricted with additional validation before, but only for 11.0, as it's technically a breaking change - https://github.com/AvaloniaUI/Avalonia/pull/8221

Solution:

Replace DataTemplates with single ItemTemplate. ItemTemplate is not inherited and isolated on the items control, that's why empty DataType will work fine.
Set DataType to a specific one.

Duplicate of #7985


lexical order within line

Currently the behavior has:
from my.package import Foo, apple, banana.
Is there a way to enforce the lexical order, so it will look like:
from my.package import apple, banana, Foo?
I checked the wiki but can't seem to find what I desire.

Hi @wisechengyi,
Sorry for the delayed answer!
What you are looking for is order-by-type: False or
isort --dont-order-by-type from the command line.
Thanks!
~Timothy


topics page: list of acronyms

It might be useful to have a visible list of acronyms somewhere on the topics page to facilitate the readers. It is probably not urgent, especially if hard to implement, but something we should keep in mind.

Maybe we should make a list of acronyms here (in GitHub) first.


Wrong model link for CTC training on commonvoice Italy

Hello,
Could you please fix the model link for CTC training on commonvoice Italy? The link you provided seems not the good one.
https://github.com/speechbrain/speechbrain/tree/develop/recipes/CommonVoice/ASR/CTC
Thanks.

Hi,
thank you for noticing. I cannot find the CTC wav2vect model for Italian in the project google drive and I cannot find it anymore in my local folders. I thus started a new training and the model will be available in few days.

thank you!

This is fixed now. Thank you!


HTTPS key & cert launch parameter for CoCo-AS

Currently CoCo AS will be launched in insecure http server mode. It is expected to configure a HTTPS cert and private key to make CoCo AS a HTTPS server.

Now CoCo-AS supports HTTPS mode for restful https://github.com/confidential-containers/kbs/blob/main/attestation-service/docs/restful-as.md#https-support
grpc is more like a protocol for interaction between internal components and thus does not require https


PR: Improve release instructions

This streamlines and simplifies our release instructions.
It also fixes some errors in our Changelog and Announcements files.

Good point! I'll add a new After the release section to mention those details.


Add the option to not consider the internal energy of the gas when performing the unbinding

Does exactly what the title says :)
I think I set the default value correctly such that users who want the normal (old) behaviour are unaffected and can keep using their normal config files.
The rationale for this is that in the EAGLE family of model, we do heat particles to high temperatures when performing feedback but these particles are clearly not unbound.

@pelahi, I'd be interested in your thoughts on this as well. I also saw that the gas' U is used in places as part of the kinetic energy when doing a kappa_rot decomposition. Is that standard?

Hi @MatthieuSchaller, this was added ages ago to keep the quantity consistent with what AHF calculated. I can provide an update so that an config option could disable it.

Thanks. I don't think that's necessary; I was just puzzled by it.
The other aspect I was wondering about is the use of U to distinguish groups in phase space. Am I maybe misreading the code or is U indeed added when considering phase-space decomposition?

From my point of view the changes look good, and from the conversation above there seems to be no reason not to merge, so I'll be doing that.


Added no button/ no outline option

Added a property to display the dropdown without the button (i.e., only the text).
Normal dropdown:

Proposed bare option:

Bare option opened:

Bare option no chevron and opened:

Bare option disabled:

Updated the demo to include the bare option:

@runn-vermel @benoitjchevalier Let me know if it makes sense to you guys or if there are updates you suggest.
I showed this to Wade but might need to show the other designers.

I forget if we already discussed this via Slack. This is already possible with CSS variables (like so), I'm not sure if it makes sense to build this in as a property.


--HEAD support for arx-libertatis

I have an open pull req with upstream to fix the build on Mac OS X. Assuming this is accepted, will need to install --HEAD until the next release is cut.

The test fails because of the stable version (1.1.2).
This patch may be a help. (auto-generated from a commit by Github)
https://github.com/arx/ArxLibertatis/commit/442ba4af978160abd3856a9daec38f5b6e213cb4.patch

Ah, I see. The buildbot only runs "brew install arx-libertatis". This will fail until upstream cut their next release. My pull req only fixed installing --HEAD for now.
I think it's best to be patient and wait. Until then, anyone hitting the problem can find this pull req.

 I don't know how much HEAD has changed since the current release; do you think it's feasible to backport the fix as a patch to the stable version?

Ping

Sorry for the delay, I was travelling. My contribution was only minor, the bulk of the work was done upstream, e.g. defect #918, together with probably many more fixes in their git repo. So I think backporting all the necessary fixes to 1.1.2 might be a lot of work.
I think it's best to wait for a new release from upstream. Anybody determined can apply this patch to their local clone & build from --HEAD.


Add support for "validation_error" at the page level

Fixes the following unhandled activity errors:
https://app.datadoghq.eu/logs?query="Unhandled Activity Error" &cols=host%2Cservice&event=AgAAAYi5jkgSkLAEJAAAAAAAAAAYAAAAAEFZaTVqa3Q4QUFEbno4eTVvcGFwNkFBSAAAACQAAAAAMDE4OGI5OGUtNjc1My00YzBlLWI0ODAtNzliMWE5ZGRhMTc2&index=*&messageDisplay=inline&stream_sort=desc&viz=stream&from_ts=1686738667085&to_ts=1686739567085&live=true
I pulled the notion pages by searching for this string: "Fetching page from Notion API." and filtering on the same workspaceId
https://app.datadoghq.eu/logs?query=service%3Aconnectors-worker "Fetching page from Notion API." %40workspaceId%3A2ddbc0204d&cols=host%2Cservice&event=AgAAAYi5gWdFiTY0EAAAAAAAAAAYAAAAAEFZaTVnWFRnQUFEaWZXZHgwWVh4MWdBQgAAACQAAAAAMDE4OGI5ODEtODg1Ny00ZmU4LWI3NWEtNzViYmE2YTEzOGE2&index=*&messageDisplay=inline&stream_sort=desc&viz=stream&from_ts=1686738639172&to_ts=1686739539172&live=true
From there I extracted the failing pageIds
I also pulled the Notion access token for this workspace and then followed the runbook to execute locally the retrieval of the page: https://www.notion.so/dust-tt/Runbook-Notion-connector-b3b082e3f8a94ad9b4d42dfa89748582?pvs=4
Notion returns an error when listing the blocks children given the pageId. Adding support for validation_errors and returning an empty list of blocks (see https://github.com/dust-tt/dust/blob/main/connectors/src/connectors/notion/lib/notion_api.ts#L304)
We still get the description of the page as part of the final rendering which still contains useful information

cc @fontanierh

(post-merge review welcome)


Fix KeyError when resolvers stats not supported

Probably wrapping too many (also feature-wise) updates here.
Gonna close this PR, let me know if you still want me to open it back.


Added relative path to /index.html

I added relative path for floeproject.org/index.html so that it works correctly even for local machines.

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

@jhung  @jobara reviews?


Create Bootnodes & ENR

I see in values.env there is
export BEACON_STATIC_ENR=
and what I understand before getting enr must have genesis.json and initialize it with datadir. or how can I get the bootnode so that it can be entered into values.env?
i made my own public blockchain sir.

You can run a generic bootnode such as this one: https://github.com/protolambda/eth2-bootnode
The generic bootnode ENR can then be used.


Setup Weapon Description for Lightsabers to hopefully stop crash being caused.

https://github.com/Servius/sepcrisistroopsgui/blob/4b32d3a7ebc6e59e4ee320db160c9aad6b8f93f1/SeparatistCrisis/ModuleData/weapons_blaster.xml#L409

No longer causing a crash, but I cant switch to the weapon for some reason


Build(deps-dev): Bump @types/node from 20.11.0 to 20.11.5 (#5461)

Pull request opened by the merge tool on behalf of #5461

Merged in 9c2a2d8f10d7a8798eaa3a1f0bf3e0793a3df3a3


has_one does not always allow a unique constraint

A has_one may be a refinement of a has_many. In this case, there cannot and should not be a unique constraint on the foreign key. Currently the gem does not distinguish.
class Blog
  has_many :posts
  has_one :most_recent_post, -> { order(published_at: :desc) }, class_name: 'Post'
  ...
end

This could potentially be addressed by getting the class of each has_one and checking: If that class has a has_many relationship as well then it would disqualify it for a unique constraint. Association reflections would be useful here:

http://api.rubyonrails.org/classes/ActiveRecord/Reflection/ClassMethods.html
http://www.rubydoc.info/docs/rails/3.0.0/ActiveRecord/Reflection/AssociationReflection

Yep, totally, makes sense.
I'm kind of wondering about similar use cases, but slightly different implementations too. When I've done this kind of thing in the past, I've used an instance method, rather than a has_one, to implement things like #most_recent_post (this has the side benefit that I don't have to worry about accidental deletion if someone uses the generated setter). If someone went in the other direction, and implemented #posts as an instance method, and left the has_one as an association, we wouldn't be able to spot it with this strategy.
That's the only concrete case I've got in mind right now, but my gut says that if there's this one use case that we've missed up until this point, there might be others as well that I'm not thinking of (which makes me just want to give users an escape hatch).

@trptcolin @adsteel Bumping this old issue! We recently ran into this with two separate models that were both referencing a polymorphic field, one with has_one, the other with has_many. We were able to work around this by defining a getter on the model for the has_one relationship.
For the escape hatch: are you thinking an additional option set on the has_one relationship / uniqueness validation? Using a comment will be hard since the gem relies on an introspection class instead of a full-blown parser. Something like:
class SomeModel
  has_one :something, disable_consistency_fail_unique_constraint_check: true
  # or
  validates :something_else, uniqueness: true, disable_consistency_fail_unique_constraint_check: true
end

Hey @trptcolin is there a solution to this (if there's a config file it doesn't seem documented anywhere)? We have a very similar use case where a has_one association is a scoped version of a has_many association. Redefining the has_one association as an instance method is not appropriate for us because we join through this association a lot, e.g. most of the time we impose an ordering based on an attribute on the association.

@louietyj hasn't been implemented yet, but I'd love to a PR for it!


How to make the mouse hover over the tooltip's content

It is very distressed can not copy tooltip's content

@Mikaaaaaaa7 did you found a solution on that? I face the same problem.


You are putting compiled code into your crate file

I've downloaded all crates on crates.io and versions 1.0.1 and 0.1.0 of cld2-sys occupy two of the top three places when it comes to size. When I open the archive files, I see a bunch of compiled artifacts inside. Are they included intentionally?

Unless something has gone seriously wrong, they shouldn't be compiled artifacts. Could you please provide a list of what you're seeing, so that I can compare it?
These crates do contain fairly large data tables used to recognize various languages, which we had to do a fair bit of work to successfully compress enough to upload to crates.io.

@emk an overview of the sizes of the crate files:
$ ls -alh crate-archives/cl/d2/cld2-sys/ | sed 's/  / /g' | cut -d" " -f5,9
[...]
392 cld2-sys-0.0.0.crate
6,5M cld2-sys-0.0.2.crate
6,5M cld2-sys-0.0.3.crate
6,5M cld2-sys-0.0.5.crate
6,5M cld2-sys-0.0.6.crate
6,5M cld2-sys-0.0.7.crate
6,5M cld2-sys-0.0.9.crate
16M cld2-sys-0.1.0.crate
6,5M cld2-sys-1.0.0.crate
16M cld2-sys-1.0.1.crate

So there seem to be two affected versions: 0.1.0 and 1.0.1.
Object files inside the 1.0.1 .crate file:
$ tar -ztvf crate-archives/cl/d2/cld2-sys/cld2-sys-1.0.1.crate | grep "\.o$"
[...]
-rw-rw-r-- 1000/1000     10328 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/cld2_generated_cjk_compatible.o
-rw-rw-r-- 1000/1000   2125312 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/cld2_generated_deltaocta0122.o
-rw-rw-r-- 1000/1000<PHONE_NUMBER>-11-19 19:22 cld2-sys-1.0.1/cld2/internal/cld2_generated_distinctocta0122.o
-rw-rw-r-- 1000/1000   9537864 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/cld2_generated_quad0122.o
-rw-rw-r-- 1000/1000<PHONE_NUMBER>-11-19 19:22 cld2-sys-1.0.1/cld2/internal/cld_generated_cjk_delta_bi_32.o
-rw-rw-r-- 1000/1000<PHONE_NUMBER>-11-19 19:22 cld2-sys-1.0.1/cld2/internal/cld_generated_cjk_uni_prop_80.o
-rw-rw-r-- 1000/1000     12920 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/cld_generated_score_quad_octa_0122.o
-rw-rw-r-- 1000/1000<PHONE_NUMBER>-11-19 19:22 cld2-sys-1.0.1/cld2/internal/cldutil.o
-rw-rw-r-- 1000/1000     37304 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/cldutil_shared.o
-rw-rw-r-- 1000/1000     65760 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/compact_lang_det.o
-rw-rw-r-- 1000/1000<PHONE_NUMBER>-11-19 19:22 cld2-sys-1.0.1/cld2/internal/compact_lang_det_hint_code.o
-rw-rw-r-- 1000/1000<PHONE_NUMBER>-11-19 19:22 cld2-sys-1.0.1/cld2/internal/compact_lang_det_impl.o
-rw-rw-r-- 1000/1000<PHONE_NUMBER>-11-19 19:22 cld2-sys-1.0.1/cld2/internal/debug.o
-rw-rw-r-- 1000/1000      6440 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/fixunicodevalue.o
-rw-rw-r-- 1000/1000      6536 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/generated_distinct_bi_0.o
-rw-rw-r-- 1000/1000     15400 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/generated_entities.o
-rw-rw-r-- 1000/1000<PHONE_NUMBER>-11-19 19:22 cld2-sys-1.0.1/cld2/internal/generated_language.o
-rw-rw-r-- 1000/1000     54736 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/generated_ulscript.o
-rw-rw-r-- 1000/1000<PHONE_NUMBER>-11-19 19:22 cld2-sys-1.0.1/cld2/internal/getonescriptspan.o
-rw-rw-r-- 1000/1000     63432 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/lang_script.o
-rw-rw-r-- 1000/1000     74944 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/offsetmap.o
-rw-rw-r-- 1000/1000<PHONE_NUMBER>-11-19 19:22 cld2-sys-1.0.1/cld2/internal/scoreonescriptspan.o
-rw-rw-r-- 1000/1000     47704 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/tote.o
-rw-rw-r-- 1000/1000     93552 2015-11-19 19:22 cld2-sys-1.0.1/cld2/internal/utf8statetable.o

The same command applied to the 1.0.0 crate file shows nothing (that one is also 10 MB smaller!), and to the 0.1.0 crate file it also shows a list of .o files.
You can have a look for yourself. The URL scheme one can use to download .crate files is:
https://crates.io/api/v1/crates/{cratename}/{version}/download

Or alternatively, the backend version (the crates.io url above is a redirect to that):
https://d19xqa3lc3clo8.cloudfront.net/crates/{cratename}/{cratename}-{version}.crate

URLs of the .crate files:
https://d19xqa3lc3clo8.cloudfront.net/crates/cld2-sys/cld2-sys-0.1.0.crate
https://d19xqa3lc3clo8.cloudfront.net/crates/cld2-sys/cld2-sys-1.0.0.crate
https://d19xqa3lc3clo8.cloudfront.net/crates/cld2-sys/cld2-sys-1.0.1.crate

Indeed, without the object files the crate files are not small, but with the object files included the crate files are more than two times bigger so it does make a difference :).
Thanks for having a look.

Ah, thank you! Looks like something went wrong with two of the builds.
Anyway, cld2 is probably about to be deprecated in favor of whatlang, which is a native Rust implementation of the same general idea.

@emk I've created a PR to fix the issue: #5
Even if cld2 is deprecated I still think it is better to fix the issue :).

Thank you for the fix. I've merged it, and soft-deprecated cld2 in favor of whatlang.


Introduce new rosVehicle specialization and LaserScanner class

This PR is to fix  #3, #15, #16.
Two new classes have been introduced:

The specialization- rosVehicle: store the per-vehicle functionality/properties.
The LaserScanner class: separate the laser scanners parts from modROS.lua to improve the readability and reusability

Apologies for taking so long.
Holidays, etc. :slightly_frowning_face:

@tckarenchiang: also noticed that luacheck isn't happy about something here.
Is that already fixed in a later commit (not part of this PR perhaps)?

@tckarenchiang: also noticed that luacheck isn't happy about something here.
Is that already fixed in a later commit (not part of this PR perhaps)?

It's fixed now

Apologies for taking so long.
Holidays, etc. 🙁

No problem. Thanks for the review :)


fix_calling_bed branch into master

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Can you add this as a new version? (1.0.1 already exists :p)


Notebook language detected as JSON despite not even being supported by kernel

Open a jupyter notebook, run a cell with something like print('hello world'). Seeing the language detection say to change to JSON, despite the kernel not even supporting JSON.

cc @bpasero , @rebornix


#15 - add property namer strategy

Issue #15

This looks great! Any chance you could add some documentation for it?

Sure!
@trappar  do you think the PropertyNamer should live in the FixtureGenerationContext class (like the ReferenceNamer)? I wasn't sure where to place it but it makes sense to me now that I am reviewing the code.

Hmm... I might have to think a bit about it since I haven't worked on this project once in a while. The question is - is the PropertyNamer a setup step or is it runtime configuration?
If I remember correctly the reason why ReferenceNamer is in the FixtureGenerationContext class is because it's quite possible that from run to run you might want to change how you name references. Is PropertyNamer similar? I'm not sure. What do you think?

Now that you are saying it, actually, we are "normalizing/serializing" objects. So the way of naming properties should not be changed on runtime. Looking at other serializer solutions, the naming strategy is not a "context" thing.
I will add some documentation for this.
Thanks!

Hi @trappar ,
If you don't see any other issue, I think it is ready.
Thanks!

Merged!

@trappar could we create a new release including this?

Done


[#3349] Samples missing deployment templates (template-with-preexisting-rg) - javascript_nodejs

Addresses # 3349
Proposed Changes
This PR adds the bots' deployment templates (template-with-preexisting-rg.json) under the samples/javascript_nodejs folder  using the new Azure Bot resource instead of the Bot Channel Registration.
Detailed Changes
Added the ARM templates of the following samples:

14.nlp-with-orchestrator
23.facebook-events
43.complex-dialog
44.prompt-for-user-input
45.state-management
language-generation/05.a.multi-turn-prompt-with-language-fallback
language-generation/05.multi-turn-prompt

Bot that were not added:

01.console-echo => Console Bot
40.timex-resolution => Console Bot

Testing
The following image shows two bots working with the new ARM Templates.

Promoted to MS # 3469


Ocelot not working with a Windows Authentication

Expected Behavior / New Feature
Ocelot should be working with Windows Authentication
Actual Behavior / Motivation for New Feature
Ocelot should be working with Windows Authentication
Steps to Reproduce the Problem
My gateway is a virtual directory hosted under default site with anonymous access.
And my services are hosted under another virtual directory with windows authentication enabled.
Note: - in IE options settings for "Enable Integrated windows authentication is enabled" from user machine. This is required otherwise the domain.com will not carry the identity.
Default website
----> Gateway = anonymous
-----> services = windows authentication.
With this from user machine when we call with servername.domain.com/gateway/ to get the user name from services, we get the user name for first user.
When the same url is hit by another user he gets the username of the first user, whereas the should have got is his own user name.
what we see is that persist-auth in headers is set to true for communication between gateway and services.
If we enable
----> Gateway = windows authentication
-----> services = windows authentication/anonymous
Authentication does not seem to work even if we add/remove testkey= "Windows".
Please assist on the same.
Specifications

Version:
Platform:
Subsystem:

@soumaz please can you debug and work out why this is. Unfortunately I do not have time to work the problem out at the moment.

@soumaz did you try setting the authPersistSingleRequest to True in your services windowsAuth settings?

Setting this flag to true specifies that authentication persists only for a single request on a connection. IIS resets the authentication at the end of each request, and forces reauthentication on the next request of the session.
The default value is false.

https://docs.microsoft.com/en-us/iis/configuration/system.webserver/security/authentication/windowsauthentication/#configuration
So it all depends on how connections are being kept open or not between ocelot and the service ( didn't check). If they are kept open and you are on the defaults, then it makes sense that there is no reauth for the second user and setting authPersistSingleRequest would solve that.
Of course it would remove a perf optimization on the other hand.

I, I got the same issue and the "authPersistSingleRequest=True" also solved my problem

Can you please show how you have Ocelot configured to get NTLM working. Startup.cs and configuration file for ReRoutes

Good afternoon, I have 2 site in IIS. 1- ocelot api gateway. 2-i microservices.
When authorizing directly through site 2, I easily go to it and log in.
When using ocelot, the authorization window constantly creeps out and is not authorized.
the above tips did not help

Hi,
Make the Authentication for Microservices as Windows and Ocelot as
anonymous. Let me know if this works for you.
On Tue, Oct 29, 2019 at 11:02 AM NoSkillNoLuck<EMAIL_ADDRESS>wrote:

Good afternoon, I have 2 site in IIS. 1- ocelot api gateway. 2-i
microservices.
When authorizing directly through site 2, I easily go to it and log in.
When using ocelot, the authorization window constantly creeps out and is
not authorized.
the above tips did not help
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ThreeMammals/Ocelot/issues/657?email_source=notifications&email_token=ACBC6HAT6IN6UHWBUJOIBTTQQ7DILA5CNFSM4F23LMP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECPJIEI#issuecomment-547263505,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBC6HCMXJDAHZ2NOCQD4BLQQ7DILANCNFSM4F23LMPQ
.

--
Thanks & Regards,
Souvik Mazumdar
Mobile: +91-8861955677

Hello everybody,
we have also struggeled with getting Windows Authentication (Kerberos) working with Ocelot. At first we received 401 unauthorized constantly. In order to fix that we changed the implementation of the HttpClientBuilder class. After we added UseDefaultCredentials=true to the HttpClientHandler, we no longer received 401 unauthorized errors:
   private HttpClientHandler UseNonCookiesHandler(DownstreamContext context)
        {
            return new HttpClientHandler
            {
                AllowAutoRedirect = context.DownstreamReRoute.HttpHandlerOptions.AllowAutoRedirect,
                UseCookies = context.DownstreamReRoute.HttpHandlerOptions.UseCookieContainer,
                UseProxy = context.DownstreamReRoute.HttpHandlerOptions.UseProxy,
                MaxConnectionsPerServer = context.DownstreamReRoute.HttpHandlerOptions.MaxConnectionsPerServer,
                UseDefaultCredentials = true // this line make the difference
            };
        }

I guess setting UseDefaultCredentials=true is required to pass the kerberos tokens to the downstream service. It was also necessary to call ChallengeAsync to generate the WWW-authenticate header required for the Windows Authentication flow (see: https://github.com/ThreeMammals/Ocelot/issues/1144)
By the way setting authPersistSingleRequest alone, didn't solve my problem.
It would be great if Windows Authentication could be integrated into Ocelot, it's still used by a lot of people out there.
Thanks in advance.
Keep up the great work!

@ArminNaish Can you provide a basic working demo. I'm struggling to get ocelot working with windows authentication. I keep getting the windows signin prompt.

Please use version 15.0.6. Versions after that don't work well with Windows
Authentication.
Thanks & Regards
On Tue 18 Aug, 2020, 6:21 AM Daudi Husbands<EMAIL_ADDRESS>wrote:

@ArminNaish https://github.com/ArminNaish Can you provide a basic
working demo. I'm struggling to get ocelot working with windows
authentication. I keep getting the windows signin prompt.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ThreeMammals/Ocelot/issues/657#issuecomment-675187248,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBC6HH3ITV2G64MCBGEQQ3SBHGALANCNFSM4F23LMPQ
.

Hello @soumaz, I'm also having trouble with Windows Authentication. My setup is as following:

ASP.NET Core web app with Angular (SPA) and Ocelot gateway (v15.0.6.) running in IIS on server X
Katana self-hosted Web API running in a console app (.NET Framework 4.8) on server Y -> can't run in IIS because reasons...

When I hit one of the endpoints (/api/test or /api/hello) from my workstation (so neither server X or server Y) with Postman setting Authentication to NTLM it all works fine (succesful requests, passes Authorize attribute, I see my Windows Identity name being returned). I have set up Fiddler as a reverse proxy on server X which forwards requests to the Web API (2). I've done this so I can see all the requests coming in to the Web API.
When I try to hit the endpoints through the ocelot gateway however, I receive a status 400 response (no details), both in the Angular web app and when using Postman (see Fiddler screenshots below).

I've tried this both with Windows Authentication enabled in IIS on server X (following this recommendation on authPersistSingleRequest link ) and without, neither seems to work.
Any recommendations and help would be greatly appreciated!
This is the link to my repo. You would need to modify ocelot.test.json and \ClientApp\src\app\api-tester\api-tester.component.ts to set the correct gateway url
Project 1. is AngularWithOcelot
Project 2. is WindowsAuthenticationWithCorsHeadless

Hi @ArminNaish,
I'm wondering if this is truly the solution to the problem, in the light of this comment by Microsoft. Wouldn't that mean that credentials are set to the identity of the account under which Ocelot is running?
Regards, James

Hello everybody,
we have also struggeled with getting Windows Authentication (Kerberos) working with Ocelot. At first we received 401 unauthorized constantly. In order to fix that we changed the implementation of the HttpClientBuilder class. After we added UseDefaultCredentials=true to the HttpClientHandler, we no longer received 401 unauthorized errors:
   private HttpClientHandler UseNonCookiesHandler(DownstreamContext context)
        {
            return new HttpClientHandler
            {
                AllowAutoRedirect = context.DownstreamReRoute.HttpHandlerOptions.AllowAutoRedirect,
                UseCookies = context.DownstreamReRoute.HttpHandlerOptions.UseCookieContainer,
                UseProxy = context.DownstreamReRoute.HttpHandlerOptions.UseProxy,
                MaxConnectionsPerServer = context.DownstreamReRoute.HttpHandlerOptions.MaxConnectionsPerServer,
                UseDefaultCredentials = true // this line make the difference
            };
        }

I guess setting UseDefaultCredentials=true is required to pass the kerberos tokens to the downstream service. It was also necessary to call ChallengeAsync to generate the WWW-authenticate header required for the Windows Authentication flow (see: #1144)
By the way setting authPersistSingleRequest alone, didn't solve my problem.
It would be great if Windows Authentication could be integrated into Ocelot, it's still used by a lot of people out there.
Thanks in advance.
Keep up the great work!

Hi,
I'am using ocelot (16.0.1) in a web api core (3.1) application (ApiGateway to manage web api core apps).
Using ocelot, I request services of a few web api core applications.
Every request is sent with Bearer Token to every app that works with a bearer token.
Therefore, I have one web api core app (say X) that is based windows authentication (and not like the others apps that work with bearer token).
So I would like to know if there is a way to catch the bearer token, in the apiGateway app, extract from it all the claims and use them to change the header authorization (from bearer... to NTLM...) and send the request to the app (X) ?
Thanks a lot in advance.
Aby

Good afternoon, I have 2 site in IIS. 1- ocelot api gateway. 2-i microservices.
When authorizing directly through site 2, I easily go to it and log in.
When using ocelot, the authorization window constantly creeps out and is not authorized.
the above tips did not help

I created a PR for this: https://github.com/ThreeMammals/Ocelot/pull/1521

@Sacrilege commented on Sep 23, 2021

Thank you for PR creation! Included in Dec'23 release.

@jpj90 commented on Oct 13, 2020

Indeed, James, we will ensure to double-check this during the testing phase of the feature delivery. We anticipate that the delivery may be complex, yet the issue has been given priority in the milestone.


Fixed table widths, added keyboard graphics

Turned on the :experimental: feature Asciidoctor to allow key combinations to appear as graphics.
Fixed table widths causing whitespace issues with the table of contents. Was set to a width of 75%, which is now removed on all tables.

Awesome, merging!


Create a Tutorials page with a new layout

Describe the use case
We would like to provide a place for those who would like to learn, hands-on. about a specific skill to land in order to use in a workshop, or for those who are not following the Launchpad curriculum
Describe the problem
Currently, it's difficult to plug and play some of the great hands-on tutorial content we have, and would like a landing page to access 'tutorial courses' where they can
Proposed Solution
Implement a new navigation and menu, building a 'tutorials' page where we can have individual tutorial sets
Acceptance Criteria
There is a new layout & navigation for the entire page, with a separate page for the tutorials, as shown in the images

Hi, @walkerlj0  is there already made content for the tutorial page you can point me to?

Completed!


Add documentation button is not visible for API creators

Steps to reproduce

Login to API Publisher as a creator
Create an API
Navigate to the Documents tab
Add a new doc and navigate to doc list page

Behavior
The Add documentation button is not visible for creators, hence more docs cannot be added

This issue is not reproducible in the latest version of the APIM pack.


Add in support for CastleDB colors

CastleDB supports RGB encoded colors. Just need to implement a conversion function inside either the CastleDB utils or generated CastleDB class to turn that value into a properly generated UnityEngine.Color.

I think this is close but ColorUtility doesn't do the swap back from the int to hex, so you'd need to pre-convert it right? If you have an idea though for the implementation, definitely submit a PR!


react-native-is-edge-to-edge  on web

Before submitting a new issue

[X] I tested using the latest version of the library, as the bug might be already fixed.
[X] I tested using a supported version of react native.
[X] I checked for possible duplicate issues, with possible answers.

Bug summary
✘ [ERROR] No matching export in "node_modules/react-native-web/dist/index.js" for import "TurboModuleRegistry"
node_modules/react-native-is-edge-to-edge/dist/index.mjs:1:19:
  1 │ import { Platform, TurboModuleRegistry } from 'react-native';
    ╵                    ~~~~~~~~~~~~~~~~~~~

Library version
0.1.0
Environment info
react-native-web

Steps to reproduce
react-native-web TurboModuleRegistry
Reproducible sample code
use react-native-is-edge-to-edge in react-native-web

Fixed in 0.1.2

Fixed in 0.1.2

new error in 0.1.2
Failed to resolve entry for package "react-native-is-edge-to-edge". The package may have incorrect main/module/exports specified in its package.json. [plugin vite:dep-pre-bundle]

package.json use "module": "dist/index.mjs", but no index.mjs file

Oops, forgot to add back esm to build format. Fixed now.


This PR should fix #261

Change quokka scoped translations related function(_, _l, _n) to use the ones imported from flask.ext.babelex. That will use a default Domain. And flask-admin will still use its own CustomDomain.
Also added a zh_CN translation .po file.
By the way, in template files, you just use _gettext, and this function is introduced by flask-admin, which in turn will use flask-admin's Domain. Code from flask.ext.admin.base:
def render(self, template, **kwargs)
    .....
    .....
    kwargs['_gettext'] = babel.gettext
    kwargs['_ngettext'] = babel.ngettext
    kwargs['h'] = h

    # Contribute extra arguments
    kwargs.update(self._template_args)

    return render_template(template, **kwargs)

And I think it's OK to use gettext function, which is set by flask.ext.babelex when babel.init_app() is invoked. You may still want to use _gettext if you want to use the flask-adimn's Domain.
And to make admin.register(Post, PostAdmin, category=_l('Content'), name=_l("Post")) work, the object _l returned (a _LazyString), should implements its __hash__ method (flask-admin will use it as a key in dict). I created PR to mitsuhiko/speaklater, but haven't got respond, so I just change the requirement.txt to use my repo temporarily.
It's my pleasure to contribute to an open-source project! :laughing:

@mrjoes maybe that change on speaklater can benefit FlaskAdmin issue about translating categories?

ive found that has been fixed im Flask Admin https://github.com/flask-admin/flask-admin/issues/922

I will merge it!


Bugsnag

Fixes #938

@micheljung you undrstand what travis problem is?

Can we somehow "push" @axel1200 work forward? It would be nice to have such information, especially now.

No you can not because i was told we dont want this.... so I stoped working on it. Go punsh @Brutus5000  and @micheljung and tell them you want it

Atm i send errors silently and also I send the username of the user. In terms of privacy this seems problematic how should i do it @Geosearchef ???

Tell me how to do it


Update Dockerfile

Prefer the shorter, more ubiquitous equivalent of >/dev/null 2>&1 to  1> /dev/null 2> /dev/null .
Forgive my OCD twitch.

IMHO this is a pointless exercise as real development is happening on branch 0.4.0 where Dockerfile changed a lot.

agreed.

:+1:


Document app review process

Fixes https://github.com/probot/probot.github.io/issues/221 by adding a documented app review process. Ideally I'd love if we could use this new doc like an issue template, and check things off as we go through a review for an app.
Thoughts/feels welcome on everything here ✨
cc/ @probot/maintainers

Test failures are unrelated. I'm gonna go ahead and merge this so that we have a basis to start with. Happy to update more in the future.


Type-checking correctness for several additional packages.

Type-checking correctness for pixeltable.func, pixeltable.index, pixeltable.io, pixeltable.iterators, pixeltable.metadata, and pixeltable.util.
This leaves only exec, exprs, and tool unchecked.

This change is


nvcc error in Windows

Setup includes :

Cuda 10.1 and cudnn7.5
Pytorch 1.3.1

Hello,
This is likely an NVCC bug, which was supposed to be fixed in the latest version. Can you try loading the latest cuda compiler from https://developer.nvidia.com/cuda-10.1-download-archive-update2
This one worked fine for me with PyTorch 1.3.1.

Hi, thank you for using Kaolin! The library went through a major rewrite with the version 0.9.0, and this issue is no longer current. Please refer to our latest documentation:
https://kaolin.readthedocs.io/en/latest/


Default values don't get applied to objects inside of an array

Hi,
It seems that the default values don't get applied to objects inside of an array.
It looks like that only the top level object gets their default values applied.
The validation in the sub-object does work (like type checking and required fields).
I'm running version 5.2.0
Example:
I expect the 'test' property to be filled with the default value if not present.
$data = [
    (object)[
        "key" => 1,
        "test" => 1
    ],(object)[
        "key" => 1,
    ]
];
$validator = new \JsonSchema\Validator();

var_dump($validator->validate(
    $data,
    json_decode('
    {
        "type": "array",
        "items": [
            {
                "type": "object",
                "properties": {
                    "key": {
                        "type": "integer"
                    },
                    "test": {
                        "type": "integer",
                        "default": 0
                    }
                },
                "required": [
                    "key"
                ]
            }
        ]
    }
    '), \JsonSchema\Constraints\Constraint::CHECK_MODE_APPLY_DEFAULTS
));
var_dump($data);

Gives:
int(0)

array(2) {
  [0]=>
  object(stdClass)#3 (2) {
    ["key"]=>
    int(1)
    ["test"]=>
    int(1)
  }
  [1]=>
  object(stdClass)#2 (1) {
    ["key"]=>
    int(1)
  }
}

@pcjacobse Thanks for reporting this.
I've found the root cause - the logic that prevents recursive application of defaults is being incorrectly triggered for arrays, with the result that the library thinks it has already applied the default value for "test" and skips it.
Currently thinking of the best way to address this.

@mathroc Do you have an opinion?
The problem is the call to $path->setFromDefault() in the array section of UndefinedConstraint::applyDefaultValues().  It needs to be called only for each element that has been set from a default, and not for the entire array the way it is now. However... we don't, at that point, actually have a path object for the array members - $path refers to the array itself.
Obviously this needs fixing, but it may require changing the recusion logic for defaults.

@erayd sorry I missed this, I’ll take a look at this in the next few days

@erayd this seems a bit tricky 😕 one think though, shouldn't $path->setFromDefault(); be called at the end of the previous if (one line up) ?
it will still cause problems when default items are defined inside arrays but that's not the case in @pcjacobse exemple

@mathroc @erayd @pcjacobse
I'm not sure if you guys have made any progress on this, but I made the following change and it appears to be working for me.
UndefinedConstraint.php:Line273
Add the following after the $path->setFromDefault();
$this->applyDefaultValues($value[$currentItem], $schema->items[$currentItem], $path);
Which is just recursively passing the array items through the same function. If this ends up blowing up on me I'll repost.

@krismckay Line 273 of UndefinedConstraint.php (branch 6.0.0-dev) is } else {. Could you please clarify exactly where you're proposing that line be inserted? If you could include some of the surrounding code in your post for context, that would be helpful. I'm assuming that you probably meant to insert after L277.
Overall, I like the fundamental approach of your idea. The implementation as you've posted it isn't quite suitable (the path passed for recursion needs to be correct) - however, recursing within this block is a good idea, and seems like an ideal solution to the problem.
If you are intending to submit a PR for this, please base it on 6.0.0-dev, and address the issues above. If not, let me know, and I'll do a PR when I have the time.

@erayd Sorry, I should have mentioned the version.. I'm on version 5.2 rather then the dev build. Here's the patch I'm applying in my build process:
`--- ./vendor/justinrainbow/json-schema/src/JsonSchema/Constraints/UndefinedConstraint.php.orig  2017-08-29 17:12:10.000000000 -0500
+++ ./vendor/justinrainbow/json-schema/src/JsonSchema/Constraints/UndefinedConstraint.php       2017-08-29 17:12:18.000000000 -0500
@@ -272,4 +272,5 @@
}
$path->setFromDefault();

           $this->applyDefaultValues($value[$currentItem], $schema->items[$currentItem], $path);
       }
   } elseif (

`

Hmm, that last "fix" only corrects the first item in an array. I have a new patch now that applies it to each item in the array. However, I've also found that the "CHECK_MODE_COERCE_TYPES" property is also affected by the same bug. It doesn't affect all items of an array. Hoping to get that working as well.
Here's the current patch to hit defaults for all array elements:
--- vendor/justinrainbow/json-schema/src/JsonSchema/Constraints/UndefinedConstraint.php.orig    2017-08-29 17:29:54.000000000 -0500
+++ vendor/justinrainbow/json-schema/src/JsonSchema/Constraints/UndefinedConstraint.php 2017-08-30 10:42:00.000000000 -0500
@@ -261,16 +261,21 @@
             // $value is an array, and items are defined - treat as plain array
             foreach ($items as $currentItem => $itemDefinition) {
-                if (
-                    !array_key_exists($currentItem, $value)
-                    && property_exists($itemDefinition, 'default')
-                    && $this->shouldApplyDefaultValue($requiredOnly, $itemDefinition)) {
-                    if (is_object($itemDefinition->default)) {
-                        $value[$currentItem] = clone $itemDefinition->default;
-                    } else {
-                        $value[$currentItem] = $itemDefinition->default;
+                foreach($value as &$val){
+                    if (
+                        !array_key_exists($currentItem, $val)
+                        && property_exists($itemDefinition, 'default')
+                        && $this->shouldApplyDefaultValue($requiredOnly, $itemDefinition))
+                    {
+                        if (is_object($itemDefinition->default))
+                        {
+                            $val[$currentItem] = clone $itemDefinition->default;
+                        } else
+                        {
+                            $val[$currentItem] = $itemDefinition->default;
+                        }
                     }
+                    $path->setFromDefault();
+                    $this->applyDefaultValues($val, $schema->items[$currentItem], $path);
                 }
-                $path->setFromDefault();
-                $this->applyDefaultValues($value[$currentItem], $schema->items[$currentItem], $path);
             }
         } elseif (

Closing as reported to be a false alarm.


OpenApi 3.x validation error when "servers" object is present

When adding the servers section with a basic object with only the url mandatory property the validation fails stating that the url property is expected.
This can be reproduced with the OpenApi 3.x Jamal samples

Sample file to reproduce, if the "servers" section is commented validation passes
`openapi: 3.0.0
info:
title: Sample API
description: Optional multiline or single-line description in CommonMark or HTML.
version: 0.1.9
servers:

url: http://api.example.com/v1
description: Optional server description, e.g. Main (production) server

paths:
/users:
get:
summary: Returns a list of users.
description: Optional extended description in CommonMark or HTML.
responses:
'200':    # status code
description: A JSON array of user names
content:
application/json:
schema:
type: array
items:
type: string`

Hmm, I can't see any errors with your example file either with the intellisense or the OpenAPI Validate  command. If you use the OpenAPI Lint command it complains about the servers.url property containing example.com, but that is expected.
Please can you let me know the filename you are using and the version of the extension you have installed?

filename: vscode_lint_sample.yaml
extension version: openapi-lint 1.0.2
Here's a sample capture of failure of "OpenApi Validate" with the above example

Sorry, I still can't reproduce this, so I can only assume there's an interaction with another vscode extension.

Tried disabling other extensions related to open api and swagger, I get the same result:

Here's my full list of extensions:
code --list-extensions

adashen.vscode-tomcat
AlanWalk.markdown-toc
Arjun.swagger-viewer
bat67.markdown-extension-pack
bierner.markdown-emoji
bierner.markdown-preview-github-styles
bungcip.better-toml
cstuder.gitlab-ci-validator
darkriszty.markdown-table-prettify
DavidAnson.vscode-markdownlint
eriklynd.json-tools
fatihacet.gitlab-workflow
faustinoaq.javac-linter
foxundermoon.shell-format
fwcd.kotlin
goessner.mdmath
ipedrazas.kubernetes-snippets
mathiasfrohlich.Kotlin
mauve.terraform
mermade.openapi-lint
momoto.binary-viewer
ms-azuretools.vscode-docker
ms-kubernetes-tools.vscode-kubernetes-tools
ms-python.python
ms-vscode-remote.remote-wsl
ms-vscode.cpptools
ms-vscode.Go
ms-vscode.powershell
ms-vsliveshare.vsliveshare
philosowaffle.openapi-designer
Pivotal.vscode-spring-boot
redhat.java
redhat.vscode-yaml
shd101wyy.markdown-preview-enhanced
shengchen.vscode-checkstyle
sidneys1.gitconfig
slevesque.vscode-hexdump
VisualStudioExptTeam.vscodeintellicode
vscjava.vscode-java-debug
vscjava.vscode-java-dependency
vscjava.vscode-java-pack
vscjava.vscode-java-test
vscjava.vscode-maven
vscjava.vscode-spring-initializr
yzane.markdown-pdf
yzhang.markdown-all-in-one
zoellner.openapi-preview
zxh404.vscode-proto3

@dariusj18 you reacted with a :+1: to the original issue, are you also running any of @hecsalazar 's extensions?

Here's my extension list
adisreyaj.swagger-snippets
alefragnani.Bookmarks
alefragnani.project-manager
Arjun.swagger-viewer
bbenoist.vagrant
bmarkovic.haproxy
bmewburn.vscode-intelephense-client
christian-kohler.npm-intellisense
CoenraadS.bracket-pair-colorizer
crasnam.elasticdeveloper
cssho.vscode-svgviewer
dakara.dakara-foldplus
donjayamanne.githistory
DotJoshJohnson.xml
dssiqueira.drupal-8-snippets
eamodio.gitlens
EditorConfig.EditorConfig
eg2.vscode-npm-script
esbenp.prettier-vscode
felixfbecker.php-debug
fknop.vscode-npm
formulahendry.auto-close-tag
formulahendry.auto-rename-tag
hdg.live-html-previewer
ikappas.composer
ikappas.phpcs
jakelucas.code-select-until
jmlv.MeteorSnippets
johnpapa.vscode-peacock
KamasamaK.vscode-cfml
karigari.chat
lindraupe.aw-watcher-vscode
marcostazi.VS-code-drupal
marcostazi.VS-code-vagrantfile
mechatroner.rainbow-csv
mermade.openapi-lint
mikestead.dotenv
mkloubert.vscode-deploy-reloaded
mrmlnc.vscode-apache
mrmlnc.vscode-autoprefixer
ms-azuretools.vscode-docker
ms-kubernetes-tools.vscode-kubernetes-tools
ms-mssql.mssql
ms-python.python
ms-vscode-remote.remote-wsl
ms-vscode.powershell
ms-vsliveshare.vsliveshare
ms-vsliveshare.vsliveshare-audio
ms-vsliveshare.vsliveshare-pack
msjsdiag.debugger-for-chrome
mtxr.sqltools
neilbrayfield.php-docblocker
oderwat.indent-rainbow
onecentlin.laravel-blade
philosowaffle.openapi-designer
ramonitor.meteorhelper
RandomChance.logstash
redhat.java
redhat.vscode-yaml
ria.elastic
ritwickdey.LiveServer
ryannaddy.laravel-artisan
santigarcor.phpunit-extended
slevesque.vscode-autohotkey
spywhere.guides
streetsidesoftware.code-spell-checker
thomas-baumgaertner.vcl
tomoki1207.vscode-input-sequence
Tyriar.lorem-ipsum
vscode-icons-team.vscode-icons
vuhrmeister.vscode-meteor
whatwedo.twig```

Common extensions are:
Arjun.swagger-viewer
ms-azuretools.vscode-docker
ms-kubernetes-tools.vscode-kubernetes-tools
ms-python.python
ms-vscode.powershell
ms-vscode-remote.remote-wsl
ms-vsliveshare.vsliveshare
philosowaffle.openapi-designer
redhat.java

Ignoring redhat.vscode-yaml and mermade.openapi-lint.

Something (presumably an extension) definitely appears to be polluting the Array prototype.

Seems to be ms-kubernetes-tools.vscode-kubernetes-tools

Seems to be ms-kubernetes-tools.vscode-kubernetes-tools

I confirm, when disabling that extension It the validation works, I reenabled and the error appears.
@MikeRalphson I suggest to debug to confirm if it's an issue in Microsoft extension and report to them or mermade itself to move forward on a fix

Please refer / subscribe to https://github.com/Azure/vscode-kubernetes-tools/issues/613

Added some forced pollution cleanup code in<EMAIL_ADDRESS>It would be great to get confirmation that this resolves the issue.

I confirmed it works with microsoft kubernetes extension enabled.
Important tip: you need to update and restart vscode, in my first try I
just updated just the extension and didn't work.
On Mon, Sep 9, 2019 at 1:44 PM Mike Ralphson<EMAIL_ADDRESS>wrote:

Added some forced pollution cleanup code in<EMAIL_ADDRESS>It would be great to get confirmation that this resolves the issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Mermade/openapi-lint-vscode/issues/9?email_source=notifications&email_token=ACSBHDL6AU2UCAR3ZBZSHELQI2DRDA5CNFSM4HBDNCW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6IORXQ#issuecomment-529590494,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACSBHDKMLNGAIVW6O6EMSJDQI2DRDANCNFSM4HBDNCWQ
.

Thanks for everyone's help in tracking down the cause of this behaviour.


Installing additional symlinks to rhash

Would it be possible to extend the SYMLINKS variable in the Makefile in order to allow install-symlinks to create a symlink for each hash method available?

Yes.
From the man-page:

The default output format can also be changed by renaming the program or placing a hardlink/symlink to it with a name containing strings ‘crc32’, ‘md4’, ‘md5’, ‘sha1’, ‘sha256’ ‘sha512’, ‘tiger’, ‘tth’, ‘btih’, ‘aich’, ‘ed2k’, ‘ed2k-link’, ‘gost’, ‘gost-cryptopro’, ‘ripemd160’, ‘has160’, ‘whirlpool’, ‘edonr256’, ‘edonr512’, ‘snefru128’, ‘snefru256’, ‘sfv’ or ‘magnet’.

Thanks, but I was hoping you could add that to the Makefile in this repository...

Many Linux distributions already have commands:
md5sum, sha1sum, sha224sum, sha256sum, sha384sum, sha512sum, crc32
I don't want to duplicate them and disturb bash completion.
Most people also don't want old hash sums like aich and snefru. Btih is hardly a hash sum, because it depends on  not strictly specified block-size parameter.
Edonr is very fast and good hash (it's a sha3 candidate), but it is not well known and is rarely used. I think, I need some more reasons to add it.
As a packager you can pass SYMLINKS variable to the make command (or patch Makefile after unpacking tgz), and you'll get all symlinks you want.

Included into RHash v1.3.6.


Add support for single-header distribution

Add generation of single-header distribution that will be easier to integrate into projects.

Added single-header generation and upload to release in latest version of master.


Add function/s to convert BIG_XXX to BIG_YYY

Goal
The goal is to facilitate converting BIG_XXX to BIG_YYY by introducing new functions to ff.c.in
Acceptance criteria

Well documented code
Tests pass with more than 80% test coverage
Static analysis should pass
Code should be reviewed by two engineers

I rather prefer a macro or  set of well defined macro. It's ok, provide the code and we'll review it with @kealan


fix(learn): missing semi-colon on use arrow functions to write anonymous functions

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

Noticed missing semi-colon on – ES6: Use Arrow Functions to Write Concise Anonymous Functions.

Congrats @shrobbins on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


LNK1181 error building bundled libzmq on MSVC 14.29 due to incorrect LIBPATH

pip install --no-binary pyzmq pyzmq fails with a LNK1181 error when attempting to build the libzmq library with environmental variable ZMQ_PREFIX=bundled on Windows 10 21H2 using Python 3.10.4 with pip 22.0.4, setuptools 62.1.0 and wheel 0.37.1, building using MSVC 14.29.30133.
The error was originally observed when attempting  pip install pyzmq==22.0.1 --install-option="--zmq=bundled" on python 3.9, but  was then replicated with the latest PyZMQ (v22.3.0) using Python 3.10 and the latest pip, setuptools and wheel as directed on https://github.com/zeromq/pyzmq/wiki/Building-and-Installing-PyZMQ
The wheel fails to build with the following error: LINK : fatal error LNK1181: cannot open input file 'libzmq.cp310-win_amd64.lib'
This appears to be caused by an error in the library path provided to the MSVC linker: /LIBPATH:build\temp.win-amd64-3.10\Release\buildutils is provided to the linker, but the relevant .lib file is stored in build\temp.win-amd64-cpython-310\Release\buildutils
Console output:
    Creating library build\temp.win-amd64-cpython-310\Release\buildutils\libzmq.cp310-win_amd64.lib and object build\temp.win-amd64-cpython-310\Release\buildutils\libzmq.cp310-win_amd64.exp
      Generating code
      Finished generating code
      building 'zmq.backend.cython._device' extension
      creating build\temp.win-amd64-cpython-310\Release\zmq
      creating build\temp.win-amd64-cpython-310\Release\zmq\backend
      creating build\temp.win-amd64-cpython-310\Release\zmq\backend\cython
      "C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Tools\MSVC\14.29.30133\bin\HostX86\x64\cl.exe" /c /nologo /O2 /W3 /GL /DNDEBUG /MD -DWINVER=0x0602 -D_WIN32_WINNT=0x0602 -Ibundled\zeromq\include -Izmq\utils -IC:\Python310\include -IC:\Python310\Include "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Tools\MSVC\14.29.30133\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Tools\MSVC\14.29.30133\include" "-IC:\Program Files (x86)\Windows Kits\NETFXSDK\4.8\include\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\winrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\cppwinrt" /Tczmq\backend\cython\_device.c /Fobuild\temp.win-amd64-cpython-310\Release\zmq\backend\cython\_device.obj
      _device.c
      "C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Tools\MSVC\14.29.30133\bin\HostX86\x64\link.exe" /nologo /INCREMENTAL:NO /LTCG /DLL /MANIFEST:EMBED,ID=2 /MANIFESTUAC:NO /LIBPATH:build\temp.win-amd64-3.10\Release\buildutils /LIBPATH:C:\Python310\libs /LIBPATH:C:\Python310 /LIBPATH:C:\Python310\PCbuild\amd64 "/LIBPATH:C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Tools\MSVC\14.29.30133\ATLMFC\lib\x64" "/LIBPATH:C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Tools\MSVC\14.29.30133\lib\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\NETFXSDK\4.8\lib\um\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\10\lib\10.0.19041.0\ucrt\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\10\lib\10.0.19041.0\um\x64" libzmq.cp310-win_amd64.lib ws2_32.lib iphlpapi.lib advapi32.lib /EXPORT:PyInit__device build\temp.win-amd64-cpython-310\Release\zmq\backend\cython\_device.obj /OUT:build\lib.win-amd64-cpython-310\zmq\backend\cython\_device.cp310-win_amd64.pyd /IMPLIB:build\temp.win-amd64-cpython-310\Release\zmq\backend\cython\_device.cp310-win_amd64.lib
      LINK : fatal error LNK1181: cannot open input file 'libzmq.cp310-win_amd64.lib'
      error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Professional\\VC\\Tools\\MSVC\\14.29.30133\\bin\\HostX86\\x64\\link.exe' failed with exit code 1181

I can't see why the build directory name is being corrupted before being passed into the MSVC linker. Is this an issue with the wheel/setup of pyzmq or could this be related to the version of MSVC/Windows Kit used?
Full console log also attached:
Pyzmq_Python310_LNK1181_Error.txt

I think this may have been fixed by bec362447c0bd5002cf813894a2fec3e8ee8ce5d.
Can you test with pip install --pre pyzmq to see if the pyzmq-23 beta works for you?

Yeah, I think you're right that it was fixed in bec3624.
The pyzmq-23 beta works correctly. Looks like newer versions of setuptools have a different format of build_temp and it didn't match the string assembly used prior to the bec3624 patch.

okay, thanks. Closing, then. Planning to push out 23 pretty soon (hoping for Cython 0.29.29 first, to get Python 3.11-compatible sources)