thread 'main' panicked at 'attempted to leave type std::mem::ManuallyDrop<xlib::Xlib> uninitialized, which is invalid'

I'm new to Rust development and thought that modifying snake would be a nice starting project.
Just cloned repo, downloaded dependencies, built it and tried to run. There wasn't any error on cargo run and cargo build --release, only a few formatting warnings. When trying to run produced binary, I'm getting the error below. Any chance you could help figure out the issue?
By the trace it seems the problem is with Xlib. Does it need to be in specific version? Is it even Ok for linux?
System: Ubuntu 20.04 (Linux meland 5.11.0-43-generic #47~20.04.2-Ubuntu SMP Mon Dec 13 11:06:56 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux)
Cargo: cargo 1.54.0 (5ae8d74b3 2021-06-22)
Backtrace:
kretyn@meland:~/projects/rsnake$ RUST_BACKTRACE=1 ./target/release/rsnake
thread 'main' panicked at 'attempted to leave type `std::mem::ManuallyDrop<xlib::Xlib>` uninitialized, which is invalid', /rustc/a178d0322ce20e33eac124758e837cbd80a6f633/library/core/src/mem/mod.rs:660:9
stack backtrace:
   0: rust_begin_unwind
             at /rustc/a178d0322ce20e33eac124758e837cbd80a6f633/library/std/src/panicking.rs:515:5
   1: core::panicking::panic_fmt
             at /rustc/a178d0322ce20e33eac124758e837cbd80a6f633/library/core/src/panicking.rs:92:14
   2: core::panicking::panic
             at /rustc/a178d0322ce20e33eac124758e837cbd80a6f633/library/core/src/panicking.rs:50:5
   3: x11_dl::xlib::Xlib::open
   4: winit::platform::platform::x11::xdisplay::XConnection::new
   5: std::sync::once::Once::call_once::{{closure}}
   6: std::sync::once::Once::call_inner
             at /rustc/a178d0322ce20e33eac124758e837cbd80a6f633/library/std/src/sync/once.rs:418:21
   7: winit::platform::platform::EventsLoop::new_x11
   8: winit::platform::platform::EventsLoop::new
   9: winit::EventsLoop::new
  10: glutin_window::GlutinWindow::new
  11: <glutin_window::GlutinWindow as window::BuildFromWindowSettings>::build_from_window_settings
  12: <piston_window::PistonWindow<W> as window::BuildFromWindowSettings>::build_from_window_settings
  13: rsnake::main
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
kretyn@meland:~/projects/rsnake$

I'm new to Rust and piston_window so I'm not really sure what is the problem specifically. My guess is Xlib had some API update and that is incompatible with older version of piston_window (this package uses 0.85.0). Updating to the newest version and fixing some code has fixed the issue. Here's the commit that introduces fix https://github.com/laszukdawid/rsnake/commit/5438ef559eb29b100595ff83d81cafa26a1ceca7.

I'm new to Rust and piston_window so I'm not really sure what is the problem specifically. My guess is Xlib had some API update and that is incompatible with older version of piston_window (this package uses 0.85.0). Updating to the newest version and fixing some code has fixed the issue. Here's the commit that introduces fix https://github.com/laszukdawid/rsnake/commit/5438ef559eb29b100595ff83d81cafa26a1ceca7.


Build multi-platform image

Description
Update the docker build to make sure we're building multi-platform images.

https://github.com/tausten/docker-cookiecutter/releases/tag/v0.5.0


[authoring] code block rendering

shouldn't render until the block is closed
but also large code blocks un-render when closing is hidden?
why is there a line at the bottom of the code block?

https://github.com/dxos/dxos/assets/4529818/c1eb9874-72c1-474b-b49c-be9b1d9356bf

shouldn't render until the block is closed

We could turn that off, but if there's any other code block below the unclosed one, the backticks for that one will be taken to close it, so that solution probably doesn't do what you are hoping for.

but also large code blocks un-render when closing is hidden?

That should be fixed as of f10c1808dc1ff5b55fe02592a8b1de7b63260ef2 already.

but also large code blocks un-render when closing is hidden?

Looks like a rounding error is causing the browser to put space between these elements. I cannot reproduce it on my platform but I expect a tiny negative margin on these blocks will fix it.

We could turn that off, but if there's any other code block below the unclosed one, the backticks for that one will be taken to close it, so that solution probably doesn't do what you are hoping for.

I think you're right. Testing this again now and it feels reasonable. For some reason I remembered the backticks being hidden after they were typed and so we couldn't tell if it had been closed or not. This doesn't seem to be happening now.

That should be fixed as of https://github.com/dxos/dxos/commit/f10c1808dc1ff5b55fe02592a8b1de7b63260ef2 already.

yes, looks to be resolved from I can tell

Submitted a PR that I expect will avoid the gap/line issue

The other thing I noticed was that I couldn't click into the backtick lines in the editor when the cursor was outside the code block. Would that be resolved by the PR as well or is that a separate issue?

Noticed this issue with hidden chracters not showing up
https://github.com/dxos/dxos/assets/35851437/77e6c1e3-9791-47d2-855b-9131fa58249c

Yeah, I think the main issue in the video is that the control characters aren't shown even though the cursor is in the code block (as we established further up the thread why the code block needs to be rendered with only opening brackets).
That does make me wonder though, could it automatically add the closing backticks when you open a code block?

The code is assuming there's an end marker on the last line of a fenced code block, and incorrectly hiding that line. This patch should help with that https://github.com/dxos/dxos/pull/5835

If you want, I could add logic to fill in a line with the three closing backticks when you type three backticks at the start of a line that isn't already in a fenced code block.

Yeah, I agree. I think it makes sense to start with fixing the bug where the code block control characters aren't visible and see if that's enough to prevent this confusion.


CANVAS-109 - Fix ESLint issues

@chern Since https://github.com/ucla/lti-media-resources/pull/28 was merged into master this has some conflicts now


options.bootClasspath build cache key differs on Windows and Linux

Expected Behavior
The build chache is hit even if source/target build platform differs
Current Behavior
The build cache is is not hit.
Context
When you set CompileOptions.bootClasspath you must use platform specific path separator (; vs :) and the cache key differs betwen platforms (windows and linux).
Steps to Reproduce (for bugs)
Set CompileOptions.bootClasspath to something with multiple paths and run the build on windows and on linux:
def boot = project.fileTree("/opt/jdk/jre/lib") { include '*.jar' }
def bootcp = (boot.files as SortedSet).join(File.pathSeparator)

tasks.withType(AbstractCompile) {
  options.bootClasspath = bootcp
}

Your Environment

Oracle Linux 6 , Fedora 25, Windows 10
Gradle 4.0.2

@lptr I see you already referenced this issue in a potential fix. Could you please provide an outlook on the Gradle version that would fix the issue?

The plan is to address this for 4.2.

Unfortunately, this won't make it into 4.2.

This problem is actually more than just os-specific path separators. The only way to configure this option is to provide an absolute path string which means you'll get cache misses even on the same OS if the JARs you're pointing to live in different locations.

This is now fixed as #3002 and will ship in Gradle 4.3.


folder naming etc in examples is corrected

close #256

@agsedova btw i did not look at the jupyter notebook, is the anything specific to look at or does it just mirror the python script?


Uninstalled app still sending notifications

What is the issue? (required)
Even though I uninstalled battery (command not found in terminal when entering "battery"), I still get error pop-ups from battery stating that certain files could not be found. These popups occur every 20s or so and make working on my machine difficult.
What exactly did you do to produce the issue? (required)
Steps to reproduce the behavior:

Downloaded battery
Used battery
Tried to upgrade battery to latest version which failed with an error message that I forgot.
Tried uninstalling battery using the battery script.

Expected behavior (required)
App completely uninstalled and no error popups stating that files are missing.
Screenshots (optional)

Error logs
zsh: command not found: battery
Did battery fail to install? Use tail -n 1000 ~/.battery/*/log instead.
zsh: no matches found: /Users/aaron/.battery/*/log
Additional information
Mac OS still only charging to 80%.

The CLI uninstall does not uninstall the GUI. If you close the GUI and reopen it, battery will reinstall. You can uninstall from the GUI, or delete the GUI app and uninstall from CLI.


I run it on docker map port to 6005. Navigated to https://iconify.design/

After I run the container. Open localhost:6005 redirected to https://iconify.design/

Yes, index page is a redirect because there is no index page.
You can change it in config, see https://iconify.design/docs/api/hosting-js/config.html


can sometimes tripple jump

example? also remember to set milestone and project

probly only way to do this is by changing the ground detection to something other than a circle radius

is that actually the problem

done?

on_collision_enter
{
    if it is the ground tag
        touching = true
}

on_collision_exit
{
    if it is the ground tag
        touching = false
}

make player box bigger and spikes smaller, adjust the things


Correct Way To Run (Pegasus) Summarization Models Locally?

We've trained a summarization model via AutoNLP that we've been successfully using with the Accelerated Inference API for a few weeks. We'd like to start running this model on our own GPUs to decrease cost.
The model we selected was PegasusForConditionalGeneration. We've successfully run the model locally using something like
device = 0 if torch.cuda.is_available() else -1
tokenizer = AutoTokenizer.from_pretrained("/mnt/models")
model = AutoModelForSeq2SeqLM.from_pretrained("/mnt/models")
pipeline = pipeline('summarization', model, tokenizer=tokenizer, device=device)
output = pipeline(input_text)

or also
tokenizer = AutoTokenizer.from_pretrained("/mnt/models")
model = AutoModelForSeq2SeqLM.from_pretrained("/mnt/models").cuda()
input_ids = self.tokenizer(input_text, return_tensors="pt")['input_ids']
output_ids = self.model.generate(input_ids=input_ids.cuda())
output = tokenizer.batch_decode(output_ids, skip_special_tokens=True, clean_up_tokenization_spaces=False)[0]

where /mnt/models points to the downloaded model repository from the Huggingface model hub.
However, neither of these methods give the same results as the Accelerated Inference API, given the same input. We suspect that perhaps tokenization is being performed differently, or the model is being called with different parameters.
Is there somewhere we can confirm exactly how the Accelerated API is loading and calling the model?

Hi @mitchellgordon95 !
The preferred point of contact for inference API related questions is the following email address:
api-enterprise[at]huggingface[dot]co
I'm closing this issue since it's not directly related to AutoNLP


Convert ProfileBarChart.js to ES6 module

Who is this PR for?
Developers who want to contribute to the codebase by adding or improving client-side features.
What problem does this PR fix?
Not all our JS files use ES6 modules, which can be confusing for new JS devs coming on to the project.
What does this PR do?
Converts ProfileBarChart.js to ES6.
Checklist

[x] Student Profile charts for attendance/behavior — tested latest commit in Internet Explorer

Screenshot (Internet Explorer)

Looks good in prod in IE. 👍


Synthetic CUBE test

[x] make script that renders "twisted cube" into a frame sequence and saves the frames (color + depth), graph, and node rotations + translations to some directory
[ ] make a pipeline option & mechanism to read node transformations instead of using the tracker to produce them
[ ] run fusion with these settings on the generated cube sequence, assess the result

I don't think this is at all relevant at this point -- fairly certain everything this would be testing is working fine.


Star Wars Battlefront 2 (2005)

The game hangs on startup, and when running the game's modding debugger it produces the following in the bf2log: Cannot create Direct3D device! Error: 0x88760868
This game does use shaders for bump maps, rain, and a few other effects but does have a working command line option for /fixedfunction. Tested on the GOG version, with an RTX 2070S.

Do you get any logs from Remix?  d3d9.log, NvRemixBridge.log

Hello! Will you please test this on the latest release and let us know if the issue persists? Thank you!


indicator component's bug

apache仓库不是最新的代码，所以在这里提交了。
第一次加载没有获取到children，导致el.children[1]报错
_getIndicatorRect也没有做children.length=0的判断处理
还有很多reset样式优先级都很高
如：.weex-root p, .weex-root ol, .weex-root ul, .weex-root dl{margin:0;padding:0;}高于一切
望处理一下。

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add missing variable

Adds a missing variable
Also, should the following values be True or False by default?

add_atom_ids: bool = False
add_atompair_ids: bool = False
directed_bonds: bool = False

@amorehead probably good to have the network know about the first two, and directed bonds is probably less essential

these two hyperparameters will have to be correspondingly set (if turned on)


Is it possible to align at top center?

Hi,
Is it possible to align the notifications at top center of the page?
Thanks!

Sorry for the delay in responding. Yes, it is. This seems to be working for me:
.alertify-logs {
    width: 50% !important;
    top: 0px !important;
    left: 25% !important;
    right: 25% !important;
    bottom: initial !important;
}

You'll just have to add it to any of your CSS files after including the Alertify css file.


Updated our CPS version

Escrow Template:
Customer scenario – This does not have a direct customer scenario, but it unblocks 3 issues that we want to target for RC2: #399, #400, and #738. Additionally, this should stop additional loads of Microsoft.VisualStudio.Shell.Immutable10.
Bugs this fixes:  #659, unblocks #399, #400, #738.
Workarounds - N/A
Risk – None. This doesn't change any code.
Performance impact  - None. there are no functional changes.
Is this a regression? - The VSSDK was changed to remove direct references to VisualStudio.Shell.Immutable.* by using a single roll-up package. This is a reaction to that change.
Root cause analysis - N/A
How was the bug found? - N/A
Tagging @dotnet/project-system @RaulPerez1 for review, @jinujoseph for the XAML changes.

@dotnet-bot retest Windows Release please

Tagging @MattGertz for approval.


Error handling is not supported on getAccessToken

Upon $http.post in OAuthProvider::getAccessToken, error handling is not returned as a promise.
Adding this would allow login validation:
return $http.post("" + config.baseUrl + "" + config.grantPath, data, options).then(function(response) {
    OAuthToken.setToken(response.data);
    return response;
}, function (error) {
    return error;
});

$http uses promises.
I can't understand what is your problem. This should be working without the change you're suggesting:
OAuth.getAccessToken(credentials)
  .then((response) => {
    // Cookie was stored.
  })
  .catch((response) => {
    // Some error occurred.
  });

My bad, works fine :)


