Polygon labels not saved & error on the setup page

Describe the bug

Labeled tasks are not shown and not saved on disk
The setup page shows an error I can't quite grasp after I've labeled an image

You've already completed some tasks, but some of them couldn't be loaded with this config: name=img, toName=img, type=polygon are expected

To Reproduce

Setup a minimal Polygon interface

<View>
  <Polygon name="poly" toName="img" value="Add Rectangle"></Polygon>
  <Image name="img" value="$image"></Image>
</View>

Import a dataset per .csv upload

image
https://images.unsplash.com/photo-1516383740770-fbcc5ccbece0
https://images.unsplash.com/photo-1516383740770-fbcc5ccbece0
https://images.unsplash.com/photo-1516383740770-fbcc5ccbece0

Label the image by creating a 4 point BB
Click Submit

Expected behavior

1 task is shown as completed
no errors on the setup page

Environment (please complete the following information):

OS: Server: Ubuntu - Browser: macOS 10.14.6
Browser Chrome 79

EDIT: This problem only seems to appear with the Polygon object and not with the PolygonLabel object. However, since I only have 1 class, I dont need the labels option.

Hi, @pietz ! This error should be fixed since v0.4.7

Please reopen this issue if the problem remains

I have error in the setup page
You've already completed some tasks, but some of them couldn't be loaded with this config: name=max_value, toName=max_value, type=textarea are expected
How to fix it?

@Vikt0riia Where do you use LS? Is it app.labelstud.io? Or you use it locally.

@makseq locally
I annotate one image. Then I understand, that I forgot some objects to add. I wanted add then, but error appears.

@makseq Do you know how to solve?
Because we are changing continuously: add/delete objects and each time I create new project as I cant modify setup at all.

@Vikt0riia what ls version do you use?

@makseq 0.7.4.post0

What setup changes do you do?

I used conda in windows

No, I mean what changes do you want to do on setup page?

I would like to add some new items.
For example, I had this:

And I would like to add:

It's specially limited LS behavior because you can

Nevertheless you can do some tricks to avoid this behavior

@makseq Ok thanks a lot !

@Vikt0riia you can try to change labeling config.xml at the project directory via filesystem. But remember you can corrupt your labeling.


tic tac toe, awesomeified

@cbas @sevresbabylone, here's my long overdue improved tic tac toe code.
Original code review @ https://github.com/peonicles/wdi-sg-playground/issues/1

:+1: Nice! Love the refactored functional code.


[Fix #10656] Mark Style/RedundantInterpolation as unsafe autocorrection

Fixes #10656.
This PR marks Style/RedundantInterpolation as unsafe autocorrection because when calling a destructive method to string, the resulting string may have different behavior or raise FrozenError.
x = 'a'
y = "#{x}"
y << 'b'   # return 'ab'
x          # return 'a'
y = x.to_s
y << 'b'   # return 'ab'
x          # return 'ab'

x = 'a'.freeze
y = "#{x}"
y << 'b'   # return 'ab'.
y = x.to_s
y << 'b'   # raise `FrozenError`.

Updating the auto-corrected code from 'a'.to_s to 'a'.to_s.dup will get safe, but it will make dup redundant when unneeded. And This would go against the cop's name "redundant". That means replacing it with a different redundancy that cannot be removed by static analysis. So I decided to mark it as unsafe.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!


Feature: Cycle tabs via mouse scrolling

What feature or improvement do you think would benefit Files?
Switch tabs using the mouse wheel while hovering over the tab bar.
Requirements
Implement mouse wheel actions over the tab bar, I guess.
Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.2428
Comments
No response

Thanks for the feedback, I've added this to the project board.

opps  I just posted this also, sorry about that


minSdkVersion 16

Hi
I'm maintaining an app that supports back to Android SDK 9 (we actually just dropped 8 for our current release a couple of months back). I was looking for a compact non-fancy number picker that we could use. However you've got minSdkVersion set to 16 even though it doesn't seem necessary from a short glance at the code (I actually recompiled with minSdkVersion 9 and tested on a real 2.3 device without problems).
Any chance you would consider lowering minSdkVersion ? I'm trying to avoid forking/rolling my own.
Thanks

Hi @simonpoole
I'm going to accept it if you open a pr about it.
also I saw new features in your fork, It'd be great to merge them with this project

hi @travijuu I'm not particularly in love with maintaining my own fork (just needed to have a version somewhere that people could use for builds), no guarantees before Christmas but I'll try :-)


Feature Request: /embed support of images vector embedddings

It would be great to add support for models that are focused on image (or image+text) vector embeddings !

https://github.com/ollama/ollama/issues/5304

closing as dup #5304


docs: fix typos

Description
Fix typos in docs

Sorry, but we're not currently merging typo fixes from external contributors.


Adding clangd language server files to .gitignore

Adding some ignore entries for files generated by the clangd language
server for editors that use it.

Marking for the various supported releases since this is could be helpful for anyone editing files in those releases while it is low risk since it's not a product-impacting change.

If we want this for 8.x can we get a backport?

If we want this for 8.x can we get a backport?

Sure thing:
https://github.com/apache/trafficserver/pull/8652

Cherry-picked to v9.2.x


I have switched to storing the password in the keychain, instead of passing it every time as a command line parameter. I've noticed this is not working, because it's passing down an instance of GenericPassword instead of the password itself.

Thanks, @plu! A new release is forthcoming.


[JS] 0.9 .prompt file issues

Running into a number of issues using .prompt files in 0.9 when used as agent tool. Documenting list here.

[x] Can't use a named .prompt as a tool without loading it elsewhere first.
[x] Even when using it, seems to be incorrectly namespaced under dotprompt/{name}
[x] Even when working around those issues, @state is not available to a delegated prompt
[x] The description of prompts is not loaded when used as a tool

Another issue: Model googleai/gemini-1.5-pro-latest not found

Another issue: Model googleai/gemini-1.5-pro-latest not found

Try googleai/gemini-1.5-pro


Fails to build an executable "test" from test.cpp

I recently had this identical issue:
http://answers.ros.org/question/201217/catkin-doesnt-create-test-executable/
There should at least be an error message if there's a naming conflict.

The CMake code mentioned in the question invokes:
add_executable(test ...)

catkin is not involved in this at all so it can't do anything about it. This is a plain CMake problem and has been addressed in CMake 3.0 (https://cmake.org/cmake/help/v3.0/policy/CMP0037.html).


Screenshot is old for Data Source settings

Issue in help/using/features/traits/client-activity-synced-audience-traits.md

The screen shot displayed on the help page does not match with the application anymore.

Current Image: https://docs.adobe.com/content/dam/help/audience-manager.en/help/using/features/traits/assets/datasource_synced.png
Expected:

Thanks for reporting this, Ankur!
We've updated the screenshot and it should be visible on the page later today.
Looking forward to even more contributions from your side.
Have a great day!
Vlad Falon
Audience Manager Senior Technical Writer


FinOps tooling in our catalog (help customers save costs)

Goldielocks saved money for customers because their apps were better sized afterwards and clusters scaled down. Other customers have used KEDA to scale down their applications. Since we added KEDA already this could be another opportunity to promote our managed apps to all customers and make them happy by easily saving some costs.
The bigger picture to connect both lose ends:
@foschubert @yasn77 and @pipo02mix are currently thinking about a bigger campaign to help a customer save costs. And imo it would be good to accompany that with a few tools in our app catalog. KEDA and goldielocks are two already. We would need to create a list of useful tools and decide which we can add easily to our catalog.

How can product support account and solution engineering in such a campaign?
Which tools would be most useful?
Promote the tools we have on the website
Write general docs about cost saving

Details, Background
Related issues:

https://github.com/giantswarm/roadmap/issues/267
https://github.com/giantswarm/roadmap/issues/275

/cc @cokiengchiara

thanks Timo! I understand Franz, Yasser, and Fer are leading this effort, right? I would happy to just stay in the loop.

@cokiengchiara yes but I think you and Halo need to sit down with them and discuss which tools are of general use for customers and how Halo can accompany such a campaign.

I am now focused on Write general docs about cost saving, I will make sure all apps I use are in the catalog

I am now focused on Write general docs about cost saving, I will make sure all apps I use are in the catalog

thank you @pipo02mix you're the best

I am now focused on Write general docs about cost saving, I will make sure all apps I use are in the catalog

Thank you @pipo02mix . @henninglange and @foschubert would be interested in written in https://github.com/giantswarm/giantswarm/issues/16412

I think it would be great to better understand how we save our customers cloud costs by using Goldilocks, Keda, Spot, etc. in which Cases do they work? How much can we save customers or how much do they save themselves by using them?


MPI-related test failure using mpich 4.2.0, gcc 13.2.0

Update: For clarity, the tests pass when using mpich 4.0, gcc 11.4.0.

I'm observing a failure using mpicc and running nc_test4/run_par_test.sh.
This issue occurs when running mpicc version 13.x, but does not occur on systems using mpicc version 11.x.  This is most easily observed on my end using Ubuntu 22.04 vs. 24.04.  I've created a couple of docker images which can be used to observe this.  They can be run as follows:
$ docker run --rm -it docker.unidata.ucar.edu/h5par:2204

and
$ docker run --rm -it docker.unidata.ucar.edu/h5par:2404

You can enter the environment by appending bash to the end of either docker command.
It seems that the issue is related to the different version of mpicc, but I'm trying to sort through what exactly is going on.  Any suggestions would be appreciated.
The error specifically is as follows:
153: Testing simple parallel I/O with 16 processors...

153:

153: *** Testing more advanced parallel access.

153: *** Testing parallel IO for raw-data with MPI-IO (driver)...

153: *** Testing more advanced parallel access.

153: *** Testing parallel IO for raw-data with MPI-IO (driver)...

153: *** Testing more advanced parallel access.

153: *** Testing parallel IO for raw-data with MPI-IO (driver)...Sorry! Unexpected result, /root/hdf5-1.14.3/netcdf-c/nc_test4/tst_parallel3.c, line: 284

153: Sorry! Unexpected result, /root/hdf5-1.14.3/netcdf-c/nc_test4/tst_parallel3.c, line: 91

153:

153: *** Testing more advanced parallel access.

153: *** Testing parallel IO for raw-data with MPI-IO (driver)...

153: *** Testing more advanced parallel access.

153: *** Testing parallel IO for raw-data with MPI-IO (driver)...Sorry! Unexpected result, /root/hdf5-1.14.3/netcdf-c/nc_test4/tst_parallel3.c, line: 284

153: Sorry! Unexpected result, /root/hdf5-1.14.3/netcdf-c/nc_test4/tst_parallel3.c, line: 91

153:
(...)

@edwardhartnett @jhendersonHDF if anything leaps out at you, feel free to chime in, it might save some time as I dig through this! And if not, no worries XD. Thanks!

