real time control using mecode.G(direct_write=True, direct_write_mode="serial") with fix

Steps to reproduce issue

connect your computer (via USB2.0) to a controller board featuring a Microchip Atmega AVR microcontroller that is designed to reset upon serial connection and that runs marlin firmware.
Run a Python3 script (I use the Python 3.7 interpreter CPython under MS Windows 10 64 bit) which contains a mecode.G(direct_write=True) command, e.g. listing 1:

import mecode
g = mecode.G(direct_write=True, direct_write_mode="serial", printer_port="COM3", baudrate=115200)   # realtime communication via serial communication at port COMx (with x=3) under MS Windows. See https://github.com/jminardi/mecode/blob/master/mecode/main.py .
g.write("M302 S0")   # send g-Code. Here: allow cold extrusion. Danger: Make sure extruder is clean without filament inserted
g.move(10, 10, 10)   # move 10mm in x and 10mm in y and 10mm in z
g.retract(10)   # move extruder motor

Expected behaviour:

I expected that all stepper motors would move
I would like to find documentation on how to create a socket and get its port number in order to use it with mecode.G(direct_write=True)  as an alternative

Observed behaviour:

Missing documentation on how to use mecode.G(direct_write=True)
console output: G91  and some Error
controller board resets, but no motor movement.

Possible Solution:
Use my fork of mecode which provides fixes (see branch serial_fix)
and add the following lines of code after the last command of the Python script that communicates with the controller board (e.g. append to listing 1, see above):
import time
time.sleep(12)

If someone confirms my issues I can create a pull request

The following works for me:

    # mecode object
    g = G(
            print_lines=False,
            aerotech_include=False,
            direct_write=True,
            direct_write_mode='serial',
            printer_port="/dev/tty.usbmodem1411",
            #printer_port="COM13",  #use for windows host machine
    )

This is on python 2.7, although I think it should work with python 3.x
What is the fix you mentioned to get it to work?

The diff is here:
https://github.com/jminardi/mecode/compare/master...DerAndere1:serial_fix#diff-7ffff7cfc8c0098f97023392516340e0L125
I guess for full python 3 compatibility, my key change was to add .decode("utf-8") to main.py line 259 and line 358 :
r = self.get_response("M114").decode("utf-8")
line = self.s.readline().decode("utf-8")

Instead of those changes, for python 2 compatibility I guess (not tested) a better solution could be to change statement_in to statement) in main.py lines 972 and line 974 . Then your encode2to3() and decode2to3() code in main.py would do the encoding depending on python version, right?
My second problem was that your loop in printer.py line 125 did recognize when my MCU finished restarting and the timeout stoped the serial connection. So I removed that loop and instead  changed printer.py line 114 from:
self.s = serial.Serial(self.port, self.baudrate, timeout=3)
to:
self.s = serial.Serial(self.port, self.baudrate, timeout=6)
sleep(5)

I updated my serial_fix branch and edited my comment above accordingly. Tested with Python3 only but It uses decode2To3 so Python 2 should also work.

At least with Python 3.x, pySerial's serial.read(data) and serial.write(data) requires that data to be transferred is a bytes object (or compatible type such as bytearray) . That is why for all occurances of s.write() and s.read() in mecode, the data to be sent should first be encoded like this: data.encode('utf-8') and received byte data has to be decoded to obtain unicode strings: data.decode('utf-8'). Along those lines, the tests have to be adjusted to make all data is always a byte object.:
https://github.com/jminardi/mecode/blob/13eef1e2a173584badf9a3ac0f231497fd5945c6/mecode/tests/test_printer.py#L24
should read
        self.p.s.readline.return_value = encode2to3('ok\n', utf-8)
https://github.com/jminardi/mecode/blob/13eef1e2a173584badf9a3ac0f231497fd5945c6/mecode/tests/test_printer.py#L57
        self.p.sendline(testline)
https://github.com/jminardi/mecode/blob/13eef1e2a173584badf9a3ac0f231497fd5945c6/mecode/tests/test_printer.py#L56
should read
        self.p.s.write.assert_called_with(encode2to3('N1 no new line*44\n'))
similarly, following lines have to be changed:
https://github.com/jminardi/mecode/blob/13eef1e2a173584badf9a3ac0f231497fd5945c6/mecode/tests/test_printer.py#L63
https://github.com/jminardi/mecode/blob/13eef1e2a173584badf9a3ac0f231497fd5945c6/mecode/tests/test_printer.py#L66

@DerAndere1 Do you believe your suggested changes are still needed? I do not current have access to a serial connected 3d printer


Separate game mode code into server and client side code versions.

Until now the overall architecture of gamemode header files is the following:
you have one file (like ctf.hpp) which contains the game mode code for both sides: server and client.
Conditional compiling is used to generate server and client side code like in this invented example:
class CTF
{
    clientinfo get_flag_holder();

    #ifdef SERVMODE
        void kick_flag_holder();
    #else
        void takeflag();
        void dropflag();
    #endif
};

this conditional compiling block represents an inheritage relation. This means there should be a class concept for this. The elaborated code block would be the following:
class CTF
{
    clientinfo get_flag_holder();
};

class CTF_serverside : public CTF
{
    void kick_flag_holder();
};

class CTF_clientside : public CTF
{
    void takeflag();
    void dropflag();
};

Also these classes could be separated into 3 files.
This would be a great replacement for the #define SERVMODE header file includes that can be found everywhere in the code.

I did thought about this for quiet a while now .. can we do the following:

