Yellow Warnings About JobDriver

Tested on game version 1.1.2598, with the Royalty DLC.
Tested on your Initial Rebirth: V2.3.0 (Update: 12 Apr).
Same errors happen even on the previous Jellypowered's version (Update: 6 Apr).
I'm getting these three yellow warnings:
(Moving the mod around the load order doesn't fix it.)
[Lecris.survivaltoolsreborn][warn] Couldn't backtrack jobDriver patch: CompInstalledPart.JobDriver_InstallPart => Verse.AI.JobDriver
(Filename: C:\buildslave\unity\build\Runtime/Export/Debug/Debug.bindings.h Line: 35)
[Lecris.survivaltoolsreborn][warn] Couldn't backtrack jobDriver patch: CompInstalledPart.JobDriver_UninstallPart => Verse.AI.JobDriver
(Filename: C:\buildslave\unity\build\Runtime/Export/Debug/Debug.bindings.h Line: 35)
[Lecris.survivaltoolsreborn][warn] Couldn't backtrack jobDriver patch: CompVehicle.JobDriver_AssembleVehicle => Verse.AI.JobDriver
(Filename: C:\buildslave\unity\build\Runtime/Export/Debug/Debug.bindings.h Line: 35)

Thank you, i remember those warnings, forgot to look intoit. Can you upload hugslib log or the section where it says [Lecris.SurvivalToolsreborn] autopatch or stats patched? (forgot which). Should be near the bottom.

Sure, here's the section. And not sure if it matters, but here are all the mods I'm currently running that your mod has patches for: Alpha Animals, Mending (Mend And Recycle), Vegetable Garden and Vanilla Weapons Expanded.
[Lecris.survivaltoolsreborn] Stats auto patched:
Patch : MiningSpeed => MiningSpeed_Tool
JobDrivers:
[ True | True ] RimWorld.JobDriver_Mine :: ! Void ResetTicksToPickHit() ! : Void b__0()
JobDefs:
Mine : Verse.JobDef Mine
WorkGivers:
RimWorld.WorkGiver_Miner :: Methods :: Verse.AI.Job JobOnThing(Verse.Pawn, Verse.Thing, Boolean) :: Properties ::: Fields :
Patch : MiningYield => MiningYield_Tool
JobDrivers:
[ True | False ] RimWorld.Mineable :: Void Notify_TookMiningDamage(Int32, Verse.Pawn)
JobDefs:
WorkGivers:
Patch : ConstructionSpeed => ConstructionSpeed_Tool
JobDrivers:
[ True | True ] RimWorld.JobDriver_RemoveFloor :: ! RimWorld.StatDef get_SpeedStat() ! : Void b__2()
[ False | True ] RimWorld.JobDriver_AffectRoof :: Void b__1()
[ False | True ] RimWorld.JobDriver_ConstructFinishFrame :: Void b__1()
[ False | True ] RimWorld.JobDriver_RemoveBuilding :: Void b__1()
[ False | True ] RimWorld.JobDriver_Repair :: Void b__1()
[ True | False ] CompInstalledPart.JobDriver_InstallPart :: ! Void b__15_2() !
[ True | False ] CompInstalledPart.JobDriver_UninstallPart :: ! Void b__15_2() !
[ True | False ] CompVehicle.JobDriver_AssembleVehicle :: ! Void b__12_2() !

Funny observation, pawns are using hammers to build sandcastles :)). I'll fix that.
Interesting, somehow your mod list has broken the main purpose of this autopatch: to not require hammers to deliver stuff. Please send the list of mods as well.
Note to self: find which mod uses namespace: CompVehicle, etc.

https://gist.github.com/4ac2967c63f612a0127bd433745102fb
Here's the link to the full log with all of my mods loaded (215 in total). And a fair warning, I added my own custom tags and changed the names for most of the mods for my own convenience. I did pay attention to patches and made sure those were changed to match the new names as well.
For the sandcastle mod, it's this one:
https://steamcommunity.com/sharedfiles/filedetails/?id=1980394843
As for mods that specifically touch hauling, I'm using these two:
https://steamcommunity.com/sharedfiles/filedetails/?id=2034960453
https://steamcommunity.com/sharedfiles/filedetails/?id=1279012058
There's also ED Enhanced Options and Mehnis Modifications that offer some pawn fixes and quality of life improvements which might mess with hauling, but I'm not sure. There's also Awesome Inventory, an alternative of sorts to RPG Style Inventory, so I believe some patching will be required there.
As far as I remember, I didn't get any additional errors from your mod during actual gameplay.
And here's a few ideas that might give you some inspiration later down the line, while I'm at it:

A crowbar, perhaps for faster deconstruction / construction of objects and structures exclusively made out of wood. Or like a combat tool that does extra damage to wooden material. Also so I can have a proper Mr Freeman running around in my game.
A craftable toolbox of some sorts, that combines all of the tools together into one item and serves as a precursor / lower tech version of the Glitterworld Multitool. As colonists / pawns come and go, it gets somewhat tedious having to manage all of the tools in the early game.
A tool /compatibility for the mod RimRoads, that makes the construction of roads faster. Or to make it so that some of the existing tools apply a faster construction bonus to roads.

Just some suggestions, your take on the Survival Tools is already awesome and greatly appreciated.  In any case, keep up the good work.

Thanks, I've almost finished patching up jectool. It proved my expectations wrong, so I had to rewrite my autopatch implementation.
Thank you for pointing out those mods, it helps a lot :-). Awesome inventory is already in the works, it will be hard that one, but I got 25% done.
I really like your ideas. Better roads should be an easy fix. The toolbox, how about a flat 100% workspeed? (I'm bad at balancing :-D). Crowbar on the other hand would be a bit difficult. I will think about it. Right now only crowbar, axe and sickle are specific to a subjob, if there will be more, I will implement a subjob specific.

@IssueBro Sorry for the long delay, but I'm back to modding. I've fixed the original yellow warning, and added a patch for Sand Castles. I'm looking now at the hauling problem, and it doesn't seem to affect the need for tools, but I'll double check just in case.


added a echo tcp server example #13/

i have problem with using remoter. simply i didnt get it idk why

and also 35. line is broken there is no actor.New() but i need to create a remoter there. i am totaly lost 👯♂

@linaund1000 Taking a look today my man! Thanks upfront

@linaund1000  I was more thinking about the example I added. The TCP server.

sorry than. now i understand better what you mean. i will be working on other examples!

sorry than. now i understand better what you mean. i will be working on other examples!


how to implement castle dynamic proxy extension registration?

how to implement castle dynamic proxy extension registration?

Hey,
Here is some info in the docs about using a castle dynamic proxy as a decorator for your service registration. If you have more questions please let me know!

Hey,
If I ignored why the subject of intercetion may be under the decorator title :)
Thanks for the quick response


How to get sessionkey from tdlib?

Hello, thanks for sharing. It's very helpful to me. I'm using tdlib to build my own telegraph client, but I'm having trouble logging in with multiple accounts. I can't get it to save the session key. Can you tell me how you got it from tdlib?

WTelegramClient is not based on tdlib. We don't support importing/exporting sessions from other libraries.
If you own the mobile phone number, you can create new session with WTelegramClient by answering the verification code.


release: 1.0.1

:robot: I have created a release beep boop
1.0.1 (2023-02-02)
Bug Fixes

add arguments validate (5d75f95)

Performance Improvements

add filename in log (b832076)
perfect route (b0465a5)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/starudream/github-asset-mirror/releases/tag/v1.0.1 :sunflower:


⚠ Samsung has degraded performance

In c79cefc, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2457 ms

Resolved: Samsung performance has improved in 3ae56c4.


Improve the Responsiveness of the webpage

Describe the Bug 🐛
The page is not viewable in small screens
Expected behavior
Make the webpage viewable in small screens

I want to work on this under JWOC

I want to work on this under JWOC

Assigned


Update 'JavaScript coding style' page

Updates the 'JavaScript coding style' page upper sections based on feedback from Frontend Community (either live or through this document (internal link)). Each commit updates a separate section for easier review.
The biggest update is the 'Linting' section, which now tries to paint a clear picture of the available options between 'standard', 'ESLint' and 'Prettier', as well as where to run the linters.
The sections about 'Modules' and 'Strict mode' could use a big overhaul as well, which I'll do in a separate PR to focus the discussion:

as we're moving to using ES Modules, 'strict mode' and isolation from other modules is provided by default rather than needing manual addition and IIFEs
we probably need to distinguish modules (as in a file that contains code, exporting features and importing other modules) from the classes that provide functionalities and are called modules as well so far
with tools like Babel providing transpilation and polyfilling, there's a chance we need to clarify whether we talk to the modules being sent to the browser or the modules being authored by the teams.

@DavidBiddle @andysellick I think I've addressed all your feedback, let me know if there's still something off 😊


Output error logs of mackerel-agent as warning log of windows event log

Since eventlog is a system-wide log, error logs of mackerel-agent are handled as warning level in the eventlog

I agree with this change.


Use nasm-rs for nasm version check

nasm-rs already has a version check.

Unfortunately, this did away with the less confusing error message in the case of missing nasm. We will follow up on that.

Fixed error message: https://github.com/medek/nasm-rs/pull/36


Openapi needs a "/" character to be reached

If I try to open the openapi resource in a browser I need to append a "/" character to the URL:
"http://<IP_ADDRESS>:8080/rest/metodoserver/openapi" doesn't work (I see only the powered by WiRL logo)
"http://<IP_ADDRESS>:8080/rest/metodoserver/openapi/" works fines

Ok thank you for the explanation!


If you really don't let it go, don't let it go, is it fun to fool?

https://docs.microsoft.com/en-us/azure/virtual-desktop/language-packs
Are you the cheating department?
I waited for two months, and finally got it, and it turned out to be all wrong when I opened it.

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

ID: a8d3194d-974c-f014-cd0a-6ec611952281
Version Independent ID: 52951f02-174b-a688-bca0-4f93cc6f7e8b
Content: Install language packs on Windows 10 VMs in Azure Virtual Desktop - Azure
Content Source: articles/virtual-desktop/language-packs.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

Hello @ilikeyi,,
Thank you for your feedback. I am currently reviewing the content on this document. I will investigate the issue and work with the content author to update the document with correct details. Please standby while I get back to you.

Hello @ilikeyi,
I have reported this to content author. Awaiting for the update here. I will keep you posted.

Hello @ilikeyi,
I have feedback this to content author. The broken links have been removed from the public document. The links for 02C_2022_LXP_ISO and 03C_2022_LXP_ISO will be added back as/when they are available. Please standby until further notice.


automatic structure detection

average the data vertically to yield a 1D array
consider a 1D Gaussian smoothing
consider baseline to be the 20 percentile value
find the column with maximum brightness
walk on each side until half the distance to the baseline is reached