Additional notes:
On ubuntu 24.04, installing libhdf5-mpi-dev installs openmpi and related tools.  This version of libhdf5 works just fine, although the nc_test4/run_par_test.sh script requires --oversubscribe be passed to mpiexec -n 16 ./tst_parallel3.  Otherwise, there is a complaint if the machine has < 16 cores/processors/what-have-you.
Using mpich and a custom-built libhdf5, we cannot oversubscribe. However, this is not an issue, because invoking mpiexec -n 2 ./tst_parallel3 results in the same issue as if we passed 4, or 8, or 16.  Running tst_parallel3 directly works, but of course it is bypassing MPI entirely.
Installing libhdf5-mpich-dev sees the same behavior as using the custom-built version of libhdf5. This suggests there is an issue when using mpich but not inherently MPI.


Releasing page layout version 2.1.2 for UnifiedSSP and SelfAsserted page.

@stevenxzhou : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@stevenxzhou : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @ktoliver, May I ask how to grant permission to Sasha or me for merging this PR?

Hi @stevenxzhou - I can merge the PR. We merge pull requests on a schedule before the twice daily publishing (weekdays).
If your question is overall about merge permissions, you could talk to the repo admin Cory Fowler.
Thanks!


Change App startup procedure.

Replace addCharger calls from frontend part to the dedicated query. Query must be executed right after truffle migrate --reset

@snumbrikvolgo @MaximMorozovMIPT PTAL


Execution hangs when browser window goes to background

Steps to reproduce

Puppeteer version: 1.6.2
Chromium: 69.0.3494.0 (Developer Build)
Platform / OS version: macos sierra
Node.js version: v10.8.0

What steps will reproduce the problem?

Run the following script. It opens google and submits form, repeating 10 times.

const puppeteer = require('puppeteer')

main()

async function main() {
  const browser = await puppeteer.launch({
    args: [
      // all related chromium flags I could find
      '--disable-background-timer-throttling',
      '--disable-renderer-backgrounding',
      '--override-plugin-power-saver-for-testing=never',
      '--disable-extensions-http-throttling',
    ],
    headless: false
  })
  const timeout = 5000
  const page = await browser.newPage()
  for (let i = 0; i < 10; i++) {
    console.log(`navigating ${i}`)
    await page.goto('https://google.com', {timeout})
    const input = await page.$('input[type="text"]')
    await input.type('test')
    await Promise.all([
      page.waitForNavigation({timeout}),
      page.evaluate(() => document.querySelector('input[type="submit"]').click())
    ])
  }
}

Switch to any other fullscreen-size window, so puppeteer browser window is not visible.

What is the expected result?
Scripts proceeds till end without errors.
What happens instead?
UnhandledPromiseRejectionWarning: Error: Navigation Timeout Exceeded: 5000ms exceeded
Some explorations
This happens only when puppeteer window is not visible at all. When even couple of pixels of this window are visible everything works fine.

Your timeout isn't sufficient
Ex. Increase it to some large value like const timeout = 30000
OR
just remove timeout and it will work perfect without any errors
UnhandledPromiseRejectionWarning: Error: Navigation Timeout Exceeded: 5000ms exceeded => means naviagtion isn't completing before 5000ms

I have default timeout (30s) in production app where I've got this error initially. I've set lower value to not wait long when was trying to localize the problem. Though when I test now with timeout: 30000 it freezes until I switch to puppeteer's browser and only then throws Navigation Timeout Exceeded: 30000ms exceeded. Same is when I remove timeout option.

script (code snippet) you gave ran successfully when I added the timeout = 30000
but I realized one more thing that if your internet connection is too low, it will throw error even after timeout = 30000

This does NOT depend on internet connection. I run this script when the browser is ON TOP of other windows and it finishes fine and quick. Please make sure you understand that problem appears only when puppeteer browser is fully hidden under other windows. The better option is to switch to any other app with expanded window. If this does not happen to you anyway, please share your OS name and version.

Ok...I will look again
On Fri, Aug 31, 2018, 4:19 PM printercu<EMAIL_ADDRESS>wrote:

This does NOT depend on internet connection. I run this script when the
browser is ON TOP of other windows and it finishes fine and quick. Please
make sure you understand that problem appears only when puppeteer browser
is fully hidden under other windows. The better option is to switch to any
other app with expanded window. If this does not happen to you anyway,
please share your OS name and version.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GoogleChrome/puppeteer/issues/3156#issuecomment-417627431,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWIcQMO4EVQpJ_a6TtFHmZmOvwjXkVEYks5uWRTAgaJpZM4WTF1R
.

With 30s timeout, make sure to wait a least 1min before switching back to puppeteer's window. Or just use lower timeout values. 5s is enough for me, because google opens in less than 2s.

I've just tested and found that this error appeared in 1.6.0 and still present in 1.7.0. It was ok in 1.5.0.

I was just about to comment the same. It's working okay with less than
v1.5.0
On Fri, Aug 31, 2018, 11:58 PM printercu<EMAIL_ADDRESS>wrote:

I've just tested and found that this error appeared in 1.6.0 and still
present in 1.7.0. It was ok in 1.5.0.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GoogleChrome/puppeteer/issues/3156#issuecomment-417752128,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWIcQMcPB_1gx1V-U77koZicoq2Zn_g7ks5uWYBFgaJpZM4WTF1R
.

Could this be linked to power-saving functions of Chrome? I noticed that when a little part of the browser window is visible, even when in the background, then execution is running. But as soon as the window is fully covered, execution seems to halt completely.

@printercu The symptoms point to AppNap feature in Mac OS X. I'd try disabling it and see if it helps.

@aslushnikov yes I think that's exactly what I described above... I didn't know it was called AppNap. It can be disabled quite easily, see: https://www.lifewire.com/control-how-mac-applications-use-app-nap-2260788

I'm fairly certain #3148 is a duplicate of this issue

Ok, so I did check for the "App Nap" option, but it doesn't show on Chromium.
And in the article linked to above it says:

If there's a Prevent App Nap checkbox present, you can put a check mark in the box to prevent naps, or remove a check mark to allow naps. If there's no checkbox, then the app isn't App Nap aware.

🤔

Can confirm the issue on Windows 10 with Chrome 72.0.3626.81. Mine freezes when attempt to click on element and chrome windows is in the background.  Is there are any workaround?

Any solutions for that? This is just so weird.

Confirm issue on linux too

Ok so before we thought this is an APpNap behavior specific to Mac, but  I see there are now reports from Linux and WIndows as well.
Everybody: can you please share some script we can try running to see this happening?

Ok so before we thought this is an APpNap behavior specific to Mac, but I see there are now reports from Linux and WIndows as well.
Everybody: can you please share some script we can try running to see this happening?

Not bear pc right now . But I think any script which do some will freeze when chrome windows is no visible . For me it always freeze on click and maybe some other operations

@aslushnikov I can not reproduce the issue on macos mojave with my snippet from https://github.com/GoogleChrome/puppeteer/issues/3156#issue-355496863.
I've tried both old 1.5 and new puppeteer.

I can not reproduce the issue on macos mojave with my snippet from #3156 (comment).
I've tried both old 1.5 and new puppeteer.

Ok so if this strikes back - please feel free to create a new issue with a script that hangs for you when you minimize browser window. We'd be happy to look into that.
Closing this for now.

@aslushnikov @printercu Please reopen this annoying bug.
Here is code to reproduce it.
The issue is in IntersectionObserver which is user in _scrollIntoViewIfNeeded function which is used in Click.
Enviroment:
OS: Windows 10
puppeteer: 1.19.0
const puppeteer = require('puppeteer');

(async () => {
	var baseTime=Date.now();

  const browser = await puppeteer.launch({
    headless: false,
   //executablePath :"C:/Program Files (x86)/Google/Chrome/Application/chrome.exe"
  });
  const page = await browser.newPage();
  await page.goto('https://github.com/');
  page.on('console', msg => {
  for (let i = 0; i < msg.args().length; ++i)
    console.log( Date.now()-baseTime+' Console: '+` ${i}: ${msg.args()[i]}`);
});

  console.log('Please minimize chrome');
  process.stdout.write('\x07');
  await sleep(5000);
   var joinNode=await page.$('a[href *= join]');
    console.log(Date.now()-baseTime+' Before  _scrollIntoViewIfNeeded');
   await _scrollIntoViewIfNeeded(joinNode,true);
   console.log(Date.now()-baseTime+' after  _scrollIntoViewIfNeeded');
  await browser.close();

  function sleep(ms){
    return new Promise(resolve=>{
        setTimeout(resolve,ms)
    })
}
  async function  _scrollIntoViewIfNeeded(element,pageJavascriptEnabled) {
    const error = await  element.executionContext().evaluate(async(element, pageJavascriptEnabled) => {
      if (!element.isConnected)
        return 'Node is detached from document';
      if (element.nodeType !== Node.ELEMENT_NODE)
        return 'Node is not of type HTMLElement';
      // force-scroll if page's javascript is disabled.
      if (!pageJavascriptEnabled) {
        element.scrollIntoView({block: 'center', inline: 'center', behavior: 'instant'});
        return false;
      }
	  	console.log('before IntersectionObserver');
      const visibleRatio = await new Promise(resolve => {
        const observer = new IntersectionObserver(entries => {
			console.log('IntersectionObserver callback (ISSUE IS HERE!!!!!!)');
          resolve(entries[0].intersectionRatio);
          observer.disconnect();
        });
        observer.observe(element);
      });

	  console.log('after IntersectionObserver');
      if (visibleRatio !== 1.0)
        element.scrollIntoView({block: 'center', inline: 'center', behavior: 'instant'});
      return false;
    }, element, true);
	  console.log('error: '+error);
    if (error)
      throw new Error(error);
  }
})();

After windows beep sound and message in console 'Please minimize chrome'- please minimize chrome window.
My log is

Please minimize chrome
7134 Before  _scrollIntoViewIfNeeded
7138 Console:  0: JSHandle:before IntersectionObserver
22261 Console:  0: JSHandle:IntersectionObserver callback (ISSUE IS HERE!!!!!!)
22265 Console:  0: JSHandle:after IntersectionObserver
error: false
22269 after  _scrollIntoViewIfNeeded

I waited 15 seconds before restore chrome window and immediately after this I received message 'IntersectionObserver callback (ISSUE IS HERE!!!!!!)'
But this ossie occur not only when window is minimized, also when windows is in background
Please fix it. Because it is really anoying issue.

[Solution]:
Add the following values to the array you assign to the args property in the options object. Now, you should be able to run Puppeteer while your window is in the foreground, background, or minimized.
const chromeArgs = [
  '--disable-background-timer-throttling',
  '--disable-backgrounding-occluded-windows',
  '--disable-renderer-backgrounding'
];

Solution and explanation available in an article written by Rohit Kumar, @rkr090 at Big Binary:
Debugging failing tests in puppeteer because of background tab


26: Add defaultValue prop

Codecov Report

Merging #27 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #27   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines          82     84    +2
  Branches       20     21    +1
=====================================
+ Hits           82     84    +2

