Tsx files?

Hi @toddlucas - I'm new to Typescript (and vscode too). Taking a look at your project, I'm not sure why vscode isn't playing nicely with the tsx files, any ideas? See screenshot below, appreciate your help!

Hi @ArdentKid, Thanks for trying the starter! First, I wanted to double check that you have run npm install and typings install as outlined in the README.
There are two problems that you're seeing here, and I've tried different solutions to them. Unfortunately, I don't have a good general purpose answer, yet. The two problems are:

The TypeScript compiler is having trouble locating the typings files
The compiler is not getting the --jsx flag when building IntelliSense

Initially, it looked like tsconfig.json might be the answer. You can see that there are two entries in the file that attempt to feed the compiler some parameters. It appears to be working for builds, but I'm not sure the IntelliSense protocol is using it (VS Code spins up a background TypeScript process to supply IntelliSense using an open protocol that MS has developed).
It is possible to work around the import warning by adding a reference comment to the top of each file, as you can see in server.tsx. It looks like this:
/// <reference path="../typings/index.d.ts"/>

Although this is a pain, it should only require one line, since the typings utility combines references for you in index.d.ts. You'll need to fix up the relative path, of course, depending on the depth of the .tsx file.
As for the --jsx issue, I don't have a solution, yet. There are others who have filed issues, so maybe we'll get some resolution at some point (hopefully soon).

I learned, for all the react types, don't install "npm install --save-dev @types/react-"
instead, use "typings install react- -save"


Connection type color switching is applied on Result grid before query is run

Description
If you run a query "in" a given connection on which is used a given connection type (and hence its color), the result grid is also colored in the convenient way - so far, so good.
But let's say you want to switch to another connection on which you want to run the same query, then when switching connection, the sql editor changes color (fine), but the result grid too - even if the query has not been yet (re)run using the newly selected connection. It then looks like as if the already-displayed data had been generated on the current connection, wheras they still are from previously selected connection.
This can be misleading, and it would be fine to fix this.
DBeaver Version
Community <IP_ADDRESS>306251800
Operating System
Win11
Database and driver
Snowflake JDBC 3.13.29
Steps to reproduce
No response
Additional context
No response

Do you use dark theme?

Do you use dark theme?

Yes I do.

Doesn't it happen in the light theme?
It looks like a duplicate of #8081

It happens the same way in Theme-enabled--Light Theme, as well as with Theming disabled.

Thank you for bug report.

Does it still reproducible in the latest DBeaver version?

Hello,
Yes it still does : once a query has been executed and its result is displayed in the grid, when you switch to another connection bearing another color coding, the color is applied on the previous connection's results' grid at the moment you click on the editor (on getFocus, I guess)
current version is Version <IP_ADDRESS>311051706
OS version is Windows 11 23H2 (Version 10.0.22631 Build 22631)
Regards,
Thomas LEILLET

