Unable to get env vars to load

Component
Forge
Have you ensured that all of these are up to date?

[x] Foundry
[x] Foundryup

What version of Foundry are you on?
No response
What command(s) is the bug in?
forge test
Operating System
Linux
Describe the bug
I am unable to load enviroment variables from .env into my foundry.toml the RPC-endpoint settings of https://github.com/foundry-rs/foundry/blob/master/config/README.md
In my test:

In my foundry.toml

Console

Inside my .env, I have set both INFURA_API_KEY and FOUNDRY_INFURA_API_KEY to correct api key.

can you paste the error hex code, that should be encoded error message.
what does forge config print for rpc_endpoints?

I did run source .env.
[2909] LootboxTest::setUp()
    ├─ [0] VM::rpcUrl("rinkeby")
    │   └─ ← 0x0bc445031644df03923eb2ab981d332f4354ceab11a95efce72a938e57beaadf0000000000000000000000000000000000000000000000000000000000000020000000000000000000000000000000000000000000000000000000000000004a4661696c656420746f207265736f6c766520656e76207661722060494e465552415f4150495f4b4559603a20656e7669726f6e6d656e74207661726961626c65206e6f7420666f756e6400000000000000000000000000000000000000000000
    └─ ← 0x0bc445031644df03923eb2ab981d332f4354ceab11a95efce72a938e57beaadf0000000000000000000000000000000000000000000000000000000000000020000000000000000000000000000000000000000000000000000000000000004a4661696c656420746f207265736f6c766520656e76207661722060494e465552415f4150495f4b4559603a20656e7669726f6e6d656e74207661726961626c65206e6f7420666f756e6400000000000000000000000000000000000000000000

Forge config (after running source .env):

Seems like it can't find the envvar in .env

can you please double check that INFURA_API_KEY is set when you run forge?

Yes and ‘FOUNDRY_INFURA_API_KEY’ is set but only inside the .env. If I run ‘echo $INFURA_API_KEY’ it returns nothing until I run ‘source .env’ at which point the echo returns the right value. But ‘forge test’ and ‘forge confit’ still can’t findit

can't reproduce with latest forge

Can you also show me your .env and foundry.toml?

[rpc_endpoints]
rinkeby = "https://mainnet.infura.io/v3/${INFURA_API_KEY}"

contract ContractTest is Test {
    function setUp() public {}

    function testExample() public {
        string memory url = vm.rpcUrl("rinkeby");
        console.log(url);
    }
}

your error is most likely in your env file, forge doesn't know env files only considers set vars, that's what
INFURA_API_KEY="123456" forge test -vvv does

Please ensure your .env file exports the environment variables if you use source, otherwise the environment variables will be local to your shell and not propagate automatically to any processes you spawn.
For example:
Doesn't work (.env does not export env vars)
$ cat .env
FOO=bar
$ source .env

# FOO is local to your shell, but no program you spawn can use it
$ echo $FOO
bar

Works (.env does export env vars)
$ cat .env
export FOO=bar
$ source .env
# FOO is available in your shell, but child processes will also have access to it
$ echo $FOO
bar

Yes! This was the problem.  Works amazingly now. It would great, for others' sake, if that little snippet was added to docs.


Fix many many join tables

Here is a pull request aiming to fix issue #46 .
I did the next things:
-> Update prisma related packages to the last version.
-> Fix the generation of many many join tables to match the new DMMF format.
-> I let some comments to facilitate the code review. I'll remove them when not usefull anymore.
@marcjulian Let me know if there is something else I can do =)

Hi @maxmousse thanks for fixing the many to many table issue. Yes looks like the DMMF has a new format, because it is an internal API it may cause breaking changes between minor versions.
I like the comments, they can be useful when the DMMF changes again.


:wrench: Update pokedex plugin

If you want to update plugin info, you need to find it in plugins folder
Plugin already auto-updated in https://github.com/Flow-Launcher/Flow.Launcher.PluginsManifest/commit/5a9edf0e60efaf533a610fbc7903cb306287d6fd

Fixed so that only description requires updating.


Output more details on task processing

I may just need to RTFM a bit harder but is there a way to get more descriptive output of the synopsis of a task during the Invoke-Build process? I've put in a helper function that I'm using for something like this that gives a bit more descriptive output as to what the particular task is doing in a project I'm trying to refine  here.
As always, thanks for the great project

Do you mean your function Write-Description? If yes, are you suggesting to make something like this built-in? If no, please clarify. Perhaps I am slightly confused by "task synopsis" which means comments like # Synopsis: ...., in IB terms.

Sorry for being obtuse. I'm thinking something like the write-description helper I used but built in as an output option when running tasks. It gives a better visual in the console and VS code when running for those not intimately familiar with larger sets of someone else's build tasks. Possibly including the synopsis output from 'Invoke-Build ? | Select Name,Synopsis)' with the build output? I don't know what that looks like though, an additional parameter perhaps?
I'm just floating the idea out there though. IB is a pretty tight script through so the counterargument might be that this kind of superfluous feature may not have a place.

Can you give an example of the suggested output, as you see it?
Is it something like this, perhaps?
The task:
    # Synopsis: Do something and something else.
    task Task1 {
        'some output'
    }

Output with the task synopsis included:
    Build Task1 C:\...\_170613_051456_fr74\.build.ps1
    Task /Task1 : Do something and something else.
    some output
    Done /Task1 00:00:00.0207639
    Build succeeded. 1 tasks, 0 errors, 0 warnings 00:00:00.5391652

That is exactly what I was thinking. I imagine that if a task was already run (output is grey) that the description would not be displayed.

This is doable and not so difficult. I do not think the feature is superfluous,
it makes the output easier to analyze and task synopsis useful not just for ib ?. I like the idea.
But let me think for some time about the details.
(1) Is it a whole build option or a script option? In any case, is it just a
"switch" or should we have some sort of configurable task header? Tomorrow we
may want to use something instead or in addition to task synopsis, e.g. task
location (file:line), so that you can navigate to a task definition in VSCode
by Ctrl+Click.
(2) Should it be written on the same like with task /<path> or on a separate
line? Note that task paths may be long. In the case of a configurable task
header this is up to a user, then the question is what to use by default.
(3) We can start with something like exposing Get-BuildSynopsis which might
be useful in any case. Then
    Enter-BuildTask { Write-Build cyan (Get-BuildSynopsis $Task) }

will do the trick right now (in some way, maybe not the best, but at least we
will know why it is not the best). It is quite flexible, e.g. you can:
    Enter-BuildTask {
        # name and synopsis
        Write-Build Magenta "$($Task.Name) : $(Get-BuildSynopsis $Task)"

        # location for Ctrl+Click in VSCode
        Write-Build DarkGray "$($Task.InvocationInfo.ScriptName):$($Task.InvocationInfo.ScriptLineNumber)"
    }

I figured that the actual implementation might be somewhat tricky based on what I've been learning about this project.
I'm just spitballing here but would it possibly make sense to include an InvokeBuild config block that pulls into the current runspace predefined variables related to the InvokeBuild settings and output configuration? That would eliminate extra parameters (which I believe you do try to keep to a minimum) and give you more room to expand features moving forward. perhaps something like:
_invokebuildsettings {
    # Any variable defined in this special task will be made global to the entire build process
    # There are several predefined variables that can be used to affect the invoke-build process or output

    # Display task output without synopsis details (default setting = $true)
    $_invokebuild_TerseOutput = $true

    # Display synopsis information on its own line if terse output is disabled (default setting = $true)
    $_invokebuild_VerboseOutputNewLine = $true
}