Impacted Files
Coverage Δ

src/DurationInputMask.js
100% <100%> (ø)
: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 d9d4bdd...8d3abff. Read the comment docs.


DSBugBash: Imported notebook creates python file with lot of extra white space

Create a python file with the following code:

#%%
print("one")
#%%
print("two")

Run all cells
Save as notebook
Open notebook in VSC and the resulting Python file has lots of extra whitespace

#%%
print("one")

#%%
print("two")

@rchiodo is this fixed already?

I think it's fine the way it is. We can revisit when we get further feedback.


Cannot install latest releases using pip

Hi  thanks for fixing the couple of opened issues it will save the day !
Following issue I tried starting from a fresh install running pip install abracadabra==0.0.5  upgrading pip to the latest version as a prerequisite. However it seems pip cannot find the latests realease ( I tried for all version starting from 0.0.2) .
So I tried running pip search abracadabra which returns abracadabra (0.0.5)  - Making hypothesis and AB testing magically simple! which is a good sign.
So I tried again pip install abracadabra==0.0.5 -vvv to get a traceback. It looks like the version are not properly named as they are prefixed with the os you're seems to be using.
See :
Fetching project page and analyzing links: https://pypi.org/simple/abracadabra/ Getting page https://pypi.org/simple/abracadabra/ Found index url https://pypi.org/simple Starting new HTTPS connection (1): pypi.org:443 https://pypi.org:443 "GET /simple/abracadabra/ HTTP/1.1" 200 987   Found link https://files.pythonhosted.org/packages/66/71/08dc217003fbdca1e41f51c6cf4552eeb2de8302dd57ac04ad0385d5ca97/abracadabra-0.0.0-py3-none-any.whl#sha256=4030febb5e8ef77167edbd3278b67b926aa8d0db99fb9e40de9f9a2b9766fc8a (from https://pypi.org/simple/abracadabra/) (requires-python:>=3.7), version: 0.0.0   Found link https://files.pythonhosted.org/packages/07/87/e5d110901f799331f7a44ff3c21c2415f24ba7b5cefa11d82eda6a95e62e/abracadabra-0.0.0.tar.gz#sha256=5eeed338934a3dee3f4ee004b8143ea38e0ef1fc89d20eec156e8f5bcae9bd12 (from https://pypi.org/simple/abracadabra/) (requires-python:>=3.7), version: 0.0.0   Found link https://files.pythonhosted.org/packages/9b/01/d220895a4c4dc1c1e5cb03268186386db9e6be0f8899e98c5e4af3075551/abracadabra-0.0.1-py3-none-any.whl#sha256=b7419a9979a7e40ad741c7106da390d91793702fc25934ac433095dbaf613fa3 (from https://pypi.org/simple/abracadabra/) (requires-python:>=3.7), version: 0.0.1   Found link https://files.pythonhosted.org/packages/be/1d/0260508bfb3cd5df77645e8f31a5239c7f91328e79711e35c72a1a401f80/abracadabra-0.0.1.tar.gz#sha256=934b6b4eae2464617de5edc9520d7433602f3bb3b5d3c66dd0d2a9116e35a2d4 (from https://pypi.org/simple/abracadabra/) (requires-python:>=3.7), version: 0.0.1   Found link https://files.pythonhosted.org/packages/e0/2b/10b48197772fa2258769e4bb380e020ab9a253d4be091d91cad6a1eadd13/abracadabra-0.0.2.macosx-10.14-x86_64.tar.gz#sha256=3a27f22f64f7d18663723dae5abd8ec32d469720fd45ffa68293faa6199e19b6 (from https://pypi.org/simple/abracadabra/) (requires-python:>=3.7), version: 0.0.2.macosx-10.14-x86_64   Found link https://files.pythonhosted.org/packages/80/de/4c0a62d459466926f1ca550c22669a3eb968c94903854c0c045bb5720706/abracadabra-0.0.3.macosx-10.14-x86_64.tar.gz#sha256=073866a99037bcbd8a32f48c511957b0b67b6ffb4b16c6bfd40b06d43681a7ce (from https://pypi.org/simple/abracadabra/) (requires-python:>=3.7), version: 0.0.3.macosx-10.14-x86_64   Found link https://files.pythonhosted.org/packages/dd/84/993353040d5549de039597b00aafc9fd0681ee99e3077a96996d53cffdaa/abracadabra-0.0.4.macosx-10.14-x86_64.tar.gz#sha256=27e69fcf368d7b2b835c784887b1b34a318d7b2f3340e3d826141cffcc4fd71f (from https://pypi.org/simple/abracadabra/) (requires-python:>=3.7), version: 0.0.4.macosx-10.14-x86_64   Found link https://files.pythonhosted.org/packages/7b/8a/aab62563ebbbd014c3e80dc4c3cfd2b9491d28e2835f969405092ee5bd1d/abracadabra-0.0.5.macosx-10.14-x86_64.tar.gz#sha256=424aa28d3f5d7bdc26129f4e6e18764be07fb75f425a5762176575a1f6ed62b4 (from https://pypi.org/simple/abracadabra/) (requires-python:>=3.7), version: 0.0.5.macosx-10.14-x86_64 Given no hashes to check 0 links for project 'abracadabra': discarding no candidates ERROR: Could not find a version that satisfies the requirement abracadabra==0.0.5 ERROR: No matching distribution found for abracadabra==0.0.5
So I finally tried pip install abracadabra==0.0.5.macosx-10.14-x86_64 which fixes the issue and install the latest release. I think this is kinda unintuitive, I advise you maybe change the versions indexes .

Sorry about that @guillaume-chech . The wheel install for 0.0.5 is now available. I've verified that
$ pipenv install abracadabra==0.0.5

installs as expected on fresh virtualenv.


Problem: 'blocks_dir' nonexistence not handled (panics)

No need for a 'blocks_dir' to exist at all, especially when a database is present.

@Isaac-DeFrain I want to up-prioritize this, as it will help with the logic related to default directories. Is it possible for the indexer to treat the blocks_dir as optional?

Yes, we can start a fresh indexer without ingesting blocks from a directory

If we make the blocks dir optional, we will need a way to tell the indexer to start watching a directory for blocks while it's running

Currently, even when you sync or replay from an existing db, we use the blocks watch dir to continue ingesting any new blocks

Right, so the watch_dir now cannot have as its default the blocks_dir if the blocks_dir is unset. The watch dir can be whatever is the default directory for the logs of the Mina node, in that case, I propose.

We have the ability to choose that default in the Mina node since we implemented it. Should it be /usr/share/mina/blocks?

Does it make sense to have both directories as optional and when

blocks_dir set   => block_watch_dir defaults to blocks_dir
blocks_dir unset => block_watch_dir is the above default


Add option to typescript-graphql-files-modules to generate complete paths

Currently the modules generated for the files have a wildcard as path. Example: declare module '*/Project.gql' {
I would like an option to generate the files with the absolute path (src/graphql/queries/Project.gql). This way auto-imports will work correctly in VS Code

I implemented it in this PR: https://github.com/dotansimha/graphql-code-generator/pull/3010
With these changed, you should be able to set now relativeToCwd: true to generate a complete path to CWD, and you can customize the prefix (default for */).
@pietdevries94

Available in 1.9.0

@dotansimha
How can I set the option for typescript-files-modules?

@tyankatsu0105 For me it works using something like:
generates:
  src/types/graphql.d.ts:
    plugins:
      - 'typescript-graphql-files-modules'

    config:
      relativeToCwd: true

Would be awesome if these would generate files in-place, so that I'd get a .d.ts file for each document


Uncaught Error: ENOENT: no such file or directory, scandir 'atom://config'

[Enter steps to reproduce below:]

Try to open a REPL

Atom Version: 1.5.3
System: Mac OS X 10.11.3
Thrown From: proto-repl package, v0.15.2
Stack Trace
Uncaught Error: ENOENT: no such file or directory, scandir 'atom://config'
At fs.js:813

Error: ENOENT: no such file or directory, scandir 'atom://config'
    at Error (native)
    at Object.fs.readdirSync (fs.js:813:18)
    at Object.fs.readdirSync (ATOM_SHELL_ASAR.js:454:28)
    at LocalReplProcess.module.exports.LocalReplProcess.getRootProject (/Users/jmgimeno/.atom/packages/proto-repl/lib/local-repl-process.coffee:35:20)
    at LocalReplProcess.module.exports.LocalReplProcess.start (/Users/jmgimeno/.atom/packages/proto-repl/lib/local-repl-process.coffee:54:20)
    at Repl.module.exports.Repl.startProcessIfNotRunning (/Users/jmgimeno/.atom/packages/proto-repl/lib/repl.coffee:89:16)
    at Object.module.exports.ProtoRepl.toggle (/Users/jmgimeno/.atom/packages/proto-repl/lib/proto-repl.coffee:178:13)
    at atom-workspace.subscriptions.add.atom.commands.add.proto-repl:toggle (/Users/jmgimeno/.atom/packages/proto-repl/lib/proto-repl.coffee:86:32)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:260:29)
    at CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61)
    at CommandRegistry.module.exports.CommandRegistry.dispatch (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:160:19)
    at AtomEnvironment.module.exports.AtomEnvironment.dispatchApplicationMenuCommand (/Applications/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:1013:28)
    at emitOne (events.js:77:13)
    at EventEmitter.emit (events.js:169:7)