8ef97837118586c77d4feaa1730e04f3506eef94


BaseActiveRecord could do with it's own setAttributes()

I came across this when needed to implement my own populateRecord due to finding that console edited documents in my MongoDB totally mess up PHP data types.
Essentially the problem is in performance. You use the Model function which looks like:
    public function setAttributes($values, $safeOnly = true)
    {
        if (is_array($values)) {
            $attributes = array_flip($safeOnly ? $this->safeAttributes() : $this->attributes());
            foreach ($values as $name => $value) {
                if (isset($attributes[$name])) {
                    $this->$name = $value;
                } elseif ($safeOnly) {
                    $this->onUnsafeAttribute($name, $value);
                }
            }
        }
    }

Which looks fine until you realise what __set does in BaseActiveRecord when calling hasAttribute:
    public function hasAttribute($name)
    {
        return isset($this->_attributes[$name]) || in_array($name, $this->attributes());
    }

As you can see it is doubling (at least) the work of the function by calling $this->$name. Instead BaseActiveRecord should have it's own setAttributes which looks like:
    public function setAttributes($values, $safeOnly = true)
    {
        if (is_array($values)) {
            $attributes = array_flip($safeOnly ? $this->safeAttributes() : $this->attributes());
            foreach ($values as $name => $value) {
                if (isset($attributes[$name])) {
                    $this->_attributes[$name] = $value;
                } elseif ($safeOnly) {
                    $this->onUnsafeAttribute($name, $value);
                }
            }
        }
    }

That would make that function a lot more performant by the looks of it.

:+1:

its good to see you are trying to find optimizations for the setter. The problem with your example is that you are ignoring the current __set, which make you ignore potential behaviors and other stuff defined in Component:
    public function __set($name, $value)
    {
        if ($this->hasAttribute($name)) {
            $this->_attributes[$name] = $value;
        } else {
            parent::__set($name, $value);
        }
    }

I tried your solution though and almost of all my tests broke. I guess you are are making less use of behaviors to set attributes.

True I didn't think of the parent __set, hmm

Thinking about this more I don't think this is solvable, I understand why it was never change. Tbh, what I am really looking for in my scenario is:
public function setAttributeDirect($name, $value)
{
    $this->_attributes[$name] = $value;
}

So I can modify the populateRecord function without having to copy near the entire baseActiveRecord

This is implemented by setAttribute already:
https://github.com/yiisoft/yii2/blob/master/framework/db/BaseActiveRecord.php#L445
    /**
     * Sets the named attribute value.
     * @param string $name the attribute name
     * @param mixed $value the attribute value.
     * @throws InvalidParamException if the named attribute does not exist.
     * @see hasAttribute()
     */
    public function setAttribute($name, $value)
    {
        if ($this->hasAttribute($name)) {
            $this->_attributes[$name] = $value;
        } else {
            throw new InvalidParamException(get_class($this) . ' has no attribute named "' . $name . '".');
        }
    }

setAttribute would not work in a good way here. Imagine this function:
    public static function populateRecord($record, $row)
    {
        $columns = array_flip($record->attributes());
        foreach ($row as $name => $value) {
            if (isset($columns[$name])) {
                $record->_attributes[$name] = $value;
            } elseif ($record->canSetProperty($name)) {
                $record->$name = $value;
            }
        }
        $record->_oldAttributes = $record->_attributes;
    }

but not i the same class as the private _attributes array.
I mean it will work in a technical sense but not in a performance sense.

Atm I have left my code like this:
    public static function populateRecord($record, $row)
    {
        foreach ($row as $name => $value) {
        	try{
        		$record->setAttribute($name, $record->resolveRecordValue($value));
        	}catch(InvalidParamException $e){
        		if ($record->canSetProperty($name)) {
        			$record->$name = $record->resolveRecordValue($value);
        		}
        	}
        }
        $record->setOldAttributes($record->getAttributes());
    }

which is just as bad since I am spinning up that exception everytime. In fact I might benchmark the two current variations of using this outside of BaseActiveRecord

Im getting confused.
In your original post, you were talking about a performance problem with the double check 'isset($this->_attributes[$name])', which can be bypassed using setAttribute()
Now you are talking about skipping hasAttribute() which basically means you can set anything in _attributes you want, skipping the allowed properties for a model.
If that is something you are after, than this is al whole different story. Why dont you implement it in you'r own function as in: _attributes_dynamic

Well my sight of the performance problem is in relation to alsp not considering the parent __set. You see I was seeing from the populateRecord function that I was trying to manipulate in BaseActiveRecord.
As I tried to fix the performance in this function, in my own BaseActiveRecord child, I realised that setAttributes would be great to use if it had that fix done. Of course, I did not foresee that the parent::__set would not be called for other scenarios.
So that gives you direction about my perspective. That is why I gave off a frank reply that really a way to override the hasAttribute method and get around it, like Yii2 does in populateRecord, without being in the same class would be the most ideal solution for me.

Why dont you implement it in you'r own function as in: _attributes_dynamic

You mean as a function in my own child BaseActiveRecord? Because 80% of the BaseActiveRecord class uses _attributes in some way. PHP's private scoping would require me to copy the entire class nearly to do it. This means bad code reuse, especially for potential Yii2 updates.

What is your biggest concern / ideal situation?

Performance and code copying.
So the first option is to use the functions that currently existin BaseActiveRecord. These functions are less than performant. In fact they are down right bad for this scenario, hence why the current code in populateRecord does not use these functions (as was said to me by Yii2 team).
The second problem is code copying. If I touch _attributes in any way in my child class I would have to copy (due to PHP private scoping properties) about 80% of the BaseActiveRecord Yii2 has in place by default. This would make for massive ode copying as well as bad reuse and for problematic updates should any part of that code experience a bug connected or in part by another part related to the baseActiveRecord and it needs updating (my own personal code will overwrite any updates).
To be frank, the ideal solution to this is to be able to skip Yii2's validation functions etc for _attributes and be able to just set it directly in a function.

why dont you override the public method hasAttribute and let it return true for your scenario. It would basically gives you the solution you are looking for

Tbh, what I am really looking for in my scenario is:
public function setAttributeDirect($name, $value)
{
$this->_attributes[$name] = $value;
}

Because of PHP scoping. It requires me to copy every piece of code that also uses _attributes in BaseActiveRecord. I cannot just use parent private variable in a child class like that. They appear as two separate variables in inheritance (I know, weird).

No I mean:

override hasAttribute() and return i.e. true
use setAttribute from BaseActiveRecord

in this way, you are still within BaseActiveRecord private scope

That would break the hasAttribute function for other scenarios wouldn't it? I only want to break it for running populateRecord function in the model

I mean hasAttribute is a very useful function and helps massively in other functions in the active record.

you can customize this hasAttribute to whatever you want. I proposed returning true, but you can put a whole piece of software behind it.
Ie.:
 public $skipHasAttribute=false;
 public function hasAttribute($name)
    {
        if($this->skipHasAttribute===true){
              $this->skipHasAttribute = false;
             return true;
        }
        return isset($this->_attributes[$name]) || in_array($name, $this->attributes());
    }

And then its up to you when & where you want skipping.

Yeah that could work actually, didn't think of that, lemme try it thanks

Oh wait yes, seen a way around that, nvm I will try it

Ha, I have just remembered my db contains data not meant for my PHP app. This would actually break that functionality and cause massive more recursion in populateRecord than is needed. I wanna know if an attribute is valid but I only wanna evaluate attributes()once for the entire function, not every time I go round.

attributes() is public and customizable

@Sammaye, can this one be closed?

expired


Additional rendering utils

This seems a good extension of the current render_type
Are there any other base types we might have missed? The current string treatment allows specifying "none" as well, but stringified nones (except when rendering) are a pain to work with in code that is generating documents.