Maybe that would allow you to bake in some of these things with a bit more freedom? I'm sorry but I've not really dug into your scary code enough to be speaking intelligently on this matter though.

I would like to avoid settings and implementing output logic inside IB depending on settings. This is less flexible. I will push the preview of (3), so that Enter-BuildTask you can do and output what you want, in particular, output Get-BuildSynopsis $Task. No settings are needed for this approach and yet you have full freedom on what and how to output, what colors to use, and etc.

Please, pull the branch preview and take a look at two added features, one is
almost final, another is yet questionable.
Feature 1 (likely to be released), the new function Get-BuildSynopsis
How to use for more details on task processing. In your build script define
Enter-BuildJob. Example:
    # Print synopsis and location (e.g. for Ctrl+Click in VSCode)
    Enter-BuildJob {
        Write-Build Cyan (Get-BuildSynopsis $Task)
        Write-Build DarkGray "$($Task.InvocationInfo.ScriptName):$($Task.InvocationInfo.ScriptLineNumber)"
    }
    # Synopsis: Some task description 1.
    task Task1 Task2, {'some output 1'}
    # Synopsis: Some task description 2.
    task Task2 {'some output 2'}

As a result, the build output looks like

Feature 2 (is it needed?), the new block Write-BuildHeader
It deals with the automatic variable $Task and one argument, the current task
path. If it is not defined in a build script the default is called.
Namely,
    {
        param($Path)
        Write-Build Cyan "Task $Path"
    }

If it is defined then it writes whatever and how it wants.
For example, for the header with synopsis this block defines:
    Write-BuildHeader {
        param($Path)
        Write-Build Cyan "Task $Path : $(Get-BuildSynopsis $Task)"
    }

Example script with the above sample:
    # Print task path and synopsis
    Write-BuildHeader {
        param($Path)
        Write-Build Cyan "Task $Path : $(Get-BuildSynopsis $Task)"
    }
    # Synopsis: Some task description 1.
    task Task1 Task2, {'some output 1'}
    # Synopsis: Some task description 2.
    task Task2 {'some output 2'}

As a result, the build output looks like

These two features cover the request completely (in my opinion), even though the way of engaging is not like the proposed. What do you think?
The question is: do we really needed the second feature? It adds some more flexibility, yes. How much is it important?

After playing with this I think we need Write-BuildHeader. It gives the full
control of task headers. Also, it is called on -WhatIf, unlike Enter-BuildJob.
I am going to release these new features soon.
P.S. Build parameters or options like TerseOutput and VerboseOutputNewLine
would be less fiddly, perhaps. But what if some users want custom colors or
task start times or something else? Then we add more (and more) parameters
or answer "Sorry, you cannot". With the new Write-BuildHeader they can.

NB: I will use the name Set-BuildHeader instead of Write-BuildHeader.

Pushed the latest to preview. All looks good, a bit later today or tomorrow I will release this.

v3.5.0 supports custom task headers. Let's see how it goes and what else is needed. I can see some possible further development but it is too early.


Collaborate with FsicalTeam on Tax Strategy and Filing

Objective:
Work with the Physical Team to address tax strategy and filing for the upcoming week. This includes reviewing relevant paperwork and assisting with simulations and other required tasks.
Tasks:
Review Tax Strategy Paperwork:
Thoroughly read and analyze the tax strategy and filing documents.
Identify key insights and areas needing attention or improvement.
Provide Feedback:
Compile feedback based on the analysis of the tax documents.
Ensure feedback is clear, actionable, and aligned with the overall strategy.
Assist with Simulations:
Collaborate with the Physical Team to understand their simulation needs.
Offer support and insights to enhance simulation results.
Coordinate with Team Members:
Maintain open communication with the Physical Team to ensure alignment.
Address any questions or concerns promptly.
Documentation:
Document the process and outcomes of the collaboration.
Prepare a summary report highlighting key achievements and areas for further improvement.

I will have effectively collaborated with the Fiscal Team by providing comprehensive feedback on the tax strategy and filing documents. This will involve delivering detailed and actionable insights that align with our strategic goals and address key areas for improvement. Additionally, I will offer continuous feedback, maintaining an ongoing dialogue with the team to provide timely suggestions and adjust insights as needed based on evolving requirements. Through this process, I aim to ensure that all tasks are completed efficiently, coordinating closely with the team to address any challenges promptly and facilitate a smooth workflow to meet all deadlines and objectives.

I will begin by scheduling a meeting with Aryab to meticulously review the numbers and itemization of the expenses. During this session, we'll examine each line item to ensure accuracy and proper categorization, addressing any discrepancies or questions that arise. It's crucial to confirm that all necessary documentation and receipts are accounted for. Following this, I will arrange a separate meeting with Suzanne to go through the document in detail. I'll provide her with an overview of the key points and objectives, highlighting sections that require her attention or input.

I will begin by scheduling a meeting with Zaryab to reconcile the numbers and itemization of the expenses. During this session, we'll examine each line item to ensure accuracy amongst the income and expenses and proper categorization, addressing any discrepancies or questions that arise. It's crucial to confirm that all necessary documentation and receipts are accounted for. Following this, I will arrange a separate meeting with Suzanne to go through the document in detail. I'll provide her with an overview of the key points and objectives, highlighting sections that require her attention or input.


Got errors when evaluating.