TypeError: ClientError.init() missing 1 required positional argument: 'operation_name'

Describe the bug
Type error when running a sql query on an RDS DB using Data API.
How to Reproduce
import awswrangler as wr

con_rds = wr.data_api.rds.connect(
    resource_arn=r_arn,
    database=db_name,
    secret_arn=s_arn
)

df = wr.data_api.rds.read_sql_query(
    sql="SELECT * FROM some_table",
    con=con_rds,
)

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Input In [10], in <cell line: 10>()
      1 import awswrangler as wr
      4 con_rds = wr.data_api.rds.connect(
      5     resource_arn=r_arn,
      6     database=db_name,
      7     secret_arn=s_arn
      8 )
---> 10 df = wr.data_api.rds.read_sql_query(
     11     sql="SELECT * FROM some_table",
     12     con=con_rds,
     13 )

File /opt/conda/envs/ml10/lib/python3.10/site-packages/awswrangler/data_api/rds.py:162, in read_sql_query(sql, con, database)
    146 def read_sql_query(sql: str, con: RdsDataApi, database: Optional[str] = None) -> pd.DataFrame:
    147     """Run an SQL query on an RdsDataApi connection and return the result as a dataframe.
    148
    149     Parameters
   (...)
    160     A Pandas dataframe containing the query results.
    161     """
--> 162     return con.execute(sql, database=database)

File /opt/conda/envs/ml10/lib/python3.10/site-packages/awswrangler/data_api/connector.py:27, in DataApiConnector.execute(self, sql, database)
     15 def execute(self, sql: str, database: Optional[str] = None) -> pd.DataFrame:
     16     """Execute SQL statement against a Data API Service.
     17
     18     Parameters
   (...)
     25     A Pandas DataFrame containing the execution results.
     26     """
---> 27     request_id: str = self._execute_statement(sql, database=database)
     28     return self._get_statement_result(request_id)

File /opt/conda/envs/ml10/lib/python3.10/site-packages/awswrangler/data_api/rds.py:91, in RdsDataApi._execute_statement(self, sql, database)
     89 if response is None:
     90     self.logger.exception("Maximum BadRequestException retries reached for query %s", sql)
---> 91     raise self.client.exceptions.BadRequestException(
     92         f"Query failed - BadRequestException received after {total_tries} tries and sleeping {total_sleep}s"
     93     ) from last_exception
     95 request_id: str = uuid.uuid4().hex
     96 self.results[request_id] = response

TypeError: ClientError.__init__() missing 1 required positional argument: 'operation_name'

Expected behavior
No response
Your project
No response
Screenshots
No response
OS
Amazon SageMaker Studio (see additional context)
Python version
3.10.5
AWS DataWrangler version
2.16.1
Additional context
Amazon SageMaker
Image: Data Science 1.0 (Anaconda)
Instance type: ml.t3.medium
custom conda environment with Python 3.10.5 installed (all latest packages through conda-forge)

Looks like something is wrong with your query however due to a bug in wrangler it obscures the original exception. Addressed in #1457

Hi @kukushking . I'm from @nrcjea001  team. Tested locally with DBeaver and no connection issues. Is this a bug in SageMaker?

Hi @leandroroser no I don't think so. But the fix was just merged to main so you can give it a try to install from main like below:
pip uninstall awswrangler -y
pip install git+https://github.com/awslabs/aws-data-wrangler.git@main

With this fix you should be able to re-run your code and see what was the original exception

Thanks @kukushking  , we were able to debug the problem after the fix. Appreciate your time.

Happy to help! Feel free to reopen if something else pops up


After a SQL results tab is pinned, executing any (new or existing) SQL query should open a new results tab

Type a SQL query
Click Execute query
Opens results in a results tab
Pin the tab
Click Execute query again
Should open results in a new results tab

Current behavior

Click Execute query again
Refreshes the query in the pinned results tab

/causedByExtension


electron kiosk seems to have problems with 4k textures

Many WebGL demos work on the host but not inside the kiosk container if big textures (like 4k: 4096xsomething) are used. Not sure what causes this problem, but it seems to be the overall cause for the white-outs we have seen recently.

Could you please specify some (one would suffice for now) of the failing WebGL applications and their inconsistent behavior under Kiosk/Chromium/Google Chrome(/Opera?) when running from inside our docker containers vs. via host-native browsers?

