Give the frontend a proper design/look, rather than being just plain old html

Again, I'm primarily a backend programmer, not a designer or frontend guy :).  Any help very gratefully accepted.

Can I start working on this ?

@aditodkar I think it's safe to say on behalf of @justinccdev - yes please!

Hi @aditodkar.  As discussed, sorry, not supporting Windows on this project.

Can I take over this issue? @justinccdev @yochannah

@NirmalKumarJoshi go ahead!

Hey @NirmalKumarJoshi, yes contributions very welcome. Any kind of design improvement is welcome.  I do want to keep the front page quite Google-like right now (i.e. just a single text box for search) but I think there's a bit that could be done, such as the search results (e.g. when you search for 'data' the 4 results are quite hard to read).
If you don't want to build the database first from the (crawler project)[https://github.com/justinccdev/bsbang-crawler] then I can sort out a Solr dump so you don't need to do this.  Please let me know if you want this.

Thanks @justinccdev and @yochannah for your recognition and response. I have started improving the UI and also framed certain mock0ups as per the requirements.
I will update you soon.

I've also put some more information on what I think I'm looking for in #3


fix: deprioritize constructors marked as obsolete

Closes #125

Codecov Report

Merging #126 (9f6870a) into main (cab0411) will increase coverage by 0.00%.
The diff coverage is 100.00%.

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

@@           Coverage Diff           @@
##             main     #126   +/-   ##
=======================================
  Coverage   89.73%   89.73%
=======================================
  Files          74       74
  Lines        2415     2417    +2
  Branches      305      305
=======================================
+ Hits         2167     2169    +2
  Misses        181      181
  Partials       67       67

Impacted Files
Coverage Δ

...Builder/NewInstanceObjectPropertyMappingBuilder.cs
85.36% <100.00%> (+0.18%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


RFC static to support MultinomialTS

Support of HMC{LastTS} and HMC{MultinomialTS}

@yebai I pointed out the two main changes. Other changes are mostly rename of files and moving around codes.

There is some weird thing happening to the folder name on git. I will try to fix them before merging.

Here a more informative name is Trajectory, I think. See also: #103

I agree. I think it should be StaticTrajectory. For the same reason, NUTS should not be existi, but just DynamicTrajectory with the type of stoping criteria specified. The current change is just to make static ones consistent with dynamic ones but I will have another PR to change them together.

@yebai Are you happy to merge this?

I agree. I think it should be StaticTrajectory. For the same reason, NUTS should not be existi, but just DynamicTrajectory with the type of stoping criteria specified. The current change is just to make static ones consistent with dynamic ones but I will have another PR to change them together.

Yes, ready to merge. Before merging, perhaps revert the changes to StaticTrajectory before doing your proposed refactoring? This is because the current PR introduce a temporary breaking change, which might be confusing.

Before merging, perhaps revert the changes to

Sure. I will do it.


Revisions for the new --help

Fixes #44798

What's the before/after look like?

There is 3 ways blue can show:

Terminal supports 256+ colors. We don't use the terminal theme blue, but use an ANSI constant which is close to TS blue, which works on dark/light terminals:

https://jonasjacek.github.io/colors/ - SteelBlue3 ANSI 68

Terminal supports 16 colors, this is windows console / powershell
Is it Windows? Then use "Bright Blue"
Is it not Windows? Then use "Blue"

@typescript-bot pack this

We don't use the terminal theme blue, but use an ANSI constant which is close to TS blue, which works on dark/light terminals:

For something with a pure white background (e.g. the default MacOS terminal theme) how does that look? On a white background, I'm getting a contrast ratio of 3.53 which is still fairly low.
Seems like if bright blue is this on CMD

and this is bright blue on PowerShell

that might work, though it's a bit off from brand colors.

This seems to set the background color to blue.

and uh

hahaha, well, I guess I need to get a windows VM up and running 🍡

This is your big chance to tell us how you feel about Windows 11. 😅

Hah, well it does look like Windows 11 doesn't ruin the UI by trying to make it touch focused - I believe that was a mistake made back in Windows 8 which I wish Apple had also learned. That said, the overall design of Win 11 feels like it falls into the same tailwindification design style that's pretty much everywhere on the web too, it looks nice but it's all just a bit samey. For OS design though, I can see that that can be an advantage.
Anyway to this PR, I got up a few VMs and explored a bunch of options. In the end, I just dramatically simplified everything and took out almost all cases of being clever. We use 'bright blue' for blue everywhere except powershell/command prompt. This has a solid contrast
'strong blue' in default themes for terminal/vscode:

'strong blue' fails in powershell though:

While I'd love to be able to target just powershell, I couldn't see any ENV differences between the two, and so they both get 'strong cyan' instead:

What are your thoughts on just using a brighter white if 256 color isn't available?

I'm not offended, will switch it now

@typescript-bot pack this

There are three different color rendering setup now: plain text, plain text + bold, color:

I've switched to have NO_COLORin defaultIsPretty and added a test to verify the plain-text-ness of the output

@typescript-bot pack this

Requests merged, I'm going to get this in so that it doesn't get overlooked before the RC


Failed to initialise Terasology java.lang.RuntimeException: Can not initialize graphics device.

What you were trying to do
First time start this game
What actually happened
Error message and then game close
How to reproduce

Step 1
Take MSI GT72s 6QD Dominator laptop =)
Step 2
Run last stable version of Terasology

Log details and game version
terasology_2017-03-13_17-59-48.872.txt
Computer details
MSI GT72s 6QD Dominator

Update Nvidia drivers to last version (378.78) and problem disappeared

Probably duplicate of #2790 and #2802
That driver version crashed a bit more than Terasology :)

I think that this issue can be closed?

If it is solved with latest drivers this can be closed indeed. Thanks for reporting and reporting back on it!


Handle standalone .scss files

Type of Issue

[x] Feature Request

Description
How to handle .scss files that are not part of a component? An example would be the styles.scss file in an angular-cli project
How To Reproduce
If you build a folder that includes a unimported .scss file it will not show up in the dist folder
Expected Behaviour
The .scss file, or a compiled version of it, should show up in the dist folder

Hi @BioPhoton,
you can copy those "static" assets to the dist folder in a separate build steps.
Currently, there are no plans to include hooks for such assets that are not referenced by/included in the Angular package. Such "out-of-band" data should be handled independently of ng-packagr. See also #123


Is there any way to use this plugin with render prop pattern instead of HOC?

Is there any way to use this plugin with render prop pattern instead of HOC?

Can you show a use case in which this would be useful? (and other established patterns are insufficient)

React, { useState } from "react";
import onClickOutside from "react-onclickoutside";

const Menu = () => {
  const [isOpen, setIsOpen] = useState(false);
  const toggle = () => setIsOpen(!isOpen);
  Menu.handleClickOutside = () => setIsOpen(false);
  return (
      //...
  )
};

const clickOutsideConfig = {
  handleClickOutside: () => Menu.handleClickOutside
};

export default onClickOutside(Menu, clickOutsideConfig);

VS
React, { useState } from "react";
import onClickOutside from "react-onclickoutside";

const Menu = () => {
  const [isOpen, setIsOpen] = useState(false);
  const toggle = () => setIsOpen(!isOpen);

  return (
      <OnClickOutside onOutsideClick={() => setIsOpen(false)}>
        ...
      </OnClickOutside>
  )
};

It will also solve https://github.com/Pomax/react-onclickoutside/issues/310 . @Pomax what do you think?

You mean can there be a normal Component that takes care of the wrapping as part of the exports in addition to the HOC? If so, I don't see why not, as long as it has a normal component name: onClickOutside is a function that yields an anonymous component, so if you want an explicit component we'd need to export something like OutsideClickMonitor or InnerClickIgnore or something.
Depending on how much code needs to be changed to effect that, of course.

I created such a Component since I came across #310 as well.

That'd be great - as long as it comes with an update to the tests that uses it of course, because it'll need a baseline against which regressions can be found =)

I wonder why this wasn't like this from the beginning... Geniuine question, not a rhetorical one. I'm curious.

Mostly because at the time this hoc was invented, around React 0.12, none of the syntax and functionality required to write that kind of code existed in JS.


jubamodel: add model transformation

While investigating or debugging models, sometimes I want to see raw results from its backend engine.  For example, in NN-based classifier, it is hard to tell how scores are calculated, because we cannot see actual distances between registered points.
In such case, it is convenient if the backend NN model can be extracted from NN-based classifier model like this:
$ jubamodel --transform nearest_neighbor -o binary -O nn_model.jubatus classifier_model.jubatus

Then we can load the nn_model.jubatus file into jubanearest_neighbor and throw query like neighbor_row_from_datum to compute distances between points.
So, I propose adding --transform <service_name> option, which transforms the input model into desired backend model.  Possible transformations may be:

classifier, regression, recommender, anomaly, clustering -> weight
classifier(NN), regression(NN), anomaly(light_lof) -> NN
anomaly(lof) -> recommender

Prototype: https://gist.github.com/kmaehashi/d85662929c3e2698daa9845fc516d1c1

implemented via #83


Volume / Change incorrectly displayed as -

Description
Console is "flip flopping" on volume and change displayed within some of our e2e tests.
After investigating I can see that volume of 1 is coming through in graphql but console is incorrectly displaying -
I have seen similar behaviour for the funding rate and index price
Steps to Reproduce

Run market-sim test test_market_selector_filter
See error

Expected behaviour
Volume and change is correctly displayed
Screenshots

If the candle is older than 24hrs then console should display - but it should have a tooltip with the change from the last 5 days.


tasks: fix handling of instrument dependencies in interfaces

Interfaces were not properly adding the instrument dependency to their dependency when instruments were specified.

Codecov Report

Merging #88 into master will increase coverage by <.01%.

@@            Coverage Diff             @@
##           master      #88      +/-   ##
==========================================
+ Coverage   98.26%   98.27%   +<.01%
==========================================
  Files         155      155
  Lines       12312    12317       +5
==========================================
+ Hits        12099    12104       +5
  Misses        213      213

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 bc84174...94d4d15. Read the comment docs.


dh/url support cab

This PR is created to fix issues which I committed on #5700

@mwrock @lamont-granquist re-review?


Cross Contract Calls: near.promiseResult(id) Error Message Unhelpful