Hi, what a great job!
I have set the cuda devices set CUDA_VISIBLE_DEVICES=0 and i run the evaluation code:python -m src.main +experiment=acid checkpointing.load=checkpoints/acid.ckpt mode=test dataset/view_sampler=evaluation dataset.view_sampler.index_path=assets/evaluation_index_acid.json test.compute_scores=true
there is something wrong with my code:
Setting up [LPIPS] perceptual loss: trunk [vgg], v[0.1], spatial [off]
Loading model from: C:\Users\user.conda\envs\mvsplat\lib\site-packages\lpips\weights\v0.1\vgg.pth
Initializing distributed: GLOBAL_RANK: 1, MEMBER: 2/2
[W socket.cpp:663] [c10d] The client socket has failed to connect to [DESKTOP-M7IQG5O]:2765 (system error: 10049
Error executing job with overrides: ['+experiment=acid', 'checkpointing.load=checkpoints/acid.ckpt', 'mode=test', 'dataset/view_sampler=evaluation', 'dataset.view_sampler.index_path=assets/evaluation_index_acid.json', 'test.compute_scores=true']
Traceback (most recent call last):
File "F:\Github\mvsplat\src\main.py", line 143, in train
trainer.test(
File "C:\Users\user.conda\envs\mvsplat\lib\site-packages\pytorch_lightning\trainer\trainer.py", line 754, in test
return call._call_and_handle_interrupt(
File "C:\Users\user.conda\envs\mvsplat\lib\site-packages\pytorch_lightning\trainer\call.py", line 43, in _call_and_handle_interrupt
return trainer.strategy.launcher.launch(trainer_fn, *args, trainer=trainer, **kwargs)
File "C:\Users\user.conda\envs\mvsplat\lib\site-packages\pytorch_lightning\strategies\launchers\subprocess_script.py", line 105, in launch
return function(*args, **kwargs)
File "C:\Users\user.conda\envs\mvsplat\lib\site-packages\pytorch_lightning\trainer\trainer.py", line 794, in _test_impl
results = self._run(model, ckpt_path=ckpt_path)
File "C:\Users\user.conda\envs\mvsplat\lib\site-packages\pytorch_lightning\trainer\trainer.py", line 943, in _run
self.strategy.setup_environment()
File "C:\Users\user.conda\envs\mvsplat\lib\site-packages\pytorch_lightning\strategies\ddp.py", line 154, in setup_environment
self.setup_distributed()
File "C:\Users\user.conda\envs\mvsplat\lib\site-packages\pytorch_lightning\strategies\ddp.py", line 203, in setup_distributed
_init_dist_connection(self.cluster_environment, self._process_group_backend, timeout=self._timeout)
File "C:\Users\user.conda\envs\mvsplat\lib\site-packages\lightning_fabric\utilities\distributed.py", line 291, in _init_dist_connection
torch.distributed.init_process_group(torch_distributed_backend, rank=global_rank, world_size=world_size, **kwargs)
File "C:\Users\user.conda\envs\mvsplat\lib\site-packages\torch\distributed\c10d_logger.py", line 74, in wrapper
func_return = func(*args, **kwargs)
File "C:\Users\user.conda\envs\mvsplat\lib\site-packages\torch\distributed\distributed_c10d.py", line 1148, in init_process_group
default_pg, _ = _new_process_group_helper(
File "C:\Users\user.conda\envs\mvsplat\lib\site-packages\torch\distributed\distributed_c10d.py", line 1268, in _new_process_group_helper
raise RuntimeError("Distributed package doesn't have NCCL built in")
RuntimeError: Distributed package doesn't have NCCL built in
I follow the instructions and i don't know why? Plz help!

Hi @karaokenoway, thanks for your interest in our work.
From your log, it seems that you are running this project on a Windows machine. We have only tested our codes on Linux machines, and unfortunately, we do not have too much experience with running codes on Windows...
You might need to search for how to ensure codes running on only 1 GPU on Windows with pythorch-lightning, since the log (Initializing distributed: GLOBAL_RANK: 1, MEMBER: 2/2) seems to show that it is running on 2 GPUs.


Add a blackHole

The library should provide a way to ensure that code under test isn't optimized away.  The only reliable way I know of to do this that doesn't cost more than a function call is:
notOptimizedAway<T>(_ x: T) {
  withUnsafePointer(to: x) { opaqueCFunctionTakingVoidConstStar($0) }
}

(storing values into Any can be costly if they need to be boxed, and anyway if the compiler can prove the Any is never read, it may one day optimize it out).

Maybe this is better, though it uses a compiler internal:
@_optimize(none)
fileprivate func doNotOptimizeAway1(_ x: UnsafeRawPointer) {}

@inline(__always)
fileprivate func doNotOptimizeAway<T>(_ x: T) {
  withUnsafePointer(to: x) { doNotOptimizeAway1($0) }
}


channels: more accurate unread counts

The improvement here is two-fold:
First, we no longer recalculate the "most recent content" timestamp when marking a channel as read. Instead, we simply reuse the .recency which we were already tracking anyway. I don't remember why we thought we had to re-derive that on the spot. (2928b4b)
Second, we update the $unread type to provide, in addition to the total unread count, sub-counts for each context in which unreads may exist: top-level and within reply threads. (be58f62)
Draft because this does not yet include the frontend updates required for the latter change. cc @patosullivan
For LAND-1393 and LAND-1394.

Actually the top-level count (not the total) provided by this is inaccurate for cases where both a top-level message with replies is below the top-level marker: in that case the replies should be included in the top-level count, but aren't.
Fixing...

Label do not merge added only because we're approaching a long holiday break, and we may prefer to merge once everyone is back to manage risks. (I'm open to reviewing this decision)


Fix incorrect LLC partitionRange extraction

Fix the extraction of partitionRange for the low level consumer.
Update unit test.

LGTM

[Current coverage][cc-pull] is 38.61%

No coverage report found for master at ef78c3f.

Powered by Codecov. Last updated by [ef78c3f...bb007df][cc-compare]
[cc-base-branch]: https://codecov.io/gh/linkedin/pinot/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/linkedin/pinot/compare/ef78c3f0acb287a7238c08dfaa102d4641c1ae00...bb007df76eb909f909885c3876882f3608b436ee
[cc-pull]: https://codecov.io/gh/linkedin/pinot/pull/214?src=pr

Can we not have partition ranges in the segment name please?

lgtm

Discarding this PR as it is obsoleted by #216 and #219.


add simple cache to cost model

this very simple implementation can have quite good hit rate:
0.0
0.0
0.0
0.0
0.2
0.16666666666666666
0.2857142857142857
0.375
0.4444444444444444
0.5
0.5454545454545454
0.5833333333333334
0.6153846153846154
0.6428571428571429
0.6666666666666666
0.6875
0.7058823529411765
0.7222222222222222
0.7368421052631579
0.75
0.7619047619047619
0.7727272727272727
0.782608695652174
0.7916666666666666
0.76
0.7307692307692307
0.7037037037037037
0.7142857142857143
0.6896551724137931
0.7
0.6774193548387096
0.6875
0.696969696969697
0.7058823529411765
0.7142857142857143
0.7222222222222222
0.7027027027027027
0.7105263157894737
0.717948717948718
0.725
0.7317073170731707
0.7380952380952381
0.7441860465116279

show by running at the same time watch -n 1 cat /tmp/cozy_cache*

but the total numbers for a query is actually low, caused by insufficient sharing between each cost_model instance I guess. I am not sure it is safe to share, but sharing should be a simple hack

also, I felt like cozy becomes faster with this. However, there is not strict validation yet

lru_cache from functools also works

I added a memcached version -- yeah I am kinda crazy

after a few warmup for memcached, cache hit rate for all cost_model instances are 80% and up

Sharing a cache between cost model instances is dangerous since there are member variables on each cost model that affect its behavior. Where you inserted the cache, I think that includes only self.assumptions and self.funcs. If you want a global cache you will need to include that information in the cache key as well.

CostModel._compare is a pure function of four inputs---just think of the self argument as a real input of equal importance to the others.
The cost model could be a plain old function of two expressions and many other inputs, but it helps efficiency a lot to do some up-front setup using the inputs that don't change very much. In particular, sending assumptions to the solver once and re-using the same solver over and over provides massive speedup.
This design is a tradeoff in code clarity: the cost model code itself is more difficult to follow, but client code is easier to read since all the extra parameters that the cost model needs have been abstracted away.


Add restarts for models and controllers, take care of tensorflow graph expanding - #13

Slowly merging changes from the more_envs branch to master.
In this pull request, I added the restarts, of the controller and the model, along with the modification we had to do to the optimizer due to the graph expanding (see discussion in issue #13).
Units tests are passing, but I haven't written new tests for the restarts.
Since

they are by their nature random
there is no Matlab equivalent to test against

what we could do is perform a number of restarts and make sure that the objective is always getting better or equal to the starting value (as it should, because when the restart fails the previous values are restored ). It's statistical, so to make sure we should perform more than one, but one the other hand we don't want the unit tests to take too long.
What needs to be added also is a restart function for the smgpr.py, because the one used for the standard mgpr.py does not work.

Codecov Report

Merging #16 into master will decrease coverage by 10.35%.
The diff coverage is 40%.

@@             Coverage Diff             @@
##           master      #16       +/-   ##
===========================================
- Coverage   96.37%   86.01%   -10.36%
===========================================
  Files           7        7
  Lines         276      329       +53
===========================================
+ Hits          266      283       +17
- Misses         10       46       +36

Impacted Files
Coverage Δ

pilco/controllers.py
88.23% <25%> (-8.44%)
:arrow_down:

pilco/models/mgpr.py
82% <30.76%> (-18%)
:arrow_down:

pilco/models/pilco.py
79.54% <53.84%> (-11.37%)
:arrow_down:

Continue to review full report at Codecov.

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

@kyr-pol many thanks for this. I have cleaned up a bit the code and I think that this can now be merged.
Let me know if you want to change something before merging.


nk.line draws faulty lines

To accurately draw lines in love, it is required to offset the coordinates by .5.
nk_love_draw_polyline() causes issues with accurate line rendering, because it ignores this rule and draws lines at integer coordinates.
This is a rectangle drawn with love.graphics.line:
function love.draw()
    love.graphics.setLineWidth(1.0)
    love.graphics.line(10.5, 10.5, 100.5, 10.5, 100.5, 100.5, 10.5, 100.5, 10.5, 10.5)
end

This is the same rectangle drawn with nk.line:
if nk.windowBegin("window", 0, 0, 200, 200) then
    love.graphics.setLineWidth(1.0)
    nk.line(10.5, 10.5, 100, 10, 100.5, 100.5, 10.5, 100.5, 10.5, 10.5)
end
nk.windowEnd()

Both rectangles should look identical, with a thickness of 1 pixel. The one drawn by nk.line is 2 pixels wide. It's not possible to draw lines with a thickness of 1 with nk.line.

This should be fixed in master and ready for the next release.


[Proposal] Rearrange how the redirect is implemented

Currently, the redirect logic is handled in the SocialController and always returns Redirect::route('home'). This causes some problems for apps who don't have a route named home. Ofc you can always override the SocialController class and write your own redirect response, but I think it would be a better user experience (for the developer) if the redirection logic is a bit more flexible by default.
If we take the login request then we could take the same approach that laravel/ui has taken for the redirect. That d mean that instead of calling authenticated() before authenticateModel(), we would call it afterwards.
// login logic
$social = $this->updateSocialInstance($request, $provider, $model, $providerUser);

$this->authenticateModel($model);

if ($response = $this->authenticated($request, $model, $social, $providerUser)) {
     return $response;
}

return redirect()->intended($this->redirectPath());

// redirectPath
    protected function redirectPath()
    {
        if (method_exists($this, 'redirectTo')) {
            return $this->redirectTo();
        }

        return property_exists($this, 'redirectTo') ? $this->redirectTo : route('home');
    }

That way we could utilize the alreadyt existent registered, authenticated, linked and unlinked methods to do custom redirects if required.
Would you be open to a PR which would make the necessary changes to the codebase @rennokki ?

This has to be implemented. The idea behind not using this was that you can extend the controller and overwrite the method, but that would mean DRY gets broken, and the redirect method is the way to go for extending the controller.

This was fixed. I was late a few months, haha. Read the new readme to get used to the new redirection stuff.


Add lib for Microsoft JDBC Drivers with maven

[INFO] Copying mssql-jdbc-6.1.0.jre8.jar to /root/traccar/target/lib/mssql-jdbc-6.1.0.jre8.jar
sample config for mssql
<entry key='database.driverFile'>lib/mssql-jdbc-6.1.0.jre8.jar</entry>
<entry key='database.driver'>com.microsoft.sqlserver.jdbc.SQLServerDriver</entry>
<entry key='database.url'>jdbc:sqlserver://[serverName][\instanceName];user=[userName];password=[password];databaseName=[database];</entry>
<entry key='database.user'>[userName]</entry>
<entry key='database.password'>[password]</entry>

I think we should be using JRE7 version.
Can we please get some confirmation from microsoft that it's allow to include it in installation package?

after get some confirmation from microsoft i tell you again for merge

dear, anton
today i see news for mssql in ubuntu lol
https://docs.microsoft.com/th-th/sql/linux/sql-server-linux-setup-ubuntu
sure can install on Ubuntu 16.10 on my test server
root@office:~# curl https://packages.microsoft.com/keys/microsoft.asc | apt-key add -
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   983  100   983    0     0    460      0  0:00:02  0:00:02 --:--:--   461
OK
root@office:~# curl https://packages.microsoft.com/config/ubuntu/16.04/mssql-server.list > /etc/apt/sources.list.d/mssql-server.list
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100    87  100    87    0     0    155      0 --:--:-- --:--:-- --:--:--   155
root@office:~# sudo apt-get update
Hit:1 http://th.archive.ubuntu.com/ubuntu yakkety InRelease
Get:2 http://th.archive.ubuntu.com/ubuntu yakkety-updates InRelease [94.5 kB]
Hit:3 http://th.archive.ubuntu.com/ubuntu yakkety-backports InRelease
Get:4 http://security.ubuntu.com/ubuntu yakkety-security InRelease [93.3 kB]
Get:5 http://ppa.launchpad.net/webupd8team/java/ubuntu yakkety InRelease [17.6 kB]
Get:6 https://packages.microsoft.com/ubuntu/16.04/mssql-server xenial InRelease [2,828 B]
Get:7 http://th.archive.ubuntu.com/ubuntu yakkety-updates/main amd64 Packages [77.0 kB]
Get:8 http://th.archive.ubuntu.com/ubuntu yakkety-updates/universe i386 Packages [35.0 kB]
Get:9 http://th.archive.ubuntu.com/ubuntu yakkety-updates/universe amd64 Packages [35.0 kB]
Get:10 https://packages.microsoft.com/ubuntu/16.04/mssql-server xenial/main amd64 Packages [940 B]
Fetched 356 kB in 4s (73.0 kB/s)
Reading package lists... Done
root@office:~# sudo apt-get install -y mssql-server
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following packages were automatically installed and are no longer required:
  authbind libcommons-collections3-java libcommons-dbcp-java libcommons-pool-java libecj-java libtomcat7-java tomcat7-common
Use 'sudo apt autoremove' to remove them.
The following additional packages will be installed:
  gdb gdbserver libbabeltrace-ctf1 libbabeltrace1 libc++1 libc6-dbg libcc1-0 libcurl3 libdw1 libjemalloc1 libunwind8
Suggested packages:
  gdb-doc clang
The following NEW packages will be installed:
  gdb gdbserver libbabeltrace-ctf1 libbabeltrace1 libc++1 libc6-dbg libcc1-0 libcurl3 libdw1 libjemalloc1 libunwind8 mssql-server
0 upgraded, 12 newly installed, 0 to remove and 0 not upgraded.
Need to get 152 MB of archives.
After this operation, 728 MB of additional disk space will be used.
Get:1 http://th.archive.ubuntu.com/ubuntu yakkety/main amd64 libbabeltrace1 amd64 1.4.0-3 [34.9 kB]
Get:2 http://th.archive.ubuntu.com/ubuntu yakkety/main amd64 libdw1 amd64 0.166-2ubuntu1 [191 kB]
Get:3 http://th.archive.ubuntu.com/ubuntu yakkety/main amd64 libbabeltrace-ctf1 amd64 1.4.0-3 [99.0 kB]
Get:4 http://th.archive.ubuntu.com/ubuntu yakkety/main amd64 gdb amd64 <IP_ADDRESS>61005-0ubuntu1 [2,652 kB]
Get:5 https://packages.microsoft.com/ubuntu/16.04/mssql-server xenial/main amd64 mssql-server amd64 <IP_ADDRESS>-6 [144 MB]
Get:6 http://th.archive.ubuntu.com/ubuntu yakkety/main amd64 gdbserver amd64 <IP_ADDRESS>61005-0ubuntu1 [217 kB]
Get:7 http://th.archive.ubuntu.com/ubuntu yakkety/main amd64 libcc1-0 amd64 6.2.0-5ubuntu12 [31.1 kB]
Get:8 http://th.archive.ubuntu.com/ubuntu yakkety-updates/main amd64 libcurl3 amd64 7.50.1-1ubuntu1.1 [190 kB]
Get:9 http://th.archive.ubuntu.com/ubuntu yakkety/main amd64 libunwind8 amd64 1.1-4.1ubuntu2 [46.3 kB]
Get:10 http://th.archive.ubuntu.com/ubuntu yakkety/universe amd64 libc++1 amd64 3.7.0-1 [226 kB]
Get:11 http://th.archive.ubuntu.com/ubuntu yakkety/main amd64 libc6-dbg amd64 2.24-3ubuntu1 [4,547 kB]
Get:12 http://th.archive.ubuntu.com/ubuntu yakkety/universe amd64 libjemalloc1 amd64 3.6.0-9ubuntu1 [78.9 kB]
Fetched 152 MB in 36s (4,192 kB/s)
Preconfiguring packages ...
Selecting previously unselected package libbabeltrace1:amd64.
(Reading database ... 174683 files and directories currently installed.)
Preparing to unpack .../00-libbabeltrace1_1.4.0-3_amd64.deb ...
Unpacking libbabeltrace1:amd64 (1.4.0-3) ...
Selecting previously unselected package libdw1:amd64.
Preparing to unpack .../01-libdw1_0.166-2ubuntu1_amd64.deb ...
Unpacking libdw1:amd64 (0.166-2ubuntu1) ...
Selecting previously unselected package libbabeltrace-ctf1:amd64.
Preparing to unpack .../02-libbabeltrace-ctf1_1.4.0-3_amd64.deb ...
Unpacking libbabeltrace-ctf1:amd64 (1.4.0-3) ...
Selecting previously unselected package gdb.
Preparing to unpack .../03-gdb_<IP_ADDRESS>61005-0ubuntu1_amd64.deb ...
Unpacking gdb (<IP_ADDRESS>61005-0ubuntu1) ...
Selecting previously unselected package gdbserver.
Preparing to unpack .../04-gdbserver_<IP_ADDRESS>61005-0ubuntu1_amd64.deb ...
Unpacking gdbserver (<IP_ADDRESS>61005-0ubuntu1) ...
Selecting previously unselected package libcc1-0:amd64.
Preparing to unpack .../05-libcc1-0_6.2.0-5ubuntu12_amd64.deb ...
Unpacking libcc1-0:amd64 (6.2.0-5ubuntu12) ...
Selecting previously unselected package libcurl3:amd64.
Preparing to unpack .../06-libcurl3_7.50.1-1ubuntu1.1_amd64.deb ...
Unpacking libcurl3:amd64 (7.50.1-1ubuntu1.1) ...
Selecting previously unselected package libunwind8.
Preparing to unpack .../07-libunwind8_1.1-4.1ubuntu2_amd64.deb ...
Unpacking libunwind8 (1.1-4.1ubuntu2) ...
Selecting previously unselected package libc++1:amd64.
Preparing to unpack .../08-libc++1_3.7.0-1_amd64.deb ...
Unpacking libc++1:amd64 (3.7.0-1) ...
Selecting previously unselected package libc6-dbg:amd64.
Preparing to unpack .../09-libc6-dbg_2.24-3ubuntu1_amd64.deb ...
Unpacking libc6-dbg:amd64 (2.24-3ubuntu1) ...
Selecting previously unselected package libjemalloc1.
Preparing to unpack .../10-libjemalloc1_3.6.0-9ubuntu1_amd64.deb ...
Unpacking libjemalloc1 (3.6.0-9ubuntu1) ...
Selecting previously unselected package mssql-server.
Preparing to unpack .../11-mssql-server_<IP_ADDRESS>-6_amd64.deb ...
Unpacking mssql-server (<IP_ADDRESS>-6) ...
Setting up libcc1-0:amd64 (6.2.0-5ubuntu12) ...
Setting up libc6-dbg:amd64 (2.24-3ubuntu1) ...
Setting up libdw1:amd64 (0.166-2ubuntu1) ...
Setting up libjemalloc1 (3.6.0-9ubuntu1) ...
Setting up libcurl3:amd64 (7.50.1-1ubuntu1.1) ...
Setting up libunwind8 (1.1-4.1ubuntu2) ...
Processing triggers for libc-bin (2.24-3ubuntu1) ...
Processing triggers for man-db (2.7.5-1) ...
Setting up gdbserver (<IP_ADDRESS>61005-0ubuntu1) ...
Setting up libbabeltrace1:amd64 (1.4.0-3) ...
Setting up libc++1:amd64 (3.7.0-1) ...
Setting up libbabeltrace-ctf1:amd64 (1.4.0-3) ...
Setting up gdb (<IP_ADDRESS>61005-0ubuntu1) ...
Setting up mssql-server (<IP_ADDRESS>-6) ...

+-------------------------------------------------------------------+
| Please run /opt/mssql/bin/sqlservr-setup to complete the setup of |
|                  Microsoft(R) SQL Server(R).                      |
+-------------------------------------------------------------------+

Processing triggers for libc-bin (2.24-3ubuntu1) ...
root@office:~# /opt/mssql/bin/sqlservr-setup
Microsoft(R) SQL Server(R) Setup

You can abort setup at anytime by pressing Ctrl-C. Start this program
with the --help option for information about running it in unattended
mode.

The license terms for this product can be downloaded from
http://go.microsoft.com/fwlink/?LinkId=746388 and found
in /usr/share/doc/mssql-server/LICENSE.TXT.

Do you accept the license terms? If so, please type "YES": yes

Please enter a password for the system administrator (SA) account:
The specified password does not meet SQL Server password policy requirements
because it is not complex enough. The password must be at least 8 characters
long and contain characters from three of the following five sets: uppercase
letters, lowercase letters, numbers, non-alphanumeric characters, or unicode
characters that are categorized as alphabetic characters, but are not
uppercase or lowercase.

Please enter a password for the system administrator (SA) account:
Please confirm the password for the system administrator (SA) account:

Setting system administrator (SA) account password...

Do you wish to start the SQL Server service now? [y/n]: y
Do you wish to enable SQL Server to start on boot? [y/n]: y
Created symlink /etc/systemd/system/multi-user.target.wants/mssql-server.service → /lib/systemd/system/mssql-server.service.
Created symlink /etc/systemd/system/multi-user.target.wants/mssql-server-telemetry.service → /lib/systemd/system/mssql-server-telemetry.service.

Setup completed successfully.

I read news this morning as well. I think Microsoft is going in the right direction.

More interesting news from Microsoft:
https://blogs.msdn.microsoft.com/jdbcteam/2016/11/17/open-source-jdbc-maven/

now we can used with in maven ?

I believe so. Merged.


Limitation of results by relevance

Hi,
is it posible to limit of results by relevance?
Something like:
SELECT *, MATCH(name,title,text) AGAINST('some text' IN BOOLEAN MODE) AS relevance
FROM `search_articles`
WHERE MATCH(name,title,text) AGAINST('some text' IN BOOLEAN MODE)
HAVING relevance > 10
ORDER BY relevance DESC

Thanks

This would be a great addition — in need of this myself...

@abufalbo
Please send a PR

The PR #91 may partially help you ... since the Models in the result collection will have now the relevance information as an attribute so that you we be able to filter on this attribute (but in PHP part)


Enhancement: Add crop image feature

Hello
Well done for your great job
Would be awesome if you could add a way to crop images directly from Obsidian. Really wanted feature!
Thanks a lot!

+1 for this


[Python] zoneinfo timezones failing during type inference

The conversion itself works fine (eg when specifying type=pa.timestamp("us", tz="America/New_York") in the below example), but inferring the type and timezone from the first value fails if it has a zoneinfo timezone:

In [53]: tz = zoneinfo.ZoneInfo(key='America/New_York')

In [54]: dt = datetime.datetime(2013, 11, 3, 10, 3, 14, tzinfo = tz)

In [55]: pa.array([dt])
....
ArrowInvalid: Object returned by tzinfo.utcoffset(None) is not an instance of datetime.timedelta

cc @AlenkaF
Reporter: Joris Van den Bossche / @jorisvandenbossche
Watchers: Rok Mihevc / @rok
Note: This issue was originally created as ARROW-16140. Please see the migration documentation for further details.

It seems this works correctly on latest master:
import pyarrow as pa
import datetime
import zoneinfo

tz = zoneinfo.ZoneInfo(key='America/New_York')
dt = datetime.datetime(2013, 11, 3, 10, 3, 14, tzinfo = tz)
pa.array([dt])
# <pyarrow.lib.TimestampArray object at 0x12ca01280>
# [
#   2013-11-03 15:03:14.000000
# ]
pa.array([dt], type=pa.timestamp("us", tz="America/New_York"))
# <pyarrow.lib.TimestampArray object at 0x12ca01400>
# [
#   2013-11-03 15:03:14.000000
# ]
pa.__version__
# '11.0.0.dev477+gc84d2dabc.d20230119'

We should verify if we have an explicit test for this, otherwise we can do a PR to add a test before closing this.
Looking for "zoneinfo", I only see it in the hypothesis strategies and in test_types.py (but that is testing more specifically the conversion to/from string for the timezone object itself). I find it a bit hard to see what the hypothesis tests exactly cover, so adding an explicit test might be a good idea.
In test_convert_builtin.py, we have a test like test_sequence_timestamp_with_timezone_inference that uses pytz, we can add something similar with zoneinfo.


imq can't read files???

okay i know I'm probably holding this wrong (I hope) and I sorry it that is the case !
[chris@chris-N150CU iqm]$ ./iqm robot.iqm robot.fbx
failed reading: robot.fbx

not sure if its talking about file access or if its loaded the file and doesn't like something inside it???
robot.zip

The README states:

FBX (7.x/ascii only)

robot.fbx probably uses a more recent version or is a binary file.

doh - sorry...


Cant start network

Hey,
Im working on a project in which i have to emulate the firmware of Foscam C2.
this firmware was a hell to extract but i got it.
the problem im running into is the following:

is there a way to solve this ?
as sh and init are rwx but it still hangs on them.

When Linux kernel booting, it executes the init programs (check main.c in kernel), and it seems like the programs are failing to initialize and returned an error (/sbin/init returns -8, /bin/sh returns -13).
It needs to debug the reason that init program fails. However, it is possible that exist another of initializing programs such as preinit. FirmAE finds initializing programs and logs them in scratch/${IID}/init log file.
If you can share the firmware and the instruction that extract the filesystem, then I can debug it together.

Hey thnx for your reply.
i still need to look into this myself will do it later this week.
in the meantime i will get you the firmware so you can look if you want do :)
its the foscam C2
https://www.foscam.eu/attachments/Foscam_C2
they have a lot of different versions with system and app firmware.
they are encrypted with ssl the password is WWzift*v4 i found that trough brute forcing variants of the old credentials.
after that you will get a broken ubi image that you need to extract manually.
modprobe nandsim first_id_byte=0x2c second_id_byte=0xac third_id_byte=0x90 fourth_id_byte=0x15
cat /proc/mtd
flash_erase /dev/mtd0 0 0
ubiformat /dev/mtd0 -f image.ubi -O 2048    "insert the app.ubi image here"
modprobe ubi
ubiattach -p /dev/mtd0 -O 2048
mount -t ubifs /dev/ubi0_X /mnt/ubifs  "use the right partition on the X it should be 1"
after this you get a ubi image in your /mnt that you can extract with binwalk.
in 1 of the folders there will be the rootfs filesystem.
i would have send you more clear steps but i lost my files due to a rollback so i got to re-do most of my steps.
i will post back later when i fixed my stuff when i have some spare time.

after giving 777 to everything and comenting 2 lines as someone else suggested i end up with this


Include listarray in rerun_py extension type bypass

What
Now that we always pass ListArray for send_columns calls from Python, we ran into an issue where the extension type was carried on when it shouldn't, causing comparision errors:
https://github.com/rerun-io/rerun/actions/runs/10366656880/job/28696922469
Note that this didn't break anything beyond this test.
Checklist

[ ] pass main ci
[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[x] I have tested the web demo (if applicable):

Using examples from latest main build: rerun.io/viewer
Using full set of examples from nightly build: rerun.io/viewer

[x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG
[x] If applicable, add a new check to the release checklist!
[x] If have noted any breaking changes to the log API in CHANGELOG.md and the migration guide

PR Build Summary
Recent benchmark results
Wasm size tracking

To run all checks from main, comment on the PR with @rerun-bot full-check.

@rerun-bot full-check

@teh-cmc urgh. I think the reason is that I'm swallowing the field name now always. But line strips seem to use list arrays with a field name that isn't item?

@rerun-bot full-check


Add delete_after_n method to When

In situations where a component is expected to automatically retry an endpoint, manually calling delete on a Mock may not be an option.
To support this scenario, add a new delete_after_n method to When that will remove the Mock from the server after it has matched the specified number of calls.
Related to https://github.com/alexliesenfeld/httpmock/issues/76
Related to https://github.com/alexliesenfeld/httpmock/issues/96

Thank you for your PR! Looks like a useful addition. I will have a closer look at it or get back to you soon!


codecov.io report shows 0.00% coverage for bin.rs

Problem
According to codecov.io there is 0.00% coverage for src/bin/bin.rs.
Yet there are many tests within src/bin/bin.rs.
Solution
Fix the job_grcov in .github/workflows/rust.yml.

Fixed https://app.codecov.io/gh/jtmoon79/super-speedy-syslog-searcher/tree/main/src?displayType=list


Serving cached response if results for a search response is already available.

@smadha are we separating tasks here? Python and frontend? ;)

Sure! this and #76 can be put together

https://github.com/MBoustani/GeoParser/commit/5fbfa3718aacfdfc5c4a2a73624aa17b2585613d
and
https://github.com/MBoustani/GeoParser/commit/8cf3db1e68d4b7112c68ba37cd57b204074404d6

@MBoustani : I will do the front end
We will need few more API's in views.py-

get list of saved searches.
get khooshe tile path for given saved search. (same as response for search_crawled_index)

@smadha
1- You can get the list of searches (same as list of tiles folder return from "list_of_searched_tiles" function)
2- Khooshe tiles path for given search would be = geoparser_app/static/search/tiles/ + <search_keyword>

@MBoustani Just the tile path wont do. It requires full response like from "search_crawled_index"

something like this - [{'rows_processed': 7207971, 'points_count': 20084026, 'total_docs': 7208416, 'khooshe_tile': 'static/tiles/search/keyword', 'popup_fields': 'url,title'}]


Add Game Creator

SteamDB app page links to a few games using this
https://steamdb.info/app/1347450/
https://steamdb.info/app/1842900/
Brief explanation of the change
Find Game Creator by it's "gcdata" file type.

missing non matching tests

Done.

After running changed rules in this pull request (this bot only tests rules.ini changes):
Changed technologies: Engine.Game_Creator
These apps now match: (9) 1138080 1347450 1512240 1570130 1768310 1842900 2706750 2727600 2893740
Please confirm that all these changed apps are correct.


README: Sentence case title

Pull Request Test Coverage Report for Build 390

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.66%

Totals

Change from base Build 389:
0.0%

Covered Lines:
2094

Relevant Lines:
2189

💛  - Coveralls


[All] Adjust base font size

Description

Change fontSize_base from 8 to 16
Fix a couple component style issues that resulted
Update borderRadius_1 theme value and remove unused borderRadius_2-5 variables (can be pulled into a separate PR)
Remove unused theme variables (can be pulled into a separate PR)

Motivation and context

The fontSize_base of 8 was chosen for two reasons:

The design system is built on an 8px grid
It reduced the significant digits in our theme values and so reduced the risk of browser rounding differences

However, setting the root-level font-size to 8px for all components also has some issues. Most notably, this is what it looks like if you place a paragraph of text inside a Card (with no CardBlock):

Using a fontSize_base of 16 instead would be much more expected by the consumer, as it is the default browser font-size. Additionally, each of the two reasons above can be addressed:

16 is simply double the 8px grid value; the mental math isn’t too much harder
browser rounding differences can be mitigated by applying a strict (i.e. set heights & line-heights) layout strategy, which is likely necessary anyway (I confirmed this is the case for Button; no visual changes in either Edge or IE11).

How to test

cd mineral-ui/packages/site && npm start
Confirm all components look as they should

Types of changes

Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] Renders and functions properly in all supported browsers
[x] Automated tests written and passing
[x] Accessibility and inclusivity considered [n/a]
[x] Rendering performance (initial load time & 60fps) and perceived performance considered [n/a]
[x] Documentation created or updated [n/a]

How does this PR make you feel?

The Site will be messed up by this change. I didn't want to update that until after discussion though, because I'd like to get it using mineralTheme at the same time.

I like the goal of this PR, to make the default font in arbitrary Card content more in line with what's expected. I'm unclear of the tradeoffs, but I trust your judgement @kylegach.

Some of the font sizes in the site are smaller than expected.

Yep: https://github.com/mineral-ui/mineral-ui/pull/228#issuecomment-320764144

Sorry, I missed your comment...

Close in favor of https://github.com/mineral-ui/mineral-ui/pull/248 ???

⬆


InfluxDB 2: Cannot find owner of TelegrafConfig

# Create TelegrafConfig
--> POST http://<IP_ADDRESS>:9999/api/v2/telegrafs
--> {
	"name": "tc1605171465258-IT",
	"description": "test-config",
	"config": "[agent]\n  interval = \"10s\"\n  round_interval = true\n  metric_batch_size = 1000\n  metric_buffer_limit = 10000\n  collection_jitter = \"0s\"\n  flush_jitter = \"0s\"\n  precision = \"\"\n  omit_hostname = false\n\n#my instance\n[[outputs.influxdb_v2]]\n  bucket = \"my-bucket\"\n  urls = [\"http://<IP_ADDRESS>:9999\"]\n  organization = \"my-org\"\n  token = \"$INFLUX_TOKEN\"\n[[inputs.cpu]]\n  totalcpu = true\n  collect_cpu_time = false\n  report_active = false\n  percpu = true\n",
	"orgID": "cf6cf88984580114"
}
<-- 201 Created http://<IP_ADDRESS>:9999/api/v2/telegrafs

### Find owners
--> GET http://<IP_ADDRESS>:9999/api/v2/telegrafs/069aea8c16a7c000/owners
<-- 200 OK http://<IP_ADDRESS>:9999/api/v2/telegrafs/069aea8c16a7c000/owners
<-- {"links":{"self":"/api/v2/telegrafs/069aea8c16a7c000/owners"},"users":[]}

Steps to reproduce:
List the minimal actions needed to reproduce the behavior.

Create TelegrafConfig
List owners of TelegrafConfig

Expected behavior:
Return list with owner - user who created the TelegrafConfig
Actual behavior:
Return empty list.
Environment info:

InfluxDB version: InfluxDB v2 nightly from 2020-11-12

I haven't investigated, but I'm curious if this has the same / similar root cause as #20006

I haven't investigated, but I'm curious if this has the same / similar root cause as #20006


An issue of including libraries

I'm writing a c plus plus program with ncurses.h
the functions in this library are very similar to curses.h, so clangd include <curses.h> automaticlly
but I just want to use ncurses.h not curses.h

clangd's issue, needs clangd return correct completion item.

clangd's issue, needs clangd return correct completion item.

excuse me but what is clangd return

The return from clangd contains <curses.h>, coc just apply and insert it to document.

The return from clangd contains <curses.h>, coc just apply and insert it to document.

then how to correct it

Needs clangd to fix this.

Needs clangd to fix this.

so you mean, I can't use coc-clangd if I want to fix this issue?

coc/coc-clangd is client, the clangd server returns the completions, client just accepts and adds to your document. If the completion item is not correct, needs the clangd server to fix and returns correct ones.

coc/coc-clangd is client, the clangd server returns the completions, client just accepts and adds to your document. If the completion item is not correct, needs the clangd server to fix and returns correct ones.

I installed clangd and I put the right path of clangd in CocConfig.json, coc-clangd will start clangd server automaticlly too (When I am editing a cpp file,  'pidof clangd' can return a correct value.) But the issue is not solved.It still will include curses.h automaticlly.

clangd is working but not correctly, this must be fixed in clangd itself.


Create blog post summary of Humans of OTel video

Blog post summary of Humans of OTel video - link to video once it's posted on the OTel YouTube channel, and include transcript. Related to #14

PR is approved. Waiting for it to be merged.

Published: https://opentelemetry.io/blog/2024/humans-of-otel-eu-2024/


ARROW-15347: [Doc][Guide] Testing

This PR adds and restructures the content of the testing section of the New Contributor's Guide.

Yes, the R part is only styling (indentation, line wrapping) changes.


Adds data commitment request

Closes https://github.com/celestiaorg/celestia-app/issues/268

Codecov Report

Merging #478 (3302693) into qgb-integration (0e740d1) will decrease coverage by 1.05%.
The diff coverage is 7.63%.

@@                 Coverage Diff                 @@
##           qgb-integration     #478      +/-   ##
===================================================
- Coverage            11.39%   10.33%   -1.06%
===================================================
  Files                   60       63       +3
  Lines                11569    12870    +1301
===================================================
+ Hits                  1318     1330      +12
- Misses               10164    11453    +1289
  Partials                87       87

Impacted Files
Coverage Δ

x/qgb/keeper/keeper_data_commitment.go
0.00% <0.00%> (-57.70%)
:arrow_down:

x/qgb/keeper/keeper_valset.go
32.11% <0.00%> (-0.48%)
:arrow_down:

x/qgb/keeper/query_data_commitment.go
0.00% <0.00%> (-82.98%)
:arrow_down:

x/qgb/keeper/query_valset.go
0.00% <0.00%> (ø)

x/qgb/orchestrator/orchestrator_client.go
0.00% <0.00%> (ø)

x/qgb/orchestrator/querier.go
0.00% <0.00%> (ø)

x/qgb/orchestrator/relayer.go
0.00% <0.00%> (ø)

x/qgb/orchestrator/relayer_client.go
0.00% <0.00%> (ø)

x/qgb/types/data_commitment.go
0.00% <0.00%> (ø)

x/qgb/types/data_commitment_confirm.go
30.43% <ø> (ø)

... and 14 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 0e740d1...3302693. Read the comment docs.


About the Twitter DataSet

It seems that the directory Market doesn't contain the sentiment extracted from the Twitter DataSet, and I don't know how to get it with other softwares you mentioned. Could you please help me with it? Thanks!

I think I have located missing dataset.
Uploaded, hope it will help you.


Improved Tutorial Sequence for Adjusting WAM-V Parameters

Eventually we want to support the ability to (easily) add custom USV models to VRX. This will be especially important for RoboBoat support, but has already been requested by users in several issues. See

#595
#487

As a starting point, we should create a tutorial sequence explaining how to customize the hydrodynamic and propulsion characteristics of the WAMV. We currently have a minimal overview of this topic here. Let's expand it to show examples of changing these parameters, similar to our environmental parameters sequence.
Once this is resolved, the next step will be a tutorial sequence showing how to add a new model like the ones being created for RoboBoat (#704) and attach the necessary plugins to get it working in the simulation.

This is mostly covered now in the dedicated "Maritime" section in the Gazebo tutorials.
https://gazebosim.org/api/sim/8/tutorials.html


Make pad more robust to non-hashable values.

The pad_width can be an iterable of iterbles, thus, we need to cast both the inner and outer list to tuples.
Also, the values should be intergers, but some use-cases (mostly Tensorflow) pass e.g. DeviceArray(1) as values in there.

You have some test failures as well.


eBPF For Windows Demo - Connection Tracking fails

reference page: https://github.com/microsoft/ebpf-for-windows-demo/blob/main/connection_tracker/README.md
Running: cd out && powershell -NonInteractive -ExecutionPolicy Unrestricted %EBPF_ROOT%\bin\Convert-BpfToNative.ps1 -ProgramName conn_track.o
Fails with: Project "C:\Users\User\repos\ebpf-for-windows-demo\connection_tracker\bpf\out\conn_track.vcxproj" on
node 1 (default targets).
C:\Users\User\repos\ebpf-for-windows-demo\connection_tracker\bpf\out\conn_track.vcxproj(1,2134): erro
r MSB4019: The imported project "C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\MSBui
ld\Microsoft\VC\v170\Microsoft.Cpp.Default.props" was not found. Confirm that the expression in the I
mport declaration "C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\MSBuild\Microsoft\V
C\v170\Microsoft.Cpp.Default.props" is correct, and that the file exists on disk.
Done Building Project "C:\Users\User\repos\ebpf-for-windows-demo\connection_tracker\bpf\out\conn_trac
k.vcxproj" (default targets) -- FAILED.
The path: C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\MSBuild\Microsoft\V
C\v170\Microsoft.Cpp.Default.props
does not exist.
Also, I searched for Microsoft.Cpp.Default.props
which also does not exist on the disk.
How to proceed?

@dahavey
I could not repro the failure following all the steps from scratch, although, the README.md has an error on setting up the EBPF_ROOT variable (step #2 of Native eBPF program generation). From the current repo's folder structure, it should be:
set EBPF_ROOT=<your workspace path>\ebpf-for-windows-demo\packages\eBPF-for-Windows.0.5.0\build\native
instead of:
set EBPF_ROOT=c:\ebpf-for-windows-demo\packages\eBPF-for-Windows.0.1.4\build\native

Copied ebpf-printk.guid into current directory and got:
C:\Users\User\repos\ebpf-for-windows-demo\x64\Release>.\conn_tracker.exe
Press Ctrl-C to shutdown
bpf_object__load for conn_track.sys failed: 131

Tested after the fix of #18, conn_tracker.exe works.
PS C:\ebpf-for-windows-demo\Debug> .\conn_tracker.exe
Press Ctrl-C to shutdown

@dahavey : Can you please verify and close?


rn-carousel-index causing an infinite digest loop - Angular 1.3.8

Hello,
I am getting an "error:infdig Infinite $digest Loop" whenever I specify a variable for the rn-carousel-index.
I am also using Angular 1.3.8.
My html is as follows:

Is this a bug ? Thanks.

my bad on this. Ignore it. It was a problem with the way I was initializing my arrayAds array. The ng-repeat for the angular-carousel was breaking when I initialize my variable like this: $scope.arrayAds = [ ] ; was causing those infinite digest loop.


Shuttle generate a huge vimswp file

I've used Shuttle for years, recently I found the ~/.vimswp/shuttle.json.swp file takes up a lot of my disk space.

/reopen


Unboxing an array of strings

How do I unbox an array of strings [String] "statuses" in the below JSON example? Looks like this is not yet supported?
{
	"userId": 3827,
	"name": "Joe Gartner",
	"thumbnailURL": null,
        "statuses": [
                "offline",
		"invisible"
        ]
}

Hey @sasikiran! It's indeed supported, just call unboxer.unbox("statuses") just like you would for any other piece of data. Here is some example code that works for the JSON you provided:
struct User: Unboxable {
    let id: Int
    let name: String
    let thumbnailURL: NSURL?
    let statuses: [String]

    init(unboxer: Unboxer) {
        self.id = unboxer.unbox("userId")
        self.name = unboxer.unbox("name")
        self.thumbnailURL = unboxer.unbox("thumbnailURL")
        self.statuses = unboxer.unbox("statuses")
    }
}


Reddit community Wiki page is disabled

Please remove or replace this link in README.md, because it does not work anymore.
https://www.reddit.com/r/LocalLLaMA/wiki/index/

Thanks!


sync api implementations for volume

What this PR does / why we need it:
It implements the sync api for volumes for a storge device.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

@PravinRanjan10 @wisererik please get this to closure


Feature/const char in the API

This allows us to work better withC++ (std::string) strings. This way
we could pass directly (after calling .c_str()) C++ std:strings to
InfluxDb C API functions. Currently I'm doing something like:
char *c_str = new char[cpp_std_string.length() + 1];
strcpy(c_str, cpp_std_string.c_str());

/* After that */

influxdb_api_function(c_str, ...);

With this change I'll be able to do:
influxdb_api_function(cpp_std_string.c_str(), ...);

And after I can call infludb_function(cpp_std_string.c_str(), ...);

Haaaallo anyone there?


Flatten content in setting bottom sheet

What

The Contribute info layout is a bit awkward.
[ ] We should flatten the content in the bottom sheet, and use an accordeon system, like in the first level

Part of

[ ] #960

Mockup

May I work on this issue?

Sure @GowthamGoush

Is anyone still working on this issue?

Yes I'm working on it

Great!

Hi @GowthamGoush ,
Do you need any help with it?

I think this has been solved. Feel free to reopen if relevant.