Provide eval-ex-units endpoint, Set ex units in finalizeTx

Context: https://github.com/Plutonomicon/cardano-transaction-lib/issues/360

I haven't yet tested the new functionality introduced in this PR, and it would be great if someone could help me out with that.

OK I've run the AlwaysMints example and the ex units are indeed being calculated and updated in the redeemers

However the tx fails to submit with a budget error:
The budget was overspent. Final negative state: ({ cpu: -369459 | mem: -394 })

@ngua Good news!
I finally managed to submit a valid transaction (AlwaysMints) with correctly calculated execution units and fees.
Some considerations:

There seems to be an unfortunate mismatch between the formal description of minfee function in Alonzo specification (Figure 4: Functions related to fees and collateral) and the way it is defined in Cardano.Ledger.Shelley.LedgerState. Namely the implementation doesn't take script fees into account.

minfee pp tx =
  Coin $
    fromIntegral (getField @"_minfeeA" pp)
      * getField @"txsize" tx + fromIntegral (getField @"_minfeeB" pp)

Because of this, I decided to extend the eval-ex-units server endpoint to also calculate script fees based on the execution units and their prices (which can be read from protocol parameters).
Response format:

{"rdmrPtrExUnitsList":[{"exUnitsSteps":0,"rdmrPtrTag":0,"exUnitsMem":0,"rdmrPtrIdx":0}],"txScriptsFee":"0"}

This PR might probably close https://github.com/Plutonomicon/cardano-transaction-lib/issues/123 as well, since the difference between expected fees and newly calculated ones (basic fee + scripts fee) is now pretty insignificant (though there is some). I tested it on the AlwaysMints example and got 2640 lovelace difference, which is more like a margin of error. Therefore I changed the fee "padding" to 10000 lovelace, which hopefully should be enough to cover this difference.

Execution units are now recalculated in BalanceTx each time calculateMinFee is called, as previously discussed in the Slack channel.

evalExUnitsAndMinFee
  :: UnattachedUnbalancedTx
  -> QueryM (Either BalanceTxError (UnattachedUnbalancedTx /\ BigInt))
evalExUnitsAndMinFee uaubTx =
  runExceptT do
    let tx = uaubTx ^. _unbalancedTx <<< _transaction
    basicFee <- ExceptT $ calculateMinFee tx
      <#> bimap CalculateMinFeeError' unwrap
    tx' <- ExceptT $ reattachDatumsAndRedeemers uaubTx
      <#> lmap ReindexErrors'
    ExceptT $ evalTxExecutionUnits tx'
      <#> bimap EvalTxExUnitsError'
        \(EvalTxExUnitsResponse { rdmrPtrExUnitsList, txScriptsFee }) ->
          updateTxExecutionUnits uaubTx rdmrPtrExUnitsList
            /\ (basicFee + txScriptsFee)

There seems to be an unfortunate mismatch between the formal description of minfee function in Alonzo specification (Figure 4: Functions related to fees and collateral) and the way it is defined in Cardano.Ledger.Shelley.LedgerState. Namely the implementation doesn't take script fees into account.

Wait... Why are you using Cardano.Ledger.Shelley.LedgerState? That module defines the fee calculation for Shelley-era transactions, which did not even have scripts. Thus, it would make sense for that era's minfee function not to include script costs.
Why not use minfee from Cardano.Ledger.Alonzo.Tx, which includes the script cost and should be what the remote node uses when validating our transactions on mainnet?
https://github.com/input-output-hk/cardano-ledger/blob/2acff66e84d63a81de904e1c0de70208ff1819ea/eras/alonzo/impl/src/Cardano/Ledger/Alonzo/Tx.hs#L339-L346

Are you also using Shelley-era ledger types elsewhere in CTL, where better types exist in the Alonzo/Babbage eras?

@GeorgeFlerovsky No, we're using Alonzo-era types


post() is successful, but the callback isn't called

Here is the relevant part of my Angular code:
$scope.addTodo = function(){
	$sails.post("/todos", {what: $scope.new_todo.what}, function(resp){
		$scope.todos.push(resp);
	});
	$scope.new_todo = {};
}

The request is successful and if I listen for any events on the todos-model, they get send. But the callback-function is never called.
I tried the same with the normal io.socket.post() function, and then the callback function is called fine:
socket.post("/todos", {what:"test"}, function(data){
	console.log(data);
});

The todos-model is managed via blueprint. Am I missing something?

It returns a promise. I'm mobile right now so I cannot post an example, but there are some examples in the README.

Indeed it does, thanks!
Although it's kinda confusing that the API documentation states that the $sails-object works exactly like the io.sails-object.

Yeah, it used to be callback based, and that documentation may be old.  It changed to be more 'angular' and use promises (and at one point did both callback and return a promise, but somewhere callbacks were removed).


Select & backspace removes only the single character preceding the selection start

Using select + backspace has stopped working between 0.2.1 and 0.2.2
How I reproduced:

on master (58ffbc8) using broccoli serve to test the demo

Test 1:

select the second line and press backspace
the newline between the first and second line is deleted, rather than the selected text

Test 2:

select both lines and press backspace
nothing is removed, the cursor moves to the beginning of the first line

Test 3:

select a few characters in the middle of a word and press backspace
the single character preceding the start of the selection is deleted

Placing the cursor between two characters and pressing backspace works as expected, deleting the single character preceding the cursor.

Thanks for the super thorough report! We are moving toward semantic actions in post-0.2.1 versions (so, starting at 0.2.2) — moving from reacting to changes to the contenteditable surface towards capturing keystrokes and modifying the internal representation based on user intent. Deletion-handling code is not yet selection-aware, which I think is the root cause of all these failures. I'm starting work on this today.


Upgrade to node 12 has breaking change with JSON.stringify

Node 12 introduced a breaking change with JSON.stringify. https://github.com/tc39/ecma262/issues/944
The result is that the JSON produced by JSON.stringify might be different from what it was with NODE 10 and is not compatible with some json decoders from other languages. That's the case with the funciton json_decode from PHP.
Without a direct obvious fix we are going to fork this one stick to node 10, that I encourage you to do too.

Specifically, the issue is caused by this:
https://stackoverflow.com/questions/58674104/php-json-decode-does-not-work-with-single-unpaired-surrogate-caused-by-node-12-w

https://hub.docker.com/r/apify/actor-node-chrome
A few issues here

Still saying "node 10" in the desc
no tag available to keep using node 10

Keep in mind that bumping to a major version of a vendor without any warning nor fallback can have lot of consequences for users

@gsouf Thanks for the report, we've fixed the descriptions.
You are right, we need to be more careful with upgrading major release. We'll consider introducing tags like node-10 or 10 for Docker images that will let you keep an older version for some time.

Closing as this seems very outdated, and we now release imagines with tags so you can stay on given node version (but we now support node 15/16 only anyway 🤷)


Loading Checkpoints After Initialization + Reset()

I had a use-case where I initialized a medium mesh cylinder FlowEnv without a checkpoint (just the default flow), and then later used env.flow.load_checkpoint. The env.render() function worked fine, and showed my checkpointed flow. However, I received an error when I next used env.reset(). Everything seems to work as expected if I use the checkpoint "restart" key in the flow config. But it seems like this is worth pointing out, because intuitively you would expect to be able to load a flow and use env.reset() as long as the meshes were identical.
EDIT: Using latest version of hydrogym on main branch.
Code:
from hydrogym import FlowEnv
import hydrogym.firedrake as hgym

flow_env_config = {
    "flow": hgym.Cylinder,
    "solver": hgym.IPCS,
    "solver_config":{'dt': 0.001},
    "flow_config": {'mesh': 'medium'}
}

env = FlowEnv(flow_env_config)

# Renders Correctly
env.render()

path_to_ckpt = '/path/to/checkpoint.ckpt'
env.flow.load_checkpoint(path_to_ckpt)

# Renders Correctly
env.render()

# errors
env.reset()

Error:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In [4], line 27
     24 env.render()
     26 # errors
---> 27 env.reset()

File /home/firedrake/sindy-rl/hydrogym/hydrogym/core.py:374, in FlowEnv.reset(self)
    372 def reset(self) -> Union[PDEBase.ObsType, Tuple[PDEBase.ObsType, dict]]:
    373     self.iter = 0
--> 374     self.flow.reset(q0=self.q0)
    375     self.solver.reset()
    377     return self.flow.get_observations()

File /home/firedrake/sindy-rl/hydrogym/hydrogym/core.py:102, in PDEBase.reset(self, q0)
     94 """Reset the PDE to an initial state
     95
     96 Args:
   (...)
     99         Defaults to None.
    100 """
    101 if q0 is not None:
--> 102     self.set_state(q0)
    103 self.reset_controls()

File /home/firedrake/sindy-rl/hydrogym/hydrogym/firedrake/flow.py:91, in FlowConfig.set_state(self, q)
     85 def set_state(self, q: fd.Function):
     86     """Set the current state fields
     87
     88     Args:
     89         q (fd.Function): State to be assigned
     90     """
---> 91     self.q.assign(q)

File PETSc/Log.pyx:115, in petsc4py.PETSc.Log.EventDecorator.decorator.wrapped_func()

File PETSc/Log.pyx:116, in petsc4py.PETSc.Log.EventDecorator.decorator.wrapped_func()

File /home/firedrake/firedrake/src/firedrake/firedrake/adjoint/function.py:117, in FunctionMixin._ad_annotate_assign.<locals>.wrapper(self, other, *args, **kwargs)
    114     tape.add_block(block)
    116 with stop_annotating():
--> 117     ret = assign(self, other, *args, **kwargs)
    119 if annotate:
    120     block.add_output(self.create_block_variable())

File <decorator-gen-48>:2, in assign(self, expr, subset)

File /home/firedrake/firedrake/src/firedrake/firedrake/utils.py:74, in known_pyop2_safe.<locals>.wrapper(f, *args, **kwargs)
     72 opts["type_check"] = safe
     73 try:
---> 74     return f(*args, **kwargs)
     75 finally:
     76     opts["type_check"] = check

File /home/firedrake/firedrake/src/firedrake/firedrake/function.py:412, in Function.assign(self, expr, subset)
    409     return self
    411 from firedrake import assemble_expressions
--> 412 assemble_expressions.evaluate_expression(
    413     assemble_expressions.Assign(self, expr), subset)
    414 return self

File PETSc/Log.pyx:115, in petsc4py.PETSc.Log.EventDecorator.decorator.wrapped_func()

File PETSc/Log.pyx:116, in petsc4py.PETSc.Log.EventDecorator.decorator.wrapped_func()