I think, I have to be a little more precise. The Habitable Zone WebGL app works in Chromium within the chrome image (I didn't test Google Chrome and Opera) regardless of the size of the texture (the actual texture size limit on my NVidia card is 16384). However, the app fails to run in the kiosk browser within docker. The application shows up for a couple of frames while the large 4k image of the milky way is still loading in the background. Then, suddenly, the app stops working (aka white-out). It looks like this happens exactly when the texture is fully loaded and transfered to WebGL/to the GPU. The is also a 2k image of the milky way in the same folders as the 4k image and it is trivial to modify the javascript code to make use of it instead of the 4k image. Once changed to 2k, the app works fine in kiosk.
Strangely, I tested the same web app last week with Chromium installed into the kiosk image and it showed the same strange behavior as electron. Inside the chrome image, however, Chromium also works with 4k.
Can you test all the following combinations on your side: (kiosk image,chrome image)×(Google chrome, Chromium, Electron)? Maybe there is just something wrong with the kiosk image?

I just rebuilt the kiosk image with electron 0.34.3 just to see if the issue is maybe just solved with a newer version, but it isn't. No need for you to test that.

Sorry but i cannot reproduce your issue: both Chromium and :kiosk handle that WebGL app. correctly (4k & 2k)...
Just to be sure that we are on the common ground:

we are talking about WebGL_Habitable_Mini/gravity_habitable_33b.html, right?
how do you get that WebGL application into browser? I use www.sh & open http://localhost:8080
how do launch :kiosk? I do : cd kiosk && make pull && make check
my NVidia diver: 352.41 (tested in "Additional Drivers")
OS: latest Gnome Ubuntu 15.10 (with all updates)

ps: WebGL_Test/test_03_4k.html (and 1k) is also fine here...

Strange. About the common ground:

yes
yes
yes
no, I didn't do make pull, instead I just used a version from last (?) week or so. But I also tried make build on the recent git HEAD (see above) and had the same error.
yes

But surprisingly today, the 4k version works on my machine as well. Unfortunately, I just tried make pull instead of testing with make check beforehand. But I also tried make build && make check today and it also works. The error kind of disappeared while it was 100% reproducible yesterday. But also Dorotea reported the 4k issue, didn't she?
I think it is important to find out what caused this issue.

make build can be used only after updating the parent image (same way).
Therefore i recommend to make pull before use just to be sure.
ps: i suspect you had some outdated :kiosk...

This is all not very reproducible. This is why I recommended version tags for the images. What has been changed in the parent image? Have you done anything on purpose or does it only work by chance?
I know that the discussion about the 4k problem came up during our discussion at the MFO. But I cant't remember how/if it has been fixed.

Well, due to apt-get update at the beginning of each package installation - each new subsequent build will very lickely get a new unique ID. It gets even worse due to the tree dependency structure of our images.
Moreover my tree dependency structure via tags seems to be ignored by docker hub automatic builds - there may be wrong builds in case of chnages to several dependent images.
Note that each time git repo is changed - docker hub rebuilds all our images and due to above.
Therefore i recommend to pull all the recent images together if you plan to rebuild any of them locally.
Do you want to start now the rearrangement from the current state into separate docker hub repositories for each of our images in order to introduce version tags?
ps: as you can see from the git log - i did not do anything related to :kiosk since 2015-11-03 all the recent changes are due to ubuntu update/upgrade...

The reproducibility discussion should be continued in #17.
I don't have any idea how to reproduce the error. Maybe it's just gone. Maybe not. If you don't have any idea how to proceed, you can also close this issue.

Will be reopened if the 4k issue shows up again.


Goes in infitie loop with some data

It's all in the title, with this simple data the script never stops. Am I missing something her e?
import Loess from "loess";

const data = {
  x: [
    19815, 19810, 19809, 19805, 19801, 19798, 19785, 19782, 19780, 19760, 19760,
    19760, 19725,
  ],
  y: [0.5, 1, 1, 0.5, 0.5, 0.5, 0.5, 0.5, 0.5, 0.5, 0.5, 0.5, 0.5],
  w: [
    2265025, 3904576, 2253001, 4108729, 1236544, 4511376, 138532900, 1234321,
    1806336, 1838736, 1838736, 1838736, 1188100,
  ],
};

const loessOptions = {};
const model = new Loess(data, loessOptions);
const fit = model.predict();

I've figured this one out --
Math.js median operation hangs on NaN inputs: https://github.com/josdejong/mathjs/issues/1214
I just stepped through the repo and verified this is what is triggered in your case.
One solution is to upgrade the dependency of math.js, but I don't have enough time to debug why exactly they're nan in the first place...

https://github.com/yongjun21/loess/issues/11

(but really the solution should be to make mathjs a peer dependency with version > 5.1.1


FINN - MarkupSafe import Error

Hi,
When I try to reach finn notebook from my Ubuntu machine I get an error like this. Has anyone encountered it before? I can access finn workspace normally except notebook feature.

Have you found any solution?

Yes, I just add the following to the finn/docker/Dockerfile.finn file. I added it to line 86.
RUN pip install markupsafe==2.0.1

I tried that but the version is incompatible. So, I can resolve include in finn/docker/Dockerfile.finn file:
RUN pip install MarkupSafe==1.1.1


Add API for a custom list formatter

With this feature I hope I'll be able to format my dumps with custom indentation, maxArrayLength and breakLength.
Edit: Nah, put it on hold, I'll implement an even better version tomorrow. Edit: done.

I'm not sure I understand the use case - but regardless, the tests (that don't have your option provided) must not change, otherwise it's a breaking change.

I've reverted the version bump as requested, and will move the whitespace cleanup to its own PR.

The new version has the better API that I announced I'd implement today: Type and size are now separate. I managed to even replicate all the old oddities so tests don't have to change and semver jump could probably be at just patch.

New version with the EOL whitespace preserved as per #11.

Thanks for your comments! Maybe someone might fix them and see whether it might then land in master. My own motivation for sharing my new feature is used up though, so I'll just fork it and use it as I see fit.

@mk-pmb adding a new feature requires eternal maintenance, so if you don't have the energy/motivation to make it through a mild code review, it's pretty clear that you won't be around long term to help maintain it either, which generally means it's a bad idea to add the feature.
Thanks for the contribution - if you change your mind, I'm happy to reopen this.

You're right, the maintainance perspective is another good argument to go separate routes from here, as I won't re-configure my editor to preserve EOL space any time soon. ;-)


The output seems slow to me. Am I doing something wrong?

Hi,
I'm using lines0.so v0.1.1.
If I run this command on this input csv file
sqlite3  tmp.sqlite "select count(*) from lines_read('input.csv');"

I have the output in 0.229 seconds.
If I apply the same to your example ndjson file
sqlite3  tmp.sqlite "select count(*) from lines_read('calendar.ndjson');"

I have the output in 51.493 seconds. Probably I'm doing something wrong.
Using wc  <calendar.ndjson wc -l I have the output in 1.053 seconds.
Thank you

Hey @aborruso , that does seem odd - can you run the following script as a file called debug.sql?
.bail on
.timer on

.load ./lines0

select lines_debug();

select count(*) from lines_read('calendar.ndjson');

And then run it with sqlite3 :memory: '.read debug.sql'?
$ sqlite3 :memory: '.read debug.sql'
Version: v0.1.1
Date: 2022-06-22T14:35:59Z+0000
Source: 37c8d2dde4c97395b6af837ec7bd6f7af639e79f
Run Time: real 0.000 user 0.000000 sys 0.000138
321981
Run Time: real 0.126 user 0.066360 sys 0.059196

It runs in ~100ms for me on a digital ocean droplet with 8gb of RAM.

Thank you @asg017
I have used your script, but I have removed .load ./lines0 because I have .load /home/aborruso/library/lines0.so in my ~/.sqliterc file.
This is the result
-- Loading resources from /home/aborruso/.sqliterc
Version: v0.1.1
Date: 2022-06-22T14:35:59Z+0000
Source: 37c8d2dde4c97395b6af837ec7bd6f7af639e79f
Run Time: real 0.000 user 0.000000 sys 0.000172
321981
Run Time: real 52.250 user 0.617902 sys 6.183273

I use the lines0 I have in lines0-linux-amd64.zip

Woah, that's pretty wild to see... Could you try compiling the library yourself and see if that changes anything? It should be straightforward to do, no external deps
git clone git@github.com:asg017/sqlite-lines.git
cd sqlite-lines
mkdir -p dist/
make loadable
sqlite3 :memory: '.timer on' '.load dist/lines0' 'select count(*) from lines_read("calendar.ndjson");'

I'm guessing the pre-compiled library might be way slower for some reason, and possibly compiling yourself may fix it? Weird how it only seems to get tripped up on calendar.json, too

Could you try compiling the library yourself and see if that changes anything? It should be straightforward to do, no external deps

It's the same :(


proxy: Rework oidc role mapper to allow multiple matching roles

If multiple claims values have a valid matching for ocis roles, we'll pick the ocis role that appears first in the mapping configuration.

E2E tests failed: https://drone.owncloud.com/owncloud/ocis/21319/68/1
:boom: To see the trace, please open the link in the console ...

npx playwright show-trace https://cache.owncloud.com/public/owncloud/ocis/21319/tracing/folder-alice-2023-4-13-12-57-38.zip
npx playwright show-trace https://cache.owncloud.com/public/owncloud/ocis/21319/tracing/folder-brian-2023-4-13-12-57-53.zip


Time 0.1 request: make fields of duration public

For Chrono I am preparing a branch that makes more methods const.
Because the Duration type of time 0.1 is part of its public API it still has a dependency on that old version (although it can be disabled).
I am not asking for major changes. But would you consider making a new point release that makes the fields of Duration public? That would make it possible to convert from time::Duration to our own implementation, even in const methods.

Sorry, this request didn't make sense. Ignore me 😄


Minimize on close

Is this possible to support the minimize on close functionality? Not necessarily moving to the menu bar, but at least minimizing the dock when clicking the close button? I miss this functionality, because I want the app to display content immediately when activated, and I don't wait for the page to load. I am looking for something similar to this functionality in Electron

mainWindow.on('close', function (event) {
    if(!application.isQuiting){
        event.preventDefault();
        mainWindow.hide();
    }

    return false;
});

Planned for likely next release.


CCCM dosn‘t work properly

Describe the bug
Under parameter I can see 6.25A current
But at the main page of Venus OS I see only -4A current with -109W
To Reproduce
Steps to reproduce the behavior:

Go to '...'
SerialBattery
Click on '....'
Parameter
Scroll down to '....'
see the expected current
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

VenusOS (please complete the following information):

Device type: Raspberry Pi3b
Firmware Version 2.90 - 14

Battery/BMS (please complete the following information):

BMS/Battery type: JKBMS
Cells: 8
Interface: USB-RS485

Additional context
Add any other context about the problem here.

Serialbattery is just setting the max limit.
This works on my side. (For example max 5A at 99%)
If its less than this setting, ask your inverter to pull more current

Knedox is correct. This does look like it is working correct.
If the Discharge Current Limit(DCL) is 6.25A it means when you see a discharge (the negative A and W value) then that current should be less than 6.25A, which it is as 4.3A.
The DCL value will change if your SOC goes low. Between 10-20% this will be 1/4 of your normal value which is where the 6.25A value is coming from.
https://github.com/Louisvdw/dbus-serialbattery/wiki/Features#charge-current-control-management

@Louisvdw
But why we have the delta of 2A? Between the current 6.25A and the real 4.3A?
Regards

Hi @testpaul999
You have two different things here. The one is a limit where we say the Maximum you should be using it 6.25A
The other is the actual usage that is being used by your system. If your inverter is only drawing 4.3A that is because that is what is being used. If you switch on more lights for instance that will increase. If we said the system must draw 6.25A all the time while only 4.3A is used, then the 1.95A difference will only be wasted as there is nothing using that power and your battery will be depleted faster.
You want the system to only supply what is needed and nothing more.

Hi,
but look at this

The need for more power is there.

OK , so if I understand what you want to say in pictures is that you expect that the battery delivers 6.5A as the limit is 6.5A, but the picture only show usage from battery of 4.3A with the rest coming from the grid. Is that what you are trying to say?
So in a perfect situation that would be the case they are the same always, but 1) the inverter can only slowly ramp up/down the power usage and 2) the measurements might not be the same acuracy everywhere.
That is not something the driver can do anything about. The driver only read the values that your BMS supply. The Victron system then has to dicide how to utilise that.
And if the BMS values is a bit off then it might need a bit of a calibration. (that does not seem to be a problem as the values match within a few Watts)

OK , so if I understand what you want to say in pictures is that you expect that the battery delivers 6.5A as the limit is 6.5A, but the picture only show usage from battery of 4.3A with the rest coming from the grid. Is that what you are trying to say?

:-) sure that’s what I mean.

So in a perfect situation that would be the case they are the same always, but 1) the inverter can only slowly ramp up/down the power usage and 2) the measurements might not be the same acuracy everywhere. That is not something the driver can do anything about. The driver only read the values that your BMS supply. The Victron system then has to dicide how to utilise that. And if the BMS values is a bit off then it might need a bit of a calibration. (that does not seem to be a problem as the values match within a few Watts)

hmmm, ok I’m not happy with that but thx for explanation.

the inverter can only slowly ramp up/down the power usage
not a problem with watts/s this will shown the whole time until the SoC under 20%


