Timestamp error

Hi, 1st of all, thanks for making this SG Weather integration. I am running on 1.1.3-fix and encountering this error:
2022-01-23 13:27:28 WARNING (MainThread) [custom_components.nea_sg_weather] Unable to parse region data from coordinator, this could be temporary
2022-01-23 13:28:54 ERROR (MainThread) [custom_components.nea_sg_weather] Timeout making GET request from https://www.nea.gov.sg/api/Weather4DayOutlook/GetData/1642886929
If you can, could you take a look at this? Thanks!

Hey,
I would suggest that you move to v1.2.0 as 1.1.3-fix pulls from a less reliable API.
As for the error messages, these might be issues from the API endpoint, and they do pop up (though rarely) every once in a while. Without detailed debug details, it will be hard to fix, and it is likely that these might not be fixable if it is an API issue.
Do monitor the warning msgs for a while and report again if you notice them persisting. In that case you might need to enable debug logs for this component for detailed error message before submitting the issue.

Have released another temporary fix under v1.2.1-fix.
This the NEA API is slightly less friendly and stable than the data.gov.sg API, so I would recommend to go back to a non-fix version of the component when the data.gov.sg service is back up.

Awesome, thanks for the speedy update at this late hour!


Improve error message if codec not found Fixes #12300

Also note that theres another implementation which should also have a similar message (AnalysisSPILoader).

@gus-asf -- looks like this one is close?  @uschindler had one more small feeback (isolate the one line that requires suppression to its own method so we don't suppress more than we need to).

Hmm, yeah I think I got busy and forgot about this. Will need to review again. Thx for the nudge.

Thanks @fsparv.

Thanks @fsparv.

heh whoops wrong browser instance ;)


How to notify data set change for FilterableListView adapter

I have to reset the FilterableListView with different values based on already selected chip inputs.
For example in my scenario I have set Friends and Groups to FilterableList, but I want that no group should come in suggestion if one is already selected.
Please suggest any solution for this.

same problem.pls help


add support for mindmap

Codecov Report
Attention: Patch coverage is 97.12644% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 98.59%. Comparing base (f985719) to head (f6f6c89).
 Consider uploading reports for the commit 4967b0b to get more accurate results

Files
Patch %
Lines

mermaid/icon.py
58.33%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #116      +/-   ##
==========================================
- Coverage   98.76%   98.59%   -0.17%
==========================================
  Files          39       43       +4
  Lines        1540     1714     +174
==========================================
+ Hits         1521     1690     +169
- Misses         19       24       +5

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


break through new Component/ComponentType approach

This is just a draft PR for potential Fleks 2.0 changes.
At the moment it is a quick breakthrough implementation of a new Component and ComponentType. That way we can remove the reflection stuff for components completely for JVM. For KMP it is no longer necessary to register a factory method for components.
Discussion continues in the "Discussions" area for Fleks ;)
example of new API:
// new Component interface; requires to override the type() function to return a ComponentType
data class Position(var x: Float = 0f, var y: Float = 0f) : Component<Position> {
   companion object : ComponentType<Position>()

   override fun type() = Position
}

// we can keep existing stuff of course like ComponentListener. They will still work as before
class PositionComponentListener : ComponentListener<Position> {
    override fun onComponentAdded(entity: Entity, component: Position) {
        println("added")
    }

    override fun onComponentRemoved(entity: Entity, component: Position) = Unit
}

// No need to inject ComponentMapper anymore; we can directly access components of an entity in a system like
// in this example via "entity[Position]"
@AllOf([Position::class])
class PositionSystem : IteratingSystem() {
    override fun onTickEntity(entity: Entity) {
        println(entity[Position])
    }
}

fun main() {
    val w = world {
        components {
            add<PositionComponentListener>()
        }

        systems {
            add<PositionSystem>()
        }
    }

    w.entity {
        // components are no longer created via reflection. They are directly created and added by the user
        it += Position(2f, 3f)
    }

    w.update(1f)
}

I will close this one and open a new one that is based on the KMP branch


ci: update GitHub Actions Linux runners to run on ubuntu-latest

The ubuntu-18.04 image has been deprecated: https://github.com/actions/runner-images/issues/6002
May explain why our Linux CI jobs started being cancelled before even starting. https://envoyproxy.slack.com/archives/C02F93EEJCE/p1661181247759129
Signed-off-by: JP Simard<EMAIL_ADDRESS>

Since (almost) all these jobs all specify a Docker container, I think it doesn't matter much what version of Linux actually hosts the Docker instance, which is why I'm specifying ubuntu-latest instead of a pinned but more recent version of Ubuntu, so this keeps working for the foreseeable future and we don't hit another surprise deprecation when the next version is dropped.


Video support.

I added video support and a very simple fix (from my testing) that prevents usernames from being stolen.
Demo: https://twitter-clone-swart-three.vercel.app/
Obviously, the code for video support is in here.
I left the original code intact in case I broke something :P

I believe there is a bug with the video audio. If you click on the video there are two audio's playing - one in the background and one in the clicked on version. Not a huge deal. Just something I noticed.  @Ketchupchh

Thank you. I will fix this.

I will review it next week @Ketchupchh hehe.
I'll be done with my final semester at uni this week.

Awesome, thanks!

Take your time man. Life comes first.

@Ketchupchh Can you check the preview? Just wanted to see if everything works. It works fine on my end. If there's no problem I can merge it.

@ccrsxx Everything works fine on desktop.

Damn, I forgot to test it on mobile. I'll take a look later, bruh I thought I could merge it yesterday.

Btw I just checked again, it's just the modal that's not coming out on mobile. I'm able to see your video tho.
Hmm, should we add a button on the video on mobile to open the modal or keep it this way? I see that Twitter doesn't open the modal when you click on a video.

Yeah, we should probably add a button to keep the Twitter theme quality.

@Ketchupchh I added a button to open the video modal on mobile, but not on desktop since you open the modal by clicking the video there.
I think it's okay to merge it now, can you check it hehe?