Commands
     -3:25.8.0 core:confirm (atom-text-editor.editor.mini.is-focused)
  5x -3:19 core:backspace (atom-text-editor.editor.mini.is-focused)
     -3:13.1.0 core:confirm (atom-text-editor.editor.mini.is-focused)
  8x -3:09.1.0 core:backspace (atom-text-editor.editor.mini.is-focused)
     -3:05.0 core:confirm (atom-text-editor.editor.mini.is-focused)
 10x -2:49.3.0 core:backspace (atom-text-editor.editor.mini.is-focused)
     -2:24 application:add-project-folder (span.name.icon.icon-file-directory)
     -1:39.8.0 proto-repl:toggle (ul.list-inline.tab-bar.inset-panel)
     -0:26.9.0 core:paste (atom-text-editor#proto-repl.leinPath.editor.mini.is-focused)
     -0:16.8.0 core:save (input#proto-repl.refreshBeforeRunningSingleTest.)
     -0:01.9.0 proto-repl:toggle (atom-text-editor.editor.is-focused)

Config
{
  "core": {},
  "proto-repl": {
    "leinPath": "/usr/local/bin/lein"
  }
}

Installed Packages
# User
Parinfer, v1.14.0
language-idris, v0.4.2
language-scala, v1.1.6
minimap, v4.19.0
proto-repl, v0.15.2
terminal-plus, v0.14.5

# Dev
No dev packages

I have the same issue. I'm using Debian Sid 64-bit.
I've upgraded to the latest version and unfortunately, the issue is also still present in v0.16.0
Stack trace:
Error: ENOENT: no such file or directory, scandir 'atom://config'
    at Error (native)
    at Object.fs.readdirSync (fs.js:813:18)
    at Object.fs.readdirSync (ATOM_SHELL_ASAR.js:454:28)
    at LocalReplProcess.module.exports.LocalReplProcess.getRootProject (/home/phillc/.atom/packages/proto-repl/lib/local-repl-process.coffee:35:20)
    at LocalReplProcess.module.exports.LocalReplProcess.start (/home/phillc/.atom/packages/proto-repl/lib/local-repl-process.coffee:54:20)
    at Repl.module.exports.Repl.startProcessIfNotRunning (/home/phillc/.atom/packages/proto-repl/lib/repl.coffee:88:9)
    at Object.module.exports.ProtoRepl.toggle (/home/phillc/.atom/packages/proto-repl/lib/proto-repl.coffee:178:14)
    at atom-workspace.subscriptions.add.atom.commands.add.proto-repl:toggle (/home/phillc/.atom/packages/proto-repl/lib/proto-repl.coffee:82:24)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/usr/share/atom/resources/app.asar/src/command-registry.js:260:29)
    at CommandRegistry.handleCommandEvent (/usr/share/atom/resources/app.asar/src/command-registry.js:3:61)
    at CommandRegistry.module.exports.CommandRegistry.dispatch (/usr/share/atom/resources/app.asar/src/command-registry.js:160:19)
    at AtomEnvironment.module.exports.AtomEnvironment.dispatchApplicationMenuCommand (/usr/share/atom/resources/app.asar/src/atom-environment.js:1013:28)
    at emitOne (events.js:77:13)
    at EventEmitter.emit (events.js:169:7)

What did you do that triggered the issue? Did you just toggle the REPL? What did you have open at the time?
It looks like a problem scanning for a project file.

I think the problem was that I has two different projects in the tree view. When there's only one open project it works fine.

These are my first steps with Clojure, so I've been following the book Clojure for the Brave and True. Instead of using Emacs as they suggest, I decided to use Atom instead.
I have a fresh installation of Atom 1.5.3, with a custom theme. I have proto-repl, proto-repl-charts and parinfer installed.
Steps:

File Menu > Add Project Folder
Navigate to src folder/project name, select .clj file
Packages Menu > proto-repel > start REPL

Then the error is thrown.
Screen shot attached.
I wouldn't be surprised if it was user error somewhere.....

I've fixed this. Once I closed the "config" folder/project, as seen in the screen shot above, based on feedback from @jmgimeno, everything now works fine.
Looks like the issues is certainly with having two projects open at once.

I just figured out and fixed the problem as well. It was only happening when the top folder open was the atom config folder. When it's open atom reports the root directory as "atom://.atom/config" This doesn't work with the standard node functions for paths. I skip trying to find the project.clj of a folder that looks like that.

Note that selecting "Start REPL" will look for a project.clj file at the root of the project. If you want to open a selected one like in the event you have multiple project folders, open the project.clj, then select "Start REPL from open project.clj" from the menu.


Paper Writing: Related works [probably finished]

I did a rewrite/correction.
@isaac-chung do you have the time to take a look at it to see if there is anything we need?

Looks great so far. Just took a look now and these came to mind:

Should we mention any work on the speedup/fast/lite tasks?
In addition to extending the number of languages, maybe we could mention what proportion of low-resource languages we are able to cover?

Should we mention any work on the speedup/fast/lite tasks?

There is some work on speeding up benchmarks, which we could def. cite. However, some of that will come as the reasoning for task selection, and the remainder I think is already partly in the introduction (feel free to update the last part of the introduction to match).
With both of these I don't believe we need a related works section on it. WDYT?

In addition to extending the number of languages, maybe we could mention what proportion of low-resource languages we are able to cover?

Hmm yes. I would probably add this to the limitations and/or discussion section. How much do we cover, what are we missing etc.

With both of these I don't believe we need a related works section on it. WDYT?

Agreed! Can omit.

Hmm yes. I would probably add this to the limitations and/or discussion section. How much do we cover, what are we missing etc.

That makes sense. That said, we do call out "low-resource languages" specifically in the abstract, intro and methods. It might be beneficial to clarify this unique contribution of the our work. Might be worth adding a short line on that, in addition to what you suggested as well?

Might be worth adding a short line on that, in addition to what you suggested as well?

Ahh right. We can add that in the text embedding benchmark section?

Yep, done.

Thanks - had a look at the section and added a few things - will you give another look?

Didn't notice any tracked changes, but added a few more suggestions in the section.

Hmm they should be there - e.g.:

Ahhh that's in Methods, not Related Works.

I will close this section for now.


[question] Why [ jwt::verify().allow_algorithm.verify() ] is void and make the runtime error exit?

Here is my code:
auto decoded = jwt::decode("eyJhbGciOiJFUzI1NksiLCJ0eXAiOiJKV1QifQ.eyJleHAiOjE2Mzc0Njk2ODQsImlhdCI6MTYzNzQ2NjA4NCwiaXNzIjoidGVzdCIsImp0aSI6ImxvZ2luLWVzMjU2ay1mYXN0cGF5bWVudGFkbWluIiwibG9naW4iOiIxMjMifQ.Mt-e32zzIwaZEnmEdM6CRFNKmK8RfV8nEj256JCptqynEsF39RNw4rog-xiA3oNnnqOJhtQ9_1Eq_K8wwxT9eg");

auto verifier = jwt::verify().allow_algorithm(jwt::algorithm::es256k("", mainConfig::ec_priv_key, "", "")).with_issuer(row["username"].as<std::string>());
if (verifier.verify(decoded)){
    ret["message"] = "verify success";
} else {
    ret["message"] = "verify failed";
}

verify throws an exception if the token is invalid, which you are probably not catching anywhere. If an exception is thrown but never caught the runtime calls std::terminate. What you need to do is either catch exceptions thrown by verify, or if you don't want to use exceptions for some reason use the overload that accepts std::error_code as an parameter (which you can them check if it had an error).
This design was made intentional, because the verification can fail for tons of reasons and a simple bool wouldn't give most applications enough information to handle the error correctly.

As mentioned you can avoid the runtimes cost and use the std::error_code overload.
https://github.com/Thalhammer/jwt-cpp/blob/66eea980b5b05ac47c09a178805f56e90568bc85/include/jwt-cpp/jwt.h#L3175

Closing as answered.
Feel free to reopen if it was unclear


[CI]: Update container images

moved pctasks-frontend to an aux server
shrink container images a bit

Trying to fix the CI failures at https://github.com/microsoft/planetary-computer-tasks/actions/runs/5579847626/jobs/10195946932#step:11:419, where we seem to be running out of disk space.
--- ErrImagePull: rpc error: code = Unknown desc = failed to pull and unpack image "localhost:5001/pctasks-ingest:latest": failed to extract layer sha256:d32bc1b36f4802dd6c638c4501c6eb836f140b71383c0d5e87db50115060811a: write /var/lib/containerd/io.containerd.snapshotter.v1.overlayfs/snapshots/285/fs/opt/az/lib/python3.10/site-packages/azure/synapse/artifacts/models/__pycache__/_models_py3.cpython-310.pyc: no space left on device: unknown

[419](https://github.com/microsoft/planetary-computer-tasks/actions/runs/5579847626/jobs/10195946932#step:11:420)

https://github.com/microsoft/planetary-computer-tasks/pull/238/commits/3403573cf46b5f5a11d5b205da53d92baf80de4e fixes an issue where local dev files like .mypy_cache might have been copied in to the container image. I don't know if this affected the images on CI (probably not) but it made my local image ~200 MB smaller)
https://github.com/microsoft/planetary-computer-tasks/pull/238/commits/e9ef5f81e890afc0651112f743ce39209dee1411 disables the pip cache, which saves ~20MB.

The next big target is the task-* images. Currently, pctasks-task-base is >2.5GB.
See the diff below for a way to get it down to ~370MB.

diff --git a/Dockerfile.task_base b/Dockerfile.task_base
index c60bbeb..2cffc3b 100644
--- a/Dockerfile.task_base
+++ b/Dockerfile.task_base
@@ -1,4 +1,4 @@
-FROM ubuntu:20.04
+FROM python:3.9-slim

 # Setup timezone info
 ENV TZ=UTC
@@ -9,31 +9,6 @@ ENV PIP_NO_CACHE_DIR=1

 RUN ln -snf /usr/share/zoneinfo/$TZ /etc/localtime && echo $TZ > /etc/timezone

-RUN apt-get update && apt-get install -y software-properties-common
-
-RUN add-apt-repository ppa:ubuntugis/ppa && \
-     apt-get update && \
-     apt-get install -y build-essential python3-dev python3-pip \
-     jq unzip ca-certificates wget curl git && \
-     apt-get autoremove && apt-get autoclean && apt-get clean
-
-RUN update-alternatives --install /usr/bin/python python /usr/bin/python3 10
-
-# See https://github.com/mapbox/rasterio/issues/1289
-ENV CURL_CA_BUNDLE=/etc/ssl/certs/ca-certificates.crt
-
-# Install Python 3.8
-RUN curl -L -O "https://github.com/conda-forge/miniforge/releases/latest/download/Mambaforge-$(uname)-$(uname -m).sh" \
-   && bash "Mambaforge-$(uname)-$(uname -m).sh" -b -p /opt/conda \
-   && rm -rf "Mambaforge-$(uname)-$(uname -m).sh"
-
-ENV PATH /opt/conda/bin:$PATH
-ENV LD_LIBRARY_PATH /opt/conda/lib/:$LD_LIBRARY_PATH
-
-RUN mamba install -y -c conda-forge python=3.8 gdal pip setuptools cython numpy
-
-RUN python -m pip install --upgrade pip
-
 # Install common packages
 COPY requirements-task-base.txt /tmp/requirements.txt
 RUN python -m pip install --no-build-isolation -r /tmp/requirements.txt
diff --git a/requirements-task-base.txt b/requirements-task-base.txt
index 3de345b..39fe26a 100644
--- a/requirements-task-base.txt
+++ b/requirements-task-base.txt
@@ -1,2 +1,2 @@
 pystac[validation]==1.*
-rasterio==1.3.0 --no-binary=rasterio
\ No newline at end of file
+rasterio==1.3.0
\ No newline at end of file

This removes a bunch of "cruft" from the task-base Dockerfile. But we need to confirm whether any of that cruft is actually load bearing.

Which base image do we need? Do we need ubuntu or can we use some other image?
Do we need conda / mamba?
Do we need to build rasterio from source?

CI passed, so I'll punt on the questions from the previous comment.


Scrolling on touch pad triggers slide changing multiple times

I usually use touch pad to browse websites. I noticed that if I use touch pad to move between slides, a slight scrolling will trigger slide changing multiple times.
I am currently on macOS 10.14.3 and my laptop is Macbook Pro.
The same problem also appears on a PC with Windows 10 as long as scrolling with a touch pad.

I have already come up with a solution using throttling function. PR #564


Development: Remove SERVER_API_URL

Checklist
General

[X] I tested all changes and their related features with all corresponding user types on a test server.
[X] I chose a title conforming to the naming conventions for pull requests.

Client
none of the checks here apply
Motivation and Context

The SERVER_API_URL is a constant used heavily in the client. However, in practice it's always empty (tested on production, ts1, locally, by looking at code).
Also the fact that it is a declared global variable injected by Webpack makes it difficult to switch to a different code bundling process, which is on the way.
Keeping this variable adds unnecessary complexity to the client code. Therefore, it should be removed.
Description

This PR removes the SERVER_API_URL in all places in the client code.
Steps for Testing

Click around in the application. I've replaced one variable in the whole app, so if it works in one place, it should work in all others.
We should pay a bit more attention in the following places:

Group Chat
Exam Mode Live Statistics
LTI 1.3 Support

The reason being is that code from those features assumed no trailing slash for SERVER_API_URL, which differed from all other places. I'm confident that this won't affect anything, but it's better to double-check.
Code Review Info
List of files where more than literally just removing the variable usage happens:

declarations.d.ts
environment.js
webpack.custom.js
interceptor.util.ts
code-editor-domain-dependent-endpoint.service.ts
build-log.service.ts
removed slash

lti13-dynamic-registration.ts
lti13-exercise-launch.component.ts
exam-monitoring.service.ts
group-chat.service.ts

Review Progress

Code Review

[ ] Code Review 1
[ ] Code Review 2
[ ] Code Review 3
[ ] Code Review 4

Manual Tests

[ ] Test 1
[ ] Test 2

Codecov Report

Merging #6544 (7bd79b6) into develop (851d4b8) will decrease coverage by 0.01%.
The diff coverage is 88.65%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #6544      +/-   ##
=============================================
- Coverage      80.48%   80.48%   -0.01%
+ Complexity     13211    13210       -1
=============================================
  Files           2349     2349
  Lines          90258    90250       -8
  Branches       13339    13339
=============================================
- Hits           72641    72634       -7
+ Misses          9701     9699       -2
- Partials        7916     7917       +1

Impacted Files
Coverage Δ

...word-reset/finish/password-reset-finish.service.ts
80.00% <0.00%> (ø)

...password-reset/init/password-reset-init.service.ts
80.00% <0.00%> (ø)

...in/webapp/app/account/register/register.service.ts
80.00% <0.00%> (ø)

src/main/webapp/app/admin/health/health.service.ts
83.33% <0.00%> (ø)

...ain/webapp/app/core/about-us/about-us.component.ts
86.95% <ø> (-0.55%)
:arrow_down:

...in/webapp/app/core/interceptor/interceptor.util.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...pp/exam/manage/students/exam-students.component.ts
80.00% <ø> (-0.16%)
:arrow_down:

...-check/exam-students-attendance-check.component.ts
96.87% <ø> (-0.05%)
:arrow_down:

...external-submission/external-submission.service.ts
100.00% <ø> (ø)

...ises/shared/feedback/long-feedback-text.service.ts
69.23% <0.00%> (ø)

... and 108 more

... and 3 files with indirect coverage changes

Flag
Coverage Δ

client
76.49% <88.65%> (+<0.01%)
:arrow_up:

server
84.92% <ø> (-0.01%)
:arrow_down:

Continue to review full report in Codecov by Sentry.

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


Add support for bind dn and bind pwd on the command line.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1538813
When mode is ldap, bind dn and bind pwd must be provided, either
in the authentication settings or on the command line.
Steps for Testing/QA

Configure MIQLDAP for Mode: LDAP (not Mode: LDAPS)
Do not get groups from LDAP
Manually create users in UPN username format.
Run SSSD conversion
e.g.:

miqldap_to_sssd -b "cn=Manager,dc=example,dc=com" -p "password" -d "example.com"

@miq-bot add_label bug

@miq-bot add_label gaprindashvili/yes

@miq-bot add_label authentication

@miq-bot assign @gtanzillo

@abellotti and @bdunne Please review. Thank you! JoeV

Gaprindashvili backport details:
$ git log -1
commit e4eb8f1bb648d061a45c4f324049577419435ef0
Author: Gregg Tanzillo<EMAIL_ADDRESS>Date:   Tue Feb 13 09:10:17 2018 -0500

    Merge pull request #16979 from jvlcek/bz1538813_miqldap_add_binddn_bindpwd

    Add support for bind dn and bind pwd on the command line.
    (cherry picked from commit 8aea84d62b00349a3e9b79926aaa48c37bc087b7)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1552785


Can I close the dialog whenever I want

the dialog will auto closed no matter i clicked which button，Can I control it? How?  I only want to replace the dialog html-content sometimes.

Hello @xiashulin The default dialogs perform specific purposes like confirmation, prompt or alert. If you want to update dialog's content then there may be a need to use the custom view feature.


Add Verdaccio to test package publishing

In theory, this could help testing versioning strategies with changesets and setting up CD without depending on an immutable (public or private) registry, we could reset the Verdaccio local package database as we ~make mistakes~ learn.

@SocketSecurity ignore<EMAIL_ADDRESS>🕊🇺🇦


Add table github_dependency

GitHub GraphQL API supports DependencyGraph* resources for quite a few package managers now.
https://docs.github.com/en/graphql/reference/objects#dependencygraphdependency
Doesn't look like go-github will be supporting it (as that relies on the REST API).

@captn3m0 appreciate your interest and detailed information. Thank you!!. We will definitely be checking this.  Pls, keep us posted for further observation.

not stale.

Rest API support is on the GitHub roadmap now https://github.com/github/roadmap/issues/495

The item on roadmap only covers alerts and updates. The issue was for DependencyGraph which is much more comprehensive (all dependencies of a repo, irrespective of vulnerability status).

Hey @captn3m0 , sorry for the radio silence on this one.
I haven't used https://github.com/shurcooL/githubv4 personally, but it looks promising and I'd be open to introducing that package to this plugin for better GraphQL querying support for new/existing tables. If that package isn't sufficient, I'd also be open to querying the GraphQL API directly, similar to what the lyft/cartography does, along with our Turbot plugin, e.g., https://github.com/turbot/steampipe-plugin-turbot/blob/main/turbot/table_turbot_resource.go.
If anyone is interested in trying to add new tables using the shurcool/githubv4 repo or hitting the GraphQL API directly, we'd be happy to work with them on the design process, review any incoming PRs, and answer any questions they have.

not stale

Hey @captn3m0 , just so you're aware, we're not actively looking at adding this table as we're currently focusing on other priorities. I believe the resources and links I've included in my previous comment can help anyone looking to implement this table, but if not, we're happy to help/provide more guidance.
The stale label will be re-attached every 30 days as per our workflow, so if you'd like to keep this open for more community visibility, please feel free to continue commenting (just like above). Thanks!


Add dataset card and model card

Add the dataset card and the model card to the repository

 Dataset Card

 Model Card


Content slightly cut off when used from firefox overflow menu

Describe the bug
If you move the extension to the "overflow menu" in Firefox (so it isn't show all the time), the extension is opened with a thick bar at the top and in a slightly thinner box which cuts off some of the content on the right side (like the scrollbar and the icon next to viewer count).
To Reproduce

Install Firefox
Install Gumbo
(optional) Sign in to Twitch in the extension, makes the bug more noticeable
Right click Gumbo's icon > "Pin to Overflow Menu"
Open Gumbo from overflow menu
Bleed from your eyes

Expected behavior
Content is not cut off
Screenshots explain this better than words

Desktop (please complete the following information):
Firefox 101 on Fedora Linux 35

Tried to reproduce on updated firefox, got a different bug:

What it should look like:

Version: Firefox 103 on Ubuntu Linux

This issue cannot be resolved on my side and should be resolved with Firefox 106 (Nightly).
I'll mark this issue as done because there's nothing I can do besides asking you to wait for Firefox 106, which is planned for October 18th, 2022.

Bugzilla Ticket: https://bugzilla.mozilla.org/show_bug.cgi?id=1783972
Release Calendar: https://wiki.mozilla.org/Release_Management/Calendar


What is the question??

We have the answer to the question of life, the universe and everything, great... but what is the question??

see #1


Increase thread count threshold for cache clearing service

The increase in workers for cache clearing service has increased
the number of processes spawned so alert on a higher thread count.

Are we doing anything with this?

We held off doing this while we investigated the original problem. Assuming it's not still a problem I'll close this.


Multiple nodes

Can you implements this for use and manage  multiple nodes in one dashboard

This is achieved in v4. Will close this issue now.


Only show extension recommendations when there are no extensions already installed

After installing the Binary Plist extension the following message is displayed when opening plist files.
"The Marketplace has extensions that can help with '.plist' files"
Clicking the Search Marketplace results in a ext:plist search in the Marketplace and it shows the extension that has already been installed.

Agree, it shouldn't. We actually do check if you have extensions installed that handle the language the file is in. So, in this case, for whatever reason, we think you don't have any.


Update home.ex.eex

No @size attr, seems like it was meant to be @text_size. :)

Fix is published to hex in 0.10.2


log said:  Connection  lost

i send vars (hostname,port,username，password) to  WEBSSH by javascript ，webssh Authenticate successfully，but log said connection to hostname:port lost
js button:
test js send host info to WEBSSH

Log:
(u'<IP_ADDRESS>', '22', u'root', u'root', None)
[I 180326 22:03:42 main:208] Connecting to <IP_ADDRESS>:22
[I 180326 22:03:42 transport:1687] Connected (version 2.0, client OpenSSH_6.6.1)
[I 180326 22:03:47 transport:1687] Authentication (password) successful!
[I 180326 22:03:47 web:2063] 200 POST / (<IP_ADDRESS>) 5151.64ms
[W 180326 22:03:48 web:2063] 404 GET /favicon.ico (<IP_ADDRESS>) 3.69ms
[I 180326 22:03:50 main:119] Connection to <IP_ADDRESS>:22 lost

js botton:
onclick="post('http://<IP_ADDRESS>/', {hostname:'<IP_ADDRESS>',port:'22',password:'root',username:'root'}); "

You need to connect to the webssh server by websocket with the worker id.
The worker only lasts for 3 seconds if no handler attached for security reason.

我也这样，这个是什么原因导致的呢？

I have added a parameter reason to the close method of worker.
You can now see more useful log message when the worker is closing.


Lambda task for RUM processing

OK, squashed out all the authentication tokens and reduced my work to one commit.
Issues we may need to resolve:

compatibility testing largely excludes the possibility of finding false negatives, because a request for 10 features, which delivers polyfills for five of them, will only include RUM tests for the five features we have polyfilled. We should probably include RUM tests for all requested features regardless of whether they got polyfilled or not.

Sample output from /v2/getRumPerfData:
"data_center","country","count","perf_dns_count_notnull","perf_dns_min","perf_dns_max","perf_dns_mean","perf_dns_std","perf_dns_95","perf_dns_99","perf_connect_count_notnull","perf_connect_min","perf_connect_max","perf_connect_mean","perf_connect_std","perf_connect_95","perf_connect_99","perf_req_count_notnull","perf_req_min","perf_req_max","perf_req_mean","perf_req_std","perf_req_95","perf_req_99","perf_resp_count_notnull","perf_resp_min","perf_resp_max","perf_resp_mean","perf_resp_std","perf_resp_95","perf_resp_99"
"IAD","US","149","13","0","91","21.17","22.52","20","33","14","0","238","65.27","60.21","38","142","130","0","505","28.64","65.96","96","383","130","0","46","5.36","7.08","21","38"
"JFK","US","89","13","0","418","78.26","107.66","57","418","12","0","311","124.04","87.8","147","311","67","0","62","12.81","13.27","48","62","66","0","39","4.25","6.9","15","39"
"LCY","GB","66","1","0","0","0","0","0","0","2","0","10","8.14","1.88","0","10","64","1","275","24.39","51.87","112","275","64","0","23","2.82","4.02","13","23"

This includes performance data for each of the DNS look up, TCP connect, request and response, in each case providing the following statistics:

Count of non-null values recorded (null values can result from beacon data from browsers that do not support the resource timing API)
minimum value
maximum value
mean value
Standard deviation
95th percentile value (probably the most useful)
99th percentile value

Sample output from /v2/getRumCompatData:
"feature_name","ua_family","ua_version","pass_rate","ip_count","total_count","is_targeted","targeting_status"
"Element.prototype.before","chrome","52.0.0","1","103","338","Yes","False positive"
"Element.prototype.prepend","chrome","52.0.0","1","103","338","Yes","False positive"
"Element.prototype.replaceWith","chrome","52.0.0","1","103","338","Yes","False positive"

These reports are still not very user-friendly. Arguably the compatibility data could be used as-is (although the current logic for measuring compatibility will exclude false negatives because we will not include tests for features that we are not pollyfilling).
However the performance data is difficult to read, so I'm thinking of adding a new section to the usage data page, potentially like this:

(this network timeline mock up uses the same colour scheme as chrome network timeline)

Done the perf visualisation:

Is it possible to cover these new features with tests?

Updated:

there is still a hardcoded AWS role in the Apex project.json file

There's precedent for this, and @matthew-andrews recommended using Apex's 'environments' feature, which I've done

Deploying the Lambda functions requires the AWS credentials to be in your ~/.aws/credentials file as polyfill and polyfill-qa profiles, and for the RUM_MYSQL_DSN and RUM_MYSQL_DSN_QA env vars to be defined in your .env file (possible solution from @JakeChampion: use env vars on command line when invoking apex deploy)

Used Jake's suggestion.

HTTP API docs are shown even if RUM env var is not set.  Should probably suppress these.

Done.

Regarding tests, what do you want to test, and how?  I'm not bothered about breaking RUM.  I appreciate that we don't want to break the service, but that should already be tested.

The AWS credentials are in Lastpass but they need sharing with the origami team.

Done

I think we should be bothered about breaking RUM, if in the future we are going to use it to inform us what features each browser supports. It is easier to add tests for it now, before it is in master than later on down the road.

Agree that if we are using RUM to directly affect targeting automatically, it needs to be tested and monitored, but right now it's an experiment.  I just want to collect some data and see if it makes sense.
Again, I don't know how you'd test it effectively.  The most serious failure is taking down the service (already covered by tests) and I'd say the second most serious is producing incorrect data, and it's impossible to know if the reported data is right until we collect some.  I'm open to suggestions though - what tests would you suggest we implement before merging?

unit level tests would be good, we can worry over intergration tests later

I don't know how I would unit test the RUM reporting routes, they require MySQL and hook directly into Express router meaning they can really only be subject to an integration test. I could unit test the lambda function but I just don't really care if it breaks, and it's totally isolated from heroku.  My tests would also probably not cover the failure scenarios because I don't know what they are yet.  The function leaves behind any files it cannot process so I can monitor for unprocessable logs.
If you can highlight exactly what you want tested, I'll look at doing that.


[BUG] Failed to delete the existing VMs after upgrade, it stuck in Terminating state

Describe the bug

After upgrade from v1.1.1 to v1.1.2-rc1, when we try to delete the existing VM before upgrade.
We can't delete the VM, it stuck in terminating state

The VM attached volume have already been deleted

To Reproduce
Steps to reproduce the behavior:

Prepare a two nodes Harvester cluster with v1.1.1 release
Set up node and disk tags on each Node's host page
Create a new storage class the set as the default
Prepare two virtual machines evenly on node1 and node2
Upgrade to v1.1.2-rc2
Wait until upgrade complete
Update the config of the first VM, increase the memory and attach a new volume
Delete the first existing VM
Delete the second existing VM

Expected behavior

Can delete both existing VMs (one of them already updated) correctly
Support bundle

supportbundle_2df5f22c-bf4c-41ad-a11f-e72b673a480a_2023-03-07T08-24-26Z.zip
Environment

Harvester ISO version: v1.1.1 to v1.1.2-rc1
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): 2 nodes bare machines

Additional Context
We can delete the newly created VM

The bug has been fix, same as https://github.com/harvester/harvester/issues/3572


titlebar background color is visible when frame: false

I have followed the README.md instructions and applied the Mica effect perfectly but the background color of the title bar destroyed the perfection.
Is there a way to disable the title bar background color?
I've tried win.setCaptionColor() but it doesn't take alpha values.

I'm on Windows 11 Home 22H2 with "Show accent color on title bars and windows borders" enabled in the Personalize Settings.
code https://github.com/ExpTechTW/TREM-RTS-Map/commit/3c42a02397122961fbed31abe599157a0ab13dd6

Hello, I can't access your repo, but why you add a caption color if you dont want this ? If you remove all win.setCaptionColor you don't have color. You cannot remove a caption color because Windows 11 does not offer it.

Repo is now public
I don't have win.setCaptionColor() in the code and the title bar background is there
https://github.com/ExpTechTW/TREM-RTS-Map/blob/3c42a02397122961fbed31abe599157a0ab13dd6/src/main.js#L8-L38

When I try your repo it's work for me, what is your windows 11 version ?

Edition	Windows 11 Home
Version	22H2
Installed on	2023/01/21
OS build	22621.1194
Experience	Windows Feature Experience Pack 1000.22638.1000.0

I didn't find a solution, I think it's a problem with your Windows 11, I'm limited with Windows 11 API, sorry.


[FEATURE] add language translator

add language translator to the website to facilitate ease and more user friendly approach

@ananyag309
Sorry i haven't planned for this yet


[Feature request] Interactive commands

Please add ability to define custom variables for interactivness:
        "custom variables": [
          {
            "variable": "$var1",
            "massage": "Write project name",
          },
          {
            "variable": "$var2",
            "massage": "Wich value to choose?",
            "options": [
              "css",
              "scss"
            ],
            "default": "css", // highlighted value in dropdown list
          },
          {
            "variable": "$var3",
            "massage": "Wich command run?",
            "options": [
              { "Hellow": "echo hellow" },
              { "GoodBye": "echo goodbye" },
            ],
            "default": "Hellow",
          }
        ]

Example.
First variable:

Second variable:

Third Variable:

Ok, thank you for your feedback. I will deal with it later when I have time


Add Payment Management System

Payment management system is added. Detailed information about this PR/payment system are listed as below:

plan options: monthly plan, quartely plan, yearly plan;
payment method include: UnionPay, WeChat Pay, Alipay;
quota management: reguar membership and professional membership.

Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration:

Test Configuration:

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

I feel the "price" should be changed from a string to a number. Other than this, the PR looks good to me.

thanks for your review. I updated this.


Fragments

I think it would be helpful to have a Fragment function that outputs the children with no actual parent element (similar to React.Fragment https://legacy.reactjs.org/docs/fragments.html)
This would be useful in scenarios where you need to spread a slice into an element like this - but don't actually want to wrap it in a <div /> tag
Body: []g.Node{
	content,
	html.Div(modals...),
	html.Script(
		html.Src("https://cdn.jsdelivr.net/npm/bootstrap@5.3.0/dist/js/bootstrap.bundle.min.js"),
		g.Attr("integrity", "sha384-geWF76RCwLtnZ8qwWowPQNguL3RmwHVBC9FhGdlKrxdiJJigb/j/68SIy3Te4Bkz"),
		g.Attr("crossorigin", "anonymous"),
	),
},

@TheManSpeaker thank you for your issue submission. 😊 I think maybe the Group function is what you're looking for? Or is it slightly different?

Ah yes that’s exactly what I need, thank you!

Excellent, good to hear.

@markuswustenberg in my case the g.group func doesn't fit very well. I try to set my components like this:
func HomePage() g.Node {
   return g.Group(
      Component("1"),
      Component("2"),
   ),
}

@markuswustenberg in my case the g.group func doesn't fit very well. I try to set my components like this:

func HomePage() g.Node {

   return g.Group(

      Component("1"),

      Component("2"),

   ),

}

You could either wrap the parameters in a Node slice or use a Div (or other such components). Often it's a matter of dividing the components differently so that you include the root of your subcomponent in the function return. How does that work for you?


[Bug]: can not install fabric

What happened?
when trying to install i get an error message
go install github.com/danielmiessler/fabric@latest
this is the output
Version check

[X] Yes I was.

Relevant log output
go: downloading github.com/danielmiessler/fabric v1.4.3
go: github.com/danielmiessler/fabric@latest (in github.com/danielmiessler/fabric@v1.4.3): go.mod:3: invalid go version '1.22.5': must match format 1.23
go.mod:5: unknown directive: toolchain

Relevant screenshots (optional)
No response

Your go.mod file in the Fabric repository specifies a Go version that doesn't match, looking for version 1.23, but found 1.22.5.
Check your Go version

I ran into the same issue:
go install github.com/danielmiessler/fabric@latest go: github.com/danielmiessler/fabric@latest (in github.com/danielmiessler/fabric@v1.4.3): go.mod:3: invalid go version '1.22.5': must match format 1.23 go.mod:5: unknown directive: toolchain
Steps to resolve:
➜  go version
go version go1.20.2 darwin/arm64

➜  brew install go
==> Auto-updating Homebrew...

➜  go version
go version go1.23.1 darwin/arm64

Then it worked

I can't reproduce it, please try again.
Here my output:
`❯ go install github.com/danielmiessler/fabric@latest
go: downloading github.com/danielmiessler/fabric v1.4.21
~ via 🐹 v1.23.1 took 14s
❯ fabric
no session, pattern or user messages provided
~ via 🐹 v1.23.1
❯ fabric -h
Usage:
fabric [OPTIONS]
Application Options:
-p, --pattern=                    Choose a pattern
-v, --variable=                   Values for pattern variables, e.g. -v=$name:John -v=$age:30
-C, --context=                    Choose a context
--session=                    Choose a session
-S, --setup                       Run setup
--setup-skip-update-patterns  Skip update patterns at setup
-t, --temperature=                Set temperature (default: 0.7)
-T, --topp=                       Set top P (default: 0.9)
-s, --stream                      Stream
-P, --presencepenalty=            Set presence penalty (default: 0.0)
-r, --raw                         Use the defaults of the model without sending chat options (like temperature etc.) and use the user role
instead of the system role for patterns.
-F, --frequencypenalty=           Set frequency penalty (default: 0.0)
-l, --listpatterns                List all patterns
-L, --listmodels                  List all available models
-x, --listcontexts                List all contexts
-X, --listsessions                List all sessions
-U, --updatepatterns              Update patterns
-c, --copy                        Copy to clipboard
-m, --model=                      Choose model
-o, --output=                     Output to file
-n, --latest=                     Number of latest patterns to list (default: 0)
-d, --changeDefaultModel          Change default pattern
-y, --youtube=                    YouTube video url to grab transcript, comments from it and send to chat
--transcript                  Grab transcript from YouTube video and send to chat
--comments                    Grab comments from YouTube video and send to chat
--dry-run                     Show what would be sent to the model without actually sending it
-u, --scrape_url=                 Scrape website URL to markdown using Jina AI
-q, --scrape_question=            Search question using Jina AI
Help Options:
-h, --help                        Show this help message`

I'm running PopOS and I get the same error:
draeician@nomnom:~$ go version
go version go1.18.1 linux/amd64
draeician@nomnom:~$ go install github.com/danielmiessler/fabric@latest
go: github.com/danielmiessler/fabric@latest (in github.com/danielmiessler/fabric@v1.4.22): go.mod:3: invalid go version '1.22.5': must match format 1.23
This is the latest version that popos has in the repository though.

Resolved my issue, but had to install a version of go that's now outside the supported popos ecosystem.


Try released finufft 2.2

(cu)finufft 2.2 was released a couple weeks ago.
We no longer need to use development versions of cufinufft we build ourselves, the wheels provided should be fine. This also means we can think about releasing pytorch-finufft on PyPI

Closing to immediately re-open just to re-run CI


feat: add support for getting DORA metrics from projects and groups

resolves https://github.com/xanzy/go-gitlab/issues/1557

:neutral_face:  I so wish there's a day that gitlab docs actually tell you that

This PR can be closed without merging, the functionality was added under https://github.com/xanzy/go-gitlab/pull/1931

Thanks for the ping @heidiberry 👍🏻


[Feature] Improve GitHub Action documentation and workflow

Is your feature request related to a problem? Please describe
OpenSearch has many checks that are run on pull requests which are displayed 'below the fold' of the UX in the pull request page, this makes them easy to overlook, especially when some checks report status directly into the pull request comments.
Improving documentation and instruction on how to troubleshoot a failures would lower the burden on maintainers and help contributors unblock themselves.
Describe the solution you'd like
Lets consolidate the 'result' comments that are featured in the pull request to include checks that are required before a pull request is merged along with links to documentation that describe what these checks are and how to troubleshoot.
Related component
Other
Describe alternatives you've considered
This could also be a new markdown document under .github/workflow that includes details about what each workflow is and how that are used that would be referenced in the project.
Additional context
No response

[Triage - attendees 1 2 3 4 5 6 7 8]
@peternied Thanks for creating this issue.


python312Packages.mdtraj: fix runtime error and support python3.12

Description of changes
closes #308847

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 309076 run on aarch64-linux 1

4 packages built:

python311Packages.mdtraj
python311Packages.mdtraj.dist
python312Packages.mdtraj
python312Packages.mdtraj.dist

The SSE detection fails with python-3.12. However, the SSE intrinsics are not turned off properly in this case.
The following patch rectifies the problem.
diff --git a/basesetup.py b/basesetup.py
index 54dd009e..9ff6ed76 100644
--- a/basesetup.py
+++ b/basesetup.py
@@ -32,6 +32,7 @@ class CompilerDetection(object):

     def __init__(self, disable_openmp):
         self.disable_openmp = disable_openmp
+        self.disable_intrinsics = True
         self._is_initialized = False

     def initialize(self):
@@ -76,7 +77,7 @@ class CompilerDetection(object):
         if os.environ.get('MDTRAJ_BUILD_DISABLE_INTRINSICS') == "1":
             self.disable_intrinsics = True
             print("Env var MDTRAJ_BUILD_DISABLE_INTRINSICS set")
-        else:
+        elif (self.sse2_enabled or self.neon_enabled):
             self.disable_intrinsics = False

Yes, I have a problem with SIMD. For some reason, only Python 3.12 cannot detect it.
But I'm able to build on Debian without any problems.
Is it correct that this patch is meant to allow the build to pass without SIMD, and not to enable it?
I think it would be better to accept this patch now, merge this PR, and postpone the SIMD issue.

Yes, I have a problem with SIMD. For some reason, only Python 3.12 cannot detect it. But I'm able to build on Debian without any problems.
Is it correct that this patch is meant to allow the build to pass without SIMD, and not to enable it? I think it would be better to accept this patch now, merge this PR, and postpone the SIMD issue.
mdtraj> Attempting to autodetect OpenMP support... Did not detect OpenMP support
mdtraj> Attempting to autodetect SSE2   support... Did not detect SSE2 support
mdtraj> Attempting to autodetect SSE3   support... Did not detect SSE3 support
mdtraj> Attempting to autodetect SSE4.1 support... Did not detect SSE4.1 support
mdtraj> Attempting to autodetect NEON   support... Did not detect NEON support

Yes, this patch allows the build to pass without SSE. This is also an upstream bug since the sse intrinsics do not get deactivated if the compiler SSE detection fails.
I agree, that we can fix the actual SSE detection issue later. You can add the patch to the PR to unbreak the python 3.12 build.

Result of nixpkgs-review pr 309076 at 34399938 run on x86_64-linux 1

2 packages built successfully:

python311Packages.mdtraj
python312Packages.mdtraj

Thanks for your support!


Hosted server slow on first request

The server is hosted on a free azure website plan that shutdown the web application after some period of inactivity. So first request should wait for the web application to warm up and start running which causes the delay on the client side and might time-out.
Resolution
Move the hosted server into a paid plan

Moved to paid plan with "Always On"


Windows VC matrix

While we have discussed this several places before, wasn't seeing this tracked anywhere. So thought it best to open an issue here. Guessing we will need to handle VC correctly here before we can address it in conda-smithy.
Currently in order to get a package to build with multiple VC runtimes, we need to add python as a build dependency and track a feature. However, this has proved impractical and in some cases limiting. Instead discussion over the past several months has concluded the best way forward is to have a separate vc package for tracking the compiler that python and other packages track. Though ultimately this must be addressed with some kind of matrix. While we can find somewhat effective ways to do this manually, conda-build-all has reigned supreme at helping convert an implicit matrix to an explicit matrix (via leveraging by conda-smithy). Hence I'm raising this here to ask that vc be handled in the matrix configuration used by staged-recipes and can picked up with conda-smithy for use in the feedstocks. There are already some considerations about how this might play out.
xref: https://github.com/conda-forge/glpk-feedstock/pull/10
xref: https://github.com/conda-forge/tk-feedstock/pull/2
xref: https://github.com/conda-forge/curl-feedstock/pull/11
xref: https://github.com/conda-forge/conda-smithy/issues/245
cc @jjhelmus @msarahan

cc @jjhelmus

Friendly nudge @pelson. 😄 Continuing to suffer without this functionality. See this re-rendering PR ( https://github.com/conda-forge/tk-feedstock/pull/8 ) for an example. Any chance we could add vc to the version matrix.

I guess the most important one to xref is: https://github.com/SciTools/conda-build-all/pull/36
That PR needs resurrecting, and we should be good to go, right?

SGTM. Would it help to have conda-build learn more about VC long term? ( https://github.com/conda/conda-build/issues/1445 )

Could you please also weigh in on this discussion @msarahan ?

We might be able to get around a matrix by doing clever hacks like PR ( https://github.com/conda-forge/tk-feedstock/pull/20 ), but I don't know if we should be rolling that out on a large scale. Maybe conda-build 3 has better solutions available?

@jakirkham - what is the status on VC nowadays? Would you like me to resurrect https://github.com/conda-tools/conda-build-all/pull/36, or shall we keep holding fire on that until we have a conda-build 3 usage plan (if we don't already)?

I'm pretty happy with the solution devised in PR ( https://github.com/conda-forge/tk-feedstock/pull/20 ) even if it may be a hack. FWIU there is a solution in conda-build 3 that is different, but have had no time to familiarize myself with it. Maybe @msarahan can comment. My guess is we can close this though.

cb3 uses variants, and one of the keys is vc:
https://github.com/AnacondaRecipes/qt-feedstock/blob/master/recipe/meta.yaml#L49
As for CB3 usage plan, I think that there are two steps, as I outlined at https://github.com/conda/conda-build/commit/a617f57a1087520c94014e78cc99efed734bcf56#commitcomment-24461476

Using cb3 with conda-build-all. I hope this is a drop-in thing. I tried to maintain the backward compatibility.
Taking advantage of cb3's build matrices while also maintaining the job splitting capability of conda-build-all. This is a little bit bigger project. It will probably largely duplicate concepts in https://github.com/conda/conda-concourse-ci (but c3i can probably be said to have duplicated conda-build-all).  Specifically, conda-build-all or something like it render output metadata, then write that output data to a temporary location to be used by the build servers, or alternately, store environment variables as necessary.

Where conda-build 3 and conda-build-all differ most is that cb3 supports an arbitrary variable space, whereas c-b-a is limited to what conda-build 2 supports, which is numpy, python, perl, lua, and R.  CB3 supports the same env vars as cb2, but the arbitrary extensibility doesn't translate into env vars.  If you can think up a good way to have it available in env vars, I'd happily accept it.

TBH I think moving away from environment variables might be healthier. The trick that I imagine with doing this is creating sensible CI matrices from it. Though that is a problem at the conda-smithy level. Depending on what the answer is to that question, it may make more sense to go to conda-build directly from conda-smithy or preserve the status quo (i.e. conda-smithy -> conda-build-all -> conda-build).

yeah, environment variables are not as nice, but they do make moving across multiple CI worker instances easier.  Having temporary files makes the build infrastructure quite a bit more complicated.

Thanks the the insights. I'm going to close this out - the proof of concept in #36 is the way I'd solve the problem for CB2, but CB3's variants is the right way to go IMO. I'm definitely up for having conda-build-all/conda-smithy/conda-whatever generate the right conda_build_config.yaml if necessary.
If anything, just having conda-build-all and conda-build 3 playing together would be a nice first step.


Update style guide plugin to UE4.20

Hello! I love this guide, but unfortunately, the plugin is not for sale nor updated for the latest engine version.
I would love a mass rename tool. Thank you!

Unfortunately Linter ceases to exist as we know it. I should really update this to reflect that.

@Allar are you saying that all references to the linter should be removed?

just giving this a bump :)


PlotlyGraph Widget's Container's height defaults to zero

After #1747  was merged, the PlotlyGraphWidget in the case of only one plot defaults to the height of the container. This seems fine in ExecutionIndex, however, in the Log/GraphViewer provides a height of zero.

This happens because addNode functions are called before onWidgetContainerResize which sets the height of the container.  At this point height's will already be set. The _.debounce function could be levaraged to adjust plot height.


HasPassword returns always false

HasPasswordAsync returns always false.

I'll update based on hash being set. For now I'd query the collection directly as a workaround.

This is what I did ;)

Same problem here. any progress on this issue?


buildkite initialization timeouts

Is your feature request related to a problem? Please describe.
As of the week of Nov 6, we've been seeing initialization timeouts much more frequently on buildkite. E.g. here and here (and here in CliMAgen which, like us, doesn't use a depot). This happens nondeterministically, and rerunning a timed out job sometimes works. It doesn't seem like this has been happening in ClimaAtmos or ClimaCoupler (I haven't looked into other repos).
Describe the solution you'd like
This problem may be due to very slow downloads of julia packages and artifacts. We could alleviate this by using a depot, as is done in some other repos (ClimaAtmos, ClimaCoupler, etc). The depot will save the artifacts at the end of a job to a central location that can be accessed by jobs, so we don't have to download them for every buildkite job.
Depot details here: https://github.com/JuliaCI/DepotCompactor.jl
#378 also has some pipeline cleanup that should speed this up
Describe alternatives you've considered

removing unnecessary dependencies would also help
it's possible this is just due to getting slow nodes, but I think that's unlikely since we didn't see this as frequently even when we had ODE as a dependency and our init took ~45 mins on average

closed by #378


Implement Virtual Texturing/Megatexture (Streaming)

Use Generic Address Allocator to make a 2D rectangle allocator for the indirection table, can assume PoT textures and PoT alignment. Its okay to be wasteful on indirection table.
Use templates to make this work for 1D, 2D, 3D, MSAA and Cubemap textures.
Arrays are handled implicitly.
Make sure page sizes are padded by MaxAnisotropy/2 texels, and are aligned to texture format block sizes.

As an extension provide a uniform LRU cache manager, gather page requests into SSBO efficiently, either straight from pixel shaders or from Visibility Buffer analysis.

1
Example, 128x128 page size and supporting Anisotropy 16x and 11 mip-map levels = 144x144 actual tile size
This results in 1-(144/128)^2 ~= 25% base wastage on high mip-levels
Then 7*144^2-128^2 = 128k texel wastage (most likely 512kb) per-texture-mip-tail
Another Example, 256^2 page size supporting Aniso 16x and 12 mip-map levels = 272x272 actual tile size
12% wastage for whole texture, then 526k texel wastage (most likely 2MB) per-texture-mip-tail
Sponza has 54 textures, 18 RGB(A), 18 R8 Bump, 18 R8 Specular, most at 1024^2 resolution.
Option 1: (18*4+36)*(128k +1.25*(1+1/4+1/16+1/256)*1024^2) = 191 MB
Option 2: (18*4+36)*(526k +1.125*(1+1/4+1/16)*1024^2) = 215 MB

Indirection Texture has full mips (if VT supports up to 11, then Indirection Texture will have 11)
Let's say our LRU cache is max 64k tiles (which with even 144^2 tiles gives a 1 MTex cache which is too much), then the Indirection Texture can just use a R16 or RG8 UINT format.
For 11 mip-level support and 64k tile support our bits-per-tile are 2.6328125
So let's explore different Indirection Texture Sizes

First Header
Second Header

Content Cell
Content Cell

Content Cell
Content Cell

The only question is how to solve the issue of "mip-tail allocation", this means do we create an additional mip-less texture array for the tail tiles and have the liberty to use a PoolAddressAllocator or do we use ContiguousPoolAddressAllocator ?
The first would probably aid allocation and deallocation speeds (streaming), but would increase our "bits-per-tile" to 2.67, and waste a texture slot. Also it would require dynamic descriptor indexing.
The only other possibility is to have the whole indirection in one texture which ensures dynamically uniform look-ups, so using ContiguousPoolAddressAllocator.
This means that the tiles of a mip-tail are all allocates contiguously (the addresses are sequential, one after the other).
This approach has upsides, being:

Mip-tail load quicker
Mip-tail can be tightly packed in the future (precomputed for anisotropic border sizes and tail lengths)
Less Indirection Texture Overhead with long mip-tails

Tile Upload/Commit is to happen with a compute shader (hopefully indirect dispatch in the future).
Should detect cache texture overflow.
Streaming and LRU cache is a concern for the future.

Render Feedback Tile-Request
Indirection Table will encode the following:

First N bits, the Tile that has the data
Last 16-N bits is whether the tile has its own data (value 0 loaded) or is pointing at a mip-map above (value is a unsigned int that will be a negative power mip-map bias).

For 11 mip-map levels and 128x128 pages, we have 1+7 levels in the mip-map tail which can only be loaded together, so max offset in mip-levels can be -3, so N=2.
Special values of 1 (default = unmapped) and 0 (error = unallocated) at the highest level mip are reserved.
Gives max 32766 tiles, which even with 128^2 tiles gives a 0.25 GTexel cache.
#Method 1
Fragment Shader or Compute Shader figures out if requested texel's page is not mapped, then with subgroup operations the list is compacted and removes duplicates (with special step that only keeps the max mip-map level if same tile is requested). If Compute Shader then we can remove duplicates for whole workgroup in shared memory.
The requested tile indices are appended to a global list, which then gets read back to CPU a few frames later, processed temporally to avoid trashing and PCIe saturation, and streamed in small increments.
Method 2
There's a storage image or SSBO (whichever is faster) the size of the indirection texture (so only practical for small megatextures), whenever we want to request a tile we tag a cell in the indirection texture copy.
This is nicer and faster because no atomics are required and lots of tag writes can be skipped thanks to subgroup operations and shared memory.
This could be blazingly fast for indirection textures 1k^2 and less.
The resulting tag-array could be scanned and cleared periodically (with more frequent scans scheduled for higher mip-levels), and made to output a compacted list of upload requests.

Data concerns of streaming
Method 1
Extend loader override to provide factory functions for assets.
Use a custom allocator backed ICPUBuffer, have the override provide lazily allocated CPU assets that are actually backed by an allocation from a persistently mapped readonly HOST side IGPUBuffer.
Then any streamer can request {IGPUBuffer,offset,range} as streaming commands to carry out later.
Method 2
Streamer has a callback/override that provides data for the portions of the already dummy IAsset objects that it wants to stream.

How to Deal with Multiple UV-Coords
Initial idea of pretransforming the UV coords to express Virtual Megatexture Coordinates flies out of the window, since different textures can be accessed with the same UV set. However the different textures will end up in different coordinate ranges of the virtual texture, or in different virtual textures alltogether.
One solution is to keep per-object data that includes virtual texture base offsets plus scale factors which can be added to the original UVs, this has the positive effect of not hindering our mesh loading speed (by not modifying the UVs).

done by @Crisspl

Render Feedback Tile-Request 2.0
##Step 1
Fragment Shader or Compute Shader figures out if requested texel's page is not mapped, then with subgroup operations the list is compacted and removes duplicates (with special step that only keeps the max mip-map level if same tile is requested). If Compute Shader then we can remove duplicates for whole workgroup in shared memory.
So we sort by (in order of importance):

MipMap level
VT layer offset
VT XY offset as a morton code

we can then knock-out duplicate entries that ask for the same texture (but different views) and the higher resolution mip-map tile requests (performing a max() of the priorities).
( we could even allow to reduce storage costs for same image view with different edge wrapping modes )
This should be doable in just 2 passes. within a subgroup/workgroup.
Then we tag the images (SSBO uint array that has same number of elements as total count of STexData) that have at least one page request outstanding. We can store the current frame number instead of a simple 0,1 (no request, request outstanding).
We also store into an image of identical dimensions as the page table at the positions of the pages requested, their priority of request.
Step 2
Run a compute shader of fixed size (but can be skipped by indirect dispatch) that analyses the whole STexData SSBO for request tags, then build a histogram (how many page each requested STexData could have) for a DispatchIndirect which will allow 1 invocation per 1 allocated page.
Step 3
Find the tile to analyse by finding the STexData given histogram and gl_GlobalInvocationID.x, then if we find that page was requested we output it to a HOST_VISIBLE download buffer (we only output the first N pages).
We can sort the pages by priority before outputting them to the buffer.
Step 4
Have the CPU read the requests, then prepare the pages for upload, finally carry out the upload on an asynchronous upload (or compute if we decide to update the page table with the same compute shader that does the uploads) queue.
P.S. Steps can be combined into one shader, if:

We use persistent threads
We stagger the stages (doing steps from previous frames)

I should still look at Dx12 Sampler Feedback to see how much next gen we can make this streaming.
Time to render 100GB Moana Scene USD ;)
( need mesh streaming for that - read bounding boxes first, then do culling on "ghosts" to determine objects that need to be streamed in)
Or better make mesh virtualization the same way we do texture virtualization (page = triangle bundle)