Asynchronously loading scripts for an isomorphic app

In my isomorphic app, I have an entry point application.js file that runs on both the client and the server.
var NotFoundPage = require("./pages/not-found");
var ErrorPage = require("./pages/error");
var LoadingPage = require("./pages/loading");
var LoginPage = require("./pages/auth/login");
var JoinPage = require("./pages/auth/join");

// some routing logic to render different pages given above
...

My problem is, when I attempt to use the following syntax to load scripts depending on the requested route :
require.ensure([], function(){
     // the page scripts
});

I get errors like TypeError: undefined is not a function, but yet if I use the synchronous syntax require(...), my scripts will all be bundled up into a giant js file.
What can I do to tell webpack to create async chunks for me given an isomorphic entry point?

Having the same problem, although I'm not quite sure if webpack context will work they way I am anticipating.  Regardless require.ensure is not being processed correctly.
require.ensure(['react-router-proxy!./'+component], function(require) {
return require('react-router-proxy!./'+component)
}) : require('./' + component))
is compiling to:
webpack_require(29).ensure(['react-router-proxy!./'+component], function(require) {
return require('react-router-proxy!./'+component)
}) : require('./' + component))

You cannot have a expression in require.ensure. Just omit the require.ensure. It wrong here and react-router-proxy already uses async loading.
If you are doing this for prerendering this won't work, because the prerendered html doesn't fit the to rendered html. React will warn you!
Instead you need to replace the proxied component with a empty component (react-router-proxy should provide it), because in the initial page load the async component is not loaded yet.

@geekyme Did you figure it out based on the tip?

also, i have the same problem

Hi, if you are using webpack-isomorphic-tools you should set patch_require: true, it is false by default.
From documentation:

// enables support for require.context() and require.ensure() functions.
// is turned off by default
// to skip unnecessary code instrumentation
// because not everyone uses it.
//
// patch_require: true, // is false by default

Closing as this is question.


[roles-0.1.0] Root

Awesome work devs. I was able to complete the whole process on this Safe for mobile - https://deploy-preview-2045.app.dev.decentdao.org/roles?dao=sep:0x40C6E5eFCE3b1f101bFB59e876F14aa5455D3E19
@Da-Colon / @adamgall - few bits of feedback.

once you create a role and you get taken back to the edit role page, the 'you haven't created any roles' should disappear

after I created a proposal, the pending proposal toast came up correctly but I wasn't taken to the proposal list page

I didn't catch it on video but the role loading icon when you navigate to the roles page and the roles are loading makes the whole component move up and down as it loads (i.e. like this used to be for hierarchy before we fixed it)

finally, when the proposal is created, clicking to view the roles didn't really work, video below. This probably is known but just highlighting
https://decentdao.slack.com/archives/C06QZTUTKRC/p1720195880327889?thread_ts=1720195806.247159&cid=C06QZTUTKRC

Awesome work devs. I was able to complete the whole process on this Safe for mobile - https://deploy-preview-2045.app.dev.decentdao.org/roles?dao=sep:0x40C6E5eFCE3b1f101bFB59e876F14aa5455D3E19
@Da-Colon / @adamgall - few bits of feedback.

once you create a role and you get taken back to the edit role page, the 'you haven't created any roles' should disappear

after I created a proposal, the pending proposal toast came up correctly but I wasn't taken to the proposal list page

I didn't catch it on video but the role loading icon when you navigate to the roles page and the roles are loading makes the whole component move up and down as it loads (i.e. like this used to be for hierarchy before we fixed it)
finally, when the proposal is created, clicking to view the roles didn't really work, video below. This probably is known but just highlighting
https://decentdao.slack.com/archives/C06QZTUTKRC/p1720195880327889?thread_ts=1720195806.247159&cid=C06QZTUTKRC

Will Update
Will Update
Yep on the list
So is this one.


Separate browser specific import maps

Since it's only few that are needed in the browser, likely it makes sense to separate them. Maybe that's something for the site configuration even.

This has been done. Now it's a filtering operation through the configuration.


Open source checklist 📝

Open source

[ ] Port Zed over to GPUI2

10/30/2023: 30% of crates ported

[ ] Extended preview period - time to find and squash all the bugs before stable
[ ] Code documentation
[ ] GPUI website
[ ] CONTRIBUTING.md guide
[ ] Channels public release

What is left / needed before we ship

[ ] Potential blog posts

[ ] Tutorials / walkthroughs
[ ] ...

This list is a work in progress. Items may be added or removed over time.

Update on the GPUI2 app port:

Update #2. Ignore the red boards around some buttons.

A couple of new screenshots of the progress (ignore the visual bugs)

The changes have been merged into preview. Blog post here, release here.

Very excited about this. To clarify, is the plan to go open source shortly after preview is promoted to stable, or are the rest of the items on the checklist expected to take a significant amount of time?

Looking the checklist I don't see an open source OS support there. I wonder if that could be prioritized, so open source OS users could enjoy open source zed, rather than letting only closed source OS users enjoy open source zed.

@jangorecki that seems to be tracked separately in issue #174

Very excited about this. To clarify, is the plan to go open source shortly after preview is promoted to stable, or are the rest of the items on the checklist expected to take a significant amount of time?

The plan is to finish off the checklist - that being said, this list is a little misleading, as a few of the unchecked items are being worked on at the same time and are almost done, so progress is going to jump really soon!

What license do you plan to use?

The Zed codebase is now open sourced:

Repo
Blog post


Intake Subsystem

Intake
What is the primary purpose?
To pick up cargo off the ground.
What operations does it need to do?

Deploy/retract roller.
Pick up cargo from the ground.
Turn on/off the roller.

What does the subsystem need to know?

What position the intake mechanism is in.
Whether the roller is spinning or not.
The color of the ball the intake mechanism picks up.

What actuators are on the subsystem?

2 motors; one for roller, one for deployment.

What sensors are on the subsystem?

Color sensor.
(Maybe) limiting switch for deployment.

How can the subsystem fail?
The subsystem will fail it if cannot pick up cargo, this can happen either from the intake mechanism being jammed, the roller failing to spin, or the intake mechanism failing to be deployed.
Limit switch failure driving the intake into the ground and/or the robot.
How can we mitigate failure?
Make sure the intake mechanism can be deployed reliably and also roller spinning successfully(in the right direction).
Current based limiting of deployment.
Time based limiting of deployment.
What can be automated?
Intake deployment, color sensing.
Notes

Having color sensors on the intake mechanism itself may be an issue because if you have two cargo on the correct color and one the wrong color it will shoot out both. Having a ball shunting system in the transport itself will solve this issue.

[x] Deploy/retract roller
[x] Turn roller on/off
[x] Limit switches
[ ] Color sensing
[ ] Automatic intake deployment

Intake has been working well! Closing this.


[FLINK-5512] [doc] Improve RabbitMQ documentation

Added information about non-parallel restriction for RabbitMQ source in order to achieve exactly-once.
Also attempted to overall improve the documentation.

Very nice, +1 to merge this

Very helpful. But I did stumble on this sentence -- I'm not sure what "of a queue" is telling me.

This limitation is mainly due to RabbitMQ's approach to dispatching messages of a queue across multiple consumers.

Would it be accurate to say "... dispatching messages from a single queue to multiple consumers" ?

Thanks @StephanEwen and @alpinegizmo for the review.
I think the re-wording for "of a queue" is definitely an improvement.
I'll change it to your suggestion and merge this by the end of the day.


🐛 Handle electron default env

Fixes #696
see https://github.com/electron/electron/issues/8212 for more info :)

Amazing

🎉


b2-hard-delete 400 file_not_present

When filing an issue, please include the following information if possible as well as a description of the problem.  Make sure you test with the latest beta of rclone.
https://beta.rclone.org/
https://rclone.org/downloads/

If you've just got a question or aren't sure if you've found a bug then please use the rclone forum instead of filing an issue.

What is your rclone version (eg output from rclone -V)

rclone v1.38-224-g829dd1ad

Which OS you are using and how many bits (eg Windows 7, 64 bit)

Linux alarmpi 4.9.60-1-ARCH #1 SMP Sat Nov 4 02:45:06 UTC 2017 armv7l GNU/Linux

Which cloud storage system are you using? (eg Google Drive)

B2 Backblaze

The command you were trying to run (eg rclone copy /tmp remote:tmp)
rclone sync -v --b2-hard-delete --checksum /home/atomi/snapshots b2:atomi/snapshots

A log from the command with the -vv flag (eg output from rclone -vv copy /tmp remote:tmp)

example log
rclone[23373]: 2017/12/10 20:08:49 ERROR : snapshots/aff0b718fb9fa0f6387def95c50e5b544caf5a2009939ab451b1d4bbc1cd6791: Couldn't delete: failed to delete "snapshots/aff0b718fb9fa0f6387def95c50e5b544caf5a2009939ab451b1d4bbc1cd6791": File not present: snapshots/aff0b718fb9fa0f6387def95c50e5b544caf5a2009939ab451b1d4bbc1cd6791 4_z9ceb62c61c604b3d6a080c10_f11003750eb26cca6_d20171211_m040416_c001_v0001097_t0057 (400 file_not_present)

Does it do that every time?
Can you replicate it on a small part of the sync?
Can you attach a log with -vv --dump responses of the problem happening please? You'll need to remove authorizationToken from the first response.  If worried about posting publically, then email to me at<EMAIL_ADDRESS>please with subject line "issue 1905" - thanks!

Does it do that every time?

Yes!
Can you replicate it on a small part of the sync?
https://gist.github.com/atomi/da795943c78e08837f4c021af7bfce5a

Thanks for sending that.  It looks like that there are some files rclone needs to delete.  b2 sends them in the listing, rclone sees that they don't exist in the source and tries to delete them but fails with that error.
eg
    {
      "action": "upload",
      "contentLength": 301440,
      "contentSha1": "c021f22cc8e7eccb995d7e2483cb0689ce91547a",
      "contentType": "application/octet-stream",
      "fileId": "4_z9ceb62c61c604b3d6a080c10_f1033bc5ec2411c76_d20171211_m161252_c001_v0001043_t0032",
      "fileInfo": {
        "src_last_modified_millis": "1513008745064"
      },
      "fileName": "snapshots/restic/data/cc/ccc429a08ab1bdb91eed3cdb6ea0d8d05e1d1df0f42a7df0c4e4d1d6272dde4b",
      "size": 301440,
      "uploadTimestamp":<PHONE_NUMBER>000
    }