Issue
The SDK currently only informs the caller that extracting a promise result has failed, with no further information from the other account that ran the code yielding the error. Here's an example:
...
    const promise = NearPromise.new(anotherContractId)
      .functionCall("other_contract_function", JSON.stringify({
        param1,
        paranm2,
      }), 0, this.GAS_FEE)
      .then(
        NearPromise.new(near.currentAccountId())
        .functionCall("internalCallbackFunction", JSON.stringify({}), 0, this.GAS_FEE)
      );
    return promise.asReturn();
}
...

internalCallbackFunction would extact the result as follows:
  @call({ privateFunction: true })
  internalCallbackFunction() {
    try {
      const promiseResult = near.promiseResult(0);
    } catch (e) {
      near.log("PR ERR", e, e.message, e.stack);
      throw new Error(`Promise result error: ${e.message}`);
    }
...

In this case, if the call to anotherContractId has failed, we get the following in the logs:
Contract logs from escrow.test.near.purchase_in_escrow({"seller_account_id":"bob.test.near","asset_contract_id":"dev-11656.test.near"}) call: [
  'PR ERR {} Promise result Failed     at promiseResult$1 (build/escrow.js:542)\n' +
    '    at internalPurchaseEscrow (build/escrow.js:1347)\n' +
    '    at apply (native)\n' +
    '    at <anonymous> (build/escrow.js:583)\n' +
    '    at internalPurchaseEscrow (build/escrow.js:1510)\n'
]

In reality, that call could have failed for a myriad reasons and we are left unaware of them. This current set up forces the developer to always pass an object and avoid throwing errors for calls that will be called by another account program/smart contract.
Proposal
For the error to be passed to the caller such that appropriate logic can be implemented.

Hi @idea404 ! Do you mind to post this in summary on https://near.social/#/devgovgigs.near/widget/IdeasList ?
There we can submit and discuss possible solutions in near social!

This is unfortunately not possible due to vm function promise_result doesn't return data when promise result is failed, only return data when promise result succeed: https://github.com/near/nearcore/blob/e8bda2381b1990fc37e04ca2ecd351963cf9997c/runtime/near-vm-logic/src/logic.rs#L2046
(see the docstring of above function)
Therefore, as a wrapper to promise_result, promiseResult cannot give error message of the failure reason (e.g. throw Error("some messagfe") provided by the callee), isn't available in the caller.
I think current practice is to design callee in a way that reduce failure cases or let failure reason determined/not important to the caller. If you are sure that promise fail reason has some important use case, we may propose a NEP to core team.


ExecuteQueue doc: support rvalue task

What problem does this PR solve?
Issue Number:
Problem Summary:
What is changed and the side effects?
Changed:
Side effects:

Performance effects(性能影响):

Breaking backward compatibility(向后兼容性):

Check List:

Please make sure your changes are compilable(请确保你的更改可以通过编译).
When providing us with a new feature, it is best to add related tests(如果你向我们增加一个新的功能, 请添加相关测试).
Please follow Contributor Covenant Code of Conduct.(请遵循贡献者准则).

LGTM


Keymap: toggleLineCommentsInSelection  small improvement

Really small UX improvement for toggleLineCommentsInSelection: after commenting/uncommenting text, cursor will move down, so you can continue commenting the following line.
Inspired by PyCharm

I'm not sure how I feel about this - I kind of like it how it is currently.

I don't want automatically moveDown, If I tell to comment-out, just want to do Atom that simple task, I basically don't want to things automatically when I said to do so.
I think this feature can achievable by using  atom.commands.onDidDispatch hook.
So maybe package to provide auto-move-dowon-on-toggle-line-comment is good as start point in my opinion though.

Ok, I'm closing this PR. Here is the information how to achieve the same goal:
init.coffee:
atom.commands.add 'atom-text-editor', 'my:comment-and-move-down', ->
  editor = atom.workspace.getActiveTextEditor()
  editor.selectLinesContainingCursors()
  editor.toggleLineCommentsInSelection()
  editor.moveDown(0)

keycap.cson:
'atom-workspace atom-text-editor:not([mini])':
    'cmd-/': 'my:comment-and-move-down'


update

an updated scheme for DM for R1.1.0

we are still working on getting it to automatically update but as R1.1.0 add some new features thought was best to add right away


During re-run of stopped bcbio analysis, coverage_summary does not enter parallel mode

I am re-running a stopped bcbio analysis from the working directory. As resource manager I use a torque which assigns jobs to a couple of worker nodes. In the beginning, bcbio (correctly) runs through all the files in local mode to see from where it has to continue the previously stopped analysis run. Everything runs fine until the analysis arrives at the Chanjo part Timing: coverage multiprocessing: coverage_summary, the first part where bcbio would have to do some new processing (I assume, anyway it decided to to some heavy local processing). Here, bcbio does not submit the jobs to the parallel backend but continues in local mode, effectively dumping all the coverage BED file in the tmp directory of the job submit node (the node where I run bcbio_nextgen.py from) during the BED cleaning step. That's bad because it concentrates all the IO and processing on one node, which in this case is not equipped to do so (mostly due to /tmp storage limitations of the submit node).
If bcbio at least would use the working directory for dumping the files that would be okay (still not parallel, though), but I suspect there is a deeper reason why bcbio does not use the parallel environment here (or why it thought it was okay to run in local mode even if there was some heavy lifting to do). The next set in the pipeline, Timing: structural variation initial is correctly run in parallel mode again, using the torque cluster.

Okay, so re-ran everything with another host, completed Timing: coverage multiprocessing: coverage_summary, and passed the next checkpoint. Then I stopped everything and re-ran again from the working directory. It's the same spiel: bcbio does not recognize that the Chanjo stuff has already been completed, and tries to redo everything in local mode on the submit node.

Thanks @schelhorn,
Sorry about that, we were memoizing based on a BED file that should have been temporary, which lead to this problem. Thanks for figuring out exactly what was going wrong, you are awesome.

Thanks for the quick fix, @roryk!


Windows Machine File Copy: Failed to Create PSDrive with Destination, An extended error has occurred

Type: Question
Enter Task Name: Windows Machine File Copy
list here (V# not needed):  2.1.9
Environment

Server - Azure Pipelines or TFS on-premises? Azure Pipelines
Agent - Private:

Capability name
Capability value

Agent.Version
2.149.2

Agent.OSVersion
10.0.14393

Agent.OS
Windows_NT

Issue Description
[Include task name(s), screenshots and any other relevant details]
Task logs
##[section]Starting: Copy files from $(build.sourcesdirectory)/site
==============================================================================
Task         : Windows Machine File Copy
Description  : Copy files to remote machine(s)
Version      : 2.1.9
Author       : Microsoft Corporation
Help         : [More Information](https://go.microsoft.com/fwlink/?linkid=627415)
==============================================================================
Copy started for - <IP_ADDRESS>
##[error]Failed to Create PSDrive with Destination: '\\<IP_ADDRESS>\d$\DocsSite\NewFolder', ErrorMessage: 'An extended error has occurred'
##[error]An extended error has occurred
##[section]Finishing: Copy files from $(build.sourcesdirectory)/site

Troubleshooting
Add a demand for a single Agent and tried the following commands from the Agent Machine, but no luck.

Remove-PSDrive from the Agent
net use /delete

Same issue since upgrade to Azure devops server from TFS (i think the error started then..)
Windows Machine File Copy V:2.*
single target specified with machine name.
2019-05-21T11:31:13.8204407Z ##[section]Starting: Copy files for Copy service 2019-05-21T11:31:13.8209862Z ============================================================================== 2019-05-21T11:31:13.8210027Z Task         : Windows Machine File Copy 2019-05-21T11:31:13.8210137Z Description  : Copy files to remote machine(s) 2019-05-21T11:31:13.8210245Z Version      : 2.1.9 2019-05-21T11:31:13.8210358Z Author       : Microsoft Corporation 2019-05-21T11:31:13.8210477Z Help         : [More Information](https://go.microsoft.com/fwlink/?linkid=627415) 2019-05-21T11:31:13.8210604Z ============================================================================== 2019-05-21T11:31:14.7890624Z Copy started for - testsrv14 2019-05-21T11:31:15.0778636Z ##[error]Failed to Create PSDrive with Destination: '\\testsrv14\D$\WinService', ErrorMessage: 'An extended error has occurred' 2019-05-21T11:31:15.1747922Z ##[error]An extended error has occurred 2019-05-21T11:31:15.2158922Z ##[section]Finishing: Copy files for Copy service

Are you running multiple agents on the same machine ? If yes, in order to solve above mentioned issue we recommend not to install multiple agent in single machine.

If were not allowed to install on agent on each machine we deploy apps/db changes, the only other way is to create a "deploy server" that has multiple agents install, all of them registered to a pool on azure devops, and then all the definitions will be filled with file copy over WinRM and powershell over WinRM.
We dont use capabilities and demands this way but we can do multiple deploys depending on no, of agents on this "deploy server". Cant the PSDrive be created in the scope of the specific agent ?

I encountered this issue message and solve it after opening SMB port 445 at destination machine


Remove UseSmallerMachineTypes feature flag

Description
Remove UseSmallerMachineTypes feature flag added in #745
Changes proposed in this pull request:
Related issue(s)

/kind enhancement


chore: replacing more 'instanceof's in examples with 'switch'

lgtm!
One question (nit): I noticed some places where we have access to logger but we are still using console.log/console.info where we can use logger.info?

lgtm!
One question (nit): I noticed some places where we have access to logger but we are still using console.log/console.info where we can use logger.info?

hrm, good call. looks like that is always in those ensureCacheExists functions? Or did you see it somewhere else? I can fix those.


UTFGrid updates

This makes it so the UTFGrid source (wish we hadn't called it TileUTFGrid) uses XHR to load by default.  A jsonp option is added for legacy support.
This also makes it so the forDataAtCoordinateAndResolution() (could have been getData()) function returns the "key" (or feature id) if the optional data member is absent from the UTFGrid.
I had completely forgotten about #5238 when I needed these changes.  So I attributed the first commit to @aberenyi since it is essentially the same as his work there.
Tests added for the loading success/failure.  I'll add a test for the dataless UTFGrid in a bit.

Fixes #5211

@tschaub many thanks for this.
I'm still up to my eyeballs with other stuff, thus haven't had the time to write the tests.

Example updated to use the v4 API.  I included upgrade notes and updated tests (though coveralls notices that coverage is actually down 0.1 %).
Thanks for any review.

One tiny comment wrt a variable name in the upgrade notes, otherwise this looks good to me. Thanks and please merge, @tschaub & @aberenyi

LGTM, thanks @tschaub

Looks good to me too. Please merge.


[Bug] [Module Name] No module named 'dbgpt.app'

Search before asking

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

Operating system information
Windows
Python version information
3.10
DB-GPT version
main
Related scenes

[ ] Chat Data
[ ] Chat Excel
[ ] Chat DB
[ ] Chat Knowledge
[ ] Model Management
[ ] Dashboard
[ ] Plugins

Installation Information

[X] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
Device: GPU 24
Models information
LLM:  azure openai
What happened
(dbgpt_env) E:\source\db-gpt> python  .\dbgpt\app\dbgpt_server.py
Traceback (most recent call last):
File "E:\source\db-gpt\dbgpt\app\dbgpt_server.py", line 14, in
from dbgpt.app.base import (
ModuleNotFoundError: No module named 'dbgpt.app'
What you expected to happen
start app
How to reproduce

git clone
pip install .....
python  .\dbgpt\app\dbgpt_server.py or dbgpt start webserver

Additional context
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Hello, @xydz ,please check if the project cloned via git clone is complete.
I suggest cloning it again or directly downloading the project.
Then, execute pip install -e ".[default]" or pip install -e ".[openai]" depending on your specific use case.
Please refer to the tutorial for docs.

remove dbgpt_env
create new  dbgpt_env
pip install
OK


tv model not render.

hi,run the project tv model not render on the plane,What should I fix ? thx:)
on iPhone XR, iOS 14.6

Messages are received, thank you.

I reviewed the project and needed to long press to place the tv model


Update gotest.yaml

What does this PR do?
What issues does this PR fix or reference?
Is your PR tested? Consider putting some instruction how to test your changes

$ git checkout master
Switched to branch 'master'
$ git merge iss53
Merge made by the 'recursive' strategy.
index.html |    1 +
1 file changed, 1 insertion(+)index.html

.patch


very slow queries getRoutesByStop and getStopsByRoute

Hi, I really like this package however I'm noticing that this two queries run very very slow like almost a minute: getRoutesByStop and getStopsByRoute.
I have 1381 stops and 87 routes in my mongodb.

Then on 'From' blur I'm calling a function to first getRoutesByStop to receive all routes that serves the 'From' stop and then on each route I'm calling getStopsByRoute to receive final list of 'To' stops that you can get to directly from the 'From' stop. This gets (1.5 - 2 minutes)
I'm getting the data eventually but it's extremely slow.
Any suggestions on how to improve it?

I'm really sorry.
parseInt(stopID) fixed the problem. My bad. Time to go to bed.

Any suggestions on how we can document this or improve it so future developers don't run into this issue?

Now, when I think about it in the stops collection the stop_id field is string and not int but for some reason when I used int stop_id as the param for getRoutesByStop: function(agency_key, stop_id, cb) it got me the results fast.


Paramaters update

Added choices for string and all available options

Have you tested this as well? Does this not break or change the API for commands without options in any way?


Can't build on windows

Severity	Code	Description	Project	File	Line	Suppression State
Error	MSB4019	The imported project "C:\BuildCustomizations\IntelOpenCL.props" was not found. Confirm that the path in the  declaration is correct, and that the file exists on disk.	bmfr	C:\Users\trsh\Downloads\bmfr-master\bmfr-master\opencl\bmfr.vcxproj	64

Seems like your Intel OpenCL driver installation is broken.

@maZZZu what has hard-coded link "C:\BuildCustomizations\IntelOpenCL.props" todo with my OpenCL drivers?

Please search the files in the repository. It is extended from:

@maZZZu I dont understand for what I have to search? $(INTELOCLSDKROOT)\BuildCustomizations\IntelOpenCL.props is for intel Graphics. I have GeForce.


Memory usage growing with time

I'm not sure if this is related to issue #62 but I noticed that when watching live HLS stream (at the moment don't have any outgoing port to share the stream) the memory usage just keeps going up. The screenshot is only after 8 minutes of run-time, but it keeps going like this (at start it was on 68 MB).

At first I thought it was related to my implementation of shaka player in our app, but the same is happening in the sample project.
I tested both apps with a VOD stream https://storage.googleapis.com/shaka-demo-assets/sintel-mp4-only/dash.mpd and the memory does grow, but once the stream is done, majority of it is freed (so I guess that's ok).
Automated reference counting is on, but it doesn't seem to affect much.
I also put the buffer configuration on by default, but that also doesn't seem to help much.
    [m_player configure:kBufferingGoal withDouble:5.0];
    [m_player configure:kRebufferingGoal withDouble:1.0];
    [m_player configure:kBufferBehind withDouble:1.0];

Any idea on a possible cause or what I could try to reduce the memory usage?
NOTE: this is a debug version so not sure if that's because of it... I wanted to test it with a release version, but since I build the framework with widevine and for some reason it crashes, I wasn't able to.
cdm_->createSession(cdm_session_type, &cdm_session_id); JS Main Thread (17): EXC_BAD_ACCESS

Not really sure what I'm doing wrong for it to crash, I built the framework the same way as for develop version, just selecting release_cdm.json (I should also check if there is a newer version available)

I see something similar, but when I use the Instruments tool to track memory usage, the leak goes away.  Usage goes up for a while, but once the leak check runs, it drops back down.  What may be happening is Objective-C or JavaScriptCore is keeping the memory alive for a while to avoid a bunch of allocations.  Or it could be memory fragmentation causing new allocations to request new memory instead of reusing old memory.  I'll see if this can happen with a more simple app.  I'll also try running for an hour or two and see if this continues forever or it drops off eventually.
I have confirmed that this is NOT caused by: (a) us leaking output images, (b) us storing too many frames, (c) us not clearing buffered ranges, (d) us leaking memory, or (e) us not freeing JavaScript objects when we should.  So I'm not sure where this leak would be coming from.  I've also tried applying memory pressure so see if JavaScriptCore would get more aggressive with freeing objects, but it didn't help; so I'm not sure what Instruments is doing to free the memory.
Note that if you try the latest master with HLS, it will go away since now we use AVPlayer for HLS.  You'd need to try playing DASH content to see this leak.

Also, if you haven't figured it out by now, you can't use the release CDM with a debugger.  You can change Xcode to not attach a debugger when it runs so you can try the release CDM.

Regarding the latest master, I tried using it, but HLS streams have some weird behavior. On some audio is distorted/high pitched or video starts to lag.

Odd, with the latest master, we should use native playback.  So if it works fine with the iOS player, it should play fine with us.  But you should file a separate bug for that.

This was my intention, but wanted to investigate more before reporting the issue. But with the current state as is, I don't have access to a mac, and don't know when I will. I made a build with an older commit so that will do for now. Once things calm down I'll check the streams again.

For us, with debug CDM and live DASH, the memory usage grows about 100MB a minute. So e.g. on iPhone XR, we're able to maintain the playback for about 20 minutes, before the app receives memory warning and we either have to destroy the player ourselves or the system will kill the app. It doesn't seem to matter whether the content is clear or Widevine-encrypted.

Static DASH seems to be more or less okay.

With release CDM, live DASH is playing for more than an hour without running out of memory or any issues at first sight. Not sure how much memory it actually uses though without the debugger attached.

I recently pushed some changes that should help the memory leak.  On my machine/device, I don't see any significant memory leak anymore.  Could you please test with the latest master?
Also, @okycelt, if you still reproduce this, please send us your manifest URL so we can test it.

@TheModMaker, the memory usage looks good on our end as well. Thank you.

I'm assuming this is fixed.  @Megued if this is still an issue for you, feel free to comment.

My apologize, I still didn't get to test the new version. Once I get the opportunity I will check and let you know if there are any issues.


Update add_event.css

Enhanced responsiveness, refined layout, modern gradients, interactive transitions, and professional shadows for a polished, cohesive design.

https://github.com/user-attachments/assets/bc02599c-7c23-47f8-87c9-7a5310fffd3d
please look at this issue the contents are not properly aligned


[Outlook] (mobile web support) Update modern Outlook on web add-in support information for mobile phones

Clarify that add-ins don't work on modern Outlook on web on iPhone and Android phones.
Internal ADO item: 5776437

Docs Build status updates of commit b894e43:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/requirements-for-running-office-add-ins.md
:white_check_mark:Succeeded
View

docs/outlook/outlook-mobile-addins.md
:white_check_mark:Succeeded
View

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/outlook-api-requirement-sets.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit ccf2c74:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/requirements-for-running-office-add-ins.md
:white_check_mark:Succeeded
View

docs/outlook/outlook-mobile-addins.md
:white_check_mark:Succeeded
View

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/outlook-api-requirement-sets.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit f5cbc4b:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/concepts/requirements-for-running-office-add-ins.md
:warning:Warning
View
Details

docs/reference/requirement-sets/outlook-api-requirement-sets.md
:warning:Warning
View
Details

docs/includes/classic-versus-modern-Outlook-on-the-web
:white_check_mark:Succeeded

docs/outlook/outlook-mobile-addins.md
:white_check_mark:Succeeded
View

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded
View

docs/concepts/requirements-for-running-office-add-ins.md

Line 82, Column 1: [Warning: include-invalid] Invalid include link extension: '../../includes/classic-versus-modern-Outlook-on-the-web'.

docs/reference/requirement-sets/outlook-api-requirement-sets.md

Line 121, Column 1: [Warning: include-invalid] Invalid include link extension: '../../includes/classic-versus-modern-Outlook-on-the-web'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 1a55f7d:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

Line 0, Column 0: [Error: docset-not-provisioned] Cannot build docset 'azure-documents' because it isn't provisioned. Please go to Docs Portal (https://ops.microsoft.com) to provision first.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit ab83196:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/requirements-for-running-office-add-ins.md
:white_check_mark:Succeeded
View

docs/includes/classic-versus-modern-Outlook-on-the-web.md
:white_check_mark:Succeeded
View

docs/outlook/outlook-mobile-addins.md
:white_check_mark:Succeeded
View

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/outlook-api-requirement-sets.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Create a sample for Canonical Data Model pattern

Create a sample for Canonical Data Model pattern.

Idea 1: use FHIR as the model, talk to Nirmal Fernando for more info.
Idea 2: use GeoJson as the model.


Release of version <IP_ADDRESS> of xWebAdministration

Releasing version <IP_ADDRESS>

This change is

Closing and re-opening to try to get AppVeyor to cooperate


How to show the sidebar file icon,not </> ,such as *.py？

Mono file icons is installed.


Set tut #6 to test ML in Spyfish

Set up tut#6 to run the fish identification model developed in https://wandb.ai/adi-ohad-heb-uni/project-wildlife-ai?workspace=user-victor-wildlife for the Spyfish Aotearoa project

@victor-wildlife I have been working on this and it seems the big hurdle right now is the format of the model that was trained. It is a PyTorch model but it isn't a YOLO model, which means that it isn't directly transferable to our current framework. Since this is likely something we want to support (custom models), I will spend some more time looking into it but it will require a bit more work than I initially thought. My aim will be to incorporate their model code in some way but keep a lot of the YOLO utils that help us work with video streams etc. We can also discuss this more next week.

Sounds good. Let's see how long it takes to develop the capabilities to support the custom model. Alternatively, we could look at some of the formats support from CVAT

@victor-wildlife Have done some more digging into this and think we can discuss the outcome during our technical session this week :)

Awesome, looking forward to the catch-up session this week.
Cheers,
Victor
On Tue, Jan 31, 2023 at 2:23 AM Jannes @.***> wrote:

@victor-wildlife https://github.com/victor-wildlife Have done some more
digging into this and think we can discuss the outcome during our technical
session this week :)
—
Reply to this email directly, view it on GitHub
https://github.com/ocean-data-factory-sweden/kso-utils/issues/6#issuecomment-1408625781,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANFVKPNDYJO6WDF37M2Q2YLWU66GHANCNFSM6AAAAAAS5JKKF4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Closing this for now as we try to train new YOLO models first and then return to testing the integration of custom models as it becomes necessary.


update gh action set-output variables

resolves #6153 and very similar to PR https://github.com/dbt-labs/dbt-core/pull/6152

Description

Update deprecated github actions output variables according to https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Checklist

[X] I have read the contributing guide and understand what's expected of me
[X] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[X] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

Closed and re-opened PR to unfreeze code/snyk check


[DCJ-627][risk=low] Remove deprecated Dataset needsApproval and active state attributes.

Addresses
Addresses the UI portion of removing deprecated Dataset properties
Summary
Removing needsApproval and active Dataset interface properties that are marked deprecated in consent.  Drive by fix of punctuation in the DEVNOTES.md that was discovered while tooling my local dev environment.
These properties were already (largely) ignored by the frontend code.
See also
https://github.com/DataBiosphere/consent/pull/2399

Thanks for the thumbs!


Added review form

-[x] create a component for review form
-[x] set react routers as to review page opens for a selected company
-[x] collect user data and pass the props to api function with a POST request

General Checks

[ ] the branch is up to date with main/master
[ ] the code works when pulled and run locally
[ ] All CI checks pass
[ ] all conflicts are resolved (if any)
[ ] PR has a descriptive title
[ ] PR has appropriate labels and milestones for easy identification
[ ] PR it is assigned to the owner
[ ] reviewers are assigned
[ ] the PR contributes only one focused change
[ ] It is in the appropriate column in the project board (if necessary)
[ ] has short and clear description
[ ] is linked to an issue (if it is related)
[ ] feedback is addressed (if any and if it is appropriate feedback.)

Markdown

[ ] the markdown source is formatted
[ ] spelling and grammar is correct in all text
[ ] The markdown looks correct when you preview the file
[ ] all links and images work

HTML

[ ] pages besides /index.html are stored in /pages
[ ] the code is well-formatted
[ ] the HTML is valid
[ ] there are no inline styles (example: style='color: red;')
[ ] there are no <style> tags with CSS, all styles are hrefs
[ ] there is no inline JavaScript (example: onclick='doSomething()')
[ ] there are no <script> tags with JS, all JS is in an separate file
[ ] ids are used for JavaScript only, not for CSS
[ ] semantic tags are used
[ ] spelling and grammar is correct in all site content

CSS

[ ] the code is well-formatted
[ ] passes all the linting checks

@danielhalasz all of this is planned. Since I worked alone on this feature it's what I have managed to do for one day. Hope with the help of my team we can finish it tomorrow.


Fixes the SwitchTraffic bug that wasn't respecting --dry_run for readonly and replica tablets during a resharding event

ref: https://github.com/Shopify/vitess-project/issues/431
Fixes the SwitchTraffic bug that wasn't respecting --dry_run for readonly and replica tablets during a resharding event. More details specifically in this comment.

Looks good to me. Submit it upstream?

Yep, might as well. Will be curious to see if they'll accept it or reopen the original PR. Either way should be fine for us.

reopen the original PR

Missing context? 🤔

Upstream PR was merged - https://github.com/vitessio/vitess/pull/12992
It had some e2e tests that have some conflicts with the v15 branch. The one-liner change represented in this PR is the same though so I think it's safe to merge this into our v15 branch. The upstream PR did have labels to backport the change to 14,15, and 16 though.
@hkdsun @brendar thoughts on merging this as is?


README - Requirements

Under  your requirements section, I see you've written Note.js which I preusme is meant to be Node.js

https://github.com/Altpapier/hypixel-discord-guild-bridge/commit/698c0cf5ec8d21583d1fb0cd92da655f8036ea94


trying to translate from french to english, asks for key

So I did the command to a mpg file as written here;
autosub ~/Downloads/Midietsoir.mpg -C 2 S fr -D en
and after it does all this conversion I get the following;
Error: subtitle translation requires spefified google translate API key see --help for further information.  I heard the key was included so whats this all about?  Thanks and regards,
Thomas

see #111

Free translation version here.


Add UITests

Add Targets UITests named RxTodoUITests
Add Three UITest case

Compare default tasks count
Add new task
Edit existed task

this is red test case to solve #9

@cruisediary Thanks for your work! However, I'm not sure if we need UI tests. Because every UI properties is binded from ViewModel's Observables and we already have tests for those Observables. Why do we need UI tests?


Update css

Added colour definitions for the announcement, page/section titles in the sidebars, admonitions. Should look like this: https://zanemit.github.io/cisp/info/intro.html

Tested locally and looks great!


Every time I open the editor I get an error (and intelephense stop working)

vscode 1.23.1
intelephense [0.8.8] - 2018-02-13
Linux Mint 18
Every time I open the editor I get this error:
Request initialize failed with message: this.send is not a function

Fixed


Conflict with metalsmith-permalinks?

Hi there,
Thank you for this plugin. I've tried to use it but it seems to conflict with metalsmith-permalinks.
Unless I'm missing something, both seem to use the path property, so the path gets overwritten when the paths plugin runs before the permalinks plugin.
Can you confirm this issue? If so, would it make sense to rename the property to avoid the conflict? I'm using this method as a workaround on my local version, but it would be nice to do it in the repository.

you are correct, and though its typical to get conflicts like this ... it's usually rare.
to that end, I made the property customizable, see updated README in v2.1.0

@ahmadnassri Thank you so much! It works perfectly.


Unable to nest Listbox

Provide a general summary of the issue here
Nesting a ListBox inside a ListBoxItem shows the list items from the parent ListBox. The use-case for this is showing a list of items in a popover action menu for a list item.
Visual representation:

ListBox

ListBoxItem

Action menu

ListBox - This renders the parent Listbox

ListBoxItem

🤔 Expected Behavior?
Nested ListBox should display its children
😯 Current Behavior
Nested ListBox displays the children of its parent ListBox
💁 Possible Solution
This might be the desired behaviour but the docs doesn't seem to cover it or I may be doing something wrong
🔦 Context
No response
🖥 Steps to Reproduce

Render a ListBox
In the ListBoxItem, render a DialogTrigger
In the DialogTrigger's popover, render another ListBox with different content

Version
1.3.1
What browsers are you seeing the problem on?
Other
If other, please specify.
Brave
What operating system are you using?
Macos
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

A ListBox cannot have interactive children of any kind. Did you mean GridList?
See last sentence: https://react-spectrum.adobe.com/react-aria/ListBox.html#text-slots


[QUESTION] - How to get Listener from server.Default() so that I could use it in different libraries?

Describe the Question
I'm trying to use hertz framework with https://github.com/pedia/endless to allow zero downtime restart. There's an example to include Listener but I'm not sure how to use it with hertz
Reproducible Code
func serve_http(addr string, ln net.Listener) (*fasthttp.Server, error) {
	server := &fasthttp.Server{Handler: func(ctx *fasthttp.RequestCtx) {
		ctx.WriteString(fmt.Sprintf("%d\n", os.Getpid()))
	}}

	go server.Serve(ln)

	return server, nil
}

func main() {
	addr := ":3030"
	var s *fasthttp.Server

	endless.Start(
		func(p *endless.Parent) error {
			ln, err := net.Listen("tcp", addr)
			if err != nil {
				return err
			}

			p.AddListener(ln, addr)
			s, err = serve_http(addr, ln)
			return err
		}, func(c *endless.Child) error {
			nf, ok := c.NamedFiles[addr]
			fmt.Printf("got %#v\n", nf)
			if !ok {
				return fmt.Errorf("inherit %s not found", addr)
			}

			c.AddListener(nf.LN, addr)
			server, err := serve_http(addr, nf.LN)
			s = server
			return err
		},
		func(ctx context.Context) error {
			return s.Shutdown()
		},
	)
	fmt.Printf("Quit %d\n", os.Getpid())
}

Expected behavior
Able to get listener
Screenshots
If applicable, add screenshots to help explain your question.
Hertz version:
Latest

I will research it and welcome it together.💕

@li-jin-gou Thanks. Please suggest if you've any idea

Seems that you wanna pass a customized listener to Hertz?
Actually it doesn't support that now, but I will looking into it - there isn't much work to do.

@welkeyever No, I want to get a listener from Hertz and use it for different purposes. So basically when Hertz is set up for Spin if it could store Listener somewhere and I could access that Listener from RequestContext, then that would also work.

@sujit-baniya It's reasonable to store it in network layer of hertz instance - you may get it through transport-related APIs. But it would be strange that relate listener to RequestContext - RequestContext is main for application-related jobs - you may save hertz instance somewhere, and get it to do whatever you wanna later.

@welkeyever Yes getting listener from hertz instance would also work. I was giving RequestContext as an example :)

