Remove **options from listen_for method

Change signature of 'listen_for' method by deleting **options
signature from it, as it was broken anyway - you had had no
option to run this with additional named options, but it was
needed to have an ability of pushing options of decorated
function. Instead of it some 'academic' code was added - now
'listen_for' method is a decorator factory which returns
decorator that have new function declared inside. It allows to
push named options of decorated function to it and get rid of
wrong signature of factory.

Got it. Anyway - you can't just leave this as is, cause your decorator maker signature says that you can pass any options to it. And then your function will run with this signature - in this case you must create same signature for decorator maker and for function it decorates, it is not good. Much better to not pass **optinons from decorator to function. As to
TypeError: 'NoneType' object is not callable
I'll fix it right now


Browser memory consumption approaches 1GB physical

I am trying to use the Chromium Browser on Raspbian on a Raspberry Pi 3 to display a WeatherStar 4000+ screen 24/7 but after less than an hour, it soaks up so much resident RAM that it crushes the machine (swap is full, load goes over 12...)  The typical RAM footprint for the browser process is 600MB-800MB resident (well over 1GB virtual).
Every time it freezes the Pi, it happens to be displaying the weather radar screen.  I am not certain if it's something on that screen or something it's preparing for the next screen, but it's at least a consistent behavior.
It runs fine on a laptop with 8GB or more.

Hello @ethandicks I don't have a Raspberry Pi 3 to test with so I need to ask you some questions.
Do you notice the increase in memory happening with the "Auto Refresh" option disabled?  Also, does the issue happen if you mute the music?
My guess that this memory leak is related to something in the core rendering engine that isn't being released.

BTW, I did some testing on Chrome in Windows 10 Pro and I saw roughly between 100-140 MB of RAM used.  I didn't see it get it any higher.  I did a quick Google search it a few people have reported memory issues in Chromium: https://bugs.launchpad.net/ubuntu/+source/chromium-browser/+bug/1751137

@ethandicks Did you happen check to see if you are able to run the site in Firefox without any issues?

To answer all at once:
I have tried it with and without music muted.  I observed no difference in memory behavior.  I have not tried it without Auto Refresh enabled.  I can add that to the testing regime.
The Chromium issues you mentioned seemed to have been reported when other software elements were also involved, AdBlocker, and a Gnome Panel Extension (which I do not have - this is a base Raspbian Install - no browser extensions).
I did try with Firefox on the RPi3.  It also consumed large amounts of memory, and in addition, the scrolling of the Travel Forecast was quite stuttery.  I did not continue testing.
I haven't done it yet, but I did start to look into engaging a memory profiler.  It won't take long to run.  I went from startup to lock-up from 2 to 4 hours on numerous occasions.
For as long as it has enough RAM, it runs just fine on a 1GB RPi3.  It might even run just fine on a $10 Pi Zero W, but I'm leaving that test for later.  In the meantime, I pulled out a Pi400 (essentially a Pi4 w/4GB RAM in a keyboard form-factor).  It won't run out of RAM as quickly.