I use the datetime in Typst to date my documents (mostly on the coverpage or in the header in the theme I'm currently working on). The benefit of passing it as a Typst datetime is that that can be displayed in a variety of formats depending on the location in the document.
While looking to cross that bridge I saw that Typst had a duration type as well. As such I thought it best to include that in the same PR, too. Got a bit carried away with quickly implementing that.
Will try to resolve that and integration tomorrow.
Most of these additions arise from my current project where I'm wrapping a script around that theme such that I can generate reports from Python with a configurable theme.

So, since I wanted to cut down on boilerplate, I also included two dataclasses to derive from for Dictionary dataclasses and another one for any arbitrary Function dataclass with arguments from its properties.
That last one could replace some boilerplate in the other classes, apart from the one thing that I started this branch with: the handling of None. In current classes such as Heading, a value of None is treated as "undefined" and results in it not being included in the rendered output. To actually set it to Typst's none, you would have to specify that string or use the new Plain.
In the end I settled on:

Stick to rendering Python None as Typst none when directly encountered in the regular render_xxx functions.
Utilized dataclass field metadata to control the rendering of positional argument and inclusion of None in the output.

I think that leaves the control of automagical behavior in dataclass-land and keeps the render_xxx functions as simple 1:1 representations of their input.

Would you mind taking a look at this?


http://pennydreadfulmagic.com/cards/ does not go where http://pennydreadfulmagic.com/cards goes!

Just add a / to the end of the route decoration
On Fri., 4 Nov. 2016, 06:14 Thomas David Baker<EMAIL_ADDRESS>wrote:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PennyDreadfulMTG/Penny-Dreadful-Tools/issues/486, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAL2zo6AoxdEYiXz2oPCKcquBp0Y0iNPks5q6jKkgaJpZM4KozUU
.

28d44d5.


Update sbt-scalafix to 0.9.29

Updates ch.epfl.scala:sbt-scalafix from 0.9.1 to 0.9.29.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.epfl.scala", artifactId = "sbt-scalafix" } ]

labels: sbt-plugin-update, semver-patch

Superseded by #148.


What injector do I use in EZFN?

in EZFN with the injector that I have it won't let me inject it and I want to know which one you use

You can just find any Kernel-Mode injector from GitHub or Unkowncheats and compile with the settings required in Globals.h and it will work.

You can just find any Kernel-Mode injector from GitHub or Unkowncheats and compile with the settings required in Globals.h and it will work.

Can you send the injector


All of Analysis seems to be....gone?

Describe the bug
Whenever loading an imported file with CodeBrowser, the "Analysis" menu option is not present. I am not sure at all what causes this. I was able to get Ghidra to work properly when I first ran it, but I have not been able to get it to work properly since, even when using the same exact file and project that worked previously. Ghidra was not updated or changed between the sessions.
To Reproduce
(Note: I am almost positive following these directions worked at first, but they have not worked since my first attempt)

Create a new project (non-shared)
Import a file
Set whatever options you want here, it weirdly won't change later behavior at all, whether it is the default or something completely arbitrary and nonsensical
Open the imported file by double clicking
No Analysis option in menu
Expected behavior
The analysis menu bar function, as well as other items, would be present

Screenshots

Above is a screenshot of what the layout typically looks like.
Attachments
crackme0x00.exe.zip
This is the file loaded in the screenshot
misaka.zip
This is the file that I was able to successfully load at first, but have been unable to properly load since.
Environment (please complete the following information):

OS: MacOS 13.1
Java Version: 19.0.1
Ghidra Version: 10.2.3
Ghidra Origin: It was initially sourced directly from GitHub, until the issue began occurring. I then switched over to the homebrew distro, which has the same exact issue as the GitHub distro
Additional context

I am just starting out with using Ghidra, so I will admit that there is a fair chance that I am simply missing some very important information. I will say that I did my best to look elsewhere for solutions, but I couldn't find anything about this issue. Any help or advice is welcome.

close the code browser window.
In the front-end window:
Tools -> Delete Tool -> CodeBrowser
Tools -> Import Default Tools -> check CodeBrowser.tool -> press OK
Open code browser window, see if that fixed it or not

This worked for me, thanks!

If that worked, please close this issue.

👍

👍


Change height of top panel

Love this theme and am very new to Awesome.  How do I change the height of the top panel?

You need to change the height property in the rc.lua file on line 470. Here
But the top panel is based on images that used as backgrounds for widgets, so you'll find some visual issues probably.

Thanks for the reply.  Yeah one of the downsides to a hidpi screen unfortunately.


[NFC] Upgrade EnumElementDecl to a DeclContext

Pure plumbing for the sake of default arguments.

@swift-ci please smoke test

@DougGregor @jckarter Can you take a quick look too? It's easy to miss a spot when adding a new kind of thing like this.

Oh, good, this'll make it easier for @griotspeak to add GADTs to Swift 15.

Like Jordan said, it seems like a good idea to set up xrefs for these now so you won't run into problems with them missing in the future.

@swift-ci please smoke test

@swift-ci please smoke test Linux platform


Context switch time order of magnitude higher than with CSPPC

Looking at the output of WarpRace regarding context switches on the 604e of the Cyberstorm and the Sonnet there is an order of a magnitude of difference (1.500ms versus 15.000ms). This is probably linked to WarpRace executing 68k code within sonnet memory and/or reading data with the 68k from sonnet memory but I am not sure. Will have to look into it.

Of course this is worth investigating, but let's keep in mind 68k and PPC have separate memories. And in our case access from 68k CPU to PPC memory is terribly slow compared to situation where both processor have equally fast access to the memory (like in CSPPC). So it might be purely a hardware design problem...

Now the Sonnet is also running with a Radeon 92xx card (and benefiting from the extra gfx speed and extra memory) it is weird to find that the context-switch has gone up to a staggering 90ms. A game like Quake (which apparently uses not much context-switches) is now even playable on 800x600x8 (15fps). However simple console programs now take forever when they do a lot of switching.
Why changing a PCI cards affects the context-switch time (PCI-latency) so much has to be figured out.

Extra time during the context switch originates from the PCI Interrupt Server from the pci.library. Looking into it. Turning it off results in a normal context switch time (15ms) and a Quake timedemo at 320x240x8 at 37 FPS. Turning it off results also in some PCI cards not working anymore (network card in my case).

Some testing resulted in the observation that at least the communication from ppc to 68k is not working. The noticed communication was actually a by-effect of SCSI interrupts on INT2 where also the sonnet card is listening to. I'm guessing that the PCI interrupt server from pc I.library should be used for correct interrupting of the 68k by the ppc. Installing a timed interrupt on a CIA B timer actually brings the context switch time within CSPPC levels (1ms) resulting in like 60fps for Quake without noticing speed decrease of normal apps due to the frequent CIA interrupts (using INT6). Rather have actual interrupts going from the ppc to 68k then have a timed interrupt, though.

Fixed it at least for the PPC -> 68K communication (using an interrupt through the pci.library). Context switch now 0.4ms so 30-40 times faster than before. Still looking at the 13ms context switch from 68K->PPC but that one is less crucial for WarpOS programs.

Seeing that the context-switch time from 68K to PPC is not really critical, i'm calling this issue closed

Addendum:
68K to PPC was fixed by timing the function RunPPC and finding out that allocating a large chunk of memory with MEMF_CLEAR was not the smart way to do that. It is now also 400 microseconds.

I know this one is closed, but after a few upgrades to the library latest tests show:
RunPPC: 280 microseconds
Run68K: 310 microseconds
Which is quite faster than the CSPPC at the moment (400 microseconds).


Responsive Design for Mobile Platforms

Kaiwa uses a lot of static pixel based scaling in CSS which makes it very difficult to use on a mobile device since the buddy list ends up taking over almost the entire screen. Ideally if the screen gets small enough, the buddy list should disappear from view, only accessible by pressing on a menu button. This would be the standard in responsive design.
I started working on my own solution since Kaiwa development appears to be stalled. I am not very skilled with CSS, so it isn't very sophisticated. Basically, it just scales the buddy list narrower if the screen is small. If I find the time to work out all the kinks, I will submit a pull request.

Here's what I have so far which just scales the right side menu according to the size of the window. It is far from perfect and should probably just be scrapped in favor of hiding the menu at certain screen widths. Since there are great mobile apps for XMPP, I'm not sure if I will go any further with it.
196a197,202
> @media all and (max-width: 1148px) {
>     #pages{
>         margin-left: 0px;
>         left: 20%;
>     }
> }
520c526,527
<   width: 230px;
---
>   width: 20%;
>   max-width: 230px;
533c540,542
<   width: 230px;
---
>   width: 100%;
>   min-width: 130px;
>   max-width: 230px;
992a1002,1007
> @media all and (max-width: 1148px) {
>     .conversation header {
>         margin-left: 0px;
>         left: 20%;
>     }
> }
1179a1195,1200
> }
> @media all and (max-width: 1148px) {
>     .conversation .chatBox {
>         margin-left: 0px;
>         left: 20%;
>     }


Update seed phrase copy and add confirmation step

User creates a wallet and wants to manually recover the seed phrase. After going to "I choose to backup my account manually", the user is led to the seed phrase section and asked to take note of their seed phrase.
The Problem
The Button in this section states "Setup seed phrase", which implies a form of the setup process. Instead, the user will be led to the wallet home dashboard.
Expected Behaviour
If there is no additional setup process, calling the button "Finished" might be more appropriate. Alternatively, retyping the seed phrase in the next section to ensure that the user took note of the seed phrase might be another option.

"Setup seed phrase" has been confusing to users, will definitely update this copy.
We're considering a lightweight "Confirm" step, possibly with just a one word confirm ("Please enter the third word of the seed phrase to continue"). @jakestutzman

Designs
Acceptance Criteria
Setup Seed Phrase

[ ] User can view their seed phrase
[ ] Selecting "Copy Phrase" copies seed phrase to the clipboard
[ ] Selecting "Continue" redirects user to "Verify Seed Phrase" page

Verify

[ ] User is directed to enter randomized word (# 1-12)

[ ] User can select "Verify"

[ ] If word matches, user is redirect to success page
[ ] If word doesn't match, user receives error message "You entered the incorrect word"

[ ] Tested

[ ] Mobile ok

The Button in this section states "Setup seed phrase", which implies a form of the setup process. Instead, the user will be led to the wallet home dashboard.

just to be clear, it's called like that because seed phrase isn't added to account before you press the button
I like the idea of lightweight verification process, full verification is too annoying.

Selecting "Copy Phrase" copies seed phrase to the clipboard

Mixed feeling about this. I wouldn't encourage copying it to clipboard. It's way too easy for spyware to steal stuff from clipboard (no access privileges needed at all usually).

Reasonable point re: copying phrase to clipboard. It's a standard practice to have it (metamask, coinbase wallet, etc), but given it's an opt in for more security-focus in our case, maybe we can exclude it.

@kcole16 if we don't mess with web layout it should be copy-able by default (just select text and copy) vs desktop/mobile apps where you have to make stuff copy-able.

Closed by #255


fix(common): Synchronize file to disk in NativeIO

Overview
Fixes a rare synchronization error in the NativeIO, where when fetching is happening very fast concurrently, the file's contents may not be synced to disk fast enough for the other end of the pipe to recognize.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#256  👈
#255
#252
#251
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @clabby and the rest of your teammates on  Graphite

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 71.3%. Comparing base (27b9970) to head (f8f9794).

Additional details and impacted files

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Just a note, this is a dud. Changed upstack.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 71.3%. Comparing base (877f22e) to head (03d3283).

Additional details and impacted files

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Hi. I'm new to development field. I just download and try to run CoreUI admin template, but it didn't load properly. Can't find this folder path in the<EMAIL_ADDRESS>Please help.. Thank you.

Before opening an issue:

Search for duplicate or closed issues
Prepare a reduced test case for any bugs
Read the contributing guidelines

When asking general "how to" questions:

Please do not open an issue here

When reporting a bug, include:

Operating system and version (Windows, Mac OS X, Android, iOS, Win10 Mobile)
Browser and version (Chrome, Firefox, Safari, IE, MS Edge, Opera 15+, Android Browser)
Reduced test cases and potential fixes using CodePen or JS Bin

When suggesting a feature, include:

As much detail as possible for what we should add and why it's important to CoreUI Admin Template
Relevant links to prior art, screenshots, or live demos whenever possible

Did you try running with NPM? I'm fairly new to this package and it may not be the best solution (please correct me if I'm wrong) but you can run it in NPM and copy any files you might need out if you're running on a different service.
# From linux (ubuntu for example) you may need to upgrade your npm command
# sudo apt install npm
# npm cache clean -f
# sudo npm install -g n
# sudo npm stable

# clone the repo
$ git clone https://github.com/coreui/coreui-free-bootstrap-admin-template.git my-project

# go into app's directory
$ cd my-project

# install app's dependencies
$ npm install

$ npm run serve


Fix case crypto quantity

It looks like there was a typo where cases for cryptocurrency orders were dividing the wrong quantity field, which always resulted in a quantity of zero.
Closes #1723

I noticed the quantity was showing zero when the vendor opened the order for all listing types, so I fixed things to always standardize the quantity for both the buyer and seller contracts.


feat!: Separate logical MySQL host from connected host

MySQL server instances have a logical host name and an instance node names.
This is particularly useful when running in clustered mode where a logical hostname is used to connect to a cluster but you also want to record what specific node is executing your query.
Trilogy exposes the instance node name via @connected_host as well as the configured host name in its connection_options.
This breaking change ensures that the net.peer.name is always set to the logical host name, while the individual instance name is being recorded in a separate attribute.
cc: @cschleiden

ah @arielvalentin looks like a couple typos, my apols on the early approval, that aside this lgtm


Added transaction protection to action of create session

I've found a multithreading problem in devise_token_auth library.
When 2 or more requests to create a new session are sending in the same time for a single user, there is a big chance to get a collision. It looks strange, because both responses will have a token keys, but only the last one will be valid.
The problem is, that tokens are stored in one row of the table. Both of synchronous requests can rewrite tokens of each other.
How it worked before this fix.
Firstly we get a row from a user table, then we generate a new token and add it to a field of user's model. Than we update a user's row in database.  But, someone else can send the same response, while we are changing and saving the updated user row. As result, only one token will be saved and valid as well.
Solution
It's simple, just wrap both of SQL requests SELECT and UPDATE in a single transaction.
Testing
Currently all tests are passed successful. Unfortunately I can't to write a new test for this fix, because sqlite doesn't support transaction mechanism as well.
I tried to use this code in the before block:
        before do
          @respones = 2.times.to_a.map do
            Thread.new do
              xhr :post, :create, { email: @existing_user.email, password: 'secret123' }
            end
          end.each(&:join)
        end

But got the same errors every time: "ActiveRecord::StatementInvalid: SQLite3::BusyException: database is locked: ... "
Anyway, I've tested this fix in real project with real heavy loading requests in MySQL and PostgreeSQL databases. The fix works pretty good, in more than 10 synchronous requests. Before the fix I've got a lot of invalid authorisations responses with only 2 synchronous requests.

Hi there @ilyaerin    ,
In an effort to cleanup this project and prioritize a bit, we're marking pull requests that haven't had any activity in a while with a "close-in-7-days" label. If we don't hear from you in about a week, we'll be closing this pull request. Obviously feel free to re-open it at any time if it's the right time or this was done in error! If you do, please rebase it with the latest master and explain why it's still needed.
We really appreciate your contribution, we're just trying to make this project manageable again to move it forward.
Hope all is well.


Make messenger options available

Since 3.0.1 it's no longer possible to access the messenger.com settings. In goofy 2 you could access it through the menu bar and with cmd ,.
I assume you could trigger a javascript click event on the currently hidden element.

Good idea. We should add that!

Yep, this is annoying, someone forgot to port the menu bar option to the new App.
Keep up the good work ;)