Anything like hertzInstance.GetListener() or hertzInstance.Listener would work.

Okay, let me have a try.
I will update if there is any progress.

@welkeyever any thought on this issue?

@welkeyever any thought on this issue?

@sujit-baniya Sorry for the late. I was stumped by other things before. I will continue with this part.

Possible solution goes here: https://github.com/cloudwego/hertz/compare/develop...welkeyever:hertz:feat/more_options_for_transporter
Please feel free to add your review comments.
The endless example works, however there is a issue related to a third party lib:

And we will finally get rid of this lib after this #541.

@welkeyever Please let me know your thought if we use Options.Listener net.Listener for Engine and pass it to transporter.
Something like this:
type Options struct {
    ....
    Listener net.Listener
}

func WithListener(ln net.Listener) config.Option {
	return config.Option{F: func(o *config.Options) {
		o.Listener = ln
	}}
}

And for each transport we assign this listener to transport
// For transporter switch
func NewTransporter(options *config.Options) network.Transporter {
	return &transport{
		readBufferSize:   options.ReadBufferSize,
		network:          options.Network,
		addr:             options.Addr,
		keepAliveTimeout: options.KeepAliveTimeout,
		readTimeout:      options.ReadTimeout,
		tls:              options.TLS,
		listenConfig:     options.ListenConfig,
                ln: options.Listener,
		OnAccept:         options.OnAccept,
		OnConnect:        options.OnConnect,
	}
}