implement a `pseudo-gamemode``
add a naming or identify property for the gamemode
let it up to the tree to decide the entire settings of the gamemode (e.g: physics, teammode ...)
when a player joins a server, it's mode tree information is synchronized via the api
that's it. single gamemode, unlimited possibilites.

Before we can do something like that we have to find ways how to prevent cheating.

Seeking them on the client side in a truly hackable game seems clearly the wrong path to go down..

I didn't say a word of implementing anti cheats methods in the client.
Still, this is going heavily towards scriptable game modes and if we implement that into the client without a plan for anti-cheat, it will be even more easy to cheat for quite some time. Not something to look forward to.

I do think that is the price we pay for a truly modable game .. and implementing anti cheat is non of public business .. @aschaeffer has been measuring anti cheat systems for server side which do work quiet well. On the other hand the server is as well modable making it easier to write comprehensive anti cheat systems
Am 12.01.2017, 23:23, Croydon<EMAIL_ADDRESS>schrieb:
I didn't say a word of implementing anti cheats methods in the client.
Still, this is going heavily towards scriptable game modes and if we implement that into the client without a plan for anti-cheat, it will be even more easy to cheat for quite some time. Not something to look forward to.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

and implementing anti cheat is non of public business

I disagree. The implementation is open so should be it's development.
But we are getting more and more Off-Topic. I just wanted to note, that we should do something to make cheating harder before a stable release, when we currently making it even more easier.

Wouldn't we avoid that discussions about cheats and anti cheats? At least until we get to a point where it makes sense.

What is with the idea of a "mother" game mode, does it sound reasonable?

By the way, anti-cheat also refers to #167

This ticket is only about the refactoring the mega-strange game modes (ctf.h, hold.h, ``) which mixes server code and client code and makes them confusing, unclear and indivisible.
Non-goals of this ticket:

Anti-Cheats
Mother-Gamemode

In fact this work is a prerequisite before we can implement a dynamic system for game modes (aka "mother" game mode).


[v1.9.x] ONNX fix node output sort

Description
Backport #20326.
Checklist
Essentials

[ ] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] Code is well-documented

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

the pylint warning basically means that len(name) are keep being redefined in each loop iteration. refer to https://stackoverflow.com/questions/25314547/cell-var-from-loop-warning-from-pylint
this is actually what we intended but to make it more formal we can try to change
def str2int(s):

to
def str2int(s, name):

the pylint warning basically means that len(name) are keep being redefined in each loop iteration. refer to https://stackoverflow.com/questions/25314547/cell-var-from-loop-warning-from-pylint
this is actually what we intended but to make it more formal we can change
def str2int(s):

to
def str2int(s, name):
...

node_output_names = sorted(node_output_names, key=lambda x: str2int(x, name))

Thanks for the heads-up!


NULL dereference after failing to save job when unloading completed jobs

Version: 2.1-current
CUPS.org User: twaugh.redhat
This bug appears to have been present for quite a long time.
When cupsdUnloadCompletedJobs() calls cupsdSaveJob(), it does not handle failures correctly.
If saving the job fails (eg no space), job->dirty will remain set. However:
   if (job->dirty)
     cupsdSaveJob(job);

   unload_job(job);

unload_job() will be called regardless of whether the job is now dirty. This later causes a NULL dereference when cupsdCleanDirty() calls cupsdSaveJob() again.
I think the fix would be to only call unload_job(job) if the job is now not dirty:
   if (job->dirty)
     cupsdSaveJob(job);

   if (!job->dirty)
     unload_job(job);

[master 233715a] The scheduler did not handle out-of-disk situations gracefully (Issue #4742)


chore: upgrade dependencies

[x] upgrade flutter: set dart and flutter version of environment
[x] major upgrade of packages
[x] fix lint issues
[x] gen (no diff)

Upgraded packages:
https://pub.dev/packages/reactive_forms/changelog
https://pub.dev/packages/entry/changelog
https://pub.dev/packages/reactive_forms_annotations/changelog
https://pub.dev/packages/rxdart/changelog
https://pub.dev/packages/theme_tailor_annotation/changelog
https://pub.dev/packages/reactive_forms_generator/changelog
https://pub.dev/packages/theme_tailor/changelog

by checking the changelogs it seems to safe to upgrade


customize: change email templates

Could be a new section under "Customize > Stlying > Change email templates"

List of file names to replace
List of template blocks and variables

Now at https://inveniordm.docs.cern.ch/customize/emails/


Deprecate tools

Old tools should be deprecated and changed by a new tool that does the previous job and maybe more:

[ ] BamBarcodeDetector, FastqBarcodeDetector and TaggedBamBarcodeDetectorshould be changed for a more general barcode detection tool independently of the input (BAM/FASTQ or tagged BAM).
[x] QualityChecker  for a tool that use the new IO system to allow all data sources allowed in ReadTools.
[x] StandardizeQuality for a tool that use the new IO system and output only SAM/BAM/CRAM files.
[ ] TrimFastq for a more flexible tool using trimmers/filters by the GATK4 plugin system.

Tools should be annotated with @Deprecated, changed the program group for a new one (DeprecatedProgramGroup) and add a short sentence about the new tool to substitute it and usage if it is very different.
Deprecation is for beta. Removal should be for version 1.0.

QualityChecker deprecated in c4aa1941f98ec15ee0ff9a620c1351f68a0cb883

StandardizeQuality deprecated in cef5cef470a79a30bc0e2269adc97b36a24990e1 (needs documentation in README/CHANGELOG)

Documentation for StandardizeQuality in README/CHANGELOG fixed in e0a4b7315fc05cfe8f97837a6dad98fd2ac6c3b5

Barcode tools deprecated in 405b49c9f310dfc3d3d1c5d99b011a68ae6727d9

TrimFastq deprecated in 53cd8ce56985b585b4baec8232bf3a2434358700

Removal of deprecated tools tracked in #94. Closing.


Implement word boundary checks in regular expressions

In the current implementation of regular expressions intended to match the GLSL keywords "in", "out", and "varying", it has been found that the patterns may match parts of variable names or within comments that contain these keywords, leading to inaccurate results.
To improve accuracy, word boundaries (\b) have been applied around each keyword in the regular expressions. This ensures that the engine would only match complete words, not substrings within longer words or comments.
The regular expressions have been updated as follows:
const outRegex = /(flat\s)?\s*\bout\b /;
const inRegex = /(flat\s)?\s*\bin\b /;
const varyingRegex = /(flat\s)?\s*\bvarying\b/ ;

While this improves the keyword matching accuracy, please note that particularly advanced cases like keyword usage inside comments might still not be handled perfectly. For deeper parsing needs, consider using a dedicated GLSL parser or compiler plugin.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Thanks!


Aliases for runParser and unParser at top-level module

It would be nice to have quick aliases for calling run/unParser from the ReludeParse module.

done in v0.6.0


Add url encoding to Whisk Object in Swift 3 container

This fixes Issues #1777, #1752.

add url encoding to the uri path of the Whisk object's REST calls to avoid issues
remove errant log statement in Whisk object that printed out communication details

PG Blue 970

Yes, it will encode the whole uri path.


pageserver: use a single tokio runtime

Before this PR, each core had 3 executor threads from 3 different runtimes. With this PR, we just have one runtime, with one thread per core. Switching to a single tokio runtime should reduce that effective over-commit of CPU and in theory help with tail latencies -- iff all tokio tasks are well-behaved and yield to the runtime regularly.
Are All Tasks Well-Behaved? Are We Ready?
Sadly there doesn't seem to be good out-of-the box tokio tooling to answer this question.
We believe all tasks are well behaved in today's code base, as of the switch to virtual_file_io_engine = "tokio-epoll-uring" in production (https://github.com/neondatabase/aws/pull/1121).
The only remaining executor-thread-blocking code is walredo and some filesystem namespace operations.
Filesystem namespace operations work is being tracked in #6663 and not considered likely to actually block at this time.
Regarding walredo, it currently does a blocking poll for read/write to the pipe file descriptors we use for IPC with the walredo process.
There is an ongoing experiment to make walredo async (#6628), but it needs more time because there are surprisingly tricky trade-offs that are articulated in that PR's description (which itself is still WIP). What's relevant for this PR is that

walredo is always CPU-bound
production tail latencies for walredo request-response (pageserver_wal_redo_seconds_bucket) are

p90: with few exceptions, low hundreds of micro-seconds
p95: except on very packed pageservers, below 1ms
p99: all below 50ms, vast majority below 1ms
p99.9: almost all around 50ms, rarely at >= 70ms
Dashboard Link

The ones below 1ms are below our current threshold for when we start thinking about yielding to the executor.
The tens of milliseconds stalls aren't great, but, not least because of the implicit overcommit of CPU by the three runtimes, we can't be sure whether these tens of milliseconds are inherently necessary to do the walredo work or whether we could be faster if there was less contention for CPU.
On the first item (walredo being always CPU-bound work): it means that walredo processes will always compete with the executor threads.
We could yield, using async walredo, but then we hit the trade-offs explained in that PR.
tl;dr: the risk of stalling executor threads through blocking walredo seems low, and switching to one runtime cleans up one potential source for higher-than-necessary stall times (explained in the previous paragraphs).
Code Changes

Remove the 3 different runtime definitions.
Add a new definition called THE_RUNTIME.
Use it in all places that previously used one of the 3 removed runtimes.
Remove the argument from task_mgr.
Fix failpoint usage where pausable_failpoint! should have been used. We encountered some actual failures because of this, e.g., hung get_metric() calls during test teardown that would client-timeout after 300s.

As indicated by the comment above THE_RUNTIME, we could take this clean-up further.
But before we create so much churn, let's first validate that there's no perf regression.
Performance
We will test this in staging using the various nightly benchmark runs.
However, the worst-case impact of this change is likely compaction (=>image layer creation) competing with compute requests.
Image layer creation work can't be easily generated & repeated quickly by pagebench.
So, we'll simply watch getpage & basebackup tail latencies in staging.
Additionally, I have done manual benchmarking using pagebench.
Report: https://neondatabase.notion.site/2024-03-23-oneruntime-change-benchmarking-22a399c411e24399a73311115fb703ec?pvs=4
Tail latencies and throughput are marginally better (no regression = good).
Except in a workload with 128 clients against one tenant.
There, the p99.9 and p99.99 getpage latency is about 2x worse (at slightly lower throughput).
A dip in throughput every 20s (compaction_period_ is clearly visible, and probably responsible for that worse tail latency.
This has potential to improve with async walredo, and is an edge case workload anyway.
Future Work

Once this change has shown satisfying results in production, change the codebase to use the ambient runtime instead of explicitly referencing THE_RUNTIME.
Have a mode where we run with a single-threaded runtime, so we uncover executor stalls more quickly.
Switch or write our own failpoints library that is async-native: https://github.com/neondatabase/neon/issues/7216

Nightly pgbench runs in staging look unchanged 👍


Improve functions and string building process.

I watched your amazing project in reddit. I hope my full request will help.

Before you merge it, you should change preview image, as I aligned informations.

Dude amazing PR, i was not able to rewrite it to this FP kinda style.

I watched your amazing project in reddit. I hope my full request will help.
Unfortunately this project didn't go as viral as i expected and we didn't even hit 100 stars 😓

Haha... I also don't know what kind of project people will like. Have a good time with FP!


Keyboard Shortcut

Is it possible to add a keyboard shortcut which will go to address bar & type tab ?

That would be amazing!

@cyrill62 @ArgonQQ
Question: are you already familiar the keyboard shortcut ⌘+L (Mac) or Ctrl+L or Alt+D (Linux/Windows) to go to the Location Bar without leaving the keyboard?

Yes I am. But I still have to type "tab ". Is it possible to create a shortcut which skips the tab  step? Or is it too much effort?

I will look into it.
By the way, I noticed that chrome now show by default an icon for all extensions, until the user explicitly hide it.
So there is this a new UX, especially for new users, and we should think about how to use it.
Teaching about the new keyboard shortcut is a good candidate of what it can do

Being unfamiliar with the chrome extension API myself and what is possible, it would be quite nice to have a hotkey if possible - I gave it a +1.

In manifest.json:
...
"commands": {
  "NAME_OF_THE_FUNCTION": {
    "suggested_key": { "default": "Alt+Shift+T", "mac": "Command+Shift+T" },
      "description": "TabTab Search"
    }
  }
}
...

But I don't know if it's possible to call the Omnibox function?
like the Ctrl+E for default search.
Any ideas?


FormatTextGenerationSFT with function calling

Description
⚠ Work in progress
This PR improves the FormatTextGenerationSFT task to allow preparing fine tuning datasets with function calling.

Closing as there's no current standard on the format for function calling SFT


Review and augment set of implicit imports

Gradle provides a service that exposes what the implicit imports should be: org.gradle.configuration.ImportsReader. It allows to get the list advertised here: https://docs.gradle.org/current/userguide/writing_build_scripts.html#script-default-imports
We should reuse that.

This is now on master.


Detect that window isn't minimized more often

Fixes https://github.com/DaemonEngine/Daemon/issues/408 and https://github.com/Unvanquished/Unvanquished/issues/1136.
Could fix https://github.com/DaemonEngine/Daemon/issues/365 and https://github.com/DaemonEngine/Daemon/issues/234.
If it doesn't, it will at least give an easy workaround: switch away from and back to the window.

To expand a bit about why this didn't work, the window was reported as unfocused in the cvar. Because of that, IN_Frame calls IN_SetFocus(false) and that's apparently what changes input handling, blocking grab and showing a cursor instead.

I don't have a way to test right now, but now while you talk about this the previous code is looking to be waiting for that! Nice finding! :tada:

I don't have a way to test right now, but now while you talk about this the previous code is looking to be waiting for that! Nice finding! :tada:

I confirm it fixes the input issue on my end (I still have to sometime click the windows to get my mouse get captured again, but that's not a big deal, no tty console-based vid_restart is required to unstuck the game anymore).

I confirm it fixes the input issue on my end (I still have to sometime click the windows to get my mouse get captured again, but that's not a big deal, no tty console-based vid_restart is required to unstuck the game anymore).


How to read a specified sheet from .xlsx file by it's sheet name.?

AlaSQL is based on unpaid voluntary work. Thank you for taking the time to make it better.
Question about how to ...

Please use http://stackoverflow.com/questions/ask?tags=alasql

Something is not working as expected:

Describe the problem
Provide code that replicates the problem
We suggest to spawn a jsfiddle from http://jsfiddle.net/98qL7m0c/

Refer to read-from-xlsx.
Make sure you include the following resources:

Then you can access xlsx like this:
alasql.promise('select * from xlsx("file.xlsx",{sheetid:"Sheetname"})')
.then(function(data){
console.log(data);
}).catch(function(err){
console.log('Error:', err);
});


Fix help.github.com link checks

For some reason, help.github.com will only serve compressed content, and
will reply with a 403 error without any "Accept-encoding" headers.
This causes 'make docs_check_links' to report help.github.com links as being
dead.
This issue is tracked in https://github.com/github/docs/issues/17358
For this fix to work, quay.io/crcont/docs-builder:latest must be rebuilt
and pushed to quay.
This can be rebuilt and tested locally with:
$ (cd images/docs-builder && podman build -t quay.io/crcont/docs-builder:latest .)
$ make docs_check_links

Why no github action run on this PR?

Maybe because it was initially based on origin/master which has no github actions instead of origin/main

(NB: the "build and publish docs" failure is expected as the quay.io docs-build image hasn't been updated yet with the changes from this PR)


youtube video, custom size

hi @joanaz
can we specified the youtube video resolution? lets say, playing 360p for all the video.
the reason is, i dont want the video cover up the entire screen, and block other module.
kindly advise, thanks
Cheers

can this module play youtube?


Prevent mining on stale block candidate

Delivers https://www.pivotaltracker.com/story/show/157929767
TODO:

[x] Add unit test if not too much effort
[ ] Check Python UAT tests and system tests pass

CI workflow on commit 3f828580 equivalent to 0eb5b447 UPDATE: Python UAT tests passed

[ ] Consider restructuring events UPDATE: Refactoring avoided as too involved. If needed for changing behaviour of any modules / subsystems it can be done in future PRs.
[ ] Squash-and-merge

One idea I had when looking at this would be to refactor the events a bit. top_changed could be really that the top changed, and we could have one event that means "we should spread this block" (current top_changed + block_created) etc... But this will work!

@hanssv I considered that too though I feel it may be a bit too involved (I even see we have tests checking on these events). I suggest not to go with that refactoring in this PR.

@hanssv @uwiger @velzevur I consider this ready to be reviewed.

I am investigating why system tests failed for this branch.


ERROR: ./bin/provision_cf

git submodule update --init --recursive --force  ERROR:
fatal: insanely long object directory
screenshot:

thanks!

Windows historically doesn't like long file paths. You can try enabling support within git by...
git config --system core.longpaths true

@dpb587-pivotal
i set already, it not works.

I see you've raised the issue upstream at https://github.com/git-for-windows/git/issues/762 and https://github.com/git-for-windows/git/issues/342. This issue can probably be closed.


ebitenutil: NewImageFromFile was not defined on Wasm

https://github.com/hajimehoshi/ebiten/blob/master/ebitenutil/loadimage.go#L15
The build tags missed js!
(GopherJS worked since it defines not only js but also the tag for the host OS).

#1088 PR Submitted


fix fix the problem of parameter mismatch when printing logs

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Html::radioList improvement

Is this possible to make set of the some values disabled?

<?= $form->field($model, 'command')->radioList($model->getCommands(), ['itemOptions' => ['disabled' => true]]) ?>
See source code \yii\helpers\BaseHtml::radioList()

@dimka3210 ['itemOptions' => ['disabled' => true]] will disable all radio buttons in a radio list, if one just want to disable a subset of the radio list, the item option would be better.

@drodata, yes. I inattentively read topic.

@dimka3210 ['itemOptions' => ['disabled' => true]] will disable all radio buttons in a radio list, if one just want to disable a subset of the radio list, the item option would be better.

How to disable item.


compile error: cast to Crystal::ClassVarContainer failed when scope is VirtualType

Hello,
Wasn't sure how to title this issue and still unsure if this describes the issue I'm hitting :smile:
I've reached the following compiler crash:
cast to Crystal::ClassVarContainer failed
*Exception#initialize<Exception, String, Nil>:Array(String) +14 [4456698206]
*Exception::new<String>:Exception +89 [4456698153]
*raise<String>:NoReturn +6 [4456504902]
*Crystal::TypeVisitor#lookup_class_var<Crystal::TypeVisitor, Crystal::ClassVar, Bool>:Crystal::Var+ +787 [4466278451]

Since the backtrace is a bit long, I've managed to synthesize the problem to be a casting issue when accessing class variables of a VirtualType.
The following code triggers this issue. I have no doubt it could be further reduced for a test case, but I'm far beyond my level of experience with Crystal internals:
abstract class Todo
  macro inherited
    @@items = [] of String
  end

  def list
    @@items
  end

  macro add(something)
    @@items << {{something}}
  end
end

class Home < Todo
  add "dishes"
  add "laundry"
end

class Work < Todo
  add "code"
  add "push"
  add "deploy"
end

todos = [
  Work.new,
  Home.new
]

# todos.class = Array(Todo+)
pp todos.class

todos.each do |todo|
  # HERE: compile error when class variable is accessed
  todo.list.each do |item|
  end
end

https://github.com/manastech/crystal/blob/master/src/compiler/crystal/semantic/type_inference.cr#L364
Please note that accessing the class variables is not an issue, it only turns into an issue when both are combined into an Array:
# this compiles without issues
todo = Home.new
todo.list.each do |item|
  puts item
end

Hope all the above makes sense and helps figure out where the issue is coming from.
Thank you in advance.
:heart: :heart: :heart:

Thanks for the detailed report!!
Yes, it's something that we didn't think about and must solve. In fact it's a duplicate of #916, so your title is a perfect description of the problem :-)
I'm actually glad that lately some bug reports are duplicates: it maybe means that we don't have many bugs but a few recurring ones. It also means we must fix them soon ;-)

@asterite ohh GitHub-search-fu failed on me, which is why I submitted the bug report in the first place.
Thank you for the fast response!
Indeed having repeated bug reports is a good thing :smile:


FT6X36 driver uses hardcoded I2C port number.

Replaced hard-coded I2C_NUM_0 by TOUCH_I2C_PORT define.

Thanks for the fix @DimDK. Did the new repository gave you any other issue?

@C47D  No, I didn't see any other issues.


Add suggestion ranking heuristics

Depends on https://github.com/nwolverson/purescript-ide-purescript-core/pull/13
The ranking format is kind of weird. VSCode uses rankText to affect ordering (actual ordering of the results has no effect), so I'm just returning  "a" or "z", where "a" forces a higher ranking.

I've made a few more additions.
Ranking based an segment prefix:

Ranking based on all caps abbreviations:

I'm testing this with Atom, but it doesn't appear as though the atom language-client observes the sortText, and the manual ordering does not seem to have any effect on the ordering Atom ultimately chooses.

This was supposedly added a while back (https://github.com/atom/atom-languageclient/commit/afef39bc5c1a551b425f78fe5e6bdac5cce266e7), but I can't get it to respect my ordering the same way VSCode does.

@natefaubion as long as we've not broken it entirely, I'm happy to look at that later

as long as we've not broken it entirely, I'm happy to look at that later

It's definitely not broken, and it does seem to influence it somewhat (and it's a little better than without it), but it's not clear to me how. It's definitely not just sorting on sortText though because I can confirm that it our language-server does sort the inferred modules first, and the Atom UI ultimately displays Data.Set first when using the Map.fromF example, and Data.Map second.

What's the intent of the last change?
This generally feels pretty good to me, I do see some differences in a couple of cases - MonadRec.tai doesn't suggest tailRec over tail but Rec.tai does.
Map.fromFo seems illustrative of how this all works out

Are you happy with this now? Setting aside the Atom question for another time.

The last change was because List.<something> would rank things like Data.List and Data.List.NonEmpty equally. That is because it's matching the same segment offset from the start. The change should rank Data.List higher because there are fewer segments after it.

MonadRec.tai only works if you've previously imported class MonadRec. MonadRec is not a prefix of an existing segment, and is not an all caps abbreviation of the segments. I could do an additional ranking based on concatenating all the segments and doing a substring search, which would handle this case at least. Wouldn't hurt.

Alright, it will now rank something like MonadRec without a previous import.

Merging this before the ranking becomes self aware.


ng-cloak, ngCloak do not work in IE

Here is some more data on the ng-cloak / ngCloak problem in Internet Explorer (including IE8, IE9, IE10).
The problem appears to occur in IE9 and IE10.  IE10 is not a legacy browser, so the Angular docs for ng-cloak do not sufficiently address this issue.
Also, adding the ngCloak class does not appear to fix the problem in IE8 or IE10.
Adding "style='display: none;'" to the element does fix the problem in IE8 and IE10.
Imported from discussion on mailing list: https://groups.google.com/forum/?hl=en-US&fromgroups=#!topic/angular/5ZMPyg5qtEM

This helped me:
https://stackoverflow.com/questions/27802094/ng-cloak-has-not-effect-whatsoever-on-blinking-angular-code
Spoiler: add the following code, because "If your angular.js file is in any way async, then it doesn't have a chance to load and add the styles before the page finishes rendering, and thus you'll see the curly brackets. That's why you must inline that ngcloak style at the top of the page".
[ng\:cloak], [ng-cloak], [data-ng-cloak], [x-ng-cloak], .ng-cloak, .x-ng-cloak {   display: none !important; }


Mwm more unit tests

Adding unit tests for nearly all of lib/*.py (except poms_wrap.py, which isn't used yet,
and dagnabbit.py which is going to need some test files, etc. )
Still also need unit tests for the subroutines in jobsub_submit itself.
Also a bugfix for return values in creds.py/get_creds() and putting back broken arg
change in get_parser.py which made DESIRED_usage... break.

This looks great. I think I just had that one comment on the parser unit tests (test_get_parser_unit.py).

I think that's addressed now.

Okay, I've added more blow-by-blow comments to the get_parser tests. It was a good exercise, I figured
out I could drop a 4-line clause in the code :-).
On Tue, 2022-06-14 at 14:44 -0700, Shreyas Bhat wrote:
@shreyb commented on this pull request.
In tests/test_get_parser_unit.pyhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_marcmengel_jobsub-5Flite_pull_47-23discussion-5Fr897347523&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=mA4qFs7Hfq-joWhWXnsXfA&m=QHB7enaDGiLbNT0wYsUiYfUrA4pvAUvUgX-lfSYWUMmLRGIM_c6kCX4UtEfnBHgX&s=KwO0oTTUkTjm1OYfZcgktTqizxGma2R_uoWqU_btGKo&e=:

+def find_all_arguments():

try to extract all the --foo arguments from the source

and track which ones are flags

f = open("../lib/get_parser.py","r")

flagargs = set()

listargs = set()

allargs = []

dest = {}

for line in f.readlines():

   p = line.find('"--')

   mq = '"'

   if p < 0:

      p = line.find("'--")

      mq = "'"

   if p > 0:

Whoo - this is a doozy, but short of using regexes, I can't see anything wrong with testing for both the " and ' cases explicitly.
Would you mind adding a quick one-line comment for each if statement in here just so we know, 2 years from now, what these clauses are doing (e.g. lines 41-42 are great)? e.g. If we find "dest=", save the value in dest dict. -d and -f you could probably explain together, as well as the store and append clauses. I'm just worried about us forgetting what all this does a while from now.
—
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_marcmengel_jobsub-5Flite_pull_47-23pullrequestreview-2D1006616257&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=mA4qFs7Hfq-joWhWXnsXfA&m=QHB7enaDGiLbNT0wYsUiYfUrA4pvAUvUgX-lfSYWUMmLRGIM_c6kCX4UtEfnBHgX&s=YqnkMbp12jOEfZwGIWvyEbur__sfSZ4uzeAGrZe2bb8&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AAB6GVY5P3XEV77GBTVAKT3VPD4MNANCNFSM5YOGAZOA&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=mA4qFs7Hfq-joWhWXnsXfA&m=QHB7enaDGiLbNT0wYsUiYfUrA4pvAUvUgX-lfSYWUMmLRGIM_c6kCX4UtEfnBHgX&s=KfFC2u7bUUC6x25EK-dZOVcpbwfaOpI2u-tZsUxJBy4&e=.
You are receiving this because you authored the thread.Message ID: @.***>


System.badimageformatexception in minkowski DLL

Hi guys, I need your help.
I'm compiling the DLL as the example described by fel88, it is correctly generating the dll.
When I put the dll together with DeepNestPort.exe folder and run the application the exception is generated.
I tried to change the dll path, but the error remains.
Has anyone been through this, or has any idea to help me.
Do I need to install something else?
Thanks.

Hi, you probably have x86/x64 mismatch problem.
Try switching the compiler to x64
https://github.com/fel88/DeepNestPort/issues/2
You can also try to set 'platform target' to x86

C isn't in my comfort zone and I've been having fun with the DLL compilations too. I'd concur with the suggestion to try flip over to x86 and found a simpler command in compile.bat works in some cases where the full didn't;

cl /Ox -I "D:\boost\boost_1_62_0" /LD minkowski.cc

Sorry, I don't have the reference of where I got that suggestion from; it's in one of the DeepNestPort or DeepNest issues.

Hello guys, thanks so much for help.
Fell88, your tip solved my problem. I did some changes im my project to use x64.
When I published to instal I have beem change in the TragetPlatform too.
In select the setup project > properties and change the TragetPlatform to x64.
Thanks.


Add ECrypto Coin (ECP)

Project background details:
ECrypto Coin (ECP Coin) was developed on the Ethereum Blockchain so its accepted at all Smart Contact vending points. So far the coin has been listed among the default coins at MyEtherwallet.com, goalabizexchange, listing on the Birrion exchange app in UK is under process, and Token.Store. More exchangers to be announced shortly.
Team background:
MR. SAMUEL ADU-YEBOAH (FOUNDER): is a young entrepreneur and a cryptocurrency expert who started his network markeng career in the year 2013. Telegram: https://t.me/goalanissi
EMMANUEL OSEI KOFI (CO-FOUNDER): He is an online entrepreneur with a BSc. Telecommunication Engineering from the Ghana Technology University College. He has been a network marketer since 2010 and a part of many successful online investments and network marketing programs. He is a Blockchain and Cryptocurrency analyst, trader and strong advocate for cryptocurrencies.  Telegram: https://t.me/osei94
MR. BISMARK KOFI ADU-LARBI (DIRECTOR/STRATEGY): Holds a BSc. Informaon Technology Degree from the University of Ghana through the Pentecost University College. He specialises in Biometric Systems, Surveillance Systems and Vehicle Security System. He is currently the Director and in charge of Strategy at ECP Ecryptopay Company Limited. Telegram: https://t.me/Bislarb
Current situation of the project:
The ideal purpose of the coin is to use it as a medium of exchange for investors for ecryptopay.com worldwide. Projects in the pipeline - ECP Store, Our own crypto trading platform, Android and Desktop wallets, merchant/vendors exchange points around the globe among others
Media exposures: INSTAGRAM: https://www.instagram.com/ecpecryptopay, TWITTER: https://twitter.com/EcpEcryptopay, FACEBOOK: https://www.facebook.com/ecpcoin/
Tradeable exchanges: TOKENSTORE (https://token.store/trade/ECP)
Recommended Gas Limit (For ERC-20 project teams) for transaction: (Default is 60,000)
Gas Limit: (Default 60,000)

logo is not qualified, the background should be transparent.


Graphite URI scheme must be either net.tcp or net.udp or net.pickled (Parameter 'BaseUri')

Trying to report to Graphite but getting the error 'Graphite URI scheme must be either net.tcp or net.udp or net.pickled (Parameter 'BaseUri')' each time (see image 1) below.
I have tried setting the option 'options.Graphite.Protocol' but this appears to be Read Only (see image 2 below).
Any ideas?
(Please note: I am using .NET Core 3.0 rc1 so hoping this is not the issue)
I am running the latest grafana and graphite in docker containers alongside Visual Studio.
Happy to provide more info if required.
Thanks,
Martin

Hey, @mabublue. As I see, your graphite uri is http based. As you can see from error message Graphite URI scheme must be either net.tcp or net.udp or net.pickled (Parameter 'BaseUri'), your scheme should be net.tcp, net,udp or net.pickled. The definite scheme depends on your graphite configuration.
So, your graphite address should be net.tcp://<IP_ADDRESS>:2003, net.udp://<IP_ADDRESS>:2003 or net.pickled://<IP_ADDRESS>:2003.

@vhatsura Thank you!! I changed http to net.udp and seems to be working. I'll test further tomorrow evening to ensure I can report the metrics, but I am no longer getting an error.
I was following the examples in the documentation which clearly show http in the url (see image below).
Anyway, Problem solved and closing comment.
Martin

Could someone update the documentation because I just had the exact same misunderstanding ?


Cannot read property 'lookup' of undefined w/ the latest 2.0 build

Team
First off -thanks for such a great OSS project! I truly stand on the shoulders of giants today!
I'm using the latest build w/ ember-cli (here is my package.json)
    "ember-validations": "2.0.0-alpha.2",

The problem statement: I get an explosion trying to boot a simple ember-cli example app. I'm trying to verify if I missed a step/ or if the README is out of date. I hope this is a helpful contribution to those looking at the library in the near future.
here is the code that blew up in ember-validations:
var lookupValidator = function(validatorName) {
      var container = get(this, 'container');
      var service = container.lookup('service:validations');  //BOOM container is undefined
}

Here is my simple model (following the README as of today)
import EmberValidations from 'ember-validations';

export default Ember.Object.extend(EmberValidations.Mixin, {
      wat: '',
      validations: {
        wat: { presence: true }
      }
});

Here is my controller where I intend to invoke the validate method
export default Ember.ObjectController.extend({
    actions: {
        save: function() {
            var model = this.get('model');
            model.validate().then(null, function() {
                console.log('hello world');
            });
        }
    }
});

In my route I simply return a new model so I get a new/fresh form each time the user enters the route

export default Ember.Route.extend({
    model: function() {
        return User.create();
    }
});

Thanks again for supporting such a great community project!

Note: I do see that in the tests you provide a container (shown below). How should/ do production apps do this exactly (or is this just just a test-helper of some kind?)
https://github.com/dockyard/ember-validations/blob/master/tests/unit/errors-test.js#L12

What type of object are you mixing validations into? By default the standard objects in ember-cli, assuming you have them in the proper directories, should have container injected into them.
Is this a unit test? Some unit test objects don't have container and will have to be worked around for now until I can develop a better solution

It's a vanilla ember object inside the models folder (no ember-data in this ember-cli project)

I'm currently trying to boot the app in browser (not a unit test currently but I do plan to test this also and looked over your tests for some guidance).

thanks for the quick reply @bcardarella

I also hit this issue now and then. @bcardarella do you think that [this function] (https://github.com/dockyard/ember-validations/blob/master/addon/mixin.js#L31-70) can be replaced by an initializer?

Same case here, objects to be validated in unit tests, but also plain Ember.Objects fail because of missing container.
@bcardarella regarding 1.: you probably know that only objects created by the container will have the container property set. When using plain Ember.Objects not generated by a container factory, but simply imported with import object from 'path/to/object'; those will be missing the container as well!
My hope would be sore something like this to work out-of-the-box:
var model = Ember.Object.extend(EmberValidations.Mixin,{
            test: null,
            validations: {
                test: {
                    presence: true
                }
            }
        }).create(),

Hi,
First, I would like also to thank the team for such a great projet. I faced the same problem, and this is my quick simple workaround. I am a new developer on Ember, so I cannot guarantee that this is the best thing to do. It is only to share it with other developers.
As the container is automatically injected to the route when you use ember-cli, you can do this when you create your model into it:
export default Ember.Route.extend({
    model: function() {
        return User.create({
            container: this.get('container')
        });
    }
});

The solution provided by @Kira2 is a good workaround for now. Any idea on how difficult it would be to fix the root cause of this bug, @bcardarella?

I use the following solution since I already had a base model:
export default Ember.Object.extend({
    // we need to do this for ember-validations
    container: App.__container__
});

import BaseModel from 'app/models/base';
export default BaseModel.extend(EmberValidations.Mixin, {

    ...
});

You would only need to remove that single statement when it is fixed.
I guess you could fix this by using the global container on the application object, which is rather nasty.

@JordyLangen where does your reference to App come from, since it looks like you're using ember-cli?

That would be
import App from 'my-app-name/app';

@green-arrow That is just the name of your app (the object hosting your application, living on window). I haven't found a way to import it yet. It's a rather nasty fix, but it's a fix that I can remove quickly when no longer needed.

@JordyLangen I may be missing something obvious, but I don't have my application living on window. Does ember-cli put your application on window behind the scenes?

It doesn't look like ember-cli puts the application on window anymore. However, @rwjblue made this nice little addon that will do it for you. If using ember-cli, you can use that to get to the global application and use the method @JordyLangen recommended.

Is there a proper fix on the way? Otherwise I will fork and work on it

Another way to get access to the app instance is to import it:
import App from 'my-namespace/app';

I agree that using plain Ember objects with validations is useful, even though they don't have a container set. Maybe the controller is the better place though, like implied in all the examples in the README. Just like Rails moved from attr_accessible to strong params, the controller might be the better place for the validations since different contexts might demand different validations on the same object

My workaround is to create an initializer which adds the container to Ember-Validations:
import EmberValidations from 'ember-validations';
var alreadyRun = false;
export function initialize(container) {
  if (alreadyRun) {
        return;
    } else {
        alreadyRun = true;
    }

    EmberValidations.reopen({
      container:container, // the magic part..
    });
}

export default {
  name: 'reopen-validations',
  initialize: initialize
};

@AyKarsi, Thank you a lot!
It fits perfect to my needs.

... But it breaks my integration tests :) It'id broken because initializers are not invoked for integration tests.
Resumed looking for better solution.

strange.. integration tests work for me. or do you mean acceptance tests?

@AyKarsi my fault. I've put initializer in app/ folder instead of addon/. So initializer hasn't been included on addon tests run.
Thank you again and sorry for the buzz.

export default Ember.Object.extend({
// we need to do this for ember-validations
container: window.'myapp'.container
});

Really don't want to have to include container on my object. Seems really unnecessary. Any one else have any other workarounds?

Using on Ember.Object is included in the docs. I feel like at the very least this should be removed from the docs for the time being.

I came up with a bit of a workaround. I just over rode the buildValidators call.
	buildValidators() {
		if (this.get('container')) this.set('container', App.__container__);
		return this._super(...arguments);
	},

For anyone dealing with this issue in unit tests, this was my solution:
// some-addon/tests/unit/components/form.js

import startApp from '../../helpers/start-app';

test('Some test requiring the container', function() {
  const App = startApp(null);

  const ComponentFormObject = Ember.Component.extend({
    container: App.__container__,
  });
});

I am going to close this issue for now, feel free to reopen this issue if you feel like it's still relevant and we should take a look at it.


skl v1.2

scikit-learn will be releasing version 1.2 soon.  I upgraded to their master branch and ran the tests here; everything appears to work except that

sklearn will start throwing an error if feature names are not all strings (source).  The test test_encoders.test_column_transformer uses tests.helpers.create_dataset which includes one column with name 321, and breaks.  I don't think there's much reason to support the mixed column name types, and suggest modifying the test dataset.
the load_boston dataset has been deprecated and will be removed in 1.2; the test test_encoders.test_string_index should be changed to some other sklearn dataset, or perhaps the internal one.

I'm surprised I don't see errors from get_features_names vs. get_feature_names_out, since #382 isn't merged yet; have I done something wrong in setting up my environment I wonder?

Hi Ben,
I just merged #382 not sure why it wasn't a problem. Do you know when sklearn 1.2 will be released as stable? For 1 I agree that we should just delete the test if it is unsupported.
2. should be more work since all our doctests rely on the boston housing. The deprecation message suggests california housing dataset instead. Maybe we should just switch to that, but the column names will be different as well.
Thanks for testing against 1.2 and pointing this out! great help!

1.2 release candidate just hit PyPI, with a regular release to follow in a "week or two".

I probably won't have time to replace the boston housing dataset in the next couple of weeks. Should we just put some sklearn < 1.2 in the requirements for the moment?


fix: accomodate for delay in secondary creation after registering a free atsign

closes https://github.com/atsign-foundation/at_libraries/issues/232
- What I did

introduced a method to check if secondary belonging to the atsign at hand exists or waits until the secondary is created in case it does not.

- How I did it

Introduced a method called waitUntilSecondaryExists() which checks if secondary exists.
When the secondary does not exist, try for 50 iterations to fetch the secondary address.
Tries to connect to the secondary address fetched.
Retries for 50 iteration if the connection to secondary was not established
Resume normal onboarding process after abovementioned checks
- How to verify it

- Description for the changelog

fix: accomodate for delay in secondary creation after registering a free atsign

abandoning this PR and branch due to an issue with rebasing it with trunk code


[Question] How to get test case name

Question
How to get the name of the test case? In the example of the following code, how to get the test_name?
TEST_CASE("test_name") {
const auto name = doctest::detail::some_thing_like_get_name(); // name will be "test_name"
}

see this issue for details how to workaround it: https://github.com/onqtam/doctest/issues/345


Navier Stokes data explanation

Hello!
We have a hard time trying to understand the different features of the Navier Stokes data sets found here  on your Google Drive.
We have a .mat-file which includes:

t (shape 1x20) - simply the time steps in seconds?
a (shape 1200x64x64), 1200 separate initial conditions for the vorticity z-component over the x-y-plane of size 64x64?
u (shape 1200x64x64x20), 1200 different time series of the vorticity z-component over the same computation domain?

In other words, does a and u contain the same physical quantity (vorticity z-component)?

Yes it is correct!


[EDQ-424] Implement Minor Changes Across Metrics Notebooks

Changes:

Removed the old metrics_over_time.py script
Changed the saving of CSV files to work for MacOS
Explicitly excluded RDR in all of the 'retrospective' (other) notebooks
Cleaned up the queries by labeling the variables inserted within
Changed the means of establishing the DATASET parameter
Trimmed the number of columns generated in the concept_success_rate file
Limited the number of imports used in the weekly notebooks
Eliminated unnecessary tables (e.g. device exposure) in some of the weekly notebooks

@nishanthpp93 PR; simple fixes.


CSS output doesn't work in Kirki 4 (worked perfectly in Kirki 3)

I am using Kirki Customizer typography settings with 'output' to automatically assign the CSS. It worked perfectly with Kirki 3 but doesn't work anymore with Kirki 4 beta as well as today's 4.0.3 stable release:
Kirki::add_field($mydesignTheme, [
    'type' => 'typography',
    'settings' => 'typography_h1_settings',
    'label' => esc_html__('Advanced Typography', 'mydesign'),
    'description'    => esc_html__('Typo settings description', 'mydesign'),
    'section' => 'advanced_typography_h1',
    'default' => [
        'font-family'    => 'inherit',
        'variant'        => '700',
        'font-size'      => '',
        'line-height'    => '',
        'letter-spacing' => '0',
        'color'          => '#111111',
        'text-transform' => 'none',
        'text-align'     => 'inherit',
    ],
    'priority' => 10,
    'transport' => 'auto',
    'output' => [
        [
            'element' => 'body.single h1, body.blog h1',
        ],
    ],
]);

Thanks for reporting, @srjlulla. We're on it trying to replicate this on our end.

The CSS generation works fine on our end. Can you try to somehow narrow the issue down? Do you get any errors in the debug log for instance? (https://wp-pagebuilderframework.com/docs/debugging/)

There's nothing in debug log too. It worked for me in Kirki 3 but doesn't anymore in Kirki 4

Is there anything else in the theme that's extending Kirki for instance? Anything that could interefere?
One thing to try is: Remove "everything" from the theme except that particular field and check if that works. If it does, gradually bring back other options to see when things start to break.
We need to narrow this issue further down.

I understand but I think it's something minor in my Kirki integration, maybe installation or integration.
Do I need to change the inc/class-kirki-installer-section.php from Kirki 4?
This link doesn't seem to be working https://github.com/aristath/kirki/tree/master/docs/files/class-kirki-installer-section.php

It works in my Customizer Preview but doesn't work on the actual website.

I tried to debug this a bit, here's what I found.
The file packages/kirki-framework/module-css/src/CSS.php line 380, this following condition is never true in my case and $fields always returns empty
if (
				( isset( $field['kirki_config'] ) && $config_id === $field['kirki_config'] ) ||
				(
					( 'global' === $config_id || ! $config_id ) &&
					( ! isset( $field['kirki_config'] ) || 'global' === $field['kirki_config'] || ! $field['kirki_config'] )
				)
			) {
				$fields[] = $field;
			}
``

Hi @srjlulla , could you please try this version (v4.0.4)?
kirki.zip
This is not available yet in WordPress plugins repository, so you'd need to update yours manually.
Please let us know if this fixes your issue.
Thanks!

Hi @contactjavas, I have tried v4.0.4 from your zip file but it doesn't work too :(

Hi @srjlulla , we've created another build (v4.0.5). Would you mind to test it against CSS output issue?
Here is the download link: v4.0.5
Thanks!

Sorry @contactjavas, v4.0.5 does not work too. I really appreciate your efforts.

Hi @srjlulla,
we have paused the 4.0 release and reverted back to 3.1.9 in the WordPress repository.
While things went smoothly for most, there were some issues we have now identified that couldn't be patched in a timely manner.
We're now looking into these issues and continue to roll out 4.0 once everything is sorted out.

Alright David. Let me know if I can help with testing or improving Kirki 4, would love to be a part of it :)

@srjlulla, can you send me a download link of your theme (Dropbox, GoogleDrive, etc.) through my contact form please? https://davidvongries.com/contact/

Yes, I have created a demo theme where I am testing and sent it to your email.

@srjlulla, can you please test this?
kirki.zip

Sorry, still doesn't work for me @MapSteps.
Same issue - works in the Customizer preview but no changes on the live website.
I tried Global Typography and Background color.

@srjlulla,
I think we got behind what's causing it. Can you please test this version?
kirki-4-0-9.zip

Oh yes @MapSteps
The changes are working now - I tried background color, global typography and heading fonts and they finally worked! 😍


Scrollbars missing

Describe the bug
There are no (vertical) scrollbars in the query pane or the results pane (in combined and individual modes).
To Reproduce
Do a query with enough results...
Expected behavior
scroll bar!
Environment:
Platform: Desktop
OS: Linux
Architecture: x86_64
WebView: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.0 Safari/605.1.15
Version: 2.0.4
Flags: featureFlags: false, models_view: true, apidocs_view: true, themes: false, newsfeed: true

Closing as it cannot be reproduced internally and there has not been further activity on this issue


Change Certificate Lifetime

Hi,
very nice script, thank You!!!
How can I change the lifetime from the certificates? When I change the values in "/etc/openvpn/easy-rsa/vars.example" it don't works.
Thank you!!
Best regards

Read the easy-rsa documentation:
https://github.com/OpenVPN/easy-rsa/blob/master/doc/EasyRSA-Advanced.md
You need to rename vars.example to vars if you want to do it that way.

Thank you very much!


cli: add option to server streams externally without a player

Add an option to the CLI name --player-external-http, to start an HTTP
server like --player-continuous-http, but without starting a
corresponding player, and streaming to all available interfaces instead
of just the loopback.
This can be useful to stream to devices like smartphones or streaming
boxes, while making use of the better compatibility and performance of
livestreamer.
An accompanying --player-external-http-port option was added to allow
people behind firewalls to add their exceptions only once.
When starting livestreamer in this mode, URLs for the server will be
printed instead of a player being started. Those are detected based
on the IPv4 addresses returned by calling
socket.getaddrinfo(socket.gethostname(), ...), which will work for sure
on UNIX platforms, and should on Windows as well (actual testing
pending), in addition to <IP_ADDRESS>.

It seems there's something wrong with Travis, it's failing to fetch packages before even starting the tests. Is an apt-get update missing?

Thanks!

Cool, did you get to test it on Windows and/or OS X?

It seems I broke player-continuous-http, I'll push a new commit fixing it (sorry!).

Cool, did you get to test it on Windows and/or OS X?
No, I don't have access to any machines with those OSes currently.

It seems I broke player-continuous-http, I'll push a new commit fixing it (sorry!).

No problem, I've already got a fix in my local repo that i'll push in a minute.

I pushed a fix as well, it just corrected the default value for the port parameter. I also updated the docstrings for the HTTP functions a bit to match the changes.


Better way to change screens

Missing feature
Screens only change in one direction on screen press.
I would like to press:

Middle right move to the next screen
Middle left move to previous screen
Bottom 1/4 goto screen 0
Top 1/4 change 12/24 time format (existing in current code)

Justification
Better user interaction
Workarounds
in loop() replace
	screen = (screen + 1) % screenCount;

with
	if (p.y < top) {
		IS_STYLE_12HR = !IS_STYLE_12HR;
	} else if (p.y > bottom) {	  // Go to page 0
		screen = 0;
	} else if (p.x > middle) {	  // Previous page
		if (screen == 0) {          // Note type uint16_t is unsigned
			screen = screenCount; // Last screen is max -1
		}
		screen--;
	} else {					  // Next page
		screen = (screen + 1) % screenCount;
	}

Added to pull request for Allow user to force a screen re-calibration


Help Required to Use ui-grid with this package

Hi Toni,
Can you please give example on how to use UI Grid, Or else suggest some good Data grid compare to UI Grid.
Thanks and Regards,
Rajan G.

From what I read in their documentation it depends on AngularJS, not sure is it working with Angular 2 and above.
Sorry I don't have any recommendation for data Grids.


Add init script to be placed in /usr/local/bin/

Note, init script does not use pwd so the paths are not dependent on the user's current directory.  The host directories are in variables at the top of the script.
This is an init script to be placed in /usr/local/bin, allowing the user to start and stop the zeek docker container, among other actions.

I think we should put this script in the AI-Hunter repository instead.

This is for general use with docker-zeek, and doesn't have anything in it that's AI-Hunter specific.  I would argue that given the complexity of the command line needed to start it up, it would be directly helpful to others using docker-zeek without AI-Hunter at all.


LongClassNameCheck.java:71-74: Introduce new class from...

The puzzle 81-f6754283 from #81 has to be resolved:
https://github.com/l3r8yJ/oop-cop/blob/2bee691998f31382e5652681bc9421e18c122ee0/src/main/java/ru/l3r8y/rule/LongClassNameCheck.java#L71-L74
The puzzle was created by @h1alexbel on 29-Dec-23.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd thank you for reporting this. I'll assign someone to take care of it soon.

@0pdd thank you for reporting this. I'll assign someone to take care of it soon.

I see you're talking to me, but I can't reply since I'm not a chat bot.

@l3r8yJ this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2024-01-09T00:30:30.937644.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@l3r8yJ it's a duplicate of #89, let's close it

sure

@l3r8yJ thank you for resolving this ticket. I've just added it to your active invoice. You can always check all your invoices and more on the Contributor Dashboard.


Added the spo app retract command solving #8

Added the spo app retract command solving #8

Coverage decreased (-1.3%) to 92.691% when pulling ab4356e05599395e7b9e322505760a26ad692c91 on waldekmastykarz:spo-app-retract into fb536f754915f7dd891f0eddc8b86dda68e16abb on SharePoint:dev.

Merged manually


Canceled tiles with  new Chrome version 69.0.3497.92

Environment : Chrome 69.0.3497.92, Widows 10.
It seems that the new version of chrome cancels some tiles from downloading making the transition when pan and zooming appear not as smooth as before. I was able to reproduce the issue on the main site https://leafletjs.com/. A lot of times tiles are missing.
Firefox and IE seem to work fine.

Are you sure tiles are missing at the end of the animation, and the cancelled tiles are not simply the tiles that are no longer needed? I just tried browsing leafletjs.com on Chrome 69 and everything works fine.

Sorry my mistake, it seems it is related to the tile server, I can reproduce the issue on Firefox.
Chrome:

Firefox :

Looks like it's not due to Leaflet issue but rather related to your network.


SSL Issues

I am continually frustrated with running SSL in my apps. I have to keep trying different ports until finally one works (yes, I am using the Enable SSL checkbox in the web app properties to get different ports.) I keep getting this error:

An error occurred attempting to determine the process id of dotnet.exe which is hosting your application. One or more errors occurred.

I do have my localhost certificate dragged to the Trusted Root Certification Authorities:

I've restarted VS, rebooted PC and none of those help. This is a really frustrating issue.
Using:
Windows 10 Pro x64, Version 1607, Build 14393.693
Intel i7-3930
64 GB RAM
VS 2015 Version 14.0.25431.01 Update 3
Project:
"Microsoft.AspNetCore.Razor.Tools": {       "version": "1.1.0-preview4-final",       "type": "build"     },     "Microsoft.NETCore.App": {       "version": "1.1.0",       "type": "platform"     }   },   "frameworks": {     "netcoreapp1.1": {       "imports": [         "dotnet5.6",         "portable-net45+win8"       ]     }   }

@Tratcher Any idea what's going on here? It seems like it might be an ANCM issue.

@pan-wang @shirhatti

it seems to be a VS tooling issue. @jhkimnew who may have experience with ANCM inside VS.

@Serjster
I am sorry but I don't know what causes the problem.
I tried to reproduce the error message you reported here with some possible repro condition such as removing certificate or using duplicated tcp port which is used by other program but I was not able to find the repro steps. So, I think you should narrow down the issue or find the repro steps. Because you told that only specific tcp port works on your machine, there should be a reason why other tcp port does not work.
If you open the launchSettings.json which is under "Properties\PublishProfiles" of your project, you can change the TCP port manually and you can find which tcp port is causing the problem on your machine and you will need to find why those tcp port does not work.
BTW, I'd like to clarify one thing. You told that you dragged your localhost certificate to the Trusted Root Certification Authorities. Do you mean you move, not copy, the certificate, right?
If so, that will cause another problem and you will need to move back to the original location.
Some more detailed information for you:
IISExpress uses predefined certificate mapping for 100 tcp ports from 44300 to 43399.
The localhost certificate is bound for all the 100 tcp ports and the location of the certificate is hard-coded such as the Personal folder. If you move the certificate to other place, it will work temporarily because http service uses cached certificate but it won't work when the cached certificate is gone, which happens when you reboot the machine or restart http service.

@Serjster
FYI,
The same error was reported from this issue which seems to happen only after upgrading VS.
Is https://github.com/aspnet/Home/issues/1608.
Is your problem related to the issue?

@jhkimnew The "dragging" issue was the problem. I dragged it back to Personal, exported from there, imported into Trusted and next time I tried running VS with SSL, I got a pop up in my VS asking if I want to trust my cert... I said yes... and now it's working again. Thanks for the free education and sorry for wasting your time.


All custom styles get overwritten with 'unset'

Using the beta version of reakit (didn't test normal version as I am already on SC 4.)
Doing
const MyBox = styled(Box)`
        background: ${rgba('black', 0.8)};
        background-color: ${rgba('black', 0.8)};
        color: red;