Update the KapoorLabs cover image

To make the cover image compatible with the minimum resolution requirements, we are updating the image taken by one of our collaborators.

These changes were merged in https://github.com/chanzuckerberg/napari-hub-collections/pull/45
For some reason the checks didn't pass on this PR, so I am closing it.


feat: [M3-8837] - Add LKE-E feature flag

Description 📝
We will need a feature flag in Launch Darkly to gate access to the UI changes for LKE-Enterprise.
Changes  🔄

The lkeEnterprise flag exists in Launch Darkly.

The flag is EVERYTHING OFF in staging and prod environments and Feature ON, LA ON, GA OFF in the dev environment.

The flag is added to the dev tools feature flag list.
useIsLkeEnterpriseEnabled hook handles the enablement of the feature.

Since there is an account capability to enable the feature, the necessary account beta endpoint updates are included.
Test coverage was added for this hook.

How to test 🧪
Prerequisites
(How to setup test environment)

Have the LKE-E customer tag on your account (prod account is fine). See the internal LKE-Enterprise project tracker to find this.

Verification steps
(How to verify changes)

[ ] Read the diff since there are no UI changes yet.
[ ] Ensure that unit test coverage passes:

yarn test kubeUtils

[ ] Confirm that the LKE-E feature flag is present in dev tools.
[ ] If you want to test the hook in code, manually add the hook to a page like KubernetesLanding.tsx, console.log the return value, and confirm you see a v4beta/account network request.

  const { isLkeEnterpriseLAEnabled } = useIsLkeEnterpriseEnabled();
  console.log({ isLkeEnterpriseLAEnabled });

As an Author, I have considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿  Providing accessibility support

[x] I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

[x] All unit tests are passing
[x] TypeScript compilation succeeded without errors
[x] Code passes all linting rules

Have the LKE-E customer tag on your account (prod account is fine). See the internal LKE-Enterprise project tracker to find this.

