Version <IP_ADDRESS>

This PR bumps the version to <IP_ADDRESS>, and will be published to hackage.

Candidate published: http://hackage.haskell.org/package/fused-effects-<IP_ADDRESS>/candidate

<IP_ADDRESS> published: http://hackage.haskell.org/package/fused-effects-<IP_ADDRESS>

<IP_ADDRESS> tagged: https://github.com/robrix/fused-effects/releases/tag/<IP_ADDRESS>


Add challenge friend button in /play

add challenge friend button in /play which outputs a modal containing self username field and target friend username field upon submission output classic /play buttons where the user chooses casual/rated, time control, and finally generates the URL. the process is not that hard as you can look at the code in Game.java and the helper modules directory. I'm giving this a good first issue. It's a wanted feature so any help is appreciated.

ok I guess no one wants to work on it well, no luck


There is no default values for components.

The generator skips the default values for Components.Schemas. It would be nice to have a default value for the property itself and in the initializer. I made an example:
Spec example:

Current:

Desired:

Thanks for filing it – you're right, default values are not generated at the moment, but it's a feature we'd like to do, and would gladly accept a pull request for 🙂 Let me slightly rename and label this issue, to properly track it.
A good starting point for anyone contributing this feature would be to add the default value to https://github.com/apple/swift-openapi-generator/blob/main/Sources/_OpenAPIGeneratorCore/Translator/CommonTranslations/translateObjectStruct.swift#L62, and work from there.

Hi, I would like to express my intention to work on this issue, if it is still available. Please let me know if I can proceed.

Go ahead!


Tooltip content text is invisible

Describe the Bug

Which package is affected (leave empty if unsure)
react-email
Link to the code that reproduces this issue
https://demo.react.email/preview/notifications/github-access-token?view=source
To Reproduce
https://demo.react.email/preview/notifications/github-access-token?view=source
Expected Behavior
Tooltip content text is visible
What's your node version? (if relevant)
No response

Seems ok for me, the tooltip seems to appear fine

@gabrielmfern  looked into some details, this bug looks like it is related to os theme, further information:
os info
windows 10
chrome version(121.0.6167.189)
windows light theme:

windows dark theme(which looks fine):

Looks like the text color should be set to white anyways, hope it helps.

is the bug still exists? Shall I give it a try?

is the bug still exists? Shall I give it a try?

Yup it's still there.
PR's are welcome! Would love to review yours if you do make one.
You also don't have to ask permission to open one either. :)

On Mac OS is the same, depends on the system theme.

is the bug still exists? Shall I give it a try?

Yup it's still there. PR's are welcome! Would love to review yours if you do make one. You also don't have to ask permission to open one either. :)

Please assign this to me.

Hello, it's my first time commenting on a issue. but I'd like to add something. I think the issue in here
<TooltipPrimitive.Content       {...props}       className={cn(         'bg-black border border-slate-6 z-20 px-3 py-2 rounded-md text-xs',         ${inter.variable} font-sans,       )}       ref={forwardedRef}       sideOffset={sideOffset}     >       {children}     </TooltipPrimitive.Content>
instead of giving bg-black we can set the color based on the theme. I haven't explored the entire code so I'm not sure what is being used for themes, but maybe use external package and conditionally change the bg color, for eg dark:bg-black for dark mode and default bg-white. Forgive me if I'm commenting anything wrong

Hey, I've checked and the issue is still present, can you please assign it to me?

@gabrielmfern Hey, I've checked and the issue is still present, can you please assign it to me?

This seems to be fixed since<EMAIL_ADDRESS>


 This sort of debug logging shouldn't have any value once the workflow is functional, right?

giving up for now and manually bumping instead


CLI for M365 version for script "Bulk library generation"

🎯Aim
The aim of this issue would be to add a CLI for M365 version of script Bulk library generation.
🤔How
I think one of the best approach would be to use the following commands:

field-add
list-add
list-view-add

I'm willing to work on this one.

thanks @milanholemans 👍
You rock 🤩


Extract code snippets from how-to-open-a-presentation-document-for-read-only-access.md

Extract the code snippets from docs/presentation/how-to-open-a-presentation-document-for-read-only-access.md and use docfx snippets to retrieve them from the samples folder and audit the text.

closing as duplicate


In this bundle compatible with SF3?

Any workaround to install this bundle with SF3?
My SF is 3.2.6
I'm getting this:
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Nothing to install or update
Generating autoload files
> Incenteev\ParameterHandler\ScriptHandler::buildParameters
Updating the "app/config/parameters.yml" file
> Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::buildBootstrap
> Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::clearCache

 // Clearing the cache for the dev environment with debug true

 [OK] Cache for the "dev" environment (debug=true) was successfully cleared.

> Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::installAssets

 Trying to install assets as relative symbolic links.

 --------- ------------------------------ ----------------
            Bundle                         Method / Error
 --------- ------------------------------ ----------------
  WARNING   EasyAdminBundle                copy
  WARNING   IvoryCKEditorBundle            copy
  WARNING   PetkoparaCrudGeneratorBundle   copy
 --------- ------------------------------ ----------------

 ! [NOTE] Some assets were installed via copy. If you make changes to these assets you have to run this command again.

 [OK] All assets were successfully installed.

> Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::installRequirementsFile
Script Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::installRequirementsFile handling the symfony-scripts event terminated with an exception

Installation failed, reverting ./composer.json to its original content.

  [Symfony\Component\Filesystem\Exception\IOException]
  Cannot rename "C:\Dropbox\Apache Xampp\injuve-control-pago\bin\symCAF6.tmp" to "bin/symfony_requirements".

require [--dev] [--prefer-source] [--prefer-dist] [--no-progress] [--no-suggest] [--no-update] [--no-scripts] [--update-no-dev] [--update-with-dependencies] [--ignore-platform-reqs] [--prefer-stable] [--prefer-lowest] [--sort-packages] [-o|--optimize-autoloader] [-a|--classmap-authoritative] [--apcu-autoloader] [--] [<packages>]...

Read the error correctly. It is not a problem of this bundle but of your installation itself.

Cannot rename "C:\Dropbox\Apache Xampp\injuve-control-pago\bin\symCAF6.tmp" to "bin/symfony_requirements".

So either fix the rights so the file can get created correctly or issue an ticket in the corresponding tracker (symfony) or even better ask someone in the official chat-channels of symfony to help you with your problem as it does not seem to be code related.

I understand that you are suggesting that if I solved this problem and others that could arrise, Luiggio could be installed correctly in SF3 version.
Thanks. I'll fix it.
@Blackskyliner


Refused to load the image

start cpp dev container
rename a symbol and make sure that refactor preview shows
:bug: there are no category icons and below CSP errors on the console

Refused to load the image 'vscode-remote://dev-container+2f55736572732f6a7269656b656e2f436f64652f7673636f64652d6465762d636f6e7461696e6572732f636f6e7461696e6572732f637070/home/vscode/.vscode-server-insiders/extensions/ms-vscode.cpptools-1.1.3/assets/ref-comment-light.svg' because it violates the following Content Security Policy directive: "img-src 'self' https: data: blob: vscode-remote-resource:".

This isn't using the asCSSUrl-util

This isn't using the asCSSUrl-util


Bidirectional wrapper does not allow specifying initial state for RNN layer