File <decorator-gen-38>:2, in evaluate_expression(expr, subset)

File /home/firedrake/firedrake/src/firedrake/firedrake/utils.py:71, in known_pyop2_safe.<locals>.wrapper(f, *args, **kwargs)
     69 safe = parameters["type_check_safe_par_loops"]
     70 if check == safe:
---> 71     return f(*args, **kwargs)
     72 opts["type_check"] = safe
     73 try:

File /home/firedrake/firedrake/src/firedrake/firedrake/assemble_expressions.py:520, in evaluate_expression(expr, subset)
    516         except ReferenceError:
    517             # TODO: Is there a situation where some of the kernels
    518             # succeed and others don't?
    519             pass
--> 520 arguments = expr.par_loop_args
    521 if cache is not None:
    522     cache[slow_key] = arguments

File /home/firedrake/firedrake/src/PyOP2/pyop2/utils.py:62, in cached_property.__get__(self, obj, cls)
     60 if obj is None:
     61     return self
---> 62 obj.__dict__[self.__name__] = result = self.fget(obj)
     63 return result

File /home/firedrake/firedrake/src/firedrake/firedrake/assemble_expressions.py:327, in Assign.par_loop_args(self)
    325 result = []
    326 grouping = OrderedDict()
--> 327 for e in self.split:
    328     grouping.setdefault(e.lvalue.node_set, []).append(e)
    329 for iterset, exprs in grouping.items():

File /home/firedrake/firedrake/src/PyOP2/pyop2/utils.py:62, in cached_property.__get__(self, obj, cls)
     60 if obj is None:
     61     return self
---> 62 obj.__dict__[self.__name__] = result = self.fget(obj)
     63 return result

File /home/firedrake/firedrake/src/firedrake/firedrake/assemble_expressions.py:270, in Assign.split(self)
    267 if indices == set([None]):
    268     if len((set(spaces) | {V}) - {None}) != 1:
    269         # Check that there were no unindexed coefficients
--> 270         raise ValueError("Saw indexed coefficients in rvalue, "
    271                          "perhaps you meant to index the lvalue with .sub(...)")
    272     rvalues = self.splitter(self.rvalue, False)
    273     return tuple(type(self)(lvalue, rvalue)
    274                  for lvalue, rvalue in zip(self.lvalue.split(), rvalues))

ValueError: Saw indexed coefficients in rvalue, perhaps you meant to index the lvalue with .sub(...)

Do I understand it correctly, that the above is already enough to reproduce the bug? :)
(Also, sorry for not getting on top of this earlier)

@ludgerpaehler—should be! I've updated to include my python and firedrake versions. The only additional assumption should be that there is already some  saved mesh (also a medium mesh from the cylinder) created from env.flow.save_checkpoint.


deleting and re-inserting view.el into dom

I have an element with event bindings. I insert them into the dom. After insertion I delete the parent element in the DOM.
Later on when I reinsert the element into the DOM the event bindings are lost.
I have to use this.delegateEvents() to get them back. I expected that the events bound would still hold even if the element was removed from the dom. What am I misunderstanding?
Thanks!

This depends on the method you use to remove the element from the DOM.  If you use remove then your bindings will be lost.  However, using detach will not remove your bindings.  Backbone only provides remove, but you can easily add a method for detach.
var MyView = Backbone.View.extend({
  detach: function() {
    this.$el.detach();
  }
});


Update play to 2.8.11

Updates com.typesafe.play:play from 2.8.7 to 2.8.11.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe.play", artifactId = "play" } ]

labels: library-update, early-semver-patch, semver-spec-patch

Superseded by #295.


Add infrastructure to test alternative planet containers with Robotest

Feature Request
In our documentation, we allow end users to specify a custom planet container when building cluster images:
https://gravitational.com/gravity/docs/pack/#custom-system-container
We should test building an image with a custom container.
Motivation
This was requested by @a-palchikov here:
https://github.com/gravitational/gravity/issues/1908#issuecomment-665098144

Also the new test should exercise builds with alternative planet container images.

Who's it for?
Gravity developers.

@a-palchikov: I'm not sure I understand this request.  What is the use case?  E.g. Are you looking for automated testing of a particular scenario/regression?  Or is this for one-off testing?
Presently, if we override PLANET_TAG (by manually editing the Makefile), and produce a custom telekube.tar, robotest will use that tar in its runs.  This seems like it would meet needs for one-off testing.
I don't understand why we'd want to automatically test a custom planet image in e.g. PR builds or nightly builds.

What I'm referring to with this is completely replacing the planet container (or customizing it for specific node profile) as described here. As an example - the current 7.0.x is broken w.r.t this - e.g. an installer that replaces the default system container image will not install. I stumbled on it by chance and it's a regression.

As an example - the current 7.0.x is broken w.r.t this - e.g. an installer that replaces the default system container image will not install. I stumbled on it by chance and it's a regression.

Do we have an issue tracking this regression and its impact/priority?  I didn't find anything when I checked here:
https://github.com/gravitational/gravity/issues?q=is%3Aissue+planet+label%3Aregression
Of particular interest to this discussion, is this a regression users are hitting?  I'm looking to determine relative priority and also maybe a real world use case -- such that any custom planet image I implement for test purposes is less contrived.

As an intermediate goal, it'd be great to add a custom planet container to our manual test plan before the our run of the plan.

Yes, I should've created the issue when I stumbled on it - will do now.

An important customer saw a regression in the functionality this would test: https://github.com/gravitational/gravity/issues/2125


Tests : Maria DB missing.

Namaskara,
Thanks for this wonderful library.
I am coding a OSS based trader. I wanted to use SQLPP11.
I cloned the git repository and built it. However I am not seeing any file (or section related to Mariadb). I see sections on Mysql, PostGresql and SQL lite. Mariadb seems missing. Can I go head and use mySql for MariaDB backend?
I will be wonderful if you can add tests for Mariadb.
With Warm Regards,
Rathnadhar K V

Hi Rathnadhar,
Thanks for the nice feedback!
Admittedly, it has been a while since last I tested with MariaDB (and right now, I cannot test it myself), but you should be able to use instead of MySQL. Please give it a try and let me know whether it worked?
That said, true, I should add tests for MariaDB.
Best,
Roland

@rathnadhar
sqlpp11 works fine with MariaDB, just use the MySQL connector and connect to the MariaDB database. Also all sqlp11 MySQL tests pass correctly with a MariaDB backend.
MariaDB and MySQL are pretty much the same database.

Essentially when the CMake option BUILD_MARIADB_CONNECTOR is enabled it checks if MariaDB is installed, but the user should use the MySQL connector, because MariaDB is (mostly) the same database as MySQL, so there is no point in duplicating the code.
The situation with SQLite3 and SQLCipher connectors is similar. We only have code for the SQLite3 connector and if the user wants to connect to an SQLCipher database, then he should use the SQLite3 connector.
I think that this is mostly a documentation issue and it could be resolved by updating the documentation.

Thanks for the confirmation, @MeanSquaredError !
Yes, it is primarily a documentation issue. It might still make sense to add a tests that make use of an actual MariaDB backend (same with SQLCipher).