And in transport.serve()
    if t.ln == nil {
    network.UnlinkUdsFile(t.network, t.addr) //nolint:errcheck
	t.lock.Lock()
	if t.listenConfig != nil {
		t.ln, err = t.listenConfig.Listen(context.Background(), t.network, t.addr)
	} else {
		t.ln, err = net.Listen(t.network, t.addr)
	}
	t.lock.Unlock()
	if err != nil {
		return err
	}
    }

I did above and got the same error as yours

WithListener

Continuously adding options like WithListener will make it confusing when user adding both WithListenConfig and WithListener - there is a potential effective priority under this scenario, but it cannot be well informed to the user.
So I just provider a config-independent way to do so.

@welkeyever
I also need to get listener/set listener from hertz.  I was wondering if the feature will be available in Hertz?  or we need to folk?  Thank you

if t.listenConfig != nil {
    t.listener, err = t.listenConfig.Listen(context.Background(), t.network, t.addr)
} else {
    t.listener, err = net.Listen(t.network, t.addr)
}

以上为 hertz 的代码，listenConfig 直接调用了 Listen  方法，并没有进行特殊处理。
我认为框架内只需传入 Listener net.Listener 参数，不应传入 listenConfig *net.ListenConfig 参数，让用户去自定义 net.Listener。
代码参考：
t.lock.Lock()
if t.ln == nil {
   t.listener, err = net.Listen(t.network, t.addr)
}
t.lock.Unlock()