@ethandicks thanks for the details!
I have not been able to find anything that would indicate why my site would consume large amounts of memory.  The fact that the issue doesn't happen in Windows leads me to believe that this is an issue with the Linux implementation of the Chromium browser.
I would suggest that you opened a bug report (https://bugs.chromium.org/p/chromium/issues/entry) with the details you provided here.
Let me know what they say!

I've been trying to do the same thing. I have a Raspberry Pi 3 A+ and a WM8960 hat with a speaker I've been setting up for use as a weather display. In stock configuration, WeatherStar 4000+ crushes it completely. I've been able to get it usable with the following performance tweaks:

Enabling zram-swap:
https://github.com/foundObjects/zram-swap
I have found that _zram_fraction="2/3" is usable.

Firefox instead of Chromium. I notice a slight performance increase. Chromium still worked, Firefox seems better.

After doing that, the system is definitely slow and fully utilized, but performance is acceptable and is even able to play that glorious audio.

The only thing I haven't figured out how to do is launch WeatherStar 4000+ at startup in fullscreen. It seems like I will need to send some sort of mouse event to trigger a click of the fullscreen button.
I tried this setup with every web browser I could find. Firefox was subjectively best for me.

@brutkows-github thanks for the information!
As far as getting the site to start in fullscreen by default doesn't work in web pages because it requires user interaction: https://www.codeproject.com/Questions/367556/Full-Screen-on-Page-Load


Support for PEP-8 Assembly language + Syntax Highlighting

Hi,
PEP-8 is not only Python's conventions for code formatting, it is also a dead simple toy assembly language, the repo of the language/environment itself is available on Google Code.
There's not a lot of use for it, being a toy language, it is never used in production environments.
However, we do use it for teaching purposes in a course at the university I'm attending, being a simplified assembly-level language, it is easier for students to approach machine-level concerns without getting too overwhelmed by the complexity of real-world assembly languages such as x86 or ARM.
I'm using Github as a platform for teaching material, since there will be some PEP-8 code hosted here, I figured this would be a good time to try and have the language recognized and highlighted by Github.
Some students also use Github as a mean to versionize their code, I have a few examples of assignments done by students during previous sessions of the course:

4-in-a-row
Brainfuck interpreter
CamelCase to snake_case converter

There might be other examples of PEP-8 code on Github, but not that I personally know of.

Hi @R4PaSs - Not from GitHub but I can help building the pull request.
Currently the Travis build is failing because the samples need to be in a samples/Pep8 directory.
Also, is the official name Pep8, PEP-8 or PEP8? (it's important because once it's set, it can't be changed :/)

Yup, sorry I just realized the PR was failing, I forgot to make a Pep-8 folder !
Hmmm, this is a tricky question for the name. As I'm not personally involved with the development of the language itself, I don't know which is the official name.
I'd go for Pep-8, on the Google code project, it is named as "Pep/8 Assembler and Simulator (8.1.3)"

Might be a good idea to ping the developers. Do you know any on GitHub?

Unfortunately no, I don't know anyone involved with the language, since the project is not hosted on Github, I don't think they have an account on the platform :/

On their Google Code project, when it's not called Pep/8, they usually write it as Pep8, without any dash character, I'd suggest going with that name in this case, I'll change the name in the grammar to comply with it.

Hi @R4PaSs, thanks for opening up this Pull Request. I'm trying to gauge how popular Pep8 is on GitHub using search like this.
There looks to be a couple of different file types here:

Pep8 files
Some kind of data files such as this

This second set of files looks to be associated with bioinformatics files. Doing some digging I think these are peptide sequences in FASTA format. .pep isn't a particularly common extension.
To add Pep8 I think we should also add an entry for these FASTA files. @larsbrinkhoff you recently added Formatted - do you think we should put the extensions for these FASTA files under this 'Language'?

Most Formatted files contain numers in columns.  However, this set looks a little bit like the FASTA files.

With this search for PEP8 files, I found only 67 repositories from 58 different users.
@arfon Since it's a conflicting file extension, should we wait for a greater adoption of PEP8 on github.com?

With this search for PEP8 files, I found only 67 repositories from 58 different users.
@arfon Since it's a conflicting file extension, should we wait for a greater adoption of PEP8 on github.com?

@pchaigno any chance you could re-run this analysis?

Hi everyone,
I'm recalling this PR has been open for over a year now (wow, nearly two, time flies), is there anything I can do to help it being merged into master aside from fixing the conflicts that have arisen since its opening ?
Are there still conflicts with FASTA files? Is there a way I can deambiguise Pep/8 (and Pep/9) files from FASTA directly from the code I'm submitting?
Sorry for forgetting this, and thanks for the previous answers !

@R4PaSs Lets see if we can get this PR moving again.  As a start, can you please update your fork with the current master of github/linguist and resolve any conflicts.

Hi @lildude, I updated the files to be rebasable on master.
I still have a license issue with the automatic tests which I'm not sure how to fix, since the license is a wtfpl and this is in the whitelist, I don't know why the test keeps failing, am I missing something?

@R4PaSs the test is failing because you're pointing to a SHA in the https://github.com/R4PaSs/Sublime-Pep8 repo from a point in time before you added the LICENSE.md file.
You can confirm this yourself by cloning locally, running script/bootstrap and then view the contents of vendor/grammars/Sublime-Pep8.  You'll see it doesn't contain the LICENSE.md file.  Running git checkout master will bring the file in and you should now find that test passes.
Changing the submodule SHA to 3c47bf219395a66f3fd2f7b922bbb58bbfdacdff (which is your master as it stands right now) should do the trick.

@lildude that indeed did the trick, thanks!

@lildude Looks good to me, sir!


Make the directories require forward slashes instead of backslashes

This would most likely break a LOT of music mods, but simple things such as:
  "levels": {     },     "7-2": {         "clean1": "7-2\7-2Intro.wav",         "battle1": "7-2\7-2Intro.wav"
don't work because it uses backslashes. I don't know much about JSON, but I've tried all of the "solutions," and now I know one thing about JSON: It gets really mad when you try and use backslashes in a string.
So please, make it so forward slashes are used instead of backwards ones because they are so much easier to work with (probably idk)
(fyi, i don't even know if this is possible judging that USTManager uses JSON entirely and it might not accept forward slashes)

nevermind it does use forward slashes im just a dumbass


Desktop Ticker

Hi,
I love Newsblur, and I'll love it more if I has a news ticker.
Alex

If you'd like to see something like this get built it'll have to be a third-party extension or app. It would be great, but could possible use better backend notification support. I'm going to close this ticket until I commit to building something like this.
If you want other folks to vote this up, head to getsatisfaction.com/newsblur and post this as an 'Idea'. I'd love to talk about it there.


:pencil2: fix typo in progressbar/tutorial004.py

In the tutorial, the length of the progress bar is set to 1000 and the update was mistakenly set to 2500. The update should be 1/4 of the length given that the loop runs for 4 iterations. This commit sets the update amount to 250 which better showcases the feature.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #63   +/-   ##
=======================================
  Coverage   99.96%   99.96%
=======================================
  Files         166      166
  Lines        2925     2925
=======================================
  Hits         2924     2924
  Misses          1        1

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 87fe78c...6cc922e. Read the comment docs.

Ah! Good catch, thanks @ValentinCalomme ! :rocket: :cake:


Update killercoda scenarios after breaking changes in k3s

What this PR does / why we need it:
Fixes broken killercoda scenarios

Corrects test for k3s installed
Updates to default ubuntu 4g image

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #46
Special notes for your reviewer:

/hold
Something is wrong with the cdi lab after rebase.

Hold has been released, current branch is working, reviewers can test at:
http://killercoda.com/cwilkers/

@cwilkers I tried the link provided to run through the scenarios. The 101 scenario did not finish waiting for the .kube/* file to exist. After 10 minutes, I exited the scenario.
The cdi scenario worked fine.

/approve


Throwing knives aren't blacklisted in the pneumatic cannon

Description

#26628 was supposed to blacklist throwing knives from entering pneumatic cannon but i just heard from someone that they tested it and it doesn't work as intended.
It is FAR too powerful in the cannon right now, and is not how i intended it to be balanced around.

Just looked into it further #26878, there should still be some sort of nerf in the cannon such as allowing it to just one knife at a time.

This adjustment could potentially be the death knell for the art of utilizing the throwing knife.
The predominant allure of the cannon, despite its occasional glitchiness in shooting in reverse, lies in the scripted cooldown for projectile throws.
If a manual were to exist that provided instruction on advanced throwing techniques, the cannon would likely fall out of favor among those in the know.

really?

blacklist was removed from pneumatic cannon, but its storage was reduced to 2x2, afaik

really?

The pneumatic cannon is pretty bad already it doesn't need more nerfs, that'd be borderline absurd.

I didnt realise that sorry


bug question

When I loaded the plane and move phone ,the track 3d model  can't quick follow ,and it seems the phone very slow .

public void  init(){

    if(mScene !=null) {
        mScene.setListener(null);
        mScene.getRootNode().removeAllChildNodes();
        mScene.getRootNode().removeAllLights();
        mScene.getRootNode().removeAllSounds();
        ((ViroViewARCore)mViroView).setCameraARHitTestListener(null);
        mScene.resetPointCloudSurface();
        controller.surfaces.clear();
        mCrosshairModel= null;
        mScene.dispose();
        mScene = null;
    }
    mScene = new ARScene();
    mScene.displayPointCloud(true);

    // Create an TrackedPlanesController to visually display tracked planes
    controller = new TrackedPlanesController(getActivity() ,mViroView);

    // Spawn a 3D Droid on the position where the user has clicked on a tracked plane.
    controller.addOnPlaneClickListener(new ClickListener() {
        @Override
        public void onClick(int i, Node node, Vector clickPosition) {
            //createDroidAtPosition(mCrosshairModel.getPositionRealtime());
        }

        @Override
        public void onClickState(int i, Node node, ClickState clickState, Vector vector) {
            //No-op
        }
    });

    mScene.setListener(controller);

    mViroView.setScene(mScene);
    ARHitTestListenerCrossHair mCrossHairHitTest = new ARHitTestListenerCrossHair();
    ((ViroViewARCore)mViroView).setCameraARHitTestListener(mCrossHairHitTest);
    AmbientLight  mMainLight = new AmbientLight(Color.parseColor("#606060"), 2500);
    mMainLight.setInfluenceBitMask(3);

    mScene.getRootNode().addLight(mMainLight);
    initARCrossHair(mScene);
    init3DModelProduct(mScene);
    Drawable drawable = getActivity().getResources().getDrawable(R.drawable.point_colud);
    Bitmap bitmap = drawableToBitmap(drawable);
    Texture chineseCharTexture  = new Texture(bitmap, Texture.Format.RGBA8, false, true);
    Material material1 = new Material();
    material1.setDiffuseTexture(chineseCharTexture);
    Surface surface = new Surface(0.25f, 0.25f);
    surface.setMaterials(Arrays.asList(material1));

    mScene.setPointCloudSurface(surface);
}

I changed the color of point but occasionally it was white. Drawable drawable = getActivity().getResources().getDrawable(R.drawable.point_colud);
Bitmap bitmap = drawableToBitmap(drawable);
Texture chineseCharTexture  = new Texture(bitmap, Texture.Format.RGBA8, false, true);
Material material1 = new Material();
material1.setDiffuseTexture(chineseCharTexture);
Surface surface = new Surface(0.25f, 0.25f);
surface.setMaterials(Arrays.asList(material1));
mScene.setPointCloudSurface(surface);

I use this do full scrren .. but in viroview I don't know why somtime it not work.
@Override
public void onWindowFocusChanged(boolean hasFocus) {
super.onWindowFocusChanged(hasFocus);
if (hasFocus && Build.VERSION.SDK_INT >= 21) {
View decorView = getWindow().getDecorView();
decorView.setSystemUiVisibility(
View.SYSTEM_UI_FLAG_LAYOUT_STABLE
| View.SYSTEM_UI_FLAG_LAYOUT_HIDE_NAVIGATION
| View.SYSTEM_UI_FLAG_LAYOUT_FULLSCREEN
| View.SYSTEM_UI_FLAG_HIDE_NAVIGATION
| View.SYSTEM_UI_FLAG_FULLSCREEN
| View.SYSTEM_UI_FLAG_IMMERSIVE_STICKY);
}
}

Hi @544531645,
1)For the 1st issue, the slow tracking, what device are you using? What Android OS is it?
2)For the 2nd issue, how often do you see the color of the point cloud turn white? Do all the dots turn white or just some of them? Can you send a screenshot if possible?
3)For the 3rd issue, the correct way to override the system UI Visibility flags is to do the following:
Create a class that inherits from ViroViewARCore and override the getSystemUIVisibilityFlags method:
 @Override
 protected int getSystemUiVisibilityFlags() {
 return (View.SYSTEM_UI_FLAG_LAYOUT_STABLE
                | View.SYSTEM_UI_FLAG_LAYOUT_FULLSCREEN);
}

1 . android pixel
2.

open twice  it is different

3.getSystemUiVisibilityFlags this method at where setSystemUiVisibilityFlags

I want the scrren is second , no stats bar , no keyback button

but sometime have ,somtime no have ... like point clound sometime is white

if I delete Drawable drawable = getActivity().getResources().getDrawable(R.drawable.point_colud);
Bitmap bitmap = drawableToBitmap(drawable);
Texture chineseCharTexture = new Texture(bitmap, Texture.Format.RGBA8, false, true);
Material material1 = new Material();
material1.setDiffuseTexture(chineseCharTexture);
Surface surface = new Surface(0.25f, 0.25f);
surface.setMaterials(Arrays.asList(material1));
these code ,
use defalut point ,  it not slow

Hi @544531645, thanks for reporting this issues. I've adding them to our backlog and will update this github issue when they are fixed.

This has been fixed in the latest release of ViroCore 1.7.2. Please checkout the update instructions here: https://virocore.viromedia.com/docs/releases


Failure when opening Tab

Our Chatbot comes with multiple tabs. Some user have problems opening this tabs. The same tabs are working for hundreds of other users. For this reason it is hard to say why the bugs appear. Different problems appear:

@sueess - We are looking into this I will get back to you soon.

@sueess - Could you please share BotID and Timestamp so that we can investigate from our end?

We get ever day multiple mails from our users, that the tabs are not working. We try to gather more information from them. Two things are for sure:

There are tab issues on MS Teams
The bugs do not occur for everyone (or in every scenario)

Since we are not able to look in the MS Teams source you have to tell me, how we can support you by solving that issue?

This is not reported bug. This comes from HTTP request to our app. We already corrected this error and fix will be published in next days.
The reported problem occurs before any requests are sent from Tabs.

@MajaEffenberg -Could you please provide a Fiddler trace by any chance?

@MajaEffenberg - We are not able to investigation on it further due to unavailability of network traces.
If you are not able to repro, shall we close it?
If you will face this issue again, feel free to re-open or raise another issue.

@MajaEffenberg -We will wait for your response.

Hello @sueess , @MajaEffenberg - Hope you are doing well !!
How everything going on ?
Did you face the error by any chance ?
Are you able to capture the network traces/logs ?


Switch to Micrometer

@objectiser wrote:

Currently the https://github.com/opentracing-contrib/java-metrics project provides a prometheus only reporter.
A micrometer based reporter should be implemented, and the current prometheus reporter deprecated.
NOTE: the current metric names are specific to the way that prometheus reports histogram data - using three parts representing the 'count', 'sum' and 'buckets'. For micrometer, it may be better to report the data differently in a way that would be consistent across different metric backends - and as a consequence it may be better to use the micrometer API directly rather than having the reporter interface.

@objectiser should the Spring Auto Configuration for Prometheus use the new Micrometer backend? Or should it also be marked as deprecated, and require the target applications to provide the necessary beans?

@jpkrohling Depends how you want to implement it - the MetricsObserver could directly interface to micrometer cutting out the MetricsReporter abstraction - although that might mean having to worry about deprecating more. And then the metrics observer spring auto config + a suitable micrometer backend in the classpath may be enough.
But if keeping the MetricsReporter and providing an alternative micrometer based impl, then I guess have a separate opentracing-metrics-micrometer-spring-autoconfigure artifact?

@objectiser do you think this can be closed?

Closed by #27


SILO_URL Doesn't Propagate to UI Deployment

Description
The SILO_URL environment variable is not propagated down to the scale-ui deployment.
Reproduction Steps
Steps to reproduce the problem:

Deploy new scale container on DC/OS cluster (ensure SILO_URL is set under the scale environment variables)
View the scale-ui service environment variables and note SILO_URL is not one of them

Expected behavior
The SILO_URL value is passed down to the scale-ui deployment
Version and Environment Details

Scale API: v7.0.0-7.0.0rc4
Can be found at /api/v6/version/
Scale UI: v0.5.0

Can be found at the bottom of the UI interface.
Cluster details below are very helpful for problem resolution, but external Scale users aren't expected to provide.

DC/OS: 1.10.9 Enterprise
Marathonlb: 1.5.12

This is as designed. SILO_URL should never be set up on the UI container unless you are wanting to skip the reverse-proxy and will likely cause CORS issues. SILO_BACKEND is the proper value and as far as I can tell, is being set correctly.  This is an inconsistency between the UI and the Scale variable and could be changed to clear up any confusion.

Closing due to not an issue

Reopening to resolve inconsistency between Scale deploy and UI.


CUPS Is ``All Hat and No Cattle''

I am running CUPS 2.3.1 on a Dell Precision 7400 running Ubuntu 20.04 (Focal Fossa), driving an EPSON XP7100 connected via WiFi.  The printer has been connected via the Ubuntu Settings/Printers page, the CUPS Administration/Printers page, and via command-line lpadmin command.  The various displays show the printer as connected to localhost,'' and the printer works--the Epson Printer Utility successfully prints nozzle tests on it.  However, it appears that the CUPS software is a pretty front end which produces job queues, but the functionality of putting ink on paper is missing.  All print jobs hang, with the message Unable to locate .local.''  The CUPS debugger returns the error messages Scheduler Not Running'' (although cupsd is running), and There is no solution to this problem.''
So what's the missing trick?  (There is ALWAYS a missing trick.  In the entire history of computer science, there has only ever been one software package which actually does what its documentation says it does, and CUPS isn't it.)  Specifically, 1) How do I force the computer to convert the printer from connected'' to located,'' and 2) how do I force CUPS to transmit a job from a job queue to the physical printer?  (I should add that CUPS occasionally puts messages like ``Printing  on  on the screen, but the physical printer does nothing at all.)  Thank you in advance for your advice, colleagues.  IHR

You have two avenues to explore:

Read the initial portions of this page and decide whether you have posted to the appropriate venue.
Read on and decide whether the EPSON XP-7100 fits into  Ubuntu 20.04.

4 August 2023
Dear Mr. Potkin,
Thank you for taking the time to consider the Issue I posted.  The document to which you direct me suggests that I certainly did post to the right venue, as it appears to offer a work-around to my ongoing CUPS problem.  The document also answers your second point, as it does list the XP-7100 as AirPrint compatible.  Obviously I've got a lot more work to do on this.  But many thanks again for your helpful suggestions.
Sincerely,
Ian H. Redmount

From: Brian Potkin @.>
Sent: Friday, August 4, 2023 2:36 PM
To: apple/cups @.>
Cc: Ian Redmount @.>; Author @.>
Subject: [External] Re: [apple/cups] CUPS Is ``All Hat and No Cattle'' (Issue #6139)
CAUTION: This email is from an external source. Do not click links or open attachments unless you recognize the sender and know the content is safe. When in doubt, please report the email to SLUAware by clicking on the three dots within your email and choosing Report To SLUAware.
You have two avenues to explore:

Read the initial portions of this pagehttps://urldefense.com/v3/__https://wiki.debian.org/CUPSDriverlessPrinting__;!!K543PA!NHncND7oSqLI-PF8sp1qsi3aZI8J_1FPuorkJ7O5DotcRhrciFJkFDRduEiJ1pitC8_ujImrAJLYABWnnLS6iPcAMu4$ and decide whether you have posted to the appropriate venue.
Read on and decide whether the EPSON XP-7100 fits into Ubuntu 20.04.

—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/apple/cups/issues/6139*issuecomment-1666083206__;Iw!!K543PA!NHncND7oSqLI-PF8sp1qsi3aZI8J_1FPuorkJ7O5DotcRhrciFJkFDRduEiJ1pitC8_ujImrAJLYABWnnLS6xoOfO_A$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/BBXCAR2M7LANPB3QACSZZ4DXTVFLJANCNFSM6AAAAAA3EMG4II__;!!K543PA!NHncND7oSqLI-PF8sp1qsi3aZI8J_1FPuorkJ7O5DotcRhrciFJkFDRduEiJ1pitC8_ujImrAJLYABWnnLS6f0bzAxI$.
You are receiving this because you authored the thread.Message ID: @.***>

Give the outputs of
avahi-browse -rt _ipp._tcp

avahi-browse -rt _uscan._tcp

driverless

Thanks for the response.  Here are the outputs you request:
browse01.txt
browse02.txt
browse03.txt
in order.  What do these tell us?
As it happens, CUPS will print its queues properly after it crashes and reboots.  (Exactly what crashes and reboots I'm not sure.)  The timescale for this is days, not minutes or hours.  So is there a command by which I can force CUPS to find its printers?
Thanks again.  IHR

The content of the three files tell us that the EPSON XP-710o is capable of driverless printing and scanning. Furthermore, it indicates a queue setup with
lpadmin -p xp7100 -v "ipp://EPSON%20XP-7100%20Series._ipp._tcp.local/" -E -m everywhere

should lead to printing taking place wiith
lp -d xp7100 /etc/nsswitch.conf

No joy.  The given lpadmin command returns the response
lpadmin: Unable to connect to "EPSONB78BD4.local:631": Name or service not known
The system shows the indicated printer as connected to localhost.  With the localhost:631 server gone, it is difficult to see what is going on.
I might point out that of the 111 open issues here, scores report problems like mine:  printers the software can't locate, job queues which hang, CUPS which only runs after a reboot, etc.  This suggests a flaw in the software, rather than a user error.


Backport resource server filter chain order fix

Spring Boot 1.5 moved the OAuth2 resource filter order from 3 to SecurityProperties.ACCESS_OVERRIDE_ORDER - 1. This should be reflected in this project for Spring Boot 2 by backporting:

https://github.com/spring-projects/spring-boot/issues/5072
https://github.com/spring-projects/spring-boot/issues/8050
https://github.com/spring-projects/spring-boot/issues/8347

I've belatedly found https://github.com/spring-projects/spring-boot/issues/7958 and so I assume this is now by design. If that is indeed the case, what is the recommended approach to correctly order OAuth security filters? For example, with Spring Boot 1.5 I had:

AuthorizationServerConfigurerAdapter (order = 0)
WebSecurityConfigurerAdapter (order = 100)
ResourceServerConfigurerAdapter (order =<PHONE_NUMBER>)

Whereas Spring Boot 2 results in:

AuthorizationServerConfigurerAdapter (order = 0)
ResourceServerConfigurerAdapter (order = 3)
WebSecurityConfigurerAdapter (order = 100)

Apologies if this is now more of a question rather than an issue.

Furthermore, the docs state:

OAuth2 resources are protected by a filter chain with order security.oauth2.resource.filter-order and the default is after the filter protecting the actuator endpoints by default (so actuator endpoints will stay on HTTP Basic unless you change the order).

This is true for Spring Boot 1.5 but not for this project.

@markhobson can you clarify what you are specifically struggling with?
It does sound like the documentation may need updating, but I also wonder if https://github.com/spring-projects/spring-security-oauth2-boot/blob/master/samples/spring-boot-sample-secure-oauth2-actuator/src/main/java/sample/ActuatorSecurityConfiguration.java answers your question?

Hi @jzheaux, I was migrating an application:

from Spring Boot 1.5 + Spring Security 4.2 + Spring Security OAuth 2.3
to Spring Boot 2.1 + Spring Security 5.1 + Spring Security OAuth2 Boot 2.1

It configures an OAuth authorization server, an OAuth resource server, and a custom form login. After migration the login form was receiving a 405 Request method 'POST' not supported upon submission. It wasn't overly clear what was happening until I discovered the change in security filter ordering. Now the resource server was incorrectly handling the login submission.
Setting the form login security configurer to @Order(2), as spring-boot-sample-secure-oauth2-actuator also does, does indeed fix it. Although this felt like a hack and wasn't mentioned in the docs so thought that this was an oversight. I since realise that this simplification is by design, but I would suggest either improving the OAuth + form login out-of-the-box experience somehow, or explicitly mentioning this in the docs.

improving the OAuth + form login out-of-the-box experience

@markhobson, would you be willing to share a minimal sample of what you are trying to do with form login? I'd be happy to take a look and see what can be done to simplify, either by using existing APIs or seeing if something can be done via auto-configuration.

Documentation updated via dab82f9

Hi @jzheaux, sure, here's a simple example of the problem consisting of a resource server and a custom login form:
https://github.com/markhobson/spring-security-oauth2-boot-issue
Running the test attempts to login using the custom form. Due to the new filter ordering the login POST is intercepted by the resource server which returns a 405. If you uncomment the @Order in Application to move web security before the resource server then it works as expected.


flutter outline plugin suggestions

would you offer us warp with Container
add padding seems to be  have bug ,The loss of new key word may lead to run errors

I really like this plug-in

@devoncarew do you think there is still something to do?

Issue moved to flutter/flutter-intellij #2566 via ZenHub


Debug causes crash

We have written the needed data into your clipboard because it was too large to send. Please paste.

/needsMoreInfo

Data not provided -> closing this issue.


Update to Go v1.18

https://github.com/stolostron/backlog/issues/22476
https://github.com/stolostron/backlog/issues/24860

/retest

/cherry-pick release-2.5

@dhaiducek: #418 failed to apply on top of branch "release-2.5":
Applying: Update to Go v1.18
.git/rebase-apply/patch:20: trailing whitespace.
  tag: go1.18-linux
warning: 1 line adds whitespace errors.
Using index info to reconstruct a base tree...
M	.travis.yml
M	Makefile
M	go.mod
M	go.sum
Falling back to patching base and 3-way merge...
Auto-merging go.sum
CONFLICT (content): Merge conflict in go.sum
Auto-merging go.mod
CONFLICT (content): Merge conflict in go.mod
Auto-merging Makefile
Auto-merging .travis.yml
CONFLICT (content): Merge conflict in .travis.yml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Update to Go v1.18
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

This is handled in #421


Acceptance test failure with SingleNamespace mode

Acceptance-test stage failed for all platforms due to this failure:
****** Waiting for rco-controller-manager to be ready...
Expect operator image version: [icr.io/appcafe/runtime-component-operator@sha256:29fed59e5aa9e01c7aa955e00e01e6643930a6c6027a01744628a3a2f3fb049e](http://icr.io/appcafe/runtime-component-operator@sha256:29fed59e5aa9e01c7aa955e00e01e6643930a6c6027a01744628a3a2f3fb049e)
Actual installed operation image version:
Install Operator image  does not match correct image [icr.io/appcafe/runtime-component-operator@sha256:29fed59e5aa9e01c7aa955e00e01e6643930a6c6027a01744628a3a2f3fb049e](http://icr.io/appcafe/runtime-component-operator@sha256:29fed59e5aa9e01c7aa955e00e01e6643930a6c6027a01744628a3a2f3fb049e)
make: *** [Makefile:348: test-pipeline-e2e] Error 1
***** End of ocp-e2e-run-X logs

***** Overall test results
[FAILED] kind-e2e-run: Exited (2) 55 seconds ago
[FAILED] ocp-e2e-run-P: Exited (2) 28 seconds ago
[FAILED] ocp-e2e-run-X: Exited (2) 31 seconds ago
[FAILED] ocp-e2e-run-Z: Exited (2) 29 seconds ago

This looks like the script is querying the wrong namespace for operator pods, and so is failing to find the actually installed image sha. So the sha comparison will always fail.

Resolved by PR https://github.ibm.com/websphere/operators/pull/94


Adding OMAPI functionality to the DHCP Server settings.

Supports setting a port and defining or generating a key. Service is enabled when settings are defined. Validation logic added on saving to verify the port is valid and not in use as well as that a key is defined or being generated. If validation fails, the user is presented with a descriptive error message.
Feature#7304
Also updated the write_settings call on the DHCP server page. No arguments were being passed so changes came up as unknown on the config history screen. Config history will now reflect the UI section and interface where change was made.

[x] Redmine Issue: https://redmine.pfsense.org/issues/7304
[x] Ready for review

Good looking out Jim. Commit added.

@Erik10206 make a base functionality to depend of an additional package being installed is not a good way to go IMO.  Based on what I saw on ISC-dhcpd 4.4.x manpage it supports following algorithms:
       The key name, algorithm, and secret must match that being used by the
       DNS server. The DHCP server currently supports the following
       algorithms:

               HMAC-MD5
               HMAC-SHA1
               HMAC-SHA224
               HMAC-SHA256
               HMAC-SHA384
               HMAC-SHA512

Can't you generate a key using any of those protocols with dnssec-keygen?

@Erik10206 make a base functionality to depend of an additional package being installed is not a good way to go IMO. Based on what I saw on ISC-dhcpd 4.4.x manpage it supports the following algorithms:
       The key name, algorithm, and secret must match that being used by the
       DNS server. The DHCP server currently supports the following
       algorithms:

               HMAC-MD5
               HMAC-SHA1
               HMAC-SHA224
               HMAC-SHA256
               HMAC-SHA384
               HMAC-SHA512

Can't you generate a key using any of those protocols with dnssec-keygen?

I agree that it isn't ideal but I was looking into the options and the version of dnssec-keygen that is included on 2.5 (9.14.3) has depreciated all hmac algorithms. That is when I came across ddns-confgen and while not included, can be added by simply installing the bind package with no further config needed. The package dependency only affects the auto key generation. OMAPI can be configured and setup without the need for the package when the key string is provided by the user.
This can also be done in code as well. I looked into it a little bit but I can dig into it some more and see if I can eliminate this dependency.

@rbgarga New commit has been pushed. I dug into the source for dnssec-keygen and just replicated the process in php.

You may want to look at phpseclib for making keys, if it has a function like this. We added phpseclib 2.0.x to pfSense 2.5.0 because OpenSSL deprecated some functions for small keys that we needed for captive portal vouchers.

@jim-p I took a look at that library. I would prefer to stick to this key string gen workflow unless you require the changes.

Got those 2 things knocked out. Let me know if there is anything around the function call that you wanted to be changed. I saw that there were other functions that had separate v4 vs v6 definitions but it's the exact same command so that is why I chose it as an arg rather than making v6 and v4 thier own functions. Let me know if you would prefer otherwise.

Hey @jim-p, I looked into squash and merge and it looks like it requires write permission to do so via github. Is there another way you would recommend?

@Erik10206 When "key algorithm" changes, shouldn't OMAPI key change as well? If a sha256 key was first generated then algorithm changes to MD5, can it keep using same key?

@rbgarga yes, the minimum length is deciding factor. The extra characters don't add much but it doesn't hurt anything. However, if you go from MD5 -> sha256, you would be forced to gen a new key if the current one isn't long enough. One thing that I got hung up on initially is that there is really no difference in how the secret (key) is created. The alg is just the method on how the secret is used so it boils down to min char length of each alg's key requirements.

I'll merge this one manually preserving you as Author just to avoid adding all these 14 commits. Thank you!

Merged. Thanks!


add go to definition

once we have type comprehension #123 then add "go to definition" -- checks if file is in current project scope, or if not, pulls up a separate textview dialog window.  should be super simple.
In VS code key is F12, or control / command + mouse click on symbol.

@rohrlich btw I just added an "open file" button to this, for gide specifically, so you can now quickly pull up entire file in editor, if you want more than just the lookup popup.  Also, it now grabs any comments prior to the starting line, which is key.


[5.0] Get the ModelAssistant source compatibility project building with the swift-5.0-branch

Explanation: There were a couple issues with how rethrows diagnostics were dealing with optional function types, resulting in a verification issue in a new source compatibility suite project.
Issue: SR-9102 / rdar://problem/45615204
Scope: Only affects code that has arguments with are optional throwing function types.
Risk: Low.
Testing: Added a new test for the case where we were failing verification.
Reviewed by: @rjmccall, @xedin
Does not affect ABI.

@swift-ci Please test

@swift-ci Please test source compatibility

@shahmishal IIRC, cross-repo testing doesn't work with the source compatibility suite. Am I remembering that correctly?

https://github.com/apple/swift-source-compat-suite/pull/298
@swift-ci Please test source compatibility


Add support for blkio.weight_device

Signed-off-by: Ma Shimiao<EMAIL_ADDRESS>

ping @LK4D4, @icecrime , @jamtur01, @calavera

can you add an integration-cli test, other wise i see nothing wrong here

@Mashimiao Thanks for the contrib! Please see the comments above.

ping @icecrime

looks like this needs a rebase :cry:
also ping @icecrime @jfrazelle PTAL

feel free to ping me after rebase

ping @jfrazelle  rebased

ping @jfrazelle , @calavera , @icecrime  rabased and fixed

Looks like we aren't validating the input from the API.
Also would be nice to put the actual weight-device into a strong type instead of []string.

Hi @cpuguy83
"Looks like we aren't validating the input from the API." what's that mean?  We have a function ValidateWeightDevice to validate the input

@Mashimiao that function is only called by the client. It also needs to be called by the server to validate what it arrives there. This is where we have other similar validations:
https://github.com/docker/docker/blob/master/daemon/daemon_unix.go#L119

@calavera  Thanks. added validations.

ping @calavera , @cpuguy83 , @icecrime

by just running a simple container (in docker dev container) like the examples you wrote I'm getting:
root@8ab3d12abb1a:/go/src/github.com/docker/docker# docker run --blkio-weight-device "/dev/sda:500" busybox true
Timestamp: 2015-09-11 11:24:31.030955354 +0000 UTC
Code: System error

Message: stat /sys/fs/cgroup/docker.service: no such file or directory

Frames:
---
0: setupRootfs
Package: github.com/opencontainers/runc/libcontainer
File: rootfs_linux.go@37
---
1: Init
Package: github.com/opencontainers/runc/libcontainer.(*linuxStandardInit)
File: standard_init_linux.go@52
---
2: StartInitialization
Package: github.com/opencontainers/runc/libcontainer.(*LinuxFactory)
File: factory_linux.go@242
---
3: initializer
Package: github.com/docker/docker/daemon/execdriver/native
File: init.go@35
---
4: Init
Package: github.com/docker/docker/pkg/reexec
File: reexec.go@26
---
5: main
Package: main
File: docker.go@18
---
6: main
Package: runtime
File: proc.go@63
---
7: goexit
Package: runtime
File: asm_amd64.s@2232
Error response from daemon: Cannot start container cc81308e7fec0434b8aa4c0c54613c6b54596105568548357d2421519ea4b55d: [8] System error: stat /sys/fs/cgroup/docker.service: no such file or directory

same happens if I specifying blkio-weight
do you know what's wrong here?

There seems to be some problems running with this new flag though, seems like writing to blkio.weight_device will always fail with invalid argument (at least in docker dev container)
@Mashimiao do you what's wrong? I'm running your examples from this pr

A fix for wrong blkio devices settings is here https://github.com/opencontainers/runc/pull/276
As soon as it's merged I'll update docker deps and you can move on with this PR and https://github.com/docker/docker/pull/15879 and https://github.com/docker/docker/pull/14466

ping @runcom , @jfrazelle , @cpuguy83, @calavera

Can we use the human units package like we do for memory

LGTM

after this is merged can we do the bps one next because I want to use it for something :P and I'm selfish

lxc failed for unrelated reasons :)

ping @jhowardmsft Is this ever something we want to support on Windows, and if so, is this parsing going to get us in trouble?

@cpuguy83 Ah, yes. Indeed, this is something we intend to support in Windows, and yes, performing client-side parsing will cause an issue. I would much rather the parsing is done server side...

@Mashimiao Appreciate your patience here. WDYT about moving parsing server side so we can support Windows?
@cpuguy83 / @jfrazelle maybe we could punt on this until Windows support is added for the sake of getting it in?

This should at least wait until the volumes on Windows are figured out.
I'd rather not have to implement hacks to support the path splitting for both platforms b/c we merged too quickly.

i think we can get this in without it for now

Hi @bfirsh  I'm not sure moving parsing server side so we can support Windows? means?
And ping @cpuguy83  I'm not sure why parsing will make us in trouble?
As this PR has been opened for a long time, I want to know what should I do if this PR could be applied?

This PR (and the related bps one) seem super impactful for closing the net around resource-greedy containers. Hope it makes it in, and looking forward to playing with it on the experimental build!

@Mashimiao The problem is this code: https://github.com/docker/docker/pull/13959/files#diff-def63b7f78492fa1f0dc4e219a650df4R183. It is looking for a Unix-specific formatted device. It would be better if you were able to follow daemon-side platform specific parsing/validation of the device (again, blank/no-op on Windows currently), such as can be found in https://github.com/docker/docker/blob/master/runconfig/config.go#L67-L74

@jhowardmsft   Sorry, as I know cgroup is only supported by Linux/Unix. So what's wrong with parsing Unix-specific formatted device? And I used getBlkioWeightDevices() to get device daemon-side.

Opts.go validation is client side (I believe, but only on phone right now to re-verify). What if a Linux CLI is targeting a Windows daemon? It would fail due to the /dev/ check. This is why the parsing and validation should be done by the daemon, not the client.

I think we're good here, for the volumes stuff @tiborvass implemented some client side parsing to handle both cases.

daemon/execdriver/driver.go:12: imported and not used: "github.com/opencontainers/runc/libcontainer/configs" as blkiodev
daemon/execdriver/driver_unix.go:103: c.Resources.BlkioWeightDevice undefined (type *Resources has no field or method BlkioWeightDevice)

ping @cpuguy83 @jfrazelle what's the status here?

LGTM

LGTM moving to doc review

Thanks @Mashimiao!
docs LGTM
ping @moxiegirl PTAL

LGTM thank you @Mashimiao for your patience.  @thaJeztah as you like sir!

restarting the userns and windows builds for good measure

@thaJeztah it's green, merging :wink:


Add cancellable action/icon to the status bar task display

Is your enhancement related to a problem? Please describe
some tasks are now cancellable, and it's possible to cancel them with the task manager
Describe the solution you'd like
Adds a cancel icon next to the progress bar (at the right of the progress bar) in the status bar
it will give the ability to cancel a task
Describe alternatives you've considered
No response
Additional context
No response

follow-up of https://github.com/podman-desktop/podman-desktop/pull/10100


fix comment in bld.bat

Only fixes a small error in a comment.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Good catch, @mbargull !


Include only needed crossbeam dependencies

When building without par-schedule feature one only needs the crossbeam-channel crate, not all the other parts of crossbeam.
This reduces compile times when building with few features. We ran into this because we are building with default-features = false and wanted to reduce amount of dependencies and improve build times.
And when building with that feature, we only additionally need crossbeam-queue. Although in practice in that case one gets rest of crossbeam due to rayon.

I have a change removing crossbeam from non-par-schedule entirely, so it may supercede this PR. SeqQueue usage causes ordering issues for CommandBuffer, so it's being removed


luci-proto-gre: remove extra parenthesis

fix the following error:
SyntaxError
Unexpected token ')'
in http://<IP_ADDRESS>/luci-static/resources/protocol/grev6.js:?
at http://<IP_ADDRESS>/luci-static/resources/luci.js:22
at async Promise.all (index 4)
at async Promise.all (index 5)
Fixes: 2b7fd1292 ("luci-proto-gre: improvement of LuCI interface")
Signed-off-by: Chuanhong Guo<EMAIL_ADDRESS>Not quite sure how the developments here works so I decided to create a PR.

I have also just noticed.
Thanks :+1:


React tooltip renders but does not show anything

i tried to  render react tooltip along with react-simple-maps.
this is the basic structure
const Page = ()=>{
  const [content, setContent] = React.useState("hesss");

  return (
    <Card>
       <MyMapImplementation setContent={setContent}/>
       <ReactTooltip>{content}</ReactTooltip>
    </Card>
  )
}

(in above case i inactivate left and top css so the tooltip would be in the page)
this occurs:

anything did i miss here? i tried to follow this map with tooltip example and seems straight forward.

I have the same issue. Tooltip won't display but is rendered in the html

actually, i solved my react tooltip case in react simple map by adding this:
<ComposableMap
data-tip=""  // <---this line
\>

i dont know why but maybe you can read the caveat part in this link


Ldanzinger/time query

adding time based query sample. @michael-tims please review and merge?

good suggestion. I didn't see that in the intellisense the first time around as I was missing the include. Updated


Incorrect suggestion to force the use of an OAuth token (Bearer Authorization header)

Because of this:
Note - If you use a parameter in a request header or body, that parameter might be visible when you view your logic app's run history and outgoing HTTP request. Make sure that you also set your content access policies accordingly. You can also use obfuscation to hide inputs and outputs in your run history. Authorization headers are never visible through inputs or outputs. So if a secret is used there, that secret isn't retrievable.
This won't work:
Under Trigger Conditions, select Add. In the trigger condition box, enter the following expression, and select Done.
@startsWith(triggerOutputs()?['headers']?['Authorization'], 'Bearer')
So, this documentation is incorrect and there is no way to force OAuth over using a signature. [It would be nice to make this possible though, possible by not stripping the Authorization header but replacing it's value with sanitized or something].

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

ID: 6b13ff46-7867-9262-53fd-a321bd683152
Version Independent ID: 4edb75cf-d61a-bd25-a9c0-179eca2ebc6f
Content: Secure access and data - Azure Logic Apps
Content Source: articles/logic-apps/logic-apps-securing-a-logic-app.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

@edwinvandeburgt
Thanks for your feedback! We will investigate and update as appropriate.

Hi @edwinvandeburgt,
I checked with the product team. Their response is that you can enable the IncludeAuthorizationheadersInOutputs option with the operation. You can then use an expression as mentioned in the doc to check whether the Bearer token is sent.
However, this solution exposes the Authorization header in the trigger outputs.
We'll update the doc to make this option clearer. Thanks!

@edwinvandeburgt: A doc update will go live after 11 AM Pacific Time on July 7, 2023.


prerendering documentation

I had a hard time prerendering. It turns out I needed to set prerender = true in my $layout.svelte or else nothing would prerender. This wasn't clear to me. At the very least we should make sure to document this

Probably makes more sense to disregard prerender for layout components, the same as we do for maxage. No real reason to have to mark the entire component chain


Do you have plan to support Http on vipAddress?

Do you have plan to support Http on vipAddress ? Call http interface between micro services。

I think what you are describing is similar to Ribbon, which is what Netflix uses for communicating between services. (https://github.com/Netflix/ribbon). This module is designed to be purely a Eureka client for registration and fetching the registry. Once the registry is fetched you can make requests to those instances using a variety of HTTP clients. You'll also have to decide what strategy to use when load balancing to the instances. You could randomly pick an instance, or use a weighted or round-robin strategy. The default Ribbon implementation is round robin. I think most users of this module are using their own logic for this, since it allows you to choose what HTTP request library to use as well as what balancing strategy.
Anyway, hope this answers your question. If you do end up building a load balanced HTTP client (similar to Ribbon), it would make a great module on NPM. I think there's probably other users searching for the same thing.

@jquatier ok,3qu,
recommend： https://github.com/resilient-http/resilient.js


docs: fix typo

The examples don't pass a param to generateId(), which leads to an empty returned string.

Thanks!


Update cat_valencia.tsv

Fixed numbers from 21 to 99, hyphen must be used.
Keep alternative forms for 8, 18 and 8[0-9] forms (huit and vuit forms are good)
Added feminine form for numbers ended with 1 or 2, but 11 and 12 (una, dues, vint-i-una, vint-i-dues,...)
Added alternate masculine form for numbers ended with 1, but 11 (u, vint-i-u, trenta-u,...).
Fixed 10^[9,10,11], because Catalan/Valencian uses long scale (one billion is equal to 10^12)

FYI, we don't use hyphens in this data set even  when they're orthographically required (since this data was primarily produced for speech synthesis, we want to keep the number of lexical entries at an absolute minimum).
And we only include one citation form per word; other forms (e.g., a feminine if the masculine is the citation form) are generated using finite-state grammars in our setup.

Thanks again! Spoke with two of our Catalan specialists at Google and I just merged in your suggestion with the long-scale numbers. We don't need the feminine forms, however, in this setup, since we only record citation forms and generate other forms via grammars. Closing for now but if we ever expand the scope of this project we'll include your commits.


Change into educational example

This PR is partially intended to spark discussion about what the chisel-template should be. Historically, I think there have been two competing concerns: education vs. template for repeated use. This PR is an attempt to pull more toward the education side since this is the example build to which we point new users. Others may disagree, but whatever we do I think the style of build.sbt I'm proposing here should exist somewhere.
Another opinion I have is that there really should not be a master vs. release of this repository. I think this should exist as purely a stable example (using normal releases). If someone wants to switch to the bleeding edge I think we should just encourage them to update the versions in this repository rather than checking out master instead of `release.
Another thought, perhaps we should move this to a g8 template since it is a little strange to start a project by inheriting a ton of git history. sbt new chipsalliance/chisel-template.g8 would give a better starting point with no git history (although we should obviously encourage users to use version control).

The master branch isn't for users. It's for testing against the current master branches of the chisel ecosystem. It should probably be renamed to something like developerBranchOnlyForTestingAgainstChiselMaster. No non-developer should be using it.

Maybe rename the current release branch to master, and the current master branch to chiselMaster or something?

I see, so there are really 3 different concerns here:

Example for new comers
Clean template for starting new projects
Testing against master branches

I'm fine with leaving master or even the entire repo for (3). I am now thinking that a repository isn't really the right model for (2) since the repository history isn't really useful for the user; SBT g8 templates or just tarballs to curl seem better here. I feel (1) should be the same as (2), with the potential distinction that (1) has lots of comments in the build files while (2) can optionally leave it out.
Historically, it seems we've tried to make this repository (and in particularly, cloning it) serve all 3 cases. It could continue to do so, if we took a tarball release approach and encourage new users to just download the tarball.
Any thoughts? Am I making sense here?

Github does have a feature where you can download the repo as a zip with no .git or git history. Maybe we can make that a link or something.

For educational usage in my opinion the setup is too long (= complicated). For a clean start, there is no need to support two Scala and two Java versions. Just 2.12 and JDK 8. And there are many files that may not really belong to a start. Do we really want to have some scalastyle here? I personally start with a single and short build.sbt and a single Chisel module. Maybe a Makefile to show the relevant sbt commands.

I agree with @schoeberl (obviously since I made this PR). What do you think about changing the distribution mechanism for the starting template (or possible templates) to either sbt g8 or just tarballs?
For example, this already works with default Github source tarballs on release:
wget https://github.com/freechipsproject/chisel-template/archive/v3.1.1.tar.gz
tar xf v3.1.1.tar.gz
cd chisel-template-3.1.1/

I very much agree with simplifying what we put in these template releases. A g8 template example would look something like:
sbt new freechipsproject/chisel-template.g8

This is similar to a recommended way of starting Scala projects: https://github.com/scala/scala-seed.g8
Also we can do both, and have multiple templates as long as we describe them on a docs page on the website for example.

I agree with @edwardcwang that the "download as zip" form GitHub is an easy to use option for starters. This works for Windows and Unix users. wget/tar is for some too much to start.
The sbt new xx.g8 looks also like a promising start.
For the template itself, I would argue for the very minimum possible. Maybe we have two: 1.) super minimal and 2.) with all the bells and whistles.
For myself, I've created a minimal template. Ok, not super minimal as it also contains a Quartus project for an Intel FPGA. But the build.sbt is super lean (although it is not up to date for the latest Chisel version).
@jackkoenig agree on dead simple, and this is master and not some git checkout release.
Good that we discuss this. All initiatives to easy starting with Chisel are important.

Missed the link to my version of a minimal starting point: https://github.com/schoeberl/chisel-empty
I guess a super minimal HW design plus a simple test should be part of the template. Simpler than the current GCD example: just one design file and one test file. Which tester is up for discussion ;-)
Cheers,
Martin

Just an FYI: I converted this to g8 a while back in https://github.com/seldridge/chisel-template.g8. You can test that to see what the workflow is like.

To add to the template fracas... ;) https://github.com/edwardcwang/chisel-template-lite is a template that attempts to explain all the parts of itself, and comes with a mill template.

This has been done separately, see https://github.com/freechipsproject/chisel-template/tree/main which is the new default branch.


Hide Keyboard

Hi,
I've seen a couple issues with the following problem - however, I am not sure how to fix this for projects iOS 7+. Do you have any ideas / know how to do this?
Thanks!

Thanks @DaliLabs ! :smile:
Please be more specific and update your issue after reading the contribution guidelines!


Problem in return getIdentity( )

In Resource I use the method $this->getIdentity() to indetificar the user, but it is returning null in this latest update.
class PedResource extends AbstractResourceListener
{
public function fetch($id)
{
$user = $this->getIdentity(); //return null
.
.
.
}
}

I think it is because of #117, fixed in #120. Can you check please PR #120 and confirm if it works with your code?

It worked!

This repository has been closed and moved to laminas-api-tools/api-tools-mvc-auth; a new issue has been opened at https://github.com/laminas-api-tools/api-tools-mvc-auth/issues/5.


Add instructions for Mixtral-8x7B inference

What does this PR do?
The Mixtral-8x7B model is too large to be loaded onto a single Gaudi2 device in 32 bits. This PR demonstrates how to run the Mixtral-8x7B model on a single card, by leveraging the --bf16 command line option to run_generation.py.
Before submitting

[X] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

@tjs-intel  as long as it's working on gaudi, it's ok. no need to put into readme unless there is any special arguments.

@libinta the --bf16 argument is required for single-card inference because the model is so large.

Taulor,
Thanks. As there is no new argument added, I don't think we need special instructions for it.
Libin
From: Taylor Jackle Spriggs @.>
Sent: Thursday, March 28, 2024 12:51 PM
To: huggingface/optimum-habana @.>
Cc: Libin Tang @.>; Mention @.>
Subject: Re: [huggingface/optimum-habana] Add instructions for Mixtral-8x7B inference (PR #843)
@libintahttps://github.com/libinta the --bf16 argument is required for single-card inference because the model is so large.

Reply to this email directly, view it on GitHubhttps://github.com/huggingface/optimum-habana/pull/843#issuecomment-2025992853, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGLGHZPHCYB4IZAQ6HGAZHLY2RYBBAVCNFSM6AAAAABFLRNF7GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRVHE4TEOBVGM.
You are receiving this because you were mentioned.Message ID: @.***>


Spike: Run from App

As a user I can run a modeled diagram directly on the Camunda Engine.

Rational and UI Sketches
Cf. slide deck

Assumptions:

we can assume that having max one process instance running in parallel is ok for this scenario
while a process instance is running, the process model needs not be changed

Suggestion:

we can have one view in two modes (modeling vs running) that work as described above.
clicking the RUN button switches to the "running" mode
closing the "running" mode makes sure the process instance is stopped
restarting is simply to hit the RUN button again from the "modeling" mode
the same way as we store the deployment config, we can store the process input variables for subsequent runs, making it easy to restart a process in the very same configuration

What about deploying the first version of my model in a process application?
Usually, if I redeploy and start a process instance I will get ClassNotFoundExceptions.
It's a different problem, but you should be aware that many users fall into this trap.

Thanks for mentioning this.
We are aware of the many ways how simply deploying a diagram via the modeler can break automation.
The aim of this feature is not to replace production deployments but to learn and discover how Camunda works:

It will be of less use for advanced setups (at least as far as this spike is concerned).

We've investigated this and came up with the camunda-playgrund.


tendermint crate uses std features in ed25519-dalek

Part of informalsystems/ibc-rs#1158.
When the Cargo.toml field is set to ed25519-dalek = { version = "1", default-features = false, features = ["serde", "u64_backend"] }, we would get the following compile errors:
error[E0277]: the trait bound `alloc::string::String: DefaultIsZeroes` is not satisfied
   --> tendermint/src/private_key.rs:47:20
    |
47  |     Zeroizing::new(String::from_utf8(Base64::default().encode(&keypair_bytes[..])).unwrap())
    |                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `DefaultIsZeroes` is not implemented for `alloc::string::String`
    |
    = note: required because of the requirements on the impl of `Zeroize` for `alloc::string::String`
note: required by `Zeroizing::<Z>::new`
   --> /home/ubuntu/.cargo/registry/src/github.com-1ecc6299db9ec823/zeroize-1.1.1/src/lib.rs:428:5
    |
428 |     pub fn new(value: Z) -> Self {
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0599]: no method named `serialize` found for struct `Zeroizing` in the current scope
  --> tendermint/src/private_key.rs:48:10
   |
48 |         .serialize(serializer)
   |          ^^^^^^^^^ method not found in `Zeroizing<alloc::string::String>`

I don't see anything there using std.
It's using alloc, and it looks like you don't have the zeroize/alloc feature enabled.

Interesting, didn't realized that it was indirectly caused by zeroized/alloc not enabled. Fixed in https://github.com/informalsystems/tendermint-rs/pull/980/commits/701f028a84fc31b2bcc7a874a2891682c3c7fd3d. Thanks!


genotype given SNPs from amplicon data

Hi Team,
Bcbio is a great tool to manipulate the NGS data.
Our lab used multiplex PCR to enrich the target regions, and then get them sequenced on illumina platform.
Our goal is to look at the specific variants we concerned.
We applied GATK GGA mode in both UnifiedGenotyper and HaplotypeCaller. UG takes shorter time, but with large proportion providing wrong callings, while HC provides more accuate results but takes long time.
As suggested, freebayes has similar performance with HC but can save great time.
I tried "freebayes -f ref.fa -@ in.vcf.gz input.bam > out.vcf" to force genotype the given SNPs.
In this in.vcf, I gave 8 variants, therefore, I expected to obtain exact same variants in the out.vcf. However, I get all SNP that can be genotype within my amplicons.
This troubles me a lot, I am wondering any suggestions that can make me forward?
Great thanks.

Thanks for using bcbio and starting this discussion. In my experience the -@ parameter for genotyping given VCFs with FreeBayes is not reliable. We shouldn't use it in bcbio, and it sounds like you may be running it outside of bcbio -- is that correct? If so you could either use FreeBayes and then subset to only your calls of interest, or revert to using GATK HaplotypeCaller which should be much faster if only calling on specific variants. Hope this helps.


Will this library work with ESP8285?

Will this library work with ESP8285?

I think it will just work with no modifications.
I'm basing that statement on my belief that the ESP8285 and ESP8266 libraries have the same API.
But i have never tested it as i do not own any 8285 hardware.
Even if it does not work out of the box, it could almost definitely be made work; All this library needs to be provided with a pointer to the raw packet. This library could be made work with any platform that can provide a pointer to memory containing the raw incoming packet.
If you try this please let us know how you get on; It may be useful for other to know.
Thanks,
dunk.


Add constructor for Line from elements at locations

Description
Add a constructor for Line which allows passing a sequence definition as discussed below and inserts drift spaces as needed.
This PR also introduces a Marker element, which is inserted at the beginning of each sub-sequence such that elements can be placed with respect to it using from.
Requires: https://github.com/xsuite/xobjects/pull/78
Related docs update: https://github.com/xsuite/xsuite/pull/256
Checklist
Mandatory:

[x] I have added tests to cover my changes
[x] All the tests are passing, including my new ones
[x] I described my changes in this PR description

Optional:

[x] The code I wrote follows good style practices (see PEP 8 and PEP 20).
[x] I have updated the docs in relation to my changes, if applicable
[ ] I have tested also GPU contexts

Original PR description
Add a constructor for Line which allows passing elements by location and adds the required drifts so one doesn't have to do it manually
from_thin_elements_at(loc_elements, line_length, **kwargs)
    Constructs a line from thin elements at fixed locations, inserting drift spaces as needed

    Args:
        loc_elements: List of (loc, element) tuples with location and element.
        line_length: Total length of line to determine drift behind last element.
        kwargs: Arguments passed to constructor of the line

    Returns:
        An instance of Line

    Examples:
        >>> line = xt.Line.from_thin_elements_at((
                    (0.7, xt.Multipole(length=0.3, knl=[0, +0.50], ksl=[0, 0])),
                    (1.4, xt.Multipole(length=0.5, knl=[np.pi / 12], hxl=[np.pi / 12])),
                    (1.8, xt.Multipole(length=0.2, knl=[0, 0, 0.1])),
                    (2.1, xt.Multipole(length=0.3, knl=[0, -0.50], ksl=[0, 0])),
                    (2.8, xt.Multipole(length=0.5, knl=[np.pi / 12], hxl=[np.pi / 12])),
                ), line_length = 5)

Thank you very much! We were thinking of something along these lines since a while.
We would propose a moregeneral API to allow optionally specifying the name of the elements or the positions with respect to other elements. We were thinking of something along these lines:
elements={
 'q1': xt.Multipole(length=0.3, knl=[0, +0.50], ksl=[0, 0])),
 'd1': xt.Multipole(length=0.5, knl=[np.pi / 12], hxl=[np.pi / 12])))
...
}
line = xt.Line.from_sequence(elements = elements, length=5,
    locations=[
        Location(s=0.7, name='q1'),
        Location(s=0.1, name='d1',from='q1')
        ....]
)

This follows the MAD-X naming convention where a "line" contains the dift, while a "sequence" is an association name/s and the drift are autogenerated.
In the future this could work also for thick elements. For now we could check that all the provided elements are thin and raise a NotImplementedError otherwise.
What do you think?

It's actually rather simple to do, if you want, I can include it in the PR:
def from_sequence(sequence):
    element_names = []
    element_locations = []
    elements = []
    for name, args in sequence.items():
        from_ = args.get('from', args.get('from_', 0))
        if isinstance(from_, str):
            from_ = element_locations[element_names.index(from_)]
        at = args.get('at', np.max(element_locations, initial=0))

        if 'seq' in args:
            # nested sequence
            element_names.append(name)
            elements.append(xt.Drift()) # TODO: here we need a marker element, for now use a zero length drift
            element_locations.append(from_ + at)
            # add elements recursively
            nam, loc, els = from_sequence(args['seq'])
            element_names.extend([f'{name}_{n}' for n in nam])
            element_locations.extend([from_ + at + s for s in loc])
            elements.extend(els)

        elif 'el' in args:
            # element definition
            element_names.append(name)
            elements.append(args['el'].copy()) # copy to allow re-using elements
            element_locations.append(from_ + at)

    return element_names, element_locations, elements

## Example

dipol = xt.Multipole(length=0.5, knl=[np.pi / 12], hxl=[np.pi / 12])
section = {
 'q1': dict(el=xt.Multipole(length=0.3, knl=[0, +0.50]), at=0.7),
 'd1': dict(el=dipol, at=0.1, from_='q1'),
 'x1': dict(el=xt.LimitRect(), at=1.2),
}
sequence = {
 's1': dict(seq=section, at=0),
 's2': dict(seq=section, at=2, from_='s1'),
 'q1': dict(el=xt.Multipole(length=0.3, knl=[0, 0, 0.50]), at=5),
#...
}

print(f'name    s     element')
for a,b,c in zip(*from_sequence(sequence)):
    print(f'{a:6s}  {b:.2f}  {c}')

which gives:
name    s     element
s1      0.00  <xtrack.beam_elements.elements.Drift object at 0x7ff8ba34cee0>
s1_q1   0.70  <xtrack.beam_elements.elements.Multipole object at 0x7ff8ba27ba60>
s1_d1   0.80  <xtrack.beam_elements.elements.Multipole object at 0x7ff8ba27bdf0>
s1_x1   1.20  <xtrack.beam_elements.apertures.LimitRect object at 0x7ff8ba27be80>
s2      2.00  <xtrack.beam_elements.elements.Drift object at 0x7ff8ba27bd90>
s2_q1   2.70  <xtrack.beam_elements.elements.Multipole object at 0x7ff8ba27beb0>
s2_d1   2.80  <xtrack.beam_elements.elements.Multipole object at 0x7ff8ba27bdc0>
s2_x1   3.20  <xtrack.beam_elements.apertures.LimitRect object at 0x7ff8ba246dc0>
q1      5.00  <xtrack.beam_elements.elements.Multipole object at 0x7ff8ba27be20>

and then use the from_thin_elements_at method to add the drifts:
def line_from_sequence(sequence, length):
    nam, loc, els = from_sequence(sequence)
    return xt.Line.from_thin_elements_at(zip(loc, els), line_length=length, element_names=nam, **kwargs)

Support for thick elements can later be added by adjusting drifts in from_thin_elements_at and by adding support for the REFER and REFPOS keywords in from_sequence.

Indeed I like the idea of keeping the possibility of defining line with a minimal amount of typing. Nevertheless I would like to keep the possibility of using a more explicit definition.
I would propose the following API allowing 3 different degrees of verbosity:

Most explicit way as defined before (allows repetition of elements, which can be useful for very large machines and for

elements with large memory footprint):
elements={
 'q1': xt.Multipole(length=0.3, knl=[0, +0.50], ksl=[0, 0])),
 'd1': xt.Multipole(length=0.5, knl=[np.pi / 12], hxl=[np.pi / 12])))
...
}
line = xt.Line.from_sequence(elements = elements, length=5,
    locations=[
        Location(s=0.7, name='q1'),
        Location(s=0.1, name='d1',from='q1')
        ....]
)

Elements defined on the fly (no dictionary), nemes are optional

line = xt.Line.from_sequence(
    locations=[
        Location(s=0.7, element=xt.Multipole(length=0.3, knl=[0, +0.50], ksl=[0, 0])), name='q1'),
        Location(s=0.1, element=xt.Multipole(length=0.5, knl=[np.pi / 12], hxl=[np.pi / 12]))), name='d1', from='q1'),
        ....],
    length=5,

)

Compact definition through list of tuples (names are optional)

line = xt.Line.from_sequence([
        (0.7, xt.Multipole(length=0.3, knl=[0, +0.50], ksl=[0, 0]), 'q1'),
        (0.8, xt.Multipole(length=0.3, knl=[0, +0.50], ksl=[0, 0])),
        ....],
    length=5,
)

The implementation should be easy:

in case 3. one builds location objects within the method and falls back on case 2.
in case 2 one builds the dictionary and the missing name and falls back on case 1.

What do you think?

Maybe one can unify our ideas by having a common "what" parameter accepting elements and sub-sequences either by name or object; by allowing to pass the name separately (to prevent conflicts of equal names when repeating elements); by allowing for nested sub-sequence (which will be useful to build a line by means of equal sections into which additional elements are placed); and maybe by renaming Location to At to reduce typing overhead such that there is no need for a separate list-of-tuple method. Something like
class At:
    def __init__(self, s, what, *, from_=0, name=None):
        """
        Args:
            s (float): Location (in m) of what relative to from_
            what (str, BeamElement or list): Object to place here. Can be an instance of a BeamElement,
                another sequence given as list of At, or the name of a named element or sequence.
            from_ (float or str, optional): Reference location for placement, can be the s coordinate (in m)
                or the name of a previous element or sequence.
            name (str, optional): Name of the element to place here. If None, a name is chosen automatically.
        """
        self.s = s
        self.from_ = from_
        self.what = what
        self.name = name

def from_sequence(locations, length, *, elements=None, sequences=None, copy_elements=False):
    """
    Args:
        locations (list of At): Sequence definition.
        length: Total length (in m) of line. Determines drift behind last element.
        elements: Dictionary with named elements, which can be refered to by name in the sequence definion.
        sequences: Dictionary with named sub-sequences, which can be refered to by name in the sequence definion.
        copy_elements (bool): Whether to make copies of elements or not. By default, named elements are re-used
            which is memory efficient but does not allow to change parameters individually.
    """
    for at in locations:
        if isinstance(at.what, BeamElement) or at.what in elements:
            ... # place element
        else:
            sequence = sequences[at.what] if isinstance(at.what, str) else at.what
            ... # place sub-sequence by recursion

Which would allow all of the above:
from xtrack import Line, At, Multipole, ParticlesMonitor
elements = {
   'quad': Multipole(length=0.3, knl=[0, +0.50], ksl=[0, 0])),
   'bend': Multipole(length=0.5, knl=[np.pi / 12], hxl=[np.pi / 12])))
    ...
}
sequences = dict(
  section = [At(...), At(...)],
)
monitor = xt.ParticlesMonitor(...)
line = Line.from_sequence(length=5, elements = elements, sequences=sequences,
    locations=[

        # by name, allowing repetition
        At(0.7, 'quad'),
        At(0.1, 'bend', from_='quad'),
        At(1.7, 'quad', name='quad2'), # repeat element 'quad', but give it a unique name
        At(1.8, 'bend'), # the implementation can also append a suffix automatically to ensure unique names
        ....

        # on the fly, most compact
        At(0.3, Multipole(...)),  # overhead to my initial proposal is negligible
        At(0.7, Multipole(...), name='quad3'),  # but it still allows to assign names as desired
        At(0.1, Multipole(...), name='bend3', from_='quad2'),
        ....

        # nested sequence (by name or on the fly)
        At(0, "section", name="section_1"),
        At(0.1, monitor, from_="section_1"),
        At(0, [At(...), At(...)]),
    ]
)

What do you think?

I like it. Could you please update the pull request accordingly?
Only one thing: I find "At" a bit misleading, and it's not that I like "Location" much better though. Should we stick to the MAD-X naming and call it "Node"?

Okay, I will update the PR. "Node" is perfect.

@giadarol I have updated the PR and it's description. Please have a look.


'Unknown cipher' error when using adonisjs in electron

Package version
AdonisJS: 4.1
Electron: 6.0.2
adonisjs/framework: 5.0.9
node-cookie: 2.1.1
simple-encryptor: 1.4.0
Node.js and npm version
NodeJS: 12.4.0
NPM: 6.9.0
Sample Code (to reproduce the issue)
const { fork } = require('child_process');

this.serverInstance = fork('./server.js', [], {
  cwd: __dirname,
  env: {
    PORT: this.port
  },
});

Because Electron is using a custom version of node, the aes-256 encryption used in the simple-encryptor package doesn't work, now this is fixed in version 3.0.0 by using the aes-256-cbc cipher. The simple encryptor package is used in the node-cookie package that is included in adonis.
Now I thought I could just include the higher version of this package myself but the node-cookie package looks specifically in the node_modules of the package itself. So now I don't know what to do..
Thanks in advance :D

Hey @luukhoeben! 👋
We are happy to accept a PR to update simple-encryptor to the version 3.0.0.

Closing since no answer from issue reporter.

I also got this isse

same problem here. How to fix?

Hey sorry for answering so late. I see you ugraded the simple-encryptor to version 3 in the framework, but the problem still exists. Here is an image of the error:

It looks like the node-cookie package get's the simple encryptor from its own node_modules folder, so upgrading the simple encryptor package in the framework won't help.

@RomainLanz Made a PR on the node-cookie repo.

@luukhoeben any update or workaround ? how to use your fork on my project?


rbd: get lastsyncbytes and lastsycduration for volrep

This commit get more information from the description like lastsyncbytes and lastsyncduration and send them as a response of getvolumereplicationinfo request.

cc @ShyamsundarR

Have you tested this end to end in a live cluster ?
testing is stuck still de to some image issue, but it shouldn't be any issue, as this follows the same flow as lastsynctime. But Still working on to get working examples and logs.
Meanwhile, feel free to review the PR if any other changes are required.

@Rakshith-R and @nixpanic please review

@mergifyio rebase


Vocab size issue

I found that the vocab size of embedding layer is 50,004. However, the vocab size of subword (bpe) tokenizer is 50,044, which causes out-of-vocab problem.
I got the vocab size of tokenizer by using this code vocab_size = len(bart.task.source_dictionary)
I faced this problem when I use bart.encode() function.
Here is how I load the pre-trained bart model in python
bart = BARTModel.from_pretrained(model_path, checkpoint_file=model_file)
where model_path contains the pre-trained PLBART, and model_file is plbart_base.pt
I am not sure if I do something wrong here. Can anyone help me?
Thanks.

Follow this.
bart = BARTModel.from_pretrained(
        args.checkpoint_dir,
        checkpoint_file=args.checkpoint_file,
        data_name_or_path=args.data_name_or_path,
        user_dir=root_dir.joinpath('source'),
        task=args.task,
        bpe='sentencepiece',
        sentencepiece_model=spm_dir,
    )

The last 4 parameters are optional. But data_name_or_path is crucial for dictionary loading. args.data_name_or_path should contain the *.dict.txt files.


urls: wait for all expected services

wait for all expected services to be returned from the CTA query

Superseded by #488


Hedera Exception: INVALID_NODE_ACCOUNT when calling client.getAccountBalance(AccountId.fromString(newAccountId1))

about the new upgraded Testnets and support for 4 nodes in Hedera Java SDK v:0.5.0;
I have updated my Java code (class ClientExample) to read 4 nodes from my .env file
(which now contains 4 pairs of Node ID & Node Addresses);
I have also included the Java statement:
hederaClient1 = new Client(Map.of(nodeId1, nodeAddress1, nodeId2, nodeAddress2, nodeId3, nodeAddress3, nodeId4, nodeAddress4)); as specified by Greg Scullard.
The program creates a new account on Hedera Testnet, with new Account ID: 0.0.10480;
but it gets an error (INVALID_NODE_ACCOUNT) when it tries to get the Account Balance:
Java statement:
accountBalance1 = client1.getAccountBalance(AccountId.fromString(newAccountId1));
Here is the Output, displayed by my Program:
Hello Hedera Java SDK World from Kamal !
clientExample --> main entered.
Client Example --> getEnv entered.
clientExample --> hederaClient --> Operator ID 1: 0.0.1613
clientExample --> hederaClient --> Operator Key 1: 302e.....
clientExample --> hederaClient --> Node ID 1: 0.0.3
clientExample --> hederaClient --> Node Address 1: 0.testnet.hedera.com:50211
clientExample --> hederaClient --> Node ID 2: 0.0.4
clientExample --> hederaClient --> Node Address 2: 1.testnet.hedera.com:50211
clientExample --> hederaClient --> Node ID 3: 0.0.5
clientExample --> hederaClient --> Node Address 3: 2.testnet.hedera.com:50211
clientExample --> hederaClient --> Node ID 4: 0.0.6
clientExample --> hederaClient --> Node Address 4: 3.testnet.hedera.com:50211
Client --> constructor entered.
New Account ID: 0.0.10480
Main Program --> Client Example --> Main error --> Hedera Exception.
com.hedera.hashgraph.sdk.HederaException: INVALID_NODE_ACCOUNT
at com.hedera.hashgraph.sdk.HederaException.throwIfExceptional(HederaException.java:38)
at com.hedera.hashgraph.sdk.QueryBuilder.mapResponse(QueryBuilder.java:168)
at com.hedera.hashgraph.sdk.QueryBuilder.mapResponse(QueryBuilder.java:10)
at com.hedera.hashgraph.sdk.HederaCall.execute(HederaCall.java:31)
at com.hedera.hashgraph.sdk.Client.getAccountBalance(Client.java:133)
at com.hedera.hashgraph.sdk.examples.simple.ClientExample.main(ClientExample.java:107)
at pkg.Program.main(Program.java:38)
Please could you solve the problem ?

There appears to be an issue with the SDK, it's picking a random node too many times. Within a single BalanceQuery, the client.pickNode method is called three times with different results
Picking node
Picked 3
Picking node
Picked 2
Picking node
Picked 0

Hi Greg, thank you for that; OK so how do we proceed ?
what should we do so that when the Java code does a Balance Query, it obtains an accurate Account balance on the first attempt every attempt ?
thank you, regards, Kamal.

I'll take a look at trying to fix this today.

Hi to Ryan, does that mean the problem has been resolved ?


Release dlp 0.7.0

This pull request was generated using releasetool.

Release is at https://github.com/GoogleCloudPlatform/google-cloud-python/releases/tag/dlp-0.7.0


Compatibility with CKAN 2.10

Description
Doesn't work with ckan 2.10
First steps
Modify Plugin.py to adapt the new function's names of IPackageController
Is there other thing to do ?
Your Setup

CKAN version: 2.10.1
Commit/version of this repo: 3.1.8

Hi @PitPitoune, thanks for reporting this. We have scheduled this and will resolve the problem in the next couple of days, hopefully it won't be require a significant change as you suggest!

The changes have been made and merged, will do a release asap.

Thanks a lot !

Released: https://github.com/NaturalHistoryMuseum/ckanext-doi/releases/tag/v3.1.9 / https://pypi.org/project/ckanext-doi/


Not waiting for the clean task to finish

del 1.2.1 doesn't return a promise and the build doesn't wait fro the clean task to finish. Bumping del's version to 2.0.2 fixes things.
Thanks for the awesome generator!

Tanks a lot for reporting!
This issue has been solved in v2.0.0.


Connection Configuration Window Lost Focus

Description
Similar to issue #13691, the focus is sometime lost on the connection configuration after change window with Alt+TAB. The window cannot be closed, the entire app hang. Have to manually end Java process.

DBeaver Version
Community Edition 23.2.0
Operating System
ubuntu 22.04
Database and driver
No response
Steps to reproduce
This happened to me multiple times

Open DBeaver.
Edit a connection.
At SSH tab, test tunnel configuration.
Change windows.
Come back to DBeaver.

Additional context
No response

Thank you for your report.

I confirm this bug.
I could not select a private key: clicking the folder icon freezes the app.


Add sourcing config

Codecov Report

Merging #25 into master will decrease coverage by 4.3%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #25      +/-   ##
==========================================
- Coverage   98.59%   94.29%   -4.31%
==========================================
  Files           8        9       +1
  Lines         285      298      +13
  Branches        6        6
==========================================
  Hits          281      281
- Misses          4       17      +13

Impacted Files
Coverage Δ

.../bluebrain/nexus/sourcing/akka/AkkaAggregate.scala
93.61% <ø> (ø)
:arrow_up:

...bluebrain/nexus/sourcing/akka/AggregateActor.scala
100% <ø> (ø)
:arrow_up:

...bluebrain/nexus/sourcing/akka/SourcingConfig.scala
0% <0%> (ø)

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 eac8dae...43b1b53. Read the comment docs.


./scripts/setup_aws_bosh_for_concourse tries to do ruby interpolation

#{cert_name} is not really valid bash
+ openssl req -new -nodes -newkey rsa:2048 -out 'generated-stubs/#{cert_name}.csr' -keyout ../deployments-luan/certs/concourse.key -subj /CN=

Thanks very much for this feedback @luan, we're marking this repo as being under heavy re-construction, and we'll incorporate this information in the new incarnation. (I'll copy-paste this message to the other issues you opened, for anyone in the future coming across any of them).


Integrate google::dense_hash_map

When aggregating allocations, the lookup and insert performance of a pair of hash tables is our biggest bottleneck. Some rough profiling with https://github.com/sparsehash/sparsehash-c11/blob/master/README suggests that we may be able to get as much as a 20% improvement in tracking speed by switching to google::dense_hash_map, at the cost of extra memory usage for maintaining our state.

Closing as probably we are not going to do it


Sticky side menu (resolves #235)

[x] This pull request has been linted by running npm run lint without errors
[x] This pull request has been tested by running npm run start and reviewing affected routes
[x] This pull request has been built by running npm run build without errors
[x] This isn't a duplicate of an existing pull request

Description
Added the following improvements to the side bar

Sticky (i.e. remains in the viewport as the page/post scrolls).
Includes a link to the page/post title (h1) so that users can jump back to the top when scrolled to the bottom of a page/post.
The hierarchy (h1 through h2) reflects in the type styles (changes to this are to be made in a following issue see Additional information section).
The currently active section is highlighted in the sidebar as per designs - the thicker side bar.
The aria-labelledby attribute of the sidebar points to a span which has the text "secondary navigation".

Steps to test

Go to deploy preview.
Go to About page.
Scroll up and down page observing side menu.

Expected behavior:
All five improvements should be applied to side menu.
Additional information
Waiting for Uttara/Lisa to complete redesigns for the hierarchy (h1 through h2) that reflects in the type styles for the side menu. These changes are to be addressed in a following issue.

Related issues
closes #235

When the thicker side bar (the current active state) is applied and more left padding is added, the menu text gets pushed a little to the right, which causes the last letter "s" in the text "inclusion challenges" be shifted into a new line. I wonder if you could use box-shadow or transparent border to add invisible padding so the total padding before and after applying the active state remain the same so the text won't shift.
Screenshots to show the problem:

Before applying the current state:

After:

When adjusting UIO line height, the side menu items doesn't respond to the line height change.
This is because the line-height is defined for #defaultContainer aside nav ul li element, which overrides the line height UIO applies to <body> tag. Please consider to remove the line-height defined for the list element or subsitute with padding-bottom or margin-bottom.

On safari, the vertical line at the left of the side menu is not showing up:


DownloadBundle.java:39-40: Create test for this class and...

The puzzle 120-b9c4d61d from #120 has to be resolved:
https://github.com/artipie/rpm-adapter/blob/fef8ecbb34f39a598e735b91b89096f349810e9a/src/test/java/com/artipie/rpm/files/DownloadBundle.java#L39-L40
The puzzle was created by olenagerasimova on 24-Apr-20.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@g4s8/z please, pay attention to this issue

The job #151 assigned to @g4s8/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

The puzzle 120-b9c4d61d has disappeared from the source code, that's why I closed this issue.

Job was finished in 19 hours, bonus for fast delivery is possible (see §36)


Add a CODE_OF_CONDUCT.md

This project needs a code of conduct document. Let's use/link to the Jupyter Code of Conduct.

Solved by PR #11.


HTTPClient: Add PATCH method and missing HTTP status codes

Just added a few missing values to the enums in HTTPClient. Builds and runs fine on Windows and Linux.
This adds support for the HTTP PATCH method and some missing standardized HTTP status codes.
For reference, see https://en.wikipedia.org/wiki/List_of_HTTP_status_codes and https://tools.ietf.org/html/rfc5789 (PATCH method for HTTP).

Fixed :)


update to latest hlsjs 0.8.5

Coverage increased (+0.1%) to 74.203% when pulling f76adf1a515bfeeee84595e5c84b227e927f3256 on update-hlsjs-0-8-5 into 6ca9ef1dd7da052a18c29e2063a11d8e798bc4fa on master.


Add READY=yes to SR appliance

We need to make sure that the SR server is up and running before reporting it as ready to ONEGATE.

carried out upstream https://github.com/OpenNebula/one-apps/issues/47


Export assemblies to lcabyg

Fixes this issue. Also, fixes from this PR is included as well.
Lastly, building components also maps to groups in LCAbyg again. Mapping is based on all old code found in the documentation backend. The correctness of this is unknown at the moment, but it seems better than just mapping to "andet" group.

@jaa-itminds I have no idea how to fix this and I can't run the tests locally because I can't install azure.storage package - Any idea how to resolve this?


Bug: Issue with Pane Arrangement

Description
I have encountered a problem when trying to open a second pane and arrange them horizontally. When I click on "Arrange Horizontally," the panes are arranged vertically instead, and the reverse happens when I choose to arrange them vertically.
Steps To Reproduce

Click on the "Tab actions menu" button
Click on "Arrange Horizontally"

Requirements
Correct the logic for arranging the panes to ensure that the "Arrange Horizontally" option arranges the panes horizontally and the "Arrange Vertically" option arranges them vertically.
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.3880
Log File

We have decided to use the term ‘Arrange Panes’ instead of ‘Split Panes’. Feedback and the response to polls on Discord indicate a preference for the current user experience, but this decision is not final, and we are actively monitoring feedback in this area.

Oh, I see. However, for me, it doesn't seem logical because when we see the icon showing the panes vertically, we expect the panes to be arranged vertically and not horizontally.

Oh, I see. However, for me, it doesn't seem logical because when we see the icon showing the panes vertically, we expect the panes to be arranged vertically and not horizontally.

I must have reversed the icons in #15843, I'll fix this shortly.


Best way to enabled/disable offside based on media query.

Hello:
The web site I am designing (a tracker implementation for the roundup issue tracker) has a nav element on the left in desktop mode. When the screen size drops below a certain amount, the nav element is hidden. Then it is displayed when the user hits the menu link. If the user resizes the screen to a larger size, the nav column is once again displayed automatically.
I have implemented display of the nav column using offside with a menu button. It works fine pushing the main section to the right and allowing the nav column to move in from the left. I can scroll the left nav column independently from the right column.
However the nav element is hidden in all cases including desktop. However in desktop mode, the menu button is hidden making the nav element inaccessible.
What is the preferred way to allow an offside menu only at certain breakpoints/screen sizes?
I can think of two ways:

add media queries that wrap all of offside.css so that the styles only trigger with a
media query spec that also displays/enables the menu button.
Add javascript matchmedia calls (along with listeners for changes) that will call
offside() or offside.destroy() as the page size changes.

Is there another method that I missed? What is the recommended way of handling this?
Thanks.
-- rouilj

Hi @rouilj,
the way default style is extended is up to users. There are no preferred/suggested solution.
Best


gettin an error

getting this error its prolly because i have the wrong api key? and hopefull its gonna resolve the buffering thing in its front end ty for helping me in advance

I'm getting the same error, here: https://github.com/chrisleekr/binance-trading-bot/issues/188
In there I've made some steps to fix it, temporarily, but fix it. Check it out.

Since may people asking about this error message, I added to Wiki - https://github.com/chrisleekr/binance-trading-bot/wiki/Troubleshooting#binance-api-error---eai_again
Please close the issue if your question is answered.


Response protocol is not returning HTTP 2 for http 2 endpoint

public class Http2Test {

    @Test
    public void isHttp2Transport() throws IOException {
        HttpLoggingInterceptor loggingInterceptor = new HttpLoggingInterceptor();
        loggingInterceptor.setLevel(HttpLoggingInterceptor.Level.BODY);

        OkHttpClient okHttpClient = new OkHttpClient.Builder()
                .addInterceptor(loggingInterceptor)
                .build();

        Request request = new Request.Builder()
                .url("http://twitter.com")
                .build();

        Response response = okHttpClient.newCall(request).execute();

        Protocol protocol = response.protocol();

        assertEquals(Protocol.HTTP_2, protocol);
    }
}

junit.framework.AssertionFailedError:
Expected :h2
Actual   :http/1.1

Do you have the ALPN jars added to your classpath?
On Wed, Jan 13, 2016, 4:15 PM Alexey Zakharov<EMAIL_ADDRESS>wrote:

public class Http2Test {
@Test
public void isHttp2Transport() throws IOException {
    HttpLoggingInterceptor loggingInterceptor = new HttpLoggingInterceptor();
    loggingInterceptor.setLevel(HttpLoggingInterceptor.Level.BODY);

    OkHttpClient okHttpClient = new OkHttpClient.Builder()
            .addInterceptor(loggingInterceptor)
            .build();

    Request request = new Request.Builder()
            .url("http://twitter.com")
            .build();

    Response response = okHttpClient.newCall(request).execute();

    Protocol protocol = response.protocol();

    assertEquals(Protocol.HTTP_2, protocol);
}

}

—
Reply to this email directly or view it on GitHub
<https://github.com/square/okhttp/issues/2236>.

@JakeWharton Im running this as pure junit test. But I run into the issue when I was debugging Android requests on Android+ device.

@JakeWharton ok this pure unit test is not correct I guess. I will try to rewrite it Android test and rerun

@JakeWharton I was probably wrong. When I run my test as Android Instrumentation it succeeded. Will do more research.

Cool I think HTTP 2 protocol is set correctly for response.
I think something is probably wrong with how HttpLoggingInterceptor log things.
In all my tests: https://github.com/square/okhttp/blob/master/okhttp-logging-interceptor/src/main/java/okhttp3/logging/HttpLoggingInterceptor.java#L151
connection is always null.
01-14 17:16:06.100 18812-18838/com.github.lexer.http2test D/OkHttp: --> GET https://twitter.com/ HTTP/1.1
01-14 17:16:06.721 18812-18838/com.github.lexer.http2test D/OkHttp: <-- 200  https://twitter.com/ (620ms, -1-byte body)
01-14 17:16:06.722 18812-18838/com.github.lexer.http2test D/OkHttp: --> GET https://twitter.com/download HTTP/1.1
01-14 17:16:06.768 18812-18838/com.github.lexer.http2test D/OkHttp: <-- 200  https://twitter.com/download (45ms, -1-byte body)

@JakeWharton I will close this story and will open separate ticket for http logger.

#2247 Response protocol is correct. This issue is invalid. Protocol is not correctly logged by HttpLoggingInterceptor.