🐳 Build docker image on different architectures

Today docker hub registry contains only amd64 architecture.
This PR modifies docker publish script in order to have multiple architectures available. In this case arm64 and amd64.
The image is published only when there is a tag on the master branch.
4 labels are published:

latest
version (ex. 1.7.1)
major.version (ex. 1.7)
major (ex. 1)

I have tested this in my fork hub.docker.com/r/cristianpb/goaccess/tags

Merged. Thank you for submitting this PR!


Use "go get" to install ethr

We don't need to clone and build the ethr manually.
Just run go get github.com/Microsoft/ethr to install the ethr.

Does this builds it also or just downloads? Also which folder it downloads? This is my first golang project so not sure of all options and want to make sure we have detailed instructions for developers who might be new to golang like me.

You can see the details of go get command with go help get. Here is the part of the go help get.
...
Get downloads the packages named by the import paths, along with their
dependencies. It then installs the named packages, like 'go install'.
...

If you want to only download the package, use go get -d <package>.

Yes, it will download the ethr and deps in $GOPATH/src/github.com/Microsoft/ethr (all folders will be created by go get command automatically), then install (including build) the ethr executable in $GOPATH/bin.
In short, go get does

clone the project in $GOPATH/<project path>.
downloads the dependencies and build the project.
install the executable in $GOPATH/bin.

in the single command.

Can you please resolve conflicts and resubmit? Then I will merge it.

Please squash the two commits :)


Update ehf-electronic-invoicing-in-norway.md

Deleted lines contain links that no longer work. Suggest to remove this until we find updated links

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav-app/LocalFunctionality/Norway/ehf-electronic-invoicing-in-norway.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Configuration dialog improvements

Next sub-page in is still API key. 'API details' because it's not just the Api Key (sic), but also the API Limit.