TypeError: Property 'authorization' must be of type state.

Hi, I'm using the ciscospark.init() (in a browser) to set programatically the clientId, clientSecret. I know you should build the browser version with ENV_VARS but it is not very practical for us. We want to set those infos at Runtime.
If I kickoff the OAuth flow with ciscospark.authorize(), it works. I get a nice json in my localStorage at the key ciscospark.
But then if I call ciscospark.authorize() again I have an error Unhandled promise rejection TypeError: Property 'authorization' must be of type state. Tried to set [object Object].
Do you have any clue on what is happening ? What can I do ? Thanks.
Example of init :
const ciscospark = Ciscospark.init({
  config: {
    credentials: {
      oauth: {
        client_id: clientId,
        client_secret: clientSecret,
        redirect_uri: redirectUri,
        scope,
      },
    },
  },
});

ok, so, i've had my coffee now and I remember why init won't work in the browser. after doing an oauth login, as ciscospark instantiates, it checks window.location for an OAuth Authorization Code. ciscospark.init creates a second instance and that instance will never see the code in the url, since the first instance already used and removed it.
When you say you're calling ciscospark.authorize again, do you mean after an oauth redirect? or multiple times after a given page reload?

I think it kind of works for the OAuth workflow, I can see the access_token (but no refresh_token) in the localStorage at the key "ciscospark".
Yes I call #authorize before making a call. So after the OAuth and multiple times per page reload. I thought it was a simple call to test you're authenticated and proceed to a call. Isn't it ?

ok, yea, the oauth flow might work now that the sdk handles localstorage, but it's only working because because both ciscospark instances talk to the same local storage key. I imagine that could produce race conditions at some point and it definitely is not something i will guarantee will work forever. (as this conversation continues, I'm trying to figure out how best to make init available without having the automatically created instance).
You should only need to call authorize once. Use it to either initiate an oauth flow or to refresh an access token. Given it's multiple behaviors, I'm working on deprecating it. In the future, you'll use something like initiateLogin to kick off an oauth flow; if you already have an access token or refresh token, you'll simply start calling methods and the sdk will refresh the access token as appropriate. That's the future though, so I think something like the following will get you unblocked.
const spark = ciscospark.init({
  config: {
    credentials: {
      // the next line is needed to make sure you get a refresh token; I fixed this a few times, but clearly there's still something not quite right with the merge algorithm for the various sources of config data
      clientType: `confidential`,
      oauth: {
        client_id: clientId,
        client_secret: clientSecret,
        redirect_uri: redirectUri,
        scope,
      },
    },
  },
});

