[BUG] LD_LIBRARY_PATH for compilation environment is overwritten

Describe the bug
LD_LIBRARY_PATH for the compilation environment is set to [] even though it looks like it should be passed through from the environment that start compiler explorer https://github.com/mattgodbolt/compiler-explorer/blob/87fea44d6ad9edf68a06dea3d1cd11dfb829a94b/lib/compilation-env.js#L63
To Reproduce
LD_LIBRARY_PATH=something make EXTRA_ARGS=--debug
Expected behavior
See LD_LIBRARY_PATH="something" in logs
Execution {"type":"executing", ... ,"env":{"LD_LIBRARY_PATH":[], "PATH":"/usr/bin"...
Desktop (please complete the following information):

OS: ubuntu 19.10

Additional context
the PATH and HOME environment variables are passed through successfully.
It looks like this code
https://github.com/mattgodbolt/compiler-explorer/blob/87fea44d6ad9edf68a06dea3d1cd11dfb829a94b/lib/exec.js#L50-L52
overwrites LD_LIBRARY_PATH

Oh blast! Good spot, thanks for noting and taking the time to investigate.

Just bitten by this - or something similar - myself. Looks like ldPath is always [], despite some support for specifying an ldPath option in the properties files. The option is being overwritten by getSharedLibraryPathsAsLdLibraryPaths .
Happy to create a different issue if its best to track separately.


Sonarr Script No Longer Working

It seems the Sonarr script has stopped working and returns a 400 error status.
21-012024 12:16:10 PM :: The response returned a HTTP status code: 400
21-012024 12:16:10 PM :: The response was: {
  "type": "https://tools.ietf.org/html/rfc7231#section-6.5.1",
  "title": "One or more validation errors occurred.",
  "status": 400,
  "traceId": "00-e90b96e3d36389003ee499675ec67a4c-09237b1ad3b924bb-00",
  "errors": {
    "$": [
      "'e' is an invalid start of a property name. Expected a '\"'. Path: $ | LineNumber: 0 | BytePositionInLine: 2."
    ]
  }
}

@bensmith2697 thank you for finding that error. I didn't even notice. Error should be fixed now.


Hero content layout should be normalized on second level sections

Pay attetion to:

space between title and top menu bar > should be equal (72px? 96px?)
space between title and text and left page margin > should be equal, one bootstrap column (1/12)
text column width > should be equal (5 or 6 bootstrap columns)
text height exception > for the page "Come funziona" we can think to use a 18px text for the subtitle to reduce height of the hero
image width for the page "Come funziona" > please evaluate to resize/cut the image more squared

Le misure

Come funziona

Supporto

presente su develop


Tools and notes for cutting releases

Always need to look this up -- putting it close to where I need it.

Codecov Report

Merging #20 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #20   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           39        40    +1
=========================================
+ Hits            39        40    +1

Impacted Files
Coverage Δ

falcon_apispec/falcon_plugin.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


[IONIC 3.0 BETA 6] Screenorientation plugin not working anymore

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[x] 3.0-BETA 6
I'm submitting a ...  (check one with "x")
[x ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Calling screenOrientation.lock throws following error : The page needs to be fullscreen in order to call lockOrientation(). On 2.3 the same code was working.
Expected behavior:
The device locks on the given orientation
Steps to reproduce:
Call screenOrientation.lock()
Related code:
 if(this.platform.is('cordova') && !this.platform.is('tablet')){
                this.screenOrientation.lock(this.screenOrientation.ORIENTATIONS.PORTRAIT);
            }

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
 ordova CLI: 6.4.0
Ionic Framework Version: 3.0.0-beta.3
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.2.5
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.10.2
Xcode version: Not installed

Fixed it by updating to 2.0.0 (Screen orientation plugin)


Mux Echo Responder is ping?

How is even work?
PS C:\Windows\system32> Enable-MuxEchoResponder -TargetVIP "<IP_ADDRESS>"
ErrorMsg Status

     Success

PS C:\Windows\system32> ping <IP_ADDRESS>
Pinging <IP_ADDRESS> with 32 bytes of data:
Request timed out.
Request timed out.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8c117af5-962f-6d09-73f4-acba27a9491c
Version Independent ID: f6017006-6005-d5f8-ca4e-8f5123af3013
Content: Enable-MuxEchoResponder (hnvdiagnostics)
Content Source: docset/windows/hnvdiagnostics/enable-muxechoresponder.md
Product: w10
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

@officedocsbot unassign @e0i
@officedocsbot assign @yogkumgit

Hello @AndreaBarr can you help us finding the right people here? Thanks

Hello @wangkanai thanks for your feedback.
After investigating this, I could find that these cmdlets have been deprecated and they do not have to be used anymore.
I created a pull request to add this info to the cmdlet's documentation.
Hope this info helps you. Thanks!

@get-itips  Thank you very much for the contribution and updating the documentation with PR. @wangkanai Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Implement  a Wrapper that determines if an App can run in the background or not. (I.e make background execution as user friendly as possible)

Most app these days has a need to run in the background and on certain event make calls to callback functions. Using flutter this has been a really tedious task that mostly requires hacks.
Searching around for possible solutions,  I found this article on medium.
https://medium.com/flutter-io/executing-dart-in-the-background-with-flutter-plugins-and-geofencing-2b3e40a1a124
But then it is not so user friendly and I think it only targets advanced developers.

I have thought about possible simple solutions;
Is it possible to Provide a wrapper around runApp() that will enable whatever app it wraps run in background mode and also accepts callback functions to be called on certain  events that might be triggered.l?
Secondly, is there a way to make the class housing the parent material app (where app theme, fonts, accent colors etc are set)  to extend another class/mixin which then provides the background execution feature and also handy functions to override and set callbacks, etc.?

Please don't post issues using all caps

Please don't post issues using all caps

Edited the title..

This is really a huge issue with flutter apps,  I hope a really easy to use solution is birthed soon


Should we implement Terraform code through AWS CDK?

Currently we implement infrastructure-as-code through Terraform HCL, using Jinja2 templates to make them dynamic.
Does it make sense to change this to an 'only Python code' implementation, through the use of AWS CDK?
That supports terraform providers since a while:

https://www.hashicorp.com/blog/cdk-for-terraform-enabling-python-and-typescript-support
https://aws.amazon.com/blogs/developer/introducing-the-cloud-development-kit-for-terraform-preview/

Perhaps we can discuss pro's and con's?

For reference, see #83


Fixed stage resize issue

Closes #62
Set fixed size across all scenes and updated rewards screen to cater to new size

After making the pull request and having the changed code reviewed from another device it was determined that this issue was larger than expected and would require adapting a more resize-responsive design to our project. A new issue will be opened to address creating a more flexible design.


Remove the ElementType from DeallocBoxInst's representation.

It was already redundant and removed from the source representation. We need to remove it now that boxes have multiple elements.

@swift-ci Please smoke test and merge


[azure-docs] Update the external links to GitHub Actions documentation

The URLs for GitHub Actions docs were recently changed. The URLs previously contained a string for free-pro-team@latest, which has since been removed. Redirects were added to ensure that the old links still worked.
To help ease redirect processing, the URLs in these articles should be updated to remove the free-pro-team@latest string:

[ ] "Deploy to App Service using GitHub Actions": [Published article] - [Markdown source].
[ ] "Deploy a custom container to App Service using GitHub Actions": [Published article] - [Markdown source].
[ ] "Set up a GitHub Actions workflow to deploy your static website in Azure Storage": [Published article] - [Markdown source].
[ ] "Continuous delivery by using GitHub Action (Azure Functions)": [Published article] - [Markdown source].
[ ] "Deploy Azure Resource Manager templates with GitHub Actions": [Published article] - [Markdown source].
[ ] "Configure a GitHub action to create a container instance": [Published article] - [Markdown source].
[ ] "Use GitHub Actions to connect to Azure SQL Database": [Published article] - [Markdown source].
[ ] "Quickstart: Use GitHub Actions to connect to Azure MySQL": [Published article] - [Markdown source].
[ ] "Use GitHub Actions with Azure Machine Learning": [Published article] - [Markdown source].
[ ] "GitHub Actions for deploying to Kubernetes service": [Published article] - [Markdown source].
[ ] "Sync your GitHub repository to App Configuration": [Published article] - [Markdown source].

Action plan
For each doc listed above:

Remove the free-pro-team@latest string in each link to docs.github.com.
Test the new link to confirm that it resolves correctly.

👋 We'll be working on this with @juliakm.

👋 We'll be working on this with @juliakm.


[FEAT] add optional param to bridgewellBidAdapter

Type of change

[ ] Bugfix
[x] Feature
[ ] New bidder adapter
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Does this change affect user-facing APIs or examples documented on http://prebid.org?
[ ] Other

Description of change

test parameters for validating bids

{
  bidder: '<bidder name>',
  params: {
    // ...
  }
}

Be sure to test the integration with your adserver using the Hello World sample page.

contact email of the adapter’s maintainer
[ ] official adapter submission

For any changes that affect user-facing APIs or example code documented on http://prebid.org, please provide:

A link to a PR on the docs repo at https://github.com/prebid/prebid.github.io/

Other information

May want to include edge case test specs.
cpmWeight should be in the range of 0 (exclusive) to inf
cpmWeight should be multiplied to cpm


bug(form-field): Extra navigation stop on label with VoiceOver and ChromeVox

This can be reproduced on the docs site:
https://material.angular.io/components/select/examples
https://material.angular.io/components/autocomplete/examples
When navigating the page with either VoiceOver or ChromeVox, you'll land on the label first before landing on the control itself (which also reads the label).

This seems like fairly expected behavior. Using the left/right arrows or VO-Left/Right arrows, Voiceover stops on headers, labels, text elements, and interactive elements.
If I put aria-hidden="true" on the label, then Voiceover navigation completely skips the mat-select in addition to the <label> .
Can you confirm that the desired outcome here is that

the text in the label would be read when the mat-select is navigated to
the <label> that contains the <mat-label> would not be accessible via the left/right arrows or VO-Left/Right arrows in Voiceover

The control should be a single navigation stop that announces itself as a combobox, even if it does read the label twice on that navigation stop. This page on Google's travel site has some MDC-based (but not Angular) components that show the behavior.

This may have something to do with our DOM structure.
Mat

Mat-MDC

MDC

Summary
Mat and Mat-MDC do not wrap the input in the label, but MDC does have the input wrapped by the label element.
Additionally, MDC uses aria-labelledby to point to the input's "label" (which is actually a span nested in the label) and aria-describedby to point to the hint/validation container.

Related comments on why we chose our current DOM structure instead of using the approach that MDC uses:
https://github.com/angular/components/blob/23a0248bc882974258a3b53bddb3c9ca077eeb2b/src/material-experimental/mdc-form-field/form-field.html#L3-L15

This page on Google's travel site has some MDC-based (but not Angular) components that show the behavior.

This page doesn't seem to work with VoiceOver navigation controls for me. Neither left/right arrow or VO+Left/Right arrow move the selection, they just repeat and get stuck on the same element. This problematic behavior isn't reproducible on our docs site or dev-app.

On https://material.io/components/text-fields, when using VoiceOver navigation, it lands on the input first and then the label. So there are still two navigation stops with the pure MDC components, they just invest the order (mat-form-field lands on the label first and then the input).

On ChromeVox:

mat-mdc only has single tab stops, but there are two navigation stops for most cases with inputs and textareas
MDC has only single tabs stops and single navigation stops

The legacy mat-form-fields don't have a navigation stop on the asterisk, but they do have stops on the label, input, and hint.

So I guess that I can't compare against the MDC components on https://material.io/components or https://material-components.github.io/material-components-web-catalog since it appears that those sites don't get updated and are using MDC from ~2016. I've asked for where I can find the catalog of components published with the latest release, but it appears that this may not be available.

As for the extra navigation stop on the asterisk, this is related to the following style from MDC
.mdc-floating-label--required::after {
    margin-left: 1px;
    margin-right: 0px;
    content: "*";
}

This might be a bug in ChromeVox as this isn't reproducible in VoiceOver.

From the MDC team

For linear SR navigation with all elements present, the text field should go through:

label
leading icon (only if interactive)
prefix
input
suffix
trailing icon (only if interactive)
helper text

The character counter is hidden from the screen reader.

This is based upon the latest release of MDC, which isn't available to demo/verify anywhere atm.
That said, it does appear that having a separate navigation stop for the label and input is expected. @jelbourn assuming the MDC team has run this all through the a11y team, should we align with them and keep the SR navigation stops as is for label and input?

Yeah, I think that's enough to satisfy this in the name of consistency, though I personally think that the extra stop for the label is a poor user experience.

Jeremy is going to check with the a11y team to determine desired outcome and next steps.

I can confirm that the extra navigation stop on the label is expected and intended.


Cargo.lock out of sync

Hi there,
Thanks for the cool project. I use it daily.
I noticed that in the last commit, (9c191279139c0fbbe494820b8702bb0d8481ea69), you bumped the version in Cargo.toml to 0.2.2, but
the version in Cargo.lock is still set to 0.2.1 (https://github.com/jrmoulton/tmux-sessionizer/blob/main/Cargo.lock#L1253).
This is causing some issues with CICD build tools which usually pass the --frozen flag to cargo build. However, due to the above,  cargo build --frozen fails with the following error:
error: the lock file /home/emilioziniades/personal/tmux-sessionizer/Cargo.lock needs to be updated but --frozen was passed to prevent this
If you want to try to generate the lock file without accessing the network, remove the --frozen flag and use --offline instead.

To fix this, I'd like to propose incrementing the patch version again to 0.2.3. I will throw together the PR, but you will have to tag the commit on your side.

Awesome, thanks for the quick response. Here's the PR: #19


change html file

Add panel to display Description, Developer and Download Link;
Clicking recommended apps could refer to Download Link.

Zhihao, please redo your PR. Check this : https://github.com/JesseZhuang/Tiger-AppStore/blob/zhihao/src/main/java/spring_boot_style/models/AppRepo.java check this file in the dev and zhihao branch, you are not up to date with dev. Please first git pull origin/dev then do your modifications and then resubmit your PR.


Local LLM llama2 extremely slow

Describe your question
I am running llama2 locally and the model is very fast when I run it with the command line ollama run llama2.
When I run it with openDevin, the request and response are extremely slow and I get devin output every 30 minutes!
Also any updates on when we can access llama3?
Additional context
config.toml
LLM_MODEL="ollama/llama2"
LLM_API_KEY="ollama"
LLM_EMBEDDING_MODEL="llama2"
LLM_BASE_URL="http://<IP_ADDRESS>:11434"
WORKSPACE_BASE="./workspace"
DEBUG=1

Did you already follow this guide?
https://github.com/OpenDevin/OpenDevin/blob/d692a72bf3809df35d802041211fcd81d56b1dc6/docs/guides/LocalLLMs.md#local-llm-guide-with-ollama-server
It looks like you're using an older approach , and not correctly (embedding model would be "local") with old instructions

I am running using make start-backend & make start-frontend. But OK now following your suggestion:
docker run
--add-host host.docker.internal=host-gateway
-e LLM_API_KEY="ollama"
-e LLM_BASE_URL="http://localhost:11434"
-e WORKSPACE_MOUNT_PATH=$WORKSPACE_DIR
-v $WORKSPACE_DIR:/opt/workspace_base
-v /var/run/docker.sock:/var/run/docker.sock
-p 3000:3000
ghcr.io/opendevin/opendevin:main
invalid argument "host.docker.internal=host-gateway" for "--add-host" flag: bad format for add-host: "host.docker.internal=host-gateway"

I think your config is correct
LLM_MODEL="ollama/llama2"
LLM_API_KEY="ollama"
LLM_EMBEDDING_MODEL="llama2"
LLM_BASE_URL="http://<IP_ADDRESS>:11434/"
WORKSPACE_BASE="./workspace"

As for LLM Debugging, you should set export DEBUG=1 in the environment and restart the backend. OpenDevin will then log the prompts and responses in the logs/llm/CURRENT_DATE directory, allowing you to identify the causes.

If you set LLM_EMBEDDING_MODEL to llama2, opendevin will use OllamaEmbedding which should be the same speed if you entered the same prompt in cli.
Maybe you can use ollama's guide to check where's the issue.

@znahas unfortunately there's not much we can do about ollama being slow. It's very resource hungry! Especially for larger models.
llama3 should work fine with ollama, from what I can tell 😄

I am running llama2 locally and the model is very fast when I run it with the command line ollama run llama2.
When I run it with openDevin, the request and response are extremely slow

While you might get a quick answer by using llama2 directly, OpenDevin's prompt is usually very long, and if you are running Ollama on a less beefy machine (e.g. GitHub Actions), the response would be very slow.

Gonna close this one since there's not much we can do.
IIRC we do have an open issue to create an agent that specializes in small local models


667 signup backbutton

🛠 Fixes Issue
closes #667
👨💻 Changes proposed
fixed signup back button by correcting path.
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
fixed issue no-667.Please merge this.

Was there any problem in the PR? It is unmerged .Please tell @SauravMukherjee44

The back button is already working on the website. Why you have created a pr for that!


analyze-libraries suite analyzes 4 new libs when using --use-sdk

When running test.py -cdart2analyzer -rnone, we run the analyze-libraries test suite, which analyzes all the libraries in sdk/lib or [build dir]/dart-sdk/lib, depending on if --use-sdk is passed.
In the dart-sdk/lib, there are copies of 4 libraries from pkg: analyzer, analysis_server, dev_compiler, and front_end.
These currently don't analyze cleanly in this context, or don't have the right status lines, so we are skipping them for now.  The skip marker is in tests/lib/analyze_library.status
We should make them pass, and remove the skip lines.

Somehow, the tests are failing on the bot while passing locally.  Suppressing analyze_libraries on the bots with use_sdk, until this is resolved

Since the test runners and tests and build configs have changed so much recently, I suspect this has been fixed. Please re-open if not.


Info - Add info on expected CSV formats

For both Service Area and Providers, add info on the expected format of the CSV.

Do you guys have any ideas for how/where to display this information? It's definitely challenging and not worth the effort to catch every variation of headers, but we should catch the most common ones... will file separate issue.

Maybe a good team work for @akegan and @sinclaireprowse ?


Usage with Islands?

Hi! I'm fairly new to both headless ui (react and solid), so this may be somewhat obvious, but I'm trying to get solid-headless to work with solid. I am able to get this library to work using regular SSR, as well as SPA. I am also able to get reactive components to work using islands when using simple Signals. For example:
// vite.config.ts
solid({
  islands: true,
  islandsRouter: true,
}),

// Counter.tsx
import { createSignal } from "solid-js";

export default function Counter() {
  const [count, setCount] = createSignal(0);
  return (
    <button class="increment" onClick={() => setCount(count() + 1)}>
      Clicks: {count()}
    </button>
  );
}

// index.tsx
import { unstable_island } from 'solid-start'
const Counter = unstable_island(()=>import("./Counter"))

export default function App() {
  return (

      <Counter />

  )
}

Unfortunately, I am unable to do this with solid-headless:
const { Popover } = unstable_island(()=>import("solid-headless"))

Looking further, it seems that unstable_island needs a default export, so I tried wrapping
// WrappedPopover.tsx
export { Popover as default } from "solid-headless"

// index.tsx
import { unstable_island } from 'solid-start'
const WrappedPopover = unstable_island(()=>import("./WrappedPopover"))

export default function App() {
  return (

      <WrappedPopover>
        {/* other elements that were also exported */}
      </WrappedPopover>

  )
}

Two outcomes occur. If I directly wrap the headless elements, then my entire page shows blank.
But if I import a component inside of index.tsx which uses unstable islands to import, the page shows, but the component becomes static and has no reactivity.

I think what I can only recommend you here is to create another module that imports the solid-headless component, export it as default, then import the new module as an island

@lxsmnsyc Isn't that what I already did in the code above?
// WrappedPopover.tsx
export { Popover as default } from "solid-headless"

// index.tsx
import { unstable_island } from 'solid-start'
const WrappedPopover = unstable_island(()=>import("./WrappedPopover"))

export default function App() {
  return (

      <WrappedPopover>
        {/* other elements that were also exported */}
      </WrappedPopover>

  )
}

Or did I miss something in that set of steps?

Hmm I missed that on first read. TBF I think the whole popover and its children should be made as a separate component, then import that component as an island. I'm not really sure about the island specifics for SolidStart so I'm not sure if my answer is accurate or helpful.

It's a bit weird because I tried both solutions, but I just noticed that when I run on dev, the headless components aren't reactive, but once I npm run build, npm run start, they are reactive. Not sure what the deal here is...

What I'm saying is instead of doing something like
import { unstable_island } from 'solid-start'
const WrappedPopover = unstable_island(()=>import("./WrappedPopover"))

export default function App() {
  return (

      <WrappedPopover>
        {/* other elements that were also exported */}
      </WrappedPopover>

  )
}

You should do
import { unstable_island } from 'solid-start'
const WrappedPopoverIsland = unstable_island(()=>import("./WrappedPopoverIsland "))

export default function App() {
  return (

      {/* component contains child into the island */}
      <WrappedPopoverIsland />

  )
}

Finally got it working. Turns out that I had imported a solid component as a non-default export which was from a Server component. I think in the Documentation repo here, they do export the  component as default from solid-start, and have to import it into an island. So I ended up having to do the same thing.
Before:
App.tsx

const FeaturesDesktop = unstable_island(()=>import("./FeaturesDesktop")
export function FeatureComponent(props) {
function FeaturesMobile() {

FeaturesDesktop.tsx

import { FeatureComponent } from './App'

After:
App.tsx

const FeaturesDesktop = unstable_island(()=>import("./FeaturesDesktop")
import FeatureComponent from './FeatureComponent'
function FeaturesMobile() {

FeaturesDesktop.tsx

import FeatureComponent from './FeatureComponent'

FeatureComponent.tsx

export default function FeatureComponent(props) {

Explanation
Poor architecture, but whatever - App.tsx uses the Mobile Features component, and imports the Desktop Features Component as an island. Both Mobile and Desktop show the same FeatureComponent, but it is displayed differently. On Mobile, it is static, and on Desktop, it is reactive.
I may have missed a few steps along the way, but this is the solution I have come up with and hopefully it helps others who get stuck in the same situation.


exec  ./scripts/pretrain.sh  error，No such file or directory: 'datasets/huawei/cpu_limit_minute.json'

ataset: traffic Total train points: 12086964 Total val points: 981818
Dataset: kdd_cup_2018_without_missing Total train points: 2925624 Total val points: 307530
Dataset: saugeenday Total train points: 23697 Total val points: 1139
Dataset: sunspot_without_missing Total train points: 73880 Total val points: 1139
Traceback (most recent call last):
File "/content/lag-llama/run.py", line 843, in
train(args)
File "/content/lag-llama/run.py", line 334, in train
) = create_train_and_val_datasets_with_dates(
File "/content/lag-llama/data/data_utils.py", line 156, in create_train_and_val_datasets_with_dates
with open(path, "r") as f: data = json.load(f)
FileNotFoundError: [Errno 2] No such file or directory: 'datasets/huawei/cpu_limit_minute.json'

Hi! You might not have updated your repo. Please pull again. The dataset path was updated, now it uses what you specify as the path in the run.py file.
Also pl check if you have followed the instructions at the top of the script before executing it.

Hi, is this resolved?


Not enough output

There should be over 2,000 files in the output with the current test data. But last I saw the load file, there were less than 200

I think it is correct, part file contains 129 lines, and I counted files in test data, and there are exactly 129 files.
Why do you think it is wrong?

The test data has pst files, and they contain 2000 emails
Mark Kerzner, Managing Partner, Elephant Scale http://elephantscale.com/
Book a call with me here http://www.meetme.so/markkerzner
Mobile:<PHONE_NUMBER>, Skype: mark.kerzner1
On Sun, Jul 31, 2016 at 7:21 PM, Aleksandr N<EMAIL_ADDRESS>wrote:

I think it is correct, part file contains 129 lines, and I counted files
in test data, and there are exactly 129 files.
Why do you think it is wrong?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/markkerzner/FreeEed/issues/130#issuecomment-236466306,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAk2eb5ptLpuIKJeLoeViJrJFUGNxOEVks5qbTwSgaJpZM4JYiLW
.

@markkerzner
Now outputs around 2400 lines. Please pull from my branch.
There were few problems - 1) readpst was not detected properly on OS X - /usr/bin/local is not in PATH by default, 2) Exceptions were swallowed in OsUtil.runCommand methods, I did some refactoring there.
I did not verify it on Linux, please check it.

I tested in Linux, it works. But do we have a test that verifies the number of output results? We should

Alex fixed it, Mark tested


register : send JWT by email instead of responding with Json User

We must validate user's email for future airdrops.
There will be a lot of scams, I feel uncomfortable recording useless entries in the system... stateless approach looks better anyway.
When a user provides its c-chain (ETH) address + email + type (client/agent/mentor/talent), the system send an email with an activation link using a JWT token including these data.
The token should expire in 1 hour.
We should store data only when tokens are well validated, preventing to store useless data with a simple flag that must be updated after activation.

Can you please add an "investors" type.

will do 💚


Fix nonsynth cache UC store hit bug

Fixes #1086
Root cause was implementation of the nonsynth cache used in the ME testbench. Uncached stores targeting valid cache blocks were not self-invalidating the block, causing the store to get lost if targeted by following cacheable load. Bug is only present in the ME testbench, not in a real BP system.

Great find!


Title is not updated when options come later

I have a situation when model value is existed, but options come a little later. In this case options are filled and values are selected, but title is empty:
<ss-multiselect-dropdown
        [options]="myOptions"
        [(ngModel)]="optionsModel">
</ss-multiselect-dropdown>

export class App {
  optionsModel: number[] = [];
  myOptions = [];
  constructor() {
    this.optionsModel.push(1); // imitate that model has value. In my real scenario I have select input(another component) and when user select value I show details and there are several angular-2-dropdown-multiselect components, and make reuqest to get options for multiselect.
  }

  fillOptions() {
    Observable.of([
            { id: 1, name: 'Option 1' },
            { id: 2, name: 'Option 2' },
        ])
        .timeout(200)
        .subscribe((data) => {
          console.log(data);
           this.myOptions = data;
        })

  }
}

I have created Plunkr for you, so you can see what I mean. Click Fill Options first and check multiselect values.

Thank you very much for the plunkr.  Its a big help for reproducing the issue.

@gund Thanks for fixing the issue. @softsimon Could you push the changes to npm?

@softsimon Also waiting for changes to be pushed to npm.

@will093 done

@softsimon I now have an issue where updateTitle() is being called before the component is initialized. I think I have fixed the problem, please see my pull request.


First draft of Bulk Data API

Coverage decreased (-0.4%) to 93.749% when pulling 36a87d42429ce904106562a42d63dfc3c0efed0e on cody-fhir-bulk-data into 3d10e2a718e18b02f756836ba4bd142baa364baa on main.


Instability of label model

Hi,
Sorry for bothering you again.
I found some weird behavior in the label model where I run the label model twice with the same data and I got the totally different result. Here you can find a snapshot of my code.

Data: basics_tutorial.pkl
Code: snorkel-metal 0.1.1
Please let me know if I am doing something wrong.
Thanks,
Sen

This a great issue, Sen!
A naive guess is that the symmetry isn't broken consistently, there is a always a symmetry e.g., p -> 1-p for labeling function accuracies. This is where the theory elects a particular winner @ajratner @fredsala any thoughts here?
Can you add in the LF function accuracies to confirm or refute that guess? Thanks!

Hi Sen,
Thanks for posting- keep them coming! At the end of the training we learn
the model up to symmetry- and then break symmetry based on the fact the LFs
are non-adversarial. But apparently this isn’t happening correctly here!
Either way it’s an easy thing to resolve- I will do so a little bit later
today
Thanks,
Alex
On Sat, Sep 1, 2018 at 10:56 AM SenWu<EMAIL_ADDRESS>wrote:

Hi,
Sorry for bothering you again.
I found some weird behavior in the label model where I run the label model
twice with the same data and I got the totally opposite result. Here you
can find a snapshot of my code.
[image: screen shot 2018-09-01 at 10 54 35 am]
https://user-images.githubusercontent.com/5580008/44948599-7b1df400-add5-11e8-8334-1e09becd9edb.png
Data: basics_tutorial.pkl
Code: snorkel-metal 0.1.1
Please let me know if I am doing something wrong.
Thanks,
Sen
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/HazyResearch/metal/issues/36, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABgw_ftyKs7sGFM_ZHNk1bDqdKGrbHskks5uWsoygaJpZM4WWT_4
.

@SenWu @chrismre Yes this is definitely what's going on- in transit now but then I'll resolve in a bit! First try is just by changing the defaults to use centered L2, which is not currently on by default, in addition to initialization to resolve this symmetry- should take care of it

Hi @SenWu I'm having a hard time reproducing the behavior you have above- have you pulled the latest version of master?  Any other differences that might explain this?
I still think this must be a symmetry-breaking issue, just confused that I can't replicate.  We can talk on slack abt too!

Looks like this is fixed on master, need to propagate to the pip installable version

If you have the latest code on master and are tagging releases to pypi, I'd highly recommend incrementing the version number in master after a release (e.g. to v0.1.6). This avoids the issue where if a user installs from pypi, they get v0.1.5, and if a user installs from GitHub, they get v0.1.5, but they are not actually the same versions.

@lukehsiao thanks, will do!
@SenWu note I added in centered L2 reg (see updated Basics tutorial) which might also generally be useful for these types of symmetry-breaking issues, if they come up
Also, unless any strong objections, I'm going to start closing issues when they are fixed on master; then I will increase the frequency of pushing new versions to pip.  Happy to hear pushback on this... but since I still view this repo as in 'dev-heavy mode' (research!!), I think this will be cleaner


api.search errors

Twitter bot which retweets,like tweets and follow users
Bot Settings
Like Tweets : False
Follow users : False
Traceback (most recent call last):
File "twitterbot_retweet.py", line 15, in
for tweet in tweepy.Cursor(api.search, q=QUERY).items():
AttributeError: 'API' object has no attribute 'search'

I'm having the same problem what was your solution


Add train script

Add train script for summarization task
Add train script for text-generate task
Add train script for token classification task
Add common libraries for environment in env.yaml

@HDThang Would you please resolve conflicts first?


Tk nbextension

Codecov Report

Merging #33 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master      #33   +/-   ##
=======================================
  Coverage   49.59%   49.59%
=======================================
  Files          28       28
  Lines        2105     2105
  Branches      210      210
=======================================
  Hits         1044     1044
  Misses       1057     1057
  Partials        4        4

Impacted Files
Coverage Δ

marvin_python_toolbox/management/notebook.py
62.5% <ø> (ø)
:arrow_up:

marvin_python_toolbox/common/config.py
25% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Error in the document

On the following page of the document for v0.2, examples/singleserver
`
sample_time!(𝐶, 0.1)  # we determine the sample rate
Sim.sample!(𝐶, SF(stats));  # we register stats() as sampling function
`
Should be
`
sample_time!(𝐶, 0.1)  # we determine the sample rate
Simulate.sample!(𝐶, SF(stats));  # we register stats() as sampling function
`
To do

[x] include example files into testing

Hello,
thank you for pointing out such errors. This was due to a former name change.
I have to ensure in future releases that all examples run properly by including them in the testing.

I like this package very much, especially the parallel calculation part. I have experience of using simulation packages in Julia, such as SimJulia.jl.  I would like to contribute to this project. Thanks for your great work.

The parallel part is not yet user ready. I hope to change that in the next few weeks.
You are very welcome to contribute. There are various ways to contribute, e.g.

pointing out errors and inconsistencies,
doing examples, tests and benchmarks,
helping in documentation,
contributing code, PR's …

I will setup a current project board in order to make the topics I'm working upon more visible. Then perhaps it is easier to contribute.

Great!

I have updated the issues to reflect the issues I'm working on.
All this is synced to a Glo board. If you want to use GitKraken Glo, please drop me an email and I can invite you to the board. Otherwise simply contribute to the issues.
Thank you in advance.

I have enhanced testing such that *.jl and *.ipynb files in examples/src and examples/nb get executed as a last test step. Thus they are integrated in CI.

@pbayer  I am not familiar with that kind of tools. I think I work on some simple things first and then contribute more.  I do not want to mess the current code.

now examples and notebooks are evaluated within their own module,
so no more namespace collisions during testing.
all examples are working and included in testing.


xLib: automation class can produce glitchy values

Due to incorrect use of xPlayPos -
Change the implementation - then update these tool that make use of it:
xRules + xStream

Fixed with xRules v0.781, but not yet pushed to xStream release


Stabilize tests

There are often recurrent failures, especially on Travis, which are all false positives due to lack of synchronization between the main thread and the thread of the test FTP server.
The solution could be using multiprocess.Process instead of threading.Thread or, in any case (no matter if we use threads or multiple processes) we should use a synchronization primitive between the client and the test server (multiprocess.Queue or threading.Queue).

We should also enable python warnings.


Differentiating between 1841 and 1851 pieces

HO107 covers the 1841 and 1851 censuses. When we developed FC2 we were under the impression that 1-999 was 1841 and anything from 1000+ was 1851.
It is now apparent that the boundary is HO107_1465 (1841) and HO107_1466 (1851).
The code needs adjustment as the system is reporting Errors in 1841 pieces from HO107_1000 to HO107_1465 as it sees these pieces as 1851 pieces.
Geoff

I am sorry that is just not true. We have always known where the transition occurred.
https://github.com/FreeUKGen/MyopicVicar/blob/master/app/models/freecen2_piece.rb#L122-L124

Back to the drawing board on the problem that I have.


clar: simplify how we handle stat(3P) on Win32

The Windows platform has multiple different definitions for stat(3P) and its struct stat with different integer widths for both the filesize and the timestamps. For our usecase, we really only need to care whether we end up using _stat() or stat(), which require us to use either struct _stat or struct stat, respectively.
The way we handle this in "clar.c" is somewhat convoluted though because we select the function and structure to use independently of each other. This can cause us to pick the wrong combination of features in some environments, like mingw-w64.
Simplify the code such both function and struct get set up under the same condition such that this cannot happen anymore.

Thanks!


possible bug in javascript: viewportHeight

Just found this problem, in a window larger the the viewport, with a lightbox opening far below the viewport, rendering it useless. (in Firefox 59, at least)
I found this code on line 97:
    var viewportHeight = $(window).height(),
    alertHeight    = alert.instance.height(),
    diff           = viewportHeight - alertHeight;

Fix:
$(window).height() returns the full height of the rendered page, while the code wants the viewport size.
I switched it to
    var viewportHeight = window.innerHeight,
        alertHeight    = alert.instance.height(),
        diff           = viewportHeight - alertHeight;

window.innerHeight will correctly return the viewport height, thus positioning the window at the correct place.

This is potentially a real fix, but I'd like someone to confirm browser compatibility before I merge.

Latest version implements something like this. It was done to improve layout for larger alerts that are taller than the window which was causing them to be too close to the top of the window.


API Error while using deepseek-coder-v2:16b with Ollama Provider

I am trying to use Ollama with deepseek-coder-v2:16b. I get a status 400 error when I make an API request against the model. I also tested codegemma:7b-instruct-v1.1-q8_0, and am getting the same error with it as well.
ERROR:
API Request Failed:
400 codegemma:7b-instruct-v1.1-q8_0 does not support tools

It does not appear to like dashes (-) or colons (:) - typing the right one that it understands is difficult.

Tested it with model id deepseek-coder-v2; still getting the same response:
API Request Failed
400 deepseek-coder-v2 does not support tools
I am assuming it is related to the way the request object is being created for ollama. It works with llama3.1 but have been getting errors similar to this on multiple other models.

See this blurb from the Ollama configuration page:

Ollama allows you to run models locally on your computer. For instructions on how to get started with Ollama, see their quickstart guide. You can use any models that support tool use (Note: Claude Dev uses complex prompts, so less capable models may not work as expected.)

"You can use any models that support tool use"
The thing that is confusing me is that I am able to use models like deepseek-coder, codestral, etc. with a different extension and the same back-end Ollama setup, so I'm not sure why the limitation exists, but I suspect it'll be fixed at some point.

And if you click that link, a bunch of models appear that support tool use. I tried the qwen that was there but the model runs with a slash in its name, so that did not work. But it is also a very small model, that could be the issue as well. The only one I have seen that will load in for me was either llama3 or llama3.1 without using the :8b at the end of it. I hope, with all the recent cool updates they have done, this gets looked at a little more.

I tried the qwen that was there but the model runs with a slash in its name, so that did not work.

So, funny story - On a whim, I decided to test a model with a slash in the name to see if I could fix the code.  It worked for me.
But... I went to the model/tool list again to search for qwen to see if I could figure out what you were seeing, and then I realized that a lot of the more popular models have tool variants.
I just used MFDoom/deepseek-coder-v2-tool-calling and it works like a charm!

I use MFDoom/deepseek-coder-v2-tool-calling, and running "Ollama run MFDoom/deepseek-coder-v2-tool-calling", then set in VScode like:

but when I start a new task, it appear " API Request Faild, Connection error"

@xiaojichao were you able to get this working? This is strange, can you pls update to the latest version 1.6.3 and see if a list of ollama models show up in claude dev for you to choose from? If not then something's blocking it from connecting to your local endpoint, hmm

Please see https://github.com/saoudrizwan/claude-dev/issues/230#issuecomment-2340575426


Demonstrate build with all 3 parameter sets in CI

Consumers of mlkem-native (e.g. other major cryptographic libraries) will likely want to include all three ML-KEM parameter sets for their users.
We therefore need to demonstrate that mlkem-native can be built 3 times with MLKEM_K=[2,3,4] and the resulting object files combined into a single library with no clashes.
On platforms that support it, this should be demonstrated for producing both a static- and a dynamically-linked library.

I agree.
@mkannwischer @potsrevennil What are your thoughts on changing the build so it generates a libmlkem{512,768,1024}.a in the interim, and we demonstrate in a separate test that all those link together into libmlkem.a?

I like that idea. Some consumers might want just one (small) library that implements just 1 parameter set. Some consumers might want all 3. We need to demonstrate that both options are possible and easy.

No, they are all namespaced. Doing things without duplication would be out of scope initially.

@rod-chapman - is this something you need relatively soon or just an idea that we may want to consider when we have time?

I agree.
@mkannwischer @potsrevennil What are your thoughts on changing the build so it generates a libmlkem{512,768,1024}.a in the interim, and we demonstrate in a separate test that all those link together into libmlkem.a?

I agree with that proposal - but for now this is just for our build system, not something that we ship in releases or want the consumer to use directly. If we actually want to ship a library that is going to be much more work - esp. w.r.t. Keccak implementation variants.

No, they are all namespaced. Doing things without duplication would be out of scope initially.

I also strongly agree with this. If you want to do that, you will have to re-work a lot of code and move further away from the reference implementation which is not something we want to do unless we have to.
If this is something that's important for one of our (potential) consumers, then we should discuss that separately.

I agree with that proposal - but for now this is just for our build system, not something that we ship in releases or want the consumer to use directly. If we actually want to ship a library that is going to be much more work - esp. w.r.t. Keccak implementation variants.

I agree. But even if we just showcase the possibility in our own build system, that's useful.

I agree.
@mkannwischer @potsrevennil What are your thoughts on changing the build so it generates a libmlkem{512,768,1024}.a in the interim, and we demonstrate in a separate test that all those link together into libmlkem.a?

I would also agree on this proposal and like the idea of showcasing the possibility in the build system.
As @mkannwischer mentioned in the long term, if we aim to ship a library as part of a release, it will require additional effort. Specifically, we need to clarify our approach to shipping the library in relation to all implementation variants.


Package jack_transport_link

Name: jack_transport_link
Suggest a package name: jack_transport_link
Short description: A bridge between Jack's Transport mechanism and Ableton's Link synchronization
Latest release version: none yet.
Website: https://github.com/x37v/jack_transport_link
Source code: https://github.com/x37v/jack_transport_link.git
AUR (if exists): none
License(s): GNU General Public License, version 2
Is the project active? yes
I have successfully built this myself on Arch: no

This appears to be an attempt to improve on jack_link.
Please consider packaging this software.

It seems that building this software from source needs the build tool conan, whch is only available in the AUR and in turn depends on two more Python packages only available in the AUR (python-patch-ng and python-pluginbas).
We can't package software in our repo, which has dependencies on the AUR, even for makedepebds.
Additionally, when trying to build the sofware wih conan installed, I get this error:
$ cmake -B build -S $(pwd) -DINSTALL_SERVICE_FILE=OFF
-- Conan: checking conan executable
-- Conan: Found program /usr/bin/conan
-- Conan: Version found Conan version 2.4.1
-- Conan executing: /usr/bin/conan install . --build missing --settings build_type=Release --settings compiler=gcc --settings compiler.version=14 --settings compiler.libcxx=libstdc++11 --settings compiler.cppstd=17
ERROR: The default build profile '/home/chris/.conan2/profiles/default' doesn't exist.
You need to create a default profile (type 'conan profile detect' command)
or specify your own profile with '--profile:build=<myprofile>'
CMake Error at build/conan.cmake:651 (message):
  Conan install failed='1'
Call Stack (most recent call first):
  CMakeLists.txt:39 (conan_cmake_install)

-- Configuring incomplete, errors occurred!

Until both these issues are resolved, I'm afraid we have to give this one a pass.

@SpotlightKid Okay, I see. Thank you for investigating this in detail.


handle screen resizing

this is work in progress. It only partially works, and I'm not happy with the implementation anyway.

I think the resizing strategy is not good. I should resize around the cursor region so that it does not go out of scope.

I think I'll merge that. It could be better but at least we now have most of the basic features xi-core provides, so we can start polishing things.


compute::ord kernel for getting min and max of two scalar/array values

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

https://docs.rs/arrow-arith/latest/arrow_arith/aggregate/index.html perhaps?

label_issue.py automatically added labels {'arrow'} from #4355


Don't explicitly generate proxy.secretToken

Thanks to
https://github.com/jupyterhub/zero-to-jupyterhub-k8s/pull/1993,
proxy.secretToken is automatically generated :)

@yuvipanda, do you know if this was this already released?

Yep!

Accordingly to this page: https://jupyterhub.github.io/helm-chart/
0.11.1 was released on Jan 15 and the reference PR was merged on Jan 27.
What am I missing here?

@damianavila ah, so I like running closer to master :D If you look at hub-templates/basehub/Chart.yaml, you'll see that we're actually on 0.11.1-n277.h61b0e003, which was released on March 3 (according to https://jupyterhub.github.io/helm-chart/). It was updated last in f51325e6c355892842e8ac1ac464c8624690ecea, to bring in https://github.com/jupyterhub/zero-to-jupyterhub-k8s/pull/2077 (which I consider a very critical fix).

ah, so I like running closer to master :D

Ah, so you like to run at the cutting edge 😛
Thanks for the clarification!

@damianavila Thanks :D More so than cutting edge, I think z2jh master is pretty stable, and we should find resources to help make releases more frequently - see https://github.com/jupyterhub/team-compass/issues/384 and https://github.com/2i2c-org/pilot-hubs/issues/233


Global Pointer value analysis is not run after rebasing (originally: MIPS rebasing doesn't adjust $gp)

Version and Platform (required):
3.5
Bug Description:
MIPS rebasing doesn't adjust $gp, references to $gp-sensitive stuff is lost on rebase, like the global offset table.
Steps To Reproduce:
Open the attached binary: libhdk-mips-elf.zip
Navigate to the memcpy plt stub:
00018900  int32_t memcpy()
00018900  1080998f   lw      $t9, -0x7ff0($gp)  {0x0}  {_GLOBAL_OFFSET_TABLE_}

It correctly gets the pointer to the _GLOBAL_OFFSET_TABLE_ at 0x2c400 using $gp and we can check the math in the python console:
>>> hex(bv.global_pointer_value.value - 0x7ff0)
'0x2c400'
>>> hex(bv.symbols['_GLOBAL_OFFSET_TABLE_'][0].address)
'0x2c400'

Now rebase the program to any address, say 0x1000 and the reference to the GOT is lost:
00019900  int32_t memcpy()
00019900  1080998f   lw      $t9, -0x7ff0($gp)  {data_2c400}

Expected Behavior:
The reference to _GLOBAL_OFFSET_TABLE_ is maintained after rebasing.

Current workaround for this is to open the binary with options rather than rebasing after open.

I tested and found that with my recent changes on the global pointer value update, the global pointer value is already corrected updated after the rebasing. However, the function is not re-analyzed so it still has the old analysis. If I manually update the analysis then it works. The next step is to understand why we missed an analysis update for it

We are missing the analysis update because once we decide a function uses the gp value and update its analysis accordingly, the function's argument list no longer contains the gp register, and it is no longer considered using the gp. In other words, our "function uses gp" detection is only one-shot, when the gp value changes, either because the entire analysis state change, or, in the future, when a gp value is specified by the user (#5648), these functions all would not be updated properly.

There is a regression on this before 4.2.6430-dev (d7f78e8c)

Closing this in favor of https://github.com/Vector35/binaryninja-api/issues/6135. The underlying issue might be incorrect relocation handling, rather than GP value analysis

This should have been closed when I made the comment above


Remove leap-into-the-abyss casting from TSTransformCreate().

[approve ci autest]

This could use a "cleanup" label.


doc(chart): add table of helm values

ref: longhorn/longhorn#5455

use helm-doc to generate table of values in readme
for default settings, modify the description in questions.yaml of each setting to one sentence.

make the descriptions in values.yaml  same as those  in questinos.yaml
for details, user can check the website

conflicts fixed

In general, LGTM. Just one comment needs to resolve.
We should make questions.yaml automatically generated as well in the future. Please help create an issue for that and add to 1.7.

Issue created: https://github.com/longhorn/longhorn/issues/6788


D3D12 pixel history execute indirect.

Cache indirect parameters pre-callback so they can be used for replay.
Move ReplayDraw implementation to device so it can be re-used.
Non-callback support left as a TODO + Assert.

Description

In D3D12 only one draw/dispatch can be contained in ExecuteIndirect arguments and it has to be the last argument, so we don't have to handle disabling other draws/dispatches like the Vulkan implementation does.
The ExecuteIndirect replay path was previously not implemented, so any draw/dispatch that was preceded by a resource assignment in its argument buffer could not have pixel history performed on it.
Sample used for testing:
https://drive.google.com/file/d/1_VCverVpLhZTNnlnj2ZpJvodJ87kA799/view?usp=sharing
Result:

It occurs to me now that there is a problem with this approach: ReplayDraw allows the caller to ask for any draw in the replay to be performed but the cached parameters are only for the event getting the callback... At the very least there should be an assert to prevent this.

I don't fully understand the mechanics of the different event IDs in the replay code, but the eventId that is passed to a PreDraw callback is not available until the return of HandlePreCallback so I can't simply cache it in the indirect replay data before the callback occurs to use for asserting in ReplayDraw.
It's not a check that would get tripped in this change since ReplayDraw is only used on the target event of the callback, but I wouldn't want to put a landmine in that won't be hit until a future change tries to use the function in a different way.

I think it is fine. The code can only be used from a callback at the moment.
For the direct draws it uses the ActionDescription to get the draw parameters but it assumes all the other state is bound correctly. I don't think ReplayDraw() is expected or designed to work except for replaying to the current event and then doing some custom setup and redrawing just the current draw.
When the code to handle Indirect draws from outside of a callback then more work is required and it is good to leave postponed until it is required (because it might never be required). At the moment ReplayDraw() is only used by Pixel History


When Source is abort'ed, persist checkpoint.

When consumers are shutdown, we expect the application to abort the signal sent to source.start. When this is the case, we should persist the last checkpoint.

This is already happening, see https://github.com/equinox-project/equinox-js/blob/3c7b839c0fa1f28c9836f0fc6fb764576fea44ab/packages/propeller/src/FeedSource.mts#L72-L76


PLUGIN-899 fix custom formats

Fixed a bug where custom formats would be disallowed during
plugin validation.

cherry-pick of #774


Roll Flutter Engine from 03c8c7724873 to 155ff372bec4 (1 revision)

https://github.com/flutter/engine/compare/03c8c7724873...155ff372bec4
2023-10-08<EMAIL_ADDRESS>Roll Skia from 3d77b5de5984 to a04f2cfaf4d4 (1 revision) (flutter/engine#46656)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


"isLoading" slightly misleading

When I saw the existence of isLoading, I assumed it would be true from the moment the react-async component mounted until the query promise I provided was completed. Instead, though, isLoading is momentarily false for the first render or so, until the state has had a chance to update, because the component doesn't know that a promiseFn was necessarily provided.
Two options I see:

on construction, set isLoading to true if there is a promiseFn provided and it produces a promise
naming clarity: provide a new param, isReady that is set to !!finishedAt
provision of some kind of warning in the documentation about isLoading not being true initially.

The first option is probably the most intuitive for the user.

Thanks for the feedback, I love to hear about these unexpected/unclear behaviors. As I was developing it, I did initially have isLoading set to true on mount, but changed that when I added the deferFn, because in that scenario, isLoading=true doesn't apply. I like your first suggestion best. I think it does make sense in most use cases and it's what people would expect.
isReady is reserved for future use.

I would love to merge a pull request if you can provide one.

Fixed in v3.6.2.


Fix caustics shadows projected position misalignment

The shadows are projected after being sampled so they align with the shadows on the scene. But it is currently using the projected position which includes a fudge factor. This calculates the projected position without the fudge factor.
The fudge factor does the following:

project along light dir, but multiply by a fudge factor reduce the angle bit - compensates for fact that in real life caustics come from many directions and don't exhibit such a strong directonality

Before

After

I guess that code was mine, probably should have written more comments..
It does look wrong so probably needs to be removed.
Did you try just removing the fudge factor, rather than making a new position var?

Did you try just removing the fudge factor, rather than making a new position var?

I just tried it. Without the fudge factor, the caustics move around with the waves more. Here is a video with and without the fudge factor applied respectively (you'll notice the change).
https://drive.google.com/file/d/13MJH7VgplYIEWZPJNi1wquaQqjblvX97/view?usp=sharing

That seems better I guess?

With the fudge factor removed completely, I notice that the caustics stretch and distort too much in certain areas. I think it is probably best to leave it untouched for now.

Stretching in the middle. Can get worse than this

Ok, still a good fix. do you mind adding comments for what you found?

do you mind adding comments for what you found?

Done

Thanks. Sorry 1 more thing - is it worth giving the two position variables different names to distinguish them? I'm thinking that relying on shadowing could be error prone ("I'm changing the variable but it's having no effect"), and could conceivably generate warnings on some platforms (maybe).

Good idea. Thanks. All done and merged.


[FEAT] Add Guide on how to use HTTP Builtin from OPA

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Some documentation and examples on how to use http builtin to for example fetch a users entity information in order to make more detailed rules.
Describe alternatives you've considered
N/A
Additional context
Imagine you want to build a rule based on an annotation that a user entity entity has, currently the permissions framework does not send the full user data, only the users name and claims.
Using the HTTP builtin, we can make a get request to the backstage catalog and fetch the users entity information like the metadata including annotations etc... which will be really useful for some rules.

We'd still want this!


strange quick plot behaviour

sometimes when comparing models quickplot doesn't plot the correct variables
e.g. try running compare_comsol_DFN and commenting out all of the comsol variables apart from positive potential
comsol_model.variables = {
    #"Negative particle surface concentration [mol.m-3]": comsol_c_n_surf,
    #"Electrolyte concentration [mol.m-3]": comsol_c_e,
    #"Positive particle surface concentration [mol.m-3]": comsol_c_p_surf,
    #"Current [A]": pybamm_model.variables["Current [A]"],
    #"Negative electrode potential [V]": comsol_phi_n,
    #"Electrolyte potential [V]": comsol_phi_e,
    "Positive electrode potential [V]": comsol_phi_p,
    #"Terminal voltage [V]": pybamm.Function(comsol_voltage, pybamm.t * tau),
}

This plots OK
Then also plot the positive particle concentration
comsol_model.variables = {
    #"Negative particle surface concentration [mol.m-3]": comsol_c_n_surf,
    #"Electrolyte concentration [mol.m-3]": comsol_c_e,
    "Positive particle surface concentration [mol.m-3]": comsol_c_p_surf,
    #"Current [A]": pybamm_model.variables["Current [A]"],
    #"Negative electrode potential [V]": comsol_phi_n,
    #"Electrolyte potential [V]": comsol_phi_e,
    "Positive electrode potential [V]": comsol_phi_p,
    #"Terminal voltage [V]": pybamm.Function(comsol_voltage, pybamm.t * tau),
}

It looks like quickplot plots the correct variables for the pybamm model, but plot the positive particle surface concentration twice for the comsol model. Seeing similar behaviour in another comsol comparison I'm doing, but not with other quickplots, so not sure where the issue is

This is a problem with known_evals, fixing now


add another fields in create property form

Description
This PR introduces a new country selection component and prepares for upcoming features. The following changes have been made:

Developed a reusable component for selecting countries, which can be integrated into various forms across the application.
Developed counter components for selecting the number of rooms, bathrooms, and other attributes.
Developed components for selecting amenities will be added to enhance the user experience when creating housing listings.

Involves the recomposition of the housing listing creation form. The form has been refactored to enhance modularity and improve maintainability. Key changes include:

Refactored the structure of the housing listing form for better code organization.
Split the form into smaller, reusable components to improve scalability.
Updated validation logic to work seamlessly with the restructured form using Zod.
Enhanced user experience by optimizing the flow of form fields.

Type of Change

[x] New feature

Checklist

[x] My code follows the project’s coding style
[x] I have performed a self-review of my code
[x] I have tested that my feature works as expected
[x] I have updated the documentation as necessary
[x] My changes generate no new warnings or errors

:tada: This PR is included in version 1.25.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Support HTTP External Processing Filter

Description:
Envoy’s HTTP External Processing Filter can be used by vendors and extension developers to enhance Envoy Proxy's data-plane feature set. The ext-proc filter implements a callout to an external grpc service in request and response flows. The external processor service can inspect and mutate requests and responses, access context data such as attributes and dynamic metadata and emit dynamic metadata.
The external processing filter is included in the envoy base image and its security posture is robust. The external processing protocol and/or filter are also used in the industry for extensibility and integration: GCP Application LB, Gloo Edge, Imperva WAF.
Envoy Gateway can support the external processing filter in a way that is similar to the ext-authz implementation. If supported, this filter will fulfill some of the GA requirements outlined in #2025 related to Envoy Proxy extensibility.

+1 to this
my preference would be to include this into #2025 as a new Policy and combine all dev extensions into one new Policy resource type
cc @Xunzhuo

we could do this with EnvoyPatchPolicy now, right? what the benifit for create an new policy?

a dedicated ExtensionAPI can do a better job

dealing with config validation
reducing surface area of extension
doing extra work such as downloading wasm from registry

doing extra work such as downloading wasm from registry

you want download wasm in controlplane?

I'm +1 for a dedicated ExtensionAPI, maybe named (Gateway|Proxy|Envoy)(Http/Networking)Filter, focus on insert/replace filters, and use ECDS to send XDS.

+1 for adding support to this, thanks for raisin it @guydc.

Please assign to me.


Review

Interesting readme, improve with icons, screenshots, present your dataset, etc;
Never push your venv;
Check the data splitting

X_train, X_test, y_train, y_test = train_test_split(...)

X_train, X_eval, y_train, y_eval
#do this instead, so it will be clear that X_test is the provided data that has no y_target included

No reason to save your models results, just present them in the notebook;
Compare models trained with and without smote;
I don't see hypothesis testing add it;

Step-by-step guide well followed but ask questions if some aspect are not well understood. Share this review with your teammates.

https://github.com/matiassingers/awesome-readme, have a look then make a more attractive readme.

https://github.com/matiassingers/awesome-readme, have a look then make a more attractive readm

Interesting readme, improve with icons, screenshots, present your dataset, etc;
Never push your venv;
Check the data splitting

X_train, X_test, y_train, y_test = train_test_split(...)

X_train, X_eval, y_train, y_eval
#do this instead, so it will be clear that X_test is the provided data that has no y_target included

No reason to save your models results, just present them in the notebook;
Compare models trained with and without smote;
I don't see hypothesis testing add it;

Step-by-step guide well followed but ask questions if some aspect are not well understood. Share this review with your teammates.

Thanks Emmanuel. About presenting my datasets, I thought maybe a download link is enough(i.e direct link to download zip file). Maybe you can clarify so that I can make the changes right away.


Need more details about how to do any troubleshooting or where to look for issues

-If we try to use the previous SP and secret, it will located in ~/.azure/aksServicePrincipal.json if we create the cluster by using "az aks create" command, could you please mentioned this as well in "https://docs.microsoft.com/en-us/azure/aks/update-credentials#update-existing-service-principal-expiration" ?
-Could you please add this Error msg to Troubleshooting page, this means the SP is expired and we need to refresh it
Status Code = '401'. "error":"invalid_client","error_description":"AADSTS7000215: Invalid client secret is provided

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0b842222-f6ac-5266-227f-7e400610e07b
Version Independent ID: 429cd3bf-2691-f633-2e0d-e70e5d8d0e03
Content: Reset the credentials for an Azure Kubernetes Service (AKS) cluster
Content Source: articles/aks/update-credentials.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@gsxmax Thanks for the Feedback. I have assigned the issue to content author for updating the document.

@mlearned Can you please look into the doc update request and add your comments.

Thank you @gsxmax for the feedback!
We've added an item to our backlog and will prioritize accordingly.
#please-close


[SPARK-43712][SPARK-43713][CONNECT][PS] Enable parity test: test_line_plot, test_pie_plot.

What changes were proposed in this pull request?
This PR proposes to enable parity test for pandas API on Spark with Spark connect.
Why are the changes needed?
To test pandas API on Spark with Spark Connect properly.
Does this PR introduce any user-facing change?
No, test-only.
How was this patch tested?
This uncommented tests should be passed CI.

merged to master


No runnable code chunks in Quarto books

Dear authors,
First of all, thank you for this excellent extension. I've tested it and think it is great. I will definitly use it for future contents!
I wanted to let you know about a problem I encountered when trying to use it in a quarto book. When I try to install the extension in a quarto book project using quarto add r-wasm/quarto-live, I get the following error after the first question in the installation process:
Quarto extensions may execute code when documents are rendered. If you do not trust the authors of the extension, we recommend that you do not install or use the extension.
? Do you trust the authors of this extension (Y/n) » Yes

ERROR: Include directive failed.
  in file C:\Users\hategeki\Desktop\CODE\R\Projets\WEBSITES\test-quarto-live\test-quarto-live-2\index.qmd,
  could not find file C:\Users\hategeki\Desktop\CODE\R\Projets\WEBSITES\test-quarto-live\test-quarto-live-2\_extensions\r-wasm\live\_knitr.qmd.

Stack trace:
  in file C:\Users\hategeki\Desktop\CODE\R\Projets\WEBSITES\test-quarto-live\test-quarto-live-2\index.qmd,
  could not find file C:\Users\hategeki\Desktop\CODE\R\Projets\WEBSITES\test-quarto-live\test-quarto-live-2\_extensions\r-wasm\live\_knitr.qmd.
    at retrieveInclude (file:///C:/Users/hategeki/AppData/Local/Programs/Quarto/bin/quarto.js:39440:19)
    at standaloneInclude (file:///C:/Users/hategeki/AppData/Local/Programs/Quarto/bin/quarto.js:39476:11)
    at processMarkdownIncludes (file:///C:/Users/hategeki/AppData/Local/Programs/Quarto/bin/quarto.js:39706:38)
    at expandIncludes (file:///C:/Users/hategeki/AppData/Local/Programs/Quarto/bin/quarto.js:39735:11)
    at eventLoopTick (ext:core/01_core.js:153:7)
    at async projectResolveFullMarkdownForFile (file:///C:/Users/hategeki/AppData/Local/Programs/Quarto/bin/quarto.js:40370:25)
    at async fileExecutionEngine (file:///C:/Users/hategeki/AppData/Local/Programs/Quarto/bin/quarto.js:41840:26)
    at async findInputs (file:///C:/Users/hategeki/AppData/Local/Programs/Quarto/bin/quarto.js:92008:36)
    at async findChapters (file:///C:/Users/hategeki/AppData/Local/Programs/Quarto/bin/quarto.js:92037:13)
    at async bookRenderItems (file:///C:/Users/hategeki/AppData/Local/Programs/Quarto/bin/quarto.js:92040:5)

To get around this, I simply install the extension in another project and copy the _extensions folder into the quarto book folder. But when I generate the final book, I realize that the extension does not work, i.e. the {webr} boxes are not considered as usable code zones rather plain code.
Do you know why this problem occurs and how to fix it?
Thanks in advance!

@CyuHat could you double-check the version of Quarto you have? You may need to update RStudio (which has a built-in version of Quarto) or directly update Quarto itself:

https://quarto.org/docs/download/
https://posit.co/download/rstudio-desktop/

In addition, be sure that you are using the live-html version of the HTML format in your book's _quarto.yml file. e.g.
format:
  live-html:
    theme: cosmo
  pdf:
    documentclass: scrreprt

This kind of problem should probably be listed in some form of FAQ in the documentation.

Thank you both for your answers! I tried your solutions but it still does not work. Also I should have given more context. Here are the version I am working with:
R: 4.2.1
Rstudio: 2024.04.2
Quarto: 1.5.55
After updating quarto to the version 1.5.56, the problem persisted. In fact, since the begining it worked for single page html or quarto website, but it still do not works for quarto book. I have done two examples, one using quarto website (see here) and the other one using quarto book (see here). Looking at the final results, it works for quarto web site (see here) but does not work for quarto book (see here).
I kept all the default options for both cases, but here are the changes I have for the quarto website (note: there is no "" added to the original code chunks):
index.qmd
---
title: "test-quarto-live-1"
engine: knitr
format: live-html
---

{{< include ./_extensions/r-wasm/live/_knitr.qmd >}}

\```{webr}
fit = lm(mpg ~ am, data = mtcars)
summary(fit)
plot(fit)
\```

And here are the changes I have done for the quarto book version:
.github/workflows/quarto-publish.yml
- name: Set up Quarto
        uses: quarto-dev/quarto-actions/setup@v2
        with:
          version: 1.5.56
          tinytex: true

_quarto.yml
format:
  html-live:
    theme: cosmo
  pdf:
    documentclass: scrreprt

index.qmd
---
title: A Quarto Live Document
engine: knitr
format: live-html
---

{{< include ./_extensions/r-wasm/live/_knitr.qmd >}}

\```{webr}
fit = lm(mpg ~ am, data = mtcars)
summary(fit)
plot(fit)
\```

My concern is that I can't get the extension using the command in a quarto book project and after adding the folder manually, it still doesn't work.

Please could you double check the contents of your _quarto.yml file? The format should be live-html, but you seem to have html-live.

Yes, you are right, it is working now. Thank you for your patience and advices. It is really an amazing project!


Updating timestamp when no other changes are made

Example: https://github.com/mrjones2014/legendary.nvim/pull/337
There are no actual changes to the docs, but it still updates the timestamp. I seem to remember this being something that was fixed a while back?

Looking at https://github.com/mrjones2014/legendary.nvim/commits/master I can see commits that don't generate a new vimdoc, is this specific to this commit or every commit? Nothing was changed in the code, the only change recently was to update a deprecated treesitter call.

Actually you know what, I changed some nerdfont icons to the non-deprecated equivalents. Those probably generated a diff but GitHub didn’t see the difference so it appeared in the PR as though there was no diff.

Actually you know what, I changed some nerdfont icons to the non-deprecated equivalents. Those probably generated a diff but GitHub didn’t see the difference so it appeared in the PR as though there was no diff.

Ty for the update @mrjones2014!


Use AddMvcWithDefaultRoute()

Use the new app.UseMvcWithDefaultRoute() which includes the default route.

Hi @henkmollema, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Nice, I didn't realize the MVC team had added this in. I am happy we finally got back this ability, it made me sad when we removed it in one of the earlier betas :).

:smile:


"New Drink" builder should open with 1 ingredient by default

One of your last commits removed the default.

@sirtastic : not a bug. just part of the previous requirements. I can make this change tho

commit 895138bf54542f0ce32cbc92347e4293835ee62d
Date:   Wed Aug 26 10:07:35 2015 -0700
"New Drink" builder should open with 1 ingredient by default. Closes #76 (github)


Automation: App breaks when creating a new namespace from a workload

Based on work done in https://github.com/rancher/dashboard/issues/8632, let's write an automated test to confirm that nothing breaks when creating a new namespace from a workload.
@cnotv can offer guidance (or maybe he has a test written somewhere)
To Reproduce:

Access to Cluster explorer
Open any Workload section list, e.g. Jobs
Create a new Job
Create a new namespace instead of picking one

Expected behaviour:

Namespace is created and the UI doesn't break

We already have a test there: https://github.com/rancher/dashboard/pull/8638/files#diff-2b5e2a92db7594b42709ac907f4475c8ee31cb630213a03af52222903c7ffb3e


uploading activation map guide

Linked issue(s):
Fixes PRO-519
What change does this PR introduce and why?

Adds an advanced guide on how to upload activation maps on Kolena
Adds API reference doc for kolena.workflow.visualization utility functions

Please check if the PR fulfills these requirements

[ ] Include reference to internal ticket and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Relevant tests for the changes have been added
[ ] Relevant docs have been added / updated

Should we mention how to generate activation maps, and common methods? It would be useful to have e.g. "Grad-CAM" show up on this page such that users searching for these keywords will land here

It would be useful to have e.g. "Grad-CAM" show up on this page such that users searching for these keywords will land here

I thought of adding a section for "How to generate an activation map" but there are many methods out there and the implementation entirely depends on the model architecture so I couldn't put together a simple generic doc for the generation part.  But I can certainly mention "Grad-CAM" on the doc so it shows up on the search!

A quick listing of various methods and links to relevant docs or papers would be great!


Label rendered with animation

Describe the bug
After update to 1.4.0 the animation (slide from right to left) appeared during the label render.
Here is my layout props:
const gridProps = {
    allowOverlap: true,
    cols: 1,
    compactType: null,
    isBounded: true,
    isDraggable: false,
    isResizable: false,
    margin: [0, 0],
    maxRows: pageConfig.endTime - pageConfig.startTime,
    preventCollision: true,
    resizeHandles: ['s'],
    rowHeight: 40 / pageConfig.partsNumber,
    useCSSTransforms: false,
  };

Your Example Website or App
https://staging.bookey.ltd/en/room/64ef91222e216d4443d4fd54
Steps to Reproduce the Bug or Issue

Click on any date in Calendar.

Expected behavior
I want to have some option to disable this animation.
react-grid-layout library version
1.4.2
Operating System Version
windows
Browser
Chrome
Additional context
You can check how the 1.3.4 version behaves here https://bookey.ltd/en/room/5ee27e4f62dabf0012c41501
Screenshots or Videos

in 1.3.4
.react-grid-item {
    transition: all .2s ease;
    transition-property: left,top;
}

in 1.4.x
.react-grid-item {
    transition: all .2s ease;
    transition-property: left,top,width,height;
}

在1.3.4中
.react-grid-item {
    transition: all .2s ease;
    transition-property: left,top;
}

在 1.4.x 中
.react-grid-item {
    transition: all .2s ease;
    transition-property: left,top,width,height;
}

Maybe you can add className.


MUMBLE_SERVERPASSWORD is not saved correctly

I set the environment variable MUMBLE_SERVERPASSWORD in my docker-compose.yml. After that I cannot login with that password. When I go into the running container and do cat /etc/murmur/murmur.ini I can see that the actual password is set to: abcdef#serverpassword=ghi where abcdef&ghi is my actual password set.
When I remove the& from the passsword it's working. So is that a character that isn't allowed in the password itself?

I have submitted a PR for this issue to be merged
#12

The & wasn't escaped into the sedcommand. This commit fix the issue.
We will check to create a fix release for that


add callback for function proxyServer

Hi, just add callback for function proxyServer. So can start multiple proxy like this:
  console.log('-----------------------------------------------------------')
  proxy.proxyServer(options1, function() {
    console.log('-----------------------------------------------------------')
    proxy.proxyServer(options2, function() {
      console.log('-----------------------------------------------------------')
    })
  })

Hi,
Thanks for contribution !
May be there should be more parameters to the callback function? Otherwise, users couldn't find out whether the proxy server is launched successfully.
e.g.
fucntion callback(err, proxyServer){
    // ...
}

Hi @callmewhy
Good catch, thanks for pointing it out:)
We cannot ensure that it's really completed when the final function is called, as the previous callbacks are called without knowning the server status.
So basically, we need to refactor it from the begin, to only trigger the callbacks when the server status is ready. And also, as @ottomao mentioned, it's better to include some parameters to the final cb.
BTW, it would be helpful if you can accept the CLA Licence, thanks!


Hyperkit leaks

Good afternoon,
I noticed(and could read it in other issues) that Hyperkit is having trouble freeing up memory when it have been allocated by the linux kernel which annoys me big time. Do any tickets exists for fixing that issue? Or tickets in underlaying projects that might lead to the issue being fixed in Hyperkit? Or basically anything that would get me closer to taking a stab at fixing it?

Sounds from your terse description like you are referring to something like the issues described in https://github.com/docker/for-mac/issues/178 and https://github.com/docker/for-mac/issues/781? That is the OSX process is actually using that memory (as guest RAM) and has not "leaked" in the normal sense.
The first comment on the former is basically an accurate summary of where we are right now with this. Only if you are seeing either actual memory leaks (that is, memory which the hypervisor has actually lost track of) or memory allocated to the hyperkit process which is unswappable (which would manifest as the RSS remaining high even when the VM is idle and there are other memory pressures in the system) would we currently consider this an issue. If you are aware of either scenario occurring then we would be grateful of the details.
For those who are concerned about swappable memory allocated to virtual machines (i.e. the VSZ of the process) the usual answer is ballooning, which in this context would start with an implementation of the virtio-balloon backend in hyperkit. However ballooning functionality in the hypervisor is only a building block and other higher level code would also be needed to manage this functionality in a way which did not compromise functionality/stability/reliability of projects which use hyperkit (or to put it more concretely and avoid giving false hope/commitment: adding ballooning functionality in hyperkit would not automatically mean it would be activated by Docker for mac, it would need careful evaluation and integration work by the docker for mac team first).
We currently don't have any plans to work on anything related to this but if, given the caveats expressed here, you are still interested then PRs to implement virtio-ballooning in hyperkit would be welcome.

@ijc25 I've had issues when memory appears to be full docker quits responding to various commands (like ps or being able to download images). The only way to fix it is the restart the process so memory goes back down.

@jhorwit2 Please open a new ticket against docker/for-mac about the issue you are seeing including a diagnostic bundle upload etc. Easiest way to do that is to use the "Diagnostics & Feedback" item in the Whale menu.
Closing this issue since there has been no further response from the OP.

Ive noticed this happening to me sometimes when i've accidentally tried to interact with a really large image.  Restarting docker, and clearing out images doesn't seem to help.  Doing a factory restart on docker does.  I think it tries to access some of the same volume information between restarts.

@kyprifog please file an issue on docker/for-mac unless you are certain that this is an issue with hyperkit itself. If you think it is hyperkit itself, please provide more details.


Value Error Read Raw EGI Files

Hi all, I am trying to read the raw egi file can anyone help me with this error. I am pretty new to this and I dont even know if I am going in the right direction.

Can you upload the file somewhere so we can look? Do you get this error if you use an EGI reader using the official EGI software?

I will have to ask my team if I can share the file or not since its not mine. We dont have NETSTATION available to us now and due to COVID its difficult to get it. Is there any other EGI reader?

You could try http://biosig.sourceforge.net/ - the main toolbox is for MATLAB, but there is also a Python port available. I think BioSig can read EGI: http://pub.ist.ac.at/~schloegl/biosig/TESTED

Thanks for the suggestion I will try this out. Any other suggestions?


Macro: R3 and R4 connection points shown as R2 and R3 on monomer preview tooltip and connection points are missing for (R1,R3,R4) peptide

Steps to Reproduce

Switch to Macromolecules view
Load this file: (R1,R3,R4).zip
Hover mouse over peptide

Actual behavior

R3 and R4 connection points shown as R1 and R2 on monomer preview tooltip
System shows R2 connection points instead of R3 and R4 ones
Expected behavior
R3 and R4 connection points shown on monomer preview tooltip
System R3 and R4 connection points
Desktop (please complete the following information):

OS: win10
Browser Chrome 120.0.6099.110 (Официальная сборка), (64 бит)

Ketcher version [e.g. v2.4.2].

Verified.


add animation-in and animation-out props to modal

animationOut property is working fine. but animationIn property is not working as expected.


Stock guestbook app crashes when running as uberjar

I am following the Guestbook tutorial.
I create the app:
% clojure -Tclj-new create :template io.github.kit-clj :name kit/guestbook
Downloading: io/github/kit-clj/lein-template/maven-metadata.xml from clojars
Generating kit project.

I package it:
% clj -Sforce -T:build all
Cleaning target
Writing Pom...
Compiling Clojure...
Making uberjar...

I start it:
% java -jar target/guestbook-standalone.jar
725   INFO  kit.guestbook.env -
-=[guestbook started successfully]=-
726   INFO  kit.config - Reading config system.edn
788   INFO  org.xnio - XNIO version 3.8.8.Final
876   INFO  org.jboss.threads - JBoss Threads version 3.5.0.Final
930   INFO  kit.edge.server.undertow - server started on port 3000

I make a request:
% curl -v http://localhost:3000/api/health
*   Trying [::1]:3000...
* Connected to localhost (::1) port 3000
> GET /api/health HTTP/1.1
> Host: localhost:3000
> User-Agent: curl/8.4.0
> Accept: */*
>
< HTTP/1.1 500 Internal Server Error
< Connection: keep-alive
< Content-Length: 0
< Date: Wed, 21 Feb 2024 03:15:27 GMT
<
* Connection #0 to host localhost left intact

In the JRE, I see this exception:
11455 ERROR io.undertow.request - UT005071: Undertow request failed HttpServerExchange{ GET /api/health}
java.lang.IllegalArgumentException: No implementation of method: :match-by-path of protocol: #'reitit.core/Router found for class: kit.guestbook.web.handler$fn__11575$router__11577

However, it works fine when started using (go) in the REPL.
I will try to attach target/classes/META-INF/maven/kit/guestbook/pom.xml.

Github doesn't support XML files, but here it is with the TXT extension.
pom.xml.txt

May have sth to do with
https://github.com/kit-clj/kit/pull/130
-Tobias
ons 21 feb. 2024 kl. 04:21 skrev Philip White @.***>:

Github doesn't support XML files, but here it is with the TXT extension.
pom.xml.txt https://github.com/kit-clj/kit/files/14353619/pom.xml.txt
—
Reply to this email directly, view it on GitHub
https://github.com/kit-clj/kit/issues/133#issuecomment-1955806327, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAADU5TG7MASEG2XLP74RVLYUVR4VAVCNFSM6AAAAABDSIXVQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJVHAYDMMZSG4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

You are right, I pushed a fix to git.
@yogthos , we may need a new release.
In the meantime, you can fix the issue by redefining :router/core in handler.clj with the following code:
(defmethod ig/init-key :router/core
  [_ {:keys [routes env] :as opts}]
  (if (= env :dev)
    #(ring/router ["" opts routes])
    (constantly (ring/router ["" opts routes]))))

Ah ok makes sense, just ping me when you've got a pr up and I can push a release out then.

Thanks -- making the suggested fix manually to my existing app solved the problem.
Does Kit support merging updates to templates into existing apps?

No. Once the app is generated from the template, only manual changes are possible.


[Bug]: Intermittent error of "You don't have permission to do that"

Why is this issue important?
Error messages are frequently coming up at various points in the workflows, including clicking "upload workbook" from main Upload page as well as going back to the main uploads page
Current State
Example of error after clicking on Upload Workbook button:

Refreshed the page and got this new error:

Refreshed again and it's back to normal.
This doesn't happen every time you click the Upload Workbook button, but has happened several times in about an hour of testing.
Expected State
Error messages should not be occurring when navigating to Uploads page and new upload page.
Implementation Plan
The following functions need to be added...
Relevant Code Snippets
No response

Not seeing this error anymore in staging, per testing from #282 , closing this out!


Floating Point Numbers comparison #203

Add's a beginner's guide to using floating-point numbers. Solves issue #203

@zoziha can you review this and merge the pr

@awvwgk  Could you please merge this PR? It has already been reviewed by zoziha. Merging it would be a great encouragement for me as a beginner in open source.


attempt to call local 'fn' (a table value)

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Announcement

[X] I have checked Breaking change announcement.

Minimal reproducible full config
if has('vim_starting')
  set encoding=utf-8
endif
scriptencoding utf-8

if &compatible
  set nocompatible
endif

let s:plug_dir = expand('/tmp/plugged/vim-plug')
if !filereadable(s:plug_dir .. '/plug.vim')
  execute printf('!curl -fLo %s/autoload/plug.vim --create-dirs https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim', s:plug_dir)
end

execute 'set runtimepath+=' . s:plug_dir
call plug#begin(s:plug_dir)
Plug 'hrsh7th/nvim-cmp'
Plug 'hrsh7th/cmp-buffer'
Plug 'hrsh7th/cmp-nvim-lsp'
Plug 'hrsh7th/vim-vsnip'
Plug 'neovim/nvim-lspconfig'
call plug#end()
PlugInstall | quit

" Setup global configuration. More on configuration below.
lua << EOF
local cmp = require "cmp"
cmp.setup {
  snippet = {
    expand = function(args)
      vim.fn["vsnip#anonymous"](args.body)
    end,
  },

  mapping = {
    ["<Tab>"] = cmp.mapping(function(fallback)
      if cmp.visible() then
        cmp.select_next_item()
      else
        fallback()
      end
    end, { "i", "s", "c" }),
    ['<CR>'] = cmp.mapping.confirm({ select = true })
  },
    sorting = {
      priority_weight = 2,
      comparators = {
        cmp.config.compare.exact,
        cmp.config.compare.locality,
        cmp.config.compare.recently_used,
        cmp.config.compare.score,
        cmp.config.compare.sources,
        cmp.config.compare.offset
      },
    },
  sources = cmp.config.sources({
    { name = "nvim_lsp" },
    { name = "buffer" },
  }),
}
EOF

lua << EOF
local capabilities = require('cmp_nvim_lsp').default_capabilities()

require'lspconfig'.cssls.setup {
  capabilities = capabilities,
}
EOF

Description
When I open nvim with some comparators defined in nvim-cmp, they are giving me error in the title.
Steps to reproduce
Open nvim with nvim -u cmp-repro.vim cmp-repro.vim and just insert any character. You will see an error in the bottom.
Expected behavior
I should be able to use all comparators.
Actual behavior
I get an error any time completion list is updated.
Additional context
❯ nvim --version
NVIM v0.10.0
Build type: Debug
LuaJIT 2.1.1713484068
Run "nvim -V1 -v" for more info

I have the same problem was able to fix by rolling back the latest neovim update.  But it looks like the for_each_child function is being deprecated not positive this is causing the issue but looks like it.

Rolling back to latest? I didn't get that. Which version of nvim are you using? What is the version of this plugin?

Yeah I could have worded that better the latest version of nvim is v0.10.0 I rolled back to v0.9.5-6 and it’s fixed this issue for me.  I am not certain haven’t looked into it yet will do it this weekend but the for_each_child function is being deprecated I think that’s what is causing the issue I’ll play around with it this weekend and see if I can fix the issue if I can I’ll do a pull request and let you know but for now rolling your nvim version back should work.

Several of the items in the compare module are tables with __call metamethods rather than simple functions. The recently_used comparator is one of those.
Commenting that one out in my comparators list fixed the problem for me.

Commenting that one out in my comparators list fixed the problem for me.

Yeah, of course. Just another "workaround". I don't want to downgrade neovim, I don't want to "hide" the problem by commenting it out. I want it to be fixed. That's why I opened this. In fact, I already know the problem is related with comparators.

When I open nvim with some comparators defined in nvim-cmp, they are giving me error in the title.

Instead of suggesting workarounds please upvote the issue so that it can get some attention from developers.

OK.  I have reproduced.

It seems misc.merge() is broken.
misc.merge = function(tbl1, tbl2)
  local is_dict1 = type(tbl1) == 'table' and (not islist(tbl1) or vim.tbl_isempty(tbl1))
  local is_dict2 = type(tbl2) == 'table' and (not islist(tbl2) or vim.tbl_isempty(tbl2))
  if is_dict1 and is_dict2 then
    local new_tbl = {}
    for k, v in pairs(tbl2) do
      if tbl1[k] ~= misc.none then
        new_tbl[k] = misc.merge(tbl1[k], v)
      end
    end
    for k, v in pairs(tbl1) do
      if tbl2[k] == nil then
        if v ~= misc.none then
          new_tbl[k] = misc.merge(v, {})
        else
          new_tbl[k] = nil
        end
      end
    end
    return new_tbl
  end

  if tbl1 == misc.none then
    return nil
  elseif tbl1 == nil then
    return misc.merge(tbl2, {})
  else
    return tbl1
  end
end

misc.merge brokes metatable.  __call is removed.  So it will be error.

I have fixed it in #1956.  Please test.

@sahinakkaya cmp.config.compare.sources does not exist. I think you can fix the error by removing nil-containing lists from your config.
local comparators = {
    cmp.config.compare.exact,
    cmp.config.compare.locality,
    cmp.config.compare.recently_used,
    cmp.config.compare.score,
    cmp.config.compare.sources,  -- == nil
    cmp.config.compare.offset
}
assert(vim.islist(comparators) == false)

misc.merge() detects dict-like tables with vim.islist() and merges them, so comparators field (list-like table) and its elements can not to be merged.
But if you put nil in the list-like table, vim.islist() returns false and then misc.merge() tries to merge comparators.

OK.  It seems configuration bug.
cmp.config.compare.sources does not exist(it is exists once?).
But I think nil check is needed for it...


Add support for arm32v6

I have a Docker container running Grocy on an original Raspberry Pi zero which has an arm32v6 processor on it. It would be useful to have support for BarcodeBuddy on this device also.
I have never built a container before, but I will give it a try. I will see what I can do and if I am successful, I will make a pull request.

I have decided to close this issue. It seems that this is more of a hassel than it is worth.


Feature request - "Image comparison log?"

Don't know if this already exists in some way and I have simply missed it. But when having the program sort out duplicates it gives the score, and what seams to be a link the the file it is comparing but not the file it is being compared too. Would it be possible to have this displayed? If so I would find it exceptionally useful.
Also could this information be putout to a specific log so that one could manually compare images.

Sorry for such a late response, I tweaked DebugOutput so that it outputs score comparisons. Unfortunately, the way the image comparison database works, all it does is store hashes that it compares to, no filenames or anything identifiable, so it's hard to tell what images it's comparing against in the output.
Are you on Windows? If you don't know how to build go projects, I can send you a build with it.
What I would recommend is to gather some very similar images, some almost identical images, and some very different images, and using a blank database see what scores they output and tweak as you want.
(lowest is around -109.7), Higher = less similar (does not really have a maximum, would require your own testing)

I am not sure if this was already tackled, i am using the following version:
Welcome to discord-downloader-go v1.6.3
main.go:58: [Version] discord-go v0.22.0 using Discord API v6
But if you attempt to download the same filename twice than it detects as a possible duplicate and does not download anything.
E.g.: upload to a channel that is monitored an "image.jpg" and let the program download it, after its been down upload a totaly different file with the same "image.jpg" name and it will drop:
 downloads.go:953: >>> SKIPPING FILE: Matching filenames, possible duplicate...
Is this image comparison part making this issue go away? or its something so far noone had?