Thanks for the feedback, I will test with MariaDB and will revert with the
results.
Again, Thank you so much.
Regards,
[image: photograph]
Rathnadhar K V
CEO
E @.***
Akshobhya Investments
introcard.in/rathnadhar
[image: facebook icon] https://www.facebook.com/rathnadhar/ [image:
twitter icon] https://twitter.com/RathnadharKV [image: linkedin icon]
https://www.linkedin.com/in/rathnadhar
[image: https://www.rotary.org] https://www.rotary.org
On Sat, Sep 14, 2024 at 4:04 PM Roland Bock @.***>
wrote:

Hi Rathnadhar,
Thanks for the nice feedback!
Admittedly, it has been a while since last I tested with MariaDB (and
right now, I cannot test it myself), but you should be able to use instead
of MySQL. Please give it a try and let me know whether it worked?
That said, true, I should add tests for MariaDB.
Best,
Roland
—
Reply to this email directly, view it on GitHub
https://github.com/rbock/sqlpp11/issues/599#issuecomment-2350945723, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANWFAAD2BXPICFUXCSBGYXLZWQGJTAVCNFSM6AAAAABOGV54BOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJQHE2DKNZSGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Code for RoQNN and Efficient On-Chip Training of QNN

Hi,
I wonder when you will release the code & tutorial for your recent papers using this codebase, including RoQNN and Efficient On-Chip Training of QNN?

Hi Haoxiang,
Thanks a lot for your interest in our work! The code for those two papers will appear as examples of TorchQuantum and we expect to release them in around 1 month.


⚠ Bofrost.at has degraded performance

In cdf53ec, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2265 ms

Resolved: Bofrost.at performance has improved in c8d7e6c after 30 minutes.


Possible to get the package name at a given call depth?

Any interest in having that feature in the package?

I'm asking because I'll put together a PR to add support for it if so.

Yes, this could be useful. I think adding a new formatting verb that produces the package name is the way to go here. One of the forks has added %k for package name (https://github.com/getlantern/stack/commit/02f928aad224fbccd50d66edd776fc9d1e9f2f2b). We should also consider %p.

@ChrisHines submitted a PR with this change after reviewing and testing it.

Implemented by #18.

@chmorgan Thanks for the feature suggestion and being patient during my code review. The new feature was just released in v1.6.0.  🎉

@ChrisHines its no trouble, thanks for fixing the code up and getting this through vs. having it sit here. I'm not good at golang yet, especially not with slices. Coming next is PR against go-kit for the package name and package filtering.


ref(glide): Using glide 0.9.0 new import resolver

glide up run with 0.9.0 that walks the import tree rather than
the filesystem tree. Pulls just the 36 dependencies in the tree.
Removes travic ci caching. The number of packages being
is small enough fetching them should be faster than copying from
the travis cache.
The glide install run Make bootstrap should be backwards
compatible to glide 0.8.3.

The output of glide up is at https://gist.github.com/mattfarina/009fad1cfbaccdce7aca

I'm doing some manual testing, and then will merge.


1 out of 7 camera not showing up for adding in homekit

Describe Your Problem:
I have 7 Tapo cameras of different models and 1 of them, a C520WS does not show up in HomeKit add accessory as the others do.
Logs:
[21/07/2024, 12:12:43] Cam Children 6809 is running on port 12345.
[21/07/2024, 12:12:43] Please add [Cam Children 6809] manually in Home app. Setup Code:
[21/07/2024, 12:12:43] Cam Garage C772 is running on port 12413.
[21/07/2024, 12:12:43] Please add [Cam Garage C772] manually in Home app. Setup Code:
[21/07/2024, 12:12:43] Cam Backyard E0D7 is running on port 53451.
[21/07/2024, 12:12:43] Please add [Cam Backyard E0D7] manually in Home app. Setup Code:
[21/07/2024, 12:12:44] Cam Study FEA8 is running on port 64312.
[21/07/2024, 12:12:44] Please add [Cam Study FEA8] manually in Home app. Setup Code:
[21/07/2024, 12:12:44] Cam Family 4E04 is running on port 51333.
[21/07/2024, 12:12:44] Please add [Cam Family 4E04] manually in Home app. Setup Code:
[21/07/2024, 12:12:44] Cam Garden FCF7 is running on port 52134.
[21/07/2024, 12:12:44] Please add [Cam Garden FCF7] manually in Home app. Setup Code:
[21/07/2024, 12:12:44] Cam Living 48E6 is running on port 56337.
[21/07/2024, 12:12:44] Please add [Cam Living 48E6] manually in Home app. Setup Code: ```

Plugin Config:
{
    "cameras": [
        {
            "name": "Cam Garage",
            "ipAddress": "<IP_ADDRESS>",
            "password": "@@@@",
            "streamUser": "@@@",
            "streamPassword": "@@@"
        },
        {
            "name": "Cam Living",
            "ipAddress": "<IP_ADDRESS>",
            "password": "@@@@",
            "streamUser": "@@@",
            "streamPassword": "@@@"
        },
        {
            "name": "Cam Garden",
            "ipAddress": "<IP_ADDRESS>",
            "password": "@@@@",
            "streamUser": "@@@",
            "streamPassword": "@@@"
        },
        {
            "name": "Cam Children",
            "ipAddress": "<IP_ADDRESS>",
            "password": "@@@@",
            "streamUser": "@@@",
            "streamPassword": "@@@"
        },
        {
            "name": "Cam Backyard",
            "ipAddress": "<IP_ADDRESS>",
            "password": "@@@@",
            "streamUser": "@@@",
            "streamPassword": "@@@"
        },
        {
            "name": "Cam Family",
            "ipAddress": "<IP_ADDRESS>",
            "password": "@@@@",
            "streamUser": "@@@",
            "streamPassword": "@@@"
        },
        {
            "name": "Cam Study",
            "ipAddress": "<IP_ADDRESS>",
            "password": "@@@@",
            "streamUser": "@@@",
            "streamPassword": "@@@"
        }
    ],
    "_bridge": {
        "username": "00:00:00:00:00:00",
        "port": 12345
    },
    "platform": "tapo-camera"
}

Screenshots:

Environment:

Plugin Version: v2.3.4
Homebridge Version: v1.8.4
Node.js Version: v20.12.2
NPM Version:
Operating System: Raspbian

Hello, please update the plugin, enable deubg mode in Homebridge and post your logs.


ES Hadoop does not retry on HTTP 429

Bulk requests in ES 2.x now respond with HTTP 429 (Too many requests) when rejecting due to threadpool queue sizes.

Confirmed the bug. It's fixed in master (along with a hefty amount of tests to make sure this works properly across the different versions). Thanks for reporting this!


Expose markdown remark plugin defaults via an export

I thought I might be able to write:
    '@elderjs/plugin-markdown': {
      routes: ['blog'],
      remarkPlugins: [
        [require("remark-shiki-twoslash"), { theme: "nord"}]
      ]
    },

But that crashes the template because it (very reasonably) assumes the availability of yml front-matter. Ideally I could do something like:
    '@elderjs/plugin-markdown': {
      routes: ['blog'],
      remarkPlugins: [
         ...require("@elderjs/plugin-markdown").plugins,
        [require("remark-shiki-twoslash"), { theme: "nord"}]
      ]
    },

@orta Huge fan of your work. As I mentioned in discord, Elder.js has a massive API surface so trying to keep the plugins relatively minimal. :)
100% open to a PR here.
Based on the current setup, not sure the ...array would overwrite:
const plugins = [
        remarkFrontmatter,
        [extractFrontmatter, { name: 'frontmatter', yaml: yaml }],
        remarkSlug,
        remarkHtml,
];

It is almost as if we'd need a dedicated code parsing key on the config.

The default remarkPlugins are exported for ease of use via the package.
const { remarkPlugins } = require('@elderjs/plugin-markdown');

But the above example still may not work because the order of imports matters.


Add support for filtering

most API end points now support filtering
allow supplying generic filter specification

implemented with f87a715e086a2c81ae0e80a899b701939106efba


Publish documentation on moryx-industry.net

Currently the documentation is only generated. For releases it should also be published on moryx-industry.net.

So I had a look at https://github.com/dotnet/docs and they do use path links as well. Their documentation completely works on Github, you can event step through previous and next in the tutorials. Look at the documentation for delegates for example.
Inline examples are broken, but still link to the correct file, see here
We should have hosted documentation, but that looks like working documentation is common on GitHub. We can also link to files using their path. I doubt the path changes more often than the namespace. Thoughts?


https://github.com/apache/flink-cdc/pull/2315 这是flink cdc 旧版本的BUG file is not a valid field name ,flink cdc 新版版已经修复了

Search before asking

[X] I had searched in the issues and found no similar issues.

Description
doris-flink-connector是否会跟进修复,
Solution
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

The flinkcdc version will be upgraded to 3.1 later.


Placing an order not working using place_order()

I'm trying to place an order, but get the below error in the screenshot. It's returning true when sending in my trade token using get_trade_token(), so I assume that doesn't have anything to do with it. I'm also able to view my account using get_account(), so logging in and authenticating doesn't have to do with it. What could be the issue here? You can see the code I use to send the order in the stack trace. I was able to place this trade manually on the Webull desktop app, so not sure why it won't go through here. Thanks

Turns out I was getting this:
{'msg': 'Fail to verify image verification code.', 'traceId': '2400b497e80d4e698902f586ba5f5cae', 'code': 'user.check.slider.pic.fail'} when logging in, so the account Id wasn't being set

I was able to get around this by setting the access token before logging in by using the inspect tool and getting the most recent access token and then setting the login timeout for to 8 hours (how long I want to stay logged in). Not sure if there is another way to get around this. I followed the previous thread to get around this by setting the did token, but was still getting this problem.

I have the same issue.
How did you:
"Set the Access Token" and "Set the login timeout"
I always get the {'msg': 'Fail to verify image verification code.', 'traceId': '24.....    after following all other threads
I set the did but still not a successful login.  However, I can view my account() data wb.get_account()
wb._did = "[did_code]"
I appear to login but I can not place_order()

Same case here ,  I logged in through work around method. (from net work tab account response into notepad style)
I am not able to place order. I am getting order history , order positions
Any advice from any one

Did you happen to get it fixed?  I'm experiencing same issue.  Getting a 405 error.


Killling saved connections

Description
All my database connections list is lost!
... after replace , to DEFAULT NULL,  in sql script editor.
DBeaver Version
23.1.3
Operating System
Ubuntu 22.10
Database and driver
Any driver (tested on MariaDB, PostgreSQL)
Steps to reproduce

Create and save any connection on Database tab
Open database and open Script editor for opened database and print ,
Press Ctrl+H, press "Replace ..."
Replace , to DEFAULT NULL, and press OK

and i get an exception:
com.google.gson.stream.MalformedJsonException: Unterminated object at line 3 column 18 path $..Mydb

Reason:
com.google.gson.stream.MalformedJsonException: Unterminated object at line 3 column 18 path $..Mydb

Restart Dbeaver and I get and exception:
com.google.gson.stream.MalformedJsonException: Unterminated object at line 2 column 17 path $.

Reason:
com.google.gson.stream.MalformedJsonException: Unterminated object at line 2 column 17 path $.

Additional context
No response

Thank you for bug report.
We had something similar some time ago, but couldn't reproduce #20065
I can reproduce it now.

This suggestion should help to eliminate the error message and restore your connections  https://github.com/dbeaver/dbeaver/issues/20065#issuecomment-1560624522

I found backup json files in
.local/share/DBeaverData/workspace6/General/.dbeaver/.data-sources.json.bak
.local/share/DBeaverData/workspace6/General/.dbeaver/.credetials-config.json.bak
.local/share/DBeaverData/workspace6/General/.dbeaver/.project-metadata.json.bak

and restore (rename) to
.local/share/DBeaverData/workspace6/General/.dbeaver/data-sources.json
.local/share/DBeaverData/workspace6/General/.dbeaver/credetials-config.json
.local/share/DBeaverData/workspace6/General/.dbeaver/project-metadata.json

but in data-sources.json I was found
    "folders": {
        "Folder1": {}default null DEFAULT NULL DEFAULT NULL,
        "Folder2": {}default null DEFAULT NULL DEFAULT NULL,
        "Folder3": {}default null DEFAULT NULL DEFAULT NULL,
        "Folder4": {}default null DEFAULT NULL DEFAULT NULL,
        "prod": {
            "parent": "Folder5"
...

(and project-metadata.json)
DBeaver was replace text in json files instead SQL editor.
I fix project-metadata.json and data-sources.json files and my DBeaver start correctly.
I became interested in the problem. DBeaver working correctly, but...
Ctrl+H - open Search window and designed for searching in files instead SQL editor.
I should press Ctrl+F instead Ctrl+H
I am sorry :)
But DBeaver should be protect own configuration files from user. I think.
Thanks.

We inherit this dialog from Eclipse platform, but we didn't expect such issue. It definitely shouldn't touch any files like data-sources.json. And I would expect that the dialog called from SQL Editor context would perform replacements only in the current SQL Editor.

We want to restrict places where the files are searched. .dbeaver should not be included there.


Windows drag window laggy

Flutter doctor:
[✓] Flutter (Channel stable, 3.3.4, on Microsoft Windows [版本 10.0.19044.1645], locale zh-CN)
[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✓] Chrome - develop for the web
[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.20)
[!] Android Studio (not installed)
[✓] VS Code (version 1.72.1)
[✓] Connected device (3 available)

drag flutter windows is very slow
https://user-images.githubusercontent.com/21303438/196341307-6a986f13-7947-4ad0-b16b-4161da750f85.mp4

@jinfagang
Did you modify any code from windows folder? Please provide a completed and minimal reproducible code sample so that we may verify this.


Is this safe to use?

It seems this is rarely updated, and the 43 Pull Requests from dependabot that have not been merged kinda puts me off using this. Can this safely be used, or is there a better alternative?

We don't have an alternative. It should be okay to use. But I agree that it needs some TLC. I will keep it in mind for a future sprint that we should get it updated and address thr dependabot PRs 😅

Well, i guess an alternative would be to use the REST API directly or our .net client library :)


[FEAT] add queryOptions to useShow hook

Is your feature request related to a problem? Please describe.
Conditionally show an item / trigger the query based on props.
Currently, I don't think there's a way to disable the show query!?
While almost all other query hooks have that option.
Describe alternatives you've considered
No response
Additional context
No response
Describe the thing to improve
Add queryOptions to useShow hook.

Hey @jsbrain ,
Thank you for contacting us! Would you like to create a PR?

Hey @jsbrain ,
We've released<EMAIL_ADDRESS>Could you approve this feature? Thank you! 🎉


More racket analysis

https://github.com/aymanosman/intellij-racket/blob/master/src/main/kotlin/org/racket/ide/analysis/RacketAnalysis.kt I think we can use annotations-mixin to get more helpful information

That's true. It could be added to one of the passes in https://github.com/aymanosman/racket-intellij.
The current implementation doesn't perform very well. It runs an external process to gather diagnostics on every code change. This, I think, is discouraged in the IntelliJ world. I think there are two alternatives.

Write all analyses in Kotlin/Java
Spawn a long running process to handle requests for updated diagnostics (basically LSP)

Option 1 is not very attractive.
I'm not sure how to do options 2 since it doesn't appear to be an approach that the IntelliJ folks push very much.
I believe the Rust plugin goes mostly for option 1, and has to implement its own type inference (among other things)!
On the other hand, TypeScript support seems to rely on an external process (presumably LSP based).
General LSP support in IntelliJ is not good, last I checked.

Option 1 will not work well with macros I believe. For option 2, LSP model is already developed for Racket, but I not sure the information they provided will fit intellij requirement.
I prefer provide customize process here, and I can work on this once we have analysis what to do here

Sounds good. I also think that that would be the best option to take.
For inspiration we can look at Greg Hendershott's excellent emacs package https://github.com/greghendershott/racket-mode.

or https://github.com/racket-tw/sauron and https://github.com/jeapostrophe/racket-langserver, they all use the same way.


procfs: Remove mallinfo from struct procfs_meminfo_entry_s

Summary
let's call mm_mallinfo directly
Impact
Code refactor
Testing
Pass CI

Hi @xiaoxiang781216 what are the benefit of this modification? I think the code became simple/easier, but are other benefits doing this way?

Hi @xiaoxiang781216 what are the benefit of this modification? I think the code became simple/easier, but are other benefits doing this way?

@acassis for consistence. You can see that meminfo_read already call mm_memdump directly, which make the abstraction through callback look very strange.


Power and Energy Hooks

Closes #85
Energy consumption requires density!

Should name the mass and volume flux equally.
Throughput is also a valid term, but more from the economical point of view. But I agree with using flux.


Create Event: After successful event creation, redirects to blank page. I think this should redirect to the user's profile page

Done.
commit ID: 28c3363


webpack library configuration does not resolve source map sources

Here is a sample webpack configuration file
module.exports = {
  target: 'node',
  devtool: 'source-map',
  entry: './src/lib.ts',
  module: {
    rules: [
      {
        loader: 'ts-loader',
        exclude: /node_modules/
      }
    ]
  },
  node: {
    __dirname: false,
  },
  resolve: {
    extensions: [ '.ts', '.js' ]
  },
  externals: [nodeExternals()],
  output: {
    filename: 'lib.js',
    path: path.resolve(__dirname, 'build'),
    library: 'lib',
    libraryTarget:'umd'
  }
};

This produces source map source urls which look like webpack://lib/./src/lib.ts and should be mapped to src/lib.ts.
Here is a sample fix against an outdated branch.

@dgozman it looks like your fork is private, can you make it public or push the fix to a branch on this repo so we can see it?

Thanks, I can see it now!
You might have seen we moved over this handling to the user-configurable sourceMapOverrides config matching the existing adapters, which doesn't support full regexes right now. I'm not sure how/whether this scenario works with the current adapters, but I was able to reproduce the issue with PWA using the sample code you provided. I think my next step would be to first see if this reproduces with the existing adapters; if it doesn't, then there's something we missed in porting the logic to PWA, if it does then we should probably figure out how to take your linked fix into the sourceMapOverrides-driven world.
This is endgame week for vscode so I don't anticipate getting too much headway on this issue this week. If you have time to look at applying the fix to the current repo, that would be great, otherwise I'll look into this issue late this week or, more likely, next week 🙂

Thanks, Connor! I don't have much context about sourceMapOverrides, that's why I didn't port the fix to the master myself, but I will be happy to review if needed.


Update jsoniter-scala-core, ... to 2.9.1

Updates

com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-core
com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-macros

from 2.7.3 to 2.9.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.plokhotnyuk.jsoniter-scala" } ]

labels: library-update, semver-minor

Codecov Report

Merging #276 (df379aa) into master (2d51420) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #276   +/-   ##
=======================================
  Coverage   43.72%   43.72%
=======================================
  Files          25       25
  Lines         263      263
  Branches        6        5    -1
=======================================
  Hits          115      115
  Misses        148      148

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 2d51420...df379aa. Read the comment docs.


Img tag attrs

Ok, this one will fix #7 for me and adds total control over each img tags attributes with a filtering.

I will need to add some documentation if you all are good with this one.

I am good with the premise of the change. :smile: Thank you!

I'll get this one updated and resubmitted this week. Thanks for all the input! You guys do great code reviews!

Superseded by #16.


XSLT function to run a datasplitter

Datasplitter is great for turning structured text into XML.
Occasionally, structured text data appears within XML elements.  This can happen when a log file is handled by an intermediate process that might wrap it, e.g. as JSON.  In these cases it would be useful to be able to run a datasplitter over the structured text from within an XSLT.
A new stroom:data-splitter() XSLT function would achieve this.  It would take the structured text and the UUID of a datasplitter configuration as input parameters, and return a <records> element

It has been suggested that xsl:analyze-string() is capable of meeting most needs.


.Query<> seems to ignore Core.Arango.Linq.Attributes.CollectionProperty Collection name

When querying a simple list:
await _dbContext.Context("gt").Query<GT_Common.DataContracts.ArangoDatabase.Tenant>("env").ToListAsync();
It seems to always search for content in the collection "Tenant". But on the model we have this property present:
 [Core.Arango.Linq.Attributes.CollectionProperty(CollectionName = "tenants", Naming = Core.Arango.Linq.Interface.NamingConvention.UnChanged)]
    public class Tenant
    {

{"{\"code\":404,\"error\":true,\"errorMessage\":\"AQL: collection or view not found: Tenant (while parsing)\",\"errorNum\":1203}"}
Is this a bug or a missuse of the property ?

Didn't even realize that attribute existed publicly^^
That happens if you integrate other open source code.
I intended the resolve collection option to be the only way
 Arango = new ArangoContext(UniqueTestRealm(), new ArangoConfiguration
{
  ResolveCollection = type => type.Name.ToLowerInvariant() + "s"
});

however, since the attribute is the I will also look for that

Awesome :D
My main problem was that my collection was named "tenants", plural and the model class "Tenant". So it wouldn't map it.
That's why I used the "CollectionName" Attribute.
In the meantime I'll use normal AQL:
await _dbContext.Context("gt").Query.ExecuteAsync<GT_Common.DataContracts.ArangoDatabase.Tenant>("env", $"FOR t IN tenants RETURN t");

@coronabytes  Thank you for your very fast fix. When can we expect a new release (NuGet) ?


Readme Update

Added data pipeline overview.

Hi @redpheonixx thank you so much for taking the effort to contribute directly to the core documentation of this project. We could use some help documenting orchestration too. Let's me know on comments whatever issues come to your mind. Happy Friday!

Made changes in overall readme structure and also removed some areas which seemed unnecessary to me. @tusharchou @mrutunjay-kinagi @abhishekkar0603  please review and add comments.


Deployment Issues

I'm having trouble deploying the app. I'm getting the error:
TypeError: JwtStrategy requires a secret or key
I have a process.env file in the root with all the variables but it doesn't seem to be working.

Hi, apologies the super delayed response. I am looking into this now and will update you!

Hi @tspanu, I added dotenv for the server code with https://github.com/mjw56/headfon.es/commit/fc754931c4e9abbfedf05390bf7f0b19ae11c5e4
This should pull in a .env file if you have one in the /server folder. Sorry about that, I had the code pulling the environment variables from the system level before.
Can you please test this and let me know if it is working?


Export doesn't work

Please Reference https://github.com/jupyter-widgets/ipyleaflet/issues/1090

Needed ipyleaflet for solving the issues. I cannot did much about this


Global sign out does not seem to invalidate tokens on another device

Describe the bug
To test this scenario, I build and run the app on 2 different simulators.
When I global sign out from one of the simulator, I expect the tokens to be invalidated on the other simulator. However, when I call fetchAuthSession to retrieve the tokens, I found out the tokens are not invalidated at all and that the session is still signed in.
Am I missing something?
To Reproduce
Steps to reproduce the behavior:

Build and run app on 2 simulators.
Sign out globally on 1 simulator.
The other simulator session is still signed in. Expected to be signed out with Cognito tokens invalidated.

Expected behavior
Expect the Cognito tokens on the other simulator to be invalidated.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment(please complete the following information):

Cocoapods [1.10.1]
Amplify [1.6.0]
AmplifyPlugins/AWSCognitoAuthPlugin [1.6.0]
Swift 5
Xcode [12.3]

*Device Information (please complete the following information):

Simulators:
iPhone 12 Pro Max (14.3)
iPhone SE - 2nd generation (14.3)

Additional context
Add any other context about the problem here.

Hi, @aizatomar
Thanks for reaching out.
There is one similar known issue.
What happens basically is, when globalSignOut() is called, the refresh token is revoked, which means that id & access token cannot be refreshed anymore, but it takes some time (1 hour or longer depends on the configuration) for id & access token to become invalid. And I believe this is what causing you to observe the behavior. We are working on the fix, but cannot tell when the fix can be done.

Ok thank you for the reply. I have also tested it on my side and yes it will take about an hour for the token to be revoked.

Upon calling globalSignOut will the id & access token be invalidated? Can this atleast be done in cognito itself or is this a bug in cognito?

Amplify library added (revokeToken)[https://docs.aws.amazon.com/cognito-user-identity-pools/latest/APIReference/API_RevokeToken.html] that invalidates the accessToken as well while invoking signOut. You can also call this api directly using the escape hatch.
Let us know if this solves your use case.

Amplify library added (revokeToken)[https://docs.aws.amazon.com/cognito-user-identity-pools/latest/APIReference/API_RevokeToken.html] that invalidates the accessToken as well while invoking signOut. You can also call this api directly using the escape hatch.
Let us know if this solves your use case.

No, its not. Initially issue says about two devices. It is not possible to call revokeToken on device A from device B

If the user thinks their account might be compromised and they changed their password whereby we globally sign them out, any malicious party who has access to the account will continue to do so for up to an hour with the current implementation.


Add Travis-CI configuration

This allows the project to be automatically built and tested with the latest version of Perl6 on the Travis-CI continuous integration service.
This time I managed to install the required libraries as prereqs, so this should run all tests on Travis.

All good, cheers!


Get WinMain parameters?

Hi all,
I'm wondering if there's a way to get the four parameters (hInstance, hPrevInstance, lpCmdLine, nCmdShow) in WinMain(), especially nCmdShow.
Solutions:

hInstance:
let hInstance = kernel32::GetModuleHandleW(ptr::null());
hPrevInstance: ?
lpCmdLine: env::args().skip(1)
nCmdShow: ?

The MSDN documentation on hPrevInstance  states:

A handle to the previous instance of the application. This parameter is always NULL.

nCmdShow can easily be obtained (along with a bunch of other stuff) via GetStartupInfo.

Thanks for your answer!


Consider Docker labels in lieu of "custom tags"

I'm evaluating sbt-docker-compose and the primary thing that first attracted me to it—as opposed to possible alternatives like docker-it-scala which leaves me to duplicate container definitions in yet another config format—is the hope of reusing perfectly good existing Compose configs that people can use in their local development with standard docker-compose, independent of sbt-docker-compose.
Adding custom preprocessed extensions to the file format like <localBuild> and <skipPull> breaks compatibility of the files with the standard tools. IMO this is a fundamental design misfeature.
I'd like to propose that custom labels could potentially fill this need instead, and likely make further extensibility easier to boot (you could abstract a contract for an sbt-docker-compose label processor instead of requiring processCustomTags specifically to be overridden).
As I said I'm still evaluating, but if the plugin otherwise meets my needs, I'll try to whip up an implementation as a PR.

Hi @ches, thanks for the feedback and evaluating the plugin!
I agree, that adding the custom tags to a label instead of as part of the image value would make the compose file more portable for direct use by docker-compose. I certainly would be open to a PR that adds the ability to be able to define custom tags within a label.
Note that the custom tags mostly play a part in relating your docker-compose definition to a multi-project sbt build. If you are just using a more basic sbt project you should not need to use custom tags at all and the docker-compose file should be directly usable with docker-compose.

Thank you for your prompt reply Kurt.
I think the plugin is 90+% of the way to serving my use case, I have a few questions about how things are intended to work that I'll ask through another issue to keep discussion here on the original topic (or if there's a Gitter or something that you'd prefer as a channel for questions, let me know).
There are some situations where the functionality currently provided by custom tags might be useful outside of multi-project builds, for instance I might already have the tagged image versions that I need pulled to my laptop and when I'm getting on a plane I want sbt-docker-compose to stop trying to hit the network to pull those images only to find the tag versions are still up-to-date locally. From what I can tell <skipPull> is a way to achieve that, even if it wasn't the intended use of it (?). There may be other ways to solve this, I'd probably prefer that such network pulls are avoided by default if a specific image tag is already available locally—as docker-compose seems to work—with an "always pull" option being the opt-in special case instead of "skip pull".
But regardless, multi-project use cases or not doesn't change whether labels might be a workable way to steer plugin behavior more portably. I think it's worth taking a stab at it.


Remove Range header after Seek to origin

When reading data using ReadAt() and seeking to the origin using Seek(0, 0) afterwards, the old Range header is still included in the request. This causes any subsequent Read() operation to start at the offset previously passed to ReadAt().
The commit in this PR fixes that behaviour.

@flobernd Thanks for the contribution. This seems to trigger some tests failures.
=== RUN   TestSetHeader
Error:     get-options_test.go:55: Test 5: Expected range header 'bytes=0-0', but got ''

This seems functionally different. bytes=0-0 would return 1 byte. So this should be investigated, if the test is wrong or the change.
=== RUN   TestGetObjectCore
Error:     core_test.go:206: Error: GetObject read shorter bytes before reaching EOF, want 946344, got 1

This also seems something that should be double checked.

@klauspost Let's try a different way :-)


Yatin: Watch agen138

Yatin requests the watch of the watch_keyword agen138. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

agen138 has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


There is a small gap under a maximized or full-screen window when "Automatically hide the taskbar" is enabled.

Windows Terminal version
1.16.10262.0
Windows build number
10.0.22621.1485
Other Software
No response
Steps to reproduce

Enable "Automatically hide the taskbar".
Enter full-screen mode.

Expected Behavior

Actual Behavior

This sounds like it may be the same issue as #12959.

Thanks for the bug report! We'll mark this as a /dup of #12959.


Not writing all entities in to_conll2003

How to reproduce the behaviour
I can't share the data because its confidential but some entities simply aren't written when using that function over documents!

Having pushed a little the analysis following the loss of many data, I realized that there were spaces included at the beginning or end of annotation empeding a correct tokenization and the associated labeling.


reintroduce state=NEW

Description
Changes proposed in this pull request.
Test plan
Outline the test plan used to test this change before merging it.

Fixes #.

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "flowlog_master",
    "component": "trireme-lib",
    "pr-id": "808",
    "commit-sha": "10988c18aeb3b6f76fa9b6d45661354a48d3f1b8"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "flowlog_master",
    "component": "enforcerd",
    "pr-id": "1212",
    "commit-sha": "d152778e4998f2338176ec1c2696e1d08b266356"
  },
  {
    "project": "flowlog_master",
    "component": "trireme-lib",
    "pr-id": "808",
    "commit-sha": "10988c18aeb3b6f76fa9b6d45661354a48d3f1b8"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "flowlog_master",
    "component": "enforcerd",
    "pr-id": "1212",
    "commit-sha": "d152778e4998f2338176ec1c2696e1d08b266356"
  },
  {
    "project": "flowlog_master",
    "component": "trireme-lib",
    "pr-id": "808",
    "commit-sha": "64edf2d3d148ab298840e5d0dbe5a542a88c01e0"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "flowlog_master",
    "component": "trireme-lib",
    "pr-id": "808",
    "commit-sha": "244e703327bb33328fa5fb2c03ece91caac72a01"
  },
  {
    "project": "flowlog_master",
    "component": "enforcerd",
    "pr-id": "1212",
    "commit-sha": "d152778e4998f2338176ec1c2696e1d08b266356"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "flowlog_master",
    "component": "trireme-lib",
    "pr-id": "808",
    "commit-sha": "244e703327bb33328fa5fb2c03ece91caac72a01"
  },
  {
    "project": "flowlog_master",
    "component": "enforcerd",
    "pr-id": "1212",
    "commit-sha": "d152778e4998f2338176ec1c2696e1d08b266356"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "flowlog_master",
    "component": "trireme-lib",
    "pr-id": "808",
    "commit-sha": "244e703327bb33328fa5fb2c03ece91caac72a01"
  },
  {
    "project": "flowlog_master",
    "component": "enforcerd",
    "pr-id": "1212",
    "commit-sha": "4be152641acc09f3471f328652aa0b4ed7bc0f3f"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "flowlog_master",
    "component": "trireme-lib",
    "pr-id": "808",
    "commit-sha": "459d449ff67fdc6f746da40aedd61f1c69ec7d8b"
  },
  {
    "project": "flowlog_master",
    "component": "enforcerd",
    "pr-id": "1212",
    "commit-sha": "4be152641acc09f3471f328652aa0b4ed7bc0f3f"
  }
]


Unable to decompile code

Getting the error below on running ghidra on any binary
----- Loading /root/Desktop/absolute/crackme02 ----- Failed to properly markup GNU Hash table at 00100298: ghidra.program.model.util.CodeUnitInsertionException: Insufficent memory at address 001002b4 (length: 36 bytes)   [libc.so.6] -> not found ----- [crackme02] Resolve 6 external symbols ----- Unresolved external symbols which remain: 6
Tried running ghidra on a vm (on the same system) but still getting the same error.
Unable to decompile or see the functions inside the binary.
OS: MacOSX (high sierra)
Java Version: 11

Probably because of ELFkickers 'sstrip' or something like it, try to manually recreate the ELF headers.

I wasn't able to reproduce this at present due to insufficient information. If there are any additional problems, please submit a new issue.


Upgrade Yarn v1 to Yarn v3

📝 Description
This Change upgrades Yarn.
Mainly for improving the experience in the containerized dev env
🎥 Demo

Please add a video or an image of the behavior/changes

/lgtm

/test kubevirt-e2e-aws

/lgtm

/lgtm

/retest


Unable to load ane/swc in Flash cs6

I have gone through the steps as shown in the read me.

Added the relevant tags to app xml.
Added ane to Flash in the library path.
It doesn't compile however and throws the error ' Unable to load ane'.

I tried with the swc but same result ie  ' Unable to load swc'.
I then replaced the dx.jar but the same thing happened.
Then I tried to check out the classes inside the ane and swc through Flash Develop but it read-
' Cannot find central directory'.
I have only added the ane(31 kb) and swc(29kb) to the extensionDirectory of my project. Is there any other file that needs to be added to the extensions directory?
There is a huge 80 mb ane file in the master directory. I tried with that too but Flash crashed as soon as I tried to add it to the library path.
Is there anything else I can do?
Thanks for your time.

I have successfully compiled with the swc. Now while packaging using adt I get the following errors.
Resource values ---common_strings xml has been skipped because of mismatch with supported languages information in app descriptor.

Got it to package by deleting most of the strings xml from the resources.

Greetings,
I'm trying to use the digicrafts ane in Flash CS6, and having this problem
Access of undefined property AdInterstitial
since you solved your problem, can you please tell me how..or maybe send me a simple sample fla file
Thank you a lot


solveEquation with negative power causes infinite loop

It turns out that the following equation x^-2 + x^-1 = x  causes a Math error: Potential infinite loop for equation: error. However, when I change the equation to 1/x^2 + 1/x^1 = x I don't get the error (FYI: 1/x^2 + 1/x^1 ==  x^-2 + x^-1).
here are some more trials where I ran into the same problem

x^-2 + x^-1= x
x^-2 + x^-1 + x^0 = x^1 + x^2
x^-2 + x^-1 + x^0 = x^2 + x^1
x^-1 + x^-2 + x^0 = x^2 + x^1
x^0  + x^-1 + x^-2= x^1 + x^2

I ran solveEquation with the debug flag, and it seems like the solver does not remember if it has already reached a step and loops forever.
Here is an illustration to tell you what I mean with that:
We are mathematically allowed to do the follow:

1 + 1 = x
1 = x - 1
1 + 1 = x
1 = x - 1
1 + 1 = x
1 = x - 1
...

I believe you get my point (infinite loop).
I suspect that solveEquation is stuck in such a case.

Hi! Thanks for reporting this :) Definitely not the behaviour we'd like haha - I know there are a few known ways things infinitely loop that are tricky to fix, but am not sure if this is one of those known issues.
I'm currently working on a conference that's happening in a few weeks and don't have bandwidth to look into this, but I encourage you to dig into the code and find more info about the issue or even a fix. I'd be happy to look into it in mid January as well.


retain other query parameters when stripping the token from the URL

Fixed

Retain other query parameters when redirecting to remove the token from the URL

ETH-877

Thanks, I was thinking about this, but forgot to mention it

I thought it was required for ETH-877, but it appears to work without this.


@types/express should be a devDependency

https://github.com/turt2live/matrix-js-bot-sdk/blob/fd950f606e76867d2bf3e8e10913755269ebea4d/package.json#L50

If it is, it doesn't build.

That smells like a bug to me.

It's how TypeScript works - the types need to be brought into the project that uses the SDK otherwise it can't build the SDK into the project's output.

Why are you trying to build the SDK in the project?

You aren't, typescript is. It takes the mappings from the bot-sdk and tries to hunt down the express mappings. Honestly this would all be fixed with #62 rather than trying to shove this specific dependency out of the way.


feat: add flag for developing dy admin apply

*Issue #155
Description of changes:
During development, to eliminate the impact on users, ensure that apply only works when --test-apply is added.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for trying to build new feature for dynein.
I understand that you are trying to create the feature in separate multiple pull requests. I think It is a reasonable tactics.
As my quick search, I found that hidden option of structopt may work for this kind of phased development. I hope this information helps to implement more user-friendly interface even in developing phase. https://github.com/TeXitoi/structopt/issues/466

memo
Failure:
ref:
https://docs.rs/clap/2.34.0/clap/struct.Arg.html#method.hidden
https://github.com/TeXitoi/structopt/issues/466
$ git diff
diff --git a/src/cmd.rs b/src/cmd.rs
index b9235a1..ba31311 100644
--- a/src/cmd.rs
+++ b/src/cmd.rs
@@ -429,7 +429,7 @@ pub enum AdminSub {
     },

     /// [WIP] Create or update DynamoDB tables based on CloudFormation template files (.cfn.yml).
-    #[structopt()]
+    #[structopt(hidden = true)]

$ cargo build
   Compiling dynein v0.2.1 (/Users/herotaka/Desktop/dynein)
error[E0599]: no method named `hidden` found for struct `App` in the current scope
   --> src/cmd.rs:432:17
    |
432 |     #[structopt(hidden = true)]
    |                 ^^^^^^ method not found in `App<'_, '_>`

Success:
ref:
https://docs.rs/clap/2.34.0/clap/enum.AppSettings.html#variant.Hidden
https://github.com/TeXitoi/structopt/issues/444
$ git diff
diff --git a/src/cmd.rs b/src/cmd.rs
index b9235a1..ed52d50 100644
--- a/src/cmd.rs
+++ b/src/cmd.rs
@@ -429,7 +429,7 @@ pub enum AdminSub {
     },

     /// [WIP] Create or update DynamoDB tables based on CloudFormation template files (.cfn.yml).
-    #[structopt()]
+    #[structopt(setting(structopt::clap::AppSettings::Hidden))]

Do not display apply subcommand in help is good idea but it is written in the README.
So I want to keep --test-apply flag so that even if the user executes it, it will not affect the user.


PUT requests don't work

Description
PUT requests appear to be broken.
Trying to use client.updateWatchlist complains about a malformed request.
It looks like there is a conditional in the request method that needs to be updated
CURRENT: if (params.method != 'POST' && params.method != 'PATCH') {
CHANGE TO: if (!['POST','PATCH','PUT'].includes(params.method)) {
Expected
A clear and concise description of what you expected to happen.
The watchlist should get updated properly.
Reproduction
Steps we can take to reproduce the bug:
Just try using client.updateWatchlist and it will generate an error from the server due to a malformed request.
Logs
If applicable, add logs to help explain the bug.
Additional
Add any other context about the problem here.

Thank you for finding the issue.


contour issue

Hi,
I am currently working with biopsy samples in Xenium. In my FFPE slides, I have 3-4 masses for each sample. After using the cv2_detect_contour function, I noticed that only one mass was successfully detected and contoured, while the others were not.
I know you suggested for contouring using coordinates, but unfortunately, the Xenium data does not include information about x-array and y-arrays.
Could you help me to handle this?

You can try this:
cnt_info=tesla.cv2_detect_contour(img, apertureSize=5, L2gradient = True, all_cnt_info=True)
binary=np.zeros((img.shape[0:2]), dtype=np.uint8)
for tmp in cnt_info:
	if tmp[2]>100:
		cnt=tmp[0]
		cv2.drawContours(binary, [cnt], -1, (1), thickness=-1)


RequestRender not firing the DoRender callback

When I use the D3D11Image control inside a Window directly, everything works fine. Calling the RequestRender method causes the DoRender callback to fire. However, if I use the D3D11Image control in a UserControl instead of a Window (the UserControl is in the Window), calling the RequestRender method will not cause the DoRender callback to fire.
Aside from moving to using a UserControl, all the code is the same between the two cases. Has anyone run into this before or gotten it to work from within a UserControl? I thought maybe the window handle for the D3D11Image was wrong but I verified it is correct.

For anyone who runs into a similar problem, I solved my issue. I was not setting the pixelsize properly when using the usercontrol.


Welcome email for new users

Do we want to have a welcome email for new signups ready?
If so this should be in a background worker for sending.

Assigning this to @DarrenMcRoy and @heathv.
For reference, the current one looks like http://i.imgur.com/plNdQDG.png. So we can't do much worse.

I can go ahead and write up some content for this, and then give it to @heathv to make it look pretty.
Will have something by the end of today or first thing Monday.

Adding it to the list.

Can you guys also look at #768 too?

I've started working on this here. @mrniaboc you might want to checkout the email content.

Hey @camallen -- sorry, confused, was going to work on this today (got delayed by other content).
Is there any actual content yet, or is this just the dev side of things to make whatever content we write appear?

Just the tech side, no copy / layout yet.

Cool. Thanks! Will really try to have that drafted by end of today.

First draft:
https://docs.google.com/a/zooniverse.org/document/d/1fu97oc0MLGBk_F8UcykOUz75yaO1oITbUgKV2rwc0gc/edit?usp=sharing
Thoughts?

I think this mailer content is ready to go?

closed via #1025


Consider CoC approach

I'm just starting with the tool, but after running my first examples the single most important thing that stroke me was the amount of boilerplate and configuration files the CLI generates.
My beloved Angular 1.x projects have a gulpfile.js and, some of them, may include an additional .json config file. Maybe a karma.conf.js. And that was it. And we are talking about projects using a complex, production-ready build system.
In a barebones project generated by angular-cli, you are presented with:

.clang-format
angular-cli-build.js
angular-cli.json
system-config.ts
tsconfig.json (x2!)
environment.ts.
environment.dev.ts
environment.prod.ts
karma.conf.js
karma-test-shim.js
protractor.conf.js

And that's on top of the regular configuration/linter files like package.json, tslint.json, typings.json, .editorconfig, etc. All of which should be versioned and maintained. There is, literally, more configuration here than actual implementation. It's rather overwhelming. Not to mention cumbersome.
Is there not a possibility here where we could benefit from using a Convention over Configuration approach?
Take angular-cli-build.js, for instance. What's its purpose? What's the need for this?
module.exports = function(defaults) {
  return new Angular2App(defaults, {
    vendorNpmFiles: [
      'systemjs/dist/system-polyfills.js',
      'systemjs/dist/system.src.js',
      'zone.js/dist/**/*.+(js|js.map)',
      'es6-shim/es6-shim.js',
      'reflect-metadata/**/*.+(js|js.map)',
      'rxjs/**/*.+(js|js.map)',
      '@angular/**/*.+(js|js.map)'
    ]
  });
};

All of that information is already available @ package.json. Same goes for angular-cli.json. It looks like its duplicating pre-existing metadata and configuring some stuff that's highly unlikely to change for most of us, tool's users.
Am I making any sense?

There was actually a discussion about this some time ago in https://github.com/angular/angular-cli/issues/443, can I ask you to comment there if you'd like to present arguments as to why some of these files should be removed?
I just merged https://github.com/angular/angular-cli/pull/902 which removed .clang-format (since it was recently removed).
The angular-cli-build.js file defines which files are actually copied over to a project build, as these aren't all the ones present in node_modules.


[Docs] Add information about domain specific user signup for Google OAuth2

Documentation Link
Relevant link that needs to be improved: https://docs.appsmith.com/third-party-services/google-login
Describe the problem
Currently our documentation doesn't have any indicators that Google sign in can be restricted to certain domains for self-hosted instances.
Describe the improvement
Information about the environment variable that controls specific domain signups can be configured needs to be added in Step 5. The environment variable to be added to docker.env file is : APPSMITH_OAUTH2_ALLOWED_DOMAINS. The value will be the domain. For eg, APPSMITH_OAUTH2_ALLOWED_DOMAINS=abc.com will restrict Google signups to email IDs belonging to abc.com.

Hi mohanarpit. Can I be assigned that change? This would be my first contribution to hacktoberfest; hoping this is right first step to first ask being assigned to that issue. Let me know.
Thanks!

@fredb1106 It's all yours! 🎉
Please don't forget to read the Contribution Guidelines. Would appreciate if you can open a PR within the next 2 days.
Also, check out the Appsmith Challenge where you can win some cool swag by inviting your friends to contribute to this project as well.

Sure, will work on this today! By looking at your code I noticed APPSMITH_OAUTH2_ALLOWED_DOMAINS was also missing from some sample docker.env files or Shell scripts to generate them e.g.:

appsmith/app/server/envs/docker.env.example
appsmith/deploy/template/docker.env.sh

... and potentially those also:

appsmith/.env.example
appsmith/app/server/envs/dev.env.example

I can add that parameter to those files as well while we're at it.
Also I can see that from the below 2 files that the domains list can be a comma-separated list so I can mention that as well.

appsmith/app/server/appsmith-server/src/main/java/com/appsmith/server/configurations/CommonConfig.java (line 54)
appsmith/app/server/appsmith-server/src/main/java/com/appsmith/server/authentication/handlers/CustomServerOAuth2AuthorizationRequestResolver.java (line 165)


ssl/t1_lib.c: Add checks for the EVP_MD_get_size()

Add checks for the EVP_MD_get_size() to avoid unexpected negative numbers.
Fixes: b362ccab5c ("Security framework.")
Fixes: 0fe3db251a ("Use size of server key when selecting signature algorithm.")

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge

merged, thank you for your contribution


Fix error on Python 3.11

Due to a change in the Enum specification in Python 3.11, the following code will cause an error:
from enum import Enum

# pyrtmp/amf/types.py
class AMF0(int, Enum):
    NUMBER = 0x00
    BOOLEAN = 0x01
    STRING = 0x02
    OBJECT = 0x03
    NULL = 0x05
    ARRAY = 0x08
    OBJECT_END = 0x09

# pyrtmp/amf/serializers.py
"0000{:02x}".format(AMF0.OBJECT_END)

>>> "0000{:02x}".format(AMF0.OBJECT_END)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.11/enum.py", line 1227, in __format__
    return str.__format__(str(self), format_spec)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: Unknown format code 'x' for object of type 'str'

This patch fixes the problem by first casting the value of AMF0 to int.

Do you know if this is backward compatible with other python versions?
Thank you for your contribution.

I ran the edited line in Python 3.6.15 and 3.9.16 (on REPL) and got no errors, so I assume it is at least backwards compatible with that versions.


trigger material out

after_manipulate trigger has been added on: 5e99a029123755f48953fdea04681b620830e20e
before_manipulate not yet

based on #17, add where deleted_at IS NULL on material_in_details_*_triggers


[CARBONDATA-3775] Update materialized view document

Why is this PR needed?
Update materialized view document synchronously,  after the materialized view is separated from the DataMap module.
What changes were proposed in this PR?

Update materialized view syntax comment.
Add comment about usage of time series.
Move document to document root directory from index directory.

Does this PR introduce any user interface change?

No

Is any new testcase added?

No

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2772/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1059/

@niuge01 Please check and fix the link for mv in README.md file also

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2784/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1071/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1080/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2792/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1090/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2802/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1096/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2809/

@niuge01 Please rebase

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2814/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1101/

@Indhumathi27 I have rebased.

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1103/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2816/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1108/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/2821/

LGTM


auctions - count display on bottom tab navigator

What would you like to be added:

Why is this needed:

Acceptance Criteria: Auction count display
Android:

[ ] Balances > Auction count should display
[ ] DEX > Auction count should display
[ ] Loans > Auction count should display
[ ] Transactions > Auction count should display

iOS:

[ ] Balances > Auction count should display
[ ] DEX > Auction count should display
[ ] Loans > Auction count should display
[ ] Transactions > Auction count should display

Auction

When counter goes to 3 digits, it’s not in the center of red oval

Expected:
• Number count displayed should be centralized

Issue created: https://github.com/DeFiCh/wallet/issues/1580


[cross_file] Migrate to null-safety.

This PR migrates cross_file to null-safety.
According to our tests (which didn't require any significant changes), this seems to be an API compatible change too:
~/github/plugins/packages/cross_file$ flutter test
00:01 +10: All tests passed!
~/github/plugins/packages/cross_file$ flutter test --platform=chrome
00:04 +11: All tests passed!

Fixes https://github.com/flutter/flutter/issues/74541

Pre-launch Checklist

[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I signed the CLA.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

/cc @mvanbeusekom @tugorez if you have time, can you guys please give this new version of the package a quick test in camera/file_selector to see that it still does what it's supposed to do in mobile/web? All tests seem to pass, so you should be able to test this just by changing the dependency of the package! Thanks!!

/cc @mvanbeusekom @tugorez if you have time, can you guys please give this new version of the package a quick test in camera/file_selector to see that it still does what it's supposed to do in mobile/web? All tests seem to pass, so you should be able to test this just by changing the dependency of the package! Thanks!!

This should be incorporated into this other PR:
https://github.com/flutter/plugins/pull/3388
Since #3388 is not backwards compatible, and it needs to bump the major version, it could be migrated to null-safety at the same time!
(/cc @cyanglaz who was migrating image_picker to null-safety)

This should be incorporated into this other PR:
https://github.com/flutter/plugins/pull/3388
Since #3388 is not backwards compatible, and it needs to bump the major version, it could be migrated to null-safety at the same time!
(/cc @cyanglaz who was migrating image_picker to null-safety)

OK, I think I finally got all the changes for the build_all_plugins script to correctly build this. (Not sure why 'file_selector' was not an issue before)

OK, I think I finally got all the changes for the build_all_plugins script to correctly build this. (Not sure why 'file_selector' was not an issue before)

/cc @mvanbeusekom @tugorez if you have time, can you guys please give this new version of the package a quick test in camera/file_selector to see that it still does what it's supposed to do in mobile/web? All tests seem to pass, so you should be able to test this just by changing the dependency of the package! Thanks!!

Hi @ditman, thank you for this awesome work. I wanted to give this a test in the camera plugin but realised that to do so I need to migrate the camera plugin also to null-safety (since I need to upgrade the Dart environment to >=2.12.0-0, which by default means I opt-in to null-safety). Is this correct, or am I overlooking something?
If this is the case, quickly testing becomes a bit of a bigger project which I am happy to do at the start of next week (this week I am on holiday and don't want to rush it because the kids are requesting my attention ;))

/cc @mvanbeusekom @tugorez if you have time, can you guys please give this new version of the package a quick test in camera/file_selector to see that it still does what it's supposed to do in mobile/web? All tests seem to pass, so you should be able to test this just by changing the dependency of the package! Thanks!!

Hi @ditman, thank you for this awesome work. I wanted to give this a test in the camera plugin but realised that to do so I need to migrate the camera plugin also to null-safety (since I need to upgrade the Dart environment to >=2.12.0-0, which by default means I opt-in to null-safety). Is this correct, or am I overlooking something?
If this is the case, quickly testing becomes a bit of a bigger project which I am happy to do at the start of next week (this week I am on holiday and don't want to rush it because the kids are requesting my attention ;))

I wanted to give this a test in the camera plugin but realised that to do so I need to migrate the camera plugin also to null-safety (since I need to upgrade the Dart environment to >=2.12.0-0, which by default means I opt-in to null-safety)

@mvanbeusekom I was expecting that you'd be able to set the dependency and use the package straight away, if it requires any bigger effort, don't even bother! :)

I wanted to give this a test in the camera plugin but realised that to do so I need to migrate the camera plugin also to null-safety (since I need to upgrade the Dart environment to >=2.12.0-0, which by default means I opt-in to null-safety)

@mvanbeusekom I was expecting that you'd be able to set the dependency and use the package straight away, if it requires any bigger effort, don't even bother! :)


Include submission title in comments page  HTML tag

Hi, great app!
It would be really helpful if the comments title were set as the title of the submission.  It seems like it would be simple to use the title from the hacker news API instead of the current title on each comments section, "Hacker News".
This would also mirror how hacker news does its own comments and help my tabbed browsing tremendously. Thanks!!

Here is a sample title tag from a Hacker News submission:
My Top 100 Programming, Computer and Science Books: Part Four | Hacker News
https://news.ycombinator.com/item?id=9835285


[FORMATS-127] Use contigName instead of contig

Fixes #127

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/bdg-formats-prb/152/
Test PASSed.

This seems fine to me.
General question - are we able to merge this into Master whenever we like or do we need to coordinate with downstream dependencies?

#131 and #132 are ready to go


VS Compilation

This is in progress.

I am closing this because we have compilation under different environment/compiler combinations in the latest master and I will reopen if a specific problem is identified.


Add format string for the assertion

ref #856
Add a custom assertion error message.
Should be useful for debugging.

the windows build error seems to be permissions issue. i'm going to force merge this


move to monorepo

CC @joshblack We're planning on executing the move to the carbon monorepo in November ... We can use this issue to track any prep work we need to do or things we should be aware of so we can have a smooth transition.

@cal-smith whats the motivation for moving into the monorepo?

@vpicone Hey Vince ... I'm sure most of the motivation should be self evident but:

simplifies keeping pace with changes to carbon-components - ideally a structure or css change in carbon-components should be propagated to react/vue/angular in the same (or a related) PR
centralizes issue logging for users - right now any cross implementation issue needs to be opened in several places
should ensure library support in the event of a maintainer change - This is more of a hope, but given that carbon seems to be here for the long run the less places things are stored the better
simplified contribution process (especially in the case of cross implementation changes)
can take better advantage of shared tooling

I'm sure there's more that I'm missing, but that should cover the ones that immediately come to mind. On the whole it's a net benefit for the ecosystem, our contributors, and our users.
Let me know if you have any concerns 🙂 And don't worry - we won't need to special case the build process for this project 😉

@cal-smith thanks! Since none of the Carbon team members are experienced with Angular or familiar with this project some of those points don't exactly track for me. Since we won't be contributing to the Angular package, we won't exactly be able to take advantage of any of the 'keeping pace' or simplified contribution advantages.
In regards to "should ensure library support in the event of a maintainer change," this is actually my primary concern. By bringing this package into the mono-repo it implies the Carbon team is at all responsible for the maintenance of this package (which we're definitely not prepared to take on).

Since we won't be contributing to the Angular package, we won't exactly be able to take advantage of any of the 'keeping pace' or simplified contribution advantages.

I'm hoping we can change that. A framework shouldn't be the inhibiting factor for contribution. And while members of the Austin carbon team may choose to not contribute, the simplified contribution flow is absolutely relevant for all other contributions.

By bringing this package into the mono-repo it implies the Carbon team is at all responsible for the maintenance of this package (which we're definitely not prepared to take on).

I feel like this deserves a huge [citation needed] ... To flip the issue around - what happens if my team disappears right now? Does Carbon stop supporting Angular? There are too many teams within IBM that rely on the maintenance of these components to continue taking that risk. At present we don't expect to be dissolving as a team any time soon, but by moving the project into the monorepo we can take advantage of all the infrastructure and policies around that.
I want to be crystal clear here - Angular is a first class citizen in the Carbon ecosystem. The sheer inertia behind us means that the Carbon organization as a whole is responsible for it's continued progress.

I imagine the product teams using the library would rather pick it up than have it live in the monorepo of a team with no experience with Angular, let alone the library itself.
Perhaps moving the project to the carbon org would be a better way to associate it without the additional complexity/maintenance burden of integrating it into the monorepo?

I suppose the big question that's emerging here is: How do we support a multitude of frameworks, produced by a multitude of geographically diverse teams, shipping carbon in the least complex and maintenance heavy fashion possible?
Ideologically the monorepo solves a huge number of issues when it comes to the maintenance and collaboration burden of a design system such as carbon - however I've now heard from at least 2 other members that the monorepo simply adds complexity ... if we don't have the bandwidth to tackle the tooling required to make a monorepo an effective solution for cross team collaboration then I'm no longer sure it's a reasonable answer to the above question.
So how do we do it? Ultimately this is one part technical and one part social - On the technical front tooling such as the compliance test suit and perhaps some unified build utilities can help manage our growing universe of packages. On the social front some more deliberate efforts to communicate and cross contribute could help - pretending that we don't work on the same platform helps nobody, pretending that we can't contribute to other projects helps nobody. I don't believe for a second that framework choice is the limiting factor here - we're not the bad guys, we're just like the rest of you.
I realize this isn't a direct answer, but between this and the icons-angular issue this has clearly moved beyond it's original scope.

Big side note here: For a long time the monorepo was positioned as the solution to carbon packaging and contribution. The assumption was the tooling would fall in place and everyone would become part of a single repository - even all the addon packages. I was unaware that the stance on the monorepo had changed (hence this issue) - again, communication is key.

@cal-smith if I gave the impression I was signaling a shift in Carbon’s official stance, that certainly wasn’t my intent. Just trying to get some more color around my concerns with this effort.

Perhaps in the future, considering being abundantly clear that you're looking to express your personal opinion.
Let me know if you have any concrete issues with this proposal, or feedback on how we can make the process easier.

Hey @cal-smith I'm going to schedule a meeting for us to discuss next week.

Sure :+1: I had some time blocked out with Vince and Josh next week, so we can use that time instead if you want?

After some further discussion we've tabled this for now ... we'll definitely investigate moving to the carbon-design-system org, but there's no clear appetite to commit to the level of work that the monorepo will require if we were to move in.


Parameter Value with -(dash) is not correctly deploying

we are doing a serverless deploy and passing parameter values to it..and one of the value starts with -(dash), reading from environment properties of Deploy Job.
But post deployment in parameters we see this value as true..rather than real value..
for eg -
serverless deploy --CLOUDANT_API_KEY_VALUE $CLOUDANT_API_KEY where CLOUDANT_API_KEY  is like -khu1232we
And if i go to the parameters of my action, it shows value of CLOUDANT_API_KEY_VALUE as true.
I tried putting /(backslash) before the value with and without doublequotes, then it doesn't show true but adding some backslash.

I'm unable to find where in the framework documentation this method of passing parameters is supported?
Reading the docs on variables, setting the env parameter CLOUDANT_API_KEY_VALUE in the shell allows you to import it into the serverless.yml as follows...
service: example

provider:
  name: openwhisk

functions:
  hello:
    handler: handler.hello
    parameters:
      CLOUDANT_API_KEY: ${env:CLOUDANT_API_KEY}

plugins:
  - serverless-openwhisk

Does this fix your issue?

I am using the same way in serverless.yml, below is the example..but still issue is not resolved. Temporarily  we are using the key that doesn't start with dash. But now again we have similar issue with some other variable.  Do we have any solution to fix this?
service: service_name
Sample serverless.yml
provider:
name: openwhisk
runtime: nodejs:8
functions:
resources:
packages:
PACKAGE_NAME:
parameters:
CLOUDANT_APIKEY: ${opt:CLOUDANT_API_KEY_VALUE}
plugins:

serverless-openwhisk

Just on top the above message, there is another observation that any value that has dash in it..is not getting deployed correctly and giving true value rather than actual value. For eg value -
PG.J4XdPBYORf--8oMODZ0eVA.PGUnaGV8T-qymbIWEIe_ccV2UifQROFiEdC-JwIygYg


Please create an outside collaborator pull request

usernames
weblate
names
Weblate (bot)
emails
<EMAIL_ADDRESS>org
WeblateOrg
reason
To allow the Weblate App to raise pull requests for translations on a demo translation project
added_by
<EMAIL_ADDRESS>review_after
20/02/2025
permission
pull
repositories
https://github.com/ministryofjustice/opg-weblate-demo-translation-project

Added opg-weblate-demo-translation-project file to the GitHub collaborators repo and merged.


Day-2-Morning-Session-Code-Review

Good job so far! i like how you have made use of variables in your project adhering to the rule of DRY.

Thanks, I appreciate the feedback :+1:


use new 'neo4j dry-run' mechanism to start neo4j process in 4.3

n.b. takes care not to run neo4j as root

We're reasonably sure that this should not be merged until drop-04 has been accepted


[Enhancement] Add support for semantic token highlighting.

Enhancement
It would be nice if onedarkpro.nvim can offer syntax highlights with the help of LSP.
Semantic token highlighting has been merged into the master branch of neovim https://github.com/neovim/neovim/commit/9f035559defd9d575f37fd825954610065d9cf96.

Completely agree. This is was raised by @mmirus in https://github.com/olimorris/onedarkpro.nvim/discussions/82#discussioncomment-3500196 and something that we will definitely incorporate.
Reading through the commits, it looks like some new highlight groups are by default, linked to vim syntax groups.
--LSP semantic tokens
"default link @class Structure",
"default link @struct Structure",
"default link @enum Type",
"default link @enumMember Constant",
"default link @event Identifier",
"default link @interface Identifier",
"default link @modifier Identifier",
"default link @regexp SpecialChar",
"default link @typeParameter Type",
"default link @decorator Identifier",

So should be very easy to incorporate. Also, that will give us some absolutely incredible highlighting. I'm running nvim 0.8 at present so if someone wants to be beat me to it, I'll be happy to accept a PR.

I've just had a play with Neovim 0.9 (no config breaking changes so far btw). I see there's a PR for Treesitter Playground which will make it easier to identify which tokens are being used for what.

I've just had a play with Neovim 0.9 (no config breaking changes so far btw). I see there's a PR for Treesitter Playground which will make it easier to identify which tokens are being used for what. Propose we wait for that before adding support.

All right. but having to install Treesitter to make use of semantic tokens seems to be a little bit unnecesarry. Hope some kind guy can give a PR so that we can enjoy that feature directly with LSP :)

All right. but having to install Treesitter to make use of semantic tokens seems to be a little bit unnecesarry. Hope some kind guy can give a PR so that we can enjoy that feature directly with LSP :)

No you don't it's built into the core of Neovim and as long as your LSP server supports semantic tokens, you'll get the highlighting.
Treesitter Playground is a separate plugin which has functionality allowing you to identify which highlight groups Treesitter is applying to your code. There's a PR to extend it so it can do the same for LSP semantic tokens. I want to use Playground to allow me to see where Neovim applies the @class semantic token, for example, and therefore what color it should be in the theme. I could probably guess but I suspect the PR will be merged in the next couple of days.

All right. but having to install Treesitter to make use of semantic tokens seems to be a little bit unnecesarry. Hope some kind guy can give a PR so that we can enjoy that feature directly with LSP :)

No you don't it's built into the core of Neovim and as long as your LSP server supports semantic tokens, you'll get the highlighting.
Treesitter Playground is a separate plugin which has functionality allowing you to identify which highlight groups Treesitter is applying to your code. There's a PR to extend it so it can do the same for LSP semantic tokens. I want to use Playground to allow me to see where Neovim applies the @class semantic token, for example, and therefore what color it should be in the theme. I could probably guess but I suspect the PR will be merged in the next couple of days.

Well I thought the playground was what I had to install to enable the highlighting, sorry for the misunderstanding. Now we just wait for that PR the be merged :)

Quick update on this:

Semantic tokens are now merged in Neovim 0.9 as per #21100
We now have the :Inspect command in Neovim 0.9 which shows the origin of highlight groups, be them Treesitter or semantic tokens, as per: #21393
The issue of modifiers (brilliantly described in #21576) is still to follow

Implementation of the tokens is a simple case of adding in the new highlight groups as indicated above. I'll give this some time as want to do some reconciliation back to VS Code but will likely implement in the next month.

In 0e641816457ab32cfc8c64bb59056a585619bf2f I've started to define semantic tokens. I've been restrained in the tokens I've defined so far (matching to sumneko_lua in VS Code) and the results are impressive:
Before

After

Visual Studio Code

I'm planning on overhauling the way styles are applied in the next week or two.

In 0e64181 I've started to define semantic tokens. I've been restrained in the tokens I've defined so far (matching to sumneko_lua in VS Code) and the results are impressive:
Before

After

Visual Studio Code

I'm planning on overhauling the way styles are applied in the next week or two.

I updated to the latest commit and it works like a charm. Thanks for the work!

I'll close this for now but expect to add more highlights over time


Use a Set instead of an Array to store key presses.

Key presses were not being removed properly if the key was pressed for more than a second or two, so it behaves as though the keys are pressed down forever. Storing them as a Set is also semantically more correct.
A simple test case
import Prelude
import Control.Monad.Eff.Console (logShow)
import FRP.Event (subscribe)
import FRP.Event.Time (animationFrame)
import FRP.Behavior (sample_)
import FRP.Behavior.Keyboard (keys)

main = do
  _ <- subscribe (sample_ keys animationFrame) logShow
  pure unit

Thanks for doing this, @ericnething. I've spent the whole day in bed with the 'flu, and have just summoned enough energy to catch up on emails. Sorry for being incommunicado.
I have no idea what @paf31's plans are, in regards to browser support, but it seems that the Set type is available (enough) after IE11 for our usage. Is that something we could consider? My gut feeling, considering my experience with the core modules, is that we're probably best to stick with the method in this PR, just to be safe, but I thought it were worth mentioning.

@i-am-tom I've been looking at javascript's Set recently, but it may be more trouble than it's worth, particularly for this use case. If we had full ES6+ support with compilation down to ES3 (or similar) then it browser support wouldn't be an issue.

@ericnething Wow, sorry for this unforgivably-slow reply. Certainly forget my idea; unless you can use the PureScript Set, the ES one is going to be more effort than it's worth. I certainly hope I'm not impeding the progress of this patch, @paf31

@i-am-tom not a problem at all! In fact, I had forgotten about this PR until you reminded me. I'll get this finished up so we can close it. I also found a dead code elimination bug that is somewhat related to this as well. I'm not sure if that should be a fix here in the library or a fix in the compiler, but there is a wa yto fix it in the library pretty easily.

Hah, I have run into the same bug a few times myself! I'll go subscribe to that one, too.

This appears to have gotten stale. I'm going to close this but we can try again with another PR if we can address the questions above. Thanks!