ECF_INCLUDE with ecFlow variables ECFLOW-1920

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

Comparison is base (7a96afc) 80.15% compared to head (3f3893c) 80.17%.
Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #46      +/-   ##
===========================================
+ Coverage    80.15%   80.17%   +0.01%
===========================================
  Files          575      575
  Lines        57874    57915      +41
===========================================
+ Hits         46391    46431      +40
- Misses       11483    11484       +1

Files
Coverage Δ

ANode/src/EcfFile.hpp
100.00% <ø> (ø)

ANode/src/Node.cpp
87.81% <100.00%> (ø)

ANode/src/Node.hpp
90.69% <ø> (ø)

ANode/test/TestEcfFile.cpp
88.25% <100.00%> (+0.60%)
:arrow_up:

ANode/src/EcfFile.cpp
75.82% <93.33%> (+0.08%)
:arrow_up:

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


Can't access component's getHandlerId() to test with Enzyme

Clone repo at https://github.com/agonzalezjr/react-dnd-enzyme.
(It is basically just a Create-React-App + Typescript -> added React-DnD, dustbin/boxes example from ReactDnD -> added Enzyme. It runs OK, and the tests run OK too.)
Debug the src/Box.test.tsx file, and there is no way in the component returned from root.find(Box), to get the handler ID needed to simulate drag and drop in testing.
This is the test code:
  it('drags - with enzyme', () => {
    const BoxWithTestContext = wrapInTestContext(Box)
    const ref = React.createRef()

    const root = mount(
      <>
        <BoxWithTestContext ref={ref} name="test" />
      </>
    )
    expect(root).toBeDefined()

    const backend = (ref.current as any).getManager().getBackend()
    expect(backend).toBeDefined()

    // How do I get the component or component's getHandlerId() to be able to call
    // backend.simulateBeginDrag(...), etc...
  });

I can see the backend is setup, and there is something registered with an id of "S1", but I need to be able to get that programmatically somehow.
Versions in use:
    "react": "^16.12.0",

    "react-dnd": "^11.1.3",
    "react-dnd-html5-backend": "^11.1.3",
    "react-dnd-test-backend": "^11.1.3",
    "react-dnd-test-utils": "^11.1.3"

    "enzyme": "^3.11.0",
    "enzyme-adapter-react-16": "^1.15.5",

This is probably a duplicate of #2774, so will be watching that one instead.

This is probably a duplicate of #2774, so will be watching that one instead.


Move candidate parsing to pion/ice

Instead of having logic in pion/webrtc and pion/sdp move all logic to
pion/ice. Users of pion/ice don't want to import these packages to use
basic pion/ice functionality.

@korymiller1489 pion/ice is ready for you to use!


AI reaction to aiming

The default auto aiming of Gothic causes NPCs to react when the player aims at them. This behavior was inconsitent/lost throughout the early development of free aiming. Check if the behavior is still present, otherwise fix it.

This issue did not in fact occur. The AI reactions work as desired.


[mimalloc] Update to 2.0.3

Update the mimalloc port from 2.0.2 to 2.0.3.
Changelog :

Improved WASM support, improved macOS support and performance (including M1), improved performance for v2 for large objects, Python integration improvements, more standard installation directories, various small fixes.

All features are tested successfully in the following triplet:

x86-windows
x64-windows
x64-windows-static

Thanks for the update!


Scrabsha/expand macros

This is an internal PR so that i can pre-review my work. You're not supposed to review it :)

Merged upstream


Screen disappears after screen orientation rotation and subsequently crashes

Version
Tell us which versions you are using:

react-native-router-flux v^3.37.0
react-native v0.40.0

Expected behaviour
App should not crash.
Actual behaviour
App crash.
Steps to reproduce

From main screen (screen 1), tap a button to go to next screen (screen 2).
On screen 2, I enable my phone screen rotation.
I rotate my screen (from portrait to landscape).
Screen 2 suddenly disappears (the moment orientation changed) and return back to screen 1.
Tap again on that button to attempt to go to screen 2.
Crash!

Appreciate kind attention on this.

Can you reproduce this in the Example app provided or just in your own app? May be something more going on here.

Hi, I tried the Example.js but couldn't reproduce. However, my use case is a little different. Assuming I am in "Home" (1 of the tabs in tabbed scene), and I click on a link that will navigate me away to "Demo" (tab_1, refer codes below) scene (single page scene). I am able to navigate to that scene, but the moment i rotate my phone orientation, "Demo" scene disappears and I return back to "Home". However, when I click on the link again in attempt to return to "Demo", it crashes. Please help.
This is my router:
<Router createReducer={reducerCreate}>
            <Scene key="modal" component={Modal} >
                <Scene key="root" hideNavBar hideTabBar >
                     <Scene
                                    key="tab1_1"
                                    component={Demo}
                                    title="Tab #1_1"
                                    onRight={() => alert('Right button')}
                                    rightTitle="Right"
                                />
                    <Scene key="launch" component={Launch} title="Launch" initial />
                    <Scene key="tabbar" component={NavigationDrawer}>
                         <Scene
                                key="main"
                                tabs
                                tabBarStyle={styles.tabBarStyle}
                                tabBarSelectedItemStyle={styles.tabBarSelectedItemStyle}
                            >
                              <Scene key="home" component={(props) => {return (<Home {...props} pubSub={PubSub} storage={Storage} />)}} title="Home" initial icon={HomeIcon} navigationBarStyle={Styles.navBarStyle} titleStyle={{color: 'white'}}
                                renderRightButton={() => {
                                return <AvatarIcon src={'https://upload.wikimedia.org/wikipedia/en/1/17/Batman-BenAffleck.jpg'} size={40} />
                                }} drawerImage={{uri: 'https://facebook.github.io/react/img/logo_small.png', width: 38, height: 38}}>

                              </Scene>
                        </Scene>
                    </Scene>
                </Scene>
            </Scene>
        </Router>

This is my Home.js:
 <View style={[Styles.containerBackgroundColor, {flex: 1}]}>
            <ScrollableTabView
                style={{marginTop: 50}}
                initialPage={1}
                renderTabBar={() => <ScrollableTabBar />}
                >
                <View tabLabel="test">
                    <TouchableHighlight onPress={() => {
                        Actions.tab1_1();
                        alert('demo');
                        }}>
                        <Text>123</Text>
                    </TouchableHighlight>
                </View>
                <Text tabLabel="All"></Text>
                <Text tabLabel="Eateries"></Text>
                <Text tabLabel="Accommodations"></Text>
                <Text tabLabel="Tours"></Text>
                <Text tabLabel="Misc"></Text>
            </ScrollableTabView>
        </View>

Same issue

facing exact same issue with same steps as @oatcrunch

Same issue.
I wrote a Drawer the sample of document.
           <Drawer                 ref="navigation"                 open={state.open}                 onOpen={()=>Actions.refresh({key:state.key, open: true})}                 onClose={()=>Actions.refresh({key:state.key, open: false})}                 type="displace"                 content={<NavigationMenu />}                 tapToClose={true}                 openDrawerOffset={(viewport) => viewport.width - 271}                 panCloseMask={0.2}                 panOpenMask={0.2}                 negotiatePan={true}                 tweenHandler={(ratio) => ({                  main: { opacity:Math.max(0.54,1-ratio) }                 })}                 tweenEasing={'easeOutCirc'}                 tweenDuration={200}             >                 <DefaultRenderer navigationState={children[0]} onNavigate={this.props.onNavigate} />             </Drawer>
Drawer's onOpen and onClose properties were the cause of this problem.


Dont support on the fly display for different spatial reference system layer

I have basemap and its spatial reference is written below. I'm adding this basemap TiledMapServiceLayer
(BaseMap) -
Spatial Reference: PROJCS["ITRF_96_UTM_Zone_35N",GEOGCS["GCS_GRS_1980",DATUM["D_ITRF_1996",SPHEROID["GRS_1980",6378137.0,298.257222101]],PRIMEM["Greenwich",0.0],UNIT["Degree",0.0174532925199433]],PROJECTION["Transverse_Mercator"],PARAMETER["False_Easting",500000.0],PARAMETER["False_Northing",0.0],PARAMETER["Central_Meridian",30.0],PARAMETER["Scale_Factor",1.0],PARAMETER["Latitude_Of_Origin",0.0],UNIT["Meter",1.0]]
And I have DynamicLayer. I am adding this ArcGISDynamicMapServiceLayer and Spatial reference is;
(DynamicLayer)
Spatial Reference:
GEOGCS["Geographic Coordinate System",DATUM["D_WGS84",SPHEROID["WGS84",6378137.0,298.257223560493]],PRIMEM["Greenwich",0.0],UNIT["Degree",0.0174532925199433]]
Only the layers having the same spatial reference of basemap are visible on the map. When basemap layer and dynamic layer have different spatial reference, dynamic layers not visible on the map. At the iOS SDK side there is not like this problem. Or anything else i'm forgetting to add?  Or it is bug ?

is this a problem in a particular sample? If it is, please mention the sample you are having problems with, if not and this a general SDK/API question please raise this on the geonet forum here:
https://geonet.esri.com/community/developers/native-app-developers/arcgis-runtime-sdk-for-android

@salihyalcin Sounds like this is not a sample specific issues, as @willcrick mentioned, you should post this in to forum and share some specific code from your app as well.

Closing and marked as GeoNet question


Account creation fails behind proxy server

Azure Data Studio Version: Latest stable 1.48.1
OS Version: All Windows versions

Steps to Reproduce:

Install Azure Data Studio (ADS) on a computer which accesses the internet through an explicit proxy server. For simplicitys sake we use a proxy without any authentication or TLS interception.
Set the enrivonment variables HTTP_PROXY=http://<proxyserver&gt>:<proxyport&gt> und HTTPS_PROXY=http://<proxyserver&gt>:<proxyport&gt> in the Windows system properties
Open ADS
Start Wireshark or another packet capture program and let it capture all connections to the proxy server
Click "Add an account" to create an Azure account in ADS
The local service providers website on http://<IP_ADDRESS> will be opened in your webbrowser, which redirects you to your IDP, which redirects you back to http://<IP_ADDRESS>. Then a GET request will be made to https://management.azure.com/tenants?api-version=2019-11-01 and the proxy server will respond to it with an error (Bad request, Bad wateway).