De : Elizabeth @.>
Envoyé : lundi 13 novembre 2023 19:04
À : dbeaver/dbeaver @.>
Cc : Leillet, Thomas /FR/EXT @.>; Author @.>
Objet : Re: [dbeaver/dbeaver] Connection type color switching is applied on Result grid before query is run (Issue #20380)
Does it still reproducible in the latest DBeaver version?
—
Reply to this email directly, view it on GitHubhttps://github.com/dbeaver/dbeaver/issues/20380#issuecomment-1808723756, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A7ECPLJTUF2WXMYZLAXA5VTYEJOK3AVCNFSM6AAAAAAZT4UAZGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBYG4ZDGNZVGY.
You are receiving this because you authored the thread.Message ID: @.***>

Thank you for update


Need guidance for PPO training

Hey, I have been working on the PPO stage for a while but haven't managed to train a Llama model with the trl library. I used mainly the code from stack_llama/scripts/rl_training.py and the dataset from Anthropic/hh-rlhf
In my case, the reward mean is just fluctuating around (-1, 0) and not going up. Objective KL is around 10 and does not change throughout training.
Has anyone successfully trained a model with the stack_llama scripts? If so, would you be so kind as to share the training logs and hyperparameters? Also, how many steps did you wait until you see some increase in the mean rewards? I'm also concerned if I'm simply not training long enough (i.e. It hasn't found a good action through exploration).

There are some training details for StackLlama in the blog, does that help?


Align data and instruction address output

This PR adds two commits to ensure the data and instruction address output by the core is always word aligned.
The core handles unaligned accesses by issuing two separate, aligned accesses. However, without these commits, the core still outputs unaligned addresses for the two accesses and relies on the memory to ignore the address LSBs.
Tested with Verilator and Vivado.

Thanks for your feedback @imphil . I implemented the requested changes, updated the documentation accordingly, and added the requested assertions.


13765 BC Registry UI: Include the support contact information

Issue #:
https://github.com/bcgov/entity/issues/13765
Description of changes:

added support contact information in the footer
UI design: https://preview.uxpin.com/e68d77b708da0ebbc185073ee5991ef2a90660be#/pages/159212743
updated README instruction

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the sbc-auth license (Apache 2.0).

This is the screenshot of the finished work in dev:

Please do not merge this PR yet. The design team is asking whether to use two "Hours of Operation" or to keep just one of them in case both of them are the same.

Just got a confirmation from the design team that only one "Hour of Operation" is required.

Just got a confirmation from the design team that only one "Hour of Operation" is required.

OK. This code change looks good to me.

Just got a confirmation from the design team that only one "Hour of Operation" is required.

Yes, but which hour should we be in operation? ;) I'll show myself out.


Contextual menus should have a context to better decide which menu options to show/hide

The context menu has way to many options. Go to definition, declaration, implementation. I never know which one to use

In theory the context menu should read the context to know which options to show instead of showing all options all the time. For example if I click on an  if statement, where should it go if I select the "go to definition"?    Shouldn't that option be shown only for variables and constants?
I asked the plugin developer who added those options to the contextual menu and he comment that he only adds them but has no way to get the context to know if they should be added or not.
Is it possible to add more context about where the user right clicked to better filter what options to show/hide in the menu?
Thank you.

For example if I click on an if statement, where should it go if I select the "go to definition"?

Unfortunately, the editor doesn't know what the meaning of "if" is, it only knows that there is one or more implementations for a "go to xyz" feature. To update said context keys extensions would need to eagerly push/update context keys as the cursor moves. To make matters more complicated: extensions run in a separate process and and the cursor might already be on a different word while extensions are still "thinking" about some other token.

Closing this as 'wont-fix' - I do understand and agree with the challenge but there is no good solution


pkgsStatic.libhwy: fix build

Description of changes
This was changed in 7328b78fb58, but I don't understand why — the only difference should be in static builds, which worked before that change and didn't work after.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

nix-repl> pkgsCross.aarch64-multiplatform.stdenv.hostPlatform.extensions.library
".so"

It was a more recent addition — you weren't checking an older Nixpkgs, were you?

Indeed I probably was, damn outdated channels

@OfBorg build pkgsStatic.libhwy

I've been getting a lot of grief from the new libhwy too.  If you pass it an -mtune= flag on aarch64 you get failed inlining errors.


The feedback and global icon is not in view for Galaxy Tab S5

Feature1448 misses the feedback and global languages.
Samsung Gal Tab S5E- Android-Google Chrome (Staging looks correct)
https://feature1448.fightpandemics.xyz/
https://staging.fightpandemics.work/

If we look at the Galaxy Tab S4 10.05- Android 8.1-Google Chrome
Both Feature 1448 and Staging looks like the display of mobile phone and looks correct (feedback and global icon in display)

Thanks @andreslarezbas there is some relation to this https://github.com/FightPandemics/FightPandemics/issues/1650
Samsung Gal Tab S5E must just meet the medium size threshold so it is showing the "desktop" view. That's not necessarily a problem but we should definitely ensure the header items are not off the screen.
I would say:

Decrease font size - 16px is big for navbar links. Maybe 14px (or at least when medium width)
As fail safe, the overflow should go to a 2nd level. This may happen with translations and we can test every combination of screen width and translation to make sure it fits perfectly. Better than going off the page.

We can ask for volunteer in #engineering-frontend channel.

This was my solution for the search bar https://fightpandemics.slack.com/files/U019926NH0W/F01CJTQ3X9D/2020-10-20_21-59-52.mp4


Show banner when in sandbox mode

Context
This is a quick note from the bunq team and mostly a feature request. Today someone contacted us with an urgent question about privacy. He was claiming to see someone else's account information through bunq desktop. After a few minutes we found out he was looking at sandbox data.
Solution
A very easy fix could be to always show a banner when you're in sandbox mode. The banner can say something like 'You're currently seeing test information, wanna go pro?'

@painjansen Please let me know if the change in #269 is sufficient enough.

looks amazing @L00Cyph3r 👍


Renaming to tabula

I would love to hear some opinions. The package was archived on CRAN.
Comment below, please. Thumbs up = yes, thumbs down = no.

I'd prefer no change.  If the API will remain the same, then I'd prefer for my older code to work without modification.  If the API will change, then changing the name makes sense to me.

https://cran.r-project.org/package=tabulapdf


Define an equal_up_to_global_phase protocol

A class implementing _equal_up_to_global_phase_(self, atol: float) should allow comparisons that ignore global phase to be made. The utility method cirq.equal_up_to_global_phase would first try for this method, then fallback to cirq.approx_eq.

This has been completed


flash : stm32l4x don'y write 0xff

for mcuboot on l4, avoiding writting 0xff solves trailer write access during swap

You might want to rebase this, so it only includes the last patch. You can force-push to your repository, this PR will be updated.

This commit is required is required only for MCU boot loader. That 's why I pushed it separetly.

now that the other flash changes are merged, can you rebase/update this.

Following some changes in mcu bootloader . this patch seems not needed . I continue testing without.
And if confirmed , i will close it.

any updates if still needed?

no

After the changes in mcuboot , the PR is not needed any more.


Set randomAccess=true on LOAD.

@uschindler Would you like to take a look?

I fixed the conflict. Sorry for force-pushing the main PR. I was not aware that you were working.


Preserve indentation when rendering partials

A fix for: https://github.com/janl/mustache.js/issues/562

not all spec tests are passing, closing...

I've added all the tests described in this spec: https://github.com/mustache/spec/blob/master/specs/partials.yml
And they are passing now

I decided to follow the original solution of @kevindew (PR: https://github.com/janl/mustache.js/pull/613) as it looks more elegant
I had to change some minor things to make it work according the the spec tests

This looks really promising! Adding the tests from the mustache spec project was a great addition 👍
I really feel @kevindew should get more credit for his contributions, as he wrote the first draft in #613 as you mention. At the moment, none of the commits has his name on it, which isn't quite fair in my eyes. Wondering if we should use his changes as the first commit, then add your commits as tweaks on top. That way you both get credit for your contributions, well deserved 😄

@phillipj
Sorry,
I didn't think about the commits/credit aspect, I just wanted the thing to work.
I will re-arrange the PR such that it will include Kevin's commits

Fully understandable!
Cool 👍

Done - please look at
https://github.com/janl/mustache.js/pull/705


Added FileShare commands.

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

Is the storage set command functional for you? When I try it, I keep getting the error Invalid request body for managed environment storage 'storage'. 'properties.azureFile.accountName' must be proivded when setting Azure File Share Storage. The length must not be more than 63 characters. I have a value set for --account-name and it's definitely less than 63 characters

Working for me, am not able to reproduce this issue


Unsupported: cannot change the io buffer size across restarts

Running any commands gives me this error: Unsupported: cannot change the io buffer size across restarts. please change it back to 8388608.
I've tried deleting ~/.config/the-way to no avail. This is version 0.7.0 on NixOS.
Running the same command with backtrace set to full gives:
Error:
   0: Unsupported: cannot change the io buffer size across restarts. please change it back to 8388608

  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ BACKTRACE ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
   1: color_eyre::config::HookBuilder::install::{{closure}}::h89493c7b19a69a32
      at <unknown source file>
   2: the_way::the_way::TheWay::start::hea72f08e14a0d57d
      at <unknown source file>
   3: the_way::main::ha4dc93d41b39356d
      at <unknown source file>
   4: std::rt::lang_start::{{closure}}::hf19916ed3db71252
      at <unknown source file>
   5: std::rt::lang_start_internal::h7d1b27a1580794aa
      at <unknown source file>
   6: main
      at <unknown source file>
   7: __libc_start_main
      at <unknown source file>
   8: _start
      at <unknown source file>

Thanks!

Hmm, this could be because of the version change - have you done the database migration as described in the release?  You'll have to download the older version and then run
the-way export snippets.json
the-way clear

then get the latest version and import with
the-way import snippets.json

Let me know how it goes!

I can't run

With the older version as well? The one that worked previously (0.6)?

I did do an export a while ago.
I had to delete both ~/.config/the-way and ~/.local/share/the-way for the error to go away.
Thanks!

Okay, that's odd but glad you found a way! Will keep an eye out for this in the sled repository. Thanks!


Istanbul not working with jasmine-node on Win7

We are using jasmine-node to test our server side code. Because we read in many blogs, that istanbul code coverage is awesome we wanted to give it a try. Therefore we installed istanbul. Jasmine-node runs fine and istanbul with karma (testacular) also runs fine. But combining jasmine-node and istanbul doesn't work for us on Windows 7 64bit. When we run the following command:
nodes_modules\.bin\istanbul.cmd cover node_modules\.bin\jasmine-node.cmd test/server/nmc/unit
We get:
//-------------------------------------------------------------------------------------------------------------------------------
(function (exports, require, module, __filename, __dirname) { @IF EXIST "%~dp0
^
No coverage information was collected, exit without writing coverage information
SyntaxError: Unexpected token ILLEGAL
at Module._compile (module.js:439:25)
at Module._extensions..js (module.js:474:10)
at Object.Module._extensions..js (D:\Studium\DiplomArbeit\git\nomocom\protot
yping\to-scaffold\node_modules\istanbul\lib\hook.js:101:13)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:312:12)
at Function.Module.runMain (module.js:497:10)
at runFn (D:\Studium\DiplomArbeit\git\nomocom\prototyping\to-scaffold\node_m
odules\istanbul\lib\command\common\run-with-cover.js:88:16)
at D:\Studium\DiplomArbeit\git\nomocom\prototyping\to-scaffold\node_modules
istanbul\lib\command\common\run-with-cover.js:191:17
at D:\Studium\DiplomArbeit\git\nomocom\prototyping\to-scaffold\node_modules
istanbul\lib\util\file-matcher.js:52:16
at D:\Studium\DiplomArbeit\git\nomocom\prototyping\to-scaffold\node_modules
istanbul\lib\util\file-matcher.js:35:9
//-------------------------------------------------------------------------------------------------------------------------------
Running the same command on Ubuntu works fine and we get  a nice code coverage. On Ubuntu we use:
nodes_modules/.bin/istanbul cover node_modules/.bin/jasmine-node test/server/nmc/unit
Are we screwing something up, or is there a lack of compatibility with Win7? Does someone know how to fix this issue?
EDIT:
Our node version is: 0.10.5 and npm is on 1.2.18, if you need more information, let me know
Thank you
Tschoartschi

@MurFyahKapi thanks! This took me all day.
For **** sake puts this in bold in the readme

@dotnetCarpenter - care to submit a PR?

I need to do some more testing, just to be sure that it will also run on linux. But yes


chore: build as cjs package

Ooh, very happy to see this!! I'd love this PR (plus a release) in order to fix https://github.com/igorshubovych/markdownlint-cli/issues/469 without needing to introduce an async code path. It looks like you've done the work already, but if you'd like a second pair of eyes or anything to get this done, I volunteer. Thank you! :)

After double checking the change I don't think it changes the behavior of the library enough to warrant a semver major.
It does allow for something new as a side effect (that I quite like, actually) and the possibility to import the library as TOML without using * as. I'll make this a semver minor bump instead of a patch since that's a new feature

v1.2.0 works great, thank you!


[usecase] Update all ManifestDemo tests descriptions

Impacted TCs num(approved): New 0, Update 5, Delete 0
Unit test Platform: Android
Android test result summary: Pass 4, Fail 1, Block 0

The fail bugId: xwalk-2737

Updated


Internal Server Error (500) for client.js on production

I deployed a build to production here using Dokku. Although everything seems to be working fine the console logs a 500 (Internal Server Error) for the client.js file.
Another note, I tried running the site on GTmetrix to test for performance and it throws an error:
Analysis Error
The URL did not appear to be an HTML page (missing or empty Content-Type)

Doesn't the server create an index.html file?

Never mind, just had to take build folder from .gitignore.


KIM Integration - GCP Trial plan

Description
Changes proposed in this pull request:

Shoot data for GCP Trial

Related issue(s)

Refers to #905


FN 45384 Goes in the code, does whatever he wants

https://www.youtube.com/watch?v=ipi7HQC-e00

additional point for the joke. Just kidding.


TVM with an LLVM Back-end giving error on resnet50

Hi,
I have been trying to go through the tutorial which helps compile the RESNET50 using an LLVM Back-end. The LLVM Back-end I am using is a cross compiler for RISC-V. So I am trying to generate the assembly for RISC-V from RESNET50. When I try to compile the code using the following command:
tvmc compile --target "llvm -march=rv64imv -mcpu=generic-rv64" --output resnet50-v2-7-tvm.tar resnet50-v2-7.onnx
It gives the following error:
LLVM ERROR: CPU 'generic' is not supported. Use generic-rv64
I have also tried to specify the target using:
--target=llvm --target-llvm-mcpu=generic-rv64
But, the error remains the same.
The LLVM I am using is only built for RISC-V and I have used it many times so I don't know, where the issue is,  but I don't think the issue would be with LLVM.

Hi,
I've got the same error as you. My LLVM is also built only for RISC-V.
Have you found already a solution for it?
BR

Similar issue https://gitlab.freedesktop.org/mesa/mesa/-/issues/7769 . You could eventually try LLVM 15, but the same tests are failing, so not really different except error code.

The error is coming from LLVM.

The right command to use for cross compilation of riscv64 is
tvmc compile --target="llvm -mtriple=riscv64-unknown-linux-gnu -mcpu=generic-rv64 -mabi=lp64d -mattr=+64bit,+m,+a,+f,+d,+c" --output cross_compiled.tar /home/bmw/aleesha/models/resnet18.onnx --cross-compiler riscv64-unknown-linux-gnu-g++
this is generation compiled  models successfully


Goal??

What is our goal to achieve. Plotting and analyzing increase of cases or getting recent data

Hi @Ritu1611 Good question, some time has passed.
"(...) When the project was kicked off, there was a lack of the APIs or data. That was quite useful I hope (saw the stats, thank you!)."
Times have changed a little now. Yup, plotting & analyzing data will be great feature here!

Can we use latest data from data.world or and other sources. You can suggest data source as well.

@mbiesiad

@Ritu1611 Thanks for your messages. I think, that the data.world is OK. Yeah, sure. 🚀
At this moment, stats for Poland are updated quite well. It could be nice to make an update for other files.

@Ritu1611 @mbiesiad I have solid programming experience in C#.NET and VB.NET but none in Python. Can I be of any help in this project? Would like to contribute.I am good at writing SQL queries in relational databases like SQL Server, Oracle. I have basic understanding of ML.


Issue with filtering by comparison

The Go package does not seem to properly execute comparison queries. For example, if I have data:
{
  "resources": [
    {
      "description": "old",
      "updated_at": "2023-04-05"
    },
    {
      "description": "new",
      "updated_at": "2024-03-06",
    }
  ]
}

and execute query
resources[?updated_at > '2024-02-01']
the filtered result should only include the "new" resource. When I run this same example in the playground on the JMESPath site, I see the behavior I expect. When I run this in my Go code, I receive no results back from the search. Clearly, something is wrong in the Go code.
Also see the related issue against the original repo.

> and similar operators only work on numbers

Will be fixed by https://github.com/jmespath-community/go-jmespath/pull/99


wpsoffice: fix crash

Motivation for this change
Fixes #125951
Use steam-run to create a fhs and launch the application to avoid possible issues, note that the closure is not minimized.
Cc @Th0rgal
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

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

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/541


no new (+) icon on database folder when hovering

items that support "new" (table, users) have a ➕ icon on the item when the mouse hovers over, but the database folder does not

verified this is in the RC build


Features/6 dnx compatibility

Backed out some changes to ReadMe as per feedback on #5.
Backed out solution structure changes as per feedback on #5.
Switched TFM to net40 to support dnx451 as per #6. This gives us much a much wider compatibility than is required, but I don't believe that will be an issue.

Note, I switched the version in the project.json to alpha9 in order to test this locally. Let me know if you want the version reverted.

Sorry, looks like I did conflict you when manually backing out those changes :disappointed:
Looks good (yes to alpha9 and removing lock file). I'll get the new version up on NuGet for testing, and check that the existing sample still works. Once that's up, do you mind adding a sample for dnx451?

I mean, once you've fixed the conflict (sorry).

Sure. So, for reference, I did the following to test this local (incase it helps you).
First, build the solution including the PR changes to produce output (or quickly replicate the change to project.json if that's easier).
In any solution that currently references SpecFlow.Dnx version 1.0.0-alpha8:

Add a Nuget.config add the root level of the solution.
Add the following configuration to the file:

<?xml version="1.0" encoding="utf-8"?>
<configuration>
  <packageSources>
    <add key="local" value="D:\\\\Projects\\Resources\SpecFlow.Dnx\\src\\artifacts\\bin\\SpecFlow.Dnx\\Debug\\" />
    <add key="api.nuget.org" value="https://api.nuget.org/v3/index.json" />
  </packageSources>
</configuration>

Note, replace the long path in there to point to wherever you store SpecFlow.Dnx but making sure to keep the escaped backslashes.

Simply reference 1.0.0-alpha9 in the project consuming SpecFlow.Dnx.

The above sounds longwinded but in practice takes less than a minute to achieve and is very handy for testing packages pre-publication and without running a full private NuGet repo.
Anyway, I'll see about these merge conflicts and takes yours for the most part. I can add a complete example to the samples directory if you want, but would it be easier to simply add the dnx451 targets to the existing projects? The TFM's would actually make one another redundant but it does prove that SpecFlow.Dnx works in both cases?

I've merged those changes, just sorting samples now.

Good idea on the Nuget.config. I was manually doing an Add > Existing Project... in VS.
Hmm, regarding the samples... my original thoughts process was it would be better to have explicit examples per framework, but can be convinced otherwise if you think it's easier.

I'll create a new PR for samples. The difficulty of course (without using aforementioned nuget.config trick) is that I won't be able to reference alpha9 until it's published without it. I'll see if I can get relative local paths working so that I could add a samples nuget.config that looks at the build output? It would assume that the core project was built before running samples or that the local build is available on nuget.

Gah, I'm suffering some problems with local testing against my consuming solution :/
I'm not sure net40 is working for me.
I'm about to go incommunicado for a while, sorry, work calls and then weekend plans. I'll look to pick up in a couple of days (unless I can squeeze in some time in the weekend).

Try out the samples PR. Should work and show it off for you?

Okay, I've got a little bit of time before the weekend, I'll see if I can get this out. I'm on a different machine now and not experiencing the same problem that I had before when testing (whatever it was), so I'll assume it's all good and push out a new NuGet. (Feel like there should be a :poop: joke there...)
I'll continue the discussion in the open issue/PR.


Pod Not Found

Bug Description
When trying to start mirrord exec, tool claims the pod is not found
Steps to Reproduce
 ./target/debug/mirrord exec ping some-service --pod-name some-pod-name -n some-namespace
[WARNING]: DEPRECATED - `--pod-name` is deprecated, consider using `--target instead.
Deprecated since: [28/09/2022] | Scheduled removal: [28/10/2022]
✓ layer initialized
x agent initializing...
thread '<unnamed>' panicked at 'failed to create agent in k8s: mirrord-layer: Kube failed with error `ApiError: pods "some-pod-name" not found: NotFound (ErrorResponse { status: "Failure", message: "pods \"some-pod-name\" not found", reason: "NotFound", code: 404 })`!', mirrord-layer/src/connection.rs:84:14
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
fatal runtime error: failed to initiate panic, error 5
zsh: IOT instruction (core dumped)  ./target/debug/mirrord exec ping some-service  --pod-name some-pod-name  -n dv-menv0

Backtrace
No response
Relevant Logs
No response
Your operating system and version
Linux
Local process
ping
Local process version
No response
Additional Info
No response

Hey @AeroNotix , thanks for reporting this issue.
Could you give us more information about the environment (list of pods with kubectl get pods mainly)?

If you're asking whether the pod exists... yes it does.

Wanted to check what's going on, as we're getting different behavior. Maybe it's something related to ping, so ping -V and more info about your Linux would also be helpful.

@AeroNotix could you also please check if you don't have any typos/ your namespace is correct
You can also try running in this format:
sudo -E RUST_LOG=debug MIRRORD_IMPERSONATED_TARGET=pod/metalbear-deployment-85c754c75f-982p5 RUST_LOG=trace LD_PRELOAD=target/debug/libmirrord_layer.so ping www.google.com

Thanks for the report @AeroNotix and sorry for the troubles.
I assume the default namespace used by kubectl is different than the one used with -n and you're specifying a pod and not a deployment/service (you can specify deployment using --target deployment/<deploymentname>).
The only possible reason for it to fail (that I can think of right now) is maybe we have bug with the namespace + --pod-name feature and we'll re-verify it locally to make sure it works.
As a sidenote, ping wouldn't work as you probably expect it to. Ping uses raw socket in order to send the ICMP packets, and we don't support raw sockets at the moment (it'll just do it locally). The DNS resolving of the remote service should be done in a remote manner though.
Is using ping a common use case for you or you just wanted to try out mirrord? We can talk about adding such feature, but we'd love the hear more about the use case so we can generalize it and think if it fits for the project at this moment.

We've tested it and there indeed seems to be an issue with the -n argument. In the meanwhile, @AeroNotix  could you try using --target/--target-namespace arguments instead? So the command would be:
./target/debug/mirrord exec ping some-service --target pod/some-pod-name -target-namespace some-namespace

Thanks for the responses. Definitely looks like you've reproduced it. After making this issue I did continue on and get things working with the --target and --target-namespace flags.
I did also find that ping doesn't work, which is fine enough, I understand how you are hooking into things and it's entirely reasonable that not all protocols and network mechanisms work. I used ping as a simple test to see if things were working, though as it didn't I just used e.g. curl.
All in all, I can continue evaluating this tool (in comparison to others). Appreciate the quick responses and independent debugging to confirm the issue.
Will keep an eye on this as the workflow for this tool seems so far superior to the other options.

@AeroNotix Thanks for the finding the bug and reporting.

Will keep an eye on this as the workflow for this tool seems so far superior to the other options.

That's great to hear. Thank you!


Import of the resources module

Just the initial import right now.
Before merging I want to array by run in the concrete module.

It would be good if Resources was in its own module file.

@NLAndreas to make Resources its own module click the module, and look for the checkbox in the settings panel


Fix import core and dev wd

This solves #15 by having each notebook change the python working dir to the project dir.

@kimleeng will you have a look at this updated pull-request?


Add ClassDiscovery::safeClassExists

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Related tickets
fixes #107

Documentation

License
MIT

What's in this PR?
This is an alternative to #110.

@localheinz could you try this patch? Does it work for you?

@dbu @Nyholm
Will do, give me a moment!

Unfortunately fails with
Warning: include(Nyholm\Psr7\Request.php): failed to open stream: No such file or directory  in /www/shop/lib/Varien/Autoload.php on line 94

Personally, though, I have worked around the issue by creating the client without using php-http/discovery. Works fine for my use case (legacy application), but might not for others.

That is strange.. I tried to reproduce and the patch do work.

@Nyholm
Maybe a cache issue, let me try again!

@Nyholm
No, issue still persists. But don't worry, I assume this Magento installation (<IP_ADDRESS>) has been patched a bit, maybe that's an issue.
Warning: include(Nyholm\Psr7\Request.php): failed to open stream: No such file or directory  in /www/shop/lib/Varien/Autoload.php on line 94

#0 /www/shop/lib/Varien/Autoload.php(94): mageCoreErrorHandler(2, 'include(Nyholm\\...', '/www/shop/lib/V...', 94, Array)
#1 /www/shop/lib/Varien/Autoload.php(94): include()
#2 [internal function]: Varien_Autoload->autoload('Nyholm\\Psr7\\Req...')
#3 [internal function]: spl_autoload_call('Nyholm\\Psr7\\Req...')
#4 /www/shop/vendor/php-http/discovery/src/ClassDiscovery.php(165): is_callable(Array)
#5 /www/shop/vendor/php-http/discovery/src/ClassDiscovery.php(63): Http\Discovery\ClassDiscovery::evaluateCondition(Array)
#6 /www/shop/vendor/php-http/discovery/src/MessageFactoryDiscovery.php(25): Http\Discovery\ClassDiscovery::findOneByType('Http\\Message\\Me...')
#7 /www/shop/vendor/foo/bar-api/src/Client.php(104): Http\Discovery\MessageFactoryDiscovery::find()
#8 /www/shop/app/code/local/Foo/Bar/Helper/Externalservice.php(37): Foo\Bar\Api\Client::create(Object(Http\Adapter\Guzzle6\Client))
#9 /www/shop/app/Mage.php(557): Foo_Bar_Helper_Externalservice->__construct()
#10 /www/shop/app/code/local/Foo/Bar/controllers/CategoryController.php(29): Mage::helper('coursemanager/e...')
#11 /www/shop/app/code/local/Foo/Bar/controllers/CategoryController.php(19): Foo_Bar_CategoryController->api()
#12 /www/shop/app/code/core/Mage/Core/Controller/Varien/Action.php(418): Foo_Bar_CategoryController->listAction()
#13 /www/shop/app/code/core/Mage/Core/Controller/Varien/Router/Standard.php(254): Mage_Core_Controller_Varien_Action->dispatch('list')
#14 /www/shop/app/code/core/Mage/Core/Controller/Varien/Front.php(172): Mage_Core_Controller_Varien_Router_Standard->match(Object(Mage_Core_Controller_Request_Http))
#15 /www/shop/app/code/core/Mage/Core/Model/App.php(365): Mage_Core_Controller_Varien_Front->dispatch()
#16 /www/shop/app/Mage.php(693): Mage_Core_Model_App->run(Array)
#17 /www/shop/index.php(94): Mage::run('default', 'store')
#18 {main}

Okey. I cloned https://github.com/magento/magento2. Tested with last commit (which is the same as  version 2.2.5).
Should we consider this as an accepted solution or should we try to target Magento 1 as well?

should we merge this?

Yes please


[RFC] Preserve empty lines between array elements

This has come up many times in the past and while going through the fb codebase, there are a few instances where we group array elements logically using empty lines and it's a shame that they are gone.

in cerebraljs (http://www.cerebraljs.com/) arrays and object literals are used to compose a tree of actions, which read like high level recipes of what happens in the app.
Being able to place the each item on it's own line is key to readability.
I'm sure that there are many use cases where an option to "preserve-new-lines" (even on a per file bases) would help to maintain readability.

@garth could you open an issue with real cerebral code where it outputs things that you are not happy with? This way we have concrete examples to look at in order to find a way to print it the best way :)

Did you mean to use hardline instead of softline, where it would maintain existing line breaks if it stayed expanded? Or did you intentionally want to keep arrays expanded?
I feared this was going to come up as arrays are inconsistent with objects in this regard. I personally wish we wouldn't do this, and I feel the same about objects.
But it's only my personal preference and I don't have a strong technical reason for it. It makes it so that if you add something, and it makes it expand, and then you remove it, it doesn't collapse again. I've definitely run into this already with objects where I had to manually collapse them.
I'm fine adapting to the community thought and doing what's needed. It's not a huge deal.

Oops, I just wanted to preserve new lines in between, not force it to be expanded if there was a single newline. I copy and pasted the object code.

Just replaced hardline with softline and added a test to make sure that if the array was not expanded, those empty lines vanish.

Ok, cool thanks!


undefined symbol 'syscall'

Updated my OpenBSD system and all system packages to the newest snapshot, and wakatime-cli failed to run. Error out put is as follows:
wakatime-cli:wakatime-cli: undefined symbol 'syscall'
ld.so: wakatime-cli: lazy binding failed!

Environment:

OS: OpenBSD Current Snapshot today 12-16-23
Platform: amd64

Logs:
Nothing appears to be pertanent in the logs, only that the cli failed to detect the local timezone.

To fix
I cloned the wakatime-cli repository. Executed gmake for gnu make, and copied the resulting binary to /usr/local/bin.
Please note: The provided prepacked binary DID NOT work.

I'm new to OpenBSD and I was trying to create a virtual machine locally on my Mac to debug this issue. Do you know how to have it running locally or any other way to run it? @anoduck

@gandarez Yes, sir.

I'm new to OpenBSD, I was trying to create a virtual machine locally on my Mac to debug this issue. Do you know how to have it running locally or any other way to run it?

What virtualization platform are you using? OpenBSD only supports open source drivers, so... you might be running into a compatibility issue. You won't be able to containerize it for use with docker or whatever... You will need to run it virtually. Qemu will work, might be slow, VirtualBox should work. Vmware is highly probable. Xen will work, but most definitely an excessive solution.

I just simply ran gmake, no flags or anything. Since OpenBSD does not use the gnu variant of make, you will have to install it on your system along with golang, and bash. doas pkg_add -U gmake go bash bash-completion git

@gandarez
README FIRST!!!
I just remembered, because I forgot! This issue has nothing to do with wakatime-cli and can be closed.
After performing the release upgrade, all go packages on my system had to be either reinstalled or rebuilt. Wakatime-cli was just the first package I encountered, because it runs all the time in the background.

@gandarez Thank, You! Wakatime has always offered superior support, I am a big fan. Sorry about the delay closing this one out.


Unable to change reference from static2.sharepointonline.com in SPFx project

Target SharePoint environment
other (enter in the "Additional environment details" area below)
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
Windows
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[X] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details
Libraries
Node version: v8.17.0
Package version(s):

@microsoft/sp-core-library: ~1.4.0
@microsoft/sp-application-base: ~1.4.0
react: 15.6.2
@office-ui-fabric-react: ^5.135.6
@uifabric/styling: 5.37.0

System Info
System:
    OS: Windows 10 10.0.22621
    CPU: (8) x64 Intel(R) Xeon(R) Platinum 8171M CPU @ 2.60GHz
    Memory: 25.51 GB / 32.00 GB
  Browsers:
    Edge: Spartan (44.22621.2283.0), Chromium (118.0.2088.69)
    Internet Explorer: 11.0.22621.1

Target SharePoint environment
SharePoint Server Subscription Edition (on-premise)
Describe the bug / error
Reproduction
https://github.com/jlara005/TestAppCustomizer
I am working on a SharePoint SE on-premises SPFx application customizer project and because of security reasons there is no internet access. The fonts keep being fetched from https://static2.sharepointonline.com/files/fabric/assets.
Seems like @uifabric/styling/lib/styles/DefaultFontStyles.js is one of the main culprits having the DefaultBaseUrl set to the above link. I have tried to set the fontBaseUrl window property before all imports on the main entrypoint file TestAppCustomizerApplicationCustomizer.ts file and the reference to static2.sharepointonline.com still remains. Seems like the imports are being evaluated before the property is being set. How can I set this on an application customizer SPFx project?
Actual behavior:

Several CSS/Font assets have to be fetched from CDN every time.
Adding the following property to the window object before all imports on the main entrypoint file doesn't work
window.FabricConfig = {
fontBaseUrl: 'https://mycdn.com'
};
Network requests stay same. There is no request for own CDN.

Expected behavior:

Setting the property will allow override to default url
All CSS/Font dependencies should be included as part of the NPM package.

Steps to reproduce

Clone repository https://github.com/jlara005/TestAppCustomizer
Install node modules
Run "gulp serve"
Look at developer tools "network" tab and notice the fonts being fetched from static2.sharepointonline.com

Expected behavior

Setting the fontBaseUrl property will allow override to default url
All CSS/Font dependencies should be included as part of the NPM package.

Hi there, I've filed a bug for the product group to take a look at this.

Hi there, I've filed a bug for the product group to take a look at this.

Hi Nick,
Any word from the PG?

No update yet. Will keep the ticket updated as I get more information.

The fontBaseUrl is set to blank by default on the pages in SP SE. Here is an example:


Refactor main view to separate out each tab

This view/viewmodel is quite large and hard to work with independently:
https://github.com/Esri/military-symbol-editor-addin-wpf/blob/dev/source/ProSymbolEditor/ViewModels/MilitarySymbolDockpaneViewModel.cs
https://github.com/Esri/military-symbol-editor-addin-wpf/blob/dev/source/ProSymbolEditor/Views/MilitarySymbolDockpane.xaml
It would be beneficial if we could separate these out into separate controls

Addressed in PR #309 - note for verification there is no different behavior to verify, just verify the PR shows separate XAML files for each tab now and that nothing broke in addin.


dwm: restored config patch interface

Motivation for this change
Add back the config patch interface for dwm. This feature was needlessly removed in a cleanup, and is now a breaking change between 20.09 and 21.05. Adding this interface back minimizes the changes people need to make when upgrading to 21.05.
Please try to backport this to 21.05 before release if possible.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[x] Fits CONTRIBUTING.md.

@viric for approval, soon please before 21.05 release.

If you read the thread on the PR you'll see that it's not actually an option declared and documented in the dwm module, which is the reason for the removal.
https://search.nixos.org/options?channel=20.09&from=0&size=50&sort=relevance&query=dwm
https://github.com/NixOS/nixpkgs/blob/nixos-20.09/nixos/modules/services/x11/window-managers/dwm.nix
You'll also see the mention that we can bring it back if it's an issue for users.
I was expecting at least one NixOS dwm user to raise an issue asking for it back long before the 21.05 releasw.
Most people, and the docs that mention patching dwm, seem to modify it using an overlay with override attributes.
https://nixos.wiki/wiki/St#Using_DWM
https://www.reddit.com/r/NixOS/comments/egs3k6/how_to_configure_dwm_in_nixos/
If you want this option back it'd be best if you add the patches option explicitly to the module but since the release is soon please at least add a comment to avoid it being removed again in the future.
Also it'd be great if you could add yourself as the maintainer of the package & module to assist viric

If you want it included in the 21.05 release you'll want to mention it in this issue: #121972

Its not defined in the dwm module because it alters the dwm package not the dwm module. Other packages use this pattern too, see retroarch.

You'll also see the mention that we can bring it back if it's an issue for users.
I was expecting at least one NixOS dwm user to raise an issue asking for it back long before the 21.05 release.

Well, I am a nixos dwm user, I just happen to use stable. I'm testing 21.05 a few days before release to see if things go smoothly, this is one extra change I had to make so I thought I'd try to PR and bring the feature back before others upgrading from stable run into the issue.

I don't really see any harm in this.
Please just add a comment as to why the change should stick around, so people aren't likely to clean it up later.

Its not defined in the dwm module because it alters the dwm package not the dwm module.

There's no reason the dwm module couldn't override it's dwm with the patches defined in an option in the module.
Similar to how home-manager wraps plugins into neovim/firefox for you once set using their plugins option
Or it could expose the package to be overriden like many home-manager modules such as dwm-status, rofi, and polybar etc

Other packages use this pattern too, see retroarch.

There's only around 33 packages using this pattern to configure themselves and only around 3 of those have been added within the last year.
The more standard nix method is to use overrides (and  where necessary overlays)

Either way thanks for adding the comment and for adding yourself as a maintainer 🙂👍

OK I added myself as a maintainer, and added a comment to try and make clear what the line does so it doesn't get removed in the future. I think I'll update the wiki with usage too, the current code examples have typos anyways.

There's no reason the dwm module couldn't override it's dwm with the patches defined in an option in the module.
Yes but that's a new feature, here I'm just trying to remove an unnecessary feature regression. And that feature would not be useful to people wanting to use dwm on a non nixos system. I used to use debian with nix, with dwm from nixpkgs.

In general I feel like this happened because of confusion around the "nixpkgs config set" and how it should be used. Is there good documentation on this somewhere?

And that feature would not be useful to people wanting to use dwm on a non nixos system. I used to use debian with nix, with dwm from nixpkgs.

This was just a potential nixos config option available in addition
Another reason I recommended the use of the ubiquitous overrides and overlays
It works with any nix setup and is a very useful thing to be aware of when dealing with any nix.
Getting users used to it as a pattern for enabling, disabling, and later even fixing things for themselves is important.

In general I feel like this happened because of confusion around the "nixpkgs config set" and how it should be used. Is there good documentation on this somewhere?

In this specific case there wasn't really any confusion, it was brought up as a breaking issue in that thread.
I had a really quick look for some docs but I couldn't find anything.
Using the nixpkgs config set is highly uncommon and the strongest use is for cross program shared settings e.g. pulseaudio support and I believe some cuda stuff with the tensorflow related programs.
The go-to configuration is with overrides and overlays because it's a common nix interface and doesn't require extra argument passthrough in all-packages.nix
polybar has its arguments to enable features just like dwm has an argument for patches.
By default polybar has i3 support disabled but many users will override the package to turn it on (or just use polybarFull 🤷), no nixpkgs config set passthrough required
If you'd like to add some docs I'm sure it'd be welcomed but IDK if it's something that should be recommended people utilize from here-on (again unless its some kind of shared config across programs)

I agree, so maybe the documentation should still recommend overlays, but when the feature was removed it should have been stubbed out and throw a build time error (like you did for dwm-git). Instead the build succeeded, but upon boot I was in a desktop with default dwm configuration. If I didn't have dmenu or xterm installed I would be stuck.

 Although in theory it is easy to get back to the last working generation 😅
Next time we remove this kind of setup then yes a throw would be best. I haven't looked into if it's been done in cases like this yet.

At the very least, there should be some deprecation story, but this will probably help with some dwm users coming from 20.09. So I think it should be added.

do you mind squashing the first two commits, I think adding the config option back and the comment are "one logical unit"
Some info, if not familiar:
Please follow CONTRIBUTING.md and manual#submitting-changes-making-patches  and squash the fix-up commits.
git rebase -i is a powerful command which achieves this, I created a small video demonstrating it's use here. A more indepth text tutorial can be found here

My bad, I just committed as I added stuff, squished.


Connection issue adding device

Added the plugin to HomeAssistant.
Now trying to add the integration/device.
Getting the following message:

What is especially strange is that it manages to find the device id and local key. But not the IP address. When i add it manually the message is the same.

No bug? Closed not planned? Ehh?


No response when run cell via command palette

Visual Studio Code - Insiders: 1.33.0-insider
python-insider: 2019.4.6365-alpha
Repro step:

Open the file manualTestFile.py in VSCode(download manualTestFile.py from https://github.com/IanMatthewHuff/vscode-python/blob/99cc5641c9d58b4059f939d015ac21f3095276b6/src/test/datascience/manualTestFiles/manualTestFile.py )
Select conda Interpreter as current environment
Press "Ctrl+Shift+p" to select "Python: run cell"

Actual Result:
No response when run cell via command palette

Expect Result:
It should start jupyter server output shows on "Python Interactive" window

This issue exists in 2019.3.6352, too, and seems to also affect the Run Above and Run Below commands.
There is an alternative command, Run Current Cell, which works fine and does what I would expect Run Cell to do. There does not seem to be any working alternatives for Run Above and Run Below, though.
It seems to me that the non-working commands are implemented as functions expecting arguments that specify the location of the operation (line/character positions). Those arguments are not available when invoked from the command palette.

I think this issue has been completely resolved at this point. Please comment on this issue or open a new one.


Identify Bottlenecks for Archive Node Sync

Context
For some node operators, it's been taking almost 6 months to sync an archive node. We believe this is taking way too long and would like to investigate the root cause of this slowness so that we can improve the sync time.
Desired outcome

Set up an archive node with metrics, logs, performance profiling so that we can monitor and identify anomalies in syncing speed
Identify root cause for slow syncing
Propose potential solutions to improving sync time

bonsai archive is tasked in different tickets now


how to use m2m_100 _12B Model??

https://github.com/pytorch/fairseq/tree/master/examples/m2m_100

The 12B model is currently not supported, as running it is quite complex (you need to split the model across multiple GPUs)

Do you know what configuration is required to run this model

Thank you


HBASE-22932 Add rs group management methods in Admin and AsyncAdmin

Not completed.
Hasn't change UTs, and plan to add

rpc GetRSGroupInfoOfTable(GetRSGroupInfoOfTableRequest)
returns (GetRSGroupInfoOfTableResponse);

and a method setRSGroup(TableName tableName).
Pushed here for some suggestions.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

:blue_heart:
reexec
1m  5s
Docker mode activated.

_ Prechecks _

:green_heart:
dupname
0m  0s
No case conflicting files found.

:blue_heart:
prototool
0m  1s
prototool was not available.

:green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

:green_heart:
@author
0m  0s
The patch does not contain any @author tags.

:green_heart:
test4tests
0m  0s
The patch appears to include 2 new or modified test files.

_ HBASE-22514 Compile Tests _

:blue_heart:
mvndep
0m 33s
Maven dependency ordering for branch

:green_heart:
mvninstall
5m 46s
HBASE-22514 passed

:green_heart:
compile
2m 36s
HBASE-22514 passed

:green_heart:
checkstyle
2m 51s
HBASE-22514 passed

:green_heart:
shadedjars
4m 56s
branch has no errors when building our shaded downstream artifacts.

:green_heart:
javadoc
1m 49s
HBASE-22514 passed

:blue_heart:
spotbugs
1m 47s
Used deprecated FindBugs config; considering switching to SpotBugs.

:green_heart:
findbugs
11m 36s
HBASE-22514 passed

_ Patch Compile Tests _

:blue_heart:
mvndep
0m 14s
Maven dependency ordering for patch

:broken_heart:
mvninstall
3m 39s
root in the patch failed.

:broken_heart:
compile
1m  1s
hbase-server in the patch failed.

:broken_heart:
cc
1m  1s
hbase-server in the patch failed.

:broken_heart:
javac
1m  1s
hbase-server in the patch failed.

:broken_heart:
checkstyle
0m 40s
hbase-client: The patch generated 18 new + 162 unchanged - 0 fixed = 180 total (was 162)

:broken_heart:
checkstyle
1m 34s
hbase-server: The patch generated 26 new + 159 unchanged - 0 fixed = 185 total (was 159)

:green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

:broken_heart:
shadedjars
4m 19s
patch has 98 errors when building our shaded downstream artifacts.

:broken_heart:
hadoopcheck
2m 18s
The patch causes 98 errors with Hadoop v2.8.5.

:broken_heart:
hadoopcheck
4m 31s
The patch causes 98 errors with Hadoop v2.9.2.

:broken_heart:
hadoopcheck
6m 46s
The patch causes 98 errors with Hadoop v3.1.2.

:broken_heart:
hbaseprotoc
0m 45s
hbase-server in the patch failed.

:broken_heart:
javadoc
0m 38s
hbase-server generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

:broken_heart:
findbugs
0m 56s
hbase-server in the patch failed.

_ Other Tests _

:green_heart:
unit
0m 41s
hbase-protocol-shaded in the patch passed.

:green_heart:
unit
1m 44s
hbase-client in the patch passed.

:broken_heart:
unit
0m 54s
hbase-server in the patch failed.

:green_heart:
unit
3m 41s
hbase-thrift in the patch passed.

:green_heart:
asflicense
0m 37s
The patch does not generate ASF License warnings.

73m 11s

Subsystem
Report/Notes

Docker
Client=19.03.2 Server=19.03.2 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/588

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool

uname
Linux a13a2f70f93a 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-588/out/precommit/personality/provided.sh

git revision
HBASE-22514 / 855c0922f8

Default Java
1.8.0_181

mvninstall
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/patch-mvninstall-root.txt

compile
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/patch-compile-hbase-server.txt

cc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/patch-compile-hbase-server.txt

javac
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/patch-compile-hbase-server.txt

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/diff-checkstyle-hbase-client.txt

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/diff-checkstyle-hbase-server.txt

shadedjars
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/patch-shadedjars.txt

hadoopcheck
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/patch-javac-2.8.5.txt

hadoopcheck
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/patch-javac-2.9.2.txt

hadoopcheck
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/patch-javac-3.1.2.txt

hbaseprotoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/patch-hbaseprotoc-hbase-server.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/diff-javadoc-javadoc-hbase-server.txt

findbugs
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/patch-findbugs-hbase-server.txt

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/testReport/

Max. process+thread count
1751 (vs. ulimit of 10000)

modules
C: hbase-protocol-shaded hbase-client hbase-server hbase-thrift U: .

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-588/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.0 https://yetus.apache.org

This message was automatically generated.


Added UI patterns

Added the ability for the item checker to see if it is a charm and if it is, display the Pattern id.

Thank you for the contribution @nick-dimeglio!
I had a couple of changes to make so that this works on inventories as well (https://github.com/csfloat/extension/pull/266). I'll close out your PR as a result, but thank you so much for putting this together.


Docker startup script is failing

I tried the docker run command and it's failing to boot.
$ docker run --name athenz --hostname localhost -itd -p 9443:9443 -p 4443:4443 -p 8443:8443 -e ZMS_SERVER=localhost -e UI_SERVER=localhost athenz/athenz
b95fd43af5af7c14d457d9f88cbf3d4edb6047eaaf510e373c6aa19613b4fa23

$ docker logs -t b95fd43af5af7c14d457d9f88cbf3d4edb6047eaaf510e373c6aa19613b4fa23
2017-09-11T13:53:16.679942414Z running start.sh as root
2017-09-11T13:53:16.680213400Z ---create athenz user---
2017-09-11T13:53:16.738793625Z ---initializing zms---
2017-09-11T13:53:16.750730893Z Generating private key for ZMS Server...
2017-09-11T13:53:16.763680289Z Generating RSA private key, 2048 bit long modulus
2017-09-11T13:53:16.830445330Z .....................+++
2017-09-11T13:53:17.033550097Z ...............................................................................................................+++
2017-09-11T13:53:17.033719201Z e is 65537 (0x10001)
2017-09-11T13:53:17.034664906Z Generating a self signed certificate for ZMS Server...
2017-09-11T13:53:17.042921739Z Generating a 2048 bit RSA private key
2017-09-11T13:53:17.075983531Z ..................+++
2017-09-11T13:53:17.103713301Z ...................+++
2017-09-11T13:53:17.104092861Z writing new private key to 'zms_key.pem'
2017-09-11T13:53:17.104301819Z -----
2017-09-11T13:53:17.106457721Z Generating PKCS12 keystore for ZMS Server...
2017-09-11T13:53:17.110411540Z ZMS Dev Enviornment setup complete
2017-09-11T13:53:17.118888318Z Setting ZMS root directory to /opt/athenz/athenz-zms-1.7.25
2017-09-11T13:53:17.121768581Z Starting ZMS...
2017-09-11T13:53:22.673017577Z ---initializing ui---
2017-09-11T13:53:22.673051838Z ---creating private/public key for ui---
2017-09-11T13:53:22.681990481Z Generating RSA private key, 2048 bit long modulus
2017-09-11T13:53:22.691038397Z ....+++
2017-09-11T13:53:22.729520209Z ...........................+++
2017-09-11T13:53:22.729870603Z e is 65537 (0x10001)
2017-09-11T13:53:22.733409203Z writing RSA key
2017-09-11T13:53:22.733899435Z ---copying zms X509 Certificate as ui X509 Certificate---
2017-09-11T13:53:22.736994948Z ---copying zms X509 Certificate for ui---
2017-09-11T13:53:22.738675002Z ---creating ntoken---
2017-09-11T13:53:23.099222643Z ---generate Athenz UI Configuration File---
2017-09-11T13:53:23.131200691Z 2017/09/11 13:53:23 Unable to retrieve zms service details, err: Get https://localhost:4443/zms/v1/domain/sys.auth/service/zms: x509: certificate signed by unknown authority (possibly because of "x509: invalid signature: parent certificate cannot sign this kind of certificate" while trying to verify candidate authority certificate "localhost")

Probably because of this change.
https://github.com/yahoo/athenz/commit/ea9775cf1f5f5c6322f600d7f3f562fac15d5972#diff-08ec88ed9751817ee4fbfffc6cd4da5e
I think this line is causing the error since we're using the cert as a self-signed cert. (Also as a CA cert.)
https://github.com/yahoo/athenz/commit/ea9775cf1f5f5c6322f600d7f3f562fac15d5972#diff-08ec88ed9751817ee4fbfffc6cd4da5eR16

I also found another error occuring.
It seems we just need to add the git package with apt-get. (The patch has been already included.)
---installing npm packages for athenz ui---
npm ERR! git clone --template=/root/.npm/_git-remotes/_templates --mirror git@github.com:christian-bromann/dargs.git /root/.npm/_git-remotes/git-github-com-christian-bromann-dargs-git-93c653bc: undefined
npm ERR! git clone --template=/root/.npm/_git-remotes/_templates --mirror git@github.com:christian-bromann/dargs.git /root/.npm/_git-remotes/git-github-com-christian-bromann-dargs-git-93c653bc: undefined


Fails for api-doc-template

The action fails on rename generated code if the folder does not exist.
See Logs: https://github.com/lukaspanni/api-doc-template/runs/4657227598 at line 280 and following

Well the reason for it to fail is because it seems a the  placeholder is not a valid folder/package name.
To run the test, you would need to use a valid name/version number for the generation to work.

I thought that it was a problem with the script not checking if the file exists


remove clock protocol, use autoclosure "now" instant

Motivation:
we don't need the clock protocol, but we do need the default clock to provide an instant.
We also add time to recordError in this PR
Resolves https://github.com/apple/swift-distributed-tracing/issues/119
Resolves https://github.com/apple/swift-distributed-tracing/issues/108

We have discussed this one with @fabianfett yesterday at length, it'll work nicely

I'll merge but post-merge review is welcome -- this mostly just swaps clock: some TracerClock for at instant: @autoclosure () -> some TracerInstant but we default impl using TracerClock so we can still mock it out globally if we wanted to 👍


Clip MenuView feature;

Hey there,
Thanks for this control - nice code style, working smoothly.
One of my project needs to have common background image for all views, but menu view itself have transparent background color. So I just starting to change MenuController view size with changing of ContentViewController view to reach such effect. Now it looks like that

Not sure if RESideMenu should have such functionality, but it's your call. Feel free to change or fix code somehow.
Thanks again,
Alexey

Added another specific feature (specific for transparent content views). Now it looks like that.


Deprecate bluebird in the API

As per Bluebird's GitHub:

Please use native promises instead if at all possible. Native Promises have been stable in Node.js and browsers for around 6 years now and they have been fast for around 3. Bluebird still offers some useful utility methods and you can use it - but please consider native promises first.
This is a good thing, the people working on Bluebird and promises have been able to help incorporate most of the useful things from Bluebird into JavaScript itself and platforms/engines. There are still missing things (.map/.filter are on their way with the iteration helpers proposal and async iterators!).
If there is a feature that keeps you using bluebird. Please let us know so we can try and upstream it :)
Currently - it is only recommended to use Bluebird if you need to support old browsers or EoL Node.js or as an intermediate step to use warnings/monitoring to find bugs.

Bluebird also messes with Typescript, since we usually import it as Promise (replacing the default).

Related: https://github.com/opencollective/opencollective/issues/1432

Started clearing this. There are a few places where map concurrency is not important and is being replaced with Promise.all(array.map(fn)) but there are places where it is crucial. I'll use a p- helper lib in these cases.


fix(columns): filters out empty geometry column

The geometry column seems to be treated special by Geopackages and contains blob data that doesn't display nicely. This filters it out of the source columns in the same way we do with geometry columns in other feature layers.

:tada: This PR is included in version 1.7.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


scheduled task service that runs at specified time

User story

As a user
I want/need a service like a scheduled task that runs at specified intervals to scrape needed pieces of information and store them in database
So that I can have access to those contents

Acceptance criteria

a scheduled service can be run to scrape information (in this case menus and exchange rates)
information will be stored in the database
scraper interval can be set

Definition of done (DoD)

Feature is done when the acceptance criteria are fulfilled
Feature is done when there are test where it is necessary and they pass
Feature is done when the linter passes
Feature is done when it is merged onto our main branch (after the code has been reviewed by at least one other person)
Feature is done when the code has been documented or commented properly

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

considering what should I use, crontab job at the operation system level or node-cron at an application level?


Login errors out 'Failed to retrieve user favorites: [object Object]'

Getting this when I try to submit:

[ERROR] session expired, please login again [code=-1]

This when I try to login:

[DEBUG] running cookie.firefox.login
[DEBUG] running cookie.firefox.signin
[DEBUG] try to copy leetcode cookies from firefox ...
[DEBUG] Successfully copied leetcode cookies!
[DEBUG] running leetcode.getFavorites
[DEBUG] http error: 401
[WARN] Failed to retrieve user favorites: [object Object]
[DEBUG] running leetcode.getUserInfo
/home/ron/.nvm/versions/node/v12.18.2/lib/node_modules/leetcode-cli/lib/plugins/leetcode.js:515
user.paid = _user.isCurrentUserPremium;
^
TypeError: Cannot read property 'isCurrentUserPremium' of null
at /home/ron/.nvm/versions/node/v12.18.2/lib/node_modules/leetcode-cli/lib/plugins/leetcode.js:515:27
at Request._callback (/home/ron/.nvm/versions/node/v12.18.2/lib/node_modules/leetcode-cli/lib/plugins/leetcode.js:421:12)
at Request.self.callback (/home/ron/.nvm/versions/node/v12.18.2/lib/node_modules/leetcode-cli/node_modules/request/request.js:185:22)
at Request.emit (events.js:315:20)
at Request. (/home/ron/.nvm/versions/node/v12.18.2/lib/node_modules/leetcode-cli/node_modules/request/request.js:1161:10)
at Request.emit (events.js:315:20)
at IncomingMessage. (/home/ron/.nvm/versions/node/v12.18.2/lib/node_modules/leetcode-cli/node_modules/request/request.js:1083:12)

Issue: new version of ubuntu did not have libsecret-tools
Solution: sudo apt-get install libsecret-tools solved the issue

Issue: new version of ubuntu did not have libsecret-tools
Solution: sudo apt-get install libsecret-tools solved the issue


Create license.md

Pashmak

whats the license brother

license key?

how do you know the license if demaded it to me?

can i see that if its true?

Yeah of course . O know

Ajoute-moi comme ami sur Discord ! L'invitation expire dans 1 semaine : https://discord.com/invite/98rrDxsp

I have the license. But its 100$ i can give it to you for 50$ if you want
…
On Tue, 30 Jul 2024 at 2:00 PM, LiEbee1123 @.> wrote: @.* commented on this pull request. ------------------------------ In license.md <#6 (comment)> : > @@ -0,0 +1 @@ + can u give it to me if you know? — Reply to this email directly, view it on GitHub <#6 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BA7V27GTY33JLWVTVJ5UUHLZO5FJRAVCNFSM6AAAAABIJTYXYCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDEMBXGAZDAMJQGM . You are receiving this because you commented.Message ID: @.***>

i will give you 5$

give me PLZ.
if it work i send ETH

کد مجوز چیه ؟

کد مجوز چیه ؟

کد مجوز چیه ؟

send license key if it works we all send crypto...is it a deal

if you want the access code i have it dm dicount price


Feature Request: Host LM Studio on server and Multi-GPU Support

Feature Request 1: Host LM Studio on Ubuntu Server
I would like to request the ability to host LM Studio on an server (not the local API). This feature would allow users to access the application via HTTP, with all components running directly on the server.
Feature Request 2: Multi-GPU Support with LLM Split
Multiple GPUs with an LLM split feature. This would enhance performance and efficiency when handling large models or workloads. I have a few AMD 580 8GB and would be great to use them for this.

This sounds like a fantastic idea! ⭐
I was wondering if we can make a Docker image out of LM Studio 🐳
I'm using RunPod for my models
and was wondering if we can run lms there 🤔

This sounds like a fantastic idea! ⭐ I was wondering if we can make a Docker image out of LM Studio 🐳
I'm using RunPod for my models and was wondering if we can run lms there 🤔

I did look in to make a docker container but i don't really have time to explore that right now

I did look in to make a docker container but i don't really have time to explore that right now

Well, if we can make LM Studio run on Ubuntu
and make it fully utilise the given hardware
then, perhaps, I can try to make a docker image out of it.
I'm open for ideas how we can deploy lms on AWS, GAE, Azure, etc.

I did look in to make a docker container but i don't really have time to explore that right now

Last time when I tried to install the LM Studio AppFile on docker
I got into all sorts of issues on running dbus ...
Not sure how to resolve it
but maybe someone can also have a look should we be able to run lms on Ubuntu Server 🙂


add create, update, delete

The tests project is not building (according to our build server).
Can you please change GoogleIntegrationTestBase.cs, Line 34 from:
                .AddMailmanServices(null, GetGoogleCredential())
to
                .AddMailmanServices(null, GetGoogleCredential())
And lines 52, 62, 91 and 94 or MergeTemplateTests: can you please change it from
EmailMergeTemplate.Create(...)
to
new EmailMergeTemplate() { Name = "...", SpreadSheetId = "...", ... }
I think that should do it..?
Thanks :)

at this function:
public void CreateMergeTemplateWithInvalidDateThrows()
{
Action action = () => new EmailMergeTemplate() {SheetName = "Test Merge Template", SpreadSheetId = "Id123", CreatedBy = "someone", DateTime.MinValue};
action.Should().Throw();
        Action action2 = () => new EmailMergeTemplate() {SheetName = "Test Merge Template", SpreadSheetId = "Id123", CreatedBy = "someone",  DateTime.UtcNow.AddHours(1)};
        action2.Should().Throw<ArgumentOutOfRangeException>();
    }

the function is testing whether the functins is passing extra parameters or not, but there is no "create" function that takes parameters.... I hope this makes sense lol

Right...with Domain Driven Design (what we had before), it was the responsibility of the domain object (the EmailMergeTemplate class) to ensure it is always in a consistent state by enforcing business rules whenever the object is constructed or mutated.
While we can't enforce our business rules at all times with our new approach anymore, I think we should still do our best.
This code:
  Action action2 = () => new EmailMergeTemplate() {SheetName = "Test Merge Template", SpreadSheetId = "Id123", CreatedBy = "someone",  DateTime.UtcNow.AddHours(1)};
calls the parameterless constructor and then initializes the properties within the class after the object has been constructed.
The rule being tested here is that trying to set the created date to a future time should throw an exception.  To enforce this rule when setting the CreatedDate property, we can implement a setter method on the property, like so (around this line of code (not tested)):
private string createdDate;
public string CreatedDate
{
    get => createdDate;
    set =>
    {
        if (value > DateTime.MinValues && value <= DateTime.UtcNow)
            createdDate = value;
        else
             throw new ArgumentOutOfRangeException(nameof(CreatedDate), value, "createdDateUtc cannot be in the future");
    }
}


Add more parameter types to Parameters

I have been playing with the core.Parameters class and really like it. One limitation is that it restricts the class of parameters that can be time-varying. This PR adds support for a few more classes: booleans, distributions, and functions.
You can imagine passing time-varying parameters of this class if, for instance you want:

To indicate whether an agent is retired = True | False for some meaningful change in its income process.
To have a time varying distribution for income shocks. Maybe this period they are normal, and the next they are lognormal.
To have a time varying tax function.

I have used this class in my ongoing work and have been very happy with it.

[ ] Tests for new functionality/models or Tests to reproduce the bug-fix in code.
[x] Updated documentation of features that add new functionality.
[x] Update CHANGELOG.md with major/minor changes.

@alanlujan91 should this change be reflected in other parts of the code like, say, https://github.com/econ-ark/HARK/blob/6a73c4f32a52ff9404f731761c3b754c1f85d650/HARK/core.py#L95 ?

Not sure this needs a test?

@MridulS is there a way for pre-commit to automatically upload changes?

There is pre-commit.ci but it makes changes to the PR branch and you need to make sure the contributors are pulling in all changes before making any new changes locally. I would just strongly suggest to new contributors to use pre-commit locally :)

well, somehow my pre-commit is different than the one in github actions?? @MridulS

MAC hates me.
It's the same tests as in https://github.com/econ-ark/HARK/pull/1415
Will come back to this when that's merged

This PR was blocked by some unrelated logistical thing.
Now its only conflict is the CHANGELOG.
Can this be fixed and merged now?

I'll try to look at this in the morning. I thought it still had a
(non-trivial) outstanding issue.
On Tue, Jun 25, 2024, 7:38 PM Sebastian Benthall @.***>
wrote:

This PR was blocked by some unrelated logistical thing.
Now its only conflict is the CHANGELOG.
Can this be fixed and merged now?
—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/1387#issuecomment-2190207349, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKRAFPECECNYOSMUO6NPCLZJH5QVAVCNFSM6AAAAABDV3AKYOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJQGIYDOMZUHE
.
You are receiving this because your review was requested.Message ID:
@.***>

GitHub is hanging on its automatic merge check. I'm trying to get this in now.


First crack at default config for ruff

Creating first rough draft PR after conversation w/ Eric about getting this started in a new branch.
Rough draft of the default settings and whatnot for implementing ruff as a linter & formatter. We'll need to take some time to go through the formatting, and the lints, and see what we do and don't want to turn on.
TODO:

[ ] Confirm current settings, making changes if required
[ ] Add github ci workflow to format & lint (without applying fixes, and failing on jobs that don't pass?)
[ ] Add precommit hook

@cmcginley-splunk @pyth0n1c
I've added a pre-commit hook & CI job to this PR. If you want to review the output of the lint job you can see the current errors, or you can run them locally to see the output. I have the pre-commit hook set to automatically fix lints when possible. Not all of our existing lint issues are auto-fixable, but the vast majority are.
I am considering adding E721 to the exclusion list, or at least inline-excluding the several issues in contentctl.py and would like your thoughts on this.
@pyth0n1c in particular- I've not committed the mass-format and mass-lint-fix changes to this branch, you're more than welcome to add them if you'd like, I imagine you want to wrap some of the other existing open PRs so we don't create a massive merge-conflict headache on every single open PR.

See the following first pass PR at resolving these:
https://github.com/splunk/contentctl/pull/289
Should we hold off on all these changes until we clear the PR backlog?

Retargeted this for the contentctl_5 branch- given the scope of changes for that, squeezing this in makes sense to me + the full formatting/lintfix commit will fit in with everything else.


ec_deployment: Add Importer

Description
Ideally, we'd like to be able to import existing Elastic Cloud deployments into a locally managed terraform ec_deployment resource. To do so, an Importer function must be written and assigned to ec_deployment resource.
It should be a safe importer and perform no changes on the resource when imported.
Terraform command
$ terraform import ec_deployment.imported 320b7b540dfc967a7a649c18e2fce4ed

@ggsood is tackling this issue

I'm taking this issue


Fix incorrect autocorrect for Style/UnneededCondition

Summary
Follow up of https://github.com/rubocop-hq/rubocop/issues/6013#issuecomment-398243569.
(This fixes a part reported at #6013.)
This PR fixes the following incorrect auto-correct.
% cat example.rb
if a
  a
else
  b ? c : d
end
% rubocop example.rb --only Style/UnneededCondition -a

Expected behavior
a || (b ? c : d)

Actual behavior
a || b ? c : d

I'm not sure whether it is easy to read nested conditional expressions as one liner.
So, this PR will not detect offenses, it does not auto-correct.
In any case, it is not good to break the code with auto-correct.
Other Information
In order to suppress the following offense, product code has been refactored slightly.
C: Metrics/CyclomaticComplexity: Cyclomatic complexity for offense? is
too high. [7/6]
        def offense?(node) ...
        ^^^^^^^^^^^^^^^^^^

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

The changes look good, but your branch has to be rebased on top of the current master branch due to merge conflicts.

Thank you for the review. I resolved merge conflict.

Nicely done, @koic! Thanks!

Thanks too!


Github change state lost after refreshing the pate

If the change wasn't committed to GitHub before refreshing the page, the change status will lost. Contents modified were saved to the project but only newer changes will be committed after refreshing.

Thanks for the report! I think this may be a duplicate of https://github.com/stackblitz/core/issues/2421. I think we can use that other issue to track this one.


problem installing jupyter-js-widgets

I have tried installing the latest version from the repo on mac os x (node v5.6.0, nam  v3.6.0) but I am hitting a problem with the javascript widgets.  The pip command seems to install ok using the standard instructions, but then I get the message "ipywidgets package not installed.  Widgets are not available." in the javascript console when I try to use the package in a jupyter notebook.    when I try to install the javascript separately (using "npm install jupyter-js-widgets") I get an error - I am including the entire npm log below).

Hi @poldrack , sorry that you are having this problem.
I have a couple of questions:

what version of npm and node are you using?

node -v
npm -v

If node is less than 5.3 or npm is less than 3.3, could you update them?

Another possibility is that the node_modules folder is screwed up.  You can rm -rf it and re-run npm install.

Looks like the build step is failing:
31 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'npm run build' ]
32 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1 signal: null
33 info lifecycle<EMAIL_ADDRESS>Failed to exec prepublish script

On Feb 17, 2016, at 2:03 PM, Jonathan Frederic<EMAIL_ADDRESS>wrote:
Hi @poldrack https://github.com/poldrack , sorry that you are having this problem.
I have a couple of questions:

what version of npm and node are you using?

node -v
npm -v
poldrack$ node -v
v5.6.0
poldrack$ npm -v
3.6.0
2) If node is less than 5.3 or npm is less than 3.3, could you update them?

Another possibility is that the node_modules folder is screwed up. You can rm -rf it and re-run npm install.

I deleted the node_modules folder and reinstalled everything (including node and npm), and still have the same problem.  The first error that appears in the log is:
sh: rimraf: command not found
is that a clue?

Ah yeah, it's trying to do a dev install but it's not installing the devDependencies specified in the package.json.  This may be related to a bug I've found in npm, and have submitted an issue for.  Are you installing from the repo?>

yes.  can I install those dependencies manually?

On Feb 17, 2016, at 3:18 PM, Jonathan Frederic<EMAIL_ADDRESS>wrote:
Ah yeah, it's trying to do a dev install but it's not installing the devDependencies specified in the package.json. This may be related to a bug I've found in npm, and have submitted an issue for. Are you installing from the repo?>
—
Reply to this email directly or view it on GitHub https://github.com/ipython/ipywidgets/issues/377#issuecomment-185454189.

Well you shouldn't have to...
from the repo root:
pip install -e .
cd jupyter-js-widgets
npm install
cd ../widgetsnbextension
npm run update:widgets
pip install -e .

This will A) build the Javascript B) Use the built Javascript with the nbextension C) Install that built Javascript and Python in a place Python can access it.
Later, when you want to make changes to jupyter-js-widgets, run this piece:
cd widgetsnbextension
npm run update:widgets

that mostly seems to run, but I get the following errors in the npm update:widgets step, and it still fails with the "Widget Javascript not detected” error - here is the output:
npm ERR! Darwin 15.3.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "build"
npm ERR! node v5.6.0
npm ERR! npm  v3.6.0
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>build: webpack
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'webpack'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the widgetsnbextension package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     webpack
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs widgetsnbextension
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls widgetsnbextension
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /Users/poldrack/code/ipywidgets/widgetsnbextension/npm-debug.log
npm ERR! Darwin 15.3.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "update:widgets"
npm ERR! node v5.6.0
npm ERR! npm  v3.6.0
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>update:widgets: rimraf node_modules/jupyter-js-widgets; npm install file:../jupyter-js-widgets; npm run build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>update:widgets script 'rimraf node_modules/jupyter-js-widgets; npm install file:../jupyter-js-widgets; npm run build'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the widgetsnbextension package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     rimraf node_modules/jupyter-js-widgets; npm install file:../jupyter-js-widgets; npm run build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs widgetsnbextension
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls widgetsnbextension
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /Users/poldrack/code/ipywidgets/widgetsnbextension/npm-debug.log

Oops an additional npm install  is missing immediately prior to the npm run update command! ! Sorry

ok - that worked, in the sense that the installation completed without errors.  however, when I try to run my notebook, I still get the error message “ [IPKernelApp] WARNING | Widget Javascript not detected.  It may not be installed properly.” and the widgets fail to appear.
below I am attaching the output from the javascript console - it appears that it still can’t see the ipywidgets package:
Default extension for cell metadata editing loaded.
main.min.js:24168 Raw Cell Format toolbar preset loaded.
main.min.js:24216 Slideshow extension for metadata editing loaded.
main.min.js:28971 actions jupyter-notebook:find-and-replace does not exist, still binding it in case it will be defined later...
http://localhost:8888/nbextensions/widgets/notebook/js/extension.js?v=20160218062137 Failed to load resource: the server responded with a status of 404 (Not Found)
main.min.js:33736 ipywidgets package not installed.  Widgets are not available.
main.min.js:23370 Session: kernel_created (eecc807c-0ccf-4253-8b2c-51980c59bb87)
main.min.js:22673 Starting WebSockets: ws://localhost:8888/api/kernels/fe72b3da-fc7c-455f-82aa-e6d0cea09bce
main.min.js:12109 Loading extension: livereveal/main
main.min.js:22329 Kernel: kernel_connected (fe72b3da-fc7c-455f-82aa-e6d0cea09bce)
main.min.js:22329 Kernel: kernel_ready (fe72b3da-fc7c-455f-82aa-e6d0cea09bce)
main.min.js:12842 Could not open comm  --  Error: Class jupyter.widget not found in registry (…)(anonymous function) @ main.min.js:12842
main.min.js:23119 Couldn't process kernel message WrappedError(anonymous function) @ main.min.js:23119
main.min.js:12842 Could not open comm  --  Error: Class jupyter.widget not found in registry (…)(anonymous function) @ main.min.js:12842
main.min.js:23119 Couldn't process kernel message WrappedError(anonymous function) @ main.min.js:23119
main.min.js:12842 Could not open comm  --  Error: Class jupyter.widget not found in registry (…)(anonymous function) @ main.min.js:12842
main.min.js:23119 Couldn't process kernel message WrappedError(anonymous function) @ main.min.js:23119
main.min.js:12842 Could not open comm  --  Error: Class jupyter.widget not found in registry (…)(anonymous function) @ main.min.js:12842
main.min.js:23119 Couldn't process kernel message WrappedError(anonymous function) @ main.min.js:23119
main.min.js:12842 Could not open comm  --  Error: Class jupyter.widget not found in registry (…)(anonymous function) @ main.min.js:12842
main.min.js:23119 Couldn't process kernel message WrappedError(anonymous function) @ main.min.js:23119
main.min.js:12842 Could not open comm  --  Error: Class jupyter.widget not found in registry (…)(anonymous function) @ main.min.js:12842
main.min.js:23119 Couldn't process kernel message WrappedError(anonymous function) @ main.min.js:23119
main.min.js:12842 Could not open comm  --  Error: Class jupyter.widget not found in registry (…)(anonymous function) @ main.min.js:12842
main.min.js:23119 Couldn't process kernel message WrappedError(anonymous function) @ main.min.js:23119
main.min.js:12842 Could not open comm  --  Error: Class jupyter.widget not found in registry (…)(anonymous function) @ main.min.js:12842
main.min.js:23119 Couldn't process kernel message WrappedError(anonymous function) @ main.min.js:23119
main.min.js:12842 Could not open comm  --  Error: Class jupyter.widget not found in registry (…)(anonymous function) @ main.min.js:12842
main.min.js:23119 Couldn't process kernel message WrappedError(anonymous function) @ main.min.js:23119
main.min.js:12842 Could not open comm  --  Error: Class jupyter.widget not found in registry (…)(anonymous function) @ main.min.js:12842
main.min.js:23119 Couldn't process kernel message WrappedError(anonymous function) @ main.min.js:23119
main.min.js:23119 Couldn't process kernel message WrappedError(anonymous function) @ main.min.js:23119
main.min.js:23119 Couldn't process kernel message WrappedError(anonymous function) @ main.min.js:23119

On Feb 17, 2016, at 10:31 PM, Jonathan Frederic<EMAIL_ADDRESS>wrote:
Oops an additional npm install is missing immediately prior to the npm run update command! ! Sorry
—
Reply to this email directly or view it on GitHub https://github.com/ipython/ipywidgets/issues/377#issuecomment-185561810.

I am seeing a similar issue as Poldrack's last comment. The installation appears to have worked, with extension.js being generated but on startup, even of example notebooks I am seeing a 404:
404 GET /nbextensions/widgets/notebook/js/extension.js (::1) 60.38ms referer=http://localhost:8888/notebooks/Widget%20Low%20Level.ipynb
The javascript file is appearing with the installed ipywidgets module:
/usr/local/lib/python2.7/site-packages/ipywidgets/static/notebook/js/extension.js
Is there any setting in ipywidget that would effect where extension.js is being looked for or being loaded from? Noticed a similar path at the top of ipywidgets/widgetsnbextension/src/extension.js but playing around with it did not seem to affect the 404 of errors.

Because you are running ipywidgets master, you also need to run notebook master which has been updated to load the ipywidgets Javascript from it's new location, ref: https://github.com/jupyter/notebook/pull/1022.

Let me know if I can close this issue if the above works for you...

thanks - yes, that worked, you can close the issue.

On Feb 22, 2016, at 12:37 PM, Jonathan Frederic<EMAIL_ADDRESS>wrote:
Let me know if I can close this issue if the above works for you...
—
Reply to this email directly or view it on GitHub https://github.com/ipython/ipywidgets/issues/377#issuecomment-187372733.

—
Russell A. Poldrack
Professor of Psychology
Bldg. 420, Jordan Hall
Stanford University
Stanford, CA 94305
<EMAIL_ADDRESS>http://www.poldracklab.org/


Rework to remove need for js-dist

It'd be nice to only have a single package.json as a single source of truth. There are other node libraries out there that are written in TS and distributed in JS; https://github.com/nats-io/nats.node is an example.

I think authzed-node should work fine for a non-TS consumer. I'm going to verify that before I cut it out, though. My understanding is that it was added when a user asked for it for a non-TS environment.

This is almost certainly why: https://github.com/authzed/authzed-node/blob/main/package.json#L31-L32


Feature64 fuel gauge filtering

Added simple exponential decay for "lag" factor IIR filter for resistive sensor inputs.  If the lag factor is set to 1.0, the decay and lag factor is ignored.

tested on car


请求网络操作报错

Steps to Reproduce

...
...
...

Logs

Unfortunately this is not enough information to investigate.
Please create a new issue with all information you can provide that would allow to reproduce if your are still experiencing this and think this is a Flutter bug.


for (const val of [1,2,3])  fails

for (const val of [1,2,3]) {
console.log(val);
}
src/SchemaToFlow.js:42:10,14: Unexpected token const
This cannot be suppressed with a suppress_comment
http://eslint.org/docs/rules/prefer-const.html and babel seem to suggest the above is valid/correct es6

This has been fixed since #831. You can build the master branch if you need this today, or you can wait until we cut another release. Thanks!


saturn console always show dashboard page

The new version forces the use of the database,so the saturn console should always show dashboard.

https://github.com/vipshop/Saturn/commit/e29876479efdb20b57bd278f36b465e10a970dbe


Use all and init to improve imports for mobot.

Agent class from mobot.brain.agent.Agent
Rate from mobot.utils.rate.Rate


carry on processing information from level to level

each product level (fits-file) should include ancestor information (used files, versions ...)
could be added to the fits header

I guess each level could add additional information
LB it would be good know which TMTC files and the packet numbers or packet number range is the file
L0 -> the LB file/s used to create it
L1 -> the L0 file/s used to create
L2 -> L1 file (should be 1to1 mapping) used to create

@samaloney what do you suggest could be a good technical solution to add this data into the fits files.
As this data might become a bit more complex as just simple strings in the header?
Current example is that i would like to store a IDB information which IDB version is uses for which time periods in the FITS file. As we need that information for L1 processing.

Yea I was just thinking the fits files in the headers as another information should be coming from our parameter database or somewhere else eventually?

Partially required and defined in the META data req see #138

I wonder can we close this now we have the parents and raw files?

With the new history and comment processing we could store the parent history file at each step? I think we should either do that or close this issue.

closing


@conda-forge-admin, please add user @ayushdg

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @ayushdg in conda-forge/dask-sql-feedstock#41.


Fixes #96 by handling RepositoryNotFoundError

Fix https://github.com/huggingface/alignment-handbook/issues/96 by adding exception handling for RepositoryNotFoundError in addition to the existing handling for HFValidationError

@lewtun I pushed another commit

@ArthurZucker I saw your recent comment on another PR, so it looks like you are online.  Any chance you can approve kicking off the workflow on this PR?

Sure!

Hey @tleyden it seems you need to reorder some imports in model_utils.py which make style should fix

Thanks @lewtun will try to find some time today!

@lewtun I pushed another commit, PTAL

Thanks!


Add ellipsis functionality to search paging

If there are a very high number of search results the number of pages listed can become unmanageable. The paging component includes an ellipsis option where pages can be listed as:

Needed design should include:

When the ellipsis is triggered
The page break where the ellipsis (or more than one) are included
The behavior when the ellipsis is clicked.

Tagging @astridu as design is needed.

Do you estimate a high number to be in the hundreds or thousands?

Here are a few options. https://invis.io/5YNLDND63KF#/376839243_pagination

Look design will be revisited as part of Advanced Search work, so closing this individual ticket.


fix: ulid validation regex in example

I noticed that the validation regex was incorrect for the generated ulid field.

Thanks. Always good to have correct doc.