// The loaded event fires when all values stored in local storage have been loaded. That said, due to some race conditions, you may still need to rely on the `change:isAuthenticated` event.
spark.once(`loaded`, () => {
  // note that at some point in the future, isAuthenticated will be replaced by canAuthorize
  spark.listentoAndRun(spark, `change:isAuthenticated`, onAuthed)

  function onAuthed() {
    spark.stopListening(spark, `change:isAuthenticated, onAuthed);
    if (spark.isAuthenticated) {
      // your app is ready
    }
    else {
      spark.authorize()
    }
  }
});

How are you deciding what client id/secret to use at runtime? Is it something your appserver is aware of? Could you dynamically generate the javascript file based on the id and secret you need? Unless you're already using a CDN, I think I can point you in the right direction for how to do that reasonably performantly.

Thanks a lot. That's great support ! I will try this tomorrow soon.
Yes our server will be aware of the configuration. It's easier for us to prebuild the assets, and be able to use it on whatever environment we need (staging, production), with dynamic configuration. I think you should really let the possibility in SDK.
I don't like to compare but Google is doing just that (https://developers.google.com/api-client-library/javascript/start/start-js). It's more logical for to know exactly when the #init is done (we decide) and with which configuration.

oh, so you're doing this at runtime more to separate your tooling than because you need multiple client ids in production? In that case, i recommend something like webpack's define plugin or babel's environment variables plugin.
In any case, yes, i agree, we need a solution where this is all doable at runtime. We had an early design requirement that const spark = require(\ciscospark`);` should give back an immediately usable object. That requirement has caused a lot of unexpected pain exactly like what you're dealing with. I've got some ideas on how to deal with this long term, but it's probably going to be a few weeks before I can get anything written beyond what I posted above. That said, I'm happy to continue troubleshooting if the code above doesn't quite work they way I think it does - this part of the api is in flux right now and I'm juggling two different implementations.

Ok so after a bit (in fact a lot) of trying, I think there may be a race condition or another kind of problem.
After an OAuth workflow, I can see that there is data in the localStorage but if I call isAuthenticated it returns false, and if I recall authorize() it raise this exception :
Unhandled promise rejection TypeError: Property 'authorization' must be of type state. Tried to set [object Object]
    at child.set (ampersand-state.js:214)
    at child.set (common.js:455)
    at eval (decorators.js:244)
    at wrapper (_createHybrid.js:87)
    at run (es6.promise.js:87)
    at eval (es6.promise.js:100)
    at MutationObserver.flush (_microtask.js:18)

For more informations after trying with Webpack definePlugin, I can that the data in the localStorage, it now contains the refresh token and its expiration.

@antoinelyset any update from here? I am facing the same situation now.

No sorry. I recommend you to use the "build time" configuration setting.

what do you mean by the build time configuration setting? do you mean to set the access token as an environment variable during build time instead? Is there any existing documentation that you can point me to perhaps?
Thanks!

Yes, you can find more informations here and if you are using Webpack, you should read this section

thanks for the info!


fix(points): dispatch swapSuccess event for cross-chain swaps

Description
For cross-chain swaps, the swapSuccess event wasn't dispatched, and points weren't tracked. This fixes it.
Context: https://valora-app.slack.com/archives/C04B61SJ6DS/p1723220938449899?thread_ts=1723219373.676809&cid=C04B61SJ6DS
Test plan
Tested manually.
Related issues
NA
Backwards compatibility
Y
Network scalability
If a new NetworkId and/or Network are added in the future, the changes in this PR will:

[x] Continue to work without code changes, OR trigger a compilation error (guaranteeing we find it when a new network is added)

@kathaypacific

what do you think about awarding points when the source tx succeeds
yes, this makes sense.


Closes #21. Add Docker

the docker-compose is what I would use for running this. Can't you accept your own PRs?

Can't you accept your own PRs?

I have been working on a Probot version of this at https://github.com/Glavin001/probot-issue-duplicate-detection instead. If I recall correctly, this PR is a work in progress. Feel free to test this branch and see if it works for you.


RFM - httpie

Current Maintainer

[x] I am the maintainer of the package and wish to pass it to someone else;

Checklist

[x] Issue title starts with 'RFM - '

Existing Package Details
Package URL: https://chocolatey.org/packages/httpie
Package source URL: https://github.com/httpie/httpie/tree/master/docs/packaging/windows-chocolatey

@jakubroztocil as it looks like you have been added as the maintainer to the package, I'll mark this request as In Progress.
Please let us know when you have pushed a new version of the package, as well as when that version have been approved.

Looks like new versions have been approved.


no location variable attribute present in netcdf file written by xugrid

in D-EcoImpact, the location attribute is required to map all variables to mesh/node/face locations. This attribute is also described in the conventions (search for Mesh2_depth:location = "node"). However, I cannot judge whether this attribute is required.
I expect xugrid just checks which dimensions are present in the variable. Since there can be only one topology dimension per variable and xugrid is aware which these are, it should be quite simple to map these.
There are two possible solutions:

xugrid automatically writes the location attribute to all data_vars. This should also be adapted after interpolating variables from nodes to faces (when implementing something like this for instance: https://github.com/Deltares/xugrid/issues/127). So actually, upon reading it should be removed, and upon writing it should be added.
D-EcoImpact also uses automatic topology dimension matching like xugrid. In that case please point us to the code that does this.

@hrajagers and @Huite could you also state whether this location attribute is required or not?

I must admit I'm a little confused by the introduction of these attributes: they seem fully redundant, because the dimension already associates data with a specific aspect (node, edge, face) of the topology as you mention.
From glancing the UGRID conventions docs, they are only mentioned for 3D layered mesh topology and onwards.
The UGRID convention pages specifically link to this section: https://ugrid-conventions.github.io/ugrid-conventions/#data-defined-on-unstructured-meshes
I think this gives a bit of background:

The coordinates attribute refers to the variables that contain the latitude and longitude coordinates. For a curvilinear grid these variables will share two spatial dimensions, here nmax and mmax: lat(nmax,mmax) and lon(nmax,mmax). In numerical models the various quantities are often computed at different locations of the mesh: staggered data. The standard CF-conventions don’t offer specific support for this functionality and thus for every stagger location coordinates need to be provided separately: cell center coordinates, corner point coordinates, u-flux point coordinates, and v-flux point coordinates. The underlying topology of the mesh, i.e. how these coordinates (variable definition locations) relate to each other isn’t stored in the file. This shortcoming is to some degree solved by the gridspec proposal by Venkatramani Balaji and Zhi Liang. We introduce here attributes that link to the topological data defined above.

So if you have e.g. staggered data on a structured grid, you need some additional attribute to indicate the location. For UGRID conventions, this isn't required: nodes, edges, and faces are explicitly represented. However, if you want to be "consistent" in some sense, you might argue that if you are specifying the location for a structured topology, that you can also do for it an unstructured topology -- even though the UGRID conventions do not require it at all.
Anyway, my understanding is possibly a bit limited, but this does not really seem part of the UGRID conventions to me. In particular, the redundancy also creates openings for conflicts. Let's say your dimension is mesh2d_nFace, but the location attribute states "node", what are you going to do? The dimension counts heavier here, because netCDF will not allow you to associate dimensions if the array shapes don't match.
I would strongly suggest a single source of truth provided by the dimensions, not these additional attributes for D-Ecoimpact.

Thanks for this elaborate explanation. Could you share a short snippet of xugrid code on how to quickly match a particular variable to node/face/edge? I looked around a bit but could not find what I was looking for real quick. If it is too much effort, I will dive a bit deeper.

Regarding this point:

D-EcoImpact also uses automatic topology dimension matching like xugrid. In that case please point us to the code that does this.

Xugrid infers the dimension based on the connectivity variables and the coordinates (so e.g. edge_node_connectivity should have dimensions (edge_dim, 2); edge_x should have (edge_dim,). Worth nothing that there's already room for conflicts here: somebody can specify edge_dimA on their edge_node_connectivity and edge_dimB on their edge_x and edge_y coordinates.
This function mostly takes care of it:
https://github.com/Deltares/xugrid/blob/ca17eb0d1cf39f36b9c7b50f7b7bd3e4e4e74f7f/xugrid/ugrid/conventions.py#L250
The upper cased "constants" define the expectations in terms of roles:
_COORD_DIMS = {
    "node_coordinates": "node_dimension",
    "edge_coordinates": "edge_dimension",
    "face_coordinates": "face_dimension",
}
...
_CONNECTIVITY_DIMS = {
    "face_node_connectivity": ("face_dimension", None),
    "edge_node_connectivity": ("edge_dimension", 2),
    "face_edge_connectivity": ("face_dimension", None),
    "face_face_connectivity": ("face_dimension", None),
    "edge_face_connectivity": ("edge_dimension", 2),
    "boundary_node_connectivity": ("boundary_edge_dimension", 2),
}

In terms of types:

string means the expected UGRID role
None means: no requirements, arbitrary name or size
int means: arbitrary name, but size should be like that.

(You can argue that mixing names and sizes is a bit (too) pragmatic.)

Ok, this would then be something like this:
from xugrid.ugrid.conventions import _get_connectivity, _get_topology, _get_coordinates, _get_dimensions
file_nc = dfmt.data.fm_curvedbend_map(return_filepath=True)
ds = xr.open_dataset(file_nc)
# ds = xu.data.adh_san_diego(xarray=True) # 1D topology
topologies = _get_topology(ds)
coordinates = _get_coordinates(ds, topologies=topologies)
connectivities = _get_connectivity(ds, topologies=topologies)
dims = _get_dimensions(ds, topologies=topologies, connectivity=connectivities, coordinates=coordinates)
print(dims)

Gives:
{'mesh2d': {'node_dimension': 'mesh2d_nNodes', 'face_dimension': 'mesh2d_nFaces', 'edge_dimension': 'mesh2d_nEdges'}}

We still need to match the variable to the topology name, but that seems easy from here on. It does seem that adding xugrid as a dependency for this is useful, or cherry pick the code a bit.

Ah okay, with your example I get what you're after. I considered something like that in the past, I think you can get all of this in a much nicer form by using the UgridRolesAccessor:
https://deltares.github.io/xugrid/api/xugrid.UgridRolesAccessor.html#xugrid.UgridRolesAccessor
ds = xr.open_dataset(file_nc)
dim = dataset.ugrid_roles["mesh2d"]["node_dimension"]

To clarify, this is an xarray accessor that'll work on Datasets.

I knew there should be an easier way to get to this. Thanks for sharing this.


UI issues in swap view

Noticed some issues in the swap view:

leading zero is not removed by backspace

after a certain number of zeros are appended, there's some sort of string parsing or conversion error, and the amount of btc decreases to zero instead of being converted properly

See video showing issues: https://user-images.githubusercontent.com/11140296/114238106-cac6e380-991f-11eb-9e2e-be9638285ba8.mov

[x] Using most recent release (0.1.0-alpha.2)

[ ] Using yarn

[ ] Using node 12.x

[ ] Using an up-to-date master branch

[ ] Link to stacktrace in a Gist (for bugs)

Expected Behavior

should be able to backspace until the field is empty
rune to btc value should be converted correctly

Current Behavior

leading zero is not removed by backspace
after a certain number of zeros are appended, there's some sort of string parsing or conversion error, and the amount of btc decreases to zero instead of being converted properly

Possible Solution

field/string manipulation errors

Steps to Reproduce (for bugs)
Open rune <-> btc swap view, and try entering a value with many zeros, or backspacing until the field is empty
Context
poor UX taints user trust, and the incorrect conversion is very concerning
Your Environment
Most recent alpha release
https://user-images.githubusercontent.com/11140296/114238106-cac6e380-991f-11eb-9e2e-be9638285ba8.mov

hey @skzv.
related to yours after a certain number of zeros are appended, there's some sort of string parsing or conversion error, and the amount of btc decreases to zero instead of being converted properly seems it caused by big slippage amount (according to your video it's close to 100%) and that's why resulted estimated result is close to zero value. we already have a task [Swap] Slip limit exceeded check #1105 for that.


[Enhancement]: add link between crocswap.com and ambient.finance

Requirements
each of the websites should have a link to the other
crocswap.com
ambient.finance
Figma
No response
Assumptions
No response

We should also discuss how crocodilelabs.io plays into the mix.

Issue is iceboxed pending internal discussions on long-term brand strategy and how to leverage our various web assets.


balloons not shown with putty terminal client

Using the test scripts from the vim help doco and from the various scripts provided under this issue: https://github.com/vim/vim/issues/2481.
The balloons do not show with these scripts when using the putty terminal client, connecting via ssh to a Linux host, with default settings (e.g UTF-8 etc). The balloons also don't show when calling :call balloon_show directly.
I have tried 32 and 64 bit builds, as well as the latest snapshot: (https://www.chiark.greenend.org.uk/~sgtatham/putty/latest.html). Putty is running on  both Windows 10 and Windows 7 machines.
The mouse does work for other activities such as resizing panes, right click popup menus in termdebug etc.
The balloons do show as expected when using the bash.exe WSL terminal client to ssh to the same machine. So it does not appear to be a problem with the vim build, test script or OS.
I have tried various settings of the 'TERM' environment variable (xterm-256color, putty-256color, screen-256color) but it has no effect.
Can anyone else reproduce the behaviour with PuTTY? Are there any logs I can generate to track down the problem? Or even a breakpoint I should set to get more info?
Vim version:
VIM - Vi IMproved 8.1 (2018 May 18, compiled Jul 13 2018 20:04:51)
Included patches: 1-178
Compiled by xx
Huge version without GUI.  Features included (+) or not (-):
+acl               +farsi             +mouse_sgr         -tag_any_white
+arabic            +file_in_path      -mouse_sysmouse    -tcl
+autocmd           +find_in_path      +mouse_urxvt       +termguicolors
-autoservername    +float             +mouse_xterm       +terminal
-balloon_eval      +folding           +multi_byte        +terminfo
+balloon_eval_term -footer            +multi_lang        +termresponse
-browse            +fork()            -mzscheme          +textobjects
++builtin_terms    +gettext           +netbeans_intg     +timers
+byte_offset       -hangul_input      +num64             +title
+channel           +iconv             +packages          -toolbar
+cindent           +insert_expand     +path_extra        +user_commands
-clientserver      +job               -perl              +vartabs
-clipboard         +jumplist          +persistent_undo   +vertsplit
+cmdline_compl     +keymap            +postscript        +virtualedit
+cmdline_hist      +lambda            +printer           +visual
+cmdline_info      +langmap           +profile           +visualextra
+comments          +libcall           -python            +viminfo
+conceal           +linebreak         -python3           +vreplace
+cryptv            +lispindent        +quickfix          +wildignore
+cscope            +listcmds          +reltime           +wildmenu
+cursorbind        +localmap          +rightleft         +windows
+cursorshape       -lua               -ruby              +writebackup
+dialog_con        +menu              +scrollbind        -X11
+diff              +mksession         +signs             -xfontset
+digraphs          +modify_fname      +smartindent       -xim
-dnd               +mouse             +startuptime       -xpm
-ebcdic            -mouseshape        +statusline        -xsmp
+emacs_tags        +mouse_dec         -sun_workshop      -xterm_clipboard
+eval              -mouse_gpm         +syntax            -xterm_save
+ex_extra          -mouse_jsbterm     +tag_binary
+extra_search      +mouse_netterm     +tag_old_static
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
  fall-back for $VIM: "/usr/local/share/vim"
Compilation: gcc -std=gnu99 -c -I. -Iproto -DHAVE_CONFIG_H     -g -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: gcc -std=gnu99   -L/usr/local/lib -Wl,--as-needed -o vim        -lm -ltinfo -lelf -lnsl  -ldl

After some experimentation and discussion with the PuTTY developer, I have confirmed that PuTTY is not sending the mouse movement reporting in response to the escape sequence:
"\e[?1002h\e[?1003h";
Which happens to be the sequence vim is using:
https://github.com/vim/vim/blob/master/src/os_unix.c#L3685
This script, when run in 'extended' mode, can be used to show what mouse events are being reported:
https://pastebin.com/iRSFjdHM
So the bug is most likely an unimplemented feature of PuTTY rather than an issue with vim. Leaving this here for reference.

In case anyone is interested, i've a working POC to implement this feature in PuTTY

I had actually sent a patch to Simon a year ago with a working implementation but it was never merged. When I get a chance I'll send it through but I assume there's not a lot of differences.

Heh. Well I also sent my patch. Let’s see :) anyway I have it working for me now, and the above linked branch can be used if anyone desperately needs balloons 🎈 in terminal.

I've rechecked with Simon Tatham about the PuTTY status on this. In general he agreed that there is something to do in this place and says that he's open for a working patch.
He found one patch - presumaby the one from @shmc - that was sent in January 2019; it is now totally outdated but most important "one the Windows frontend, not in the GTK one".
He also gave @puremourning's one a review and noticed "that's much better (also including changes for both frontends, but unfinished" - so it can't be applied either. I'll try to make some notes in Ben's PR and hope he find's the time to work on those - which would get us very likey to working balloons "out of the box".

Oh neat. I didn’t realise. I can finally stop using a fork :)


Impelimeted TextureView in CustomVideo Renderer

We face a problem with the SurfaceView in provided renderer, so after spending time figure out what can be a better solution to replace the SurfaceView with TextureView.
In this code we have implemented OTTextureView which will be handle by OTTextureRenderer and the top most interface it OTVideoRenderer which basically implement BaseRenderer of open tok and pass event to OTTextureRenderer.
The implementation is improve video rendering performance drastically and we can now also have animation on video view as currently it is TextureView.
The implementation won't be possible without help of "https://github.com/romannurik/muzei" refer the GLTextureView from there and change the code as per our project requirement.
Hope the solution will be helpful for other android developers who are using the OpenTok solution in their app.

Hey, I hope the solution is helpful for your clients like me.

Hey, I hope solution will be helpful for your clients like me. Please provide an update here.

I had problems original opentok's sample on resolutions other than 640x480:

when flipping device orientation and changing renderer dimensions, a strange stripes appear in rendered video
when capturing frame shots getting a deadlock

Tried this implementation hoping it might fix these problems but it doesn't unfortunately

On which device you are using the sample?

For capturing the frame shots you need to move your code of capturing image from view in background thread else it will create a ANR.

Moto Z 2016, but I guess it is device independent, it must be something in opentok's renderer code that breaks it when dimensions change from portrait to landscape.

Please try with applying this
<activity
    android:name=".MainActivity"
    android:configChanges="orientation|screenSize">
</activity>

once hope it will work.

android:configChanges="orientation|screenSize" is already there

There is a way to enable TextureViews instead of SurfaceView inside the SDK.
When building the Session, you need to provide a subclass of https://tokbox.com/developer/sdks/android/reference/com/opentok/android/Session.SessionOptions.html overriding the method useTextureViews to return true
When doing this, publisher.getView() will return a TextureView instead of a GLSurfaceView


Footer Added

Related Issue

Info about the related issue

[x] KSOC Participant

[x] Contributor

Closes: #5
Describe the changes you've made
Added Footer
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Ran on Local Machine
Checklist:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

Screenshots

Original
Updated

Original Screenshot

Updated Screenshot

ated

@RajavJain  Do one thing , remove the term Repo Link :- and keep only the github icon but move that to the extreme right of the footer
I think it will look good

@soujo Okay will be doing that.


Arc

Codecov Report

Merging #82 (cae1924) into master (a384e68) will decrease coverage by 20.93%.
The diff coverage is 77.27%.

@@             Coverage Diff             @@
##           master      #82       +/-   ##
===========================================
- Coverage   95.40%   74.46%   -20.94%
===========================================
  Files           7       15        +8
  Lines         674      928      +254
===========================================
+ Hits          643      691       +48
- Misses         31      237      +206

Impacted Files
Coverage Δ

lib/run.js
50.00% <54.54%> (ø)

bin/arc-tool.js
100.00% <100.00%> (ø)

bin/arc.js
100.00% <100.00%> (ø)

lib/context.js
100.00% <100.00%> (ø)

lib/parser.js
95.59% <100.00%> (+0.23%)
:arrow_up:

cmds/meta/arc-tool.js
100.00% <0.00%> (ø)

lib/interactive.js
3.37% <0.00%> (ø)

cmds/meta/arc.js
100.00% <0.00%> (ø)

lib/completion.js
10.00% <0.00%> (ø)

... and 5 more

Continue to review full report at Codecov.

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


Parent link in QueryModelNode is set wrong in some cases

The parent link in QueryModelNode is set wrong in some cases when we reuse a node without cloning it first.

See https://github.com/eclipse/rdf4j/pull/2180 for example of failures.

Can you point to somewhere where we reuse a node without cloning it first, or perhaps a particular example of a query where optimizing/evaluating results in the parent nodes being out of date? I've briefly gone over how some of our optimizers replace and move nodes, and from what I can tell they're fairly conscientious about the book keeping.


Deadline cutoff for 0.9.6

Will support all requests prior to this in the upcoming release. The proceeding issues filed will be processed for next quarter's version. Thanks for all of the engagement and stay tuned for 0.9.6!

https://github.com/cncf/landscape#current-version


Support for Spigot Library loader

Spigot added a Library loader functionality lately.
https://www.spigotmc.org/threads/spigot-bungeecord-1-17.510208/#post-4184317
What this means is that it should no longer be required that bStats be shaded within the plugin and relocated as Spigot should be able to handle the version control itself.
According to the post, in some circumstances this would even allow different plugins to use a different version of the same library according to the post.
However, I'm not entirely sure in which circumstances that refers to.

I have done the PR to apply this change: #106


feat: remove all dependancies to Bootstrap styles, refactor semantic …

…according to #29

Checklist

[x] Resolve issue/feat. n° #29
[ ] tests are included
[ ] documentation is changed or added
[ ] existing message properties have been updated with new phrases
[ ] new (i18n) language set have been added
[x] improvement to make view conforms with WCAG 2.1 AA

Description of change

Refactor semantics

title h2 is the first element,

Accessibility improvement

:visible-focus is improved
the menu is child of title h2

Add Typography workarounds

Remove bootstrap

Screenshot

Thannks for the review @ChristianMurphy


i did not find the java\executables\Backtest\target\Backtest.jar file,could you correct it?

in the configuration.py
thanks

you need to build the backtest project on java side first! then it will appear there


Add method to get max content length to HttpClient

Looks good

It might be more interesting to be able to override this setting on a per-Request basis.
Incidentally, the behavior of .execute() and .executeAsync() is surprisingly different when this limit is exceeded.


Update request on backbone.marionette

Hi,
Is it ok to make a request if you can update package.json specifically the backbone.marionette dependency from 2.1.0 to 2.4.4?
Or
Are you willing if I can make a fork of your code and I'll make an update on the package.json instead?
Thanks,
Neil

I just submitted a pull request so it's a proper request.


fix: live reloading when running as a module

Description
This PR fixes #654
how to run: python3 -m app --dev

Hey @VishnuSanal 👋
I am unable to understand the changes. Could you add an example on why we need the new parameters?

Hey @VishnuSanal 👋
I am unable to understand the changes. Could you add an example on why we need the new parameters?
And how you'd invoke the changes?

Hey @VishnuSanal ,
Could you please update the PR description with the above??

uh, oh. my previous comment never got sent.

Could you add an example on why we need the new parameters?

I have changed the implementation, PTAL. removed the need for new params.

And how you'd invoke the changes?

running as a module usually, like python3 -m app --dev would do.
please CMIIW.

@VishnuSanal , I just saw your description. That is not what we want to solve for 😄
We want to allow the dev mode to work like robyn -m src --dev
where src
src/
  - __init__.py
  - __main__.py

The --dev flag should rerender based on app changes.

thank you for the review, Sanskar! this currently works as expected in my testing, PTAL & LMK what you think. :)

Hey @VishnuSanal 👋
Thanks for the PR, I have some comments 😄

Hey @VishnuSanal 👋
Thanks for the update. I have some follow up comments

current issues:

doesn't work when using robyn -m src --dev. sys.argv[0] returns the path to the executable. but, from here, it seems like we shouldn't run it like it, but like python3 -m src --dev.

(will edit & update this comment...)

note to self:
the flow of control is like: cli#run => cli#start_dev_server => reloader#setup_reloader => EventHandler#reload => Robyn#start and, we are getting the file object at the last stage. my current implementation starts a new dev server from there.
we need a way to bypass it the first time (inorder to reach robyn#start) and, then kill the previous reloader & start new one from there.


Refactor fabric:cleanup_index_files

Overview
Previous implementation of fabric:cleanup_index_files assembled a regexp by concatenating active signatures.
The approach caused regexp to exceed system limit in the case of huge number of them.
The stacktrace looked like follows:
2> fabric:cleanup_index_files().
** exception error: no match of right hand side value {error,{"regular expression is too large",37555}}
     in function  fabric:cleanup_index_files/1 (src/fabric.erl, line 519)
     in call from fabric:'-cleanup_index_files/0-lc$^0/1-0-'/1 (src/fabric.erl, line 501)
     in call from fabric:'-cleanup_index_files/0-lc$^0/1-0-'/1 (src/fabric.erl, line 501)

This implementation uses different approach. It relies on the fact that view files are named as ${signature}.view. This means that the signature of the file can be easily extracted using filename:basename(FilePath, ".view").
Testing recommendations
make exunit tests=src/couch/test/exunit/fabric_test.exs

Related Issues or Pull Requests
N/A
Checklist

[x] Code is written and works correctly
[x] Changes are covered by tests
[ ] Any new configurable parameters are documented in rel/overlay/etc/default.ini
[ ] A PR for documentation changes has been made in https://github.com/apache/couchdb-documentation

@jaydoane PR is updated


Operator install stays in unknown state due to incorrect YAML in docs

Describe the bug
Following the install for the operator doesn't work. The Subscription has an invalid source value. Currently points to enmasse-operators, but should be pointing to the named CatalogSourceConfig, which is: installed-enmasse-operators
To Reproduce
Following docs @ https://github.com/EnMasseProject/enmasse/blob/e44c8c596d187171e580746d1caba84a3ce13aa9/documentation/modules/proc-installing-using-olm.adoc
Expected behavior
Operator installs successfully following docs

Fixed by #2822


Fix error message for MarkApplicant

displays updated application status in message box upon markapplicant command success

Codecov Report

Merging #288 (167cc6e) into master (e5edaac) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #288      +/-   ##
============================================
- Coverage     70.02%   70.00%   -0.03%
  Complexity      826      826
============================================
  Files           131      131
  Lines          2886     2887       +1
  Branches        375      376       +1
============================================
  Hits           2021     2021
  Misses          738      738
- Partials        127      128       +1

Impacted Files
Coverage Δ

...ddress/logic/commands/applicant/MarkApplicant.java
89.47% <100.00%> (ø)

.../java/seedu/address/model/applicant/Applicant.java
83.63% <0.00%> (-0.77%)
:arrow_down:

...java/seedu/address/logic/commands/job/EditJob.java
94.80% <0.00%> (ø)

Continue to review full report at Codecov.

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


Support tool calling when a structured result is provided

This PR addresses behavior when an agent requests multiple tool calls, one of which is a "final" result call. I believe it closes #161 and follows #179.
In this PR:

agents have a new end_strategy parameter that can be either "early" or "correct". Early means that the presence of a "final" result short-circuits all other tool calls, and they are not made. Passing "correct" will call all tools, even if a final result was provided. correct is the default value.
no matter what the end_strategy is, a ToolResult message is always generated for every tool call. This ensures that messages from the agent call can safely be used as the input to another call.
Ruff (correctly) claimed that the _handle_model_response method was very complicated, so I split it into a few straightforward method calls. If that's not desirable, it's easy to recombine.

Note: this PR does not address the streaming case, as that is more complexing pending conversation with @samuelcolvin .

All good feedback. Will clean up today or tomorrow as time allows.

@samuelcolvin I think this is all cleaned up now


Removes spaces between file and line number

This is a small fix that changes the two places I know of that prints file + linenumber.
From
at /path/File.php: 767
/path/File.php : 767

to
at /path/File.php:767
/path/File.php:767

I find this to be more consistent, easier to read and most importantly it makes it possible to open the file in an editor and have it scroll to the specific line.
E.g in iTerm2 you can Cmd+click the link and have it opened at the correct line in Sublime Text.

@perifer Hey! Thanks for the contribution. But tests are failing, do you mind of take a look?

Here you go.

@perifer Thanks! 👏

@perifer Is this your twitter? https://twitter.com/perifer

You’re welcome! ❤ the package.

@perifer Is this your twitter? https://twitter.com/perifer

Yes.


Unofficial unusable

at ChatGPTUnofficial.cs line 258
  var client = new HttpClient();
        var request = new HttpRequestMessage
        {
            Method = HttpMethod.Post,
            RequestUri = new Uri($"{Config.BaseUrl}/backend-api/conversation"),
            Headers =
            {
                {"Authorization", $"Bearer {AccessToken}" }
            },
            Content = new StringContent(JsonConvert.SerializeObject(requestData))
            {
                Headers =
                {
                    ContentType = new MediaTypeHeaderValue("application/json")
                }
            }
        };
        var response = await client.SendAsync(request);

        response.EnsureSuccessStatusCode();
        var stream = await response.Content.ReadAsStreamAsync();
        var str = await new StreamReader(stream).ReadToEndAsync();  // always an empty string

Currently auth endpoint is stopped working and its patched by OpenAI, we may remove the session-token and use access-token directly for the unofficial method

Hi @PawanOsman, can you explain how the access token should be formated to use it? I am trying to set the field AccessToken from the ChatGptUnofficial to allow for the access token directly, but the ValidateToken method is returning the following error:
The input is not a valid Base-64 string as it contains a non-base 64 character, more than two padding characters, or an illegal character among the padding characters.
in line:
var decodedPayload = Encoding.UTF8.GetString(Convert.FromBase64String(tokenPart));
I wonder how I can use it in the meantime. I can also support you with the modification from session-token to access-token if you explain your strategy.


Misc optimisations and bug fixes

Fixed driver types syntax for DB adapters
URI-Encode Database adapters name upon requests
Fixed field types

@jguittard you need to open a new PR because this contains old commits. Be sure to rebase from zf-apigility-admin-ui/develop and after commit your changes. Thanks!


feat: initial work for trusted bidding signals

Description

Add in trustedBiddingSignals part of the proposal
Additional context (optional)

https://github.com/WICG/turtledove/blob/main/FLEDGE.md#31-fetching-real-time-data-from-a-trusted-server
Checklist
Have you considered all of the following?
Please confirm by checking off each item to ensure you've considered it:

[x] Did you create or update tests?
[x] Did you create or update any documentation?

Fixes:

Codecov Report

Merging #75 (324b715) into main (8e3010e) will decrease coverage by 2.40%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #75      +/-   ##
==========================================
- Coverage   49.03%   46.62%   -2.41%
==========================================
  Files          10       10
  Lines         310      326      +16
  Branches      127      143      +16
==========================================
  Hits          152      152
  Misses         86       86
- Partials       72       88      +16

Flag
Coverage Δ

unittests
46.62% <0.00%> (-2.41%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/frame/interest-group.js
90.90% <0.00%> (ø)

src/frame/utils.js
18.26% <0.00%> (-3.33%)
:arrow_down:

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 8e3010e...324b715. Read the comment docs.


added acronym definitions and helpful links to README

I'm new to microcontrollers and these acronym titles and links were helpful to me.
Help needed with EVT acronym.

I think it means something like "Evaluation/Test" but why the acronym worked its way into the software stack, I have no idea.

I would just not worry about it and treat it as a nonsense acronym.

Just added the approximate definition you suggested.

@dwillmore As someone not  in the industry and just looking at the intro it is hard reading to lookup the numerous acronyms and determine which one is applicable.
Also, I think there is a hobby of interest in this chip since EEVBlog posted about it https://www.eevblog.com/forum/blog/eevblog-1524-the-10-cent-risc-v-processor-ch32v003/

Conflicts now anyway. We might add a few of these here and there in the future.


Remove incorrectly tagged test annotation from test case

The test case is just a simple unit test which was incorrectly tagged with the
UsesTestStreamWithProcessingTime test category annotation. The annotation
prevents the test case from running with Flink or other runners which exclude
this test category.

Run Flink ValidatesRunner


Feat(web): Introduce RadioField component

I have just one objection and that's the inline margin of radio items. Then it's ✅. Thank you! 🙏🏻

Ok, I changed back to margin-left


[FLINK-16252][python] Optimize the progress of the process_outputs in Python UDF

What is the purpose of the change
This pull request will optimize the progress of the process_outputs in Python UDF
Brief change log

Using value_coder to encode_stream directly in ScalarOperation

Verifying this change
This change added tests and can be verified as follows:

It is a perforce improvement without extra function test

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

How does this patch test
Test Code
@udf(input_types=[DataTypes.INT(False)], result_type=DataTypes.INT(False))
def inc(x):
    return x

t_env.register_function("inc", inc)

# num_rows = 100000000
num_rows = 100000
num_columns = 100

select_list = ["inc(c%s)" % i for i in range(num_columns)]
t_env.register_table_sink(
    "sink",
    PrintTableSink(
        ["c%s" % i for i in range(num_columns)],
        [DataTypes.INT(False)] * num_columns))

t_env.from_table_source(MultiRowColumnTableSource(num_rows, num_columns)) \
    .select(','.join(select_list)) \
    .insert_into("sink")

beg_time = time.time()
t_env.execute("perf_test")
print("consume time: " + str(time.time() - beg_time))

Test Results
num rows, num colums |  Consume Time(Before) | Consume Time(After)
1000w,1                         |     86.13                           |   76.47
1000w,10                       |     164.36                         |   148.87
100000000, 1               |     784                              |   739

@HuangXingBo Thanks for the PR. LGTM. The test failure is not related to this PR and has been tracked in https://issues.apache.org/jira/browse/FLINK-13553.


feat: support prometheus metrics

proposal: make opengemini-client-go support prometheus metrics, please do not merge this PR till fully discussed, it is just for discussion here.

Background
To enhance monitoring and observability, we have introduced Prometheus metrics support in the project. These metrics will help us better understand the system's performance and identify potential bottlenecks.
New Features
In this PR, we have added the following six Prometheus metrics:

opengemini_client_query_total Count all queries
opengemini_client_query_counter Count of opengemini queries and classify using measurement
opengemini_client_query_latency Calculate the average of the queries
opengemini_client_write_total Count of opengemini writes
opengemini_client_write_counter Count of opengemini writes and classify using measurement
opengemini_client_write_latency Calculate the average of the writes

Problems

the SDK needs to configure the http port to support Prometheus collection.
how to quickly obtain measurement attributes using the query method.
other indicators that may need to be added.

Impact
This change does not affect existing functionality; it solely adds monitoring metrics.
Ensure that the Prometheus server configuration in the deployment environment is set up to scrape these new metrics.
Prometheus dependency was introduced for this project, and the project dependency is no longer pure.

how to define the metric of writing when using auto-batch point write method？using inner batch writing request's time costing as the metric?

how to define the metric of writing when using auto-batch point write method？using inner batch writing request's time costing as the metric?

specify a tag to distinguish between batch and single write?
whether the batch feature is enabled or not, there is always one request sent. We only need to count the time spent on this request.

how to define the metric of writing when using auto-batch point write method？using inner batch writing request's time costing as the metric?

specify a tag to distinguish between batch and single write? whether the batch feature is enabled or not, there is always one request sent. We only need to count the time spent on this request.

I agree, it seems no valuable to split them.

@shoothzj @Chenxulin97 PTAL,it can work well on my local machine.


Error on npm install command on linux

Node Version: Node: v13.5.0 NPM: 6.14.4
Platform: Linux raspberrypi 4.19.75-v7l+ #1270 SMP Tue Sep 24 18:51:41 BST 2019 armv7l GNU/Linux
Compiler: https://pastebin.com/JtSRAh0g
Module: trying to run npm install (https://pastebin.com/N7Vzt7F9)

Verbose output (from npm or node-gyp):
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:194:23)
gyp ERR! stack     at ChildProcess.emit (events.js:305:20)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:275:12)
gyp ERR! System Linux 4.19.75-v7l+
gyp ERR! command "/usr/bin/node" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/pi/dc/Kuzumi/node_modules/ref
gyp ERR! node -v v13.5.0
gyp ERR! node-gyp -v v5.1.0
gyp ERR! not ok
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"arm"})
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/ref):
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm WARN optional SKIPPING OPTIONAL DEPENDENCY: Exit status 1

You omitted the actual build error but chances are it's an issue with the module you're trying to build, ref, not node-gyp.

its working on the computer but not on my raspberry pi and i need it on the pi

Still omitting the build error. :-)


Bump to 3.3.2

Updated to the latest version since the 3.3.0 distro is not available in the tinkerpop mirrors.

Thanks @nagytech !


Importing multiple jsons

Hi,
I export with 3 different jsons from Jira, I have to do this because the path and area fields of all of them are in different Azure environment. They all have different jqls but I can also aggregate them in a single jql.
3 different json's are exported to JiraExport folder. When I import only one json, it imports all the outputs in JiraExport to Azure. However, the path and area fields are whatever is written in the import json.
I also tried emptying JiraExport each time, this time custom links between issues are not created.
Looking forward to your support, thank you.

You can either map the areapath, similar to https://github.com/solidify/jira-azuredevops-migrator/issues/715
Or an easier fix would be to transform your exported issue .json files manually and set the correct areapath


Adding log facilities

Disclaimer
The work in this PR is heavily inspired by other projects, and tries to combine the best ideas from:

Novoda's simple Log  wrapper - included in NoTils
Jake Wharton's Timber

I just want to use Log.v("whatever") in both my Java and Android modules.

A Log class has been defined as a drop-in replacement of android.util.Log, but tries to keep the Android dependency out of the core idea, in order to have a single entry point for logs for both core and mobile modules. At its core Log is just a static handle to access logging facilities, but allows to inject as many Logger instances as needed and delegate them to actually output the messages. The concern of formatting and deliver the output is delegated to each logger, therefore any behaviour can be potentially added in there as needed. Two Logger implementation have been provided:

OutputStremLogger: it will use a PrintStream to output the given messages/throwables. Note this class has only plain Java dependencies, so it's been defined in the core module.

AndroidLogger: it's just a thin wrapper on top of android.util.Log, supporting log filtering by priority. All the logs with a priority not specified in the class constructor will be skipped. This will allow to fine tune the level of messages in the output according to any valid heuristic.

In order to provide more context to each log, the messages are prefixed with some information about the current stack trace (ie: current thread name, current class, method, line of code). The composition of such information is provided as utility (FancyLogs#addStackTraceInfoTo(String message, int stackDepth)) and can be easily reused across different Logger implementations. When dealing with lack of information (ie ProGuard-ed classes) the formatting utility will just serve the original message without additional info about the stack trace.
By default the Log holds an instance of this class using System.out, but it exposes a public way to override that setup in a static way. As shown in PlaygroundApplication it's easy to override the default behaviour of Log, injecting an AndroidLogger or disabling every log as needed:
        ...
        if (BuildConfig.DEBUG) {
            Log.initWith(AndroidLogger.newInstance(TAG, Log.DEFAULT_STACK_DEPTH, Log.ALL_LEVELS));
        } else {
            Log.disable();
        }
        ....

/cc @blundell @ataulm

:+1: yeah I just wanna Log.d("whatever"); whenever an wherever, if thats the api then :100:

Oh yeah, forgot to mention that with the right formatting you can click through the code from the logs:

@mr-archano ya link all the things! https://github.com/novoda/notils/pull/60


Alwas 256 error time

I am getting always the erorr below, it doesn't matter what parameter I pass to the createView
message = "Error internally in Apple's NSData trying to read from URL 'E1BEB61654A6904B.png'. Error = Error Domain=NSCocoaErrorDomain Code=256 "The operation couldn\U2019t be completed. (Cocoa error 256.)" UserInfo=0x7f8678f29a40 {NSURL=E1BEB61654A6904B.png}";
Maybe it's related to this tsackoverflow post:
http://stackoverflow.com/questions/10106087/nscocoaerrordomain-code-256
Unfornately does your examples not wortk,
it would be great if you would offer an working sample.

there is a working test app : https://github.com/GeraudBourdin/Ti.SvgView-TestApp
did you tryed it ?  Wich version of titanium are you using  ?  could you give some code ans svg that does the error ?

Thanks !
Yes this example works, with your *.svg files.
But I cannot use my *.svg files, I have exported with illustrator,
is there an special format I have to use.

@MichelBahl, it looks like you're trying to load a png file, not svg

It's defintely a svg file, I took a closer look at the file there is some reference to a png.
I don't know a lot about svg standard maybe you can tell me what's wrong with it.
Here you can get a sample file: http://s000.tinyupload.com/index.php?file_id=00235262090135616583

i see

in your file. try to cleanup your svg and i think it will  be fine :)

Thx for your help and thanks for the great module, now it's working fine.

you 'ré welcome :)


Update Pages “the-role-of-digitalization”

Automatically generated by Netlify CMS

👷 Deploy Preview for famous-kulfi-ae21db processing.

Name
Link

🔨 Latest commit
ad05b4ff31a6d5608ed1d3f39cb7895a4481922e

🔍 Latest deploy log
https://app.netlify.com/sites/famous-kulfi-ae21db/deploys/657f2a6c29259e0009303e69


#355 Digital Clock

#355  I have completed the Digital Clock project.
Screenshot of project

https://github.com/Vikash-8090-Yadav/Future.WebNet/assets/103979298/5b88ecb2-0a01-4d05-bbaa-8de7b9df39b6
Recording of project

@shubhisingh184  I have completed #355  Digital Clock .Please have a look😊

@shubhisingh184  Thank you


Bump Apicurio Registry version to 2.4.2.Final

First of all, thanks for this plugin!
Fix: https://github.com/Apicurio/apicurio-registry/issues/3301
The significant changes are because ArtifactType is now mapped to a simple String to allow for user-defined extensions (e.g. we got requests for BigQuery).

Thank you for the contribution to the project. Means a lot!
Left a comment to look out for. I'll be happy to integrate the additional changes if necessary.


nil UI after allowing app to access photos

I'm hitting a fatal error: unexpectedly found nil while unwrapping an Optional value after allowing the app to access photos.

I'm opening it using the example code in the readme from a button tap. Also if I re-run the app, everything works fine.
Any thoughts?

Seems to be working now that I initialized the picker view controller locally in a function instead of as a member variable of my own view controller


fix tensor and index aren't on the same device error

var_indexed = torch.from_numpy(var)[index].float() this line will result in an error with torch==1.13.0+cu117.
because the tensor torch.from_numpy(var) and index index may not on the same device.
Change to var_indexed = torch.from_numpy(var)[index.cpu()].float() resolves the issue.

Hey @zeakey thanks for you contribution. Can you add a screenshot  in the description to proof the correctness of the modification? Then this PR can be merged.

@LeoXing1996 added.


Mouse focus in an RDP window

Whilst RDP'd to a VM via Bastion, and with Windows Explorer open, normally the forwards and backwards mouse buttons would go up and down a folder structure. Even though the mouse focus is in the VM, it doesn't go up / down the folder structure as you'd expect. Is this something that can be added in the future?
Cheers

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

ID: 196cd28d-c7b6-9e3f-ef55-1c85d6c9be5f
Version Independent ID: 6daf260e-526d-c5c5-50ca-c7a85e17c78a
Content: Create an Azure Bastion host
Content Source: articles/bastion/bastion-create-host-portal.md
Service: bastion
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@d83194 Thanks for the question! We are investigating and will update you shortly.

@d83194 can you elaborate what you mean by "forward and backward mouse buttons"?

Just the side buttons on the mouse, the forward and backward button. In Windows Explorer they traverse forward and back, up and down the folder tree that you've already gone through. This doesn't happen though when you're connected via Bastion. Cheers

@d83194 If you would like us to add function to those buttons in the future, please leave your feedback at the Azure Networking's Feedback.Azure.Com page.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


False Flags on User Home Directories

In the misc.py module, the functions "check_all_users_home_directory", "check_users_home_directory_permissions", and "check_users_own_their_home" have inconsistent script checks with what is available in "CIS_Ubuntu_Linux_16.04_LTS_Benchmark_v1.1.0.pdf", in that they do not skip the users that have a shell of "/bin/false" or "/user/sbin/nologin". This is causing the user "nobody" to be flagged as failing because it has a UID of 65534, but should not apply because it is non-login. The max_system_uid should not be increased to account for this, because it would then skip users that should have home directories and be validated as such.
Here is a snippet of the custom Nova profile I am using to conform with the spec update for Ubuntu 16.04 that calls these functions:
misc:
  check_home_directory:
    data:
      Ubuntu-16.04:
        tag: CIS-6.2.7
        function: check_all_users_home_directory
        kwargs:
            max_system_uid: 999
    description: Ensure all users' home directories exist
  home_directory_permission:
    data:
      Ubuntu-16.04:
        tag: CIS-6.2.8
        function: check_users_home_directory_permissions
    description: 'Ensure users home directories permissions are 750 or more restrictive'
  check_owner_home:
    data:
      Ubuntu-16.04:
        tag: CIS-6.2.9
        function: check_users_own_their_home
        kwargs:
            max_system_uid: 999
    description: Ensure users own their home directories

@anuragpaliwal80 @vishsriv I think this is your code. Would you mind taking a look when you get a moment?

I can confirm this on Ubuntu 16.04. Running the CIS provided test script returns nothing while check_home_directory fails.


[Service Bus] Ship updates for v1 of Service Bus

Creating this epic to track the work we wont be taking up for GA, but will consider for the version after GA

@ramya-rao-a what to do with this epic?

This one is to track the Track 1 issues

Closing this as we no longer have any planned updates to make for v1 of @azure/service-bus package


help_pages: Highlight headings targeted by URL fragments.

This PR aims to make the help documentation more user-friendly by highlighting links targeted by URL fragments. It also ensures that targeted headings are not obscured by the floating menu bar at the top of the help pages.
One caveat: While the approach here works fine when loading a new docs page whose URL includes a fragment, there appears to be something about simplebar that clears out the :target reference. If you click a heading link on a help page, for example, you might momentarily see the highlighted style appear before it disappears.
From spot-checking, that seems to be the only way that link targets are referenced from help pages. There isn't, for example, any on-page navigation to move through longer pages. In either case, the primary motivation for this change is help links coming from external references, which this PR addresses.
See CZO discussion.

Screenshots and screen captures:
Highlighted and properly top-aligned anchor target on "Customize organization settings":

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Updated PR to move CSS comments to line above.

Ideally we’d switch the main scrollbar from a SimpleBar to the normal browser scrollbar on <html> (with the navigation as position: fixed or position: sticky). This would also fix various accessibility problems—e.g. you currently can’t scroll the help pages with the keyboard until first clicking on the scrolling pane to focus it—and naturally prevent the introduction of new ones.

Ideally we’d switch the main scrollbar from a SimpleBar to the normal browser scrollbar on <html> (with the navigation as position: fixed or position: sticky). This would also fix various accessibility problems—e.g. you currently can’t scroll the help pages with the keyboard until first clicking on the scrolling pane to focus it—and naturally prevent the introduction of new ones.

I'd be 100% on board with this. I'd even want to go further and set up the layout with CSS Grid, and see whether it's worth having the navigation fixed or sticky at all, with perhaps an unobtrusive "Return to top" link or icon floating in the navigation column.

Updated to reflect some color and rounded-corner adjustments from Vlad. Changes in this PR will now render onscreen like this:

One more fix: I've added an additional commit that preserves the scroll target, so that on-page URL fragments are highlighted and scrolled to as expected.

Looks good to me! I tested for a 2-line heading in a narrow window as well.

This works for me; we will almost certainly want to remember to redo this color when we redo the visual design for this page more broadly, but that's OK.


Facebook failed with error 500

Error message: No results for this query.
Query string: action=display&bridge=Facebook&u=118603330411&media_type=all&format=Atom

I cannot reproduce on my instance. Are you sure the version that you are using is up to date ?

Seems to be working for me as well.
Possibly related to #994

https://github.com/RSS-Bridge/rss-bridge/issues/2047


[ffmpeg] Build error

Copy issue body from /vcpkg_installed/vcpkg/issue_body.md
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:32 (message):
  Failed to download file with error: 1
  If you are using a proxy, please check your proxy setting. Possible causes are:

  1. You are actually using an HTTP proxy, but setting HTTPS_PROXY variable
     to `https://address:port`. This is not correct, because `https://` prefix
     claims the proxy is an HTTPS proxy, while your proxy (v2ray, shadowsocksr
     , etc..) is an HTTP proxy. Try setting `http://address:port` to both
     HTTP_PROXY and HTTPS_PROXY instead.

  2. If you are using Windows, vcpkg will automatically use your Windows IE Proxy Settings
     set by your proxy software. See https://github.com/microsoft/vcpkg-tool/pull/77
     The value set by your proxy might be wrong, or have same `https://` prefix issue.

  3. Your proxy's remote server is out of service.

  If you've tried directly download the link, and believe this is not a temporary
  download server failure, please submit an issue at https://github.com/Microsoft/vcpkg/issues
  to report this upstream download server failure.

Call Stack (most recent call first):
scripts/cmake/vcpkg_download_distfile.cmake:270 (z_vcpkg_download_distfile_show_proxy_and_fail)
scripts/cmake/vcpkg_acquire_msys.cmake:26 (vcpkg_download_distfile)
scripts/cmake/vcpkg_acquire_msys.cmake:67 (z_vcpkg_acquire_msys_download_package)
scripts/cmake/vcpkg_acquire_msys.cmake:216 (z_vcpkg_acquire_msys_declare_package)
C:/Users/tzuk9/AppData/Local/vcpkg/registries/git-trees/14577b12f56accfce4428caf17e4b47542f365ec/portfile.cmake:48 (vcpkg_acquire_msys)
scripts/ports.cmake:147 (include)
error: building ffmpeg:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest port files with git pull and vcpkg update.
Then check for known issues at:
https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+ffmpeg
You can submit a new issue at:
https://github.com/microsoft/vcpkg/issues/new?title=[ffmpeg]+Build+error&body=Copy+issue+body+from+C%3A%2FUsers%2Ftzuk9%2FDocuments%2FsimulatorMapping%2Fvcpkg_installed%2Fvcpkg%2Fissue_body.md

Please check if your firewall isolates vcpkg.


[FR] Rename DisplayName -> ModelName

It also adds links to composed model sample

/check-enforcer override


Fix for run.sh

Fix for run.sh
Fixes

Currently run.sh requires an auth file as the first parameter for pokecli.py to run, which forces everyone to use auth files.
Swapped config and auth position so old command of ./run.sh ./configs/myconfig.json works again.

Note: Those running ./run.sh ./configs/myauth.json ./configs/configalexnew.json need to swap the parameters back.

:+1:


pushDataLayerEvent not sending any data to GTM

Environment
React 16.3.1
React Native 0.55.3
iOS Simulator
Steps to Reproduce

Create a new Tag in Google Tag Manager (UA Event).
Create a trigger in GTM that looks for a custom event named "DEBUG" and fires the tag in #1.
Run the code at the bottom of the issue.

Expected Behavior
Within Google Analytics, an event should show up in the real time reports.
Actual Behavior
No event in real time reports.
Several things were done to test this:

On a website running the exact same instance of GTM, the console was opened and a dataLayer event pushed:
dataLayer.push({"event": "DEBUG", "message": "test from website"});

The event showed up nearly instantly in the Google Analytics interface.

Using the GoogleAnalyticsTracker code right next to the GTM code also pushed a successful event (see it in the code sample below). This "proves" that the simulator has access to the internet.

Code sample/demo project
import {
    GoogleAnalyticsTracker,
    GoogleTagManager
} from "react-native-google-analytics-bridge";

const tracker = new GoogleAnalyticsTracker("UA-XXXXX-X");

GoogleTagManager.openContainerWithId("GTM-XXXXXXX")
    .then(() => {
        GoogleTagManager.setVerboseLoggingEnabled(true);
    })
    .then(() => {
        return GoogleTagManager.pushDataLayerEvent({
            event: "DEBUG",
            message: "test from pushDataLayerEvent()"
        });
    })
    .then(success => console.log(success))
    .catch(err => {
        console.log(err);
    });

tracker.trackEvent("DEBUG", "test from tracker()");

Hi! Could you provide logs from the XCode output window when doing this?
Also, I don't have the same setup with a website, so wondering if maybe the container type matters. I.e. when you create the container:

One thing to test would be creating an "iOS" container and check if that matters. It does matter on the Google Analytics side which type of GA "property" you create: https://github.com/idehub/react-native-google-analytics-bridge/wiki/Creating-a-Google-Analytics-property-for-mobile-analytics

@cbrevik
I'm having the same problem. If I use the GoogleAnalyticsTracker, I can push events and see them in GA, but if I try using the GoogleTagManager to push enhanced ecommerce events, then it doesn't seem to work. It's also my first time trying to implement GTM, so I'm not really sure if I've configured the container or the tags incorrectly.
As you can see in my code, I'm simply trying to push an event taken from the GTM enhanced ecommerce docs:
GoogleTagManager.openContainerWithId("GTM-T9VS75T")
  .then(() => {
    return GoogleTagManager.pushDataLayerEvent({
      'event': 'productClick',
      'ecommerce': {
        'currencyCode': 'EUR',                       // Local currency is optional.
        'impressions': [
         {
           'name': 'Triblend Android T-Shirt',       // Name or ID is required.
           'id': '12345',
           'price': '15.25',
           'brand': 'Google',
           'category': 'Apparel',
           'variant': 'Gray',
           'list': 'Search Results',
           'position': 1
         },
         {
           'name': 'Donut Friday Scented T-Shirt',
           'id': '67890',
           'price': '33.75',
           'brand': 'Google',
           'category': 'Apparel',
           'variant': 'Black',
           'list': 'Search Results',
           'position': 2
         }]
      }
    })
  })
  .then(success => {
    console.log(success);
  })
  .catch(err => {
    console.log(err);
  });

As for the tag configuration, here's a screenshot in case something's wrong on my end:

The container is Android (Legacy). Tag type is GA - Universal Analytics, and enhanced ecommerce features are enabled alongside the data layer, and it should be triggered by any event. So assuming everything is correct, I'm not sure why calls to pushDataLayerEvent() don't seem to appear in my GA.

@clundberg1 The same thing is happening with me. Did you find any work around?

It would be cool if you could do a GoogleTagManager.setVerboseLoggingEnabled(true) and check adb logcat in terminal (if this is Android we're talking about).
This will narrow down if the problem is with the library, or a problem with GTM <-> GA. If the logs says that the event was sent, then I'm leaning towards the latter.

@cbrevik
I tried to investigate in the library, what i found is if pushDataLayerEvent is triggered then in native side it checks if everything is good it sends a promise back through the bridge. See the below code snippet for reference.
if (mContainerHolder != null && isValidMapToPushEvent(dictionary)) {
          getDataLayer().push(ConvertReadableToMap.getMap(dictionary));
          promise.resolve(true);
}

Please see below logs, it seems suspicious.
10-04 11:41:29.682 12431 17372 W GoogleTagManager: Failed to load the container. No default container resource found with the resource ID -1
10-04 11:41:29.683 12431 17372 V GoogleTagManager: loadAfterDelay: containerId=GTM-XXXXXXX delay=0
10-04 11:41:29.684 12431 17372 D GoogleTagManager: Setting CTFE URL path: /r?id=GTM-XXXXXXX&gtm_auth=XXXXXXX&gtm_preview=5
10-04 11:41:29.684 12431 17372 D GoogleTagManager: Setting previous container version:
10-04 11:41:29.687 12431 17389 V GoogleTagManager: Start loading resource from network ...

It would be cool if you could do a GoogleTagManager.setVerboseLoggingEnabled(true) and check adb logcat in terminal (if this is Android we're talking about).
This will narrow down if the problem is with the library, or a problem with GTM <-> GA. If the logs says that the event was sent, then I'm leaning towards the latter.

Is something wrong?

10-18 17:10:53.933 12969 13025 I GoogleTagManager: Obtained fresh AdvertisingId info from GmsCore.
10-18 17:10:53.937 12969 13011 E GoogleTagManager: Invalid macro: _gtm.loadEventEnabled
10-18 17:10:53.939 12969 13028 V GoogleTagManager: Attempting to load resource from disk
10-18 17:10:53.939 12969 13028 D GoogleTagManager: Failed to find the resource in the disk
10-18 17:10:53.940 12969 13028 V GoogleTagManager: loadAfterDelay: containerId=GTM-T9VS75T delay=0
10-18 17:10:53.940 12969 13028 D GoogleTagManager: Setting previous container version:
10-18 17:10:53.942 12969 13029 V GoogleTagManager: Start loading resource from network ...

@codesinghanoop
I tried uninstalling the app from the emulator and running again and it didn't work.

@clundberg1 After reading the issue description i think you are using the same container for web and mobile. Please add .then in dataLayer.push({"event": "DEBUG", "message": "test from website"}); to check that promise is resolved and also add .catch to see if there is a error.

@codesinghanoop
If you look at the code I posted, there's then() and catch(). The promise resolves successfully and prints true (no error).

I also have the same issue, I've pushed several events but I don't see them on GA. My container for iOS/Android is Legacy. The promise return 'success' however.
GoogleTagManager.openContainerWithId(gtmContainerId)
  .then(() =>
       GoogleTagManager.pushDataLayerEvent({
         event: "open_app",
       })
         .then(success => console.log(success))
         .catch(err => {
           console.log('GTM err', err);
         })
     );

@clundberg1 do you find any solutions to this issue?
@codesinghanoop if the event is pushed successfully, then you can see the events on GA right away?


self is not accessed

https://github.com/microsoft/pylance-release/issues/272#issuecomment-678511791
Could you please do something with this problem?

You can remove hint from enabledDiagnosticLevels in your settings.json:
    "errorLens.enabledDiagnosticLevels": ["error", "warning", "info"]


gpio: Define slew rate (output), drive strength and hysteresis (input) in the API

Right now, the gpio API is fairly basic, you can specify the pull for input but that's pretty much it.
There are more parameters, and they would be useful in a few contexts:

Slew rate is the duration for an output GPIO pin to go from one level to another. It is usually specified in nanosecond. Fast slew rate can generate EMI interference due to overshoot and can interfere with neighbouring pads (like SDCard and DRAM). A slow slew rate limits the frequency at which communication can happen. It is essentially a low-pass filter.
Drive strength is the current that a GPIO will try to drive. It is usually specified in mA. Specifying a too large value at the system level can cause the CPU to drive too much total current, which can burn it. Specifying a too low value can burn the GPIO itself. It is important to note that this doesn't really limit the current the GPIO will produce, it is only specifying the guarantee that the voltage will be stable at this current draw level. Low value reduces overshooting.
Hysteresis is wether the cross over point for input reading is 50% or larger via a Schmitt trigger.

For particular hardware:

bcm283x:

https://matt.ucc.asn.au/mirror/electron/GPIO-Pads-Control2.pdf documents the configuration the bcm283x driver should implement.
https://raspberrypi.stackexchange.com/a/8320 and http://www.mosaic-industries.com/embedded-systems/microcontroller-projects/raspberry-pi/gpio-pin-electrical-specifications have a great deal of information.
In practice, the values can only be set per GPIO groups; 0-27, 28-45, 46-53, not per pin.

ftdi:

http://www.ftdichip.com/Support/Documents/DataSheets/ICs/DS_FT232H.pdf partially document the one supported by this device.
See eepromFt232h in https://github.com/periph/extra/blob/master/hostextra/d2xx/d2xx.go
ft232h slew, hysteresis and drive current can only be set per bus side (CBus and DBus), not per pin.
ft232r only supports a generic "high current" value. What to do about this?

Options:

I think drive strength and slew can be queried at least in a generic way.
Setting, because of the fact it affect GPIO groups, should probably be device specific (?)
Hysteresis is generally meant to be left on, so I'm ambivalent about this. It is also generally only set on a whole bus basis.

Finally fixed after manual testing. Will write a blog post about it.

Tweeted about it; https://twitter.com/periphio/status/995757845059723264


added names to register

Addresses issue:
https://github.com/microservices-demo/microservices-demo/issues/229
Added first and last name because it is required for displaying username on sock shop.

Coverage decreased (-0.05%) to 25.158% when pulling 841da60740fac024feebf669c4186618108ebd95 on registernames into e3ad75a57428db163468622a8ddbd892e297406b on master.


Bug: Nordic characters are ignored

What are the steps to reproduce this issue?

Use Orval to generate typescript files with the following schema:

openapi: '3.0.0'
info:
  version: 1.0.0
  title: Swagger ÆrligØnskeÅpen
  license:
    name: MIT
paths:
  /ærligØnskeÅpen:
    get:
      summary: List all ærlig-ønske-åpen
      operationId: listÆrligØnskeÅpen
      responses:
        '200':
          description: A paged array of ærlig-ønske-åpen
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/ÆrligØnskeåpen'
components:
  schemas:
    ÆrligØnskeåpen:
      type: object
      required:
        - ærlig
        - ønske
        - åpen
      properties:
        ærlig:
          type: string
        ønske:
          type: string
        åpen:
          type: string

What happens?
It generates the file rligNskePen.ts:
export interface RligNskePen {
  ærlig: string;
  åpen: string;
  ønske: string;
}

What were you expecting to happen?
It should generate the file ærligØnskeåpen.ts:
export interface ÆrligØnskeåpen {
  ærlig: string;
  åpen: string;
  ønske: string;
}

Any logs, error output, etc?
Nope.
Any other comments?
I've been testing with different versions of Orval, and the issue starts with 6.18.0.
What versions are you using?
Operating System: MacOs 13.6.3
Package Version: 6.23.0
Browser Version: Chrome 119.0.6045.123

My gut feeling is this is the ticket that broke it: https://github.com/anymaniax/orval/pull/909

I have a PR comming where I belive I've fixed the issues. My testing found that #951 was probably the cause.


Template prim sum's return type on eigen matrices (code cleanup #987)

Summary
This fixes a little bug that @bbbales2 noted in #987: prim's implementation of sum for Eigen::Matrix was always returning double, rather than its argument's (templated) element type. Per our discussion in the issue, I've also replaced sum's implementation for std::vector with the standard library algorithm std::accumulate.
Tests
One new test that makes sure that sum has the right return type for various argument types. Any other return cases I should check?
Side Effects
None.
Checklist

[x] #987

[ ] Copyright holder: (fill in copyright holder information)
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[ ] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

Thanks @cqfd!

Oh whoops, @cqfd, could you add a copyright holder to the pull req.?

Whoops—just added it (that look right?).


Google Plus example?

Is there an example for making a post on your public Google Plus feed using this module?

@chovy Were you able to figure out how to do this?

19 days later...I'm still trying to figure this out :(

This feature is available only to Google App customers through the Google+ Domain API
https://developers.google.com/+/domains/

@bantini Thank you for replying.
Ahh so this library can't be used for that? We have to use the API directly? I'm trying to figure out how the likes of buffer.com or hootsuite schedule posts to g+ profiles and pages.


Fix #2: link references, footnotes

For reference, pulldown_cmark with the input
[link 1](https://link)

[link 2][ref]

[link 3][]

<https://website>

[^1]

<EMAIL_ADDRESS>
[ref]: https://example.corn
[link 3]: https://collapsed

[^1]: footty

Produces this event stream
Start(Paragraph)
    Start(Link(Inline, Borrowed("https://link"), Borrowed("")))
        Text(Borrowed("link 1"))
        End(Link(Inline, Borrowed("https://link"), Borrowed("")))
    End(Paragraph)
Start(Paragraph)
    Start(Link(Reference, Borrowed("https://example.corn"), Borrowed("")))
        Text(Borrowed("link 2"))
        End(Link(Reference, Borrowed("https://example.corn"), Borrowed("")))
    End(Paragraph)
Start(Paragraph)
    Start(Link(Collapsed, Borrowed("https://collapsed"), Borrowed("")))
        Text(Borrowed("link 3"))
        End(Link(Collapsed, Borrowed("https://collapsed"), Borrowed("")))
    End(Paragraph)
Start(Paragraph)
    Start(Link(Autolink, Borrowed("https://website"), Borrowed("")))
        Text(Borrowed("https://website"))
        End(Link(Autolink, Borrowed("https://website"), Borrowed("")))
    End(Paragraph)
Start(Paragraph)
    FootnoteReference(Borrowed("1"))
    End(Paragraph)
Start(Paragraph)
    Start(Link(Email, Borrowed("abcd@example.com"), Borrowed("")))
        Text(Borrowed("abcd@example.com"))
        End(Link(Email, Borrowed("abcd@example.com"), Borrowed("")))
    End(Paragraph)
Start(FootnoteDefinition(Borrowed("1")))
    Start(Paragraph)
        Text(Borrowed("footty"))
        End(Paragraph)
    End(FootnoteDefinition(Borrowed("1")))

ref is lost for link references
Upstream issue: https://github.com/raphlinus/pulldown-cmark/issues/434

Ended up going with a different markdown library (for now)


Futher Doc improvements

From @deltaidea on #254:

[ ] Performance tips: avoiding ambiguity, using moo. (#81, #238, #243, #250)
[x] Error handling:

[x] List of possible exceptions. I imagine it's short, but I'm not sure about it myself.
[x] Empty parser.results. (#253)

[ ] Debugging tips: inspecting that the right tokens are parsed, using nearley-test. (#253)
[x] IDE support: syntax highlighting is supported at least in VS Code, probably in other editors too.
[x] Go through the source code and update references to the readme.

@hardmath123: Introduction should also describe why people like nearley: it's user-friendly, easy to get started, works on both the browser and node, etc.
@hardmath123: I think "parse complex data structures" is not quite what we wanted to say, even in the original README. This is a good opportunity to find better wording. Complex languages, perhaps?
@tjvr: Document save() / restore()

@Hardmath123

Is .save() and .restore() even a thing anymore?

Is .save() and .restore() even a thing anymore?

Sure! rewind() is the thing that's deprecated.

Okay, if you get a chance could you document .save() and .restore()? I think it should be a new section in the "using a parser" page. Importantly, it would be good to have some motivating examples for why you would use this feature.

I wonder if we should be pushing lexers from the start, and move the syntax/explanation of scannerless parsing to a dedicated section...

No, I think it's important for people to be able to get a parser working as fast as possible. We can mention lexers earlier, but the "getting started" and "home" pages should be scannerless, I think.

Are there good examples for save() and restore() anywhere outside of this repo, or maybe even here?

@diegoperini That depends; what are you looking to do? 🙂
Sent with GitHawk

I want to be able to recover from an invalid state to ignore error causing lines during the parse. I see no other way to detect multiple errors in a single source file. If there is a way, maybe save()/restore() is unnecessary, I really don't know. :)

Perhaps looking at the tests would help? :man_shrugging: I'm sorry it's not documented better.
Save/restore was added in this commit:
https://github.com/kach/nearley/commit/b64ffe6bc3c0edef04defa7583c4c266f1bff069
The idea is that you call save() at a point that you want to be able to restore to later. If you want to be able to restore to any token, then you should use the built-in keepHistory option.
You call restore() with the object that was returned from save(), or alternatively use parser.table[index] to restore to the parse state "at" index N (I think it's the parse state after the Nth token).
The built-in rewind(index) method (which we might have removed since then) does this for you.
Hopefully that helps? :-)

It's good enough. Thank you so much. Maybe after I get a grasp of it, I can write the docs for you as my thanks.

@tjvr I saw something like // TODO: moo save/restore in the parser tests. Does this mean we cannot use save() and restore() if we use moo as a lexer?

@diegoperini You should absolutely be able to use save() and restore() with Moo.


Upload models again (error 404)

Seems like the default mediafire URLs for the pretrained models are down. Could you upload them somewhere else and update the code? Thank you

I just saw that the problem is up to 0.8.4, since from 0.8.5, the link points to a github URL which works. I suggest removing prior versions from pypi due to deprecation.

I just saw that the problem is up to 0.8.4, since from 0.8.5, the link points to a github URL which works. I suggest removing prior versions from pypi due to deprecation.

Thanks for raising this! I've yanked all package versions before v0.8.5.  The versions will still be available but installation will trigger the message: "The URLs for weights have changed. Please upgrade."

Thanks for raising this! I've yanked all package versions before v0.8.5.  The versions will still be available but installation will trigger the message: "The URLs for weights have changed. Please upgrade."


refactor: Refactor model configuring for CmsDbContext

Overview
This PR cleans up the configuration of the CmsDbContext. It moves all the different entity type configurations out of the OnModelConfiguring method, by creating an IEntityTypeConfiguration for each one. We already had some of these, particularly the more complex configurations, but it was inconsistent. It's now consistent, and the CmsDbContext looks much cleaner.
Changes
Minor

Move all entity type configuration in CmsDbContext to their own concrete classes
Other very minor changes, mostly things like ordering methods by access level, refactoring the GetService calls in the CmsDbContext, etc.

How to review the PR

Run the web app
Ensure nothing is broken
Success!

Would be good to test Page content changes from Contentful, i.e. CRUD operations on a content that is a Page content type. The change I made is minor and should not impact anything and seems to work fine but 🤷
Checklist
Delete any rows that do not apply to the PR.

[x] Title uses Angular commit convention
[x] PR targets development branch
[x] Unit tests have been added/updated
[x] Documentation has been updated where relevant

Nicely done
... tho your fix to the button with entry ref isn't showing up on this branch 🤔 Maybe the last PR broke it and I missed it

Seems to be there to me? I did remove a method from CmsDbContext, which was part of my original attempt at the fix, but ended up being unused. So I hope that's what you're talking about since I'm gonna merge this now!


Asset reference tag does nothing

Well, maybe i am doing something wrong. But I noticed that when i put asset reference tag into markdown text, i end with something like that: {asset:5}.
When i use parseRefs filter in twig template, this reference tag is not compiled to anything. When i manually change it to {asset:5:url}, url of asset is outputted, so I can tell that refence tags are working.
Ideally, it would be if <img> tag was outputted. I tried using {asset:5:getImg()}, but it didnt worked. Maybe doxter should output reference tags like this?
![]({asset:5:url})

Have you tried the image shortcode?
Be sure to turn on shortcode parsing in the plugin settings.

Can't reproduce this, sorry!


Discusion: Migrate from JS to TS

we use a lot of custom JavaScript web_src/js, witch we could migrate to TypeScript.
any thoughts on this topic?

cc @silverwind

If we're talking about typedefs for API responses; that might be benefitial, those could be generated from the swagger.json..
But for everything else? The current code is a hodgepodge of various imperative APIs with mostly ad-hoc types. I don't see a benefit really, it will mostly slow down the build..
otoh, if we're talking long term about a frontend rewrite, Typescript would be a good option

Gonna have to say no thanks because:

No obvious advantage or win for gitea?
More complex, slower build process.
Increased barrier for development.
ECMAScript has been evolving so quickly lately that it will probably get types and supersede TS in the near future anyway. The true "futureproof" choice is to continue evolving with the latest ECMAScript spec.

How to handle the code which using vue is also a problem ...

A bit off topic but I feel as if vue will be replaced with the ability to send POST data + show returned content as simple tag attributes as part of the upcoming HTML standard, which is the true "futureproof" way. See https://htmx.org as an example of this.

I have never been fun of TS, ecmascript has evolved quite much so I don't see much use for TS to be honest

I think the more urgent thing is maintainable for frontend codes. We may should rework the frontend but have to consider below things:

[ ] The new frontend should not depend on a backend technology like Go template, so that frontend and backend developers could cooperate easier.
[ ] The new frontend should be friendly with SEO so that repositories on public Gitea site could be found by search engine.
[ ] The new frontend could be introduced step by step because it's difficult to replace all of the frontend codes once.

I also dislike TypeScript because it has no runtime support anywhere and last I checked the compilation is a major hindrance loss when developing because it easily doubles the build time. I'd rather wait to have official types support in ECMAScript.

The new frontend should not depend on a backend technology like Go template, so that frontend and backend developers could cooperate easier.

Yes, we need to extract the JS from the templates as well as nuke that conglomerate of gotemplate and vue that is used on the frontpage repo list. Some inline scripts are still necessary for performance-critical cases, but maybe they could also be extracted to files.

If the separate build step is not wanted JSDoc  https://www.typescriptlang.org/docs/handbook/jsdoc-supported-types.html may be a possibility (but keep in mind that it has some limitations (which don't matter here I think)).


Errors if key an underscore in it

If you attempt to do like
try unboxer.unbox(keyPath: "routes.overview_polyline.points")
It'll give error of
[UnboxError] An error occured while unboxing path "routes.overview_polyline.points": The key "overview_polyline" is missing.
yet in the JSON, the key does exist and is spelled correctly. Looks to error on the underscore.

Problem looked to be that route is an array and you can't key path to an item in an array since it doesn't know which array item to look at.

So what's the solution to that problem?


Test result audit response update

This PR is created to add feature test result counts and type in test result audit response

Can you add some test cases for this?

tests added


add configuration command

considering a command that would, ideally

stop the daemon
open the config file in $EDITOR
start the daemon after the user quits the editor

i'm not 100% sold if this is needed, or if we should just document how to edit the file and restart dlite and be happy

I would love that, especially if the init script which installs docker is added (see https://github.com/nlf/dhyve-os/issues/14)

The README says Next, edit your config file for DLite via dlite config. Set the value of the "extra" option to "--bip=<IP_ADDRESS>/24 --dns=<IP_ADDRESS>" and exit your editor..
However, the master branch does not contain dlite config.

yes it does.. it only exists in the master branch (which corresponds to the latest 2.0.0-beta pre-release). if you installed via homebrew you won't have it because 2.0.0 isn't finished yet, and as such isn't in homebrew


Add Notifications

We should use notifications to alert users when something like that happens. I suggest implementing an aggressive setting first. When we see which notifications are too much, we can reduce them.
Any hint/open/connect/message/close received while the application is not shown should increment a counter (one per types hint/open/connect/connectionMessage/close).
After N seconds without new messages, a notification, such as: "X new Match(es), Y chats requests, Z chats accpted W chats closed"
should be shown.
We need to filter hints that do not create new connections and hints that point to own needs.
Asking for permission should be done after posting the first need in each session.

docs: https://developer.mozilla.org/en-US/docs/Web/API/notification

not sure if we can implement this without putting the ws handling (messaging-agent.js) and rdf store in a webworker first so we can use these in a separate somewhat progressive webapp approach which can also enable notifications, so moving these things into a separate part that can run in its own thread is probably a necessary step before figuring out the notification system

@peacememories' research result: notifications are available on Android only using service workers, on IOS not at all. We're waiting with this.

closes #2284
closes #2658

we already have notifications... we might need a refactoring/adaptation to enhance the UX but for now this item is done


rm pkg from DESCR. and linux pkg from travis (will fail)

Probably have to delete travis cache first...


NullPointer in VersionSet.numberOfFilesInLevel()

I write data to levelDB in multi-threads, the exception below is thrown. why?
java.lang.NullPointerException
at org.iq80.leveldb.impl.VersionSet.numberOfFilesInLevel(VersionSet.java:227) ~[leveldb-0.7.jar:?]
at org.iq80.leveldb.impl.DbImpl.makeRoomForWrite(DbImpl.java:809) ~[leveldb-0.7.jar:?]
at org.iq80.leveldb.impl.DbImpl.writeInternal(DbImpl.java:676) ~[leveldb-0.7.jar:?]
at org.iq80.leveldb.impl.DbImpl.write(DbImpl.java:665) ~[leveldb-0.7.jar:?]

I did a manual review of this code and I don't see any problems with the locking code, so without a test case this to reproduce this, I can not work on this.
Reopen the issue if you can reliably reproduce this problem


Some exploding style cleanup

@keybase/react-hackers the animation colors aren't finalized, but o/w this is ready!


nixos/mbpfan: add aggressive option

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[ ] 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/)
23.05 Release Notes (or backporting 22.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

[ ] Fits CONTRIBUTING.md.

Can you please drop the formatting changes in this PR? They go against the norm for NixOS modules and make the diff impossible to read.

Can you please drop the formatting changes in this PR? They go against the norm for NixOS modules and make the diff impossible to read.

Sure. You can review again now.

@ofborg eval

Thank you!

Thanks for merging!


text color when editing a node is white, same as default background color.

When editing a node, text color still white, same as background color, so I have to change the background color.

style - updated.txt
Please use the stylesheet that should help you .. tried and tested

I don't think it's a good idea to replace the entire css. For now, i'm fixing this within my app custom style using the code below:
.jstree-proton .jstree-clicked {
  color: inherit;
}

A proper fix should be done and merged in a new version of this package. Any contributor have suggestions? (@vedraan , @tihomiro , @brunowego)