The video modal just infinite loads for me :(. Maybe it's just a problem on my end? :p

I doubt it? My own video also causes the modal to infinite load.

Hmm works fine on my end, even using another account that's not an admin.

Can you check the web console, and see if there's any error there?

Sorry, I should've mentioned it infinite loads on mobile. Desktop works just fine.

Works fine on my phone.

Maybe there's some ad block/extension that blocks the video? Have you tried it on another browser, to see if it works?

Yeah, I've tried it on Safari and OperaGX. Has to be something on my end since for some reason on mobile your video fails to load all together but on desktop it loads fine except the video doesn't show just the audio.

Check the web console on the desktop, is there any errors there?
I'll ask my friends to check the video tomorrow, if it's working, then it must be from your end.

This is all I get. Probably is just an issue on my end.

I think it's a Safari issue, I remember something broke from my website on Safari but not on other browsers.
Can you try using the Chrome browser on your desktop?

Okay yeah, Chrome makes everything work fine lol.

Nice, will close this PR tonight.


How to reconnect device with app

Problem
I can control the device after pairing successfully in my App, but when I kill the App and restart it, I'll lose the connection between my Matter device with the App. To keep the device  connectied all the time in the App,  could you tell me the reconnection mechanism how to recennect the Matter device with my App?
Proposed Solution
I guess before sending commands to the App, we should try to reconnect the matter device, is it correct?

See https://github.com/project-chip/connectedhomeip/issues/17298 which once resolved will fix this.

Issue Review: We believe this is complete in the current SDK, can you please verify, and re-open if this is still an issue?


collect language detection metrics in datadog_cluster_agent integration

What does this PR do?

Collect language detection metrics in datadog cluster agent integration.
Motivation

Collect language detection metrics in datadog cluster agent integration.
Additional Notes

Review checklist (to be filled by reviewers)

[x] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[x] Changelog entries must be created for modifications to shipped code
[x] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

/merge


[FRONT] Prometheus rules : removed screen split when entering edition mode

Removed screen split when entering edition mode
/!\ preliminary step for prom rule edition which is still not implemented

Codecov Report

Merging #56 (d6d8cdd) into master (27b1594) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #56   +/-   ##
=======================================
  Coverage   13.21%   13.21%
=======================================
  Files          54       54
  Lines        2006     2006
=======================================
  Hits          265      265
  Misses       1692     1692
  Partials       49       49

Continue to review full report at Codecov.

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

also we can also remove the button used to split the screen here: https://github.com/perses/perses/blob/master/internal/api/front/perses/src/app/project/prometheusrule/prometheusrule-list/prometheusrule-list.component.html#L29-L36
and the associated typescript code


API key

Hi, I'm trying to trigger an incident using the API. But I can't figure out how to get the API key. Is it created automatically with the creation of an application or how does it work? Was also unable to create a new application from UI. Is this something that has to be done via the config file?

API keys are created automatically when an application is created. You can make an application by logging in as an admin user and clicking the "Create application" button in the "Applications" tab. This should take you to a page where the API key is visible.

Thanks that worked! How is the user management for Iris done?

server:
    host: <IP_ADDRESS>
    port: 16649
    disable_auth: True

With this config you should be able to login with any user.
If disable_auth: False, then you can configure oncall to manage users. Here's an example of LDAP config for oncall:
  user_sync:
    module: 'oncall.user_sync.ldap_sync'
  ldap_sync:
    url: 'ldaps://my-ldap-server'
    base: 'dc=company,dc=net'
    user: 'cn=binduser,ou=services,dc=company,dc=net'
    password: 'xxxxx'
    cert_path: '/etc/ldap_cert.pem'
    query: '(uid=*)'
    attrs:
      username: 'uid'
      full_name: 'cn'
      mail: 'mail'
      mobile: 'mobile'
    image_url: 'https://image.example.com/api/%s/picture'

Then you can run something like:
/home/oncall/env/bin/oncall-user-sync /home/oncall/config/config.yaml
/home/iris/env/bin/iris-sync-targets /home/iris/config/config.yaml

Which will import the users from ldap into iris. Haven't tried doing anything with users local to iris yet.

Disclaimer: I'm probably missing half of what's going on. Still figuring this out myself.

Thanks for the info. Disable_auth could probably be one of the issues, will do a clean install and try it

Oh, and if you disable_auth: False, you might also run into an issue where if you're running iris through http, cookies won't be set correctly when logging in and you'll be redirected to the login page (after you just logged in). Either run iris through https or change the code. Can't remember where. Grep for cookie and set the secure value to false.

I'll give a plug for https://github.com/jrgp/iris-admin and https://github.com/dwang159/oncall-admin for user management in Iris/Oncall as well, if you don't want to deal with setting up LDAP. wleese's steps for setting up LDAP users for Oncall and Iris looks good to me. We're a little short staffed on writing docs for setup, but I'll try to get something going. Really appreciate the patience you guys have had with setting it all up; thanks for being the guinea pigs.


Ranking Performance Monster Ticket

Problem
Telos cannot handle more than 100 table rows of operations safely
Existing algorithm failing on mainnet because we have > 1000 accounts.
To Do

[x] Solve reputation score
[x] Solve community building score
[x] Solve planted score
[x] Solve transaction points
[x] Solve transaction score
[x] Solve contribution score

Solution 1:
Incremental calculations of everything.
There are several sub-considerations in this - telos multi index tables, the only data structure in telos, have linear performance for many table operations. Example: table size.
So we have to be really creative with how we calculate rank on a large numbers of items

Table Size is kept in a separate k-v store for 3 tables in harvest contract, and 2 tables in accounts contract
We want to delete entries with 0 values from all tables - this way we don't need to keep a separate counter for "number of items > 1". Only items with > 0 points should be counted.
We cannot delete users - we should have a separate reputations table, rather than storing reputation in user table. This is a cleaner solution.
Add separate reputation table, keep reputation there as well as in user accounts for now - later remove from user accounts.

Increment algorithm:

kv-store table stores current index, and current table, then we can iterate
algorithm starts at index 0 at current table, goes to end index, then starts at index 0 on the next table, etc. We have 6 tables to go over total, including the contribution score table.
we can store not the index (as to get to an index we would have to linearly increment) but we store the next value instead, the continue on from there with lower_bound(valie) which is log N performance
buckets will not be precise because values might / will change while we are calculating.

caveat:
calculating transaction points is extremely expensive and in itself, calculating these for 1 account could exceed the time limit.
We need to make this calculation much more efficient. I think we should do this each time a transaction is recorded, and collapse all transactions into days.
New transactions table, scoped by username:
score, day,
 of users transacted with, # of times so we can limit users to 26 times and not count the rest.
Solution 2:
Incremental calculations of bucket limits, on demand calculation of user bucket
This is potentially much better as users can have their scores calculated on demand
Just like in Solution 1, there would be a crawler going over all scores continuously on each table, and populating a table of buckets
Then each user can use this bucket table to figure out their score.

Planted score update:

[ ] Planted table in harvest will need to be expanded with a new field for rank - this is a data migration task that will take probably half a day to do without interrupting service.

[ ] New ranking method will be the same we already use for reputation and community building

[ ] Adding computations to scheduler again so they run automatically

Done


Cannot use Mmanos\Billing\Facades\Billing as Billing because the name is already in use

I'm getting this fatal error when trying to charge a customer. Here's the code where it's called:
$user = Auth::user();

try {
  $user->billing()->withCardToken(Input::get('stripeToken'))->create([
    'email' => $user->email,
  ]);

  $user->charges()->create(19999);
}
catch (Exception $e) {
  return Redirect::back()->withErrors([
    $e->getMessage(),
  ]);
}

If I made these changes in Charges.php everything works just fine:
use Mmanos\Billing\Facades\Billing as BillingFacade;
[...]
$gateway_charge = BillingFacade::charge(...

Do I have something misconfigured, or am I running into a weird edge case?

It looks like the error happens only in CustomerBillableTrait since there's already a Billing class in the Mmanos\Billing\CustomerBillableTrait namespace.


Support composite gravities

Instead of requiring gravity to be set as just a simple value, e.g.
Gravity.BOTTOM, allow composite ones which are based on TOP, BOTTOM and
CENTER such as BOTTOM | CENTER_HORIZONTAL. This is achieved by a bitwise
check.

The last two are to demonstrate the expected behaviour I'd like. Reverting https://github.com/Kisty/dialogplus/commit/fea1e322aa75fe39eca47bd6d383e5a7eef0b974 makes it fill up the width.

Part of #79

I've managed to fix the PR and seems safe to merge. Reckon you could do that, @orhanobut?

@Kisty sorry for a very long reply, wasn't available for a long time.
Currently there is already a method to define content width. This should do the trick.
.setContentWidth(ViewGroup.LayoutParams.WRAP_CONTENT)

Thanks for having a look at it. Although that is the case that you can use the .setContentWidth(ViewGroup.LayoutParams.WRAP_CONTENT) call it's not immediately obvious. I think developers might use this.
The other feature which I worked on and that I needed was the ability to use composite gravities so that you can have the Dialog in centre top. Currently, if you try to do something like that, it crashes because it can't find the right animation to play. I do a bitwise check to overcome it. Did you think it would be worth merging that? I can clean the branch so that it includes only that.

How's this now?

I've removed the wrap content code now but have left the ability to use many gravities so it doesn't crash. Would that suffice?

Hi! I made some changes to remove the wrap content but fixed the crash when using more than one gravity. Is it ready to merge?

@Kisty Hi there, let me check this out this weekend, thanks for the pull request

Afternoon, @orhanobut! Did you manage to have a look at this PR over the weekend?

@Kisty Hi, it looks good to me, but for convention maybe we can avoid using "else" word but it's not a must tbh, if you can do that I'll wait for the merge or can merge as well.

Okiedokes. Will make that change.

LGTM, thanks


initial work, simple file templating

Purpose

Automatically generate a basic CWL-executing DAG
STILL NEEDS: actually connecting to the ogc api properly

Proposed Changes

Adding cwl template file
Changing deploy_process method to create file if executionunit is present

Issues

unity-sds/unity-sps/issues/92

Testing

tested output of templating script in personal airflow deployment, successfully ran echo cwl task

Successfully deployed templated DAG through a modified OGC endpoint!

Going forward, the request payload schema will be just a little bit different (still abiding by https://docs.ogc.org/DRAFTS/20-044.html)

Example request (for a cwl dag):
creating
URL (POST to): http://k8s-sps-ogcproce-64159ac933-118635354.us-west-2.elb.amazonaws.com:5001/processes
Body:
{
    "executionUnit":{
        "type": "application/cwl",
        "href": "https://raw.githubusercontent.com/unity-sds/unity-sps-workflows/main/demos/echo_message.cwl"
    },
    "processDescription": {
        "id": "test_cwl_dag1",
        "title": "testing cwl dag1",
        "description": "placeholder description",
        "version": "1.0.2",
        "jobControlOptions": [
            "async-execute",
            "sync-execute"
        ],
        "inputs": [],
        "outputs": [],
        "links": [
            {
            "href": "https://processing.example.org/oapi-p/processes/EchoProcess/execution",
            "rel": "http://www.opengis.net/def/rel/ogc/1.0/execute",
            "title": "Execute endpoint"
            }
        ]
    }
}

deleting
URL (send a DELETE to): http://k8s-sps-ogcproce-64159ac933-118635354.us-west-2.elb.amazonaws.com:5001/processes/test_cwl_dag2
(no body necessary)


Creating the FHIR Proxy generates an error at the function app

Storing Redis Connection String in KeyVault...
Creating Secure FHIR Proxy Function App [sfp-proxy5791]...
ERROR: Could not find a runtime version for runtime {} with functions version {} and os {}Run 'az functionapp list-runtimes' for more details on supported runtimes.

Anything will be helpful! Thank!

Same problem here.

I am also running into this error.  Any throughts?

Deployment Issue Closing for repo archiving


The debugger command doesn't seem to work when using multiple threads at the same time.

Your environment

ruby -v: 3.2.2
rdbg -v: 1.8.0

Describe the bug
Doesn't seem to work well with multithreaded debugger statements. The program does not print out expected local variables and seems to hang forever after last continue statement.
To Reproduce
Given the following ruby code:
require "debug"

thr1 = Thread.new {
  hello = "world"

  debugger

  puts hello
}

thr2 = Thread.new {
  hello = "hello"

  debugger

  puts hello
}

thr1.join
thr2.join

The first time I reach a breakpoint it shows me in the hello = "hello" block. When I type out hello though it than jumps me to hello = "world" block. After typing out hello a second time, it prints out the string "hello" instead of "world". If I type continue it hangs forever after that. If I run the program without the debugger statements it doesn't hang.
Expected behavior
I would expect debugger to be somewhat threadsafe. Obviously debugger cannot necessarily control the order of these two debuggers. The second debug statement may execute first but I'd expect that the breakpoint would match the local variable and I'd expect typing continue would than make the program keep running and go to the next debugger statement or terminate the program if no more debugger statements.

Thank you. This code is great to reproduce the issue.
Honestly speaking current thread management is not strict and this is a time to consider strict management...

I couldn't complete to solve this issue so I'll try it on 1.9.1

I can't continue after hitting a breakpoint with Puma and I believe this might be the same issue. Just curious.. Is it a known issue that debug can't be used with puma due to this? Or am I doing something else wrong? (I've tried and failed on several rails projects. Changing to webrick makes it work)


Make biohazardous barricades take CMRods not SS14 Rods

About the PR

Made biohazardous barricades actually craftable.
Why / Balance

Boilers are really messing up cades, and this should be an option.
Technical details

Just want to give the heads up that this is a web edit because I am unable to build SS14 at the moment, but this should be a relatively quick fix. Just changed the SS14 MetalRod to CMRodMetal
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

fix: Biohazardous barricades are now constructible with six metal rods.

Also @DrSmugleaf does the caustic resistance on the biohazardous barricades actually reduce boiler damage? I haven't looked into that unfortunately.

Also @DrSmugleaf does the caustic resistance on the biohazardous barricades actually reduce boiler damage? I haven't looked into that unfortunately.

It should if it's just changing burn resistances


Refactoring to octomock

Refactoring to use octomock lib.  Will move this to regular PR once I get the final action refactored as well and have all three actions using the same version of octomock

This ports the tests to use the mocking library so running the unit tests should confirm that it works.
We have migrated most of the other actions over to it, so this PR just makes it so it matches some of the others.
Still up for debate whether this is a step in the right direction...  If you want to glance at octomock and give some feedback that'd be great too :)  I am not a javascript person so there is probably a lot of low hanging fruit.


Improving the Windows installation process

Since both curl and python aren't pre-installed on Windows, curl -sSf https://raw.githubusercontent.com/denoland/deno_install/master/install.py | python is not a very convenient way to install deno.
Creating a proper .msi or .exe installer is probably too much work at the moment. However, adding deno to a Windows package manager is fairly easy.
As far as I know, there are two major package managers for Windows: Chocolatey and Scoop.
Since scoop is more actively maintained than choco, arguably easier to use, and focused on developer tools, I decided to make a PR for adding deno to scoop. Since it has already been merged, deno can be installed with scoop install deno on Windows.
I think in future, there should be multiple options for users to install deno on Windows:

[ ] Download deno.exe file
[X] Download deno.zip file
[ ] Download deno-setup.exe file
[x] Install from powershell.exe: iex (iwr https://raw.githubusercontent.com/denoland/deno_install/master/install.ps1) (see: denoland/deno_install/#4)
[x] Install from pwsh.exe: iex (iwr https://raw.githubusercontent.com/denoland/deno_install/master/install.ps1)
[X] Install via Scoop: scoop install deno (see: lukesampson/scoop/#2570)
[ ] Install via Chocolatey: choco install deno
[ ] Install via npm: npm install deno

Can we have deno.land/deno.ts?

I think this issue can be closed for now. The Windows PowerShell installer is on par with the python installer. And in regards to package managers, I think Windows is ahead of MacOS with scoop install deno already working compared to brew install deno which is yet to be implemented.

@ry I think scoop is the modern package manager for Windows. Might be that I live in a bubble, but I don't know anyone who's still using choco... It's basically Homebrew for Windows - not least because it is inspired by Homebrew. :)
I think a reference would be great. Even though I wrote the inital version of the PowerShell installer, I'm already using scoop to manage my deno installation.

Will Chocolatey installation come in the future? I usually use Chocolatey.

Will Chocolatey installation come in the future?
I usually use Chocolatey on my laptop.

At the moment only scoop is available. If someone want to work on this it's totally welcome but it's not in the scope of the development.


Assign BEP044 to "Stimuli BEP"

Attn @neuromechanist  @dorahermes @VisLab @smakeig

will probably have to open a similar PR against main

will probably have to open a similar PR against main

oh, it isn't automagically somehow populated from this metadata?

Na. The main branch was created from a completely different repo. So changes on the old website have to be mirrored on the new one.
Hence why we have been avoiding changing the old one unless 100% necessary.


The word "goto" breaks YUI Compressor

FYI: YUI Compressor (2.4.8 - including, but not limited to) breaks when using reserved words as property names. The word goto is a "future reserved keywords in older standards", and is therefore not allowed by YUI Compressor. Be on the safe side and avoid using reserved (or previously reserved) words.
Thanks for making this awesom´ library!
Reference: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Lexical_grammar

Unfortunately, this is part of the public API and cannot be changed at this point.
Also, people still use YU Compressor?!

I think a workaround could be to use bracket notation instead of dot notation for the goto property. And you're right, "people" don't really use YUI Compressor nowadays. However, it's sometimes bundled with enterprise CMSs, for instance. My current client uses Adobe Experience Manager, which (unfortunately) uses YUI Compressor under the hood.

Haha, YUI Compressor seems to have some serious issues with getters and setters...
/* Source */
var foo = {
    get bar() {
        return 'foobar';
    }
}

/* YUI Compressor output */
var foo={get barfunction(){return"foobar"}};

Yes. It renames bar to barfunction. Why it does this is beyond me. :(

Seems issue with getters and setters was fixed in the latest version of YUI Compressor (2.4.8).

LOL, wow. Why not use UglifyJS which is way more updated and doesn't have such issues?

Don't shoot the messenger! Tell that to Adobe... It's really a pain when an enterprise CMSs bundle and enforce tools and libraries that can't be easily replaced. AEM also bundles (and depends on) a pretty outdated version of jQuery, which can also be quite troublesome. I don't know what they are thinking, since web technologies evolve much faster than their CMS.

 I feel ya. :(


add tags

I have been able to add tags upon starting an entry using the 'tags' property.
The problem i have run into is that the select box might not contain these tags and certainly does not have them marked as selected.
So when the user clicks teh 'done' button in the popup it sends an update to toggl with no tags, effectively deleting the tags.
Does anybody have a way to handle this?
Step 1: add the newly added tags to TogglButton.$user.tagMap in background.js
Step 2: mark them selected
Step 3: submit them as selected when clicking 'done' -> updateTimeEnrty
Alex

Since the user really didn't need to add any other tags, i just created a way to hide the popup at all times.
Did this by adding a wrapper around the popup:
$editForm:
  '
' +
    '
' +

and then adding a stylesheet in the document to always hide this:
document.addStyle = function (str, hoo, med) {
      var el = document.createElement('style');
      el.type = "text/css";
      el.media = med || 'screen';
      if (hoo) el.title = hoo;
      if (el.styleSheet) el.styleSheet.cssText = str;//IE only
      else el.appendChild(document.createTextNode(str));
      return document.getElementsByTagName('head')[0].appendChild(el);
    };

    document.addStyle('#toggl-button-edit-form-wrapper {display: none}');

if you're interested I can make a PR for this.

Currently there is no functionality to add new tags from the edit popup. Also there is a setting that hides the hides the edit popup. It's called Show post-start popup and unchecking that will not show any popup.

You can open settings from the extension icon menu.

@IndrekV  It is possible to add new tags on the fly by just adding them to in the configuration object under 'tags', any new tags are automatically added.
I know about the settings menu, but I wanted to make it as hassle free as possbile.
Now I just created a custom crx that is distributed in the company, the sales manager is satisfied!
Thanks

@cobrowserAlex I'm not exactly sure what you mean by adding tags. Do you mean adding tags from the edit popup or do you mean something else.

I created a toggle button for ZohoCRM and needed to add tags based on lead source and Lead or Deal name. All of which initially aren't in Toggl.
So when creating the button i added tags like this, and it works well:
    link = togglbutton.createTimerLink({
      className: 'zoho',
      description: description,
      projectName: 'Sales',
      tags: tags
    });

ok, thanks for the clarification.

As the button is quite specific for our purpose I won't be making a pull request, unless there is strong demand...


maintenance(lint): adher to new linting rules

@nicojs I guess prettier won't allow you to make it in one line since our limit is 150letters per row(?)

Yes, strange how the rules suddenly got stricter. I guess bugfixing in their part, and since we're not locking our versions, we get the latest and greatest each time.
Run this locally to reproduce:: rm -rf node_modules packages/*/node_modules && npm i

Whut???? No file changes? 😲

Abandon 🙈

yea nico, there were some strange caches in package.json file. I had to reinstall whole stryker coz i had some problems with doubling references. happens


Disable ligatures

Hi,
Although I like ligatures for coding with fonts like Fira Code, some other fonts (Space Mono) have ligatures that I would not like to see in the code (e.g. fi).
Is it possible to disable ligatures as a setting somewhere?
Many thanks,
Nikos

Hi @corelon ,
Thanks for trying out Oni and for the feedback!
There is no setting that exists today, ligatures are always on, but I believe this should be relatively straightforward - the ligatures are controlled by a CSS property font-variant-ligatures. So we'd just need to add a config that controls this property:
https://developer.mozilla.org/en-US/docs/Web/CSS/font-variant-ligatures

Hi @extr0py ,
Thanks for the reply.
If I am not mistaken both Atom and VS Code offer that option.
For fonts like Space Mono which are very nice for coding, but have non-dev related ligatures ("fi" I mentioned in the original post) it would be good to be able to disable this, if it does not require too much effort.

Definitely! Thanks for your comments on the PR. I'll reopen this to track more fine-grained control of the ligature support as you suggested.

I experimented and dug around a bit, and it looks like VSCode implemented this a bit differently:
https://github.com/Microsoft/vscode/issues/192
.monaco-editor.enable-ligatures {
    -webkit-font-feature-settings: "liga" on, "calt" on;
    font-feature-settings: "liga" on, "calt" on;
}

I was hoping that this would enable a bit of finer-grained control over the behavior of the ligatures, to exclude some and not others, but it still causes the fi ligature to be rendered via Space Mono. So I tried out with Space Mono, and it doesn't help the situation there.
Looking at Space Mono, the best bet is to disable ligatures for that font via editor.fontLigatures = false. It seems like it is mostly a stylistic font as opposed to a development font.
I'm also tracking some extensibility to this with #412 - having CSS theming support for Oni plugins. With that, a plugin could do some more fine-grained control over text rendering too.
I'll close this for now as I think the best bet for Space Mono is to use editor.fontLigatures false, but feel free to reopen if there is something I missed.


How to use Jest for Node.js projects

TLTR: since Node.js 4 is still supported many companies use it and many Node.js projects (including PostCSS, Autoprefixer, Browserslist) must support it. What is a solution for them?

Node.js 4 is not like Node.js 0.12. It is still supported and it is OK to still use it in production until April 2018.
If you are developing popular Node.js library you must support all current Node.js version.
If you support some version you should run tests for this Node.js version of Travis CI. It is the only way to be sure that you didn’t brake Node.js 4 support accidentally.

Of course, it is OK to run the only Node.js 9 on Travis CI for client-side projects like Jest. But Node.js projects is different.
What is your recommendation for the big Node.js project? Stick on Jest 21 until August 2018?

We do not have the time to support a version of Jest that is two versions behind the current LTS. Also, this decision was made for us by the jsdom team. There is no recommendation from us beyond using older versions of Jest. That said, if you'd like to make the latest version of Jest work with Node 4 and you are committed to supporting it until mid 2018, we are happy to accept pull requests and re-enable CI on Node 4.

That said, if you'd like to make the latest version of Jest work with Node 4 and you are committed to supporting it until mid 2018, we are happy to accept pull requests and re-enable CI on Node 4.

What was the problem of supporting Node.js 4? I can’t test it since I get yarn warning right now. It is about some very useful Node.js API or just using let without "use strict"?

Also, Node.js 4 is current LTS ;). All 4, 6 and 8 are LTS.

You'll have to ask the folks working on jsdom about that. Jest frequently segfaults on Node 4 which was fixed in later versions of Node. I generally don't recommend using Node 4 (for Jest) and would encourage you to push your company to upgrade to a later version which is about 2x faster, more stable and likely more secure.
Please note that it doesn't matter to this project which versions of Node.js are LTS. At Facebook, we only use the latest LTS release or unstable version. I'd even go so far as to drop support for Node 6 but I understand that's taking it a bit far for some folks, and it isn't causing major issues for us right now (like, segfaults).

@ai See their changelog regarding v10: https://github.com/tmpvar/jsdom/blob/master/Changelog.md#1000
Also, JSDOM v9 started becoming a burden, because of all sort of bugs, which needed to be patched in setup files.

I generally don't recommend using Node 4 (for Jest) and would encourage you to push your company to upgrade to a later version which is about 2x faster, more stable and likely more secure.

We don’t use Node.js 4 too. But PostCSS has 25 M downloads, so we have many Node.js 4 users, which I must support until Node.js 4 is still current LTS.

The problem that PostCSS doesn’t even use JSDOM :(.
Can I install Jest 22 without JSDOM to use it for pure JS tests?

What was the problem of supporting Node.js 4?

Right now we've done nothing to remove node 4 support beyond upgrading to the latest version of jsom in jest-environment-jsdom (which fixes bugs and add features everybody has to polyfill for older versions). There are still TODOs in the code marking removal when node 4 is dropped.
We might have some issues as we haven't run node 4 on CI for a couple of months, but nothing major I wager.

I can’t test it since I get yarn warning right now.

YARN_IGNORE_ENGINES=true yarn.

It is about some very useful Node.js API or just using let without "use strict"?

Nope. That's more relevant when we wanna drop node 6 as we use a ton of async-await in the code base.

All 4, 6 and 8 are LTS.

True, only the latter two are active LTS though, which I think counts for something.

Can I install Jest 22 without JSDOM to use it for pure JS tests?

If you use a lock file and ignore the warning from yarn about engines there should be no issue upgrading

You should be good to do it. Just use "testEnvironment": "node" in Jest config.

Awesome! Thanks for this advice. I run Jest on Node.js 4, but have this issue right now (destructor syntax?):
SyntaxError: Unexpected token ...
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:373:25)
    at Object.Module._extensions..js (module.js:416:10)
    at Module.load (module.js:343:32)
    at Function.Module._load (module.js:300:12)
    at Module.require (module.js:353:17)
    at require (internal/module.js:12:17)
    at _load_normalize (/home/travis/build/postcss/postcss/node_modules/jest-config/build/index.js:14:73)
    at Object.<anonymous> (/home/travis/build/postcss/postcss/node_modules/jest-config/build/index.js:173:30)
    at Module._compile (module.js:409:26)

What do you think about some unofficial Node.js 4 support. You don’t use Node.js 4+ syntax, but this support is not official and may require some hacks?

What do you think about some unofficial Node.js 4 support. You don’t use Node.js 4+ syntax, but this support is not official and may require some hacks?

Something like that 🙂
PR welcome for you specific error here: https://github.com/facebook/jest/blob/d1845116ebe818e6d945d50ad4857752758970ee/packages/jest-config/src/normalize.js#L284-L289. Just concat instead of spread syntax should work out fine

I will send PR tomorrow :). And try to grep other ... symbols.

I sent PR with ... fix https://github.com/facebook/jest/pull/5134

I fixed all ... in Jest repo, but now has this issue:
 FAIL  test/container.test.js
  ● Test suite failed to run

    SyntaxError: Unexpected token ...

      at _load_jestJasmine (node_modules/jest-runner/build/run_test.js:146:355)

I look into jest-runner and there is no ... there. Seems like it is in some dependency and stacktrace is broken. Maybe somebody have a idea how find this dependency?

https://github.com/facebook/jest/blob/1521c516c2cae8f4bfe41605640f273f9c17fb87/packages/jest-jasmine2/src/index.js#L53

@SimenB my bad :sweat: PR was sent https://github.com/facebook/jest/pull/5142

Done. Now Jest works for Node.js 4 too. Thanks to everyone.

Thanks for pushing on this @ai!

Is there something specific I need to do to run jest under node 4 environments?

@shawnmclean Node 4 gets it's end of life in 6 days, better use that time to migrate to Node 8 (latest LTS version) than figuring out your tools to work with soon unmaintained language.

@thymikee thats my issue now. We're migrating to Node 8, but during that time, we want to maintain backwards compatibility with 4, hence having tests running under Node 4 for the time being.

Jest 22 should work fine on node 4 as long as yiu use node env and not jsdom. Use jest 21 (or a custom jsdom env) if you need it


disable scroll beyond bounds (in osx)

not really a bug, but not a feature i want here

I looked into this, but was not able to find an easy API to disable it. It's really more of an OSX thing. There are CSS hacks to disable the elastic scroll, but they do not work well when resizing the window. Closing for now as it's not a big deal.


use explicit and compiler-friendly type conversions

In all of those cases, I'm seeing type conversion errors from static analysis. The "extra-tours" in this PR are clearing up typing and type conversions for the compiler

Merged into other PR https://github.com/gitgitgadget/gitgitgadget/pull/823


page fragment loaders

This adds a server option to fenced code blocks. When present, the code is run on the server when the page is rendered in the same fashion as a data or page loader; the interpreter is passed the source of the fenced code block as stdin. The stdout of a page fragment loader must be HTML, which then replaces the comment placeholder (e.g., <!--:85902a01:-->).

Marking this as a draft because I haven’t thought through how caching should work. Currently page fragment loaders run every time the page is loaded, but I think they should only run when the page is edited, similar to page and data loaders. That means we’ll need to cache the output of page fragment loaders (or more likely the HTML of the entire page).
I also think the implementation could be cleaned up slightly, for example by moving the rendering of page fragments into a function, and by exposing the normalized interpreters on the config instead of on loaders… or maybe the loaders object should have a method for evaluating fragments.
Fixes #234. Probably also fixes #145, which we should consider duplicate.

It's immensely satisfying to run DuckDB (from sh) and bake-in some statistics without having to write an external data loader. Makes it easier to stay in your train of thought.
A few observations as I'm testing this:

How can we pass the page params to the fragment loader?

Crashes in the code block are fatal (we should instead surface the error).

It would be super nice to give a name to the output and make it available as a reactive variable, similar to what we do with sql blocks with id= (except here the value would be the DOM node). If I add an id manually then use querySelector(#id) I can work something in build, but it's a bit tedious and doesn't work in preview.

I think it would make sense to cache the fragment outputs aggressively, maybe based on a hash of the code (& interpreter). It would be faster to play with, and closer to the build (where date is the build date, not the current date).

I'm not always seeing hot-reloading; related, or not, with the following content I'm getting a loop of page reloads (which sometimes stops after a while):

```js server echo
console.log(Math.random());
```

showing ↓ { type: 'reload' } messages. Same with new Date(). (I don't understand how this can even be possible.)

I added support for params to page fragment loaders in f5b5a13. This required passing a couple additional arguments to the interpreter so that you can run commands from stdin while passing additional arguments to the script. In the case of sh, it looks like:
sh -s -- --theme=air

Whereas for Node:
node --no-warnings=ExperimentalWarning -- --theme=air

We should allow the config to define the arguments used when the interpreter is invoked, so we’ll probably need to bifurcate the interpreters option to distinguish between the interpreter being invoked with a file versus being invoked with stdin. Or maybe we should promote the page fragment source to a temporary file and use that, rather than using stdin. That’s one less thing for users to worry about.

Crashes in the code block are fatal (we should instead surface the error).

Maybe… I think the build should crash if a page fragment loader errors, in the same fashion as a page loader.

It would be super nice to give a name to the output and make it available as a reactive variable

I’ve been thinking about making HTML the base representation rather than Markdown. So, Markdown pages would first compile to HTML, but a flavor of HTML (”Observable HTML” say) that allows us to define <script type="observable">. And Observable HTML would be compiled into vanilla HTML as today. I originally thought page fragment loaders could emit Markdown rather than HTML, which could do something similar; but that would also allow page fragment loaders to emit page fragment loaders recursively. Whereas if HTML is the base representation, page fragment loaders are only expressible in Markdown and thus cannot be recursive.
Anyway, my goal with this PR is to introduce a minimal abstraction that serves as a foundation — to not add more than we need. We can add more in the future.
A pattern that I like from oss-analytics is defining JSON scripts:
https://github.com/observablehq/oss-analytics/blob/937a8af225c839c25b48ebfba3953a49a7774770/src/%40[scope]/[name].md.js#L305-L308
These are then exposed as top-level variables using JSON.parse:
https://github.com/observablehq/oss-analytics/blob/937a8af225c839c25b48ebfba3953a49a7774770/src/%40[scope]/[name].md.js#L95-L98

I think it would make sense to cache the fragment outputs aggressively, maybe based on a hash of the code

Maybe. I was thinking it should be based on the modification time of the file like we do with page & data loaders. That would mean that all page fragment loaders re-run when you edit the Markdown, but if we use hashes, it’s harder to force a re-evaluation. (And we don’t trace transitive dependencies of loaders, so sometimes they need a nudge.) That said, we use hashing for client-side code blocks, so it makes sense to have similar behavior for server-side code blocks, so you’re probably right!

I'm not always seeing hot-reloading; related, or not, with the following content I'm getting a loop of page reloads

The loop I expect if the loader is nondeterministic, and maybe is related to the other problem. It’s because we don’t have any caching. When the client connects, the first thing it does is send the hash of the page, and if it doesn’t match what the preview server’s hash, the client reloads the page. If you have a nondeterministic page fragment loader, since there is no caching, the hash will never be consistent and will reload continuously. Let’s implement caching before investigating this further.


"refresh" thinks remote QueueIamPolicy resource doesn't exist, but I think it does

What happened?
I have a GCP Cloud Tasks queue resource and an associated IAM policy. These are managed in Pulumi.
function createQueue(args: QueueArgs): gcp.cloudtasks.Queue {
    const {name, location, maxConcurrentDispatches} = args;
    const queue = new gcp.cloudtasks.Queue(name, {
        location: location,
        rateLimits: {
            maxConcurrentDispatches,
        },
    });

    const members = [
        serviceAccounts.backend,
        serviceAccounts.shellServer,
    ].map(member => mapIamPrincipal(member));

    new googleNative.cloudtasks.v2.QueueIamPolicy(name, {
        queueId: queue.name,
        bindings: [
            {role: 'roles/cloudtasks.enqueuer', members},
        ],
    }, {parent: queue});

    return queue;
}

Note: I'm using the gcp provider for the Queue (and for almost everything in my stack) but that provider doesn't support QueueIamPolicy, so I had to use the google-native provider for just that resource type (https://github.com/pulumi/pulumi-gcp/issues/747).
The QueueIamPolicy current exists in my Pulumi state. Here's an excerpt from pulumi stack export --stack=anrok/staging:
{
    "urn": "urn:pulumi:staging::anrok::gcp:cloudtasks/queue:Queue$google-native:cloudtasks/v2:QueueIamPolicy::background-jobs-default",
    "custom": true,
    "id": "v2/projects/anrok-main-staging/locations/us-central1/queues/background-jobs-default-7409822:getIamPolicy",
    "type": "google-native:cloudtasks/v2:QueueIamPolicy",
    "inputs": {
        "bindings": [
            {
                "members": [
<EMAIL_ADDRESS><EMAIL_ADDRESS>                ],
                "role": "roles/cloudtasks.enqueuer"
            }
        ],
        "location": "us-central1",
        "project": "anrok-main-staging",
        "queueId": "background-jobs-default-7409822"
    },

And the policy exists according to gcloud:
$ gcloud tasks queues get-iam-policy --project=anrok-main-staging --location=us-central1 background-jobs-default-7409822
bindings:
- members:
  -<EMAIL_ADDRESS>  -<EMAIL_ADDRESS>  role: roles/cloudtasks.enqueuer
etag: BwXva_AV-SE=
version: 1

But when I do pulumi refresh, Pulumi wants to delete it from the Pulumi state:
$ pulumi refresh --stack=anrok/staging \
    --target 'urn:pulumi:staging::anrok::gcp:cloudtasks/queue:Queue::background-jobs-default' \
    --target 'urn:pulumi:staging::anrok::gcp:cloudtasks/queue:Queue$google-native:cloudtasks/v2:QueueIamPolicy::background-jobs-default'
Previewing refresh (anrok/staging)

View Live: https://app.pulumi.com/anrok/anrok/staging/previews/7e7fc425-a3a9-4591-8a8c-4a6a78a60669

     Type                                              Name                     Plan
     pulumi:pulumi:Stack                               anrok-staging
     └─ gcp:cloudtasks:Queue                           background-jobs-default
 -      └─ google-native:cloudtasks/v2:QueueIamPolicy  background-jobs-default  delete

Resources:
    - 1 to delete
    1 unchanged

Expected Behavior
No changes from pulumi refresh
Steps to reproduce
See above.
Output of pulumi about
$ pulumi about
CLI
Version      3.53.1
Go Version   go1.19.5
Go Compiler  gc

Plugins
NAME    VERSION
nodejs  unknown

Host
OS       darwin
Version  12.6.1
Arch     x86_64

This project is written in nodejs: executable='/REDACTED/.volta/bin/node' version='v18.13.0'

Backend
Name           pulumi.com
URL            https://app.pulumi.com/kannan-anrok
User           kannan-anrok
Organizations  kannan-anrok, anrok

Pulumi locates its logs in /var/folders/z9/2stfl58s23l38j8gbwwns0m80000gn/T/ by default
warning: Failed to get information about the Pulumi program's dependencies: could not find either /REDACTED/anrok/pulumi/main/yarn.lock or /REDACTED/anrok/pulumi/main/package-lock.json
warning: Failed to get information about the current stack: No current stack

Note: I'm using Yarn workspaces, so the "yarn.lock" is in a parent folder. I can't figure out how to get pulumi about to look there, so I just ran yarn info to get my dependency versions:
$ yarn info --name-only @pulumi/gcp @pulumi/google-native @pulumi/pulumi
├─ @pulumi/gcp@npm:6.48.0
├─ @pulumi/google-native@npm:0.27.0
└─ @pulumi/pulumi@npm:3.53.1

Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

That does seem odd 🤔 Let's see if we can unravel what's happening here.
What does pulumi preview --diff output?

pulumi preview --diff (and pulumi up) show no changes ("209 unchanged"). But I think my Pulumi code is in sync with my Pulumi state, so that is expected, right?

I think my Pulumi code is in sync with my Pulumi state, so that is expected, right?

I'm not sure what to expect :-) I'll try to recreate the situation from scratch -- can you supply the implementation of mapIamPrincipal (or if it's unimportant, just describe roughly what it does)?

export function mapIamPrincipal(principal: Principal): pulumi.Output<string> {
    if (principal instanceof gcp.serviceaccount.Account) {
        return pulumi.interpolate`serviceAccount:${principal.email}`;
    }
    return pulumi.output(principal);
}


Feat/civ 2735 document and note

Screens for judge upload evidence
[### JIRA link (if applicable) ###
https://tools.hmcts.net/jira/browse/CIV-2735
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ x] No

These are PO approved and can be merged. They already made it past review. No need to further review.


Releasing  0.2.2

Do you know when 0.2.2 is going to land on hackage ?
Just asking because it prevents our (ghc-8.4.1) travis-test to succeed.
Cheers,

I will do it this afternoon, thanks for the heads up.

It is uploaded: https://hackage.haskell.org/package/protolude-0.2.2
Cheers


Exception guarantees

Javadocs for dynamic buffers and other appropriate types should have prescribed exception guarantees.

flat_buffer and multi_buffer have exception specifications now


Convert caffe to tf

Platform (like ubuntu 16.04/win10): ubuntu 16.04
Python version: 2.7
Source framework with version (like Tensorflow 1.4.1 with GPU): caffe GPU
Destination framework with version (like CNTK 2.3 with GPU): tf GPU
Pre-trained model path (webpath or webdisk path): https://drive.google.com/file/d/15Ah6Z3dJlTP3ZkLdML-kKYI6miGipu3w/view?usp=sharing
Running scripts:  mmtoir -f caffe -n deploy.prototxt -w realvideos_refinedet_resnet101_512x512_iter_40130.caffemodel -o smalltf_ir --inputShape 512,512,3
I got the ERROR: TypeError: object of type 'NoneType' has no len()
Thank you very much in advance!

Hi @HongyiDuanmu26 , please put the whole error traceback and caffemodel file here.


Improve pkg/searchhandler.go ServeHTTP error handling

The error handling in ServeHTTP should be refactored.
Currently there is a function that handles errors
func (h *searchHandler) handleError(err error, w http.ResponseWriter) {
	if err != nil {
		w.Header().Set("Content-Type", "text/plain")
		w.WriteHeader(404)
		fmt.Fprintf(w, "Error: %v\n", err)
	}
}

This gets called once, and its in ServeHTTP
key, err := surt.SsurtString(uri, true)
if err != nil {
  h.handleError(err, w)
  return
}

The 404 return status seems to be a bit eager as the error can be caused by uri parsing causing an error. There are also other cases where errors are not dealt with properly.

A review was done on PR 34 in the gowarc repo. I will push changes directly to master here ASAP


Paths for themes installed via Package Control

Instructions about how to specify paths for themes installed via Package Control. Related to Issue #10.

Thanks for doing this!


Do you support SPM officially?

README.md says 'ReactorKit officially supports CocoaPods only.'
But It seems like you guys support SPM as well.
How about writing a SPM installation guide in README.md?

Hi @lbr0452000
I added the spm guide to README.md
Thank you for your suggestion.


Allow Other Mobile Platforms Easy Integration

Google should make it so Eddystone allows easy integration for other mobile platforms like Cyanogen, Tizen, Firefox OS, Ubuntu Phone.
One contextual protocol to rule them all \o/

Hey Benjamin. Anything in particular you'd recommend here? (Are there issues with the bluetooth stacks on those platforms we're not addressing?)

Hi @bkerensa — any follow up to @mashbridge's question? Would be curious to hear what you're thinking!


Testing Application Factory without existing CMDT config data

John,
In working on creating unit tests for the Application Factory, my team stumbled upon a difficulty in trying to test it. As far as I can tell, there is no easy way to give it "mock" CMDT records with which to work, in writing unit tests for the application factory code itself.
I handled the same issue when I wrote my own SObject type to SObject type binding capability that I have described to you previously. In that instance I had to make my own binding provider. When I did that, I had it pull the important mapping data from the CMDT into an @testVisible private static map  property.
    @testVisible
    private static Map<String,Map<SObjectType,SObjectType>> SObjectTypeBindingMap{
        get{
            if(SObjectTypeBindingMap == null){
                SObjectTypeBindingMap = new Map<String,Map<SObjectType,SObjectType>>();
                List<base_SObjectTypeBinding__mdt> sotBindings = [SELECT Id
                                                                    , DeveloperName
                                                                    , Purpose__c
                                                                    , BindingSObjectType__c
                                                                    , BindingSObjectType__r.QualifiedApiName
                                                                    , BoundSObjectType__c
                                                                    , BoundSObjectType__r.QualifiedApiName
                                                                    FROM base_SObjectTypeBinding__mdt];
                for(base_SObjectTypeBinding__mdt sotBinding : sotBindings){
                    if(!SObjectTypeBindingMap.containsKey(sotBinding.Purpose__c)){
                        SObjectTypeBindingMap.put(sotBinding.Purpose__c,new Map<SObjectType,SObjectType>());
                    }
                    SObjectTypeBindingMap.get(sotBinding.Purpose__c).put(sObjectTypesMap.get(sotBinding.BindingSObjectType__r.QualifiedApiName),sObjectTypesMap.get(sotBinding.BoundSObjectType__r.QualifiedApiName));
                }
            }
            return SObjectTypeBindingMap;
        }
        set;
    }

This way I could easily "mock" the configuration data for unit tests, by just setting that property.
I was thinking of modifying your other application factory classes to do something similar. Before I did that, I thought it would be a prudent to ask how you have dealt with this? I want to make sure I'm not just missing something obvious.
Thanks, as always, for your time and attention.

@torynet -- Circling back to this issue.  You mentioned translating CMDT usage into a mapping architecture.  Is that something you're willing to share with the project, and if not, I'd still enjoy seeing your work, just for my edification.  Thanks regardless.


Add README to Security Observability with eBPF folder

Signed-off-by: Bill Mulligan<EMAIL_ADDRESS>

I just combined my wording and #465 since we both had the same idea

hmm @xmulligan I merged @lizrice updates which then collided with yours. Can you folks sort it out and rebase.

I'm also happy with Liz's PR so I'll just close this one


Fix/gitlab remove tokenset and duplicate

Fixes #1006

[x]  functional review: @esthercheran

@esthercheran this contains the fix for the mutlifile push in Gitlab and the duplicate + rename issue. Let's test this before release of course; also wondering if we should provide a beta package to verify this fix because the issue is very specific.

@esthercheran can you functionally review this? I'm not aware of what to look for when reviewing this.

@LiamMartens @six7 I think we should release a beta as the Gitlab push issue is very specific to the commit message rule. I have tested the other functionality, it is working fine.

This also fixes #1006

@six7 @esthercheran checked this with the user in the figma tokens slack


Config File with Python Input Interface

Ok so my config.js from yesterday is back.
I branched off and added config.py. This opens and direct text entry window for the user which will automatically edit the config.js file with their chosen options. This will prevent any issues with having users directly editing files.
Only tested on Windows 10, no issues and I think there's enough checks to prevent improper data entry (such as a making sure they fully write "true" or "false", ensuring numbers are used where needed etc)

Alrighty, let me test and we can move this in.
My plan is to just throw config.py into our install.bat and install.sh.

sounds good, definitely give it a test, i think i thought up all the possible incorrect answer for the config setup but i may have forgotten some, also i only have windows to test on

don't worry i'll get this done by the end of the night but i'm working on a yt video atm :)

No worries I'm gonna setup my github properly so the "merge" crap goes away on my PRs

ok, YT video rendering, cloning and testing.

Can you make the recommend for use random password true?

Alright, so it worked for me, once you make the changes above I can merge it and update documentation accordingly.

alright yea i can get both those to happen, have it up in a bit

ok that wraps that up

Alright, awesome. I'll merge this, update install.bat/.sh and whatnot

great


Can't see installed icon packs

Hi Florian
I have a vault hosted online on Google Drive with several plugins installed among one of them is Icon Folder.
I installed several icon packs and even a custom one. When I close Obsidian and come back, Icon folder doesn't see the packs anymore. The Icon Pack section in Options is empty. This is weird cause the packs are there on  the Google Drive : G:\Mon Drive\Valombreuse FR.obsidian\plugins\obsidian-icon-folder\icons
The icons I set up are mostly gone. I can see some of them but not many.
My Icon Packs folder path is .obsidian\plugins\obsidian-icon-folder\icons which is correct.

When I want to change an icon with #change icon, the list is empty :/

Note : My custom icon pack "valombreuse" features 2312 icons.
Can you  please help me out ?
PogS

When you go to your Console in obsidian (View > Toggle Developer Console), do you see any errors or a message that your icon pack was loaded?

It seems like some icon or file in your icon pack might be corrupt. Because there are a lot of icons in your icon pack, it is hard to debug which icon might be corrupt. I will try to fix (and reproduce) this issue while developing v2.0.0 (#159). Thanks for reporting!

Yeah thank you :)

On my side I uninstalled the plug in and reinstalled it. I lost all my icons so i had to remake all the hundreds of entries :/
The problem is still here.
I do not find how to open the Obsidian console :/

On my side I uninstalled the plug in and reinstalled it. I lost all my icons so i had to remake all the hundreds of entries :/ The problem does not occur only for custom packs but also to downloaded fontawesome etc... packs. All are missing. The problem is still here. I still see icons I already set for notes and directories but I can't place more as the list is empty.
There is also a problem adding more icons to a pack : I select multiple icons and it fails to add them to an existant pack. It seems to happen if i select too many icons and when the pack is big. Is there a number limit into a pack ?
I do not find how to open the Obsidian console :/

There is no limit to adding icons to an icon pack. It can happen that your syncing service for your vault might cause all of these problems. We had some similar issues before with Obsidian sync.

+1 I have the same issue, all icon packs disappear after a restart, when I add them again there are a lot of "renamed" notifications. I don't use any sync services

This should be fixed in 2.0.0.

Cool :) Thank you
De: "Florian Woelki" @.>
À: "FlorianWoelki/obsidian-icon-folder" @.>
Cc: "Frédérick Pogu" @.>, "Author" @.>
Envoyé: Mercredi 3 Mai 2023 09:37:39
Objet: Re: [FlorianWoelki/obsidian-icon-folder] Can't see installed icon packs (Issue #160)
This should be fixed in [ https://github.com/FlorianWoelki/obsidian-icon-folder/releases/tag/2.0.0 | 2.0.0 ] .
—
Reply to this email directly, [ https://github.com/FlorianWoelki/obsidian-icon-folder/issues/160#issuecomment-1532580418 | view it on GitHub ] , or [ https://github.com/notifications/unsubscribe-auth/A6R376BQPVMNJTAYSEIDYZ3XEIDMHANCNFSM6AAAAAAV6YGQNU | unsubscribe ] .
You are receiving this because you authored the thread. Message ID: <FlorianWoelki/obsidian-icon-folder/issues/160/1532580418 @ github . com>


Glob 7.2.2 broke our jasmine test suites

We reverted to 7.2.0 and everything works fine again. Not sure if this is a jasmine issue or a glob issue, but I figured I should let you know. Please see full writeup here: https://github.com/jasmine/jasmine/issues/1970#issue-1236201037

Duplicate of #471, please see that issue for more details.

Should be fixed by<EMAIL_ADDRESS> If it isn't, please post a new issue at https://github.com/isaacs/node-glob/issues, ideally with a reproduction case.  Thanks!


[uamqp] connection idle timeout doesn't work in uamqp

through investigation, the idle timeout works as expected, but the connection status transition when idle timeout happens (DISCARDING) is misleading to some extent.
Created an issue in uamqp to track the problem: https://github.com/Azure/azure-uamqp-python/issues/151


Why there is no more kerb preset for highway crossing ?

Why there is no more kerb preset for highway crossing ?

I believe the recommended practice these days is to assign the kerb ramp tags on the crossing-sidewalk junction nodes, not on the crossing way itself.
see #6078

Hi @bhousel
Just curious: do you have a source for such recommendation?
I think @phiphou was asking about highway=crossing specifically, which is used for nodes, rather than highway=footway + footway=crossing, which you describe.
I think it is rather impractical to assume that everyone is keen to move to tagging sidewalks separately to roads, especially in areas where there is little data mapped. Removing the kerb presets for single crossing nodes might mean that we'll loose useful information because mappers will not want to dive into such a level of detail when they simply want to add a pedestrian crossing.
What do you think?


plugin.toml missing

I have the same issue. Maybe an incompatibility with the newer dokku versions?

I just copied plugin.toml from any other plugin and it fixed it...

@nemanjan00 I switched to dokku’s own mariadb plugin (still in beta, but working fine): https://github.com/dokku/dokku-mariadb


Syncs 2.5 images with 1.3

Checklist

[X] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@vagimeli - Is this ok to merge?


Update to operator-sdk 0.9.0

This PR introduces the following changes

upgrades to operator-sdk version 0.9.0 which makes use of go mod
moves from dep to go-mod
updates vendor/
adapts to changes to make build

Signed-off-by<EMAIL_ADDRESS>

/retest

/retest

/lgtm


Entity.InsertChild

It'd be handy to have a method to insert a child element at a certain index. I've gone ahead and written an implementation of this by taking most of the Entity.AddChild() method and just changing it a bit.
I've also just made AddChild() call InsertChild(child, _children.Count) since that seemed the most straightforward way to do it.
I haven't got time right now, but I can do a PR in the next couple of days.
/// <summary>
/// Add a child entity.
/// </summary>
/// <param name="child">Entity to add as child.</param>
/// <param name="inheritParentState">If true, this entity will inherit the parent's state (set InheritParentState property).</param>
public void AddChild(Entity child, bool inheritParentState = false)
{
    InsertChild(child, _children.Count, inheritParentState);
}

/// <summary>
/// Insert a child entity at a specific index.
/// </summary>
/// <param name="child">Entity to add as child.</param>
/// <param name="index">The zero-based index at which <paramref name="child"/> should be inserted.</param>
/// <param name="inheritParentState">If true, this entity will inherit the parent's state (set InheritParentState property).</param>
public void InsertChild(Entity child, int index, bool inheritParentState = false)
{
    // make sure don't already have a parent
    if (child._parent != null)
    {
        throw new System.Exception("Child element to add already got a parent!");
    }

    // do this first, so that any exceptions short-circuit this method
    // before we update any values on child
    _children.Insert(index, child);

    // set inherit parent mode
    child.InheritParentState = inheritParentState;

    // set parent
    child._parent = this;
    child._indexInParent = index;

    // update any siblings which need their index updating
    for (int i = index + 1; i < _children.Count; i++)
    {
        _children[i]._indexInParent += 1;
    }

    // reset child parent dest rect version
    child._parentLastDestRectVersion = _destRectVersion - 1;

    // mark child as dirty
    child.MarkAsDirty();
    MarkAsDirty();
}

Hi @craigmjohnston,
I agree, it could be handy. I implement it in a slightly different way than suggested above, its in dev branch.
Thanks! :)

Ah great, that way works too.


chore: 合并scow-vagrant仓库的文件

合并 PKUHPC/scow-vagrant 的文件。类似#316

Codecov Report
Base: 64.57% // Head: 64.57% // No change to project coverage :thumbsup:

Coverage data is based on head (4284a96) compared to base (1798661).
Patch has no changes to coverable lines.

:exclamation: Current head 4284a96 differs from pull request most recent head bacdbf0. Consider uploading reports for the commit bacdbf0 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #342   +/-   ##
=======================================
  Coverage   64.57%   64.57%
=======================================
  Files         101      101
  Lines        2837     2837
  Branches      353      353
=======================================
  Hits         1832     1832
  Misses        959      959
  Partials       46       46

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Executables on release don't run

On my Windows 7 x64, I've downloaded the 3.1-beta2-b001-SNAPSHOT jmonkeyplatform.zip, then started the bin/jmonkeyplatform64.exe and got this:

Which translate roughly in: "this executable isn't compatible with the OS. Please check if you need 32bit or 64bit"
Also bin/jmonkeyplatform.exe does the same.

Did you try jmonkeyplatform-windows-x64.exe  also?

That one worked, but on about I see:
Product Version: jMonkeyEngine SDK 3.1.0-beta2-b001-SNAPSHOT
Updates: Updates available to version NetBeans 8.1 Patch 1
Wasn't it supposed to be based on Netbeans 8.2?

Wasn't it supposed to be based on Netbeans 8.2?

Yep
Just to get sure can you please check this also :

If it is on netbeans 8.1 then i think @MeFisto94 has forgotten to remove the previous netbeans folder before building the sdk ? (So build script skipped downloading new netbeans)

Apparently, I am sure

Has Travis trolled you?

Well yes, master should still be 8.1 travis wise, but I thought that each tag has it's own cache and hence I would expect beta2 to use 8.2. In addition to that it uses engine version beta1 (https://travis-ci.org/jMonkeyEngine/sdk/builds/179301297#L764). That's a strange thing I have to check out but I guess this can be solved by just removing travis' cache (And hoping that re-building everything doesn't take multiple iterations due to running out of disk space...)
However https://travis-ci.org/jMonkeyEngine/sdk/builds/179301297#L219 shows it had troubles with my specification of the beta2, so I wonder how it could fall back to beta1?
Anyway this should be unrelated to the bug itself and even worse: I had an idea where that error comes from, but since this also happens for netbeans 8.1 and it also happened for beta1, I am clueless.
If someone is bored and has a fast connection he can try out everything roughly between may and beta1, somewhere it starts to break. 😛
Usually those exe files shouldn't change at all, possibly not even between netbeans versions.

I can confirm that 3.1-beta1_unstable-SNAPSHOT works (i.e. the executable is not affected by the bug)

3.1-beta1-b002-SNAPSHOT is affected by the bug

I have tried to investigate on that one and for some reason, the files are simply broken:
working/jmonkeyplatform/bin/jmonkeyplatform.exe: PE32 executable (GUI) Intel 80386, for MS Windows

beta2/jmonkeyplatform/bin/jmonkeyplatform.exe: MS-DOS executable

In addition to that, objdump says Unknown File Type for the broken files. I tried to look at the hex diff but apart from some zeroed out regions.
Using https://github.com/jMonkeyEngine/sdk/compare/3.1-beta1_unstable-SNAPSHOT...3.1-beta1-b002-SNAPSHOT (Thanks @Pesegato for your hint) I could find the problem:
It's 5acbb89. For some reason gradle destroys the binary file. I never liked that task anyway, but it is better than a platform dependent bash/awk script.
I have a locally working solution which is even much faster (4 times as fast at least :


Testability - controlled randomness for mutators.

I spent some time playing with ODfuzz. I tried to design a new API for the existing mutators to make them more testable. At the moment, if we want to test the mutators, we are required either to mock the random module or to set a seed for the random module. The first approach is already implemented here: https://git.io/JfVv2. The second one is implemented here: https://git.io/JfVvr. I think that the later one should not be used at all because it looks more like a cheating to me. The first approach has got some disadvantages too. Once we modify some parts of a mutator, which utilizes the random module, the tests will not work anymore and we will have to update all values passed to mocked classes.
So, I decided to follow a different path. I extracted parts which use the random module into separate functions. For example, I changed the function flip_bit of the mutator StringMutator from:
    @staticmethod
    def flip_bit(self, original_string):
        string = original_string[1:-1]
        if not string:
            return string

        index = round(random.random() * (len(string) - 1))
        ord_char = ord(string[index])
        ord_char ^= 1 << round(random.random() * ord_char.bit_length()) + 1
        ord_char = 0x10FFFF if ord_char > 0x10FFFF else ord_char # see https://stackoverflow.com/questions/52203351/why-unicode-is-restricted-to-0x10ffff
        ord_char = normalize_surrogates(ord_char)
        generated_string = ''.join([string[:index], chr(ord_char), string[index + 1:]])
        return generated_string

To:
    @staticmethod
    def flip_bit(self, original_string):
        string = original_string[1:-1]
        if not string:
            return string

        index = random.randint(0, len(string) - 1)
        bit_index = random.randint(0, ord(string[index]).bit_length())

        return StringMutator._flip_bit(self, string, index, bit_index)

    @staticmethod
    def _flip_bit(self, string, index, bit_index):
        ord_char = ord(string[index])
        ord_char ^= 1 << bit_index
        ord_char = 0x10FFFF if ord_char > 0x10FFFF else ord_char # see https://stackoverflow.com/questions/52203351/why-unicode-is-restricted-to-0x10ffff
        ord_char = normalize_surrogates(ord_char)
        generated_string = ''.join([string[:index], chr(ord_char), string[index + 1:]])
        return generated_string

I changed the function replace_char of the mutator GuidMutator from:
    @staticmethod
    def replace_char(string_guid):
        # get the proper index to the string array by skipping the prefix "guid'"
        # and trimming the last single quote enclosing the GUUID part
        index = round(random.random() * (len(string_guid) - 2 - 5)) + 5
        if index in GuidMutator.GUID_DASH_INDEXES:
            index -= 1
        rand_hex_char = HEX_BINARY[round(random.random() * (len(HEX_BINARY) - 1))]
        mutated_guid = ''.join([string_guid[:index], rand_hex_char, string_guid[index + 1:]])
        return mutated_guid

To:
    @staticmethod
    def replace_char(string_guid):
        # get the proper index to the string array by skipping the prefix "guid'"
        # and trimming the last single quote enclosing the GUUID part
        index = round(random.random() * (len(string_guid) - 2 - 5)) + 5
        rand_hex_char = HEX_BINARY[round(random.random() * (len(HEX_BINARY) - 1))]

        return GuidMutator._replace_char(string_guid, index, rand_hex_char)

    @staticmethod
    def _replace_char(string_guid, index, rand_hex_char):
        if index in GuidMutator.GUID_DASH_INDEXES:
            index -= 1
        mutated_guid = ''.join([string_guid[:index], rand_hex_char, string_guid[index + 1:]])
        return mutated_guid

Then, I wrote tests using the library Hypothesis (https://hypothesis.readthedocs.io/en/latest/).
import re

from collections import namedtuple

from hypothesis import example, given, seed
from hypothesis.strategies import sampled_from, composite, from_regex, integers, text

from odfuzz.constants import BASE_CHARSET, HEX_BINARY
from odfuzz.mutators import GuidMutator, StringMutator

StringPropertyMock = namedtuple('StringPropertyMock', 'max_length')
guid_regex = r"guid'[a-fA-F0-9]{8}-[a-fA-F0-9]{4}-[a-fA-F0-9]{4}" \
             r"-[a-fA-F0-9]{4}-[a-fA-F0-9]{12}'"

@composite
def guid_and_index(draw):
    guid = draw(from_regex(guid_regex, fullmatch=True))
    index = draw(integers(min_value=5, max_value=len(guid) - 2 - 5))
    return (guid, index)

@seed(0)
@given(guid_and_index=guid_and_index(), rand_hex_char=sampled_from(HEX_BINARY))
@example(guid_and_index=("guid'00000000-0000-0000-0000-000000000000'", 28), rand_hex_char='A')
def test_guid_mutator(guid_and_index, rand_hex_char):
    guid, index = guid_and_index
    mutated_guid = GuidMutator._replace_char(guid, index, rand_hex_char)
    assert re.match(guid_regex, mutated_guid)

@composite
def string_and_char_index_and_bit_index(draw):
    string = draw(text(alphabet=BASE_CHARSET, min_size=1, max_size=20))
    char_index = draw(integers(min_value=0, max_value=0 if len(string) == 0 else len(string) - 1))
    bit_index = draw(integers(min_value=0, max_value=ord(string[char_index]).bit_length()))
    return (string, char_index, bit_index)

@seed(0)
@given(string_and_char_index_and_bit_index())
@example(("abcdefgh", 3, 2))
def test_string_mutator_flip_bit(string_and_char_index_and_bit_index):
    string, char_index, bit_index = string_and_char_index_and_bit_index
    StringMutator._methods = ['_flip_bit']
    mutated_string = StringMutator._mutate(StringPropertyMock(20), string, char_index, bit_index)
    assert re.match(r"'.*'", mutated_string)

    # reset the list of available methods
    StringMutator._methods = []

And I was able to test the mutators without any problems. As long as the API does not change, the tests do not need to be altered. We may use this technique for testing all mutators and generators as well. @phanak-sap, what do you think of this approach? I am not planning to work on this anytime soon but I would really appreciate if you could take a look at it. Feel free to address your concerns.
Also, I had to modify a couple of unrelated things because the tests would not work properly:

The value GUID_DASH_INDEXES needed to be fixed. This is a bug because I forgot to update the values in my last commit. Surprisingly, the bug was discovered via the hypothesis tests which I have attached.

 class GuidMutator:
-    GUID_DASH_INDEXES = (8, 13, 18, 23)  # mandatory part of GUUID, "-"
+    GUID_DASH_INDEXES = (13, 18, 23, 28)  # mandatory part of GUUID, "-"

In order to pass arbitrary data to a mutator from the tests (i.e. StringMutator._mutate(StringPropertyMock(20), string, char_index, bit_index)), it was required to do the following change.

class Mutator:
     @classmethod
-    def _mutate(cls, proprty, value):
+    def _mutate(cls, proprty, *args, **kwargs):
         """Select a random mutation function from the list of all available functions."""
         if not cls._methods:
             cls._methods = [name for name in cls.__dict__ if not name.startswith('_')]

         chosen_mutator = random.choice(cls._methods)
-        mutated_value = getattr(cls, chosen_mutator)(proprty, value)
+        mutated_value = getattr(cls, chosen_mutator)(proprty, *args, **kwargs)

Wow this is a huge post and will take me some time to read it thoroughly.

hypothesis was one of my my choices as well. I have also thought about behave for e2e/integration knd of scenarios
My first gut-feeling is to put this together with better modularization - split runtime, database and networking from URL generations (generators, mutators) for odata v2 + potentionally V4 in the future. This testing looks like good approach for something that could be a library style project for the url generators.


Create authentication features in API

[ ] login
[ ] register
[ ] logout
[ ] password resets

@Psami-wondah Suggested JWT authentications, i feel the same too


Update contribution.md

Hey you can contribute any extensions of your choice removing space adding , violates code of conduct ---- you can try styling complete readme file


Compara test db

I have added a test database and tests for these 3 methods:

Bio::EnsEMBL::Slice::get_all_compara_Syntenies()
Bio::EnsEMBL::Slice::get_all_compara_DnaAlignFeatures()
Bio::EnsEMBL::Gene::get_all_homologous_Genes()

If you find any other method that connects to the Compara API, let me know

The first build is failing because I forgot to add ensembl-compara to the dependencies. Should work now

Now failing because tables.sql is not compatible with sqlite. Will fix that now

Coverage increased (+0.2%) to 79.913% when pulling a477eb188b7bb2e70e93284887088ef952834aad on muffato:compara_test_db into 775ec7546f70aafe28823de99b6812b3d3f3f347 on Ensembl:master.

There was some problem in the data. Fixed now. The 999th Travis build has just started !

Coverage increased (+0.2%) to 79.913% when pulling 9e9759f583ad36503a3deb77cc03e111f5516e47 on muffato:compara_test_db into 775ec7546f70aafe28823de99b6812b3d3f3f347 on Ensembl:master.

That's great work, thank you :-)

Fair enough. I'm not thrilled at the core API testing getting circularly dependent on related APIs, but I suppose that ship has already sailed.

@nerdstrike We were asked that (some time ago, admittedly) ! In fact, Compara's Travis build runs the few .t tests from ensembl-rest that use Compara API, and I will soon make it run this new .t from ensembl.
The idea was that we should not break your API, and fix it if we do


Mobile Follow Button z-index incorrect on posts

[ ] This is a question about using the theme.
[ ] This is a feature request.
[X] I have updated all gems with bundle update.
[X] I have tested locally with bundle exec jekyll build.
[X] I believe this to be a bug with the theme --- not Jekyll, GitHub Pages or one of the bundled plugins.

Environment informations

Minimal Mistakes version: minimal-mistakes-jekyll 4.5.0 from git://github.com/mmistakes/minimal-mistakes.git (at layout-adjustments@1f58867
github-pages or jekyll gem version: jekyll 3.5.2
Operating system: 10.12.6

Expected behavior

Steps to reproduce the behavior
the main index page when looking from a mobile device, the "Follow" button displays as expected.

However, when viewing an actual post, it seems the z-index is off.

If I insert a z-index: -1; on the article class, it looks correct, however, I'm not sure if that is the correct solution... nor if it has a negative impact on other things.

-1 was what it previously was but it appeared below the main content. There's probably something else at play I'll have to adjust.

I can't reproduce it on any of demo site's sample pages. Do you see the overlap there?
https://mmistakes.github.io/minimal-mistakes/
Perhaps it has to do with customizations you've made.

You need to use the latest on master. There's some additionally CSS that was added in https://github.com/mmistakes/minimal-mistakes/commit/173f18404725c8930603a1aa356a51aadefaa905 to the .sidebar element to position it above the main content. The CSS you're using on your site is missing that.


Add repository field to package.json

Hello @3846masa, 👋🏼
Thank you for creating the excellent package!
In this issue, I recommend adding the repository field to package.json. If added, npm view or npm repo will be much more helpful. For example:
$ npm view 'stylelint-browser-compat' repository

$ npm view 'stylelint' repository
{ type: 'git', url: 'git+https://github.com/stylelint/stylelint.git' }

$ npm repo --browser=false 'stylelint-browser-compat'
npm error no repository
npm error A complete log of this run can be found in: /Users/masafumi.koba/.npm/_logs/2024-07-12T02_52_40_138Z-debug-0.log

$ npm repo --browser=false 'stylelint'
stylelint repo available at the following URL:
https://github.com/stylelint/stylelint

It would be great if you could consider it. Thank you. 😃

Thank you for your report.
The repository field will be included in package.json after v1.0.0-beta.133.

Excellent! Thank you! 👍🏼


Move settings.numberFormat properties to settings.defaults

I added settings.defaults in de259b8 to support properties like showToolbar that can be overwritten later. I was thinking that this is also the case with the settings.numberFormat properties, so we can move them to settings.defaults and get rid of that numberFormat wrapper key.

Sounds reasonable to me. @mrkwanzaa your thoughts here?

Makes sense to me


Use SPDX license and copyright identifiers

:notebook_with_decorative_cover: Description
This was mostly done in an automated fashion:
banner='\/\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\n \*                    _   _____   __________                                  \*\n \*                   | | \/ \/ _ | \/ __\/_  __\/     Visibility                   \*\n \*                   | |\/ \/ __ |_\\ \\  \/ \/          Across                     \*\n \*                   |___\/_\/ |_\/___\/ \/_\/       Space and Time                 \*\n \*                                                                            \*\n \* This file is part of VAST\. It is subject to the license terms in the       \*\n \* LICENSE file found in the top-level directory of this distribution and at  \*\n \* http:\/\/vast\.io\/license\. No part of VAST, including this file, may be       \*\n \* copied, modified, propagated, or distributed except according to the terms \*\n \* contained in the LICENSE file\.                                             \*\n \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\/' && for file in $(git ls-files); do gsed -i -z "s/$banner/\/\/ This file is part of VAST. It is subject to the license terms in the LICENSE\n\/\/ file found in the top-level directory of this distribution.\n\/\/ SPDX-FileCopyrightText: © $(git log --format="format:%ci" --reverse -- $file | head -1 | cut -d- -f1) Tenzir GmbH<EMAIL_ADDRESS>SPDX-License-Identifier: BSD-3-Clause/" $file; done

:memo: Checklist

[x] All user-facing changes have changelog entries.
[x] The changes are reflected on docs.tenzir.com/vast, if necessary.
[x] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions
🤷♂

Why did you add the first 2 lines?

Why did you add the first 2 lines?

See link in the associated story. They aren't necessary, but they refuse some possible confusion by mentioning the LICENSE file explicitly.

Can you re-run the script without the 2 first lines? I think we have a window to get this through now.

Can you re-run the script without the 2 first lines? I think we have a window to get this through now.

Done. Please check back.

somehow fwd.hpp and partition.hpp as indented? Can you run git format?

somehow fwd.hpp and partition.hpp are indented? Can you run git format?

👀 fixed


[ADD]: added CodeStudio link in Readme.md file

Added a Problem Solving site to Websites for code challenges section.

@DhanushNehru Hey could you please review this?


When disable and keyboard props are true the calendar icon is not disabled

Environment

Tech
Version

material-ui-pickers
1.0.0-rc.7

material-ui
1.0.0-beta.44

React
16.3.1

Browser
Chrome

Platform
Windows

Steps to reproduce

Open example.
Tab to "Test" field.

Expected behavior
When disable and keyboard props are true the calendar icon is also disabled and not focusable, hoverable, clickable.
Actual behavior
Calendar icon is focusable, hoverable, and clickable. In addition the label also moves upward if the value is null.
Live example
https://codesandbox.io/s/92o1640qvp

Hi Jonathan,
thanks for raising this issue!
Would you like to submit a PR?
The idea is to pass disabled prop to InputAdornment component (as well as TextField component, like it is now)

I'll take a stab at it thanks for verifying the issue.

@cherniavskii @dmtrKovalenko When do you think the next release will be?

@lolJS Now Im working on one of important features, Id like to see in the next release. I'm planing finish it today or tomorrow. Thus it would be merged I will do a release. Friday or Saturday.

299 stars 😱

Published in rc.8


process is not defined

console error:process is not defined

This will be true if you haven't enabled nodeIntegration in webPreferences. Cloning latest master should fix this issue.


fix: run vm.$forceUpdate in setData and setProps method

After update from vue-test-utils from 1.0.0-beta.11 to current version, I noticed that children are not always updated:
it('download button is disabled when content is empty', () => {
    wrapper.setData({ content: [] });
    expect(wrapper.find(vButton).vm.disabled).to.be.true;
});

Using wrapper.vm.$forceUpdate() fixed this issue. To be honest I'm not sure why :)

Thanks for the PR. Can you add a test case that was broken before the bug fix?

Sure, I'll try to add test that covers my situation

After investigation I noticed that this bug occurred in very very uncommon scenario which is not worth attention here.


C# "new Process" Hooking

Hello,
I'm trying to hook process creation in the process that I'm injected into. It's a C# application which just creating new process on button submiting. As far as I know C# "Process" creation just calling native CreateProcess function. I tried everything, but I cannot hook this unmanaged function, can anyone tell me what is wrong?
public void Run(RemoteHooking.IContext InContext)
   {
var createProcessHook = LocalHook.Create(
           LocalHook.GetProcAddress("kernel32.dll", "CreateProcessW"),
           new CreateProcess_Delegate(CreateProcess_Hook),
           this);
       createProcessHook.ThreadACL.SetInclusiveACL(new Int32[] { 0 });

       RemoteHooking.WakeUpProcess();
}
[UnmanagedFunctionPointer(CallingConvention.StdCall,
               CharSet = CharSet.Auto,
               SetLastError = true)]
   delegate bool CreateProcess_Delegate(
      string lpApplicationName,
      string lpCommandLine,
      ref SECURITY_ATTRIBUTES lpProcessAttributes,
      ref SECURITY_ATTRIBUTES lpThreadAttributes,
      bool bInheritHandles,
      uint dwCreationFlags,
      IntPtr lpEnvironment,
      string lpCurrentDirectory,
      [In] ref STARTUPINFO lpStartupInfo,
      out PROCESS_INFORMATION lpProcessInformation);

   [DllImport("kernel32.dll", SetLastError = true, CallingConvention = CallingConvention.StdCall, CharSet = CharSet.Auto)]
   static extern bool CreateProcessA(
      string lpApplicationName,
      string lpCommandLine,
      ref SECURITY_ATTRIBUTES lpProcessAttributes,
      ref SECURITY_ATTRIBUTES lpThreadAttributes,
      bool bInheritHandles,
      uint dwCreationFlags,
      IntPtr lpEnvironment,
      string lpCurrentDirectory,
      [In] ref STARTUPINFO lpStartupInfo,
      out PROCESS_INFORMATION lpProcessInformation);

   bool CreateProcess_Hook(
      string lpApplicationName,
      string lpCommandLine,
      ref SECURITY_ATTRIBUTES lpProcessAttributes,
      ref SECURITY_ATTRIBUTES lpThreadAttributes,
      bool bInheritHandles,
      uint dwCreationFlags,
      IntPtr lpEnvironment,
      string lpCurrentDirectory,
      [In] ref STARTUPINFO lpStartupInfo,
      out PROCESS_INFORMATION lpProcessInformation)
   {
       Console.WriteLine("[=] Hooking process creation!");

       bool result = false;
       lpProcessInformation = new PROCESS_INFORMATION();

       result = CreateProcessA(lpApplicationName, lpCommandLine, ref lpProcessAttributes, ref lpThreadAttributes,
           bInheritHandles, dwCreationFlags, lpEnvironment, lpCurrentDirectory, ref lpStartupInfo, out lpProcessInformation);

       Console.WriteLine("Creating process {0} with PID: {1}", lpApplicationName, lpProcessInformation.dwProcessId);

       return result;
   }

What was done to fix this? I'm having the same issue at the moment.


Bugs of presenter mode and embed terminal's local variable

Describe the bug

Here are two bugs from #1739 @3b1b
Bug from https://github.com/3b1b/manim/pull/1739/commits/9a9cc8bdacb7541b7cd4a52ad705abc21f3e27fe
The first scene.wait won't wait for key press because of https://github.com/3b1b/manim/blob/b3b7d214adff2088f6c75cbc814cbbf8f1a9b4e4/manimlib/scene/scene.py#L66
and https://github.com/3b1b/manim/blob/b3b7d214adff2088f6c75cbc814cbbf8f1a9b4e4/manimlib/scene/scene.py#L497-L500
I think here self.hold_on_wait should be equal to self.presenter_mode (or True because if self.presenter_mode == False then self.hold_on_wait won't be used in anywhere)
Bug from https://github.com/3b1b/manim/pull/1739/commits/9df12fcb7d8360e51cd7021d6877ca1a5c31835e
The locals() variables in the iPython terminal opened by typing e in the interactive mode are not local variables in the users file:

So users can't do anything like play(FadeOut(circle)) (because both FadeOut and circle are not in the locals() dict or globals() dict)
This issue happend because embed() is executed by scene.on_key_press() instead of UserScene.construct(), so the local_ns in scene.embed() is read from scene.on_key_press's frame.
https://github.com/3b1b/manim/blob/b3b7d214adff2088f6c75cbc814cbbf8f1a9b4e4/manimlib/scene/scene.py#L137-L143
But the whole stack of frame is:
<frame at 0x7fe5aaf59040, file '/.../manimlib/scene/scene.py', line 140, code embed>
<frame at 0x7fe5aae3ddd0, file '/.../manimlib/scene/scene.py', line 639, code on_key_press>
<frame at 0x7fe5aae49c80, file '/.../manimlib/window.py', line 106, code on_key_press>
<frame at 0x7fe59fea05a0, file '/.../site-packages/pyglet/event.py', line 408, code dispatch_event>
<frame at 0x7fe5a9dd5040, file '/.../site-packages/pyglet/window/__init__.py', line 1343, code dispatch_event>
<frame at 0x7fe5aaf58210, file '/.../site-packages/pyglet/window/cocoa/pyglet_view.py', line 199, code pygletKeyDown_>
<frame at 0x7fe5aaf58040, file '/.../site-packages/pyglet/libs/darwin/cocoapy/runtime.py', line 1141, code objc_method>
<frame at 0x7fe5aaf51840, file '/.../site-packages/pyglet/libs/darwin/cocoapy/runtime.py', line 775, code __call__>
<frame at 0x7fe5aae4aa60, file '/.../site-packages/pyglet/libs/darwin/cocoapy/runtime.py', line 805, code __call__>
<frame at 0x7fe5aae48220, file '/.../site-packages/pyglet/window/cocoa/__init__.py', line 317, code dispatch_events>
<frame at 0x7fe5aae44ba0, file '/.../site-packages/moderngl_window/context/pyglet/window.py', line 181, code swap_buffers>
<frame at 0x7fe5aae49ac0, file '/.../manimlib/scene/scene.py', line 175, code update_frame>
<frame at 0x7fe5aae47c40, file '/.../manimlib/scene/scene.py', line 179, code update_frame>
<frame at 0x7fe5aae1ea60, file '/.../manimlib/scene/scene.py', line 113, code interact>
<frame at 0x7fe5aae17ba0, file '/.../manimlib/scene/scene.py', line 102, code tear_down>
<frame at 0x7fe5aae11950, file '/.../manimlib/scene/scene.py', line 83, code run>
<frame at 0x7fe5a1eb5210, file '/.../manimlib/__main__.py', line 25, code main>
<frame at 0x7fe5a1908a90, file '/.../bin/manimgl', line 33, code <module>>

It won't go through the user's code. So I don't know how to fix it elegantly ¯\_(ツ)_/¯

Bug from 9a9cc8b

Very right! The effect seems to be that it doesn't halt on the first wait call, then does after. Thanks for the note.
To be honest, the "presenter" mode was just something hastily thrown in there as I wanted to record certain intereacitons for the wordle video, so it may not have been fully thought out.

Bug from 9df12fc

Very true, my original intent with this was just to be able to drop into a scene and call the scenes methods, even if the local variable of construct might not be accessible. I also can't think of a clean way to access the construct locals, unless there's some deep python magic I'm unaware of.
What do you think, is this useful enough to keep, or should we just remove it entirely?

For the second one, I can't solve it either.
I'm taking a wild guess that there isn't a way to read the construct's locals without changing the user's file, since they're dynamically generated when running, and locals in user's constructs have no connection to scene.py.
So I think removing this feature is the right choice for now.

What I'll do for now is just change the keybinding to something that requires being more deliberate. (https://github.com/3b1b/manim/pull/1746).  Even if it's not something where you can drop into an environment with the local variables of construct, I can still see it being useful for recording certain presentations where at some point I want to interact with the scene in a way that doesn't yet have GUI support.
Also, in a longer-term future, it may be nice to have a more seamless relationship with the interactive mode and the embedded mode, such that someone could do things like click a Mobjct and copy an id for it to their clipboard, which they could then reference in the terminal, in which case access to local variables wouldn't matter as much.