Please make sure that the boxes below are checked before you submit your issue. If your issue is an implementation question, please ask your question on StackOverflow or join the Keras Slack channel and ask there instead of filing a GitHub issue.
Thank you!

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[x] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[ ] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[ ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

rnn_size = 10
init = K.variable(value=np.zeros(rnn_size), dtype='float64', name='init_state')
gru_1 = GRU(rnn_size)(x, initial_state=init)
runs without error but running
rnn_size = 10
init = K.variable(value=np.zeros([rnn_size, 2]), dtype='float64', name='init_state')
gru_2 = Bidirectional(GRU(rnn_size))(x, initial_state=init)
gives
TypeError: call() got an unexpected keyword argument 'initial_state'

Still running into this issue.


Argument of type "Handler" is not assignable to parameter of type "Handler"

TS tells the following error:
Argument of type "Handler" is not assignable to parameter of type "Handler"
Types of parameters 'string' and 'never' are incompatible. ts(2345)
in
examples/blog/src/index.ts
for all routes (get,post,,put,delete) like:
app.get('/', Controller.root)

Hi @thgick
I fixed this issue by #134 . Please upgrade hono to v0.5.4.
And, the examples are updated. See: https://github.com/yusukebe/hono/tree/master/examples/blog
Thank you!

Hi @yusukebe!
Great! It works!
Thank you very much for that quick fix.


Consider adding custodian to settlement steps / instructions

In the following account hierarchy (brown lines), settlement step 2 can be approved by D in two ways:

using D's account at C
using D's account at B

We could have a mismatch if B's allocation does not match D's approval. The ambiguity would disappear if the Step included the target holding custodian.

Had a discussion with @matteolimberto-da this morning. The ^ would also remove the need for https://github.com/digital-asset/daml-finance/issues/257. Following the discussions from last week, we think it makes sense to do this change although the settlement is pretty flexible as is (under the assumptions that allocations/approvals happen in a consistent manner).

As discussed, we could try to split up the work ^ as follows:

Add custodian to the Instruction/Step (and make minimal changes to the settlement factories).
Let the settlement factories source path information from standing instructions (path-to-root-custodian or parent-custodian preferences).
Allow for compression of Instructions (forming a circle) before they are created (as part of a Batch).


console error when clicking on bars in Firefox

To reproduce, open this link in Firefox, open developer tools and click on the bars.
The following error will appear:
TypeError: Value being assigned to SVGPoint.x is not a finite floating-point value.
Chrome doesn't report this error.
Seems it's due to the fact that when the focus event is fired, clientX and clientY are both undefined: https://github.com/hshoff/vx/blob/master/packages/vx-event/src/localPoint.js#L32.

thanks for reporting this @gnijuohz ! Just getting back from PTO but will try to fix this soon 👍 PRs welcome if you'd like it sooner!

Should be fixed in 0.0.77


RuntimeError: CUDA out of memory

Hi develop, thanks for your great repo, when I try to use convnext, I met this trouble, it confuses me, what should I do?
RuntimeError: CUDA out of memory. Tried to allocate 196.00 MiB (GPU 0; 15.78 GiB total capacity; 14.49 GiB already allocated; 84.94 MiB free; 14.51 GiB reserved in total by PyTorch)
the command I run is
python tools/train.py configs/convnext/convnext-base_32xb128_in1k.py

please set a smaller batch size, modify this line https://github.com/open-mmlab/mmclassification/blob/d2e505415040bf5329ab218bb6fe3d899f176cd5/configs/convnext/convnext-base_32xb128_in1k.py#L8
change 128 to 64 or 32


Update cutadapt and BUSCO

This PR updates cutadapt and BUSCO; this update is required for the VGP workflow.
Installation sequence for tool-installers

[ ] Test using @galaxybot test this
[ ] Inspect CI output for expected changes
[ ] Deploy using @galaxybot deploy this if test install was successful
[ ] Merge this PR

@galaxybot deploy this


[#294] Added 'Brokers' entry within the Workloads section

Since Brokers are a type of workload, a shortcut to our project has been added within the Workloads section as 'Brokers' which displays a list of brokers.
fixes: #294

When I'm selecting the brokers, I can see the proper page. However two sections are getting selected:

Can you update the contribution so that, when I click on Brokers, only this entry gets selected in the page?
Alternatively, if that isn't possible, let's completely remove the ActiveMQ Artemis entry.
@RoddieKieley do you have an opinion on this?

Hello @lavocatt , about the selection of both the entries i will take a look. but, we can't remove the entry with Activemq Artemis because in future we need to update this name with Messaging on openshift. so i think we cannot remove this entry as per our discussion atm. but will confirm from Roderick.

Hey @Msarawan after checking with @RoddieKieley, the correct approach is to remove the ActiveMQ Artemis entry and keep only the Brokers one under Workloads

Hey @Msarawan after checking with @RoddieKieley, the correct approach is to remove the ActiveMQ Artemis entry and keep only the Brokers one under Workloads

I will update now, thanks!


Remove semicolons within js file.

I'm not sure if it's happening because of this fix.
But it's removing all the semicolons from the end of statements.

Are you using standard or semi-standard? Can you provide details on the package settings you are using where this is causing issues?

I'm using standard one. Only settings I got is format on save.

@stephenkubovic Related: When using in semi-standard mode, it removes semicolons on commented lines. 😠

@jsejcksn that sounds like it might be an issue with https://github.com/Flet/semistandard, not this package.


Merge BuildContext with ScopedReader

Is your feature request related to a problem? Please describe.
IDE does not seem to play nicely with typedefs, so when overiding build method of consumer widget it would generate this ugly code:
  Widget build(BuildContext context, T Function<T>(ProviderBase<Object, T> provider) watch) {
    // TODO: implement build
    throw UnimplementedError();
  }

Describe the solution you'd like
It looks like any class implementing method named "call" can act as function, so this seems possible:
abstract class WatchingContext extends BuildContext {
  T call<T>(ProviderBase<Object, T> base);
}

/// assuming ConsumerWidget was changed to use WatchingContext
class HomePage extends ConsumerWidget {
  @override
  Widget build(WatchingContext context) {
    final value = context(_state);
    return _buildSomething(context, value.text);
  }

  Widget _buildSomething(BuildContext context, String value) => Text(value);
}

which does not require any typedefs, and looks less noisy.
Describe alternatives you've considered
I can probably write my own solution on top of river_pod, but it would be nicer to have it main package.
Additional context
Close if it's too silly.

I'd suggest making a custom auto-complete for this instead.
I don't like using context here, because:

it confuses people into thinking that they can't obtain a provider without a BuiltContext
people would forget to replace BuildContext with WhateverContext and would wonder why they don't have access to the method

After playing with this more, it seems to be dart analyzer issue, as it only happens with typedefs that look like this:
typedef Closure = T Function<T>(T input);

so I will report this to dart issues, but hard to say if/when they will fix it.
Anyway, snippet will work, at least for this example. There may be other cases where I'm overriding a method that involves ScopedReader, but I don't have any concrete examples yet, so closing for now.

FYI https://github.com/dart-lang/sdk/issues/43667


Added RaiseUserException (equivalent to RaiseSystemException)

Similar to RaiseSystemException, but only for exceptions thrown by the particle code.
Currently they are still just console.log'd, they aren't treated specially.

Had another go at this in https://github.com/PolymerLabs/arcs/pull/2782 -- PTAL!


Cannot seem to join a game when the host is running PML

I have just taken the time to delete every mod I'm running one by one and hosted a server with the same mods as the joining client each time, and yet still cannot join. Now if I'm running PML but the host is not, I can join. I am getting some sort of error which I think may be the cause of this. Started happening after the latest Pulsar update.
Full logs:
Joining-Side log
Host-Side log

Fixed with 0.10.3


publish delete host on uninstall

waiting until host updates is completed

resolved on https://github.com/gravitl/netclient/pull/137


Add a contact form and finish the mobile version

Add a contact form to your portfolio.
Style the page as per Figma design.

Add a contact form to your portfolio.
Style the page as per Figma design.
setup Linters
Fix the Linters issues
test the form using formspree service


Windows-core incorrectly propagates the std feature to subcrates

Summary
Windows-core always enables the std feature on Windows-strings and Windows-results
Crate manifest
[package]
name = "scratch"
version = "0.1.0"
edition = "2021"

[dependencies.windows]
version = "0.58.0"
default-features = false
features = []

Crate code
# build.rs
fn main() {
    println!("cargo:rustc-link-arg=/SubSystem:Console");
    println!("cargo:rustc-link-arg=/ENTRY:main");
}

# main.rs
#![no_std] #![no_main]

#[no_mangle]
extern "C" fn main() {
    // result incorrectly re-imports std into a no-std project
    let _ = windows::core::HRESULT(0);
}

// found duplicate lang item `panic_impl`
#[cfg_attr(not(test), panic_handler)]
fn _panic(_: &core::panic::PanicInfo) -> ! {
    loop {}
}

I think I figured out how to test this - PR coming soon.

Thanks for reporting this issue! Testing revealed that there was a bit more involved in fixing this problem. #3180 provides the fix as well as the test repro and verification.


Problems with Strike-through (--) and Thematic break (---)

Markwon version: {4.2.0}

Hi!
In the screenshots (https://noties.io/Markwon/#screenshots) and in the description I found that the library supports Strike-through (--) and Thematic break (---).
But they work neither in my project nor in the demo.

I use such plugins:

and artifacts:

Hello @Grimald !
Syntax to define strike-through is ~~ and not --, so: ~~hello~~ will work.
A line of text with thematic break may not contain other characters. It is described in the commonmark spec. So, add a new line after your thematic break definition:
---
Hello!

Oh, sorry. In a small screenshot I mixed up ~~ with a --.
Thanks for the quick response and the link to the specification. It will be possible to talk substantively with the manager :)

But an online editor can do this (https://dillinger.io/):

Well, there are a lot of different markdown implementations, so it's not a surprise. But from your screenshot I see that dillinger simply replaces -- and --- and —(—) and has nothing to do with strikethrough or thematic break 😄

Yes, I agree with you. Need to understand what the customer wants and inform the manager what can be done.
Thanks a lot!


Adding unique key that is case-insensitive

When using uniqueKeys as below, it is letting me insert two rows with same type and same name but in different cases. I want to add unique constraint that should not allowing inserting same name regardless of CASE
  @override
  List<Set<Column<Object>>>? get uniqueKeys => [
        {
          name,
          type,
        }
      ];

I tried using customConstraints but it is giving me SQLITEException
  @override
  List<String> get customConstraints => [
        'UNIQUE (lower(name), type)',
      ];

throws
SqliteException(1): while executing, expressions prohibited in PRIMARY KEY and UNIQUE constraints, SQL logic error (code 1)

I can't even return name.lower() in either uniqueKeys or TableIndex as it gives me Expression but drift wants Column. How to solve it?

while executing, expressions prohibited in PRIMARY KEY and UNIQUE constraints

As the error message indicates, that's not supported by sqlite3. This is also mentioned in the documentation: "As with PRIMARY KEYs, a UNIQUE table-constraint clause must contain only column names — the use of expressions in an indexed-column of a UNIQUE table-constraint is not supported."
It seems to work with generated columns though:
  TextColumn get lowercaseName => text().generatedAs(name.lower())();

  @override
  List<Set<Column<Object>>>? get uniqueKeys => [
        {lowercaseName, type}
      ];


If an object with a PID specified in "-n" exists, log and skip it

If an object exists, we don't do anything. If it needs to be restored from an exported version (e.g. a Bag), it will need to be deleted first, then we can replace it.

As of 98f2bb3565f1179b429823e1514919a067f43b12 this is in place, but we have an additional problem (what if there is more than one object input directory?). We can only specify one PID on the command line.


Legacy projects

This would fix an issue where builds for deployment target of iOS 12.4 are failing due to a missing @availability

@kerekesmarton , would you mind sending this pull-request to master? I'll rebase right after

@kerekesmarton , I'm going to decline this pull-request because I've noticed this is fixed in origin/master, I just didn't update this branch after my last version.
I'll tell you what, I'm going to add a rebase step to my CI so that this doesn't happen again.
Thanks anyway


Create a "Past Events" page

This page will list all the past events, ordered by the most recent on top and with pagination.

Done and in production.


Add server API

Problem solved
Short description of the bug fixed or feature added
Changes made

[ ] Public API changes: list the public API changes made if any
[ ] Internal API changes: explain the internal logic changes

How to test

[ ] Automated tests: link to unit test file
[ ] Manual tests: step by step instructions on how to test

Contributor NFT
Paste in your wallet address below and we will airdrop you a special NFT when your pull request is merged.
Address:

PR-Codex overview
The focus of this PR is to add a basic Express server with a /search endpoint to filter tasks based on a query parameter.
Detailed summary

Added Express server setup
Defined /search endpoint to filter tasks
Implemented task filtering based on query parameter

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#3253  👈
main

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

Join @gregfromstl and the rest of your teammates on  Graphite


Representation of MDPs

Description of the problem

Since, RL aims to solve MDPs i.e., Markov Decision Processes so our first aim should be decide on their representation. It should be designed in such a way that RL algorithms can easily use these representations for finding optimal/sub-optimal solutions.
MDPs have the following elements,

State
Actions
Transition Probabilities
Transition Rewards
Policy
Performance Metric

SMDPs have an additional element called Time of Transition.
How each of the above elements can be represented? One idea can be to use a class for encapsulating the above elements.
Example of the problem

References/Other comments

MDPs and its associated concepts may be represented using the following class structure,
class Action
{
    private string description;

   private Action(const string& description="");
   public static getObject(const string& description="");
   private ~Action();
   public void deleteObject();
   public string getDescription();
};

template <class _type>
class State
{
    private string description;
    private vector<Action*> actions;
    unordered_map<Action*, _type> transitionProbs;
    unordered_map<Action*, _type> iTransitionRewards;

   private State(const string& description="");
   public static getObject(const string& description="");
   public void addAction(Action& action);
   public void setTransitionProb(Action& action, _type transitionProb);
   public void setITransitionRewards(Action& action, _type reward);
};

template <class _type>
class MarkovDecisionProcess
{
    private vector<State<_type>*> stateSpace;
    private unordered_map<State<_type>*, Action*>  policy;
    friend _type performanceMetric();

    private MarkovDecisionProcess();
    public static getObject();
    public void addState(State& state);
    public void updatePolicy(State& state, Action& action);
};


Trim material registry asset name is incorrect

based on Registry Data - wiki.vg, asset_name is a namespace. however in Minestom, the namespace lacks the first part ("diamond" instead of "minecraft:diamond", for example)

wiki.vg is wrong. The asset name specifically cannot include a namespace. The client validates this structure using Codecs.IDENTIFIER_PATH which only allows path characters (notably not :). The namespace used for the material asset is the namespace of the path. You can check net.minecraft.item.trim.ArmorTrim (Yarn mappings) for the associated logic to get the asset, or net.minecraft.item.trim.ArmorTrimMaterial (also Yarn mappings) for the Codec used to deserialize this registry.


[FR]: Nested field column type that only needs the partent field.

Contact Details
No response
Present your request
i have deeply nested metadata for my formulas, where every subfields have their own names based on the formula note names.
currently the DB folder can't show the content of nested fields right and just shows it in the written out JSON style.
another problem is that it is breaking the links that are inside the nested list inside the YAML.
into this:
Basic_Power_Variables:
undefined:                                 - "[[Power]]"
path: Variables/Power.md
type: file
embed: false
i have added an example file of one of my formula notes with the normal dataview table on how i would like to have it inside DB folder too, but it would be fine if everything is in just one column and not split into 3 like in the note.
Formula Basic Power.md
For which platform do you request this request??
Desktop

hi @xDovos thanks for your suggestions,
As we talk in discord we will try to create another kind of cell or a new config to render objects as pure markdown

https://user-images.githubusercontent.com/11924043/209936461-68472aab-7e58-459f-9824-5c090bfb996a.mov
Found a way parsing it with yaml stringinfy and some tricks

Commit 52bbc520


Ubuntu image in Vagrant file no longer available

Ubuntu pulled the box file you have in your Vagrant file.
Updated link:
https://cloud-images.ubuntu.com/vagrant/trusty/current/trusty-server-cloudimg-amd64-vagrant-disk1.box
shell.vm.box_url = "https://cloud-images.ubuntu.com/vagrant/trusty/current/trusty-server-cloudimg-amd64-vagrant-disk1.box"
shell.vm.box = "ubuntu/trusty64"

web.vm.box_url = "https://cloud-images.ubuntu.com/vagrant/trusty/current/trusty-server-cloudimg-amd64-vagrant-disk1.box"
web.vm.box = "ubuntu/trusty64"

Thanks. I've updated this.


"edit" icon missing

V1.12, for some reason pretty "edit" icon is missing:

What should I check?

Those are actually emojis, pencil icon is: 📝 :pencil:. I think that you have missing that emoji in your operating system (what OS are you using or was this on mobile?) or you have somehow disabled emojis in Discord or something like this.

I think that you are missing that emoji in your operating system (what OS are you using or was this on mobile?)

Linux.

Does others see the emoji or are you the only one missing it?

Uhm, I don't know.
If I just copy/paste :pencil: into Discord message window, it displays properly:

Interesting. Is it linux desktop version, that could have bug with rendering emojis... Can you check with mobile or web version of Discord to verify this?

It's exactly the same in web version (Firefox), icon missing.

Hmm... 🤔 Not sure what is causing that then...

Btw did that work before or it has always been missing?

Emojis always worked, the one from wiki hoo - never.
Is there a way to log/see the payload that's being sent to Discord?

Yeah, I was also wondering if the message sent from your MediaWiki server is being encoded in some weird format. To log the payload to PHP error log file, you can:

Open file DiscordNotificationsCore.php
Add this after line 511: error_log($post);
View your PHP error log file (location depends on your installation of PHP)

OK, so it's what's being sent: [08-May-2020 08:52:33 UTC] {"embeds": [{ "color" : "2993970" ,"description" : "???? [Magus](https://
PHP user locale is UTF8, db is in utf8mb4, everything is utf8 in pretty much any place I can think of.
What else can I check?

Ah! So that is the issue. What method do you use the send the data, cURL?

yes, curl

If you open the language file i8n/en.json of the extension in your mediawiki installation, do you see messed up emojis or proper emojis? Maybe that file got messed up at some point or has wrong character encoding?

It does look good

$ file extensions/DiscordNotifications/i18n/en.json
extensions/DiscordNotifications/i18n/en.json: UTF-8 Unicode text

I am not sure what the issue might be then... 🤔


feat: allow misc keys to be modifiable

Fixes #386, though I'm sure you could do a better/more refined job than this :)

Just fixed this in https://github.com/folke/which-key.nvim/commit/6aa1b2fa93a2a26a1bd752080ec6a51beb009e75
keymap desc will always override the label of a preset when present, so it should work without any customizations.

thanks!!


Centralize pyenv install logic

Context
As part of the work to allow Dependabot to work reliably when isolated from the internet some code cleanup has been identified.  This is the beginning of the work to limit the places where we make attempts to talk to the public internet
Approach
I've added a new helpers.rb for python which will contain various methods which are currently copy/pasted between the various python package manager implementations.  The first method I've moved over is the method that installs new python versions via pyenv install.
Ongoing
I am pretty sure the python_version and the methods it calls in the various implementations can also be moved to the new helpers.rb and I will be working on that next.

@mctofu will this log as you are expecting to allow us to search in a central place?


[INFINITY-1737] Enable statsd for Kafka

Just in case, I reverified via the stub universe build that the agents were getting metrics. :+1:

@nickbp and @triclambert since you reviewed this, do you happen to know where @NimaVaziri got https://downloads.mesosphere.com/kafka/assets/kafka-statsd-metrics2-0.5.3.jar from? Upstream is only 0.5.2: https://github.com/airbnb/kafka-statsd-metrics2/blob/master/gradle.properties#L1
Now I'd like to replace it with yet another version and I'm worried I might drop some important changes that Nima applied...

Note for posterity:
I built upstream 0.5.2, and the contents of the resulting jar is byte-for-byte identical to the aforementioned "0.5.3", except for one class file, MetricNameFormatter.class.
Simple decompile shows no difference. After decompiling that class on both sides with procyon -r -v -ln -cp -lv -ta (raw bytecode output with some more details), there only difference is a single missing line number annotation in Nima's version:
@@ -233,7 +233,6 @@
           45: invokevirtual   java/lang/StringBuilder.append:(Ljava/lang/String;)Ljava/lang/StringBuilder;
               linenumber      56
           48: invokevirtual   java/lang/StringBuilder.toString:()Ljava/lang/String;
-              linenumber      49
           51: areturn
     LocalVariableTable:
       Start  Length  Slot  Name        Signature

So I'm going to assume there were no functional changes and the file is safe to replace with 0.5.2.

FTR, @elezar found the source for 0.5.3: https://bintray.com/airbnb/jars/kafka-statsd-metrics2/0.5.3#files/com%2Fairbnb%2Fkafka-statsd-metrics2%2F0.5.3 although it's still not clear what the exact difference is.


Extending AbstractField does not work for mutation fields

Hi @viniychuk,
Maybe, I am doing something wrong, but when I try define mutations in such a way:
private function getRootMutation() {
        return new ObjectType([
            'name' => 'RootMutation',
            'fields' => [
                // here's our likePost mutation
                'likePost' => new LikePost()
            ]
        ]);
    }

and then define LikePost in the following way:
class LikePost extends AbstractField
{

    public function build(FieldConfig $config) {

        $config->addArgument('id', new NonNullType(new IntType()));
    }

    public function getType()
    {
        return new IntType();
    }

    public function resolve($value, array $args, ResolveInfo $info)
    {
        return 2;
    }
}

then it does not work. I receive error that likePost field is undefined.
I tried also extending AbstractInputField, however InputFieldConfig does not contain addArgument function.
Is it possible to do in the current version?

Can you show me the mutation request you're sending?

@tommyJS it looks like you're making a query instead of mutation

I do it in the following way:
$processor = new Processor($productSchema);
$payload = 'mutation { likePost(id:1) }';
$processor->processPayload($payload);
$result = $processor->getResponseData();

@tommyJS , your example will work just fine in v<IP_ADDRESS> although there's a couple of things to mention that could benefit you in the future.

By a convention try to name fields with the "Field" suffix.
When adding a field as an object – you do not need a key (it's simply won't be used), because Field instance itself has a name which you can override by implementing ->getName method.

Here's how it could look like:
class LikePostField extends AbstractField
{

    public function build(FieldConfig $config) {

        $config->addArgument('id', new NonNullType(new IntType()));
    }

    public function getType()
    {
        return new IntType();
    }

    public function resolve($value, array $args, ResolveInfo $info)
    {
        return 2;
    }
}

$processor = new Processor(new Schema([
    'query' => new ObjectType([
        'name' => 'RootQueryType',
        'fields' => [
            'currentTime' => [
                'type' => new StringType(),
                'resolve' => function() {
                    return date('Y-m-d H:ia');
                }
            ]
        ]
    ]),
    'mutation' => new ObjectType([
        'name' => 'RootMutation',
        'fields' => [
            new LikePostField()
        ]
    ])
]));

$payload = 'mutation { likePost(id:1) }';
$processor->processPayload($payload);
echo json_encode($processor->getResponseData()) . "\n";

Thank you @viniychuk. "Field" suffix seems to obligatory.

@viniychuk I think that it might be good to do the following change in Readme.
'likePost'   => new LikePostField(),

instead
'likePost'   => new LikePost(),

Sorry, this convention is not obligatory in <IP_ADDRESS>. (I had <IP_ADDRESS>).

I think that I know what was the root cause of this issue.
It seems that in <IP_ADDRESS> version, the following convention was allowed for fields:
'likePost' => new LikePost()

And additionally in <IP_ADDRESS> version also the following thing is allowed:
'likePost' => new LikePostField()

But nothing different than that. It seems that I need to follow this convention and I think that it is worth adding to Readme. In fact, I can resolve this issue by implementing getName function, but it is not mentioned in Readme (also worth adding).
@viniychuk What about changing getName function to getIdentifier in AbstractField? It would be more intuitive, I think. Normally, Types have names in GraphQL.


Install Python in Windows CI

Currently, the CoreCLR build silently fails in 2.1 because it can't find Python. This was caught in the auto-merge to 2.2 where the CoreCLR build fails correctly/loudly. Fix this by installing python to a build-local location, similar to creating a bash to use.

Latest commit fixes this (still silent) error:
BUILD: Laying out dynamically generated files consumed by the native build system
BUILD: Laying out dynamically generated Event test files and etmdummy stub functions
Traceback (most recent call last):
  File "F:\vsagent\53\s\src\coreclr\src\scripts\genEventing.py", line 17, in <module>
    from utilities import open_for_update
ModuleNotFoundError: No module named 'utilities'

The utilities.py file it's trying to import is in the directory with genEventing.py, but fails as described in https://bugs.python.org/issue34841. The python37._pth that comes with the python zip prevents the same-directory import from resolving (and PYTHONPATH is even ignored). Removing that file works locally.

Success:
  Microsoft.NETCore.TestHost -> F:\vsagent\33\s\bin/obj/x64/Release/blob-feed/packages/specs/runtime.win-x64.Microsoft.NETCore.TestHost.nuspec
  Microsoft.NETCore.Jit -> F:\vsagent\33\s\bin/obj/x64/Release/blob-feed/packages/specs/runtime.win-x64.Microsoft.NETCore.Jit.nuspec
  Microsoft.NETCore.TestHost -> F:\vsagent\33\s\bin/obj/x64/Release/blob-feed/packages/specs/Microsoft.NETCore.TestHost.nuspec
  Microsoft.TargetingPack.Private.CoreCLR -> F:\vsagent\33\s\bin/obj/x64/Release/blob-feed/packages/specs/Microsoft.TargetingPack.Private.CoreCLR.nuspec
  Microsoft.NETCore.ILAsm -> F:\vsagent\33\s\bin/obj/x64/Release/blob-feed/packages/specs/runtime.win-x64.Microsoft.NETCore.ILAsm.nuspec
  Microsoft.NETCore.ILDAsm -> F:\vsagent\33\s\bin/obj/x64/Release/blob-feed/packages/specs/runtime.win-x64.Microsoft.NETCore.ILDAsm.nuspec
  Microsoft.NETCore.Native -> F:\vsagent\33\s\bin/obj/x64/Release/blob-feed/packages/specs/Microsoft.NETCore.Native.nuspec
  Microsoft.NETCore.ILAsm -> F:\vsagent\33\s\bin/obj/x64/Release/blob-feed/packages/specs/Microsoft.NETCore.ILAsm.nuspec
  Microsoft.NETCore.ILDAsm -> F:\vsagent\33\s\bin/obj/x64/Release/blob-feed/packages/specs/Microsoft.NETCore.ILDAsm.nuspec
  Microsoft.NETCore.Jit -> F:\vsagent\33\s\bin/obj/x64/Release/blob-feed/packages/specs/Microsoft.NETCore.Jit.nuspec
  Microsoft.NETCore.Runtime.CoreCLR -> F:\vsagent\33\s\bin/obj/x64/Release/blob-feed/packages/specs/runtime.win-x64.Microsoft.NETCore.Runtime.CoreCLR.nuspec
  Microsoft.NETCore.Runtime.CoreCLR -> F:\vsagent\33\s\bin/obj/x64/Release/blob-feed/packages/specs/Microsoft.NETCore.Runtime.CoreCLR.nuspec

Build succeeded.
    0 Warning(s)
    0 Error(s)

Time Elapsed 00:00:44.69
Command execution succeeded.
BUILD: Repo successfully built.  Finished at 22:13:51.71
BUILD: Product binaries are available at F:\vsagent\33\s\src\coreclr\bin\Product\Windows_NT.x64.Release

We might want to make sure each repo actually produces something to prevent this kind of silent error, since it's basically impossible to notice and manifests as custom changes not being flowed.

(Merging despite Jenkins OSX failure because AzDO OSX was fine.)


feat: initial exchanges page

Summary
https://app.clickup.com/t/861mruakp
TODO:

[x] Tests
[x] Mobile Layout (currently uses standard mobile table formatting)
[x] Placeholder Logo
[x] Dropdowns
[x] Currency Formatting
[ ] ~Handle "upto 4 decimals" as it currently shows 4 DP regardless~

Requires https://github.com/ArkEcosystem/laravel-foundation/pull/562 & https://github.com/ArkEcosystem/laravel-foundation/pull/563

Checklist

[ ] I checked my UI changes against the design and there are no notable differences
[ ] I checked my UI changes for any responsiveness issues
[ ] I checked my UI changes in light AND dark mode
[ ] I checked my (code) changes for obvious issues, debug statements and commented code
[ ] I opened a corresponding card on Clickup for any remaining TODOs in my code
[ ] I added a short description on how to test this PR (if necessary)
[ ] Documentation (if necessary)
[ ] Tests (if necessary)
[ ] Ready to be merged

corners are cut off

border on icons is missing
Design:

Implementation

On sm creen the submit wallet button is below
Design:

Implementation

also card header color is not correct, is blueish in design
Design

Implementation

Md screen does not show the volume
Design

Implementation

Button says "submit wallet" but should say "submit exchange"

lg screen (1024) is broken
Design:

Implementation

margin above blue box should be 32px and is 24px

@alfonsobries Fixed all the points apart from:

lg screen (1024) is broken - this is because there's no md-lg breakpoint yet - added here https://github.com/ArkEcosystem/laravel-foundation/pull/568
Md screen does not show the volume - Same as above
Button says "submit wallet" but should say "submit exchange" - Not implemented yet - https://app.clickup.com/t/861mrubj4
also card header color is not correct, is blueish in design - that's a mistake in the designs, I confirmed with Oleg that it should be gray-100

I'll re-open the PR once the foundation change is merged


Notifier interface and plugin

Motivation for changes:
Notification are currently just output plugins that operate only on task.accepted. This new interface and plugin allow to tweak that.
Detailed changes:

Maintain backwards compatibility, no upgrade actions required.
Converted pushover plugin to use a new interface
Added render()  to Task for consistent render calls.
Added a way to suppress missing phase plugin messages in log.
Added traceback to task.traceback on crash.
Created notify plugin that operates on these notifiers group plugins:

Can now select to iterate over any entry container, not just accepted.
Can set the scope to task, and send only 1 notification per task, regardless of entry containers

Operates on exit phase only.
Created notify_crash plugin that operate on_task_abort and calls notify plugin with a hardcoded config to operate on task scope.

Config usage if relevant (new plugin or updated schema):
Schema:
schema = {
        'type': 'object',
        'properties': {
            'to': {'type': 'array', 'items':
                {'allOf': [
                    {'$ref': '/schema/plugins?group=notifiers'},
                    {'maxProperties': 1,
                     'error_maxProperties': 'Plugin options within notify plugin must be indented '
                                            '2 more spaces than the first letter of the plugin name.',
                     'minProperties': 1}]}},
            'scope': {'type': 'string', 'enum': ['task', 'entries']},
            'what': one_or_more({'type': 'string', 'enum': ENTRY_CONTAINERS})
        },
        'required': ['to'],
        'additionalProperties': False
    }

Usage:
test_notify:
  mock:
    - {title: "foo", url: 'http://bla.com'}
    - {title: "bar2", url: 'http://bla2.com'}
  regexp:
    accept:
      - bar2
  notify:
    to:
      - pushover:
          userkey: userkey
          apikey: apikey
    what:
      - undecided
      - accepted
  scope: entries
  disable: [seen]
  manual: yes

Notify crash schema:
    schema = {
        'type': 'object',
        'properties': {
            'to': {'type': 'array', 'items':
                {'allOf': [
                    {'$ref': '/schema/plugins?group=notifiers'},
                    {'maxProperties': 1,
                     'error_maxProperties': 'Plugin options within notify plugin must be indented '
                                            '2 more spaces than the first letter of the plugin name.',
                     'minProperties': 1}]}}

        },
        'required': ['to'],
        'additionalProperties': False
    }

Usage:
notify_crash:
  to:
    - pushover:
        userkey: userkey
        apikey: apikey
        message: "Task has crashed {{task.abort_reason}} {{task.traceback}}"

notify_crash plugin could probably best be used in a global template.
To Do:

[ ] Convert all notifier plugin to follow new interface:

[x] notify_osd
[x] notify_sns
[ ] notify_xmpp
[x] notifymyandroid
[ ] prowl
[x] pushalot
[x] pushbullet
[x] rapidpush
[ ] send_email
[x] slack
[ ] sms_ru
[ ] telegram
[x] pushover

[ ] Think of way to test this

@liiight Yeah, I dig the general structure now. Should be very easy to implement new notifiers, as well as to send messages about new things.

Closed and opened #1510 on public branch for easier testing


npm install modifies .zshrc

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
7.2.0
Electron version
28.1.0
Operating system
macOS 14.2.1
Last known working Electron Forge version
No response
Expected behavior
npm install does not change .zshrc.
Actual behavior
npm install rewrites the users .zshrc adding this code:
# tabtab source for electron-forge package
# uninstall by removing these lines or running `tabtab uninstall electron-forge`
[[ -f ~/Projects/eletron-forge/test-project/node_modules/tabtab/.completions/electron-forge.zsh ]] && . ~/Projects/eletron-forge/test-project/node_modules/tabtab/.completions/electron-forge.zsh

The tabtab uninstall electron-forge command results in this error:
zsh: command not found: tabtab

Steps to reproduce

npm install

Additional information
Is there some context on why this is needed and can this be made optional to avoid rewriting bash config without warning?
See #528 which was closed without follow-up.

Sounds like you ran npm install electron-forge which is the legacy package name, @electron-forge/core and @electron-forge/cli don't have any dependency on tabtab as can be seen in this code search --> https://github.com/search?q=repo%3Aelectron%2Fforge tabtab&type=code
Ref:


Don't cache the mutations in the PostCss plugin

The mutations are currently being cached "globally" to the PostCss plugin resulting in it becoming stale when combined with Gulp Watch (for example).
This change means that the mutations map is regenerated on each run.
This fixes issue #37

Thanks!

Published to npm as<EMAIL_ADDRESS>

Great! And many thanks for your work on this, it's great when combined with Tachyons.


BLE "Terminating because EAFNOSUPPORT, Address family not supported by protocol"

As stated in https://github.com/AlCalzone/ioBroker.ble/issues/422#issue-677958385 I have a problem with bluetooth low energy access (running the BLE adapter from within iobroker) in docker container on a Raspi 4 4GB (Raspian Buster).
It might be a general Docker+NOBLE problem, but the issue that AlCalzone referred to is as of February 2020  (https://github.com/abandonware/noble/issues/43#issue-567050740) and there is little conversation on this. I believe Docker+NoBLE is a common setup, whether on Raspi4 or not, no?
Can anyone report the setup working in his environment? A report whether that should work would be awesome to avoid investigation in the wrong direction ;-)
Maybe it is a problem, that can be solved through a change in the container build or parameters?
Cheers!

Hi,
please let me know what you did already and how you start your container.  Maybe i can reproduce it and check.
Regards,
André

BTW, my first research gives me the information, that you need to run your container in network mode host to access Bluetooth from inside a container because Bluetooth is a network device.. As I read in you ble issue this seems not to be the case in your setup.
Regards,
André

Thanks for that advice! Would be a simple and reasonable fault by me! Started with the docker command (bridge) from your readme, and did not expect I would need to alter it.
Unfortunately, I was not yet able to access iobroker port 8081 when running the container in host mode (connection refused on 8081 and vis port 8082), but that is another issueI need to solve now first.
I believe it is not an issue then? Would like to keep it open for the moment, but will then likely be closed ;-)
Since I am not a docker expert, you might give me a hint? Would that command be right?

docker run -it --net host  --name myIObrokerHost2  --restart unless-stopped -v iobrokerdata:/opt/iobroker -e IOB_OBJECTSDB_PORT:10000 -e IOB_STATESDB_PORT:10001 buanet/iobroker:v5.0.0

Portainer runs on 9000, so I changed the db ports, which seems to be running fine.

I'm not sure if changing the database ports on a master is working that way. Never tested this....
Think it would be a better choice to redirect the portainer port and keep the iob database defaults....
But what ever. Without the ioBroker container log I have nothing to analyze. :)
Regards,
André

Hi André,
I also had that impression, but thanks for looking on this again! The redirect of portainer ports indeed was the easier way. I finally chose to change portainer! By the way: Changing the iobroker database ports indeed had no effect ;-) They still ran on 9000 and iobroker searched on 9000 etc. as well ...
What works now is:
docker run -d -p 9500:9000 -p 8500:8000 --name portainer --restart always -v /var/run/docker.sock:/var/run/docker.sock -v portainer_data:/data portainer/portainer:arm
and
docker run --privileged  --net host  --name myIObroker  --restart unless-stopped --mount src=/home/pi/iobroker,target=/opt/iobroker,type=bind -v  buanet/iobroker:v5.0.0
Erased the data folder of iobroker container before and started from scratch then again.
We can close this issue. It was neiter a noble bug nor ioboker or docker container problem, but just false configuration. Seems to work like a charm now! Thanks for your efforts and contributions!
May I sugggest to add a remark in your readme, that BLE adapter will not run when not running your container in host mode?
Cheers!
C4mper


Move tests out of package; no longer installed to site-packages

Tests are no longer unnecessarily installed to users site-packages
directory. Tests are still included in the source distribution.

it would be really good to merge this PR.


ReflectMapEntities on Children Component Fails to Complete with Partial EntityMap.

Bevy version
Tested on Bevy v0.8.1, but the code in question hasn't changed in Bevy 0.9.0.
What you did
In the bevy_ggrs plugin we use the ReflectMapEntities data to update all of the Children and Parent component's stored entity ids whenever we restore a world snapshot.
The issue happens when we try to map entities on Children components, but we don't necessarily have every existing entity in the entity map.
The problem is in this line
https://github.com/bevyengine/bevy/blob/e954b8573c085a01c62007c4c6232870e0b5c891/crates/bevy_hierarchy/src/components/children.rs#L26
When we don't have a certain entity in the map, this function returns with an error, but in my scenario, I still need it to map all of the entities that it can, and leave any entities that it can't map the same as what they already were.
For me, patching this so that it only updates the entity if it is in the map, and ignoring it if it isn't fixes my issue. This is similar to what is already done for the Parent component.
I'm not sure if there is a scenario where it's important to report an error or not.
Edit: Opened #6791 to fix.

This is similar to what is already done for the Parent component.

It seems there was a concern for this but it was never addressed https://github.com/bevyengine/bevy/pull/2424#discussion_r678257338

I have a somewhat related issue with mapping in #6702. The code that fails for you also fails for me, but in my case, it highlights the incorrect behaviour of the write_to_world function. Hopefully, the fix you provided will not make it harder to debug my original issue, as it will silently ignore the entity remapping bug.

@sQu1rr apparently we can run into issues with an invalid hierarchy if we use my fix ( https://github.com/bevyengine/bevy/pull/6791#pullrequestreview-1197003848 ) so I'm not sure if my fix will work anyway. 🤔
I may just need to make a custom RollbackMapEntities trait for my use-case, that allows partial maps.
The only other thought I have right now is to provide maybe a second function to the MapEntities trait for allowing you do partial maps.


just to trigger the test

just to test it

@dlorenc this is just a testing PR to double-check the actions :)
/close

forgot we dont have prow 😂


Add PWA support to the react application

Reference: https://web.dev/progressive-web-apps/

I can work on this feature 💯

I can work on it.
@gmelodie your views on this?

I would love to join you on this issue. It seems like a great opportunity!

I would like to work on this with both of you as well @CIPHERTron @harshsharma6401

I'd love to jump in if there's any room! 😂

OMG we have so many folks interested to work on this haha
Let me open a PR with the basic service worker setup. Then, maybe we can work on that i.e. the manifests, favicons and stuff.
Does that sound good @ditsuke @KshitijDarekar @harshsharma6401?


[New Feature]: Workflow for closing old PRs

🎮 feature Request
Over time, open pull requests (PRs) can accumulate in a repository, cluttering the interface and making it difficult to focus on active development tasks. Additionally, outdated or abandoned PRs may no longer serve their intended purpose and should be closed to maintain repository cleanliness.
Implement a workflow that automatically closes old PRs based on predefined criteria, such as their last update timestamp. By regularly closing PRs that have not been updated within a certain timeframe, the repository can stay organized, and contributors can focus on active and relevant PRs.
Point down the features
This feature aims to automate the management of open PRs in a repository, ensuring that outdated or abandoned PRs are closed in a timely manner. By doing so, it helps maintain repository hygiene, improves workflow efficiency, and provides a better experience for both maintainers and contributors.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

@Durgesh4993 please assign me this issue under gssoc'24

Please raise any other issue, i'll assign you...
If you have any doubt you can directly send message on our discord (https://discord.gg/rZb46cCMmK), we have created separate channals for all projects...
Thank you :)


#80 preserve search items

Changes:
For not clearing the List
this.preserveSearchItems = this.searchItems.length > 0;
ng-click="viewModel.selectItem(item, viewModel.preserveSearchItems)"
For making the selected Items unavailable for re-selection

store a clone of the searchItems in preservedSearchItems
remove the selected items from searchItems in this.setSelectedItems = function(items)

i have some big troubles installing procrator. sorry for no tests.

@guylabs ping never did a PR on a Fork, so no clue if you got informed ;)

Hi @thedanielhanke,
this PR is not needed anymore when we fixed #57 with the new functionality of the items-method.
Thanks again for the contribution!
Regards,
Guy

:+1:


<< is not a valid property - support yaml merge keys

Hi
Currently the reader does not support the use of yaml "<<" merge keys
Repo:
    [Fact]
    public void TestMergeKeysOnParameters()
    {
        // Arrange
        OpenApiDocument document;
        var diagnostic = new OpenApiDiagnostic();

        String spec = @"
          openapi: 3.0.0
          info:
            description: This is a simple API
            version: 1.0.0
            title: Simple Inventory API
          paths:
            /inventory:
              get:
                summary: searches inventory
                operationId: searchInventory
                parameters:
                  - $ref: '#/components/parameters/custom-header-example1'
                responses:
                  '200':
                    description: OK
          components:
            parameters:
              custom-header: &custom-header
                name: custom-header
                in: header
                description: The requested API version
                required: false
                schema:
                  type: string
                style: simple
              custom-header-example1:
                <<: *custom-header
                example: specific example
          ";
        document = new OpenApiStringReader().Read(spec, out diagnostic);

        var parameter = document.Paths.First().Value.Operations.First().Value.Parameters.First();

        parameter.ShouldBeEquivalentTo(
            new OpenApiParameter
            {
              Name = "custom-header",
              In = ParameterLocation.Header,
              Description = "The requested API version",
              Required = false,
              Schema = new OpenApiSchema
              {
                Type = "string"
              },
              Style = ParameterStyle.Simple,
              Example = new OpenApiExample
              {
                Value = new OpenApiString("specific example")
              } as IOpenApiAny
            }
        );
    }

The diagnostic contains the following errors:
  [0]: {<< is not a valid property at #/components/parameters}
  [1]: {The field 'name' in 'parameter' object is REQUIRED. [#/components/parameters/custom-header-example1/name]}
  [2]: {The field 'in' in 'parameter' object is REQUIRED. [#/components/parameters/custom-header-example1/in]}

custom-header-example1 should "merge" in the properties of the referenced "custom-header" parameter, redefining "example". However once read in the document has null for all the other properties of "custom-header-example1".
Do you have any plans to support the merge syntax as it is valid yaml and useful for very large open api specs with reused components?
Thanks in advance.

The OpenAPI spec only supports YAML that uses the JSON Schema.  This ensures OpenAPI descriptions can be roundtripped between JSON and YAML.

Please feel free to reopen this issue if you feel it has not been resolved.


Add vimeo CSP

Description of changes:

Break CSP rules into several lines for readability
Add Vimeo CSP

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #1944 (ab09db4) into main (1447dfd) will decrease coverage by 0.01%.
Report is 3 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1944      +/-   ##
==========================================
- Coverage   69.74%   69.74%   -0.01%
==========================================
  Files         440      440
  Lines       13354    13355       +1
  Branches      985      985
==========================================
  Hits         9314     9314
- Misses       3862     3863       +1
  Partials      178      178

Flag
Coverage Δ

metweb
62.51% <ø> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes


Remove one of XLSX file download way

https://3.basecamp.com/3198361/buckets/3856681/messages/5577317219#__recording_6245485551

Both reactable-bundle and dataclips-bundle are after yarn build:prod


Adds init info to fix apt-get upgrade and other related stuff

Why?

Ran into this issue with #137

What changed?

This fixes the puma init script so that it doesn't cause apt-get upgrade to fail (which in turn causes a whole host of other things to fail)

This is just commented out code. What are you trying to do here?

@BrandonMathis this file is being added to init.d and is causing problems when running upgrades (which the letsencrypt renew script does automatically). These comments are LSB Tags https://wiki.debian.org/LSBInitScripts/DependencyBasedBoot, but the long and short of it is that without these comments, software updating would not complete, the ssl cert would not renew, and other problems ensued.
Here's the relevant error message from the full dump on #137 :
insserv: warning: script 'puma' missing LSB tags and overrides
insserv: There is a loop between service monit and puma if stopped
insserv:  loop involving service puma at depth 2
insserv:  loop involving service monit at depth 1
insserv: Stopping puma depends on monit and therefore on system facility `$all' which can not be true!
insserv: exiting now without changing boot order!
update-rc.d: error: insserv rejected the script header


Rewrite restore to guard against missing migrations

Initial implem of #1658
I had to resort to several TS hacks to get it right, but it can likely be improved.
What it guards against:

missing migrations for new element types
missing properties (this one was the most common error)
extra properties - we no longer need to remove obsolete properties because we're basically recreating the element objects from scratch

I've split selection element from the generic element to be able to exclude it from the ExcalidrawElement type (because we're filtering it out on restore). If anyone can come up with a better solution (that still keeps the validation for missing element types), have at it.
Several problems:

kinda unreadable, especially some of the TS error (e.g. when you're missing an element type migration in the switch case)
right now, we invoke 2 functions per element, and do a shallow clone. Didn't notice a perf regression though (also 2nd commit makes a different perf improvement that may offset it).
As we now need to "migrate" even base properties, those which were there from the start, we need to duplicate initialization. In some cases I don't (text baseline...) and simply take it from the migrated element.

In 2nd commit I've rewritten normalizeDimensions to be pure and not mutate the element (which was triggering a rerender for each element --- not something we'd want in restore).

please double check that adding a new property to ExcalidrawElement breaks the type checking for this function. thanks!

It does. It didn't for you?

No I didn’t end up testing it. Just couldn’t tell if it’d break in the code review for sure. If it looks good let’s merge it


Dev server does not serve routes expected to be built by static export

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
In dev mode, navigating to /blog/posts/my-post-title should be picked up by the router as a destination that will eventually be available on static export.
Current Behavior
In dev mode, navigating to /blog/posts/my-post-title is not be picked up by the router, and you get a 404 where a static HTML file would normally be located after exporting the static site.
Steps to Reproduce (for bugs)

Use the following next.config.js:

// next.config.js
module.exports = {
  exportPathMap: function () {
    return {
      "/": { page: "/" },
      "/p/hello-nextjs": { page: "/post", query: { title: "hello-nextjs" } },
    }
  },
}

Run the dev server with next
Navigate to localhost:3000/p/hello-nextjs
The result will be a 404

Context
The expected behavior is extremely relevant to using Next to build static websites. One key component to developing a static website is the ability for the local development server to resemble as closely as possible the conditions where the static site will be served. In this case, I would expect that the paths exported from exportPathMap would be used to serve the correct page.
You may be thinking that next-routes + <Link as='...' /> is a good solution to this problem, where the browser push state masks navigation to /blog/posts/my-post-title with something like blog-post?slug=my-post-title, but this only works when navigating from a root page or index, not when reloading or opening a new window.

I'm not familiar enough (yet) with the source, but if it's any help to someone who is, my guess is that this could be easily fixed by adding the result of exportPathMap to the the built-in routing paths. Roughly around here, we could detect dev mode and serve the correct page template specified in exportPathMap for any routes it matches.

This also helps with #1972 (but not solve it, as part of it is unrelated)

Hey @tannerlinsley, what I did on my team is extract the routes into its own module that just returns an object with the route as the key and data about the route (e.g., page and query) as the value. We then consume this module in next.config.js by returning the object returned by the module and in our development server, where we route all get requests through middleware that looks for a route with the req's name and returns that page if found. Happy to provide some code examples if you'd like.

Yeah, we've tried almost that exact same thing, though we don't like relying on a custom server, given we are only using next to export static sites. Would love to see your code regardless :)

Hey @tannerlinsley, sorry for the delay. We have three files:
routes.js:
module.exports = () => {
  return {
    '/': { page: '/', query: { data } },
    '/404': { page: '/404', query: { data } },
    '/about': { page: '/about', query: { data } },
    ...
  }
}

developmentServer.js:
const express = require('express');
const next = require('next');
const { parse } = require('url');

const DEV = process.env.ENVIRONMENT !== 'production';
const PORT = 4567;

const app = next({dir: '.', dev: DEV});
const handle = app.getRequestHandler();

const getRoutes = require('lib/routes');

const routes = getRoutes()
app.prepare().then(() => {
  const server = express();
  server.get('*', (req, res) => {
    const parsedUrl = parse(req.url, true);
    const { pathname, query } = parsedUrl;
    const route = routes[pathname];
    if (route) {
      return app.render(req, res, route.page, route.query);
    }
    return handle(req, res);
  });

  server.listen(PORT, (err) => {
    if (err) throw err;
    console.log(`> READY FOR LIFOTFF http://localhost:${PORT}`);
  });
});

and finally, next.config.js:
const getRoutes = require('lib/routes');

module.exports = {
  exportPathMap: (tes) => {
    return getRoutes();
  }
};

Let me know if you have any questions (or suggestions!).

This works perfectly. Thanks guys.

@ldthorne I guess it'd be good to document this in the wiki 👌

Thanks for this solution @ldthorne. I just have a little issue about it;
I have a dynamic link like /episode?id=s01e01&title=pilot as /e/s01e01-pilot this works very well on dev server but I have an error when I serve /episode?id=s01e01&title=pilot around id query: Cannot read property 'id' of undefined
Is it a way to redirect to a 404 without having theses errors?
Thanks and cool for adding this solution on the wiki 🤓


Add a tutorial on batch shaping in Pyro 0.2

Pyro 0.2 will have stricter batching requirements (after #806) to allow new features like parallel enumeration #742 and nested iarange #780. This will require more careful use of pyro.iarange, TorchDistribution.reshape() and pyro.scope() for masking. The tutorial should also cover how to write broadcastable PyTorch code, such as using Ellipsis as in x[..., i, j] and setting flexible sample_shape in sample statements.

What about two smaller tutorials, one upstream in PyTorch on distribution batch shape semantics and one in Pyro about Pyro-specific features?

this is a great idea.
the tutorial should include a clear statement of when a function / block can be batched. (eg. it can't include certain operators which take a tensor to a non-tensor. other restrictions?)

Didn't notice this thread. The tutorial will be greatly helpful! I have to spend a few days to work with Pyro distribution shape semantics and still not sure I make things right. :(


Upcheck endpoint

Create a basic app which responds to Upcheck

Uses PicoCli to define a http end point
Uses Vertx to create http endpoint using config from cmdline
Success = able to ‘get’ http://a.b.c.d/upcheck (using curl), and displays a fixed string (“OK”).
Requires AT

This has been superseded by #9


RELATED: RAIL-1815 bump sdk8 dependencies

The code coverage is the same after your change 🤟
This is a good thing if you are not sure ;)
Report (compared to build #140)
Covered lines: 26.8%, change: 0%
Covered branches: 22.58%, change: 0%
Coverage report
https://checklist.intgdc.com/job/client-libs/job/gooddata-create-gooddata-react-app-unit-tests-zuul-docker-el7/143/artifact/ci/results/coverage/lcov-report/index.html

Build succeeded (check pipeline).

https://checklist.intgdc.com/job/client-libs/job/gooddata-create-gooddata-react-app-unit-tests-zuul-docker-el7/143/ : SUCCESS in 51s
https://checklist.intgdc.com/job/compliance/job/gooddata-create-gooddata-react-app-oss-check/119/ : SUCCESS in 3s

Build succeeded (gate pipeline).

https://checklist.intgdc.com/job/compliance/job/gooddata-create-gooddata-react-app-merge-compliance-check/53/ : SUCCESS in 16s
https://checklist.intgdc.com/job/client-libs/job/gooddata-create-gooddata-react-app-unit-tests-zuul-docker-el7/144/ : SUCCESS in 49s
https://checklist.intgdc.com/job/compliance/job/gooddata-create-gooddata-react-app-fossa-scanning-zuul-el7/18/ : FAILURE in 4m 03s (non-voting)


Implement telemetry reporting

Proposed changes:

added telemetry reporting
fixes #6613

Status (please check what you already did):

[x] added some tests for the functionality
[x] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)

@akelad can you please take a look at the docs?

thanks a lot @akelad for the comments :rocket:

@federicotdn ready for another review 🙂
I've changed the reporting function to make sure it runs in the background :+1:


Skipping test_xpl.py::test_save on v24.2

As the title.
Close #2904

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

Project coverage is 84.42%. Comparing base (cda0865) to head (8b05b61).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2905      +/-   ##
==========================================
- Coverage   86.57%   84.42%   -2.16%
==========================================
  Files          48       48
  Lines        9402     9449      +47
==========================================
- Hits         8140     7977     -163
- Misses       1262     1472     +210


Popover

Hi,  I was hoping you could show be a basic example of how to implement the bootstrap popover using your PrototypeJS based bootstrap.   I have had no luck getting any components working other than a modal.
here's a basic page I'm testing with which you used in another support page.   I've tried even initializing a tooltip using a document.observe but it just throws an error stating that tooltip is not a function.
the modal on this page does work though.
here's a link to the basic page i'm working with: https://app.box.com/s/x8xo5n4j3ytbn5hksotc9ur4c6v4us6h

I have an example page where I built all the components here http://cwaadmin.com/bootstrap/
I'll take a look at your example and see if I see anything

thank you for this!  helps big time.

Jason, I tried to run your script but you've deployed to Box and so downloading your samples is not complete, you have relative linked script files that are not found.  Can you instead put up a working sample within your domain or create a jsFiddle?

Hi guys, thank you for your help, I actually got it working by following jwestbrooks example component page, I should have closed this.


Add missing export functionality for more layers

Resolves #87. This PR adds missing export functionality (to JSON) for the following layers:

[x] Concatenate
[x] DepthwiseConv2D
[x] SeparableConv2D

You really fast @mkaze ! Many thanks!


How to do Apple versioning in CMake?

working branch: https://github.com/madebr/SDL_image/tree/cmake_work
I don't know how to reproduce the Macos dylib versioning using CMake.
Since I also don't own Apple hardware, I'm unable to build it locally.
The goal is to reproduce the versioning of lbiraries generated by the SDL2_image project in the Xcode subfolder.

The CMake build system defines libtool version variables for versioning the .so libraries on Linux (these match the autotools build).

For Macos, the following variables are available for dylib versioning DYLIB_CURRENT_VERSION and DYLIB_COMPATIBILITY_VERSION.

SOVERSION and VERSION contain a section about MACH-O versioning.

CMake provides MACHO_COMPATIBILITY_VERSION and MACHO_CURRENT_VERSION, but these are for CMake 3.17+ (=march 2020, which is perhaps a bit too new for ubuntu 20.04 LTS)
The current minimum cmake required version is 3.14.

@SpaceIm Can you please help me out?

I don't know, does it ever work in CMake 3.14+ with another generator than XCode?

I didn't test with the XCode generator.
But I think it will work for all generators, even for Xcode, but it just won't set the versioning.
I ended up peeking into SDL2's cmake script and borrowing their approach:
https://github.com/libsdl-org/SDL/blob/e9c7b5191c3ed783489d66ba825f2101810a9b58/CMakeLists.txt#L610-L611

I ended up peeking into SDL2's cmake script and borrowing their approach:
https://github.com/libsdl-org/SDL/blob/e9c7b5191c3ed783489d66ba825f2101810a9b58/CMakeLists.txt#L610-L611

Yes, I think there is no other choice.


[OSD-19300] allow STS based access to new AWS accounts

What is being added?
Is this a fix for a bug?  What's the bug?  Is this a new feature? Please describe it. Is this just a small typo fix?  That's fine too!
Checklist before requesting review

[x] I have tested this locally
[ ] I have included unit tests
[ ] I have updated any corresponding documentation

Steps To Manually Test
Please provide us steps to reproduce the scenarios needed to test this.  If an integration test is provided, let us know how to run it. If not, enumerate the steps to validate this does what it's supposed to.

Start the operator
Run the thing
Observe X in the spec for the thing
Clean up the thing

Ref OSD-0000

/remove-do-not-merge

Codecov Report

Merging #793 (62d7770) into master (54feee1) will increase coverage by 1.32%.
The diff coverage is 62.65%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #793      +/-   ##
==========================================
+ Coverage   44.91%   46.23%   +1.32%
==========================================
  Files          44       44
  Lines        6120     6356     +236
==========================================
+ Hits         2749     2939     +190
- Misses       3092     3115      +23
- Partials      279      302      +23

Files
Coverage Δ

api/v1alpha1/accountclaim_types.go
45.00% <ø> (ø)

controllers/account/iam.go
56.72% <50.00%> (ø)

pkg/awsclient/client.go
16.73% <0.00%> (-0.43%)
:arrow_down:

...ontrollers/accountclaim/accountclaim_controller.go
58.02% <64.37%> (+13.07%)
:arrow_up:

... and 2 files with indirect coverage changes

/retest

/lgtm

/retest

/retest-required

/lgtm


Should AbsorbPointer also have a ignoringSemantics argument?

Since untappable UI shouldn't be selectable via accessibility anyway.

Sounds like it probably should.


premis export: check for unclosed file objects

unfortunately the issue still exists... either in the plugin or a library being used by it

We found the leak in the nanite/droid dependency, fixed it, and send a pull request to the maintainers: https://github.com/openpreserve/nanite/pull/36


SDKS-2948 crash fix and improvement

JIRA Ticket
Please link jira ticket here
Description
Briefly describe the change and any information that would help speedup the review and testing.

Addresses the comments


[BUG] LiteDb can't read Bson.ObjectId

Version
LiteDB version: 5.0.10 (nuget)
OS: Win10 64bit
.NET framework: 3.1 Core
Describe the bug
When reading an object with Bon.Object ID I get the following error:
Value cannot be null. (Parameter 'bytes')
It seems LiteDB is calling the public ObjectId(byte[] bytes) constructor but with null
When i Look at the DB with LiteDB Studio all ID fields have data in them.
Code to Reproduce
public class MongoEntity
{
    [BsonId]
    public ObjectId _id { get; set; }
}

var collection = db.GetCollection</* child of MongoEntity*/>("some name");

var result = collection.FindAll();
foreach (var row in result) // here the error occours
{

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots/Stacktrace
 at MongoDB.Bson.ObjectId..ctor(Byte[] bytes)
   at LiteDB.BsonMapper.Deserialize(Type type, BsonValue value)
   at LiteDB.BsonMapper.DeserializeObject(EntityMapper entity, Object obj, BsonDocument value)
   at LiteDB.BsonMapper.Deserialize(Type type, BsonValue value)
   at LiteDB.LiteQueryable`1.<ToEnumerable>b__27_2(BsonDocument x)
   at System.Linq.Enumerable.SelectEnumerableIterator`2.MoveNext()

Additional context
as I saw, Bson.ObjectId has no arg free Constructor

@fritz-net You're using the MongDB ObjectId type. The LiteDB mapper doesn't know how to deserialize it and ends up using the wrong constructor. You can get it working by adding the following code to your project:
BsonMapper.Global.Entity<MongoDB.Bson.ObjectId>().Ctor(doc =>
            new MongoDB.Bson.ObjectId(
                doc["TimeStamp"].AsInt32,
                doc["Machine"].AsInt32,
                (short)doc["Pid"].AsInt32,
                doc["Increment"].AsInt32));

While this should work, you'll be storing the oid as a document, which will perform worse than using the native LiteDB ObjectId type.

Thank you @lbnascimento this fixed the issue for me.
Also thanks for the hint regarding perfomance. The issue with Chaning the ObjectId is that I want to be able to both use Mongo or LiteDb debending on the Setup of the program (stanalone = litedb OR server = mongo)

@fritz-net In theory, it should be possible to use pre-processor directives for this purpose:
public class MyClass
{
#if SERVER
	MongoDB.Bson.ObjectId _id { get; set; }
#else
	[BsonId]
	LiteDB.ObjectId _id { get; set; }
#endif
}

You would also have to do the same for every time you access the _id property. Honestly, I wouldn't bother with it unless I actually experienced performance issues.

@lbnascimento thanks for your response. I think I will use your first suggestion.
Performance is no issue at the moment. If there are issues I will benchmark and decide if its worth the effort to have 2 executables.


Rule idea: dollar-variable-pattern

Following my question on SO (http://stackoverflow.com/questions/36304921) I'm proposing to add a rule to enforce a naming convention for variable names in SCSS.
Something like https://github.com/sasstools/sass-lint/pull/257 would be great I reckon, where we could either specify a string for a preferred common format (camelcase, snakecase etc...) or a custom pattern.
For example I may want my variable names to always be camel case, in which case:
// this wouldn't be correct
$my-variable: #000;

// but this would be correct
$myVariable: #000;

I don't have experience in writing plugins for stylelint but I may give it a try based on the other rules in this repo and try to work something out in a PR if it's of any use to anybody else.

I don't have experience in writing plugins for stylelint but I may give it a try

Hi @arnaudvalle. I'm not a contributor here, but I help out on the stylelint linter itself. Perhaps I can be of some help here...
There's some stylelint documentation that you might find useful when getting started: working on rules & writing plugins.
There's also some documentation on naming rules and how rules are built to work together.
Finally, there are a handful of similar *-pattern rules in stylelint that might provide a good starting point this plugin:

custom-property-pattern - probably the most similar to this proposed plugin.
custom-media-pattern
selector-class-pattern

...
You might want to follow the same convention and call this plugin variable-pattern?
Good luck :)

@jeddy3 thanks for the links, they'll definitely be useful when I have a go at this!

Thanks! This is something that should not be that hard to implement.

You'll probably want to call this plugin: stylelint-dollar-variable-pattern so that the precss can use it too :)

Yeah, if the rule is not going to be only used for scss syntax, then it makes sense to make it a plugin and just wrap it in stylelint-scss.

This was implemented in #10, so closing. Thanks to everyone who participated!

I think the docs for this need updating. No mention of how to use this. https://github.com/kristerkari/stylelint-scss/blob/master/src/rules/dollar-variable-pattern/README.md

@Arcanorum if you or someone else have not solved this already, I had luck by adding this to my .stylelintrc.yaml in the rules part:
scss/dollar-variable-pattern: "^-?[a-z][a-z0-9]*(--?[a-z0-9]+)*$"


Implement fingerprint method in Core classes

Proposed changes:

Implements #9133
I implemented the fingerprint method for Event, Slot and TrackerWithCachedStates.
This already existed in Domain and StoryGraph so left untouched.
Instead added some missing tests for Domain.fingerprint() - StoryGraph had an existing test in test_graph.py

Status (please check what you already did):

[X] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[X] reformat files using black (please check Readme for instructions)

I thought it's testing if an identical event (i.e, with the same data) returns the same fingerprint 😅
I just tested with copy.deepcopy instead and the test passed

(For some reason I can't reply to this directly)
Yup I just meant that this won't test the instability of the builtin hash. As that only changes when starting a new python process.


Fix left/right padding documentation for format specifiers

Relevant source locations:

Positive values result in padding on the left: fmtstrformatter/parser.rs
Padding being applied: fmtstrformatter/mod.rs

Related to https://github.com/newsboat/newsboat/pull/744#discussion_r376712119

Coverage remained the same at 53.052% when pulling 78b6518a557f4919ab5927810d152a18d6c67fd4 on dennisschagt:feature/fix-format-padding-documentation into a7490633b007fdea50144a94c7d419de0feb4b42 on newsboat:master.

I checked the tests in our original release, 2.10, to see if C++ version behaved like this. It did. So this isn't a bug we introduced while porting to Rust.
Thanks for fixing this! I'll merge once the build has finished.


Updates

Update Cake
Update xunit.runner.console (previously forgotten)
Update libgit2sharp (bugfixes)

Deleting *.pdb files in all sub-folders before packaging

😻  Thanks for taking care of this!


ide+vscode: Show full function signature

Before it just displayed function name(), now it displays the whole signature.

Thanks @u9g I fixed it up 👍

thanks!


SublimeLinter pep8 PATH problems

Hi,
I'm trying to get SublimeLinter-pep8 working on SublimeText3 (I'm on windows)
I got this error from the console : SublimeLinter: WARNING: pep8 deactivated, cannot locate 'pep8@python'
I'm looking at the troubleshooting section( at sublimelinter.readthedocs.org). And to verify my path I'm typing, in windows command prompt, "where pip", but nothing is found.
I have already added C:\Python34\Scripts to my PATH.
Can someone help me ?
Thank you

You need to install pip and pep8.

I already installed pip.

Strange. And it doesn't show up when you run where pip? Could you run echo %PATH% and post the results? And also verify that pip.exe is in C:\Python34\Scripts?

I've investigated such problem myself before finding this issue and came up with a patch (#490) that should address this issue.


Parse error: result fied set in delta hunk

Hi,
I have no idea what I'm doing, and unfortunatly the documentation is light so I might be way off. But I feel like I've found a bug when trying to clone a remote git repo from mirage (compiled with the unix backend for now) :
module Main
  (C: Mirage_types_lwt.CONSOLE)
  (RES: Resolver_lwt.S)
  (CON: Conduit_mirage.S) = struct

  let start console resolver (conduit:CON.t) =
    let (module Context) = Irmin_mirage.context (resolver, conduit) in
    let module GitMem = Irmin_mirage.Git.Mem.KV(Context)(Git.Inflate.None) in
    let module Store = GitMem(Irmin.Contents.String) in
    let module Sync = Irmin.Sync(Store) in
    let config = Irmin_mem.config () in
    let upstream = Irmin.remote_uri "https://github.com/Ulrar/test-irmin.git" in
    Store.Repo.v config >>= Store.master
    >>= fun t  -> Sync.pull_exn t upstream `Set
    >>= fun () -> Store.get t ["README.md"]
    >>= fun r  -> C.log console r
end

Running that I get :
2018-07-08 17:49:43 +01:00: INF [tcpip-stack-socket] Manager: connect
2018-07-08 17:49:43 +01:00: INF [tcpip-stack-socket] Manager: configuring
2018-07-08 17:49:46 +01:00: INF [git.sync] remote: Counting objects: 3, done.
2018-07-08 17:49:46 +01:00: INF [git.sync] remote: Total 3 (delta 0), reused 3 (delta 0), pack-reused 0
2018-07-08 17:49:46 +01:00: INF [git.sync] Received a pack file of 226 bytes.
Parse error: result fied set in delta hunk
2018-07-08 17:49:46 +01:00: ERR [application] main: Mstruct.Parse_error("result fied set in delta hunk")
Raised at file "map.ml", line 122, characters 10-25
Called from file "src/core/lwt.ml", line 817, characters 20-60

Seems like there is at least a typo (I assume it's field and not fied), and maybe even a bug since the same code using Irmin-unix seems to work fine. Unless irmin-mirage is supposed to be used another way, I'm piecing it together by reading Canopy's sources and reading the doc, but a lot of it is magic to me so I might just be doing it wrong.
I most uncertain about Git.Inflate.None, maybe I'm supposed to use something else there, I saw a few issues mentionning it but everything got closed without any infos on what.

Well that's it, replacing Git.Inflate.None by Git.Inflate.M solves the issue. Although there is still a typo in the error, and at least another in the doc of ocaml-git ("Minimaal signature provided by Zlib.") but that's not the place for this.

The sync and push examples are now working fine on the master branch, thanks to the great work done by @dinosaure on ocaml-git. We are planning a new release pretty soon with the fixes.


How can I prevent showing Xcode warning <Result of call to 'done(on: flags:_:)' is unused>?

I am newbie in PromiseKIT. I am using version 6.
I get Xcode warning message <Result of call to 'done(on: flags:_:)' is unused> in some case that I dont care about the result of function and do nothing after finishing processing ( dont implement ensure / catch / finally )
My func is like this :
func test(id:String) -> Promise<Void> {
    return Promise { seal in

        if id == "1" {
           // fulfill
        } else if id == "2" {
           // reject
        }
    }
}

and I get the message when I call it
test(id: "1").done { // Xcode warning here
        print("Test OK")
        }

So Can you help me to prevent showing this Xcode warning message ?
Thank you very much for reading my issue

This is in the troubleshooting guide.

Troubleshooting Guide

Thanks so much ! You saved my day


UI Fixes: Sticky Parameters, and Plot log rationalization

The Windows GUI for plotting and the plot logs is in particular need attention.

It's a pain to repeatedly input all of the plotting parameters:  other than size, threads, and stripes, you have to specify number of plots, memory, stagger, bitfield, and temporary and final destinations.  These should be be sticky, prepopulated with whatever their last values were.  Given that once folks get a particular plot strategy in place they don't change it, it would seriously aid us if the params were sticky.

The plot logs are poorly designed. They have no consistent formatting and are both under-and over-inclusive in the data included.

a) Formatting. Ideally, the logs should be set up so they can be easily ingested into a database for analysis. This means consistent format in terms of fields and delimiters.   What we have are some fields separated by ":" others by tabs, others with spaces, others with "...." , others not at all.  The format  is entirely non-standard and requires custom code for parsing on ingest.  Some lines the logs are separated by 1 line break, others by 2 line breaks, in no consistent manner. Here is the start of a log:
Starting plotting progress into temporary dirs: F:\ChiaTempF and F:\ChiaTempF
ID: 847ed183e02f1feeaf7619d85fc6aa4364aacd872878892d23f4833e8df16d6e
Plot size is: 32
Buffer size is: 1000MiB
Using 128 buckets
Using 2 threads of stripe size 65536
Starting phase 1/4: Forward Propagation into tmp files... Mon Mar  1 18:42:34 2021
Computing table 1
F1 complete, time: 75.737 seconds. CPU (149.77%) Mon Mar  1 18:43:49 2021
Computing table 2
Notice the inconsistency:  "Plot size is:" and then "Using 128 buckets".   A normal design would be
Plot size: 32
Buckets: 128
Why is .... used in "Starting phase 1/4 to separate the date and time, but in the "F1 complete, no such delimiter is used.
b) The logs are over-inclusive--every "bucket" sort or write is logged.  And bucket lines are basically the same 128 lines, over and over again:
          Bucket 127 QS. Ram: 0.917GiB, u_sort min: 1.125GiB, qs min: 0.562GiB. force_qs: 0

The only thing that changes is the bucket number. And I have yet to find resource that explains what this is information is useful for in terms of analytics or even real time monitoring.
Remember: we look at the plot logs in real time to see actual progress. Having to scroll past hundreds and hundreds of lines of "Bucket..." details to find out what phase is going on it is waste of time and space.
Yet useful information--the actual plotting parameters (esp. number of parallel plots of which the current plot is included), as well as stagger, etc. are not included. If you are doing analysis, you need this and the only way to get it now it so manually add it after you ingest the data into a database.
c) Plot logs names: I have yet to figure out what the log files names are based on--I'm sure there's an explanation--but regardless they are needlessly long and complicated, and really mean nothing to the user.  A rational name would be based on the date and time the plot was started, because that's what humans remember, not arbitrary strings.
Recommendation.

Adopt either a comma delimited, or tab delimited file format.
2a) Either each line should have the format .  Data values should not include any text of units.
We don't need the bucket information. If you think the bucket detail is important, the provide a -verbose option in the UI that folks can select.
What information should be there? Again thinking analytics, I want to know the following:
--Log file name. Here, give a plot log a serial number (the jth plot run) followed by start date and time.  The value J is the same for all plots in this run (whether serial or parallel). This enables all of the logs for each plot run J to be linked together in a database and the performance metrics (time, CPU) averaged over all plots in the run.
-- Plot Run (again this is the J value)
-- Plot ID
-- Number of plots total in this run
--Plot size (and don't say "Plot size is:" "is"? Really?
-- Instance number N of this plot in this run
-- Series or Parallel: S or P
-- Stagger time
-- Memory allocation
--Threads
--Bucket size
--Bitfield setting
--Stripe size (never changes, but whatever)
-- Overall Start time:  Use a conventional date time format, please. mm/dd/yyyy, hh:mm:ss.ssss or similar.  Ideally something that can be natively recognized by Excel or other database software as a date.
-- Phase 1 total time (we don't need start and end time if you provide this; start and end are just means to the end of computing total time. Just give us total time)
-- Phase 1 CPU
-- Phase 2 total time
-- Phase 3 CPU
-- Phase 4 total time
-- Phase 4 CPU
--Total time (again we don't need the end time if we have total time.
--Total CPU

(Just to be clear the "--" above are NOT to be included in the file).

Don't put the "View Log" RIGHT NEXT TO "Delete Plot"!  That's terrible design. That's like the Army putting "View Log" next to "Launch Nuclear Missile." Sure there's a confirmation screen ("Are you sure you want to launch a nuclear warhead? "), but why create needless anxiety?  Yes View Log and Delete Plot are both actions, but find a way to separate them.

Why does "View Log" disappear after the plot is complete.  Sure I can go fetch the log file from the plotter directory, but there's no reason to kill the link?

Why is the View Log window modal and you only can view one at time?  The windows should be independent and non-modal so you can view several simultaneously.    (Of course, ideally, you'd have a dashboard that shows the actual run time status per the above so you can monitor all plots, but I'm not asking for that. )

Fixing the plot log format and stickiness are trivially easy changes in the coding and shouldn't take more than a couple of hours.
Thank you for your consideration.

Hi @specracer35 . Thank you for your comments. We will improve logs in the long run

A lot of these changes are for the chiapos repo

And are going to be a while before they're addressed. There are still fundamental changes to the plotting algorithm that mean that the best logging strategy isn't known yet.
This should be split into two issues and the logging moved to chiapos.


Is there anyway to ignore SSL validation?

In my case：
At the development environment, I use the dynamic routing with Consul，and all Services start up with HTTPS。
I got SSL connection error， like this：

Steps to Reproduce the Problem
ocelot.json config ：

Specifications

Version: 15.0.6

I tried resolve like this, and it worked.
By the way, In multiple configuration file case , just need to configure "DangerousAcceptAnyServerCertificateValidator" to "true" for this problem. But it doesn't work for dynamic routing.
So I‘m not sure it's the best way to resolve it.
I'm looking forward to your help, tks very much.
Here is my way.
Startup file
services.AddOcelot(Configuration).AddConsul().AddPolly().AddDelegatingHandler<IgnoreSSLValidateDelegatingHandler>(true);
IgnoreSSLValidateDelegatingHandler
`
public class IgnoreSSLValidateDelegatingHandler : DelegatingHandler
{
    private readonly X509CertificateCollection _certificates = new X509CertificateCollection();
    public IgnoreSSLValidateDelegatingHandler()
    {
    }
    protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, System.Threading.CancellationToken cancellationToken)
    {
        var inner = InnerHandler;
        while (inner is DelegatingHandler)
        {
            inner = ((DelegatingHandler)inner).InnerHandler;
        }
        // inner is HttpClientHandler
        if (inner is HttpClientHandler httpClientHandler)
        {
            if (httpClientHandler.ServerCertificateCustomValidationCallback == null)
            {
                httpClientHandler.ServerCertificateCustomValidationCallback = (message, cert, chain, errors) =>
                {
                    return true;
                };
            }
        }
        return await base.SendAsync(request, cancellationToken);
    }
}

`

@v2codes It is very strange that DangerousAcceptAnyServerCertificateValidator property did not help you.
Do you think the PR #1377 will solve the problem? I think soo...
Did you debug the code to figure out why dynamic routing switched off the DangerousAcceptAnyServerCertificateValidator property?


PlainEncoding uses raw value pointer to write data to disk.

Closes #113.
The reason of the bug was that Arrow re-uses the underneath buffer object for multiple arrays. Each array is with different offsets on the same buffer.  PlainEncoder used to write Array::data() which always pointed to the first batch.

I have not figured out how to use arrow to make unit tests (i.e., making multiple arrays share the same buffer) yet.


Instagram error : Unterminated string at 65631 [character 65632 line 1]

When launching RipMe on this IG :
https://www.instagram.com/janduxo
It stops and says
Error : Unterminated string at 65631 [character 65632 line 1]

Does it workfor you ?

Yes

It works completely ?
For me after 90~ photos it stops
What should i do ?

Confirmed it's not working.
It's messing up while trying to get the next page.

Found the issue, it seems that the json is randomly cut off, it's ends with :{"edges":[{"node":{"text":"How many siblings do you have?  </body></html>.

Looks like this is a bug with Jsoup it self
If you run the following code
URLConnection connection = new URL("https://www.instagram.com/graphql/query/?query_hash=" + qHash
                        + "&variables=" + vars).openConnection();
                connection.setRequestProperty("User-Agent", USER_AGENT);
                connection.setRequestProperty("x-instagram-gis", ig_gis);
                BufferedReader in = new BufferedReader(new InputStreamReader(connection.getInputStream()));
                String line;
                StringBuilder sb = new StringBuilder();
                while ((line = in.readLine()) != null) {
                    sb.append(line);
                }
                in.close();

sb.toString() returns the full JSON but Jsoup.parse(sb.toString())) only returns the first 65632 chars