Missing memfill op in IR

I encountered the Wasm memory.fill instruction in the GlobalAlloc::alloc_zeroed function, which is used to zero out the memory allocated by GlobalAlloc::alloc.
Since the memory in Miden VM is guaranteed to be initialized to zeros, I ignored this instruction for now, but we're going to need it in the future.
See semantics at https://webassembly.github.io/spec/core/exec/instructions.html#exec-memory-fill
### Tasks
- [ ] Add `memfill` op in IR and lower it to MASM;
- [ ] Translate Wasm `memory.fill`;

The initial state of memory is guaranteed to be zero (for now at least), but alloc_zeroed is still needed because the allocator itself might free and reuse memory that was previously written to, which will no longer have that guarantee. So we'll definitely need this.
I'll make a note to add memset/memory.fill to the instruction set while I'm in there working on other things.

Implemented in #242


[DO NOT MERGE] check disk space

You're probably only interested in the root filesystem, so you can get straight to the point with df -h / which will print the usage stats on the root filesystem only.

Thanks! Yeah, that would be cleaner if that would be smth we decide to merge :) but I'm just trying to figure out what's going on here - we use c5d.xlarge AWS instances for CI (https://aws.amazon.com/ec2/instance-types/c5/) yet we are only getting ~39GB disks instead of 100GB ones it seems which is weird.


Update sequencer-batch-appended.ts

Defines Turing heights as L2 blocks rather than L1 blocks

Codecov Report

Merging #469 (66bedc8) into develop (093772b) will increase coverage by 14.54%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop     #469       +/-   ##
============================================
+ Coverage    72.17%   86.72%   +14.54%
============================================
  Files           59       34       -25
  Lines         1826     1122      -704
  Branches       281      162      -119
============================================
- Hits          1318      973      -345
+ Misses         508      149      -359

Flag
Coverage Δ

contracts
86.72% <ø> (ø)

core-utils
?

data-transport-layer
?

message-relayer
?

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

Impacted Files
Coverage Δ

.../l1-ingestion/handlers/sequencer-batch-appended.ts

packages/core-utils/src/provider.ts

packages/message-relayer/hardhat.config.ts

...layer/src/services/l1-ingestion/handlers/errors.ts

packages/core-utils/src/common/index.ts

packages/core-utils/src/common/misc.ts

packages/core-utils/src/watcher.ts

...ices/l1-ingestion/handlers/state-batch-appended.ts

packages/core-utils/src/common/hex-strings.ts

packages/core-utils/src/fees.ts

... and 15 more

Continue to review full report at Codecov.

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

Test coverage would solidify this!


Implementation Coverage is wrong?

Hi,
I'm new to this great library so please if my understanding about it is wrong just let me know.
I would like to point that I've found discordance between the examples on the README.md and the IMPLEMENTATION_COVERAGE.md.
In the README.md there is an example that uses describe_instances
from . import add_servers
from moto import mock_ec2

@mock_ec2
def test_add_servers():
    add_servers('ami-1234abcd', 2)

    client = boto3.client('ec2', region_name='us-west-1')
    instances = client.describe_instances()['Reservations'][0]['Instances']
    assert len(instances) == 2
    instance1 = instances[0]
    assert instance1['ImageId'] == 'ami-1234abcd'

And then on the IMPLEMENTATION_COVERAGE.md the describe_instances is not marked as implemented.
Is the IMPLEMENTATION_COVERAGE not up to date or is this an error?
Thanks.

Hi @omar-ogm, thanks for raising this!
describe_instances() is definitely supported. The IMPLEMENTATION_COVERAGE doc is generated automatically, so it's either out of date or there's a bug that causes it not to pick up that specific method. Will look into it.

Hi @omar-ogm, thanks for raising this!
describe_instances() is definitely supported. The IMPLEMENTATION_COVERAGE doc is generated automatically, so it's either out of date or there's a bug that causes it not to pick up that specific method. Will look into it.


Metric server is not able to display metrics on Kubernetes dashboard

Rancher versions:
rancher/server or rancher/rancher:
rancher/agent or rancher/rancher-agent:
rke version: V0.19
kubernetes version:1.11.1
Infrastructure Stack versions:
healthcheck:
ipsec:
network-services:
scheduler:
kubernetes (if applicable):
Docker version: (docker version,docker info preferred) Docker version 17.03.2-ce, build f5ec1e2
Operating system and kernel: (cat /etc/os-release, uname -r preferred) Ubuntu 16.04
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
3 node cluster through rke
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Kubernetes
Steps to Reproduce:
Deploy the kubernetes through rke and then deploy the kubernetes dasboard.You will see the dashboard but not able to see the cpu and memory usages on dashboard
Results:
Here is the logs of kubernetes dashboard logs
No metric client provided. Skipping metrics
Metric client health check failed: the server could not find the requested resource (get services heapster). Retrying in 30 seconds.

Metric client health check failed: an error on the server ("Error: 'dial tcp <IP_ADDRESS>:8082: i/o timeout'\nTrying to reach: 'http://<IP_ADDRESS>:8082/healthz'") has prevented the request from succeeding (get services heapster). Retrying in 30 seconds.


Is there a way to set the expiration time?

It would be nice if default expiration time could be set here, so when the log group is created it would set it to that time.

Is that about the setting here? 👇
https://docs.aws.amazon.com/AmazonCloudWatch/latest/logs/Working-with-log-groups-and-streams.html#SettingLogRetention
If so, I think that it is not the responsibility of this plugin.

Yeah, and that's fine.


Integrate Test Framework

This is a prototype for issue #12

Thanks @zhjwpku !
What I would be interested to know is the compile and link times, respectively, for both GTest and Catch2.
My experience in Arrow is that GTest's main header gtest.h imposes a large compilation overhead when you have many independent test modules, and I'm curious if Catch2 improves the situation.

Thanks @zhjwpku !
What I would be interested to know is the compile and link times, respectively, for both GTest and Catch2. My experience in Arrow is that GTest's main header gtest.h imposes a large compilation overhead when you have many independent test modules, and I'm curious if Catch2 improves the situation.

I did 3 tests:

using -E to see the preprocessed file size

4301941   -- gtest
4086315  -- catch2

using time to see the compile time

0.40s user 0.06s system 78% cpu 0.587 total   -- gtest
0.32s user 0.06s system 76% cpu 0.494 total   -- catch2

using time to see the link time

0.04s user 0.02s system 79% cpu 0.080 total   -- gtest
0.09s user 0.03s system 98% cpu 0.120 total   -- catch2

As the results shows, catch2 is better at preprocessed file size and compile time, but gtest is better when it comes to link time. I don't think the above tests is adequate to prove which is better than the other, since the UT in this PR is fairly simple.
I think we should focus closely on the specific benefits that the test framework is expected to deliver for this project.

Thanks for the number @zhjwpku .

As the results shows, catch2 is better at preprocessed file size and compile time, but gtest is better when it comes to link time. I don't think the above tests is adequate to prove which is better than the other, since the UT in this PR is fairly simple.

Well, it does tell us that they are close in terms of inclusion overhead, which was my concern.

I think we should focus closely on the specific benefits that the test framework is expected to deliver for this project.

Agreed. Is there a detailed (hopefully unbiased) comparison somewhere?

I found this link: https://yurigeronimus.medium.com/guide-for-choosing-a-test-framework-for-your-c-project-2a7741b53317

Seems like a lot of fluff with no substance, unfortunately.

@raulcd @Fokko can you please review this PR?

BTW, should we run ctest in build_iceberg.sh (or a separate script) from pre-commit check in the CI?

BTW, should we run ctest in build_iceberg.sh (or a separate script) from pre-commit check in the CI?

I've added the ctest command in build_iceberg.sh, this will run test on all platforms we are targeting.


Clang frontend rework

Invoke host clang to compile C++ to IR instead of shipping a frontend in the plugin
There are multiple of motivations for this change:
(1) Less code and complexity of the plugin: binary size from 70M down to 30M
(2) Avoid IR version difference between host clang and shipped frontend
(3) Host clang configuration is easier to match, i.e. we can just pass the same arguments
Points (2) and (3) guarantee compatibility between injected IR and the host IR module. This is critical because LLVM IR gives no such guarantees between different toolchain versions or forks. We can live with that when we target Android, because we
know and we can reproduce the exact target compiler version from Goolge's android toolchain repository.
iOS is a different story, because our host compiler (the official AppleClang) is known to carry a surprising amount of internal modifications on top of its public upstream. While we don't know the actual patches, we can measure that the difference is huge by comparing the ABI of the two binaries. This suggests that our previously shipped clang frontend from apple/llvm-project may generate different IR than the host compiler. Failures would be arbitrary and there is no good way to detect them. It could be errors from middle-end passes, backend crashes, silent miscompilation or it may also just work. We wouldn't know.

There is some polishing todo and I have to fix a test failure on Linux. However, what do you think about the approach in general?

I love this design idea! (I didn't look at all the changes though)

Minor: the changes in commit Polishing after Clang frontend rework related to StringEncoding can be squashed into the previous commit, IMHO doesn't make sense to have another commit for fixes of newly-added changes in the PR.

Minor: the changes in commit Polishing after Clang frontend rework related to StringEncoding can be squashed into the previous commit, IMHO we don't need to have another commit for fixes of newly-added changes in the PR.

Yes and no. I'd like to keep the patch with the actual change as small as possible. If we ever end up here when bisecting for a regression, the patch we have to understand is smaller. And that's good.
That said, I didn't spend the time to sort out my polishing into formatting fixes and code reduction. The commit with the actual change made the code simpler in a few spots and I used to second commit to remove some code. It's probably not worth the effort to sort this out. Otherwise I would agree with you request.

There is some polishing todo and I have to fix a test failure on Linux. However, what do you think about the approach in general?

In fact, this uncovered an issue in my proposed environment for regression tests that is under review in https://github.com/open-obfuscator/o-mvll/pull/15. The test passes for me locally and the difference to the output from the build bot above is this:
--- test-dumps-local.log	2023-07-12 11:45:23
+++ test-dumps-github.log	2023-07-12 11:45:23
@@ -1,6 +1,6 @@
 Running tests with: /test-deps/bin/clang
 Testing plugin file: /o-mvll/src/build_ndk_r25/libOMVLL.so
-Available features are: {'system-linux', 'shell', 'host-platform-linux', 'host-arch-x86', 'aarch64-registered-target', 'bpf-registered-target', 'x86-registered-target', 'arm-registered-target'}
+Available features are: {'system-linux', 'shell', 'host-platform-linux', 'host-arch-x86', 'target=None', 'aarch64-registered-target', 'x86-registered-target'}
 OMVLL_PYTHONPATH: /Python-3.10.7/Lib
 -- Testing: 9 tests, 2 workers --
 PASS: O-MVLL Tests :: core/plugin/find-omvll-config-cwd.c (1 of 9)
@@ -11,7 +11,21 @@
 PASS: O-MVLL Tests :: passes/arithmetic/xor-x86_64-darwin.c (6 of 9)
 PASS: O-MVLL Tests :: passes/flattening/basic-native.c (7 of 9)
 PASS: O-MVLL Tests :: passes/flattening/basic-aarch64.c (8 of 9)
-PASS: O-MVLL Tests :: passes/strings-encoding/basic-aarch64.cpp (9 of 9)
+...
++ : 'RUN: at line 22'
++ env OMVLL_CONFIG=/o-mvll/src/test/passes/strings-encoding/config_replace.py /test-deps/bin/clang++ -fpass-plugin=/o-mvll/src/build_ndk_r25/libOMVLL.so -target aarch64-linux-android -fno-legacy-pass-manager -O1 -c /o-mvll/src/test/passes/strings-encoding/basic-aarch64.cpp -o /dev/null
+/test-deps/bin/clang -S -emit-llvm -std=c++17 -o /tmp/lit-tmp-gtl5vm7b/omvll-x86_64-unknown-linux-gnu-0e7f2a.cpp.ll /tmp/lit-tmp-gtl5vm7b/omvll-x86_64-unknown-linux-gnu-0e7f2a.cpp
+/test-deps/bin/clang -S -emit-llvm -target aarch64-unknown-linux-android -std=c++17 -o /tmp/lit-tmp-gtl5vm7b/omvll-aarch64-unknown-linux-android-4cc414.cpp.ll /tmp/lit-tmp-gtl5vm7b/omvll-aarch64-unknown-linux-android-4cc414.cpp
+No available targets are compatible with triple "x86_64-unknown-linux-gnu"

-Testing Time: 0.37s
-  Passed: 9
\ No newline at end of file
+--
+
+********************
+********************
+Failed Tests (1):
+  O-MVLL Tests :: passes/strings-encoding/basic-aarch64.cpp
+
+
+Testing Time: 0.92s
+  Passed: 8
+  Failed: 1

I will make a note in the respective PR. It's unrelated to this review.


Update scalaz-core to 7.3.6

Updates org.scalaz:scalaz-core from 7.2.33 to 7.3.6.
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!

Applied Scalafix Migrations

org.scalaz:scalaz-.*:7.3.0 (created no change)

https://raw.githubusercontent.com/scalaz/scalazfix/v0.1.2/rules/src/main/scala/scalaz/ScalazFix.scala

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

labels: library-update, early-semver-minor, semver-spec-minor, scalafix-migrations, commit-count:1

Superseded by https://github.com/coursier/coursier/pull/2602.


Remove redundant Travis config.

Travis will automatically run gradlew check

@elharo

Lgtm. Now that we have a bunch of violations, can you file a bug for us to look at the violations.


Add LLVM-13 installation to Docker setup

Add LLVM-13 installation to Docker setup:

Installs LLVM 13 in for Ubuntu 18.04 Docker images
This is needed as a requirement for #9425

cc @masahi @jtuyls @Mousius for reviews

@tqchen @areusch @jroesch @u99127 and all other previously mentioned, I wonder how many LLVM version you consider we should maintain on our pipeline. At this moment we have 5 versions. I was thinking we could remove 9 and 10?
Thoughts?

This log confirms the fix actually works when rebuilding the Docker images: https://ci.tlcpack.ai/blue/organizations/jenkins/docker-images-ci%2Fdaily-docker-image-rebuild/detail/daily-docker-image-rebuild/109/pipeline


File upload for challenge not accessible

It seems that after deploying the server using a fresh clone and default settings that uploading a file results in the file being uploaded properly, but when going to access the file by the embedded link in the challenge text, the URL generates as:
http://ctfdomain:4000//file.ext
and clicking on this link returns a 404 and the "Whoops, looks like we can't find that.  Sorry about that" page.

Can you upload the associated sqlite database that's generated?

Never mind I see the issue, thanks for the heads up


Support a configuration option to disable wrapping optional attributes in MAYBE

Is your feature request related to a problem? Please describe.
I get these TS issue in my code which forces me to either use maybe in my function input definition but I don't want to use that. I just use { tvName?: string }.

It would be great if I can generate the Schema without any InputMaybe and Maybe.
I use generate types in my NodeJS backend so I dont have to maintain a secondary type library and just tap into the Schema types. That works great.
But I dont want to get rid of the Maybes in my  code everywhere, it is quiet annoying.
Like I have a function

async function getSomething({ id, name}: {id: string, name?: string}) {
....
}

and I call that function somewhere else using a payload coming from my graphql query (I use AWS Amplify).

const handler = async (event: SomeInput): Promise<any> => {
   const { id, name } = event;

 await getSomething(id, name) ;
}

I will now get a type error on getSomething because name is InputMaybe<Scalars['String']>; and my TS code only expected a string or undefined.
So now I have to cast my name or use something like getSomething(id, name: name || undefined) which is ugly.
Or I have to start using Maybe in all my functions and I dont want to because it will couple my code to the Generator.
Describe the solution you'd like
Add a configuration option to generate the types without Maybes.
So instead of :
export type ConnectTvInput = {
  code: Scalars['String'];
  facilityId?: InputMaybe<Scalars['String']>;
  mode: TvMode;
  residentId?: InputMaybe<Scalars['ID']>;
  tvName: Scalars['String'];
};

we get
export type ConnectTvInput = {
  code: Scalars['String'];
  facilityId?: Scalars['String'];
  mode: TvMode;
  residentId?: Scalars['ID'];
  tvName: Scalars['String'];
};

Describe alternatives you've considered
Run a regex Replace over my generated code and try to manually remove InputMaybe and Maybe, but it may be a bit difficult with nested maybes in Arrays.
Any additional important details?
No response

Hello,
You could use inputMaybeValue to change InputMaybe  to just undefined e.g.
generates: {
  'src/types.generated.ts': {
    plugins: ['typescript'],
    config: {
      inputMaybeValue: 'T | undefined'
    }
  }
}

Note that a null value is a valid case in that your resolvers can receive, so be careful when you make this change because you can only do falsy checks:
{
  yourResolver: (parent, { input } ) => { // your input can be `T | null | undefined`, but if you've changed it to `T | undefined`...
    // then, you won't be able to do this safely:
    if( input !== undefined ) {
      console.log(input); // input is `T | null` at runtime but your type is saying `T`, which is wrong
    }

    // instead, you can only do falsy checks like `if(!input)`
  }
}

@eddeee888 I will give it a try.
I am not afraid of input: null. If I don't have a input in a query, my query will just look like
query {
  yourResolver {
    id
  }
}

If there are arguments, they are either required not required.

Closing this as this seems to fix the issue. Please let me know if it doesn't. Thanks!


mem_ss - create the basic tb with rd/wr/ request tasks (d_mem/i_mem)

add basic RD/WR request to the mem_ss -> i_cache/d_cache
add trackers on the top-level interface -> what goes in, and out from core side and the FM side
create the "FM" behavioral to accept the RD/WR from the mem_ss

use the cache tb as a reference for all!!

We can mark this as done once we have this sequence working with trackers:
From D_MEM side:

write to 5 CL in the same set. (force a dirty evict to FM) (with a 20 cycle delay between each write)
Read the 5 CL and make sure the data is as you wrote it. (with a 20 cycle delay between each read)

From I_MEM side:

We support only read, so to be able to validate this we must "backdoor" load the FM with some data
Send (with a 20 cycle delay between each read) - make sure it matches the Backdorr data

Note:

Please don't overlap the address.
due to an issue in MC we can only use tq[0] - this is why we need the big delay.
no need to add delay (latency) between the mem_ss -> SRAM response at this point

Created test wr_rd_dmem with 5 write to differents CL same Set then 5 reads
Trackers look good i think, RD values are the ones we wrote


Reviewer and applicant user accounts

Description
A user account is assigned both a reviewer and applicant role. How does this impact their experience with accessing the applications they submit on the platform? There are parameters preventing a reviewer from submitting and reviewing their own submission. However, will this user account be able to access their own applications  on their dashboard, and have the same functionalities designated to the reviewer user group?

Reviewers do not need the "applicant" role but I do not think anything will change if they do.

A 'Reviewer' that has submitted an application will not be able to view and access their own application.


⚠ Turnos 24x7 has degraded performance

In 9198d38, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 833 ms

Resolved: Turnos 24x7 performance has improved in 3e4eaf1.


fixed label issue for last node when hydrating relationship

I checked out this branch and confirmed that it does in fact fix the issue.


[WIP] Add ORAS exclusive moves.

I scraped the learnsets for Pokémon in ORAS from Serebii and added them to the db with the new version_group.

I think you should also add the changes from versions.csv, version_names.csv, version_groups.csv and all the other versions related .csv files

I'm sorry, I'm not sure what you mean. I based this on origin/oras, hence changes made there to include the new versions are 'included'. The only thing I can see concerning versions that may be unhandled is in version_group_move_methods. I did not yet add the move methods that are available in ORAS, but I shall do so ASAP.

Misses the moves that are completely new in ORAS, i.e. Dragon's Ascent, etc.
Thought those were already in the branch, but it was the new abilities.

@Cu3PO42 @magical If you guys need data ripped from ORAS (as opposed to scraping it or manually entering it), have you asked anyone at Project Pokémon if they have the data handy? I'm sure Kaphotics or someone would be willing to contribute.

@codemonkey85 I haven't asked specifically, but scraping was like a 20 line job and it seemed to be the easiest thing. I might ask or have a look at my own decrypted copy though, if I need more data.

@Cu3PO42 The only thing that concerns me about scraping data from any website is the reliance that the data was flawlessly entered into said website.

@codemonkey85 That's a valid concern. Though I'm not specifically worried about Serebii due to high traffic of the site and ORAS being released for quite some time. Maybe he even got the data from the game. When I have any more time I will probably double check with the learnsets from the ROM. (And add the three missing moves...)

Is there any example/tutorial on how to rip data directly from the game? I think this is the main problem here...
If more people would know how to do this the data would be more consistent, accurate and up-to-date. And then we do not have to copy data from other websites...
I would really like to rip some data but don't know anything about how to use python or this script.

@Cu3PO42 Veekun has traditionally has data scraped from games whenever feasible, which means Serebii/Bulbapedia have more data (and faster), but of lower quality. (High traffic doesn't matter for obscure moves -- why would anyone check the exact level for Remoraid to learn Hydro Pump?)
@RichardGottschalk Unfortunately, ripping data is a bit of an art and requires specific background knowledge. (It could, however make a nice blog post, maybe when @eevee starts running out of topics to blog about...)

@encukou Alright, I'll have a look at a few GARCs when I have the time and fix mistakes or just confirm the data is right. Thanks for providing insight about the best practices here.

This has now been ripped directly, but thanks regardless  :)


Schema add UI: fix passing wrong data in the request header

ui bugfix
Issue

The issue is that UI is passing wrong data in the request header

Content-Type: text/plain;charset=UTF-8

Whereas Backend API expects application/json
@POST
  @Produces(MediaType.APPLICATION_JSON)
  @Consumes(MediaType.APPLICATION_JSON)
  @Path("/schemas/validate")

Solution

The issue occurred because UI is sending a stringify version of the request object, so contentType: text gets auto-added. Sending the data as json object solves the issue.

Codecov Report

Merging #11602 (5ff0234) into master (4292356) will increase coverage by 0.12%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #11602      +/-   ##
============================================
+ Coverage     62.92%   63.04%   +0.12%
- Complexity     1103     1106       +3
============================================
  Files          2326     2326
  Lines        124984   124984
  Branches      19080    19080
============================================
+ Hits          78641    78795     +154
+ Misses        40730    40578     -152
+ Partials       5613     5611       -2

Flag
Coverage Δ

integration
<0.01% <ø> (ø)

integration1
<0.01% <ø> (ø)

integration2
0.00% <ø> (ø)

java-11
63.02% <ø> (+48.51%)
:arrow_up:

java-17
62.91% <ø> (+48.41%)
:arrow_up:

java-20
62.91% <ø> (+<0.01%)
:arrow_up:

temurin
63.04% <ø> (+0.12%)
:arrow_up:

unittests
63.03% <ø> (+0.12%)
:arrow_up:

unittests1
67.44% <ø> (+0.17%)
:arrow_up:

unittests2
14.50% <ø> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 25 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[bug] Zooz ZEN25: toggling 2nd switch shows both as on/off

Version
Checklist:

[X] I am not using HomeAssistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog problem was not mentioned there.

Build/Run method

[X] Docker
[ ] PKG
[ ] Snap package
[ ] Manually built (git clone - npm install - npm run build )

zwavejs2mqtt version: 3.0.2
zwave-js version: 7.0.0
Describe the bug
With the ZEN25 when the 2nd switch is changed to on/off it also toggles the 1st switch so that the first switch shows it's on when it physically is not.
Steps to reproduce the behavior:
Open Control under ZEN25 Binary Switch toggle, target value 37-2 to on.  Both current values for 37-1 and 37-2 change to "true", despite only 37-2 physically turning on.  Changing it back to off will also change both values to false.
Expected behavior
Only 37-2 changes to true.
Additional context
Just updated from V2.0.0 where it was working and it also seems to work on V2.1.0 but, all versions after that give me the same result.  The result is the same regardless of where you trigger the change, in the control panel, in Home Assistant, or physically hitting the switch.  Also, I have 4 of the same switch ( Nodes 8, 9, 19, 20 ), and all 4 produce the same results.
zwavejs_1.log

This should be fixed by https://github.com/zwave-js/node-zwave-js/pull/2160


.eslintrc extends not working

Our .eslintrc file "extends": "eslint-config-airbnb", but with this configuration we don't receive the expected errors from our code (no errors are returned). Without this line, some errors are returned but obviously not all errors we expect.

I'm having this issue too. Any ideas?

Interesting. The reason is that eslint is a npm package. What it normally would do is require('eslint-config-airbnb'), which you probably have installed using npm.
However, this is some 'standalone' version of eslint running in the ruby environment, which doesn't know about node_modules and all of that.
Correct me if i'm wrong, but I don't think this is going to work.
@sandlerr ? ideas?

Couple of thoughts...

ExecJS says they don't support require - but commonjs.rb does - so theres that...
Executing eslint from ruby and capturing the output (like @ocke mentioned in #6) seems like a reasonable approach, but I'd prefer not to have a global eslint installed since (for me) this is running on CI. My pre-commit hook runs the linter with npm, so it makes sense that this plugin would execute the same command.
I did play (unsuccessfully) with the --print-config option for eslint to see if I could fully hydrate the .eslintrc file.  If all the configuration was fully generated before running this process (it's just a JSON object at the end of the day) this library would probably work as expected.


Add support for Google OAuth2

The add-on don't work out of the box  (at least very stable)  with Google OAuth2 (aka google plus authentication). The problem is that the response hook is dynamic (increasing number in the address).
My solution was just to make the response address static:
https://github.com/mstahv/vaadin-oauthpopup/commit/b409b34624054c5357314944394f51ac24defde1
Don't know if it is a properly working generic solution, but works for me.

This has been fixed with pull request #11


Can't pledge for an Aid Package

Is your feature request related to a problem? Please describe.
Currently, nothing happens when trying to pledge for a package.
{donorID}/aidpackages/{packageID}/pledge throws a CORS error

It seems elixir.redcross.lk domain isn't whitelisted. @manuranga could you please assign someone from the team?

This now should work

https://apim.docs.wso2.com/en/latest/deploy-and-publish/deploy-on-gateway/choreo-connect/troubleshooting/error-handling/

This happened due to the wrong configuration values for the BE DB.
Issue was identified by looking at the Choreo observability logs.
This is now fixed in the development and production deployment of the donor API

@jayasanka-sack


Nodejs using HTTP server instead of HTTPS

Rule finding: https://semgrep.dev/r?q=problem-based-packs.insecure-transport.js-node.using-http-server.using-http-server
Code link: https://github.com/lucky-org/website-backend/blob/c71fccf73e40927a1a8bd700949469c99ec7a5e7/server.js#L31

I talked with Prakash regarding this issue, he said that as we would be running our tests locally it would take some extra work to mock the TLS connection, and also as the backend also uses Cloudflare the requests are going to be secured anyways, so it won't make a difference.

Hey, I don't have enough context on how our tests run, can you give me some context on our tests? then I would be able to look into what kind of options we would have. I think if we are using any modern library for the HTTP connection it would automatically support HTTPS.
As for Cloudflare, Cloudflare gives HTTPS between the user and the services but in the future if we go for a microservice architecture.  These services would have to talk to each other on HTTP since we don't have HTTPS on a service level.
So in the attached example image if Service 1 has a command execution vulnerability i.e. I can run shell commands in the service. I can run a program  TCP dump which will let me record all traffic on the network and since it's all plaintext HTTP I would be able to read all the data.

So let's say one of your front-end services gets hacked, it doesn't have anything sensitive but there are other services e.g payment service running in the same network.  I would be able to dump all the traffic to and from the payment service since it's all HTTP.
If you need any more clarification let me know I will add more details or we can get on a call as well.


Rewrite to Entra UI

Could you rewrite the labs to Entra UI

In process.  Should be done in the next week to 10 days.  I was waiting on the final UI updates.


Missing data / link

In this link : Guest VM Setup , when clicked on 'Guest Owner Setup', the data/link is missing which is supposed to explain how to setup a guest VM.

Hi Nitish,
I'm currently updating the install instructions for the Guest Owner's infrastructure. I'll update this issue when it's posted. Thanks!
Sincerely,
Jesse

Hi Nitish,
I've added the Guest Owner setup instructions here. Thanks for the report!
Sincerely,
Jesse


Reparenting frozen RigidBody3D breaks smoothing

Hi there. I have a CharacterBody3D character in my game that when they pick up a RigidBody3D, it static-freezes that object and reparents the object to the character, as well as adding the necessary collision-exclusion stuff. This mostly works, except for smoothing. The physics object itself will attach and move with the character as desired, which can be checked using Debug > Visible Collision Shapes, but the mesh stops moving with its parent object entirely.
Screencast from 07-27-2024 08:15:02 AM.webm
In the above video, the left object is a normal cube, while the right object is using Smoothing. The expected behavior is that the mesh on the right will track the right collider, and generally look similar to the cube on the left. Both cubes reparent to the player after a 1 second delay.
SmoothingDesync.zip
And here's the repro project. Godot 4.2.2 + Smoothing 1.2.1. I've also verified it affects Godot 4.3-rc1 mono.
I don't have a workaround yet, but I was thinking of reparenting the mesh node depending on whether the item is being held. But also I don't really want to do that :stuck_out_tongue:
Thanks for looking!

Sorry I took a while, I've been afk for a few days.
It looks like the problem is that on reparenting, you need to reset what the smoothing node is following. For convenience it automatically selects the parent node on _ready(), but you are changing the node path by reparenting.
The solution is to reset this path by assigning the target again after reparenting:
$Smoothing.target = ".."
You may need to use a different script for the two different blocks in this example for it to work (as the control block doesn't have a smoothing node).
This could possibly be handled better by the addon, but not a great reason to change as core physics interpolation should be available soon even in 4.x (after 4.3).

Is the problem the target node_path is resolved to an absolute path at runtime?

Is the problem the target node_path is resolved to an absolute path at runtime?

Yes, it does a few things like this for safety, should the node be deleted, if I remember right (I wrote it years ago).

The more I dive into this, the more interesting it gets :slightly_smiling_face: I think I see the problem.
The good news is _enter_tree is executing as expected when reparenting the object -- it's just the code isn't working the way we hope. And the culprit is...this line:
https://github.com/lawnjelly/smoothing-addon/blob/0c4ae88fd752e011bb925bfdaec9669a33798917/addons/smoothing/smoothing.gd#L144
During _ready, set_as_top_level(true) is called, which obviously sets the object as a top-level object:
https://github.com/lawnjelly/smoothing-addon/blob/0c4ae88fd752e011bb925bfdaec9669a33798917/addons/smoothing/smoothing.gd#L88
And we know _ready gets called after _enter_tree, because...in DOM terms, "enter tree" is called during the "capture" phase, while "ready" is called during the "bubbling" phase. That is, we call _FindTarget() (in _enter_tree) before calling set_as_top_level(true) (in _ready).
get_parent_node_3d is a little weird:

Returns the parent Node3D, or null if no parent exists, the parent is not of type Node3D, or top_level is true.
Note: Calling this method is not equivalent to get_parent() as Node3D, which does not take top_level into account.

(emphasis mine)

Basically, get_parent_node_3d will only ever work when the Smoothing scene is first created because it's called before top-level is enabled.
I think the solution is simply using get_parent. I'll test that out and sent our a PR for it if so.


Insert Rows and Columns - A Thank you!!!

Assuming you are interested in adding them, here is code for inserting rows and columns.  Guess you will want to tweak things a bit as well as making sure I didn't mess up something.  Tried to keep things in line with your other code.  Check on the cCount thing I did in the row insert.  Probably a better way since you will know what you are doing.
Thanks again.
jim
    /**
     * @brief   Insert row by index.  Inserts at postion so new row will preceed specified row.
     * @param   pRowIdx               zero-based row index.
     */
    void InsertRow(const size_t pRowIdx, const string defValue = "", const int cCount = 0)
    {
        const ssize_t rowIdx = pRowIdx + (mLabelParams.mColumnNameIdx + 1);
        vector<string> insVector(GetColumnCount()+cCount, defValue);
        mData.insert(mData.begin() + rowIdx, insVector);
    }

    /**
     * @brief   Insert row by name.  Insert at postion so new row will preceed specified row.
     * @param   pRowName              row label name.
     */
    void InsertRow(const std::string& pRowName, const std::string newRowName = "", const string defValue = "")
    {
        ssize_t rowIdx = GetRowIdx(pRowName);
        if (rowIdx < 0)
        {
            throw std::out_of_range("row not found: " + pRowName);
        }

        InsertRow(rowIdx, defValue, 1);
        SetRowName(rowIdx, newRowName);
    }

    /**
     * @brief   Insert column by index. Inserts at specified position so will preceed column at current position.
     * @param   pColumnIdx            zero-based column index.
     */
    void InsertColumn(const size_t pColumnIdx, const string defValue = "")
    {
        const ssize_t columnIdx = pColumnIdx + (mLabelParams.mRowNameIdx + 1);
        for (auto itRow = mData.begin(); itRow != mData.end(); ++itRow)
        {
            itRow->insert(itRow->begin() + columnIdx, defValue);
        }
    }

    /**
     * @brief   Insert column by name.  Inserts at specified position so will preceed column at current position.
     * @param   pColumnName           column label name.
     */
    void InsertColumn(const std::string& pColumnName, const string newColName = "", const string defValue = "")
    {
        ssize_t columnIdx = GetColumnIdx(pColumnName);
        if (columnIdx < 0)
        {
            throw std::out_of_range("column not found: " + pColumnName);
        }

        InsertColumn(columnIdx, defValue);
        SetColumnName(columnIdx,newColName);
    }

Thanks a lot! I will take a look and see how to integrate this. It makes sense to support inserting columns/rows.


[vue-child] fix interfaces of main function

Passing the vue router to the createMicroFrontendChild function is working as expected, and transpile from TS to JS as expected, to but IDE shows warnings like the router is a mistake.

Here you go, hope that helps.


Sleep while RateLimitExceeded instead of crash the whole server

Fixes #527

I'm wondering if there's some way that we can inject the Octokit::Client onto the github_action or create the Octokit::Client in the github_action in order to retrieve rate_limit, scope. It'd be easy to do:
github_action(access_token:) do |client|
    client.whatever action
end

or even to make it less confusing:
github_client(*params) do |client|
   client.whatever_action
end

@minuscorp that's a good idea. What if we just require passing in client as a param instead of access token?

Fair enough


Backports 0.18 pr3

Pls rebase and drop 14006 (backported in #4220) and 13765 ("backported" via #2319)


feat: Optional token scale prefix

This PR adds the ability to remove scale from CSS variables.
i.g.:
Currently:
export const { styled, css, config, theme, createTheme } = createStitches({
  theme: {
    colors: {
      "my-pink": "#FFBDC9"
    },
  },
});

the injected CSS variable is: $colors-my-pink: #FFBDC9

Proposed feature:
export const { styled, css, config, theme, createTheme } = createStitches({
  hasScalePrefix: false, // default true
  theme: {
    colors: {
      "my-pink": "#FFBDC9"
    },
  },
});

the injected CSS variable is: $my-pink: #FFBDC9

I understand the config flag is optional, but since the config is global, it's gonna break cases when you need to access other scales from within CSS properties?
This example is super incorrect, it's just to show the cases when you need access to scales:
$$size: 'calc($scales$md - 200px)',
width: 'calc($scales$md - 200px)',
background: 'linear-gradient(to left, $colors$my-pink, #333 $$size)',

I understand the config flag is optional, but since the config is global, it's gonna break cases when you need to access other scales from within CSS properties?
This example is super incorrect, it's just to show the cases when you need access to scales:
$$size: 'calc($scales$md - 200px)',
width: 'calc($scales$md - 200px)',
background: 'linear-gradient(to left, $colors$my-pink, #333 $$size)',

Some would argue that magical mapping might not be the best for consistency (I love it though).

Oh well, this is a good point of view.
We suggested this new feature because we want to write the entire token when writing the CSS styles.
i.g.
export const { styled, css, config, theme, createTheme } = createStitches({
  prefix: "dy",
  theme: {
    colors: {
      "my-pink": "#FFBDC9"
    },
  },
});

so, the injected CSS variable is: $dy-colors-my-pink: #FFBDC9
and we want to write the CSS style like this:
$$size: 'calc($dy-scales-md - 200px)',
width: 'calc($dy-sizes-200 - 10px)',
background: 'linear-gradient(to left, $dy-colors-my-pink, #333 $$size)',

If there was such a feature that allowed to write the token completely? You can help us?

@lcoalves Thanks for your contribution but unfortunately, we can't merge this feature for the reason @LexSwed outlined.
In addition to that, we are also trying to keep the API as concise and consistent as possible to make the documentation easier to follow.
Gonna close this issue but again thank you so much ❤
Please feel free to create a feature request if you'd like to validate something with the community or the core team before working on it.


compilation error | fail

OS: Ubuntu 16.04
I'm having trouble compiling with 'make'. The errors and warnings appear to be related to hdf5 sources but I can not really tell. The make log is attached and any advice is greatly appreciated!
Thanks and Cheers,
Ben
make.log

Hi,
It looks like it compiled to me, does the program not run?

Thanks for the rapid reply! I'm just getting the generic:
~/nanopolish$ nanopolish
The program 'nanopolish' is currently not installed. You can install it by typing:
sudo apt install nanopolish
-Ben

Can you try running as ./nanopolish

Success, thank you! I added to my $PATH and now we're cookin'.

Great!


Obtain path to elm file without query

Use resourcePath instead of getRemainingContext to obtain absolute path to elm entry point.
Using the getRemainingContext caused an error when this loader is used with any other loader (aka when the loaderContext.loaders array contains more loaders than just elm-webpack-loader).
Tests actually mock resourcePath and leave loaders as an empty array:
https://github.com/rtfeldman/elm-webpack-loader/blob/master/test/loader.js#L40

BTW, this was the only change I needed to make a first working version of elmx-webpack-preloader to precompile .elmx files to .elm before using elm-webpack-loader
#39

seems good, thanks!


Update the page for Streaming Integration profiles

Update [1].
[1] https://github.com/wso2/docs-ei/blob/master/en/streaming-integrator/docs/ref/wSO2-Stream-Processor-Profiles.md

Obsolete.