Cause:
When looking at the connections in Wireshark you can see the following request to the proxy server:
GET https://management.azure.com/tenants?api-version=2019-11-01 HTTP/1.1
This is a violation of the HTTP specification in RFC 7230: To fetch an object via HTTPS through an HTTP proxy, the CONNECT method must be used. The correct syntax is:
CONNECT management.azure.com:443 HTTP/1.1
<Establish a TLS connection...>
<Then the HTTP request is sent within the encrypted tunnel...>

There are countless bug reports in the issue tracker of Axios (eg https://github.com/axios/axios/issues/4531) about this behavior.
Possible workarounds:
Delete the https_proxy and HTTPS_PROXY environment variables and keep only http_proxy OR HTTP_PROXY. Then Axios (at least in the currently used version 1.6.2) will create correct HTTPS proxy requests.
If you need the environment variables for other applications under the same user account, you can dynamically remove them before starting ADS in a batch file:
set https_proxy=
"C:\Program Files\Azure Data Studio\azuredatastudio.exe"

Possible solutions:

Do not use Axios.
Pass another proxy agent into Axios instead of using the built-in agent.
Dynamically unset https_proxy and HTTPS_PROXY environment variables before calling Axios.

This is still a relevant issue


Wrong ')' in RHS of eq 16

The ')' in the bottom of the RHS of eq. 16 should be after dphi/dn

[ ] Wrong ')' in RHS of eq 16
The ')' in the bottom of the RHS of eq. 16 should be after dphi/dn

[ ] In the Figures that have (a) and (b) replace (top) and (bottom) to make reference to (a) and (b)

Check list

[x] 1 Please specify base of logarithm or use natural log (ln).
[x] 2 Please specify base of logarithm or use natural log (ln).
[ ] 3 Please note that there must be a citation of Figure 12 somewhere after the citation of Fig. 11 (your only citations of Fig. 12
occur earlier in the paper). Please check this inserted citation and relocate if necessary (but make sure it’s after the Fig. 11
citation).
[ ] 4 Please provide page number.
[ ] 5 Please check page number.
[ ] 6 Please provide city of publication for Refs. [28,30].
[ ] 7 Please check insertion of name and location of publisher.
[x] 8 Please provide working URL.
[ ] 9 Please verify update volume number.
[ ] FQ: This funding provider could not be uniquely identified during our search of the FundRef registry (or no Contract or Grant
number was detected). Please check information and amend if incomplete or incorrect.
[ ] Q: This reference could not be uniquely identified due to incomplete information or improper format. Please check all
information and amend if applicable.

Talk to Lorena about:

[ ] Comment 3
How PyGBe is written

For items 1 and 2, add in the response to the galley proof:

The base of logarithm used is technically irrelevant, because the big-O in front indicates that we are talking about the scaling of the computational work with problem size N. The scaling is simply a slope in a log-log plot, with any base. Specifying the base of the logarithm in the text would make an incorrect narrative.

Reference DOIs missing:
[10] https://doi.org/10.1103/PhysRevB.72.155412
[35] https://doi.org/10.6084/m9.figshare.154331.v3
[41] https://doi.org/10.6084/m9.figshare.7504271.v1
[42] https://doi.org/10.6084/m9.figshare.7504307.v1
[44] https://doi.org/10.6084/m9.figshare.7504484.v1
[45] http://doi.org/10.5281/zenodo.2511600
[46] http://doi.org/10.5281/zenodo.2513715
[47] https://doi.org/10.6084/m9.figshare.7506167.v1
Note: the problem with missing links to references originates from pasting the .bbl file into the .tex after processing with the RevTeX style (which strips all DOIs from references!!).
Reference [26] -- Update URL to: https://github.com/pygbe/pygbe
Reference [36] -- Update URL to: http://pygbe.github.io/pygbe/docs/


Crash with ViaFabricPlus

Description
Minecraft crashes when opening multiplayer screen.
To reproduce

Install Numeral Ping
Install ViaFabricPlus
Install necessary libs
Launch Minecraft
Open multiplayer screen
See the error

Info

Fabric Loader 0.14.21
Fabric API 0.83.0/0.83.1
Minecraft 1.20/1.20.1
YACL 3.0.1
Debug mods (StackDeobf, Better log4j)

Logs
latest.log
crash-2023-06-16_04.21.26-client.txt
Related
ViaFabricPlus#134

Thank you for reporting this! I found a compromise that will probably work. Hopefully I'll get around to releasing this! :P

Turns out I initially made a mistake, but it seems to work now!
There's just a bit of duplicate information, but if you're not using ViaFabric it'll hide the tooltip entirely.


"identityIds" property is depreciated. Examples are no longer valid

User assigned identity example are not longer valid. Attempting to use those will end up with error below
New-AzResourceGroupDeployment: 2:43:21 PM - The deployment 'sql' failed with error(s). Showing 1 out of 1 error(s).
Status Message: ResourceIdentity property identityIds is deprecated. The userAssignedIdentities property should be used instead. (Code:InvalidParameter)
CorrelationId: 5a4595db-9f66-4c01-978f-755945bf7ac7

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

ID: 6dada714-f867-2a46-6b94-eaea6e1da966
Version Independent ID: 25dc8f50-52c6-3c2b-ade6-c5d8258b74ec
Content: Configure managed identities on Azure VM using template - Azure AD
Content Source: articles/active-directory/managed-identities-azure-resources/qs-configure-template-windows-vm.md
Service: active-directory
Sub-service: msi
GitHub Login: @barclayn
Microsoft Alias: barclayn

@artisticcheese,  Thanks for leveraging Azure docs feedback channel to raise this question. Since you have closed the thread I presume the issue is resolved, if it's not the case or you need further assistance kindly just let us know.
As a side note, this channel is reserved for ‘Azure docs’ feedback- for sharing doc feedback and suggesting content improvements.

@artisticcheese,  Thanks for leveraging Azure docs feedback channel to raise this question. Since you have closed the thread I presume the issue is resolved, if it's not the case or you need further assistance kindly just let us know.
As a side note, this channel is reserved for ‘Azure docs’ feedback- for sharing doc feedback and suggesting content improvements.


Update index.md

Added release notes for today's Heimdal release

@Lilith-mx Hi, when to publish this? Also, please sign the CLA (just write "Mendix").

Hi Adam,
We have just released today and immediately realized that we forgot the release notes (we didn’t need any extra documentation for this release happily)
so the sooner the better for us.
Sorry for late communication, we will do our best to not get there again.
I just signed the CLA as well
Lilith,
Sent from Mail for Windows 10
From: Adam Dupaski
Sent: Tuesday, August 7, 2018 8:25 AM
To: mendix/docs
Cc: Lilith; Mention
Subject: Re: [mendix/docs] Update index.md (#1306)
@Lilith-mx Hi, when to publish this? Also, please sign the CLA (just write "Mendix").
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Publishing now, thanks.
From: Lilith<EMAIL_ADDRESS>Sent: August 7, 2018 8:40 AM
To: mendix/docs<EMAIL_ADDRESS>Cc: Adam Dupaski<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [mendix/docs] Update index.md (#1306)
Hi Adam,
We have just released today and immediately realized that we forgot the release notes (we didn’t need any extra documentation for this release happily)
so the sooner the better for us.
Sorry for late communication, we will do our best to not get there again.
I just signed the CLA as well
Lilith,
Sent from Mail for Windows 10
From: Adam Dupaski
Sent: Tuesday, August 7, 2018 8:25 AM
To: mendix/docs
Cc: Lilith; Mention
Subject: Re: [mendix/docs] Update index.md (#1306)
@Lilith-mx Hi, when to publish this? Also, please sign the CLA (just write "Mendix").
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/mendix/docs/pull/1306#issuecomment-410950394, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATZPMQn-rD40l0sUlKEJ-SEm7kVYjxh0ks5uOTZagaJpZM4Vxhcl.


Tagless final pt2

Using scalafix to rewrite all other interfaces not covered by #591 into Tagless Final-style. This does not actually alter the interpreters themselves, that will come in (a) subsequent PR(s).

[x] ProtocolSupportTerm
[x] ModelProtocolTerm
[x] EnumProtocolTerm
[x] ArrayProtocolTerm
[x] PolyProtocolTerm
[x] ServerTerm
[x] FrameworkTerm
[x] SwaggerTerm

Previously, on #591

[x] ClientTerm
[x] ScalaTerm

Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

[x] I acknowledge that all my contributions will be made under the project's license.

BTW, this was all almost completely mechanical due to https://github.com/blast-hardcheese/guardrail-scalafix-rules/blob/master/rules/src/main/scala/fix/GuardrailInterface2TF.scala . scalafix was a tremendous help to make these restructures much less painful than they needed to be.


Add docs for TextGenerationRequest

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

Comparison is base (67f2d45) 100.00% compared to head (9cd2bb3) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #39   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           21        21
  Lines          238       238
=========================================
  Hits           238       238

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


Implement IEnumerable extensions

Okay I just figured out how pull thing this works, I removed the property I added since it was no longer needed.

Works great! Thanks for your contribution!


Keeps track of tasks in scheduler grouped by status

I've been seeing half-second get_work runtimes recently on my scheduler. The
cause of this has been a large backfill that scheduled a lot of done tasks. With
about 100k done tasks and about 350 pending tasks, a lot of time is wasted
looping over done tasks just to ignore them. This essentially caches these
computations by keeping a dictionary to group tasks by status and updating it
whenever task statuses are changed. This has sped my scheduling up by an order
of magnitude, now taking about 20ms per get_work.
In order to ensure updates happen every time, the task state has to take over
any functions that modify status. This is a necessary for remote task storage
anyway, so it had to be done eventually. There are no new tests here because
there are no functionality changes, only improvements in efficiency and
different internal representations.

Was just going to merge this but now it looks like there's a merge conflict. Would you mind fixing and then I can merge? Thanks!

Fixed


Refactor core, add more collection and array functionalities

Codecov Report

Merging #2 into master will increase coverage by 0.86%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##             master     #2      +/-   ##
==========================================
+ Coverage     99.13%   100%   +0.86%
- Complexity       54    108      +54
==========================================
  Files             2      4       +2
  Lines           115    222     +107
==========================================
+ Hits            114    222     +108
+ Misses            1      0       -1

Impacted Files
Coverage Δ
Complexity Δ

src/UnderscoreBase.php
100% <100%> (ø)
32 <32> (?)

src/UnderscoreCollection.php
100% <100%> (ø)
67 <67> (?)

src/UnderscoreArray.php
100% <100%> (ø)
9 <9> (?)

src/functions.php
100% <100%> (ø)
0 <0> (?)

Continue to review full report at Codecov.

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


Count operands index from 1

Fix a issue that if you add constant as the first op, DML prototype will crash. The root cause is here: constants_info->memory_info.insert(index, std::move(memory_info)); in mojo_model_info.cc, WTF::HashMap can't allow index = 0 as key.
Now fix it by counting operands index from 1 and using map to replace array in mojom definition. PTAL, thanks! @fdwr

Thanks for the investigation and fix. It's too bad WTF::HashMap doesn't support 0 (quite surprising, given 0 has many useful cases 🤨). Would it be easier to use another map like std::map which allows zero, or is WTF::HashMap the preferred Chrome map?

WTF::HashMap

It's generated by mojom definition in webnn_graph.mojom:
// Represents the constant operands in the model.
[Stable]
struct ConstantsInfo {
  // The key is index of model's operands array to identify constant operand.
  map<uint64, MemoryInfo> memory_info@0;
  mojo_base.mojom.ReadOnlySharedMemoryRegion shared_memory@1;
};

Then in the generated file webnn_graph.mojom-blink.h:
WTF::HashMap<uint64_t, MemoryInfoPtr> memory_info;
Please see details https://chromium.googlesource.com/chromium/src/+/master/mojo/public/tools/bindings/README.md, map<S, T>  is Mojom Syntax. Not sure how to generate std::map by mojom. @fdwr


On mobile: hamburger dropdown is not escapable and language selector disappears

https://github.com/techequitycollaborative/tenantprotections2/pull/46


Feat/va sidebar

close #513

minimized version of sidebar

@nastassiadanilova
At this moment sidebar looks very strange, can you merge all latest updates from develop and fix conflicts?

@nastassiadanilova

Sidebar looks weird on small resolutions - http://piclair.com/data/emkdx.jpg
The color of the links in the sidebar does not match the design
Dropdown styling (dots, images)
Expectetions: http://piclair.com/data/yvc26.jpg
Reality: http://piclair.com/data/hvfre.jpg
Collapsed navigation looks weird http://piclair.com/data/l4b4v.jpg


fix(style) - refactor padding inside accordion panel

Fixes https://github.com/international-labour-organization/designsystem/issues/981
Notes :-

Currently when the accordion opens up the padding inside changes which breaks the sentence.

Fix :-

Add consistent padding inside the panel

Hey @Shashika6 were you able to reproduce this in Storybook?

Will move it back to ready for review after testing in bb8

@Shashika6 so you were able to test this?

Yes @justintemps tested this out in bb8


Cascated select with autocomplete - Can't access field.parent (possibly undefined)

Angular version: 16.0
Formly version: 6.1.8
I'd need to create a cascated select for my app. The first field is a Material Autocomplete and then the next one a simple select
I'm struggling, however, using hooks because field.parent appears to be possibly undefined:
(property) FormlyFieldConfig<FormlyFieldProps & { [additionalProperties: string]: any; }>.parent?: FormlyFieldConfig<FormlyFieldProps & {
    [additionalProperties: string]: any;
}> | undefined

So TypeScript is complaining when I try to access the previous field:
'field.parent' is possibly 'undefined'.ts(18048)
Cannot invoke an object which is possibly 'undefined'.ts(2722)

Thing is that I've tried to replicate the error because I thought that it might be related with my version of Angular:
https://stackblitz.com/edit/ngx-formly-ui-bootstrap-va2yrr?file=src%2Fapp%2Fapp.component.ts
But it seems to be working well (with neither field nor field.parent possibly "undefined") and I'm confused about what I'm doing wrong in my context to have this object that's potentially "undefined".
form = new FormGroup({});
  sendForm: FormlyFieldConfig[] = [
    {
      key: 'thirdParty',
      type: 'autocomplete',
      props: {
        createLink: '/third-parties/create-thirdparty',
        required: true,
        label: 'Tercero',
        displayProp: 'displayName',
        valueProp: 'id',
        placeholder: 'Empresa XYZ',
        filter: (term: string) => {
          return this.thirdParties.asObservable().pipe(
            map(thirdParties => term ? this.filterThirdParties(term, thirdParties) : thirdParties)
        )},
      },
    },
    {
      key: 'user',
      type: 'select',
      props: {
        label: 'Usuario',
        options: [],
        required: true,
      },
      hooks: {
        onInit: (field: FormlyFieldConfig) => {
          const thirdParty = field.parent.get('thirdParty');
        }
      },
      expressions: {
        hide: '!model.thirdParty',
      }
    }
  ];
  model = null;
  options: FormlyFormOptions = {};

if you're sure the field has a parent, just add ! to omit the typescript error:
- field.parent.get('thirdParty');
+ field.parent!.get('thirdParty');


Taking array element of a predefined function return doesn't work in Calc

I want to get the numeric value of "A" by chars('A')[0] but what we get is [ 65 ]; [ 0 ]

Other funny things here:

If we define C = { [1, 2, 3] } and then C[0] doesn't work ("Error: Variable C already has a non-array value"), while C()[0] does work ...
Should we (?) and do we (?) get an error if we were to fix the above but then do: C[1] = 3?

Another example: in "library.calc" we have this:
def pop($s) = { _v = splice($s._values, -1, 1); _v[0] }
which should be able to work as:
def pop($s) = { splice($s._values, -1, 1)[0] }
But it doesn't work currently; gets this:
[ 0 ]
So, basically we need both syntactic (in the grammar) and semantic (in the visitor) changes.

Currently (version 2.14.0) C[0] gives a different error: "Error: Cannot index into variable C which is type FUNCTION." which should be easy enough to fix (same deal as inside "evaluate" and a no-param function).

The problem is that the syntax for a function call is id '(' ... and should be expr '(' ... I haven't tried that to see if the grammar will compile or even work, and I don't know what implications this would have for LValueContext to be able to implement this ...


Include docs about Python logs auto-instrumentation

Included documentation about Python logs auto-instrumentation. Included info on the OTEL_PYTHON_LOGGING_AUTO_INSTRUMENTATION_ENABLED flag, and accompanying examples. Also included usage of this flag for OTel Operator Auto-Instrumentation.
Preview:

https://deploy-preview-3195--opentelemetry.netlify.app/docs/instrumentation/python/automatic/logs-example/
https://deploy-preview-3195--opentelemetry.netlify.app/docs/kubernetes/operator/automatic/

@chalin I added a bit more of an explanation to the beginning of logs-example.md. If you don't mind giving it another look. 😄

@avillela there's a build log warning, but otherwise this looks good. Could you address it?
@open-telemetry/python-approvers can you take a look please?

@avillela there's a build log warning, but otherwise this looks good. Could you address it?
@open-telemetry/python-approvers can you take a look please?

@cartermp I fixed the build issue, but now I'm getting some errors related to a file that I didn't change, and when I try to run the same verification commands locally, I can't reproduce the errors. Any idea as to what's going on??

@avillela I'm not sure anything was fixed. The latest change has an invalid link, and there's still issues with the file format/markdownlint jobs.

@avillela I addressed an issue on main and fixed a bad link here, but there's still evidently an issue with the link:
docs/instrumentation/python/automatic/logs-example/index.html
  target does not exist --- docs/instrumentation/python/automatic/logs-example/index.html --> /content-modules/opentelemetry-specification/specification/logs/bridge-api.md

@avillela I addressed an issue on main and fixed a bad link here, but there's still evidently an issue with the link:
docs/instrumentation/python/automatic/logs-example/index.html
  target does not exist --- docs/instrumentation/python/automatic/logs-example/index.html --> /content-modules/opentelemetry-specification/specification/logs/bridge-api.md

@cartermp finally managed to fix it. 😀


Add test helpers

Helper to create results (success and failures)
Assertions/Expectations (ex: expect(result).to have_value/have_error(1))

Resources:

https://relishapp.com/rspec/rspec-expectations/v/3-8/docs/custom-matchers/define-a-custom-matcher

Implemented in this PR: https://github.com/Fretadao/f_service/pull/35


Your config files SHOULD NOT be writable to this user!

i using admin acc logon on，see this warning!
how to solve?

Change permissions on the configuration file so the web server user account can't overwrite it.

how to solve it?

Change the permissions on the file. Basic Linux command. This is out of scope of MPOS and you should read up on Linux Administration prior to running a pool. It's stated in the README and Quick Start Guide that running a pool is not an easy tasks and requires good knowledge of Linux systems, Databases and PHP Code.


Moves created within the item bar do not have move results

In GitLab by @Asacolips on Apr 11, 2021, 15:04
Summary
Unlike moves created within an actor's character sheet, moves created within the item sidebar do not have move results added to their data object properly.

Labels (do-not-change)
/cc @Asacolips

In GitLab by @Asacolips on Jun 22, 2021, 17:32
I'm unable to reproduce this in the latest beta version. Closing this one as resolved.


Feature: add option to change port

Allows user to change the port of the dev
server by adding -p or --port= on the command
line.

NB: I'm pretty novice when it comes to Python. So review carefully!  ;-)


Adds Gitlab provider

Adds a Gitlab provider based on the one for github.
The mapping between Github and Gitlab entities:
Repo - Project
Org - Group
Team - Group

@dinosk we need to change the client options and allow setting a different hostname for SSH, please see fluxcd/toolkit#171

added an sshDomain field in https://github.com/fluxcd/go-git-providers/pull/36/files#diff-427f0edd6e28f8dd9251b1e762ca2b85R62 , will check if there is sth missing for this on the next iteration

Works! sans the caching request, our gitlab instance doesn't support that.
$ ginkgo gitlab
Running Suite: GitLab Provider Suite
====================================
Random Seed:<PHONE_NUMBER>
Will run 10 of 10 specs

• Failure [3.214 seconds]
GitLab Provider
/Users/simon/src/go-git-providers/gitlab/integration_test.go:125
  should list the available organizations the user has access to [It]
  /Users/simon/src/go-git-providers/gitlab/integration_test.go:200

  Expected
      <int>: 0
  to equal
      <int>: 1

  /Users/simon/src/go-git-providers/gitlab/integration_test.go:246
------------------------------
•••
------------------------------
• [SLOW TEST:5.222 seconds]
GitLab Provider
/Users/simon/src/go-git-providers/gitlab/integration_test.go:125
  should update if the org repo already exists when reconciling
  /Users/simon/src/go-git-providers/gitlab/integration_test.go:301
------------------------------
• [SLOW TEST:16.630 seconds]
GitLab Provider
/Users/simon/src/go-git-providers/gitlab/integration_test.go:125
  should update teams with access and permissions when reconciling
  /Users/simon/src/go-git-providers/gitlab/integration_test.go:344
------------------------------
• [SLOW TEST:6.614 seconds]
GitLab Provider
/Users/simon/src/go-git-providers/gitlab/integration_test.go:125
  should create, delete and reconcile deploy keys
  /Users/simon/src/go-git-providers/gitlab/integration_test.go:439
------------------------------
• [SLOW TEST:10.660 seconds]
GitLab Provider
/Users/simon/src/go-git-providers/gitlab/integration_test.go:125
  should be possible to create a user project
  /Users/simon/src/go-git-providers/gitlab/integration_test.go:501
------------------------------
•
------------------------------
• [SLOW TEST:14.436 seconds]
GitLab Provider
/Users/simon/src/go-git-providers/gitlab/integration_test.go:125
  should update if the user repo already exists when reconciling
  /Users/simon/src/go-git-providers/gitlab/integration_test.go:545
------------------------------
Deleting the user repo:  test-repo-194
Deleting the org repo:  test-org-repo-217
Deleting the shared org repo:  test-shared-org-repo-812

Summarizing 1 Failure:

[Fail] GitLab Provider [It] should list the available organizations the user has access to
/Users/simon/src/go-git-providers/gitlab/integration_test.go:246

Ran 10 of 10 Specs in 67.313 seconds
FAIL! -- 9 Passed | 1 Failed | 0 Pending | 0 Skipped
--- FAIL: TestProvider (67.31s)
2020/09/22 15:42:53 GET https://gitlab.com/api/v4/groups/9146106: 401 {message: 401 Unauthorized}

Ginkgo ran 1 suite in 1m10.758218143s
Test Suite Failed

Closing this, opened #51 to run the integration tests


Upgrade to Faraday 2.x

Fixes #41
I made it require a version greater than 2.0.1, as recommended by their own guide.

Thank you for the contribution! This commit was included in https://github.com/Shopify/oktakit/pull/68.
Closing ❤


Buffalo buffalo Buffalo buffalo buffalo buffalo Buffalo buffalo

Buffalo buffalo Buffalo buffalo buffalo buffalo Buffalo buffalo Buffalo buffalo Buffalo buffalo buffalo buffalo Buffalo buffalo Buffalo buffalo Buffalo buffalo buffalo buffalo Buffalo buffalo
Had to add the Buuffalo word also :/

Hi Vincent,
Thanks for the PR. I think to make the integration of the new Buffalo dialect, you have to modify this file: https://github.com/remusao/Hodor/blob/master/src/Main.hs to make it usable with the main test program.
Also if you could add some simple sample program to make sure Buffalo is turing complete, that would be great. (e.g: you could implement a simple turing machine in Buffalo, nothing to fancy).
Cheers,
Rémi


Refactor how app_data is computed in backend views

Purpose
We have now many views and all are rendering app_data and some of them are common to all views. We must refactor how they are computed because we had several cases where some new attributes are added at one place and not in an other.
Proposal

[x] create functions to manage common app data in views
[x] flags can be missing in app_data leading to a white page

Nice fix on missing flags :+1:


Add if a tool is available in a galaxy instance to the workflow

It would be good to add weather a tool is available on a specific instance to this workflow, therefore the following script could be used and extended to query the tools from the 3 big instances
https://github.com/bio-tools/biohackathon2022/blob/353570d03a3cccc73294b907c7c79ac9f814d815/scripts/workflowhub_galaxy_biotools.py#L24-L32
@supernord can the Australians take on this part ?

One thing to consider is, that the initial list we produce has a list of tool ids for each suite, so we would need to match each tool id and then decide what to show as output .... maybe tell the user each server that has at least one tool or the suite or all tools of a suite, but we cannot really give links to all tools, e.g. that would be 100 for mothur ...

I can help with this as well, it should be easy to do in BioBlend if you are not against adding that as a Python requirement.

I think with bioblend should be fine. What are your thoughts about the way how to present it at the final list:

links for each tool to the instance (will be a bit much info in my opinion
a check [x] if at least one tool is in the instance
a check [x] if all tools are in the instance

Not sure if there is even a case where not all tools of a suite are installed

good question
there are examples of this - I don't think Galaxy Australia has all OpenMS tools installed
for the moment it depends on how comprehensive the list needs to be for it to be useful for the community - for some tools it may be important if not all tools are available

For reference, this is the script from BioHackathon 2022 for getting tools from an instance
https://github.com/bio-tools/biohackathon2022/blob/main/scripts/workflowhub_galaxy_biotools.py#L24-L49

Since we extract the bio.tools id directly from the wrapper, I don't think we need this part here.

good question there are examples of this - I don't think Galaxy Australia has all OpenMS tools installed for the moment it depends on how comprehensive the list needs to be for it to be useful for the community - for some tools it may be important if not all tools are available

Maybe we could also present it like this e.g.:
Galaxy Shed Id: OpenMS
Instance .eu (5/10)
Instance .org.au (7/10)
Instance .org (10/10)

Since we extract the bio.tools id directly from the wrapper, I don't think we need this part here.

The key bit is the Galaxy API access, which is very straightforward.

I'm looking into writing a function that given a list of tool IDs in the form toolshed.g2.bx.psu.edu/repos/galaxyp/openms_falsediscoveryrate/FalseDiscoveryRate/2.6+galaxy0 and returning a pandas DataFrame (Galaxy servers x tool_ids) of True/False.
Let me know if this works!

@nsoranzo we do not really have a tool ID list in this way to start with, but only the wrapper id, but if we could produce the dataframe for all tools on the 3 main servers (~ 3-4.000 tools), we could just match the wrapper id to the tool id and that would work

Can we use the existing python function and update to extract the wrapper ids into a list?

The proposed function is ready for review in https://github.com/bebatut/galaxy_tool_extractor/pull/23 .

Done: https://github.com/galaxyproject/galaxy_tool_metadata_extractor/pull/23


Testing for the menu.
May never use.
Test URLs:

Before: https://main--24life--hlxsites.hlx.page/drafts/rrusher/load-menu-fragment
After: https://fragment-block--24life--hlxsites.hlx.page/drafts/rrusher/load-menu-fragment

seems not used


Option to change passphrase or 2 factor authentication

There doesn't appear to be an option to change the passphrase to the desktop client, this may be that the 12 word phrase is fixed indefinitely against that wallet.
How do you import other wallets?  Is it just a case of adding another using the 12 word passphrase for the other wallet?
Will there be 2 factor authentication

duplicate #14


Jetson Env

Dear GustavZ,
I want to know the environment that you run on Jetson for this model. Are you using tensorRT and cuDNN? We tried to run tensorflow zoo model on tensorRT, but it seems not supported well.
Thanks for your reply in advance!
Michael

I started with TensorRT, but i did not get along well with it. In my opinion it is not really recommendable.
My jetson has the following setup:
OpenCV 3.3.1
CUDA 9
cuDNN 7
Tensorflow 1.4

Hello Gustav! How did you install Tensorflow 1.4 on your Jetson?

I build it from source.
There are multiple tutorials if you dont know how to build tensorflow from source on the jetson.
This one helped me to understand it better:


Prevent direct access to the API behing Kong

Hey, sorry if this question has already been asked before but I can't find it in the issue tracker.
Summary
I wonder how you guys would do to prevent direct access to an API which is hosted on App Engine.
I want to use Kong in front of my API to handle the user authentication for me, but what if the user finds out the URL of the API and decides to bypass Kong?

If the backend is publicly accessible, you simply can't...
A quick way might be using basic-auth (or requiring a special header or something like that) on the backend, which you insert in Kong. Anything bypassing Kong wouldn't have the proper credentials/settings and be refused by your app. But this smells like 'security by obscurity'...

I see... that is what I was thinking. Thank you for the answer.


DFM 72AUC02 in Ubuntu 16.04 case:448962

I want to install the camera DFM 72AUC02 in Ubuntu 16.04. I have finished all the work following your instructions, it seems all right, however,
When I tried to detect the camera through tcam-ctrl -l , I got the message:
No device found.
Does DFM 72AUC02 have the Linux support?
How I can use this camera under Linux?
Thank you so much!

Hi
I suppose, you did not change the firmware in your camera.
Please refer to https://github.com/TheImagingSource/tiscamera/wiki/Getting-Started-with-USB-2.0-CMOS-Cameras
Stefan

Thanks for your reply.
When I issue the command: sudo ./firmware-update -ud 7214101 -f ../../../data/firmware/usb2/dfk72uc02_3012.euvc
the firmware file matches my camera, but it was rejected:
Firmware Size: 19968 EEPROM Size: 16384
Firmware does not fit in EEPROM. Aborting!

Thank you so much!!
So it can just used as a monochrome camera under Ubuntu?  But it works well under Windows.

Hello
I tried the beta-release branch and with the new tcam-capture I get a color image using this firmware. Same with a standard pipeline like
'gst-launch-1.0 tcambin serial=15810834 ! video/x-raw,format=BGRx,width=640,height=480,framerate=30/1 ! videoconvert ! videoscale ! ximagesink'
Stefan

Hello
You changed the firmware in the camera for Linux. For Windows, you must change it back to dfk72uc02_129.euvc
Stefan

Wow, thank you so much!


bug: it looks like String type as argument doesn't work

This function does not revert, whatever the string I pass to it through the SDK:
fn accepts_dynamic_string(s: String) -> () {
    assert(s.capacity() == 0); // this is true
    assert(s.as_bytes().len() == 1_000_000); // this is true as well
    assert(s.as_bytes().capacity() == 0); // this is also true and seems not coherent
}

Upon further testing, returning the s.as_bytes under an array form shows that the bytes received appear to be random, which further indicates an encoding error at the compiler level (because the script data I send out is coherent).
I can provide a better repro if needed.

So I actually cannot reproduce this inside a Sway function. I'm going to double check my SDK encoding.

@FuelLabs/sway-compiler I showed that the bug is not on the Sway-side in this branch https://github.com/FuelLabs/sway/tree/iqdecay/bug-string-encoding.
Process:
git checkout iqdecay/bug-string-encoding
fuelup component add<EMAIL_ADDRESS>fuelup component add<EMAIL_ADDRESS>./doit.sh

It will display receipts, the one we're interested in is the ReturnData one, which contains the bytes of the String received in the function. They are [0x43,0x47,0x54] which translates in ASCII to "CGT" which is the String we're passing to the contract in the script. Sorry for the false alarm.


Monitoring / logs for website build

It would be easier to figure out and fix issues like nixos-homepage#232 and #41 with log access.
Monitoring the website build, even for simple "hey, look, I failed", would at least let us act more quickly into figuring out a solution. A month is kinda slow :/.
(I have no actual knowledge or propositions for this.)

@samueldr is this still needed?

Indeed it's not an issue anymore.


Document baseline picking logic

This is going to be awesome to have up!  Will clear up so much confusion.  :)

Because SVGs are hard to review in GitHub:

This is looking great. I'm taking a quick pass to add a few updates to add details for how the "diff base mode" will impact baseline picking and how to configure the baseline branch to be something other than master. Will push up here in just a min.

Hey @djones let me know what you think of the changes I made here.

Some styling updates — not using "Build" as proper noun, so calling them "baseline build". Also removing "merge requests" since all pull request behavior is specific to GitHub right now.
Added sections for how to configure the baseline branch and how to enable manual diff base mode (which we don't want to encourage, so I didn't include any images or anything).
Elaborated on why we pick baselines in certain cases.

Also, the images look amazing! Would it look too weird to move the (2) in the first image to be below the base? So it reads as 1 --> 2 like the others.

Will update the diagrams tomorrow and then merge.


Implementing Insertion-Deletion

Short Description
Implementation of Insertion Deletion Metric for evaluating saliency maps
Fixes #15 .
Long Description
Implementation of required classes and functions
Added unit tests
Checklist

[x] Code linting are passed. (Please run pre-commit run --all-files).
[x] Mypy checks are passed. (Please run mypy saliency_metrics).
[x] Unit tests are added for new code.
[ ] Documentation is added for the new code.
[x] Unit tests are passed. (Please run sh tests/run_tests.sh).
[ ] Docs are successfully built. (Please run cd docs/ && make html and host the webpage locally).

@rjagtani Please fix the lint issue first.

@rjagtani In this iteration, I mainly reviewed the implementation. Please check the comments and solve the issues. After the implementation is corrected, I will review the unit tests. Please note that the expected test coverage is > 90%. You can check the coverage report in the CI logs, and see which lines are not covered by your tests.

@rjagtani Please mark the conversations as resolved after you make corresponding changes in your code.

I thought I'd do it once the changes are approved

@rjagtani The numbers in the test cases are too big, like 3072, 1024. Please note that the test cases are run on only a machine with 2GB memory. As more tests are written, they will be run in parallel, and the large tensors will result in a big memory footprint.

@rjagtani The numbers in the test cases are too big, like 3072, 1024. Please note that the test cases are run on only a machine with 2GB memory. As more tests are written, they will be run in parallel, and the large tensors will result in a big memory footprint.
Also, you can squash your commits before pushing them. In this PR there are already 50+ commits and the commit list is very long.

Thanks for reviewing the code, I think all implementation related changes are done. I will continue with testing related changes tomorrow, I will also read up on squashing commits since I've never done it before.

@rjagtani One addtional note for you:
Please add an extra argument in the initialization function of the metric: device: Union[str, torch.device] = "cuda:0".
Then create an attribute self.device = device. Then, send the self.classifier to self.device. Make sure you have freeze the whole classifier and turn on the eval mode in the initialization function.


Next not shipping with themes

Thought I'd try spinning up an app with the Next branch but notice it doesn't ship with the base themes?
However the docs for the next branch still show you being to use themes.
Thanks

Sounds like we should be porting the Theme documentation section to the next branch.
You have a look at how the documentation is handling the theme.

Thanks I've got it working.

The link is here now https://github.com/callemall/material-ui/blob/next/docs/src/components/App.js

Sorry but why is this issue closed we should still update the documentation so people would understand how to use Theme with the new version ?

@NitroBAY I don't understand your point. What's missing from the documentation? Do you want to improve it?


Async import-solution action times out after 1 hour but solution gets imported anyway

I'm using the pack-solution and import-solution actions to import an unmanaged solution from a GH repo to a dev environment. This is how I use the import-solution action:
- name: Import unmanaged solution to dev
      uses: microsoft/powerplatform-actions/import-solution@v0
      with:
        environment-url: ${{ env.DEV_URL }}
        app-id: ${{ env.APP_ID }}
        client-secret: ${{ secrets.CLIENT_SECRET }}
        tenant-id: ${{ env.TENANT_ID }}
        solution-file: out/solutions/${{ env.SOLUTION }}.zip
        run-asynchronously: true
        force-overwrite: true
        publish-changes: true

Most of the time this works as expected, but sometimes the action runs for an hour and then times out with Error: The HTTP request is unauthorized with client authentication scheme 'Anonymous'. But when I check the target environment, the solution has actually been imported and works just fine.
I've experienced this with different solutions and different environments. It's also happened with small solutions that usually only take a few minutes to import. Any ideas why this might be happening? It's a bit frustrating because every time this happens a full 60 minutes of run time get added to your billable GH actions minutes.

@nkirstein the underlying reason for the authN error is that for appID/clientSecret, the MSAL library offers no way to refresh the OAuth access token (i.e. no refresh token). And our pac CLI does not store the actual secrets on the runner/agent, so we have no good way to re-acquire a fresh token from the actual secrets.
We're considering a design change to capture the provided secrets within the nodejs process that runs a particular GH Action to improve this situation.
Ideally, importing solutions shouldn't take that much time to begin with, but alas, that's a current perf limitation within Dataverse.

@davidjenni Thanks for the detailed explanation. And also for the great work you're doing here, these actions are extremely useful.
Regarding the original issue, it now seems to me that it only happens with default and "developer" (aka community plan) environments but not with sandbox or production environments. The exact same solution might take 5 minutes to import into a sandbox, but may time out after an hour when importing to a default env. Not sure what that means, but that's what I'm seeing.

in recent releases of both pac CLI and the pp-actions tasks, we've added a more robust renewal for MSAL confidential apps by holding on to the client secret for the duration of the action. This is necessary, since in the MSAL design, confidential apps do not have a client-side refresh token that can be used


Make collection edit page wording consistent

Fixes https://github.com/samvera-labs/hyrax/issues/1020

Change "save" button text to refer to "update collection".
Make "Edit" and "Edit Access" showing of permissions both say
"Edit access"
Add a new method (permission_options), to be used for the dropdown selection of rights.
Apply the new method to all partials which use a dropdown selection of rights.

@samvera-labs/hyrax-code-reviewers

Thanks for the PR, @laritakr. I have a few quick questions :point_up:

Given that I had already added this to the spanish translation issue, I just added a link to this PR for additional reference on that issue. Translations should be able to be done at a later point.


[destination] implement loader for Snowflake with PUT stage

Background
Implement Snowflake destination based on PUT stage. It will be quite similar to the BigQuery implementation (just simpler).
Tasks

[ ] use jsonl as preferred type
[ ] use PUT to stage
[ ] allow option to remove files when COPY INTO succeeded.
[ ] our jsonl uses base64 to store binary so enable BINARY_FORMAT  in COPY INTO https://docs.snowflake.com/en/sql-reference/sql/copy-into-table
[ ] fail on any error on COPY INTO, return up to 100 errors
[ ] i think you must use TRUNCATE for replace load, do TRUNCATE COPY in one transaction!
[ ] implement cluster hint with CLUSTER BY

Authentication
Derived from ConnectionStringCredentials:

[ ] must understand sql alchemy syntax https://docs.snowflake.com/en/developer-guide/python-connector/sqlalchemy

[ ] should expose several query parameters as fields warehouse role

[ ] key pair auth should be supported https://docs.snowflake.com/en/developer-guide/python-connector/sqlalchemy#key-pair-authentication-support

Tests

[ ] we should enable it as destination in tests. all tests using ALL_DESTINATIONS must pass

[ ] it should pass the same common tests which BigQuery job client and sql client is passing

[ ] note that to pass sql_client tests you'll need to map DBApi exceptions into 3 categories that dlt needs: relation not found, terminal exceptions, transient exceptions. see bigquery and postgres implementations. that's manual work

all in all writing the test suite is more work than the code here

@adrianbr if you have any comments to the ticket...


Remove ESLint jest/valid-expect-in-promise

This PR refactors the way the tests catch errors. Currently they are wrapped in promises, but instead it can be refactored to use a jest trick of await expect(function).rejects.toThrow('error message'). Using this trick, we can remove this lint rule.

I decided to go ahead and merge this now, to reduce the chance of future conflicts


Fix spelling errors on Chromium Chronicle 7

What's changed, or what was fixed?

In the featured snippet add missing e's

Fixes: #8274
CC: @petele

:tada: This has been pushed live to https://developers.google.com/web/


can't see the  TLS response of request

Hi, thanks for providing this tool, after following the steps in README, I successfully configured the connection between Wireshark and my java-httpclient demo, and now Wireshark captured the TLS request sent in httpClient(the test https link is https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Set-Cookie), but I can't see the TLS response of this request, Is this because of my Wireshark filter setting, or need extra configuration?
The Wirshark screenshot:

The reason is your filter settings. You can right click and Follow stream to see the full communication for the particular request.

Thanks for the tip,I will give it a try.

Indeed, after change the filter setting to (http || tls) and (tcp.port==8081 || tcp.port==443),I can see the TLS traffic sent by HttpClient in my spring boot application which listens on port 8081.


When will mac be supported?

When will mac be supported? I am using teams to communicate with colleagues from different countries and this will be of great help to me

Unfortunately, never (or at least not anytime soon). There are too many dependencies on windows-only components.


Fix version in Readme

Uses the latest version (1.0.2) in the Readme examples.

thanks so much for that @jkostolansky


[SPARK-5363] [PySpark] check ending mark in non-block way

There is chance of dead lock that the Python process is waiting for ending mark from JVM, but which is eaten by corrupted stream.
This PR checks the ending mark from Python in non-block way, so it will not blocked by Python process.
There is a small chance that the ending mark is sent by Python process but not available right now, then Python process will not be used.
cc @JoshRosen @pwendell

Test build #27463 has started for   PR 4601 at commit 05e1085.

This patch merges cleanly.

Test build #27463 has finished for   PR 4601 at commit 05e1085.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class Params(

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

Can we add logging for the uncommon cases here?  I'd add a log message for the case where the next integer is not available and a second case for when it's not END_OF_STREAM (this log message should contain the actual integer received).

Test build #27598 has started for   PR 4601 at commit 656d544.

This patch merges cleanly.

Test build #27600 has started for   PR 4601 at commit 890329c.

This patch merges cleanly.

Minor log-level nitpicking aside, this looks good to me.

@JoshRosen updated

Test build #27602 has started for   PR 4601 at commit e15a8c3.

This patch merges cleanly.

Test build #27602 has finished for   PR 4601 at commit e15a8c3.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

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

Test build #27600 has finished for   PR 4601 at commit 890329c.

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

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

Test build #27598 has finished for   PR 4601 at commit 656d544.

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

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

LGTM.  I'm going to merge this into master (1.4.0), branch-1.3 (1.3.0), and branch-1.2 (1.2.2).

I merged this into master (1.4.0), branch-1.3 (1.3.0), and branch-1.2 (1.2.2), but did so right before I noticed that there's a comment on JIRA suggesting that this didn't fix the freeze.  I guess I was a bit too trigger-happy here since I wanted to try to squeeze a fix in for 1.3.0.

Reverted in master (1.4.0), branch-1.3 (1.3.0), and branch-1.2 (1.2.2).


Allow unlimited API results with limit=-1

Is your proposal related to a problem?
We are trying to use WAGTAILAPI_LIMIT_MAX = None to return all API results in a single query, but it appears there is no way to invoke an unlimited request with the limit parameter.
Describe the solution you'd like
I would like to pass?limit=-1 to get all the results in a single query, but this currently raises a ValueError
https://github.com/wagtail/wagtail/blob/5be9bd12d3ecd198e7f9b581df99aa07e8b5a3ed/wagtail/api/v2/pagination.py#L24-L27
Describe alternatives you've considered
We can pass a sufficiently large integer (e.g. limit=999), but we would like to have a true "unlimited" option.
Additional context
Initial issue: https://github.com/WGBH-MLA/ov-wag/issues/217
Working on this
Looking at the source code, it seems like a straightforward change, and we would be willing to create a PR, if that would be helpful.
Thanks!

Thank you @mrharpo! I can see how this could be convenient for some projects but I’m hesitant to recommend we’d proceed. The WAGTAILAPI_LIMIT_MAX setting is there to make sure the limit provided in a query parameter stays reasonable, though it can be set to None to enforce no limit. Which would need to be done for limit=-1 to work.
Once that’s done, your proposed ?limit=-1 can already be achieved by setting an arbitrarily large limit as you mention? If you set limit=999999 that’s more pages than I’ve seen in 99.99% of sites. One more 9 to reach 10M and that’s more pages than I’ve ever heard of in any site. Either way, true "unlimited" likely wouldn’t be supported as there’d be a bottleneck at some point down the stack.
I’ll tag this as "Needs design decision" so other core team members review. Not sure if supporting-1 is worth the effort when there are such clear, simple alternatives – but please let me know if 999999 is problematic in some way I might not understand.

@allcaps I see this relates to your suggestion in RFC 100 to support a "All page URLs endpoint". What do you think of making it easier to have a "limitless" API response?

@thibaudcolas Thank you for considering this feature! (Even if seems like a trivial use case)
We are implementing the 999... solution for now, which will be more than enough for our project.
I created the issue because as a developer, I assumed setting WAGTAILAPI_LIMIT_MAX = None would have been enough to allow a client application to get everything with limit=-1 (or some other common syntax). I agree, a true "unlimited" option would hit real-world limits, but in practice, I think there is a benefit to allow a consistent way to easily request "all" records (even though the actual number may never exceed 100 in the lifetime of this particular app).
A limit=-1 call would still be subject to the WAGTAILAPI_LIMIT_MAX if set, so a backend can still be configured to have reasonable limits, but this allows clients to use a common syntax for cases when they want to get "all" the records in a single call, or at least as many as possible.
In any case, a note in the documentation for WAGTAILAPI_LIMIT_MAX would be appreciated with whatever is decided!

hey, can I work on this issue? I may be able to integrate the option for 'unlimited'. Please assign this to me

@Illucious this issue is marked as "needs design decision" – a decision needs to be made by the core team on whether we do want to proceed with this, before it can be worked on.


Call git config instead of trying to parse ~/.gitconfig

zigmod init always fails for me. My gitconfig is apparently not ini compatible and it fails with "error: IniSyntaxError" in guessCopyrightName. It would be safer to call "git config --get user.name".

Please open an issue for this first. This is not the approach I want to take for the error. Sorry for this result.

Sure, no problem.


add Rob Hirschfeld

includes RackN as company

Thank you for contributing to stackforge/stackalytics!
stackforge/stackalytics uses Gerrit for code review.
Please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


Master branch differs from published PyPi version

TextAnalyticsWrapper is not exported in the current published version, 0.1.0. The following code fails when executing against the published version, but works fine when building from source:
from pathlib                                import Path
from copy                                   import deepcopy
from pprint                                 import pprint
from collections                            import Counter

from presidio_evaluator                     import InputSample
from presidio_evaluator.evaluation          import Evaluator, ModelError
from presidio_evaluator.models              import TextAnalyticsWrapper
from presidio_evaluator.experiment_tracking import get_experiment_tracker
import pandas as pd

def main():
    print('hello world')

if __name__ == "__main__":
    main()

There are no versions or tags, so this will build whatever the current state of the master branch is: pip install git+https://github.com/microsoft/presidio-research.git, but you can always fork and tag it yourself as a workaround for this issue.

You might need to install plotly as well. It doesn't seem to be listed in the documentation or required.

Thanks @seanlindo. We periodically release a new version, and will make sure all requirements are handled.

The package installed via pip misses this file: presidio_evaluator/data_generator/raw_data/us_driver_license_format.yaml
This leads to the 1_Generate_data.ipynb notebook failing add the driver license provider to the faker.
And even if the yaml is manually copied into the package and the faker can successfully add the provider, generating the data still fails with:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
File [~/.pyenv/versions/3.11.6/envs/piidetection/lib/python3.11/site-packages/faker/generator.py:92](https://file+.vscode-resource.vscode-cdn.net/Users/gustav.vonzitzewitz/workspace/buzok_experiments/PII_detection/presidio/~/.pyenv/versions/3.11.6/envs/piidetection/lib/python3.11/site-packages/faker/generator.py:92), in Generator.get_formatter(self, formatter)
...
AttributeError: 'RecordGenerator' object has no attribute 'us_driver_license'
...
AttributeError: Failed to generate fake data based on template "My driver's license number is {{us_driver_license}}
".You might need to add a new Faker provider! Unknown formatter 'us_driver_license'

@gustavz I'm still facing the issue with us_driver_license template
AttributeError: Failed to generate fake data based on template "My driver's license number is {{us_driver_license}}".You might need to add a new Faker provider! Unknown formatter 'us_driver_license'

A new version (0.1.2) was released to PyPI and is now aligned with the master branch.

Reopening as there's still an issue with the us driver license generation.

Fixed in 0.1.3


[BEAM-9615] Add string coder utility functions.

This adds initial utility functions for encoding and decoding utf8 strings in the Go SDK.
Doesn't make use of them yet. In practice this is already how strings are encoded in the Go SDK, but marked as "custom" coders rather than the built in URN.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

R: @tysonjh
cc: @youngoli

Ah dang it. I thought I had pushed the commit with the rename, but it was waiting on a password. I'll have that as another PR.


Upgrade linter versions to the newest versions that still install under Python 2

We picked up some bugfixes along the way, so I fixed the lint they detected, and also seemingly a bug in pylint, which I reported: https://github.com/PyCQA/pylint/issues/3039

We run linting under Python 3.6, but unfortunately the way that pipenv is currently set up requires the linters to successfully install under all environments, and pylint==1.9.5 is marked as not supporting Python 3.7. Similarly, pylint>=2 are Python 3-only, so we have to use pylint==1.9.4 as the closest version that isn't blacklisting Python 3.7. I suspect that version doesn't actually support 3.7, but since the development environment (and by extension the lint environment) is using 3.6, that doesn't really matter here.
TL;DR: My solution is ugly, but it works well enough to not be worth spending additional time on at the moment.


Not beginner friendly with Main() and explicit

I was teaching OOP one of my students and was covering interfaces. I gave them the following task:
Fishes don't talk but fish is an animal
Cat is an animal and can talk
Dog is an animal and can talk
Human is not an animal, but can talk
They talked they sent me this:
interface ITalk
{
    void Talk();
}

class Animal
{
    public virtual void WhatAnimal()
    {
        Console.WriteLine("Animal");
    }
}

class Fish : Animal
{

}

class Cat : Animal, ITalk
{
    public void ITalk.Talk()
    {
        Console.WriteLine("Meow");
    }

    static void Main()
    {
        ITalk obj = new Cat();
        obj.Talk();
    }
}

class Dog : Animal, ITalk
{
    public void ITalk.Talk()
    {
        Console.WriteLine("Woof");
    }

    static void Main()
    {
        ITalk obj = new Dog();
        obj.Talk();
    }
}

class Human : ITalk
{
    public void ITalk.Talk()
    {
        Console.WriteLine("Hello");
    }

    static void Main()
    {
        ITalk obj = new Human();
        obj.Talk();
    }
}

static void Main()      //Polymorphism
{
    ITalk a1 = new Cat();
    ITalk a2 = new Dog();
    a1.Talk();
    a2.Talk();
}

Why? Because of this page. Please fix it and make it clear, don't make lazy docs, make it intuitive.

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

ID: ef21cb62-9cbd-84c8-cfae-6dd1a421180b
Version Independent ID: d931e899-d6f0-55ff-ab3a-dfff6e31f5a2
Content: interface - C# Reference
Content Source: docs/csharp/language-reference/keywords/interface.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

@Almantask We agree that this is the wrong article for beginners that don't yet have a background in OO techniques.
I'm curious how they landed here, instead of the introductory material on interfaces in the programming guide. That material, while still not perfect, is more approachable and explains the concepts you were trying to teach.
The language reference is for the syntax and semantic rules of the language, not style and design questions.
I'm going to close this, as the concerns you raise should not be addressed on this reference article. If you have comments on how we can improve the introductory OO material, you can open issues in the programming guide section.


add board definition of eetree gamekit rp2040

This is a board made by eeTree tech
The schematic, gerber files and more infomations are as follows:
https://www.eetree.cn/project/detail/698
The board uses a Puya flash named P25Q16. We tested that it works well with #define PICO_BOOT_STAGE2_CHOOSE_W25Q080 1

Replaces #709


How to handle Authentication Required popup

I have a problem with Authentication popup. How to input username and password into the Authentication popup with codeceptjs sdk automatically? Please help me to resolve this issue.
Thank you so much.

If you mean a Basic Auth dialog, you can provide user and password in the url in your codecept.json file like this:
{
  "tests": "./*_test.js",
  "timeout": 10000,
  "output": "./output",
  "helpers": {
    "WebDriverIO": {
      "url"<EMAIL_ADDRESS>      "browser": "chrome",
  ...

Close due inactivity


Font flashes as wrong font

Problem: When a page loads on the first try, the font is not as specified in the theme.
Use: https://nextjs.org/docs/pages/building-your-application/optimizing/fonts

I've tried:

Using the style tag: The issues persists.
<link rel="preconnect" href="https://fonts.googleapis.com">
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
<link href="https://fonts.googleapis.com/css2?
family=Kumbh+Sans&family=Outfit:wght@300&family=Permanent+Marker&display=swap" rel="stylesheet">

Using @fontsource: This was the original state and has the issue.

Working solution:
import { Kumbh_Sans, Outfit, Permanent_Marker } from "next/font/google"
import { Theme, ThemeOptions, createTheme } from "@mui/material"

const permanentMarker = Permanent_Marker({
  subsets: ["latin"],
  weight: "400",
})
const kumbhSans = Kumbh_Sans({ subsets: ["latin"], weight: "400" })
const outfit_font = Outfit({ subsets: ["latin"], weight: "300" })
const outfit = outfit_font.style.fontFamily

export const base: ThemeOptions = {
  typography: {
    fontFamily: [
      "-apple-system",
      "BlinkMacSystemFont",
      outfit,
      "Roboto",
      "Arial",
    ].join(","),
    h1: { fontFamily: permanentMarker.style.fontFamily, fontSize: 110 },
    subtitle1: { fontFamily: kumbhSans.style.fontFamily, fontSize: 41 },
}


Make sure information logs are removed after one day

https://linux.die.net/man/8/logrotate
https://stackoverflow.com/questions/28696069/elasticsearch-how-to-configure-logging-yml


AubsUK - Added Scan option to scan List1, List2, All Channels, All Lists

Added Scan option to scan List1, List2, All Channels, All Lists
New list options in menu item 19:

LIST1  - Scan through all channels added to List1
LIST2 - Scan through all channels added to List2
ALL CHANNELS - Scan through all channels (List1, List2, and channels not added to any list)
ALL LISTS - Scan through all channels added to List1 or List2

Scan status in the top left will now show:

1 - List1
2 - List2
C - All Channels
L - All Lists

I will add another option to the list, so will close this PR.


Depth of field should be enabled on webgpu

          I believe this should just check for webgl, since webgpu should support depth textures. This currently disables it for webgpu too.

Originally posted by @IceSentry in https://github.com/bevyengine/bevy/pull/13009#discussion_r1597707303

Please correct me if I'm wrong, but this seems to be fixed already by https://github.com/bevyengine/bevy/pull/13374. If so, maybe we could close this issue?


bcache-tools: 1.0.7 -> 1.0.8

Description of changes
Practically none.  Between 1.0.7 and 1.0.8:

The code was changed to use builtin kmod, which our patch already did (so this lets us remove it from the patch)
The bcache-register script was replaced with a small executable, but we don't use it - we already delete it entirely and use a line of shell in the rule - so it doesn't matter at all.

https://github.com/g2p/bcache-tools/compare/v1.0.7...v1.0.8
In all honesty, the only reason why I am providing this PR at all is because I found it very entertaining that technically this package is nine-and-a-quarter years out of date.
Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

There also seems to be version 1.1 at a different url:
origin: https://git.kernel.org/pub/scm/linux/kernel/git/colyli/bcache-tools.git/
repology: https://repology.org/project/bcache-tools/versions

There also seems to be version 1.1 at a different url: origin: https://git.kernel.org/pub/scm/linux/kernel/git/colyli/bcache-tools.git/ repology: https://repology.org/project/bcache-tools/versions

Oh, now I remember: https://github.com/NixOS/nixpkgs/issues/271977

@ofborg test installer

@lukegb wrt the concerns we discussed, does https://hydra.nixos.org/build/253333786 show that the installer tests are already failing in the same place as the ofborg run you performed with the above comment?

@lukegb wrt the concerns we discussed, does https://hydra.nixos.org/build/253333786 show that the installer tests are already failing in the same place as the ofborg run you performed with the above comment?

Good timing; you commented just as I was investigating ^^
Yes, I think nixosTests.installer.bcache is broken at HEAD. This is... surprising and unfortunate.
In any case, I don't think this makes the situation any worse, LGTM, ship it, whatever.


Code editor not resizable in Chrome

Can't resize the code editor in Chrome 90. Works correctly in latest Edge tho.
Also, related: currently code editor width is restricted by side design. Would be nice if I could resize it as wide as reasonable.

I'm on Chrome 90, and I can extend the code editor vertically, and similarly in edge too. Is this occurring on a specific problem or is it a site wide issue for you?

I can't reproduce this today anymore. Could have been a fluke on my end. Thanks!


Creating a panel to update multiple entities

I doubt this is something that has built in functionality but I've thought that a number of times before and been pleasantly surprised to find there is, hence asking here.
I'm trying to develop a panel page, which has a parent object and then two child objects (through a one-to-many relationship) that will be created at the same time.  While I appreciate it would be easier from our point of view to have the parent object created then add the child objects as separate steps the users' existing system implements it in one form so don't really want to add more steps in.
I've tried basing the development on Entity Dialog as Panel without success. Is there a simple way of achieving this through Serenity or am I just overcomplicating the Entity Dialog as Panel approach?
Any guidance you're able to give would be greatly appreciated.

Have you tried with a mix of example in http://serenity.is/demo/BasicSamples/OtherFormInTab or http://serenity.is/demo/BasicSamples/OtherFormInTabOneBar and http://serenity.is/demo/BasicSamples/EntityDialogAsPanel/11077  ?
I do not know, however, it could be a starting point

how about calling the other dialogs on AfterSave() and passing on the information there? If it's a matter of simply duplicating the data onto the appropriate columns on the other grids without using joint fields, I think it should work.


Minimal working ex

V2 of the shiny app integrated into a minimal working example workflow of the reproducible reports.
A lot of the files changed are only white space updates, the main changes occur in:

/app/R/renderReport.R
app/modules
app/templates/EN/generic_intro.Rmd

Most of the other files are files in the reports/ directory, which consists of all of the individual sections of the report and should be largely independent to how the application functions.

This is now pretty out of date and not necessary given the latest updates, so I'll close this PR now.


Implement backtracking

Currently, parsers generated by abnf won't match rules like
foo = *("a" / "b") "b"

Backtracking is implemented in https://github.com/declaresub/abnf/commit/273d54452f5e5a14236668f8b9d86440fc139cca.

Implemented in v 2.0.0.


Redis client changes: Add retries

I'm going to close this for now as I think we're going to want to revisit the general approach not just in terms of retries, but also with the concept of having a shared connection pool across the whole context


Google Chrome - Websockets Error - (Opcode -1)

I'm running into an issue in Chrome where websockets are erroring out with "(Opcode -1)". This was originally with 0.9.2, but I just updated to 0.9.3 today and am still having the issue.

It doesn't seem to be an application error (happens with both good/bad apps)
No log entries
Firewall rules allow all relevant ports in both directions

Opcode -1 isn't a valid opcode, which led me to Wireshark the connection. This confirmed that data was being sent after the server sent a close connection.
The signal appears to be sent from here, but I'm not that familiar with Shiny, and our R devs aren't aware of a reason for this. Firefox reports Firefox can’t establish a connection to the server at `. I understand Shiny should fall back to Ajax, but it doesn't look like that's happening.

I'm not positive this is a ShinyProxy problem, but I'm not really sure what's causing it at this point.

Hi @cmattoon,
Thanks for the detailed description.
Did you always have this issue, or did it start happening at some version or update?
I cannot imagine what part of ShinyProxy could cause this issue. It uses Undertow as its underlying web server, and beyond setting up a ProxyHandler to the docker container, ShinyProxy does nothing to the proxied connection.

We've had the issue intermittently, and it seems that this is a generic manifestation of "something went wrong". Currently, everything is working as expected after updating apps and adjusting Netscaler/Haproxy settings, though this was happening with direct connections as well.
I'm going to mark this as closed; thanks for your help!

``


Missleading "Returning Focus" example

Hi,
the Readme "Returning Focus" example is not working, the containerRef passed to the hook is empty and not assigned to an element. Also I guess it's now outdated.
So what would be the current "proper" way to return focus on a different element than the triggering one?
Thanks!

I ended up removing this behavior just because I couldn't get the ref setup working correctly, but meant to re-add after finishing up the rest of the package.
I think I can re-add this quickly, so I'll get to that now!


FAQ (please look for a solution here before asking)

Q: The extension doesn't show up in the Editor menu.
A: Please make sure Copilot is turned on in System Settings.app > Privacy & Security > Extensions > Xcode Source Editor Extension.
Q: The extension says it can't connect to the XPC service/helper or the XPC Service version loading never ends.
A: If you have just updated the app from an old version, make sure you have restarted the XPC Service.
Please make sure you have set up Launch Agents, try running launchctl list | grep com.intii from the terminal, and see if com.intii.CopilotForXcode.ExtensionService exists. If not, check ~/Library/LaunchAgents to see if com.intii.CopilotForXcode.ExtensionService.plist exists and the program path matches the location of the app. If they don't, and the button in the app fails to create them, please try to do it by hand.
This is an example of the .plist file. Please note that the application path may need to be adjusted based on your installation location.
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
    <key>Label</key>
    <string>com.intii.CopilotForXcode.ExtensionService</string>
    <key>Program</key>
    <string>/Applications/Copilot for Xcode.app/Contents/Applications/CopilotForXcodeExtensionService.app/Contents/MacOS/CopilotForXcodeExtensionService</string>
    <key>MachServices</key>
    <dict>
        <key>com.intii.CopilotForXcode.ExtensionService</key>
        <true/>
    </dict>
</dict>
</plist>

After creating the file, set the proper permissions: chmod 755 ~/Library/LaunchAgents/com.intii.CopilotForXcode.ExtensionService.
If you are installing multiple versions of the extension on your machine, it's also possible that Xcode is using the older version of the extension.
Q: Host app shows Server unavailable, please make sure you have installed Node, or GitHub Copilot not working.
A: Please ensure that the Path to Node correctly points to a valid Node executable. Avoid using ~ in place of /Users/username
If you are using Node version v16.13.x or anything lower, consider updating to a newer version of Node.
If you're using nodenv, use the path that points to a specific version of Node, such as /Users/username/.nodenv/versions/19.8.1/bin/node.
You can also try setting "run node with" to /usr/bin/env in the host app.
After making changes, be sure to restart both the host app and the helper app.
If none of the above is working for you, you can observe the logs posted by the language server using the Console.app.

Set up the filter to the process:CopilotForXcodeExtensionService subsystem:com.chimehq.JSONRPC to show only the logs from the language server. See if there is anything you can do.
Set up the filter to process:CopilotForXcodeExtensionService subsystem:com.intii.CopilotForXcode to show only logs from the helper app.

You can also run the project from Xcode, add take a look at target CopilotService. Set localServer.logMessages to true and see if anything useful is printing on the console.
Q: Host app shows timeout when signing in to GitHub Copilot.
A: The language server seems to be not working correctly. You can observe the logs posted by the language server using the Console.app. Set up the filter to the process CopilotForXcodeExtensionService and subsystem com.chimehq.JSONRPC to show only the logs from the language server. See if there is anything you can do.
Also, make sure you are not doing any thing in you hosts file about GitHub.
Q: The extension complains that it has no access to the Accessibility API
A: Please check if the Accessibility API permission is set up correctly.
Q: I turned on real-time suggestions, but nothing happened
A: Try typing something in the editor and wait for a little while, if you see an animation from the real-time suggestion indicator or the floating widget, that means the real-time suggestion is correctly triggered.
Q: I can't cancel real-time suggestions with mouse clicks, or pressing esc.
A: Please make sure the real-time suggestion debounce is long enough (1 second by default).
Q: I have signed in to my GitHub account, but the app doesn't generate any suggestions.
A: Please make sure the GitHub Copilot status is OK. If it's not, it's likely that your GitHub Copilot subscription is not valid.
If you are new to GitHub Copilot, it's recommended to visit its official website to learn about its behavior.
Q: The GitHub Copilot status is Not Authorized, or anything not OK.
A: Please make sure your GitHub Copilot subscription is valid.
Q: The widget presents a LanguageServerError that has no enough information about what the cause is.
A: The error is posted by the GitHub Copilot language server. Most of the time you can just ignore it. If the error persists, please try:

Check you GitHub Copilot settings in the host app, make sure the status is OK.
You can observe the logs posted by the language server using the Console.app. Set up the filter to the process CopilotForXcodeExtensionService. See if there is anything you can do.

Q: I quit the helper app accidentally, how do I restart it?
A: Every time you run a command, open Xcode, open Copilot for Xcode.app, the helper app will be launched.
Q: Will it work in future Xcode updates?
A: I don't know. This extension uses many tricks to do its job, and these tricks can break in the future.
Q: How do I remove commands from the KeyBinding settings page if they are no longer available?
A: The key binding configuration is stored at ~/Library/Developer/Xcode/UserData/KeyBindings. You can open the configuration file in any text editor and remove any item whose CommandID contains com.intii.CopilotForXcode.EditorExtension.
Q: When I tried to use gpt-4 models, I got message "The model: gpt-4-xx does not exist"
A: To access GPT-4 models, you must join the waitlist and await approval from OpenAI, even if you've subscribed to ChatGPT Plus. The available models in the app are based on OpenAI's documentation, but not all GPT-4 models are currently available to the public.

Hey, Bro,  Sorry to trouble you.  Would you share me the implementation of  how to launch the CopilotForXcodeExtensionService.app while executing the editor command?

@alexlee002 The key is to use a launch agent to launch an XPC Service. For reference, you may examine this smaller project: https://github.com/intitni/XccurateFormatter. In its readme, there is a link available that explains how to configure the launch agent.

@intitni yes, I read that document last weekend, but I failed to do it in my demo... always something wrong

And another question,  The XPCService commonly without UI,  so , How the source editor extension launches the CopilotForXcodeExtensionService.app?   by calling getService()?  but I launch it in Terminal via launchctl,   the xpc service is launched, but I couldn't find the UI in menubar ...

Nobody says an app with UI can't be an XPC Service. So CopilotForXcodeExtensionService.app is the XPC Service and it will launch when getService is called.
2023年5月15日 +0800 20:24 Alex Lee @.***>，写道：

And another question, The XPCService commonly without UI, so , How the source editor extension launches the CopilotForXcodeExtensionService.app? by calling getService()? but I launch it in Terminal via launchctl, the xpc service is launched, but I couldn't find the UI in menubar ...
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Nobody says an app with UI can't be an XPC Service.
I'm using Xcode 14.3, If I create an App Target, and Add XPCService configuration into Info.plist,  But I Can create a build phase named 'Embed XPCService', and I can only add it to Embed Foundation Extensions, but it could work


fix build when build path has spaces - 2.0.x

Change Description

Change Type
Select ONE

[ ] Documentation

[x] Stability bug fix

[ ] Other

[ ] Other - special case

Consensus Changes

[ ] Consensus Changes

API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

@skoyya-bo can you indicate if this is a clean cherry-pick of fix_build_space_path, or if you had to make changes, so we know how close we need to inspect this?

Missing change for eosio-wasm-spec-tests/generated-tests/CMakeLists.txt (should have submodule change here and also change in that file in the eosio-wasm-spec-tests repo)
Missing change for libraries/appbase/version.cmake.in (should have submodule change here and also change in that file in the appbase repo)

Closing as this is not complete and not on the radar to have someone else take over.


Load administration even if plugin js does not exist

1. Why is this change necessary?
If a plugin's administration javascript file can't be loaded the whole administration fails to initialize and only shows a white page.
Currently, if a plugin's javascript file fails to load, the administration will fail to initialize since the error is not handled adequately (https://github.com/shopware/platform/blob/master/src/Administration/Resources/app/administration/src/core/application.js#L378-L383). This pr allows the loading of the plugin to fail but ensures that the administration will load.
2. What does this change do, exactly?
This change allows the including of administration files of plugins to fail without taking down the whole administration.
Since the build process only uses core-js in version 2, I refactored the method to handle the error correctly. With core-js >= 3 we could have used allSettled here so a version upgrade would be great.
3. Describe each step to reproduce the issue or behaviour.

Install a plugin with an administration javascript file.
Rename the javascript file so that the administration can load it.

4. Please link to the relevant issues (if any).
5. Checklist

[ ] I have written tests and verified that they fail without my change

I can't find an adequate way to tests this change and the application.js does not seem to be tested yet. Do you need a test here? If so can you please point me into the correct direction how I would write a test for this.

[x] I have squashed any insignificant commits
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hey @niklasbuechner, could you please put the error-handling into the injectPlugin(plugin) method? then you could also just test that method.
Regarding core-js: major update of dependencies can potentially break Plugins, therefore we will have to test these extensively beforehand. We do already have a ticket for this but i cant say when this will be done.

Hey @niklasbuechner, could you please put the error-handling into the injectPlugin(plugin) method? then you could also just test that method.
Regarding core-js: major update of dependencies can potentially break Plugins, therefore we will have to test these extensively beforehand. We do already have a ticket for this but i cant say when this will be done.

Hello, we're closing this pull request due to inactivity. If this change is still important to you, feel free to create a new pull request.
For more information about our contribution guidelines, see https://docs.shopware.com/en/shopware-platform-dev-en/contribution/contribution-guideline.

New pr: https://github.com/shopware/platform/pull/1633