and
{
  "code": "file_not_present",
  "message": "File not present: data/cc/ccc429a08ab1bdb91eed3cdb6ea0d8d05e1d1df0f42a7df0c4e4d1d6272dde4b 4_z9ceb62c61c604b3d6a080c10_f1033bc5ec2411c76_d20171211_m161252_c001_v0001043_t0032",
  "status": 400
}

Unfortunately I don't see what rclone is sending with that dump, so if you could first make run the sync to make sure it transfers any files, then run it again with -vv --dump bodies --retries 1 this will log all the traffic.  Since the sync is clean there won't be any of your file data in the dump.  Adding --retries 1 will mean it doesn't retry.
Could you also try downloading one of those files rclone can't delete using rclone and the web interface?
Also can you try deleting one of those files using the web interface and see if it gives the same error?
So either rclone is sending something strange to b2, or b2 has reported files that can't be deleted - we'll find out shortly!

I just noticed that in the response the file name is wrong, so likely rclone is sending b2 the wrong file name.  I didn't think that mattered since rclone supplies the ID, but maybe it does...
Hold on sending me stuff, I'm just going to try something...

Yeah fwiw, downloading and deleting the file works fine in the B2 web interface although it translates forward slashes as %2F which I assume is normal i.e snapshots%2Frestic%2Fsnapshots%2Fa8d3a4942453c449165066b138f3a6966602a0dee0f3f1175710c3641bf6e7ed as the file name.
I think that perhaps, adding a nested directory to the bucket could be the cause i.e b2:alarmpi/snapshots/restic  but the "filename" in the logs for hard deletes is missing /snapshots/restic/ so maybe it's attempting a relative location without the prefix.

Hmm, in my testing b2 seems to be ignoring the file name when deleting by id.
Can you try this https://pub.rclone.org/v1.38-222-gff5cb663-issue-1905β/ (uploaded in 15 mins or so) which has a fix to make rclone always send the correct file name and see if it makes any difference?

Yeah fwiw, downloading and deleting the file works fine in the B2 web interface although it translates forward slashes as %2F which I assume is normal i.e snapshots%2Frestic%2Fsnapshots%2Fa8d3a4942453c449165066b138f3a6966602a0dee0f3f1175710c3641bf6e7ed as the file name.

Thanks for trying that and yes the / transation is normal

I think that perhaps adding a nested directory to the bucket could be the cause i.e b2:foote22/snapshots/restic but the "filename" in the logs for hard deletes is missing /snapshots/restic/ so maybe it's attempting a relative location without the prefix.

That is my theory too, but we'll see!

Thanks for testing. I've merged that to master now - it will be in the latest beta from now on, the first of which is
https://beta.rclone.org/v1.38-226-g66558213/ (uploaded in 15-30 mins)

Thanks again for all the solid code you've written! I really do appreciate it.


inconsistent variable names

Hi,
i'm trying to use DLC2Kinematics following the instructions. but getting errors about joints_dict not being defined.
it took me 2 hours, but i figured out that in this section:
"
Computations in joint coordinates
To compute joint angles, we first create a dictionary where keys are the joint angles and the corresponding values are the set of bodyparts:
joint_dict= {}
joints_dict['R-Elbow']  = ['R_shoulder', 'Right_elbow', 'Right_wrist']
and compute the joint angles with
joint_angles = dlc2kinematics.compute_joint_angles(df,joints_dict)
"
you define:
joint_dict= {}      so           "joint_dict"
but then try to asign into "joints_dict":
joints_dict['R-Elbow']  = ['R_shoulder', 'Right_elbow', 'Right_wrist']
best,
Shahaf

closed with https://github.com/AdaptiveMotorControlLab/DLC2Kinematics/commit/02db6880de34436b7d1650ccd8aedf595f22bd74


Meaningless Code？

In com.netflix.eureka.registry.PeerAwareInstanceRegistryImpl method syncUp, why need call method isRegisterable? and how to understand logic of method isRegisterable(Instance instance)? the method code as follow:
/**
     * Checks if an instance is registerable in this region. Instances from other regions are rejected.
     *
     * @param instanceInfo  th instance info information of the instance
     * @return true, if it can be registered in this server, false otherwise.
     */
    public boolean isRegisterable(InstanceInfo instanceInfo) {
        DataCenterInfo datacenterInfo = instanceInfo.getDataCenterInfo();
        String serverRegion = clientConfig.getRegion();
        if (AmazonInfo.class.isInstance(datacenterInfo)) {
            AmazonInfo info = AmazonInfo.class.cast(instanceInfo.getDataCenterInfo());
            String availabilityZone = info.get(MetaDataKey.availabilityZone);
            // Can be null for dev environments in non-AWS data center
            if (availabilityZone == null && US_EAST_1.equalsIgnoreCase(serverRegion)) {
                return true;
            } else if ((availabilityZone != null) && (availabilityZone.contains(serverRegion))) {
                // If in the same region as server, then consider it registerable
                return true;
            }
        }
        return true; // Everything non-amazon is registrable.
    }

这是来自QQ邮箱的假期自动回复邮件。你好，我最近正在休假中，无法亲自回复你的邮件。我将在假期结束后，尽快给你回复。

this code is used to determine whether a service instance should be registered with the Eureka service registry based on its data center information and the client's configuration. It enforces conditions for registration in AWS environments but defaults to allowing registration in other environments. The practical significance of this code depends on the specific deployment and use cases, and it may appear to have limited usefulness in some situations.


Update devIPv6 tip to main