`;

produces:
.dGkyAZ {
    margin: unset;
    padding: unset;
    border: unset;
    background: unset;
    font: unset;
    font-family: inherit;
    font-size: 100%;
    box-sizing: border-box;
    background-color: unset;
    color: inherit;
}

.Box-sc-6sssmg-0 {
}

.kEorAp {
    background: rgba(0,0,0,0.8);
    background-color: rgba(0,0,0,0.8);
    color: red;
}

I am not sure where these 'unset' properties come from but they take presence over my custom styles.
const MyBox = styled(Box)`
      &&{
        background: ${rgba('black', 0.8)};
        background-color: ${rgba('black', 0.8)};
        color: red;
     }
`;

does work.

Thanks for reporting it @ConneXNL. The unset styles come from Box. I'll investigate it.

Hmm, I couldn't reproduce it: https://codesandbox.io/s/m4zj3o5q6p
Could you please provide a sandbox/repo with a reproduction?

I upgraded to SC 4.1.* (I was at 4.0.*).
This seems to have resolved the issue.
I am closing this comment as it might be a SC issue rather then a reakit issue.


fix: gh action authentication issue

This fix tries to use a generic user instead of the bot user in an attempt to use the right secrets, it works on a branch, but we can only be sure that it really works after merging (previous fixes also worked on branches ...)

After fruther investgation, since march 01, 2021 dependabot pull-requests can't access secrets 🙄


Remove apply button in each selection/settings

Hi,
It would be nice if we didn't have to press apply each time we:

choose priority
choose context/project.
etc

It just auto-apply/auto-save our current selection without manually pressing apply as this is a redundant step.
Thanks! 🥞

Maybe Related #44


docs: introduce prettier and md-padding to format all docs

I found that the project lacks some automated mechanism to format the document.

Please fix commit message and PR title. There is a typo, it's not pettier but prettier

Please fix commit message and PR title. There is a typo, it's not pettier but prettier

OMG 😮
DONE.

Very useful tool. Thank you! I left some comments, looks like it breaks the GitHub-flavored alerts

This is an issue that has not been resolved by the prettier community. The solution is as follows: https://github.com/prettier/prettier/issues/16454.
I don't know if you can accept it.

Very useful tool. Thank you! I left some comments, looks like it breaks the GitHub-flavored alerts

This is an issue that has not been resolved by the prettier community. The solution is as follows: prettier/prettier#16454.
I don't know if you can accept it.

you mean "add an empty line between note and text" ?
> [!IMPORTANT]
>
> **Lorem ipsum dolor sit amet, consectetur adipiscing elit**

I can accept this.

LGTM


Adds format for webpage content

Everything looks good. Does this change affect the appearance of your website, or is it only reformatting the code?
If it does affect the appearance, would you mind including some before and after screenshots so we can see the difference?

Of course.
Before: https://screenshot.googleplex.com/KpTWASCTnMX.png
After: https://screenshot.googleplex.com/6OGga99N38o.png
@alekseychaikovsky (@ing you on this in case you want to see as well)


chore(auth): Removed redundant languageCode null-check.

Description
` String? get languageCode {
if (_delegate.languageCode != null) {
return _delegate.languageCode;
}
return null;

}`
This code has a redundant check. Getter languageCode will always return String? for _delegate.languageCode. It is more easier to understand the simple and straight logic if is done as:
String? get languageCode {     return _delegate.languageCode;   }
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See [Contributor Guide]).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (melos run analyze) does not report any problems on my PR.
[x] I read and followed the [Flutter Style Guide].
[ ] I signed the [CLA].
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[x ] No, this is not a breaking change.

hey @Naveed-dev30, if you sign the CLA, we can look to get this merged. Thanks 👍

Hey @russellwheatley thanks for the reminder. I have signed. Kindly merge my PR.


MSB4019: The imported project "c:\Microsoft.Cpp.Default.props" was not found (was: Unable to install on windows 10 x64)

I'm running on windows 10 x64, Node x64 8.1.2 and I seem unable to install sharp. I originally noticed the problem via a plugin which has sharp as a dependency and traced it through to here.
( Original issue: https://github.com/gatsbyjs/gatsby/issues/2419 )
If I npm i sharp i get the following error, does anyone see where this is failing what I could try?
c:\github\gatsby-blog>npm i sharp
npm WARN read-shrinkwrap This version of npm is compatible with lockfileVersion@1, but npm-shrinkwrap.json was generated for lockfileVersion@0. I'll try to do my best with it!

><EMAIL_ADDRESS>install c:\github\gatsby-blog\node_modules\sharp
> node-gyp rebuild

c:\github\gatsby-blog\node_modules\sharp>if not defined npm_config_node_gyp (node "C:\Users\douga\AppData\Roaming\npm\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
c:\github\gatsby-blog\node_modules\sharp\build\libvips-cpp.vcxproj(20,3): error MSB4019: The impor
ted project "c:\Microsoft.Cpp.Default.props" was not found. Confirm that the path in the <Import>
declaration is correct, and that the file exists on disk.
gyp ERR! build error
gyp ERR! stack Error: `msbuild` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\Users\douga\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:125:13)
gyp ERR! stack     at ChildProcess.emit (events.js:213:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:197:12)
gyp ERR! System Windows_NT 10.0.15063
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\douga\\AppData\\Roaming\\npm\\node_modules\\npm\\node_modules\\node-gyp\\bin\\node-gyp.js" "rebuild"
gyp ERR! cwd c:\github\gatsby-blog\node_modules\sharp
gyp ERR! node -v v8.1.2
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN The package gatsby-cli is included as both a dev and production dependency.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\douga\AppData\Roaming\npm-cache\_logs\2017-10-11T16_27_40_868Z-debug.log

I see a number of issues here relating to the same issue which all seem to point to problems with C++ redistributable etc. I should mention I have VS2017 installed, plus the 2015 build tools and have tried setting to both 2015 and 2017 without any joy.
Additionally, other node-gyp packages, such as node-sass build ok for me, so I'm not convinced it's an issue with that (or at least one which is generic enough to cause everything to fail for me)

error MSB4019: The imported project "c:\Microsoft.Cpp.Default.props" was not found. Confirm that the path in the declaration is correct, and that the file exists on disk.

Please see #949 #420 #483 for details of how others have solved this problem when using multiple MSVC installations.
You'll probably find npm install --build-from-source node-sass fails in the same manner.

@lovell thanks for the pointers, but this is what I'm getting at, other node-gyp based installs are working for me, just not this one. e.g.
c:\github\gatsby-blog>npm install --build-from-source node-sass
npm WARN read-shrinkwrap This version of npm is compatible with lockfileVersion@1, but npm-shrinkwrap.json was generated for lockfileVersion@0. I'll try to do my best with it!

><EMAIL_ADDRESS>install c:\github\gatsby-blog\node_modules\node-sass
> node scripts/install.js

Cached binary found at C:\Users\douga\AppData\Roaming\npm-cache\node-sass\4.5.3\win32-x64-57_binding.node

><EMAIL_ADDRESS>postinstall c:\github\gatsby-blog\node_modules\node-sass
> node scripts/build.js

Binary found at c:\github\gatsby-blog\node_modules\node-sass\vendor\win32-x64-57\binding.node
Testing binary
Binary is fine
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN The package gatsby-cli is included as both a dev and production dependency.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

+<EMAIL_ADDRESS>added 114 packages, removed 8 packages and updated 4 packages in 25.239s

Then if I run, npm i sharp directly afterwards:

c:\github\gatsby-blog>npm i sharp
npm WARN read-shrinkwrap This version of npm is compatible with lockfileVersion@1, but npm-shrinkwrap.json was generated for lockfileVersion@0. I'll try to do my best with it!

><EMAIL_ADDRESS>install c:\github\gatsby-blog\node_modules\sharp
> node-gyp rebuild

c:\github\gatsby-blog\node_modules\sharp>if not defined npm_config_node_gyp (node "C:\Users\douga\AppData\Roaming\npm\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
c:\github\gatsby-blog\node_modules\sharp\build\libvips-cpp.vcxproj(21,3): error
 MSB4019: The imported project "c:\Microsoft.Cpp.Default.props" was not found.
Confirm that the path in the <Import> declaration is correct, and that the file
 exists on disk.
gyp ERR! build error
gyp ERR! stack Error: `msbuild` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\Users\douga\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:125:13)
gyp ERR! stack     at ChildProcess.emit (events.js:213:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:197:12)
gyp ERR! System Windows_NT 10.0.15063
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\douga\\AppData\\Roaming\\npm\\node_modules\\npm\\node_modules\\node-gyp\\bin\\node-gyp.js" "rebuild"
gyp ERR! cwd c:\github\gatsby-blog\node_modules\sharp
gyp ERR! node -v v8.1.2
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN The package gatsby-cli is included as both a dev and production dependency.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\douga\AppData\Roaming\npm-cache\_logs\2017-10-11T18_20_00_821Z-debug.log

I'm happy to try and debug things but if you know where I can start that'd be great. I've read the other issues but none of the solutions are working for me.

The following suggests node-sass is not compiling from source:

Cached binary found at C:\Users\douga\AppData\Roaming\npm-cache\node-sass\4.5.3\win32-x64-57_binding.node

...so let's try another example. Does npm install --build-from-source bcrypt fail?

Have you tried everything mentioned in the two Stack Overflow questions linked to in https://github.com/lovell/sharp/issues/420#issuecomment-215375110 ?

Aha, progress, yes you're right the non cached version didn't build, I didn't spot that, my mistake. (Sorry not too experienced with node-gyp). I'm currently removing and re-adding the cpp distro as for some reason I can't seem to correctly set the msvs:
c:\github\gatsby-blog>npm config get msvs_version
2015

c:\github\gatsby-blog>npm config set msvs_version 2017 --global

c:\github\gatsby-blog>npm config get msvs_version
2015

c:\github\gatsby-blog>npm config set msvs_version 2013 --global

c:\github\gatsby-blog>npm config get msvs_version
2015

Hmm, ok, re-installed the cpp 2015 build tools and used the manual set:
set VCTargetsPath=C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\v140 as it would be on my system.
Got further but now have different errors:

c:\github\gatsby-blog>npm i sharp
npm WARN read-shrinkwrap This version of npm is compatible with lockfileVersion@1, but npm-shrinkwrap.json was generated for lockfileVersion@0. I'll try to do my best with it!

><EMAIL_ADDRESS>install c:\github\gatsby-blog\node_modules\sharp
> node-gyp rebuild

c:\github\gatsby-blog\node_modules\sharp>if not defined npm_config_node_gyp (node "C:\Users\douga\AppData\Roaming\npm\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
  VError.cpp
  VInterpolate.cpp
  VImage.cpp
  win_delay_load_hook.cc
     Creating library c:\github\gatsby-blog\node_modules\sharp\build\Release\li
  bvips-cpp.lib and object c:\github\gatsby-blog\node_modules\sharp\build\Relea
  se\libvips-cpp.exp
  Generating code
  Finished generating code
  libvips-cpp.vcxproj -> c:\github\gatsby-blog\node_modules\sharp\build\Release
  \\libvips-cpp.dll
  libvips-cpp.vcxproj -> c:\github\gatsby-blog\node_modules\sharp\build\Release
  \libvips-cpp.pdb (Full PDB)
  common.cc
  metadata.cc
  operations.cc
  pipeline.cc
c:\users\douga\.node-gyp\8.6.0\include\node\uv-win.h(32): fatal error C1083: Ca
nnot open include file: 'winsock2.h': No such file or directory (compiling sour
ce file ..\src\metadata.cc) [c:\github\gatsby-blog\node_modules\sharp\build\sha
rp.vcxproj]
  sharp.cc
c:\users\douga\.node-gyp\8.6.0\include\node\uv-win.h(32): fatal error C1083: Ca
nnot open include file: 'winsock2.h': No such file or directory (compiling sour
ce file ..\src\common.cc) [c:\github\gatsby-blog\node_modules\sharp\build\sharp
.vcxproj]
  utilities.cc
c:\users\douga\.node-gyp\8.6.0\include\node\uv-win.h(32): fatal error C1083: Ca
nnot open include file: 'winsock2.h': No such file or directory (compiling sour
ce file ..\src\pipeline.cc) [c:\github\gatsby-blog\node_modules\sharp\build\sha
rp.vcxproj]
c:\users\douga\.node-gyp\8.6.0\include\node\uv-win.h(32): fatal error C1083: Ca
nnot open include file: 'winsock2.h': No such file or directory (compiling sour
ce file ..\src\operations.cc) [c:\github\gatsby-blog\node_modules\sharp\build\s
harp.vcxproj]
  win_delay_load_hook.cc
c:\users\douga\.node-gyp\8.6.0\include\node\uv-win.h(32): fatal error C1083: Ca
nnot open include file: 'winsock2.h': No such file or directory (compiling sour
ce file ..\src\sharp.cc) [c:\github\gatsby-blog\node_modules\sharp\build\sharp.
vcxproj]
c:\users\douga\.node-gyp\8.6.0\include\node\uv-win.h(32): fatal error C1083: Ca
nnot open include file: 'winsock2.h': No such file or directory (compiling sour
ce file ..\src\utilities.cc) [c:\github\gatsby-blog\node_modules\sharp\build\sh
arp.vcxproj]
gyp ERR! build error
gyp ERR! stack Error: `msbuild` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\Users\douga\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:125:13)
gyp ERR! stack     at ChildProcess.emit (events.js:213:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Windows_NT 10.0.15063
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\douga\\AppData\\Roaming\\npm\\node_modules\\npm\\node_modules\\node-gyp\\bin\\node-gyp.js" "rebuild"
gyp ERR! cwd c:\github\gatsby-blog\node_modules\sharp
gyp ERR! node -v v8.6.0
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN The package gatsby-cli is included as both a dev and production dependency.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\douga\AppData\Roaming\npm-cache\_logs\2017-10-11T18_49_39_458Z-debug.log

YES! finally got it working! there seemed to be two issues primarily.

No VCTargetsPath set - I set mine as a system level environment variable to the following path C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\v140 this seemed to solve the first issue where it couldn't find the Microsoft.Cpp.Default.props file.

Missing 8.1 SDK from the MS visualcppbuildtools_full.exe - I had only originally selected every except 8.1 and had selected 10...because i'm on windows 10, node-gyp needs 8.1 it seems.

What didn't aid the confusion was that the paths to the VC build tools are not very clear, here's my registry structure for example:

Either way it's all sorted now, if anyone wants to ask specific questions, best be quick whilst I remember the process!

Great, glad to hear you managed to get node-gyp to play nicely with multiple MSVC installs.
Thank you for the detailed updates - these should help others in a similar situation.

 (C:\Users\admin\AppData\Roaming\npm\node_modules\npm\lib\utils\lifecycle.js:279:16)
115687 verbose stack     at emitTwo (events.js:135:13)
115687 verbose stack     at EventEmitter.emit (events.js:224:7)
115687 verbose stack     at ChildProcess. (C:\Users\admin\AppData\Roaming\npm\node_modules\npm\lib\utils\spawn.js:40:14)
115687 verbose stack     at emitTwo (events.js:135:13)
115687 verbose stack     at ChildProcess.emit (events.js:224:7)
115687 verbose stack     at maybeClose (internal/child_process.js:943:16)
115687 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:220:5)
115688 verbose pkgid<EMAIL_ADDRESS>115689 verbose cwd E:\silex\Silex\Silex
115690 error Windows_NT 6.1.7601
115691 error argv "C:\Program Files\nodejs\node.exe" "C:\Users\admin\AppData\Roaming\npm\node_modules\npm\bin\npm-cli.js" "install"
115692 error node v9.1.0
115693 error npm  v4.3.0
115694 error code ELIFECYCLE
115695 error errno 1
115696 error<EMAIL_ADDRESS>postinstall: npm run build:fonts && npm run build
115696 error Exit status 1
115697 error Failed at the<EMAIL_ADDRESS>postinstall script 'npm run build:fonts && npm run build'.
115697 error Make sure you have the latest version of node.js and npm installed.
115697 error If you do, this is most likely a problem with the cloud-explorer package,
115697 error not with npm itself.
115697 error Tell the author that this fails on your system:
115697 error     npm run build:fonts && npm run build
115697 error You can get information on how to open an issue for this project with:
115697 error     npm bugs cloud-explorer
115697 error Or if that isn't available, you can get their info via:
115697 error     npm owner ls cloud-explorer
115697 error There is likely additional logging output above.
115698 verbose exit [ 1, true ]
please help me to solve this issue

kindly provide me installation details of silex static website builder in step by step manner.

@srinivasants Wrong repo.


PARQUET-653: Build conda artifacts with -static-libstdc++ again

The final stage of this work will be to move dev/testing packaging builds to centos6 images on Circle CI (this may be possible in Travis -- which allows you to use docker -- but easier with Circle CI)

+1, LGTM

+1, thanks

So, this unfortunately has broken the Arrow builds. I think we're going to have to set up a separate repo with a Dockerfile that uses the RHEL devtoolset to build these artifacts. static linking libstdc++ is far too finicky it turns out

We could also just use conda forge and keep bumping the build number when we're ready to upgrade the artifact


Support setting the default size of secondary windows

How to set the default size of the secondary window?
As is shown in，I want to resize this window by default

This is some thing I really want help on too. Second window opened is too small to use. Like while previewing documents before printing it only opens doc in 44%

hope the answer +1
在 2021-06-30 13:46:27，"Xitiz Havoc" @.***> 写道：
This is some thing I really want help on too. Second window opened is too small to use. Like while previewing documents before printing it only opens doc in 44%
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

This is not currently possible, but we would accept a PR to support this.


Couldn't find workspaceMetadata of workspace id "..." [Solution Package Generation]

Describe the bug
Generated a package for a solution and when I verify the resource deployment I encounter an error saying it couldn't find the workspaceMetadata for the workspace.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://portal.azure.com/?feature.IncidentTriggering=true&feature.investigationrawdata=true&feature.MultiplePlaybooksForRule=true&feature=&EntitiesFromGS=true&feature.mstimatching=true&feature.Dynamics365Connector=true&feature.Solutions=true&feature.EntityMapping=true&feature.AnalyticsEfficiency=true&feature.IncidentTimeline=true&feature.msticonnector=true&feature.mtpconnector=true&feature.BringCustomConnector=true&feature.teamscollaboration=true&feature.AlertDetails=true&feature.ZscalerPrivateAccessConnector=true&SearchKeyFieldMapping=true&feature.WindowsSecurityEventsFeature=true&feature.AdvancedSearch=true&feature.includePreviewTemplates=true&playbooksgallery=true&feature.management=true#create/Microsoft.Template'
Click on 'Build your own template in the editor'
Paste the content from mainTemplate.json into editor
Click on 'Save'
Select a Subscription, Resource Group and enter a workspace name.
Click on 'Review + Create'
Click on 'Create'
See error

Expected behavior
Expecting workspaceMetadata to be available for other resources. Not quite sure why it is not found.
I did find in documentation a part that talks about the metadata for the workspace being stored in Cosmos DB and managed by Microsoft (https://docs.microsoft.com/en-us/azure/machine-learning/how-to-manage-workspace?tabs=python#advanced). I couldn't find any other information related to workspace metadata to try to figure out why it would not be found.
Screenshots

Additional context
Source files:
1.0.2.zip
Template Validation:

@ericlanteigne-semperis, It seems Azure Sentinel is not added to your workspace. Can you please add the same and try again to redeploy the solution. let me know if you still facing any issue.[PFB screenshot]

@ericlanteigne-semperis , Can you please check and confirm on this ?

@v-maudan Sorry for the delay. I was trying to find what to do yesterday afternoon but couldn't quite understand. This morning I realized after I press "+ Create" in Azure Sentinel I then get the page to add Azure Sentinel to a workspace. Once I added it to my workspace I ran the deployment again and it was successful!
I am trying to think about how we would handle this for customers though. Because the workspace is created by the template and if this service is not added, the other resources in the template will fail. So unless we split this into a two part deployment, we would need some way to handle it within the template or during the deployment. Is there a way for the ARM template to be configured to add services to resources once they are created?

I am having some further issues validating the deployment although it was successful. Maybe you will have some thoughts about it. I ran deployed this into a new resource group to help isolate the resources for verification.
Ran the deployment with the error, added Azure Sentinel to workspace, reran deployment and it was green:

Resource Group after deployment:

When trying to find alert rules, none could be found although I can see the detailed information for each when selecting it from deployment page. Where would I go to view them?

Data connector could also not be found, even when selecting it from the deployment page. I am a bit confused by this considering it said it was successful.

I should add that once I manually added the dependsOn attributes like it mentions in documentation and I got past this issue. I wanted to bring it up though in case it can be incorporated into package generation.

@ericlanteigne-semperis,
prerequisite step before solution deployent is to have resouce group and log analytics workspace with azure sentinel enabled.
We already mentioned in the instruction for same. Don't create new resource group while solution deployment. known issues

Post solution deployment
Please check this documention
Post solution deployment steps.docx
Please let me know if you still facing any issue.Thanks!

@v-maudan Thanks for the explanation and reference. I was able to find my resources that were deployed.
We can close this issue since I am not blocked by this anymore. Thank you


Adding --output_text and --remove_endnotes options

Added a feature to output the text of the EPUB file to a plane text file. This helped me to review the text as I worked on the next feature, remove endnotes.
The remove endnotes option now just applies another regex on the text in the same area other regex's are run against the text. This one applies one that removes all numbers that are immidiatly preceeded by a letter or punctuation. This is probably not perfect, but I ran several chapters through it and it caught all the endnotes I had.

Thanks for your great contribution. The change LGTM. Let's see if any edge cases will happen in the future and fix them at that time.


StorageApplicationPermissions throws exception in unpackaged app build

Which version of the app?
WinUI 3 Gallery
Description
In the FilePickerPage.cs there is the async void PickFolderButton_Click(object sender, RoutedEventArgs e) method:
[Line 187]
StorageApplicationPermissions.FutureAccessList.AddOrReplace("PickedFolderToken", folder);
If compiling as an unpackage application, this line throws an ArgumentException (see screenshot).
Screenshots

Windows version
Windows 10 22H2 (19045)
Additional context
No response

This is likely an API that doesn't work on unpackaged, and should be providing appropriate documentation.


Import Marten.AspNetCore into the official repository

With @jokokko 's sign off, let's import the Marten.AspNetCore library into the official mothership repository and add documentation for that here.
Tasks

[ ] Import the code
[ ] Add documentation
[ ] Target IHostBuilder in .Net Core 3.* apps
[ ] Target IWebHostBuilder in all targets (PITA to switch between the two things, but what can you do?)
[ ] Optionally start the Async Daemon as an IHostedService (might break this out to a separate issue)
[ ] Register container resolved IDocumentSessionListener objects that would be added to newly created IDocumentSession objects at resolution time (might break this out later)
[ ] Might need to allow users to register an all custom factory for IDocumentSession building

I think now we'll just stick the extension methods on ServiceCollection into the main Marten lib

This was done through other pull requests to v3.12


Poetry dependency groups

I can see that an upcoming poetry release has support for "dependency groups".
https://python-poetry.org/docs/master/managing-dependencies/
Will this plugin need any updates to accommodate this?

I would hope that dependency groups will make nox-poetry entirely redundant. But I haven't looked at them in detail yet.

https://github.com/cjolowicz/cookiecutter-hypermodern-python-instance/pull/679 has some initial work that demonstrates how dependency groups could replace nox-poetry.
It looks like Poetry still only supports editable installs for the root package. Also, the noxfile needs to export requirements for safety (a tool for checking dependencies against a database of security vulnerabilities). So there's still some amount of glue code that might be easier to maintain in a PyPI package.


feat: rework security models

At this point, Security models are directly pulled from FastAPI.
Interestingly, they already reflect some of the Binder concepts we have here:

Every model has a .model attribute that is the OpenAPI model.
Every model has a __call__ so that it can serve as a dependency..

So I think we should try to formalize this by making a creating a Binder Protocol for them.
Something like:
class SecurityGuard(Protocol):
     @overload
    def verify(self, connection: HTTPConnection) -> Any:
         ...
    @overload
    def verify(self, request: Request) -> Any:
        ...

class SecurityGuardMarker(Protocol):
    def register_parameter(self, param: inspect.Parameter) -> SecurityGuard:
        ...

class SecurityOpenAPI(Protocol):
    def get_openapi(self, model_name_map: ModelNameMap, schemas: Schemas) -> SecurityModel:
        ...

class SecurityOpenAPIMarker(Protocol):
    def register_parameter(self, param: inspect.Parameter) -> SecurityOpenAPI:
        ...

I think as part of this we may want remove the ability to create security models via the dependency injection system.
Yes, it is possible, but it ads a lot of complexity.
Yet the original use case of lazy-loading models is still valuable.
A good compromise would be to accept an instance or a callable that returns an instance.

On the other hand, it is pretty useful to say "this auth model depends on a dynamic config" and to be able to swap that out in tests. But that config is (hopefully) just some env vars, so setting those env vars before a test would not be a huge deal.
What does need dependency injection is something that actually validates tokens. But  I reckon that'd be a 3rd party lib anyway