Any thoughts about adding a Mock Service Worker preset for this? Doesn't help too much now since there aren't any UI changes, but might be handy once development ramps up a bit more.
I took a quick stab at a preset (that does the absolute bare minimum, there's probably a lot of room for improvement):
// New file at `src/mocks/presets/extra/account/lkeEnterpriseEnabled.ts` or similar

import { http } from 'msw';
import {
  accountFactory,
} from 'src/factories';
import { makeResponse } from 'src/mocks/utilities/response';

import type { MockPresetExtra } from 'src/mocks/types';

const mockLkeEnabledCapability = () => {
  return [
    http.get('*/v4*/account', async ({ request }) => {
      return makeResponse(
        accountFactory.build({
          capabilities: [
            // Other account capabilities might be necessary here, too...
            // TODO Make a `defaultAccountCapabilities` factory.
            'Kubernetes',
            'Kubernetes Enterprise',
          ],
        })
      );
    }),
  ]
};

export const lkeEnterpriseEnabledPreset: MockPresetExtra = {
  desc: 'Mock account with LKE Enterprise capability',
  group: { id: 'Account', type: 'select' },
  handlers: [mockLkeEnabledCapability],
  id: 'account:lke-enterprise-enabled',
  label: 'LKE Enterprise Enabled',
};

(And then there are some accompanying changes required to the MockPresetExtraId type in src/mocks/types.ts, and the preset needs to be added to the extraMockPresets array in src/mocks/presets/index.ts to get it to show up in the dev tools)

I'm going to stick this back in Draft for now after talking about RQ and cache management with Banks. (relevant comments here and here
(Thanks for your comments, Joe. MSW preset is a good idea.)

Putting this back up for review with some updates:

The LKE-E API folks report that they used v4beta/account to be cautious; after discussion, it seems more likely this will become v4/account
Until then, we are opting to use a separate beta query and query key because regular and beta endpoint caches don't really need to be in sync. The purpose of the cached beta endpoint is to read the capabilities only. While not very DRY, beta-specific query and cache seems less risky than updating the cache logic for the existing query, as we needed to with /clusters.
@jdamore-linode Added a preset for LKE-Enterprise Enabled! I think this, and the ability to dynamically mock capabilities in general, could be useful to improve in the future, for sure. Some of the devs have discussed this in office hours before. Let's work with what you suggested for now.


Clangd cpp install fail

It appears the curl request to get the platform clangd link returns 2 URLs (see below) resulting in this error:
Warning: Binary output can mess up your terminal. Use "--output -" to tell
Warning: curl to output it to your terminal anyway, or consider "--output
Warning: <FILE>" to save to a file.

URLs:
https://github.com/clangd/clangd/releases/download/12.0.0/clangd-indexing-tools-mac-12.0.0.zip
https://github.com/clangd/clangd/releases/download/12.0.0/clangd-mac-12.0.0.zip

The error can be repeated by running this:
curl -L -o "clangd.zip" https://github.com/clangd/clangd/releases/download/12.0.0/clangd-indexing-tools-mac-12.0.0.zip https://github.com/clangd/clangd/releases/download/12.0.0/clangd-mac-12.0.0.zip

Will fix, related: https://github.com/clangd/vscode-clangd/issues/180


KeyError '23' while running examples

Has anyone else seen this while running the examples?
export MODEL_PATH=../examples/ft/prom-core/$KMER
export DATA_PATH=examples
export PREDICTION_PATH=examples
python ../examples/run_finetune.py \
    --model_type dna \
    --tokenizer_name=dna$KMER \
    --model_name_or_path $MODEL_PATH \
    --task_name dnaprom \
    --do_predict \
    --data_dir $DATA_PATH  \
    --max_seq_length 75 \
    --per_gpu_pred_batch_size=128   \
    --output_dir $MODEL_PATH \
    --predict_dir $PREDICTION_PATH \
    --fp16 \
    --n_process 48

error message we are seeing is
12/14/2020 01:54:26 - INFO - transformers.data.processors.glue -   Writing example 0/5
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/home/ubuntu/anaconda3/envs/pytorch_latest_p37/lib/python3.7/multiprocessing/pool.py", line 121, in worker
    result = (True, func(*args, **kwds))
  File "/home/ubuntu/DNABERT/src/transformers/data/processors/glue.py", line 120, in glue_convert_examples_to_features
    label = label_map[example.label]
KeyError: '23'
"""
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
  File "../examples/run_finetune.py", line 1281, in <module>
    main()
  File "../examples/run_finetune.py", line 1152, in main
    prediction = predict(args, model, tokenizer, prefix=prefix)
  File "../examples/run_finetune.py", line 484, in predict
    pred_dataset = load_and_cache_examples(args, pred_task, tokenizer, evaluate=True)
  File "../examples/run_finetune.py", line 761, in load_and_cache_examples
    features.extend(result.get())
  File "/home/ubuntu/anaconda3/envs/pytorch_latest_p37/lib/python3.7/multiprocessing/pool.py", line 657, in get
    raise self._value
KeyError: '23'

@Zhihan1996 halp?

Do you use your own data to finetune the model?

Nope. We were running your example with KMER=6.
On Wed, Dec 16, 2020 at 5:18 PM Zhihan Zhou<EMAIL_ADDRESS>wrote:

Do you use your own data to finetune the model?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jerryji1993/DNABERT/issues/6#issuecomment-747141642,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMR7EOZCVWIE6H4Z5BRC343SVFL6RANCNFSM4U3QHTOA
.


ModuleNotFoundError: No module named 'darknet_ros_msgs'

Hello
I have built the YOLO-ROS on my Ubuntu 18.04 system with Ros eloquent.
This works so far. My camera is displayed with the baunding boxes.
If I now try to display the following Ros Topics with ros2 topic echo cih get the following error:
Topic:

/darknet_ros/found_object
/darknet_ros/bounding_boxes

Error:
ros2 topic echo /darknet_ros/bounding_boxes
Traceback (most recent call last):
File "/opt/ros/eloquent/bin/ros2", line 11, in
load_entry_point('ros2cli==0.8.8', 'console_scripts', 'ros2')()
File "/opt/ros/eloquent/lib/python3.6/site-packages/ros2cli/cli.py", line 69, in main
rc = extension.main(parser=parser, args=args)
File "/opt/ros/eloquent/lib/python3.6/site-packages/ros2topic/command/topic.py", line 43, in main
return extension.main(args=args)
File "/opt/ros/eloquent/lib/python3.6/site-packages/ros2topic/verb/echo.py", line 83, in main
return main(args)
File "/opt/ros/eloquent/lib/python3.6/site-packages/ros2topic/verb/echo.py", line 97, in main
node.node, args.topic_name, args.message_type, callback, qos_profile)
File "/opt/ros/eloquent/lib/python3.6/site-packages/ros2topic/verb/echo.py", line 131, in subscriber
msg_module = import_message_type(topic_name, message_type)
File "/opt/ros/eloquent/lib/python3.6/site-packages/ros2topic/api/init.py", line 74, in import_message_type
module = importlib.import_module(package_name + '.' + middle_module)
File "/usr/lib/python3.6/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 994, in _gcd_import
File "", line 971, in _find_and_load
File "", line 941, in _find_and_load_unlocked
File "", line 219, in _call_with_frames_removed
File "", line 994, in _gcd_import
File "", line 971, in _find_and_load
File "", line 953, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'darknet_ros_msgs'
The following topic works without problems:

/darknet_ros/detection_image

I have checked the cMake list of the package darknet_ros_msgs. This seems to be ok so far.
Can someone help me with this?
Thanks in advance!

HI ! Did you succeed to solve it ?

HI ! Did you succeed to solve it ?

Yes I have solved it. I had a problem with the source command. Which was a bug in my operating system.
After I solved it I could source the built workspace normally again and display the message with ros2 topic.

Yes I have solved it. I had a problem with the source command. Which was a bug in my operating system.
After I solved it I could source the built workspace normally again and display the message with ros2 topic.


NFC: Move ReplacePushConstants pass to Common/ for Metal

We'll use the same pass to replace push constant access as normal uniform buffer access later.
Progress towards https://github.com/openxla/iree/issues/4370

I think we want to use argument buffers and shouldn't need this pass. Is there a reason you aren't going to use them?

(I'm guessing because you're routing through SPIR-V instead of generating msl?)

I think we want to use argument buffers and shouldn't need this pass. Is there a reason you aren't going to use them?

Yes I'm using argument buffers for descriptor sets; that's pretty straightforward. Though I don't find good resources about push constant equivalences in Metal. It looks like https://developer.apple.com/documentation/metal/mtlargumentencoder/2915771-constantdataatindex is? From its doc:

index
This value corresponds to either the index ID of a Metal shading language declaration or the index field of a MTLArgumentDescriptor object.

It suggests we can do either do buffer(N) style or id(N) style? (Although "within the argument buffer" is confusing to me.) The latter is encoding data into an argument buffer ("descriptor set"), where the former is more at the "pipeline" level so what we want for push constant. I'm not sure how to mix them though. Maybe call constantDataAtIndex: on the argument encoder before setting any active argument buffer?

(I'm guessing because you're routing through SPIR-V instead of generating msl?)

Yeah, this is another problem. I tried to play with constantDataAtIndex:, but didn't figure out a way to let SPIRV-Cross generate the proper MSL code to mix the argument buffer style and old buffer(N) style and not let everything just crash..
But maybe it's due to my lack of experience. If you know pointers / resources, that'd be great! :)

I think we want to use argument buffers and shouldn't need this pass. Is there a reason you aren't going to use them?

Yes I'm using argument buffers for descriptor sets; that's pretty straightforward. Though I don't find good resources about push constant equivalences in Metal. It looks like https://developer.apple.com/documentation/metal/mtlargumentencoder/2915771-constantdataatindex is? From its doc:

...

But maybe it's due to my lack of experience. If you know pointers / resources, that'd be great! :)

I think setBytes is what you want.
See https://developer.apple.com/documentation/metal/mtlcomputecommandencoder/1443159-setbytes?language=objc

Discussion
Using this method is equivalent to creating a new MTLBuffer object from the specified data and then binding it to the compute shader with the setBuffer:offset:atIndex: method. However, this method avoids the overhead of creating a buffer to store your data; instead, Metal manages the data.

Use this method for single-use data smaller than 4 KB. Create a MTLBuffer object if your data exceeds 4 KB in length or persists for multiple uses.

@dneto0: yup indeed! thanks for the pointers!
After some searching I found out the way to teach SPIRV-Cross to emit the MSL I'd expect: https://github.com/KhronosGroup/SPIRV-Cross/issues/2098#issuecomment-1406377858. So this is not needed any more.
Thanks everyone for the help! :)


Python-based, cross platform buildResourcePack.py

TODO: shit's still broke.

So everything works, but for some reason the game code can't read the zip file and sees it as completely empty.
The hell.

Can you add a requirements.txt to list all the dependencies you'll need to install?
pip freeze > requirements.txt

@psykzz onlyuses first party modules in Python 3.5. If colorama is installed it'll output colored text but that's completely optional.

PIL is first-party?

@ComicIronic the sprite dummy is for debugging the script. It doesn't get ran unless you set the sprite renderer executable to it.

And yet, if someone did do that it wouldn't work, because they don't have PIL.
Either remove the DummyScript or add its dependencies.
What harm is a requirements.txt file anyway?

Alright I'll put a requirements.txt next to the dummy.

Should be done. Works on Windows. Requires Python 3.5.

Approved


Question of ioRatio in NioEventLoop and two backed array in SelectedSelectionKeySet

Hi,
I was reading netty source code these days, netty is really a great project and the source code is so beautiful！
Now I have two questions to ask for your help：
purpose of ioRatio in NioEventLoop.
Does ioRatio mean the elapsed time of io operation?As the methodprocessSelectedKey above   is cpu intennsive(copy bytes from tcp buffer to jvm direct buffer), is ioRatio simply used to control the proportion of copy bytes and business task?Is there any other purpose,like efficiency? ^^
perhaps two backed array in SelectedSelectionKeySet may be reduced to one.
I see this set is backed by two arrays, but I think use one array can also do the same thing as every time after NioEventLoopcall processSelectedKeysOptimized(SelectionKey[] selectedKeys), the two SelectionKey[] is filled with Null point

flip method in SelectedSelectionKeySet

SelectionKey[] flip() {
        if (isA) {
            isA = false;
            keysA[keysASize] = null;
            keysBSize = 0;
            return keysA;
        } else {
            isA = true;
            keysB[keysBSize] = null;
            keysASize = 0;
            return keysB;
        }
    }

perhaps if the flip method is replaced with end method as follows can do the same and also avoid execif (isA) ^^

end method

   SelectionKey[] end() {
        keys[keysSize] = null;
        keysSize = 0;
        return keys;
    }

Thank you for your help!!!

purpose of ioRatio in NioEventLoop.

This provides a rough mechanism to control how much time is spent processing IO related tasks (socket reads, connects, closed, hangups, etc..) vs "non-IO" related tasks. The "non-IO" the Executor related events the IO thread has queued. The lower the number the more time will be spent on "non-IO" tasks, and currently 100 disables the timeout all together and runs all pending "non-IO" tasks.

perhaps two backed array in SelectedSelectionKeySet may be reduced to one.

Some of the methods may return "invalid values" if we are operating on the internal array (e.g. size) ... however this may be OK if the interface is only inconsistent while we are using it ... @normanmaurer @trustin  - WDYT?
Also I wonder if the iteration to bounds in processSelectedKeysOptimized can be handled better:
https://github.com/netty/netty/blob/4.1/transport/src/main/java/io/netty/channel/nio/NioEventLoop.java#L582
// to call the updated method:
int size = selectedKeys.size();
processSelectedKeysOptimized(selectedKeys.flip(), size);

// alternatively if we make the changes suggested in this issue:
// processSelectedKeysOptimized(selectedKeys.array(), selectedKeys.size());

private void processSelectedKeysOptimized(SelectionKey[] selectedKeys, int end) {
  for (int i = 0; i < size; ++i) {
   final SelectionKey k = selectedKeys[i];
   // null out entry in the array to allow to have it GC'ed once the Channel close
   // See https://github.com/netty/netty/issues/2363
   selectedKeys[i] = null;

   ...

    if (needsToSelectAgain) {
      for (int x = i + 1; x < size; ++x) {
        selectedKeys[i] = null;
      }
      ...
    }
  }
}

let me take a stab at this...

@Scottmitch thank you for eliminating my doubts, this is my first time to create an issue in github. I am so happy for receiving your reply^^

还是不太明白ioRatio 到底是什么作用,能给解释下吗

@lxchinesszz - can you translate to English please?

@lxchinesszz

This provides a rough mechanism to control how much time is spent processing IO related tasks (socket reads, connects, closed, hangups, etc..) vs "non-IO" related tasks. The "non-IO" tasks are Runnable objects queued in the EventLoopGroup due to using the Executor interface (e.g. Executor#execute(..). The lower the number the more time will be spent on "non-IO" tasks, and currently 100 disables the timeout all together and runs all pending "non-IO" tasks.

上面团队成员已经回答了啊.用来大致控制IO相关任何和非IO相关任务的时间分配比.

thanks @jiakme


Two icons problem

As you can see, shows two different icons on notification:

Same issue in my rails app

If I had to guess, you've probably got a CSS conflict. I was using a pre-built theme that included the Toastr plugin, and some of the custom styling in that theme was causing two icons to appear.
Here's the code that I removed solved the issue:
#toast-container > .toast-warning:before {   content: "\f0e7"; }
#toast-container > .toast-error:before {   content: "\f071"; }
#toast-container > .toast-info:before {   content: "\f005"; }
#toast-container > .toast-success:before {   content: "\f00C"; }
Do some digging in your CSS, and I bet you'll find the same sort of thing going on.

@murbano83 loading toastr css before any other css, fix this problem for me.

Nothing to be done with any of the above.Just keep your
toastr css immediately after boostrap(if using)  and before any other css links.Its will work without any issues.
likes this:
     <link href="~/Content/bootstrap.min.css" rel="stylesheet">
     <link href="~/Content/plugins/toastr/toastr.min.css" rel="stylesheet">

....remaining links here after.....comes


Lots of uknown objects with .net core 2.x

When I get a memory dump of any .net core project I am seeing lots of uknown objects with no clear description as below  :

This in turn diminishes the value of memory dumps for Perfview.
For above trace I have used the below code with .net core 2.2:
using System;
using System.Collections.Generic;
using System.Linq;

namespace ConsoleApp89
{
    class Program
    {
        static List<string> l;//= new List<string>(10000);
        static void Main(string[] args)
        {
            l = Enumerable.Range(1, 100000).Select(k => k.ToString()).ToList();

            Console.WriteLine("Hello World!");
            Console.ReadKey();
        }
    }
}

We are experiencing the same issue. Again with netcore 2.2

Same here, from log file:
Error: Did not have a type definition for typeID 0x7ffb85f94f38
Error: Did not have a type definition for typeID 0x7ffb82c11a30
Error: Did not have a type definition for typeID 0x7ffb82535e48
Error: Did not have a type definition for typeID 0x7ffb82bef878
Error: Did not have a type definition for typeID 0x7ffb85f962f8
Error: Did not have a type definition for typeID 0x7ffb82d978ba
Error: Did not have a type definition for typeID 0x7ffb85f9a5f0
Error: Did not have a type definition for typeID 0x7ffb85f9abb8
Error: Did not have a type definition for typeID 0x7ffb85f9bf00
Error: Did not have a type definition for typeID 0x7ffb82d78678
Error: Did not have a type definition for typeID 0x7ffb82c4e4d0
Error: Did not have a type definition for typeID 0x7ffb82c602ca
2.5s: Shutting down ETW session
2.8s: ETW Listener dieing
2.9s: Waiting for shutdown to complete.
[  2.9s: Done Dumping .NET heap success=True]
Finished reading .NET Dump AHCoreB2.gcdump, 387778 Nodes 4981 Types
Creating a GC Dump from a liver process 2588
Object count 387K > MaxDumpCount = 250K, sampling
3.1s:   Started Sampling.
************* SAMPLING GRAPH TO REDUCE SIZE ***************
Original graph object count 387,778, targetObjectCount 250,000 targetRatio 1.55
Original graph Size MB 50,950,084 TypeCount 4,981
Stats of the top types (out of 4,955)
OrigSizeMeg SampleSizeMeg   Ratio   |   OrigCnt  SampleCnt    Ratio   | Ave Size | Type Name
10.485880   10.485880       1.00 |          5         5       1.00 |  2097176 | UNKNOWN 0x7ffb823aee7a (Bytes > 1M)
8.694912    4.597048       1.89 |    116,757    75,273       1.55 |       74 | UNKNOWN 0x7ffb82445f18
4.680456    4.680456       1.00 |          1         1       1.00 |  4680456 | UNKNOWN 0x7ffb82660b12 (Bytes > 1M)
4.516560    4.516560       1.00 |         33        33       1.00 |   136865 | UNKNOWN 0x7ffb823aee7a (Bytes > 100K)
2.585952    2.585952       1.00 |          4         4       1.00 |   646488 | UNKNOWN 0x7ffb82617ba2 (Bytes > 100K)
0.970752    0.625680       1.55 |      1,536       990       1.55 |      632 | UNKNOWN 0x7ffb836652b0
0.841608    0.542592       1.55 |     11,689     7,536       1.55 |       72 | UNKNOWN 0x7ffb8295e558
0.654584    0.422016       1.55 |     11,689     7,536       1.55 |       56 | UNKNOWN 0x7ffb82a3b398
0.576168    0.576168       1.00 |          1         1       1.00 |   576168 | UNKNOWN 0x7ffb82660b12 (Bytes > 100K)
0.550224    0.354912       1.55 |      3,110     2,005       1.55 |      177 | UNKNOWN 0x7ffb8286f63a
0.544512    0.350688       1.55 |      3,075     1,982       1.55 |      177 | UNKNOWN 0x7ffb82ba03d2
0.485400    0.312920       1.55 |     12,135     7,823       1.55 |       40 | UNKNOWN 0x7ffb823b8408
0.460076    0.299976       1.53 |      8,571     5,526       1.55 |       54 | UNKNOWN 0x7ffb82455772
0.455210    0.325150       1.40 |          7         5       1.40 |    65030 | UNKNOWN 0x7ffb82549d7a (Bytes > 10K)
0.396832    0.255840       1.55 |     12,401     7,995       1.55 |       32 | UNKNOWN 0x7ffb82962790
0.384192    0.247680       1.55 |      4,002     2,580       1.55 |       96 | UNKNOWN 0x7ffb82560488
0.374048    0.241152       1.55 |     11,689     7,536       1.55 |       32 | UNKNOWN 0x7ffb8295f2f8
0.370680    0.239256       1.55 |      6,597     4,253       1.55 |       56 | UNKNOWN 0x7ffb829f7cea
0.342288    0.220656       1.55 |      7,131     4,597       1.55 |       48 | UNKNOWN 0x7ffb82a12888
0.334889    0.236169       1.42 |         17        11       1.55 |    19699 | UNKNOWN 0x7ffb823aee7a (Bytes > 10K)
0.304200    0.196144       1.55 |      2,925     1,886       1.55 |      104 | UNKNOWN 0x7ffb8253fe00
0.294232    0.212520       1.38 |          9         7       1.29 |    32692 | UNKNOWN 0x7ffb823ac0c2 (Bytes > 10K)
0.291888    0.222560       1.31 |          9         6       1.50 |    32432 | UNKNOWN 0x7ffb82455772 (Bytes > 10K)
0.290872    0.290872       1.00 |          2         2       1.00 |   145436 | UNKNOWN 0x7ffb82455772 (Bytes > 100K)
0.272928    0.175968       1.55 |     11,372     7,332       1.55 |       24 | UNKNOWN 0x7ffb823b4a50
0.246240    0.246240       1.00 |      3,078     3,078       1.00 |       80 | UNKNOWN 0x7ffb8295ee60
0.246240    0.221280       1.11 |      3,078     2,766       1.11 |       80 | UNKNOWN 0x7ffb8295dac8
0.224784    0.144936       1.55 |      3,122     2,013       1.55 |       72 | UNKNOWN 0x7ffb8264c018
0.221616    0.142848       1.55 |      3,078     1,984       1.55 |       72 | UNKNOWN 0x7ffb82b87580
0.221616    0.142848       1.55 |      3,078     1,984       1.55 |       72 | UNKNOWN 0x7ffb82b668b0

Same problem here. It is not finding the definition of any type, everything is "UNKNOWN". Using netcore 2.2.

@vancem @sharwell - do either of you know what is up with this?  We're trying to do some profiling of ASP.NET Core 3.0 and running into the same issue.

@Pilchie this commonly occurs when the circular buffer size is not large enough and the events that map the type name to the class ID get overwritten.  Can you try re-collecting with a larger circular buffer?  You can do this by specifying /CircularMB on the command line.

@brianrob I've tried this with max circular buffer setting:
 ./PerfView.exe HeapSnapshot 117320 my-dump.gcdump -CircularMB:2147483647
But I'm still getting mostly UNKNOWN in the output. I can see the errors while running the command:

This is also for a .NET Core 2.2 application.


make some tests build

some simple dependency fixes to make tests build.
note the test folder is out of scope - we're moving to deprecate it because I don't really know how anyone is expected to be able to maintain it (very brittle). and of course rpc tests are still failing, ala at least #109

looks good to me


Make Notebook images configurable

Hi,
Can we make: ``spawner_ui_config.yaml``` configurable?
We need to have an option on the charms to allows us to set custom notebook images so users can build their own.
I think this is a nice reference from upstream: https://github.com/kubeflow/manifests/blob/master/apps/jupyter/jupyter-web-app/upstream/base/configs/spawner_ui_config.yaml

If I understand @phvalguima correctly, he wants to be able to configure the default images. This makes sense for bigger organizations that would probably want to provide their own default images.

I was able to use my own images as default by editing the ConfigMap jupyter-ui-config-config. But I found that the ConfigMap reverts back to default values after a few days. Looking for a way to persist these kind of changes.

This is supported in the latest version of CKF 1.8
https://charmhub.io/jupyter-ui/configure?channel=1.8/stable


Skip tutorial button added.

Solved issue #224

@opticod
I have deliberately kept it that way.
Skip tutorial button usually appears only the first time user installs the app (if the user is in a hurry and just wants to get the work done).The app is implicitly showing the tutorial.
But later on the user is explicitly viewing the tutorial, there is no point of adding skip button there.
Should i keep it this way or

Seems logical. Thanks @free4murad ..Merging

@opticod
Should I make a new PR just push changes to this one?

You have to create a new PR. @free4murad


Is there way to extract coordinates

Hi,
I am working with pdf files and I came across the box-detect library. Thanks for creating this amazing library. I am using the following PDF file
Form_49A.PDF
I am trying to annotate over the pdf file, however, for achieving this I was looking at ways to extract these boxes and then annotate. Is there a way to extract the coordinates for the boxes present in the pdf-file?

Hi @SuryaViswanath11 ,
Glad to hear you found BoxDetect useful.
To use BoxDetect functions you need to first convert your PDF to images which is a fairly simple task. You can use one of few available packages to do it, like pdf2image

Hi @SuryaViswanath11 , Glad to hear you found BoxDetect useful. To use BoxDetect functions you need to first convert your PDF to images which is a fairly simple task. You can use one of few available packages to do it, like pdf2image

Seems there is no direct answer to the question, Is there a way to extract the coordinates for the boxes present in the image file?

Hi @teohsinyee
Each function from BoxDetect takes an image as input and returns a collection of coordinates for detected boxes (based on config params).
Example:
from boxdetect.pipelines import get_boxes

rects, grouping_rects, image, output_image = get_boxes(
    file_name, cfg=cfg, plot=False)

print(grouping_rects)

OUT:
# (x, y, w, h)
[(276, 276, 1221, 33),
 (324, 466, 430, 33),
 (384, 884, 442, 33),
 (985, 952, 410, 32),
 (779, 1052, 156, 33),
 (253, 1256, 445, 33)]

import matplotlib.pyplot as plt

plt.figure(figsize=(20,20))
plt.imshow(output_image)
plt.show()

Another:
from boxdetect.pipelines import get_checkboxes

checkboxes = get_checkboxes(
    file_path, cfg=cfg, px_threshold=0.1, plot=False, verbose=True)

print("Output object type: ", type(checkboxes))
for checkbox in checkboxes:
    print("Checkbox bounding rectangle (x,y,width,height): ", checkbox[0])
    print("Result of `contains_pixels` for the checkbox: ", checkbox[1])
    print("Display the cropout of checkbox:")
    plt.figure(figsize=(1,1))
    plt.imshow(checkbox[2])
    plt.show()


🐛 BUG: ERR_UNSUPPORTED_ESM_URL_SCHEME Prettier v3

Describe the Bug
When formatting .astro files in VSCode this error shows in the console. Prettier v3
Error: Only URLs with a scheme in: file and data are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'f:'
    at new NodeError (file:///F:/Developer/node_modules/.pnpm/prettier@3.0.0/node_modules/prettier/index.mjs:19091:5)
    at throwIfUnsupportedURLScheme (file:///F:/Developer/node_modules/.pnpm/prettier@3.0.0/node_modules/prettier/index.mjs:20134:11)
    at defaultResolve (file:///F:/Developer/node_modules/.pnpm/prettier@3.0.0/node_modules/prettier/index.mjs:20171:3)
    at resolve (file:///F:/Developer/node_modules/.pnpm/prettier@3.0.0/node_modules/prettier/index.mjs:20191:12)
    at importFromFile (file:///F:/Developer/node_modules/.pnpm/prettier@3.0.0/node_modules/prettier/index.mjs:20206:15)
    at importFromDirectory (file:///F:/Developer/node_modules/.pnpm/prettier@3.0.0/node_modules/prettier/index.mjs:20470:10)
    at file:///F:/Developer/node_modules/.pnpm/prettier@3.0.0/node_modules/prettier/index.mjs:20487:12
    at async loadPlugin (file:///F:/Developer/node_modules/.pnpm/prettier@3.0.0/node_modules/prettier/index.mjs:20492:28)
    at async Promise.all (index 0)
    at async Promise.all (index 1) {
  code: 'ERR_UNSUPPORTED_ESM_URL_SCHEME'
}

Steps to Reproduce
Format any .astro file in VSCode

Hmm, seems like a Windows issue. What does your Prettier config look like?

It worked, it worked. A clean refresh and re-install. Ty @Princesseuh


add machine to kubernetes

Organization Name:

Short summary about the issue/question:
Brief what process you are following:

How to reproduce it:

OpenPAI Environment:

OpenPAI version:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Hardware (e.g. core number, memory size, storage size, GPU type etc.):
Others:

Anything else we need to know:

@qzwlinux - I'm closing this issue seems you had created another duplicate one: #2369


You, sir, are a wonderful person

Just some thanks 👍

Ah! You have no idea how much this means to me. Thank you!
On Tue, Nov 15, 2016 at 5:57 PM, Noah Overcash<EMAIL_ADDRESS>wrote:

Closed #20 https://github.com/zachlatta/school/issues/20.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zachlatta/school/issues/20#event-860386036, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA8j-AxcvV5F0Kg5_YOuRyMrme3sgvZnks5q-mMMgaJpZM4KzQTx
.

--
Zach Latta
Hack Club |<EMAIL_ADDRESS>


First Version

My first submission. Using memory-mapping to split the file parsing in chunks, the rest is pretty standard stuff. Using manual parsing of lines to avoid building huge UTF-8 strings.
I have a few ideas where I can save a bit more time, especially for avoiding the String parsing / unparsing & Hashing. I think I can plug the vector API to parse the digits faster but I'm not sure if it's really going to be worth it.
Check List:

[X] Tests pass (./test.sh <username> shows no differences between expected and actual outputs)
[X] All formatting changes by the build are committed
[X] Your launch script is named calculate_average_<username>.sh (make sure to match casing of your GH user name) and is executable
[X] Output matches that of calculate_average_baseline.sh

Execution time: ~6s
Execution time of reference implementation: ~2m 10

Note that I have a test that fails (with UTF-8 chars) but when I tested on other implementations it also failed for all of them, so I'm assuming it was an error in the provided code, I was a dozen commits behind. Comparing against the baseline yields correct results.

This currently fails tests:
Validating calculate_average_samuelyvon.sh -- src/test/resources/samples/measurements-3.txt

Using java version 21.0.1-graal in this shell.

real	0m0.114s
user	0m0.211s
sys	0m0.050s
1c1
< {Bosaso=-15.0/1.2/20.0, Petropavlovsk-Kamchatsky=-9.5/0.0/9.5}
---
> {Bosaso=-15.0/1.3/20.0, Petropavlovsk-Kamchatsky=-9.5/0.0/9.5}

Little rounding error. This should be fixed. I have an error with an UTF-8 test but it seems non-sensical to me. Let me know if it happens on your test setup as well (it fails for other submissions as well locally)

@gunnarmorling Ok, applied your changes.
I still have errors running this test: src/test/resources/samples/measurements-20.txt.
Unsure why the output expects the UTF-8 bee when it does not appear in the input. Odd :( It fails for other submissions so I'm unsure what's going on (same on "royvanrijn"'s submission)

All looking good now, tests pass too. 00:12.471, nice! Thx for participating!


Auto-create Bot profile during installation.

Profile ID should be stored in sys_profile_bot setting.

Related ticket #2148


Add an option to encourage keeping installed packages if possible

Built on top of #37

Moved to https://github.com/ocaml-opam/opam-0install-cudf/pull/3


SNBT Parsing Is Horribly Slow

The SNBT parser appears to rely heavily on Regex but the regexes aren't compiled.
Specifying the RegexOptions.Compiled in the Regex constructor will yield a massive speedup.

I didn't do extensive benchmarking on the parser with large files, not sure how to interpret the given description in any empirical sense being it is heavily context-dependent, but adding the compiled flag will be a simple fix. It will add some load time to the assembly, but it should be insignificant for all the more there are.

As of https://github.com/ForeverZer0/SharpNBT/commit/f27bebf6c9057ab6e2ba9e3726d5911b176c1ab7, regular expressions used in the library use the Compiled flag.


Hitting backspace doesn't move cursor to previous indent column.

VSCode Version: Code 1.6.1 (9e4e44c19e393803e2b05fe2323cf4ed7e36880e, 2016-10-13T15:43:46.054Z)
OS Version: Linux x64 4.4.0-45-generic
Extensions: DavidAnson.vscode-markdownlint-0.5.0, EditorConfig.EditorConfig-0.3.4, ajhyndman.jslint-1.2.1, dbaeumer.jshint-0.10.15, dbaeumer.vscode-eslint-1.0.8, eg2.tslint-0.5.39, jakethashi.vscode-angular2-emmet-1.0.9, ms-vscode.cpptools-0.9.2, msjsdiag.debugger-for-chrome-2.1.1, vscodevim.vim-0.4.0

Steps to Reproduce:

Type a line, hit return
press Tab
press backspace

Cursor moves back single space, doesn't move back to previous indent column.
This also happens with exiting code, html tags and inside backticks.
You can use the following code see this.
@Component({
  selector: 'lifecycle',
  template: `

      {{title}}

      <ng-content></ng-content>

    Count: {{count}}

      <button (click)="showTitle=!showTitle">Toggle Title</button>

      <button (click)="incCount()">+</button>

  `,
  styles: []
})

@rajinder-yadav
I cannot reproduce. Here I've made a test.js file with the template content. The editor identifies it is indented with spaces @ 2.
Hitting backspace removes two spaces.

There might be an issue with:

you might have turned this feature off via editor.useTabStops
the indentation detection code might not be working as expected for your file (i.e. check the bottom right corner)

Closing, I think it's VIM plugin, I tested without plugins and it's work file!


[Enhance] Add .gitignore file

Add a.gitignore file to the project, which helps to exclude certain files and folders from being tracked by Git.

https://github.com/github/gitignore/blob/main/Python.gitignore


chore(sequencer): Add instrumentation

Summary
Added instrumentation to astria-sequencer
Background
Adding instrumentation to all async calls will aid in tracing since spans will be emitted even if no events happen under them.
Changes

Disabled clippy::blocks_in_conditions lint
Added instrumentation to all async function calls that are not long-lived.
Removed instrumentation from long-running functions.
Minor refactoring of two functions to get rid of clippy exceptions and move logging within spans

Related Issues
Part of #1321
Closes #1331

I think we should be using err(Debug) throughout this PR since we're dealing with anyhow::Results, and they are usually fairly unhelpful if just Display is used.

Reverted Debug instances for this PR, will make a followup to transition to eyre instead of anyhow


Update ArtistAlbumsListFragment.java

In artist view, show albums sorted by year. I tested this change with Android Studio.

Ideally one should be able to choose the sort, but this is better than sorting by name, so i'm merging it.

I agree that ideally one should be able to choose the sort, but this is beyond my capabilities. Thanks for this solution!


Added option to set current directory setup as the default

Details of Changes

Toolbar: Added a Button that let you show/hide the "Tags" column of "Details Layout". When you click it, the layout is fixed

Validation

Built and ran the app

@ferrariofilippo @gave92 thank you!


Wagtail 4.0 - Documentation - fix up legacy release note refs & use consistent format

fix some broken refs that were lost in the migration to markdown files
use lower_snake_case for all internal refs (most follow this convention)
release process sub-section in docs should be named release schedule instead

Flagging for 4.0 - this avoids us adding new refs (mostly in the settings) that do not align with the convention + fixes some broken refs that would be a regression on the 3.0 docs.

Cherry-picked in c579a93eae4148a79c3e828e52bb554f60d73c05


https://github.com/ionic-team/ionic/issues/11690 fix

The bug is in swiper-zoom.ts#L512. The event s.touchEvents.move is not defined. Furthermore you passing the the object {passive: false, zone:false} as config to plt.registerListener. But the subscription require a non passive listener to prevent the event from bubble up to the slides controller that handle the page slide event.
Short description of what this resolves:
Changes proposed in this pull request:

Ionic Version:  2.x / 3.x
Fixes: #11690

Hello and thank you for contributing to Ionic! We have been working on porting all of the Ionic components to web components and have recently updated master to reflect this. This significant change has caused this pull request to break. While we really appreciate the time and effort you put into creating this, we are not able to merge it because of the newly introduced conflicts. We are extremely sorry about this. We will not be merging any more features in to v3. If this is a feature and you have the time, please resubmit this PR against the master branch. If this is a critical security issue in v3, we would greatly appreciate it if you would resubmit the PR against the new v3 branch. Thanks so much for your time!


Test task missing in build.xml?

How can i run the tests? There doesn't seem to be a test task.
Would it be better if the project just used a maven pom.xml? That way, you can build, test and deploy with one file? Let me know if you like me to submit a patch.

Tommy, I think you are exactly right that moving to a single pom.xml is the right approach -- I've just been postponing the work because looking at poms make my head spin (especially when adding the refs to do releases).
I have that work piling up here: https://github.com/thebuzzmedia/imgscalr/issues/41
If POMs are second nature to you and this is something you could whip up quickly and it could give us build parity to what the Ant build gives us now, that would be fantastic.

Tommy, now that the pom.xml is in the project can I mark this as fixed or did you specifically want a test task in the Ant's build script as well?

Yes  this is a closed issue. thanks!


Current Date and Time

User Story
AS AN employee with a busy schedule,
I WANT to add important events to a daily planner,
SO THAT I can manage my time effectively.

Acceptance Criteria
GIVEN I am using a daily planner to create a schedule,
WHEN I open the planner,
THEN the current day is displayed at the top of the calendar.

complete
https://nestibry.github.io/umnbc-mod05-work-day-scheduler/


Add PWA using its homepage URL

We currently only support adding PWAs from it's manifest URL, this PR allows PWAs to be added by them homepage URL.

Some PWAs, such as https://www.pureformulas.com/, have their manifests dynamically injected via JS. I doubt this is common, and they can still be added by explicitly passing the manifest URL, but if there's a nice way of working around this it's probably worth doing. (Can't think of one though, unless Lighthouse can extract for us.)

Interesting. I can't see the Manifest information on Chrome Developer Tools.

I could not find the manifest for pureformulas.com either. At the moment, I am only reading the html value.

@ithinkihaveacat @andreban PTAL

We can create another PR for handling the JS case, do you have any examples?

I added the Slugified field generation, I will create another PR to start using the field.

I will create another PR for further refactoring of Save and its test later.


exotica: 6.1.1-1 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository exotica to 6.1.1-1:

upstream repository: https://github.com/ipab-slmc/exotica.git
release repository: https://github.com/ipab-slmc/exotica-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.10.3
previous version for package: 6.0.2-1

exotica
* [exotica] Update version to 6.1.1 for documentation
* Contributors: Wolfgang Merkt

exotica_aico_solver

No changes

exotica_cartpole_dynamics_solver

No changes

exotica_collision_scene_fcl_latest

No changes

exotica_core
* Fix broken examples and expose new features (#736 <https://github.com/ipab-slmc/exotica/issues/736>)
  * Add EndPoseTask::GetS
  * SamplingProblem: Rename IsValid(q) to IsStateValid(q)
  * SamplingProblem: More debug output for IsValid
  * KinematicTree: Use same timestamp for tfs
  * Scene: Fix uninitialised quaternion
* Fix unit tests on Debian Buster (#734 <https://github.com/ipab-slmc/exotica/issues/734>)
* Fix test robot URDF with collision links
* Contributors: Wolfgang Merkt

exotica_core_task_maps
* Fix unit tests on Debian Buster (#734 <https://github.com/ipab-slmc/exotica/issues/734>)
* Contributors: Wolfgang Merkt

exotica_ddp_solver

No changes

exotica_double_integrator_dynamics_solver

No changes

exotica_dynamics_solvers

No changes

exotica_examples
* Fix broken examples (#736 <https://github.com/ipab-slmc/exotica/issues/736>)
* Fix unit tests on Debian Buster (#734 <https://github.com/ipab-slmc/exotica/issues/734>)
* Fix test robot URDF with collision links for ROS Noetic
* Python 3 fixes
* Contributors: Wolfgang Merkt

exotica_ik_solver

No changes

exotica_ilqg_solver

No changes

exotica_ilqr_solver

No changes

exotica_levenberg_marquardt_solver

No changes

exotica_ompl_control_solver

No changes

exotica_ompl_solver
* SamplingProblem: Rename IsValid(q) to IsStateValid(q) (#736 <https://github.com/ipab-slmc/exotica/issues/736>)
* Contributors: Wolfgang Merkt

exotica_pendulum_dynamics_solver

No changes

exotica_pinocchio_dynamics_solver
* Reduce compilation time and required memory
* Contributors: Wolfgang Merkt

exotica_python
* Fixes for Python 3 and newly exposed features (#736 <https://github.com/ipab-slmc/exotica/issues/736>)
  * Fix issues identified by static analysis
  * Expose EndPoseTask::GetS
  * Remove transformations.py (usage replaced by exo.KDLFrame)
  * Install convert_moveit_scene_to_sdf & remove use of transformations.py
  * SamplingProblem: Rename IsValid(q) to IsStateValid(q)
* Python 3 fixes
* Contributors: Wolfgang Merkt

exotica_quadrotor_dynamics_solver

No changes

exotica_scipy_solver
* EndPoseSolver: Specify solver options only where applicable (#736 <https://github.com/ipab-slmc/exotica/issues/736>)
* Python 3 fixes
* Contributors: Wolfgang Merkt

exotica_time_indexed_rrt_connect_solver

No changes

The sync is small so I'm going to make an exception for this so we can get it in before the final sync.


File Explorer Sorting: Unable to Sort Files by Creation Date

I have noticed that the Visual Studio Code file explorer does not currently support sorting files by creation date. The files are sorted alphabetically by default, but it would be highly beneficial to have the option to sort files by their creation date instead.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

add this feature

once again start the voting


rename database

Describe this PR
The database will most likely not be a mirror so renaming to reflect this/
Checklist

[x] Code pipeline builds correctly

Follow up actions after merging PR
Check the terraform and init pipeline stage

Should we leave the Postgres version as 11.1?

I checked the other repos and they are 11.1, but I'm not sure

Should we leave the Postgres version as 11.1?

I checked the other repos and they are 11.1, but I'm not sure

Yeah, I'm not sure either


typo: Double word "card"

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

#sign-off


Create an exception for BungeeCord namespace in plugin message namespace creation

Referencing #16

Resolved by adding bungeecord:main support for 1.13+ contexts


Erratic behavior in autocorrect for deletetag command

Input "deletetaa friends" is autocorrected to "deletetag friends" but input "aeletetag friends" is treated as unknown command. Both have edit distance of 1. Why are they handled differently?

I will preprocess the first alphabet to reduce the processing time to autocorrect a command. I have this consideration is because it has very low chance that the first input alphabet is wrong, usually the first input alphabet is correct while the subsequent alphabets are wrong unless we deliberately typed the first letter wrongly.

Alright. Please document this setting in the developer and user guide.


Join should not include columns twice

Consider
from first_table
join side:left second_table [some_index]

which currently outputs
SELECT
  first_table.*,
  second_table.*,
  some_index
FROM
  first_table
  LEFT JOIN second_table USING(some_index)

but probably should output
SELECT
  *
FROM
  first_table
  LEFT JOIN second_table USING(some_index)

I'm going to start tagging issues that we're relying on the semantic branch with the semantic label.
If anyone needs a fix urgently to unblock them, please post and I can try and rush something. Some will be possible, some less so...
(CC @aljazerzen, hope that's OK your end)

(I'm ok with tagging semantic)
Unfortunately, this is not possible in general, at least not without knowledge of table DDL. A counter example is this:
from first_table
select [some_index, foo]
join second_table [some_index]
join third_table [some_index]

which would, in your proposal, probably translate to:
SELECT
  *
FROM
  first_table
  JOIN second_table USING(some_index)
  JOIN third_table USING(some_index)

But this includes all columns from first_table too, even though we wanted only some_index and foo.
I know that current behavior is not ergonomic at all, because results contains basically three copies of some_index and because it (maybe) doesn't even compile in all DB engines, due to ambiguity of some_index in SELECT.
Fortunately, upcomming language changes on semantic branch include how USING is handled. I haven't written about it anywhere yet, so I'll use this issue to open the discussion:
The change
I added "self equality" operator ~, which does exactly what plain ident did before in join:
from first_table
join second_table [~some_index]

which translates to:
SELECT
  first_table.*
  second_table.*
FROM first_table
JOIN second_table ON first_table.some_index = second_table.some_index

It does not translate to USING, but to ON left = right, which means that the result also contains both columns from left and right.
The reason
We currently have custom behavior on join where plain idents are interpreted as USING and resolved in each of the tables separately. This is not possible anymore, because transforms are normal functions that cannot have custom resolving behavior.
I may be able to get around this limitation by extending rules on functions, but I'm not sure if this is possible.

It's a small language change, but I want your opinions on it anyway, @max-sixty @snth .
I think it's unfortunate consequence that reduces ergonomics of the language a little, but is tolerable compared to upsides that it brings.

The change
I added "self equality" operator ~, which does exactly what plain ident did before in join:
from first_table
join second_table [~some_index]

which translates to:
SELECT
  first_table.*
  second_table.*
FROM first_table
JOIN second_table ON first_table.some_index = second_table.some_index

It does not translate to USING, but to ON left = right, which means that the result also contains both columns from left and right.
The reason
We currently have custom behavior on join where plain idents are interpreted as USING and resolved in each of the tables separately. This is not possible anymore, because transforms are normal functions that cannot have custom resolving behavior.
I may be able to get around this limitation by extending rules on functions, but I'm not sure if this is possible.

It's a small language change, but I want your opinions on it anyway, @max-sixty @snth .
I think it's unfortunate consequence that reduces ergonomics of the language a little, but is tolerable compared to upsides that it brings.

I will need to ponder this a bit as I don't know enough about the resolving behaviour to understand what you mean by

because transforms are normal functions that cannot have custom resolving behavior.

In the example
from first_table
select [some_index, foo]
join second_table [some_index]
join third_table [some_index]

, can it not turn the first two rows into a CTE and then use the SELECT * FROM ... form?
What if in first_table the column was called first_index so that it has to be renamed with a derive, would it generate a CTE then as in?
from first_table
derive [some_index=first_index, foo]
join second_table [some_index]
join third_table [some_index]

The current playground version translates this to:
SELECT
  first_table.*,
  first_table.foo,
  second_table.*,
  third_table.*,
  some_index
FROM
  first_table
  JOIN second_table USING(some_index)
  JOIN third_table USING(some_index)

which seems wrong to me since in my hypothetical example first_table has no some_index column.

I'd also like to highlight that for a LEFT JOIN, first_table.some_index and second_table.some_index are not equivalent since the latter will contain NULLs when no match is found.

I don't know if this is much help but that's all that I could think of for now.

With "normal functions" I mean that we resolve their arguments the same way as we resolve arguments to any other function. Which allows you to define things like:
func left_join_by_name right_tbl filter left_tbl -> (
  join side:left right_tbl [left_tbl.name == right_tbl.name] left_tbl
)

from a
left_join_by_name b

More about that is written here.

can it not turn the first two rows into a CTE?

Like so?
WITH _table_0 AS (
  SELECT
    *
  FROM first_table
  JOIN second_table USING(some_index)
)
SELECT
  *
FROM second_table
JOIN third_table USING(some_index)

This still has the problem in _table_0 CTE: we have included all colums of first_table, where we only wanted some_index and foo.
(and disregard current implementation, it has many problem like this)

are not equivalent since the latter will contain NULLs when no match is found

Good point. That's another reason to include both columns in the table after join.
(which does not happen if we compile to USING)

Regarding the CTE example, what I envisioned was
from first_table
select [some_index, foo]
join second_table [some_index]
join third_table [some_index]

turning into
WITH _table_0 AS (
  SELECT
    some_index,
    foo
  FROM first_table
)
SELECT *
FROM _table_0
JOIN second_table USING(some_index)
JOIN third_table USING(some_index)

and
from first_table
derive [some_index=first_index, foo]
join second_table [some_index]
join third_table [some_index]

turning into
WITH _table_0 AS (
  SELECT
    first_index AS some_index,
    foo
  FROM first_table
)
SELECT *
FROM _table_0
JOIN second_table USING(some_index)
JOIN third_table USING(some_index)

I tend to write the joins out manually and haven't used USING much so I'd have to refresh my memory on how most RDBMs treat SELECT * FROM t1 LEFT JOIN t2 USING (index). My guess would be that they only include t1.index and leave out t2.index.

Ah, yeah that does make sense and would work!
It's not the easiest to implement, though. In any case, it's not needed if we stop compiling to USING.

I too haven't used USING before @max-sixty mentioned he uses it a lot. It's convenient, but only if you have columns with the same names in both tables, so I'm guessing there is a lot of people that are not familiar with its behavior, who would want to have something similar in PRQL.

Update: original issue has been circumvented with semantic branch, which now compiles:
from first_table
join side:left second_table [~some_index]

... to ...
SELECT
  first_table.*,
  second_table.*
FROM
  first_table
  LEFT JOIN second_table ON first_table.some_index = second_table.some_index

We have discussed new behavior of join and self equality operator on the call and we will be adopting this new behavior in 0.3 release and possibly amend it in patch releases.

Self equality is now discussed in #1168