Description
Add following changes :
Adding CMakeList.txt in posix GCC demos (#1020)
Revert "Remove coroutines (#874)" (#1019)
Create GitHub Workflows that execute the p3 FreeRTOS Kernel Demos (#1018
risc-v: Fix build flags and linker scripts (#906)
Updating kernel release script to update source files with tags and r…
Update FreeRTOS-Kernel Source submodule
Add unit tests for kernel PR #679 (#1015)
Fix file path in the project file (#1013)
Update FreeRTOSconfig.h for M2351 (#1005)
Added Posix Demo run in github action (#997)
Set configMAX_SYSCALL_INTERRUPT_PRIORITY to 4 for QEMU demos (#986)
Update FreeRTOSConfig.h (#1008) - [Already Present]
Add IAR projects for LPC55S69 (#1007)
Test Steps

Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks

/bot run checks


Support for building images base on different architectures at the same Board

In GitLab by @sokril on Jun 8, 2020, 24:44
example： k8s cluster + kubeedge.
K8s cluster is x86 arch and kubeedge's edge node is arm, Board should support build x86 images and arm images at the some time.

In GitLab by @sokril on Jun 8, 2020, 24:45
closed

In GitLab by @sokril on Jun 8, 2020, 24:45
reopened

In GitLab by @sokril on Jun 8, 2020, 24:45
changed the description

In GitLab by @luotian-github on Jun 8, 2020, 08:56
assigned to @wknet123 and unassigned @luotian-github


Add unbound

post_install is not an appropriate place for starting services.

https is available for all URLs

@lukesampson the post_install just replace a string on an extracted file.
@r15ch13 will update it.

Sorry, I meant pre_install — what's that doing?

it is just proxing unbound command without parameter to provide right default configuration,
user should run unbound -c ~/scoop/apps/unbound/current/service.conf to run unbound
but with the pre_install i created, just  unbound is enough. also it runs in bg as it should.
is that too much?

Thanks for explaining. I'm a bit slow — I thought it was running that command rather than writing it to a file. It is a more code than is ideal, but it's okay in the absence of a scoop service command.


Counterwork seemingly unclickable repo button labels

As title, the change would counter-work the effect from #14926 that links seem unclickable (especially in the default gitea theme), while maintaining some sort of visual harmony. General design language was stolen from GitHub (d'oh).
The "after" screenshots are from my own branch, not master, but I think you get what I mean...
Before:

After:

Thanks, looks nice! But why is the watch counter in blue? 🤔

Thanks, looks nice! But why is the watch counter in blue? thinking

That's the new hover effect. Should have put a mouse cursor on it.
I've removed the original hover effect completely (!important). The label part no longer behaves like a basic label (button-like, pressable), but a hard surface with a normal gray link (i. e.: looks like text, could be darker for contrast, and turns into primary color when hovering over it).
To be fair, GitHub ditched the global opacity approach and just lightens text color for this, which is especially appealing considering accessibility - but this PR at least maintains good consistency (and we got official dark theme much earlier than they did, hehe).

make lgtm work


dialog: focus lock

fixes: #260

[x] new <FocusLock />
[x] <Dialog /> uses focus lock

TODO:

[x] rename <FocusLock /> to <FocusManager />
[x] add trapped prop to activate/deactivate functionality
[x] remove as prop
[x] with no focusable children - make container focusable
[x] update <Dialog /> to always use the <FocusManager />
[x] <Dialog /> is trapped if has onClose or modal props

@jaketrent this is ready for you to give it a second pass

Added one relevant story.  Please have a look.  Overall, looks great to me, plz merge.

Now that I've committed, I'm not sure I have an approve/review button any more.

that's weird


chore(master): Release 0.40.1

:robot: I have created a release beep boop
aztec-package: 0.40.1
0.40.1 (2024-05-14)
Miscellaneous

aztec-package: Synchronize aztec-packages versions

barretenberg.js: 0.40.1
0.40.1 (2024-05-14)
Miscellaneous

barretenberg.js: Synchronize aztec-packages versions

aztec-packages: 0.40.1
0.40.1 (2024-05-14)
Bug Fixes

avm-simulator: Unencrypted logs hash mismatch in kernel (#6399) (35645e4)
avm-transpiler: Fix cast to u1 (#6402) (16ab1f7)
unconstrained: Add missing debugLog oracle (#6397) (87eb8ab)

barretenberg: 0.40.1
0.40.1 (2024-05-14)
Miscellaneous

barretenberg: Synchronize aztec-packages versions

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/AztecProtocol/aztec-packages/releases/tag/aztec-packages-v0.40.1 :sunflower:

:robot: Release is at https://github.com/AztecProtocol/aztec-packages/releases/tag/aztec-package-v0.40.1 :sunflower:

:robot: Release is at https://github.com/AztecProtocol/aztec-packages/releases/tag/barretenberg.js-v0.40.1 :sunflower:

:robot: Release is at https://github.com/AztecProtocol/aztec-packages/releases/tag/barretenberg-v0.40.1 :sunflower:


Clarify REDC implementation (TODO comments)

Context: P256VERIFY.yul#L240
Description:
In the REDC implementation, there are two TODO comments related to integer overflow. Both overflows can happen and must be properly handled.
The first one is tested e.g. by p256verify_wycheproof_0_should_pass.
The second one is tested e.g. by p256verify_wycheproof_115_should_pass.
Recommendation:
Remove the TODO comments.

Resolved in #200


Consider adding Microsoft.ApplicationInsights.AspNetCore as a dependency in 2.1-based package

In ASP.NET Core 2.1, the default project template does not pull in the Microsoft.ApplicationInsights.AspNetCore so I had to add it in order for the newer 2.1-based profiler package to work.  It is better to specify it as a dependency so end users don't need to include it by themselves.

https://www.nuget.org/packages/Microsoft.ApplicationInsights.Profiler.AspNetCore/1.1.3-beta1


Keyboard Toolbar not displaying

Hey I'm trying to ad a toolbar to the RichText Editor in SwiftUI and it doesn't seem to display. Here is what my code looks like:
`    @StateObject var context: RichTextContext = RichTextContext()
@State var text: NSAttributedString = NSAttributedString(string: "type here....")

var body: some View {
    RichTextEditor(text: $text, context: context)
        .toolbar {
            ToolbarItemGroup(placement: .keyboard) {
                RichTextKeyboardToolbar(context: context) {

                } trailingButtons: {

                }

            }
        }
}`

After some more digging this seems to be an issue with the underlying view being a UITextView and not a SwiftUI view.

Hi @willmorris44
Yes, the keyboard placement unfortunately doesn't work due to the UIKit integration. You can place this toolbar anywhere, but it has to handle presentation state manually.


feat(interactive): Add field id and name in graph schema

In flex schema definition, we have graph id and graph name. But in schema implementation we have no such field.
More other, having field id in schema.h will make it easy for us to get the current running graph's id.

Closed since not going to be adapted.


Scrollbar in Lepiter cahnging size and jumping around when scrolling

When using mouse to scroll a page with dynamic content in Lepiter, scrollbar handle changes size and jumps around, which makes scrolling harder.
I understand why it is the case when scrolling first time - size of some elements and thus size of the page is not known beforehand.
But it keeps behaving like that even after the page is scrolled to the end and fully loaded.

That is the current behavior of the infinite element used by the tree of lepiter.


Support tls client certificates and kerberos auth

It would be great if milena supported user authentication, introduced in kafka 0.9.

I agree! I'm working on the new 0.9-style groups right now. Either auth or compression are next. This is a free-time only project right now, so sorry new features take so long. :disappointed:

I don't want to demand anything. Great that you are doing the work. For my part that issue is mostly a keep-track-of-it.
BTW: I would appreciate if you could open issues (maybe with a "wip" label) when you start working on something. Even sketching the desired API (if you thought so far) would be a great help for others so they develop patches that are in line with your vision. (Of course I would love to see the progress in a "wip: 09-consumers" branch directly as well ;)))


IF Website - clearer CTA to docs

Hi @jmcook1186
Would it be possible to make a big. "Documentation" CTA button on the IF website?
I couldn't find it myself until I spotted the small "Docs" button in the top left navigation.

HI @adamj89 this is something you'll need to ask @osamajandali about.

It need to be more bolded. and I think it is better to have a cta after the second sections
@adamj89 please added to the board if this something you want me to do

closing as not needed


Added unjail upon command executor leave

Added unjail upon command executor leave
Whenever the executor of the :jail command leave the victims of his jail that are in-game or not get unjailed automatically.
Proof of functionality: (Thanks to @l0wk3yomega) (his discord)
https://github.com/Epix-Incorporated/Adonis/assets/122803145/009790b4-4651-4ef0-9ab4-315ca32b5ee2

dont merge this is a terrible idea

at least make it a setting pls

at least make it a setting pls

Okay

you need to add this to defaultsettings as well
you need to add this setting to the settings.descs and settings.order

Do we really need another setting for this?

dont merge this is a terrible idea

It's quite cool as an idea but I don't think it should be on by default though

Maybe a plugin then

Maybe a plugin then

It's fine as a feature


(feat): Transaction feed paginated

Description
When scrolling to the bottom of the transaction feed, the app requests more transactions to the blockchain-api.
Tested
Locally with an emulator, hitting local blockchain-api with these changes.
How others should test
Until blockchain-api PR is merged, this can't be tested without setting up the blockchain-api service.
Once the blockchain-api PR is merged, you should be able to scroll to the bottom of the transactions feed and new transactions will be shown (if there are any).
Related issues

Fixes #1808

Backwards compatibility
This app is not compatible with the current version of blockchain-api. We have to deploy blockchain-api service before this is released in order to avoid any issue.
Screen Recording
Scrolling until no more transactions are available:
https://user-images.githubusercontent.com/12504748/163847783-3fb529d0-aa69-4ce1-b589-43a2edeb740d.mov
Network issues at the beginning:
https://user-images.githubusercontent.com/12504748/163847977-4568bd5e-3924-4507-a4ec-95480c2e5bdd.mov
Network issues when some pages were already fetched:
https://user-images.githubusercontent.com/12504748/163848124-3a8d0848-1c6b-4c25-b935-1915a140de81.mov

@kathaypacific @gnardini @jeanregisser
Thank you for the reviews!!
The PR is ready to be reviewed again. :)

Hey @MuckT, We have verified above task on Latest Android internal release build , Test flight release build V1.31.0 and updated status in test rails.
Test rail links:
iOS15: https://valoraapp.testrail.io/index.php?/tests/view/30606
Android 9: https://valoraapp.testrail.io/index.php?/tests/view/30637
iOS 14: https://valoraapp.testrail.io/index.php?/tests/view/30668
Devices: iPhone 13 mini(15.1.1), Redmi Note 5 pro (9.0), iPhone 12(14.7.1)
CC: @dievazqu
Thanks..!!

Hey @MuckT , We have verified above task on Latest Android internal release build and updated status in test rails.
Observation :

user is able to see the alert incase if the page is not loaded properly due to internet connectivity issues
User is able to scroll till the last page of transaction loads and activities are seen.

Test rail links:
Android 11: https://valoraapp.testrail.io/index.php?/tests/view/30699
Devices: Google Pixel 2XL(11.0)
CC: @dievazqu
Thanks..!!


Enhancement Add setter and getter for certain classes

so that LeftAndMain no longer updates config during init
Please check https://github.com/silverstripe/silverstripe-admin/issues/264 for more details

Looks good, do you think it'd be worth updating i18n to have this behaviour as well? @tractorcow

@flamerohr i18n already has a static var to override the config; It's probably ok as it is for now.

@flamerohr updated, but I've restored the non-static $rewriteHashlinks variable in SSViewer.

Also, dontRewriteHashlinks() no longer irrepairably modifies global state :)

Ok, cause of the issue was test state no longer being resettable solely via config nest / unnest; I've resolved this by adding a new SSViewerTestState to reset the new vars. :)


Arbitrary limitation that images dir must be under /home

We currently have an arbitrary limitation that forces the default images dir to be a subdir to /home --
cpp/piscsi/piscsi_image.cpp
string PiscsiImage::SetDefaultFolder(const string& f)
{
    if (f.empty()) {
        return "Can't set default image folder: Missing folder name";
    }

    string folder = f;

    // If a relative path is specified, the path is assumed to be relative to the user's home directory
    if (folder[0] != '/') {
        folder = GetHomeDir() + "/" + folder;
    }
    else {
        if (folder.find("/home/") != 0) {
            return "Default image folder must be located in '/home/'";
        }
    }

I think it's time to remove this limitation and allow users to define the default images dir anywhere on the file system. F.e. they may want to mount external storage media under /mnt and use that for images storage. (Although the mount point could certainly be under /home but that is not a good practice IMHO)

Discarding after discussion in PR


Dapr status for self-hosted

Describe the proposal
Add a -s, --self-hosted flag to the dapr status command.
This will allow checking the status of a self-hosted Dapr instance ,which is not currently supported.
Release Note

RELEASE NOTE: ADD -s, --self-hosted flag to the dapr status command for checking the status of a self-hosted instance.

I'd love to work on this if the proposal is accepted.

This proposal seems interesting.
dapr status -k is a command for tracking the control plane status in k8s.
Currently not all control plane services might be deployed in the self hosted mode. And there is no straightforward way to track all the control plane services.
/cc @yaron2 what do you think about this proposal ?

@guyzyl Are you interested in contributing a PR towards this?

@mukundansundar I would've loved to, but unfortunately don't have the capacity for it ATM.


Variance explained by grouping in PERMANOVA

Perhaps I'm missing this, or it is statistically inappropriate but:
Is there some sort of AOV table returned from skbio.stats.distance.permanova ? I'm specifically looking for some sort of variance-explained statistic. In the R-version of this test, adonis returns a partial R2.
If there is not, this would be a nice feature to implement.

Currently this is not implemented, but I believe it is not too hard to put in place. From some googling around we are already calculating some of the necessary pieces to return the r^2. I'll see if I can put something in place.

If somebody sees this and still needs the R2, you can calculate it from the values provided in the permanova output like stated on stackoverflow. For instance:
p = permanova(D, groups)
r2 = 1 - 1 / (1 + p[4] * p[3] / (p[2] - p[3] - 1))
p["R2"] = r2
print(p)

bumping this up as this will be a super useful feature for skbio's permanova function to be able to truly replace r's adonis

110% agree. @tommyfuu, do you have bandwidth to issue a PR?

110% agree. @tommyfuu, do you have bandwidth to issue a PR?

currently not, but i can potentially try over the summer! another feature that I think can be beneficial is extracting the PERMANOVA feature coefficient (importance) based on dissimilarity. can potentially work on that as well when i have time

That would be amazing!


add a texture uv modifier node (only supports the variables currently…

… available in three.js textures)

lgtm


Testing attributes type

This PR is not for merging, checking, android-sdk should run compatability with java master.

build

build

build

build

build

build


Users stop being matched after a while

For the past year or two we've gotten occasional reports of people not being included in meetings after a while. Pausing, adding or removing does not affect it, and the users that are ignored appear in the user lists just fine.
I have tried to debug this issue but have had no luck so far, especially as it pops up every few months and I myself haven't been matched in a long time. Any idea what might be going on? I checked the source once or twice but found no obvious errors.

Thanks, I have no clue, but I will try to find the issue as soon as possible.


Sidds features

Added the test files
Added the view files
Added the test folder files
Added the helper files
Added the controller files
Changed the README.md file

STATUS: INVALID CODE REVIEW REQUEST 🛑 🛑
Hi @siddghosh108 and @lily4178993,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
This is a group project so, following gitflow, after merging all your changes to the development branch, you should raise a PR from development to main and then submit that PR. Also, kindly avoid submitting merged PRs.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. And please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


add edgeIdConfig for stage and prod environments

Adding the edgeIdConfig parameters provided by Harshavardhan in the franklin-stock channel. This should enable analytics to begin aggregating data in the right place.
Test URLs:

Before: https://main--stock--adobecom.hlx.page/pages/artisthub/
After: https://new-edge-config--stock--webistry-development.hlx.page/pages/artisthub/

Closed for now as things are being tested in the staging environment for analytics.


Texture atlas

I finally came around to implementing the texture atlas for images. I'm using guillotiere to manage the atlas space. In the current implementation the atlas only grows if necessary. It never shrinks.
I also wrote a small build script that compiles shaders at compile time using shaderc. Manually compiling shaders during development was super annoying.
Let me know what you think.

Just rebased on current master. Go ahead and rename or move anything you like.

If you're happy with it, then I'm happy with it as well.

I have changed the renderer so it doesn't create a useless image::Pipeline (and therefore, an empty atlas) when both image and svg flags are disabled.
I think this ship is ready to sail. Thank you for all the work here! :tada:


[Cancelled] feat: Pydantic models of function signature and parameters

Use Pydantic models for data-centric handling of funcdef signature params

Cancelled to test Linear integration (backlinks)


finish Cantillon support

[x]  review worker api PR #16
[ ] throughout code use isEncointer, isEncointerCantillon isEncointerGesell flags to change behaviour and use worker api

split into remaining tasks. will close this


Whole-form Zod validation messages

I am using Zod refine to create a validation that takes in consideration multiple fields.
Individual fields are validated using <FormMessage /> tag. How can I display whole-form validations?
My custom validation is like that.
export const changePasswordSchema = z
  .object({
    password: PasswordSchema,
    passwordRepeat: PasswordSchema,
  })
  .refine(
    (data) =>
      data.password == data.passwordRepeat,
    {
      message: "Passwords do not match.",
    }
  )

Found out a workaround. Replacing .refine with .superRefine, I could set in which field(s) should I display the custom error.
.superRefine((data, ctx) => {
    if (data.chatwootUserTempPassword !== data.chatwootUserTempPasswordRepeat) {
      ctx.addIssue({
        code: z.ZodIssueCode.custom,
        message: "Passwords do not match.",
        path: ["chatwootUserTempPasswordRepeat"],
      });
    }
  })

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Add Service Binding Support

The PR adds support for Service Bindings

@zebp For the name, I think now (in the dashboard) the previous Worker Script is now totally renamed to Service, So I think a Service Binding implies the Remote Service Binding, which is difference from the current local binding.
I don't think my name is right, but I don't think just name itservice is good either.

For test, I think we must use wrangler2, and it seems that the wrangler.toml should be changed too, maybe like this (remove type and change build.upload to main):
name = "testing-rust-worker"
workers_dev = true
compatibility_date = "2021-11-27" # required
main = "build/worker/shim.js"

kv_namespaces = [
    { binding = "SOME_NAMESPACE", id = "", preview_id = "" },
    { binding = "FILE_SIZES", id = "", preview_id = "" },
]

vars = { SOME_VARIABLE = "some value" }

[durable_objects]
bindings = [{ name = "COUNTER", class_name = "Counter" }]

[build]
command = "worker-build --release"

And after the migration to wrangler2 is done, I think I can add a test easily.
BTW, the current support for service binding in wrangler2 is merged (in https://github.com/cloudflare/wrangler2/pull/906) but isn't release now, so maybe we need to wait their release?

miniflare 2.6.0 allegedly has bineings support: https://github.com/cloudflare/miniflare/issues/301

Friendly ping @zebp @ImSingee
Any chance making some progress here anytime soon?
If not, any objections if I pick it put and push to an acceptable form?

Closing in favor of #243


[Installer] Add selection of IIS website where Myrtille will be installed

Myrtille is now installed on the "Default Web Site".
In the installer, you can choose from a list of all IIS sites.
If you select a non-Default site, Myrtille should work on this site with its SSL certificate.
Currently, you have to manually implement this behavior through Internet Information Services (IIS) Manager.
Myrtille is supposed to be available at https://subdomain.example.com/Myrtille, but not at https://localhost/Myrtille

The Myrtille installer was originally designed for automatic installation, so on the default website (knowing that 90% of people use the default site), but indeed it might be useful to be able to choose the website


Profile resets each time app is opened

I am having an issue where the profile resets each time the app is opened meaning all decoders that was enabled are reset to the default and I have to go into datawedge and manually enable them again.
is there a way to enable the decoders from dart? or a way to stop a new profile being created if one already exists?

The devices are currently running Android 11 and Datawedge 11.3.28.

Thank you for your input. Currently we are creating a new profile every time FlutterDatawedge is instantiated, this seems to overwrite the old profile. We will implement that if a profile with the given name already exists, it will not be overwritten.

Is there any way to stop this from happening or can I send an intent to enable the decoder that keeps getting disabled?

The following branch: skip_profile_creation_option contains changes which I think will fix the issue.
flutter_datawedge won't create a profile and configure it during initialisation, the should fix the issue of the profile resetting when the app is opened.
I migrated that functionality to the function createDefaultProfile.
Additionally the active profile can be fetched using requestActiveProfile and waitForActiveProfile as well as a list of all installed profiles with requestProfiles and waitForProfiles.
Please have a look if this changes will fix the issue, then we will release it on pub.dev.
flutter_datawedge:
    git:
      url: git@github.com:circus-kitchens/flutter_datawedge.git
      ref: skip_profile_creation_option

@james-rich We just released the Branch to pub.dev as a PreRelease. https://pub.dev/packages/flutter_datawedge/versions/3.0.0-dev.1


rbvmomi2 version 3.4.0 is missing cluster.summaryEx object

Tried rbvmomi2 version 3.4.0 and still the cluster object is missing summaryEx object.
Need to get / set vCLS placement property which is part of summaryEx.
summary	ComputeResourceSummary	[summary](https://aer1-cvcsa80.asm.delllabs.net/mob/?moid=domain-c1033&doPath=summary)
**summaryEx	ClusterComputeResourceSummary**	[summaryEx](https://aer1-cvcsa80.asm.delllabs.net/mob/?moid=domain-c1033&doPath=summaryEx)

@agrare

Discussion from gitter :point_up: September 29, 2022 10:53 AM

Hey @sushilrai that is a weird one, I pulled the latest WSDL VMware vSphere Management SDK 7.0U3f
and there is no properly summaryEx in the wsdl files, it exists only in the docs to say it is deprecated yet it was added in vSphere API <IP_ADDRESS>
the recommendation is to use computeResource.summary instead of cluserComputeResource.summaryEx per the documentation

Deprecated. Do not use this property. The cluster summary. The same information could be obtained via Summary

tried python version to pull the information of the vCLS datastore using below snippet
 if hasattr(cluster.configurationEx, 'systemVMsConfig'):        vCLS_config = cluster.configurationEx.systemVMsConfig     else:       # return False, self.allowedDatastores_names, None        return False, [], None     changed = False

This was surprising to me because I tested this property for you specifically in https://github.com/ManageIQ/rbvmomi2/pull/16
What I found was after updating the vmodl.db with the sms.wsdl for the vSphere SMS endpoint, it cleared the properties which were added.

This has been fixed in v3.4.1


Add unfold

Hello @tsnorri, thanks for your contribution!
Calculating Fibonacci numbers is a quite abstract example. Especially since most calculations in web apps are not done in templates, but rather in controller.
Could you please provide a more down-to-earth example of use?

I just noticed your question, sorry about the rather long delay. A more down-to-earth use case would be traversing a page hierarchy from a leaf towards the root, i.e. by using parent() or similar. This would be useful in e.g. creating a breadcrumb list. I don’t have sample code right now but I can try to produce something.

Something like this:
{% set breadcrumbs = page|unfold(=> (_ is null or _.root) ? null : [_, _.parent])|reverse %}

Why do you need the _.root check? Isn’t your root’s parent null?

Why do you need the _.root check? Isn’t your root’s parent null?

Apparently in Grav (at least version 1.4.5) the root isn’t actually a page but a directory. The directory may contain other pages in addition to the designated home page.


[prim_edn_req] Fix protocol violation, enable use of prim_sync_reqack_data

This PR contains two commits:

The first commit fixes a protocol violation. Without this commit, the req_i signal is directly forwarded to the handshake synchronizer and de-asserted 1 cycle after receiving the last ACK. This means the handshake synchronizer sees a REQ without having sufficient time to ACK. The fix implemented in this PR is to not forward the REQ whenever the packer FIFO has valid data output.
The second commit instantiates the prim_sync_reqack_data primitive that also feeds the data through the synchronizer prim. This will allow to waive all CDC violations in one place.

LGTM from my side, but would be good to get an ack from @mwbranstad as well.

LGTM from my side, but would be good to get an ack from @mwbranstad as well.

Thanks for the feedback guys.
Regarding Mark's point: yes it means the fips value is not forwarded to the actual consumer. I think that's bad and that the decision to use it or not should be to the consumer. I'll either do another commit to this PR or do a follow-up PR when I get the chance.

Thanks for the feedback guys.
Regarding Mark's point: yes it means the fips value is not forwarded to the actual consumer. I think that's bad and that the decision to use it or not should be to the consumer. I'll either do another commit to this PR or do a follow-up PR when I get the chance.

Update, I've appended another commit to also forward the fips signal brought up by @mwbranstad.

Update, I've appended another commit to also forward the fips signal brought up by @mwbranstad.

Needed to rebase, now also CI should pass.

Needed to rebase, now also CI should pass.

@vogelpi thanks for added the fips signal. LGTM.

@vogelpi thanks for added the fips signal. LGTM.


v1.20.0 does not start as a windows service

Description
After upgrade from v1.19.4 to v1.20.0 gitea can not start as a windows service. It gives 1053 (Service didn’t respond in a timely fashion) almost immediately after start attempt. No records are written to log to help debug this issue. Service starts successfully after downgrading back to v1.19.4.
Gitea Version
1.20.0
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
Windows Server 2019
How are you running Gitea?
Self hosted as a windows service.
Database
None

Please take a look at the error log. It's possbile your Gitea instance has some deprecated app.ini configurations.

@lunny, thanks for hinting. As I said gitea did not create any log files. So I started is manually and fixed removed fallbacks found in console. After that service has started successfuly.


Ability to set Fees and "coins' when sending a transaction

Context
Right now, the user can not chose how much fee/coins he put on a smart contract call transaction.

@gregLibert must-have for next (post-ICO) release. Confirm please.
@peterjah can you specify the Action plan?

Is it for the front ? the API ?


Update quickstart.md

What/Why/How?
Substantive edit of this page. I used Open Weather Map’s Current Weather Data API for testing as I know we need to stop using petstore. This is the API that is used throughout Tom Johnson’s API docs course. I tested all commands and everything worked fine.
Reference
Testing
All done.
Screenshots (optional)
Check yourself

[X] Code is linted
[ ] Tested with redoc/reference-docs/workflows
[X] All new/updated code is covered with tests

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

Great work! :clap: :partying_face: Looks good to merge :+1:


NO-ISSUE: Implement Go tools pattern

Implement the Go tools pattern to manage versions and dependencies of the tools we use for code generation and testing via tools/go.mod.

Overall the usage of the pattern LGTM

👍  on my side too, this will avoid already some issues with the grpc generation we had and versions skews CI/devenv

+100 :)


HBAsets

Thanks for submitting your project to be featured in the Open Leadership Zone at MozFest 2017 :sparkles:
This is open to any project in Mozilla Open Leaders round 4 -- even if you're not coming to MozFest. This process ensures that you're following open practices and will help set you up for a sustained contributor community afterwards. If you have any questions, feel free to reach out to @acabunoc or ask questions in our chat room.
Open Project Checklist :clipboard:
Your project must complete the following to be a featured on Mozilla Pulse and the Open Leadership Zone at MozFest 2017. As you complete each exercise, check off the box and comment with a link to your completed resource. This template repository is here to help you if you get stuck!

[x] Complete Open Leadership 101
[x] Provide a GitHub repository for work and discussion on your project in a comment

new to GitHub?  Here's a step-by-step guide on using the #mozfest template

[x] Create file: README.md in your project repository. This file should help newcomers understand what your project is, why it's important, and kinds of help you're looking for.
[x] Create file: LICENSE to give your project an open license, allowing for sharing and remixing.
[x] Create file: CONTRIBUTING.md so others know how they can contribute. If you'd like, you can remix this template
[x] Create file: CODE_OF_CONDUCT.md. This can be the Mozilla Community Participation Guidelines or a code of conduct of your choice
[x] Turn on your Issue Tracker and create issues to describe each task that you need help with and how a contributor can get started on that task. If you're participating in Hacktoberfest, Create a label called hacktoberfest and apply it to your issues.

Ready to be featured :tada:

[x] Fill out this form to submit your project to Mozilla Pulse. Add the tag open leadership zone.
[x] Leave a comment on this issue with the text This is ready for the Open Leadership Zone. @acabunoc will review this issue. If accepted, your project will be approved on Mozilla Pulse and featured in the Open Leadership Zone.

If you get stuck at any point, feel free to look at the project templates or reach out to your mentor or @acabunoc. We're here to help you through this process.

Hey @derekhoward! Gentle reminder that there's only one week to MozFest :) I'd love to see you project featured in the Open Leadership Zone!

Hey @acabunoc. Thanks for the reminder!
This is ready for the open leadership zone now 🎉

Thanks @derekhoward! You're in the Open Leadership Zone now 🎉
Pulse: https://www.mozillapulse.org/entry/479


colord-kde: fix missing buildInput

Added a buildInput that is not strictly required for the package to compile, but without which the package will not actually work at all.
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 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.

The Colour Management kcm still complains about QML errors, namely that "org.kde.kirigamiaddons.treeview" is not installed. Does the package need to be added to the QML import path explicitly or something?


Pub/Sub publisher retrySettings: runtime errors that should be compile time

Publisher built with these retry settings fails at runtime, with generic "illegal argument exception."
The error does point to specific failed pre-conditions, which have to do with minTotalTimeout -- which I had not set here.

        RetrySettings retrySetting = RetrySettings.newBuilder()
                .setInitialRpcTimeout(Duration.ofMillis(500))
                .setMaxAttempts(10)
                .setJittered(Boolean.TRUE)
                .setMaxRpcTimeout(Duration.ofMillis(500))
                .build();

Moved to feature wiki.


toil-lib cleaning and merging

(putting this toil-lib issue here just to keep things in one place)
toil-lib needs to get merged back up with bd2kgenomics/toil-lib so people don't need to a special version to run toil-vg.
on a related note:  copying the file in read_input_file()
https://github.com/cmarkello/toil-lib/blob/master/src/toil_lib/toillib.py#L485
https://github.com/cmarkello/toil-lib/commit/5f5efa882d926bbaafad8529c56a70d54b51aa2e
is a bit dangerous.  better to use atomic copy as is done in write_output_file
https://github.com/cmarkello/toil-lib/blob/master/src/toil_lib/toillib.py#L573-L585
i'd actually done this independently so you can just copy over or cherry-pick in this change:
https://github.com/glennhickey/toil-lib/commit/61840bc1ad2c7e3dc6c698eccf835533bf30cca9
not normally so pedantic but toil's actually a place where these types of things can come back to haunt us.

The problem with merging to bd2kgenomics/toil-lib is that it takes quite a bit of time to coordinate with Hannes to merge with master, but I can do that. The other issue with merging, which was the main problem, was that the docker_call command in programs.py for integrating pipe commands within a single docker image was not implemented exactly the way Hannes desired. He had some ideas, but that sort of went into the backlog while he was working on toil things that held a higher priority. The docker_call thing is the main issue why I couldn't merge with bd2kgenomics/toil-lib. I can try to work with him on figuring out how he wants docker_call written, but it'll likely take a few days time.

no problem.  it's nothing urgent, just needs doing eventually.
On Mon, Dec 5, 2016 at 3:16 PM, Charles Markello<EMAIL_ADDRESS>wrote:

The problem with merging to bd2kgenomics/toil-lib is that it takes quite a
bit of time to coordinate with Hannes to merge with master, but I can do
that. The other issue with merging, which was the main problem, was that
the docker_call command in programs.py for integrating pipe commands
within a single docker image was not implemented exactly the way Hannes
desired. He had some ideas, but that sort of went into the backlog while he
was working on toil things that held a higher priority. The docker_call
thing is the main issue why I couldn't merge with bd2kgenomics/toil-lib. I
can try to work with him on figuring out how he wants docker_call
written, but it'll likely take a few days time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BD2KGenomics/toil-vg/issues/54#issuecomment-264964341,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2_7siRGINXcG1OIa0SN45KAAIQZUJaks5rFHEAgaJpZM4LEm_2
.

Closed by #111


allow np.integer in validation of param.Integer

param                     1.10.1             pyhd3eb1b0_0
This is a very small issue,  but I bumped into it.
The validate method of param.Integer currently does this:
if not isinstance(val,int):
    raise ValueError("Parameter '%s' must be an integer."%self.name)

But for example when you get integers from a random number generator, very often they are of some numpy int type, which do not inherit from built-in int. However, they are just fine integers for param.Integer purposes I would say.
So maybe the validation can be changed to:
if not isinstance(val, (int, np.integer)):
    raise ValueError("Parameter '%s' must be an integer."%self.name)

here is a relevant thread on stack overflow about numpy int checking.

Thanks for the suggestion!
The proposed change looks good to me (will some additional logic needed to ensure np is an optional dependency).

Right; we can't assume Numpy is installed, as it's not a dependency.  I think we have a couple of options here, both of which , require special logic to avoid a dependency on np:

Simply allow Numpy ints as suggested here
Coerce any provided Numpy int into a supported int type.

For option 1, I'd be concerned that downstream code already written for param.Integer could break if someone provides a Numpy integer to Parameterized code not expecting it. At a minimum, if that parameter value is ever passed to isinstance(val,int) in Parameterized code, it would now fail.
I see Param as offering a contract to code authors, where Param takes care of ensuring that it only lets through a particular universe of values, and then the code using that parameter need only ever consider the items in that universe, never needing to check or test other Python objects that a user might supply. Here, I don't know how big an issue this might be; I personally have found Numpy ints to behave like regular ints in general, but there will definitely be cases where they don't, and the question is whether the benefit of accepting such values outweighs those potential problems.
Option 2 seems safer to me, i.e. to convert the provided int-compatible value into an int.  That way downstream code sees only a regular int, and users can still pass in Numpy ints as desired.  Such conversions can cause problems for mutable data structures, but I don't think they'd be an issue for an int, right?  So I think I'd favor option 2, but am open to arguments in support of option 1 or some other option 3.

I think the main difference is that numpy ints are fixed length, like int64 , while python built-in int can become arbitrarily large. One situation I can think of  where this difference matters is large prime numbers.
Numpy ints are mainly introduced for speed I believe, so type casting to python int should be safe. A param.Integer is just one int, so the whole speed thing is negligible.  Option 2 should be fine.
Now of course, if somebody specifically chose an numpy unsigned int, you don't want negative results to be possible all of a sudden. However, the bounds option of param seems like the right solution for those kind of things.

I'd like to see this in Param, but it would require a PR to convert a numpy into to Python when it is supplied, implemented in a way that does not require importing Numpy, and with tests.

Duplicate of https://github.com/holoviz/param/issues/205, except that the conclusion reached here (convert to int) is different than in that previous issue (accept more types and don't convert).
While I get the intent with converting to make the life of downstream user easier, I'm a little uncomfortable by the fact that:

Param gets into the business of converting/casting user provided values, I think that would be the first time, and that it can also lead to bugs
It isn't consistent with how Date (and DateRange and maybe more) work, as it accepts pure Python and Numpy datetime objects, and does not cast them to pure Python datetime objects.

I personally have found Numpy ints to behave like regular ints in general

I am under the same impression.
I prefer option 1: Simply allow Numpy ints as suggested here.

I personally have found Numpy ints to behave like regular ints in general

While generally true, I have recently discovered a case where inserting an integer into a SQLite database fails because it is a numpy.int64 object and not a Python int.
As for whether we should cast to int or support numpy integers I don't really have a strong opinion!

While generally true, I have recently discovered a case where inserting an integer into a SQLite database fails because it is a numpy.int64 object and not a Python int.

Ok thanks for that piece of info.
I don't know why I started with Date, without mentioning the obvious one which is Number! Number accepts any object that is a number given the _is_number internal function:
https://github.com/holoviz/param/blob/main/param/__init__.py#L696-L705
Number does not do any casting/conversion.
This reinforces my opinion that Integer should behave like Number and simply accepts np.integer objects, without doing any conversion.


Added Missing Images of Internships

fixed issue : #276
before

after

before

after

before

after

hello @HimanshuNarware  and @Shubh942
can you please review this PR and merge it

@SadafKausar2025  share a 30 sec video

https://github.com/HimanshuNarware/CareerZunction_Intern/assets/137484958/e4a50426-dc3d-4e9c-9152-0e2222279f1c

hello @HimanshuNarware
I have made vedios , there were 3 - 4 images of the internship was missing I corrected it

hello @HimanshuNarware
please give levels also

Thank You


Switch from browserify to webpack to build admin and analytics bundles

Related: #170, #171.

This has been done and seems to work well.