Relative html img links are referencing non existing locale files since v1

If I define a link to an img in my documentation and use markdown style
![screenshot](./assets/live_toggle.png)

The image path will be adapted to always find the non localized image in either language file
If I define a link to an img and use html
<img src="./assets/live_overview.png" alt="screenshot"/>

The image path will only work for the default language since the relative path is kept intact and will not find anything at .../fr/assets/live_overview.png
I understand that there have been changes to improve efficiency for non locale assets as described in #252
Though I would expect that html (img) links are treated similar to markdown image tags and are being converted.
Maybe I am just missing something.
I use mkdocs_static_i18n-1.1.1
with mkdocs-1.5.3
and mkdocs_material-9.4.6
Thanks a lot for your beautiful work!

Issue has been fixed with v1.2.0
Sorry for the hassle, probably could have seen it coming with the pending pull request. Thank you!

Actually version 1.2.0 didn't add anything after 1.1.1 that would influence this behavior.
Also I don't think MkDocs processes the links of images written in HTML instead of Markdown, but as you say it's fixed, then perhaps some other version fixed that and you haven't used it before 🤔

Uh, I'm sorry the problem is not fixed actually. I was looking at the wrong part of the docs .
So am I right that, since images (files) are not moved to the language dir anymore, html links will "break" because they are not processed?
Do you have an idea how to deal with this? Since html has more features in how images are displayed it is the favored choice in some places.

If I understand correctly the image name and extension doesn't change, just the relative path of directories to it.
As mentioned in this comment (https://github.com/ultrabug/mkdocs-static-i18n/pull/256#issuecomment-1712484289) you could use absolute paths, but as I point out in the comment below (https://github.com/ultrabug/mkdocs-static-i18n/pull/256#issuecomment-1714816686), you need to keep in mind the real base path, as it might be /repository-name/ instead of just /. Also in the case you change the name of the base path you would have to modify every link ✌

Since html has more features in how images are displayed it is the favoured choice in some places.

What other features do you have in mind? You can add CSS classes or other attributes with the attr_list extension to display the images in a different way. https://python-markdown.github.io/extensions/attr_list/
markdown_extensions:
  - attr_list

![screenshot](assets/live_toggle.png){ style='width: 50%;' }

I see, thanks for the clarification!
The attribute list hint is great! Didn't come to my mind even though we already used the extension for another purpose. I will have a go with it. Some 200 html tags need to be replaced but then the style is also more consistent. I may not forget to add this in the docs contibution guidelines 😉

You can add CSS classes or other attributes with the attr_list extension to display the images in a different way. https://python-markdown.github.io/extensions/attr_list/

Serves as a very good solution for me.
I will leave a note that, without knowing anything about the underlying process, I would love relative links in html tags to be working with multiple language directories.


TypeError: iter_rows() got an unexpected keyword argument 'column_offset'

When trying to use:
${row_header_data}=          Read Excel Row          row_num=1

I receive the following error:
TypeError: iter_rows() got an unexpected keyword argument 'column_offset'

I assume this could be related to an update in an underlying dependency.

Presumably, this issue is caused on line:
https://github.com/peterservice-rnd/robotframework-excellib/blob/92e8e91bcbf6fc5aadcd23820036e8b952130bef/src/ExcelLibrary.py#L241

it works for me.
TypeError disappeared.
Otherwise, Read Excel Row and Read Excel Column do not returned correct data as expected.
The returned list only contain one cell's value.


Added suggestions form testing

Hi,
here are the changes, which I would suggest after testing the three notebooks.
All the best,
Hendrik

Thanks Hendrik, much appreciated!


.inf file

Tested on W7 and W8. It seems, driver .inf file is necessary to extend usable OS range.
Thank you
Tadas LY1CE

Thank you for your report! I haven't tested the firmware on those. I should have, though. You right, an inf file is needed to make it work. I will address this issue shortly.

@TadasLes Would you mind testing the inf file when it is ready. I have a Windows 7 VM and I'm going use it for testing, but it would be nice if you can confirm the inf file works on Windows 8 as well.

@TadasLes Would you mind testing the inf file when it is ready. I have a Windows 7 VM and I'm going use it for testing, but it would be nice if you can confirm the inf file works on Windows 8 as well.

Of course I will test it on W7, W8 and W XP

@TadasLes Thank you! I really appreciate that. I think the inf file will be available either later today, or during the weekend.

@TadasLes Looks like I managed to create an INF file that works just fine on Windows 7 x64. Could you please test it on your set of operating systems?
Here is how I do it on Windows 7 x64:
Method 1

Copy both inf and cat files to your machine;
Open the Command Prompt with administrative privileges;
Run the following command:

pnputil -i -a stm32-blue-pill-serial-monster.inf

It should warn you that you are installing a driver from an unidentified developer. That's OK, the driver has been signed with a self-signed cert.

Plug in the device. Windows will first try to find the driver in Windows Update, and after that will install the local driver you just installed.

Method 2
Go to the Device Manager, right click on one of the STM32 Blue Pill Serial Monster devices, and select Update Driver Software. Then choose browse my computer for driver software and point it to a directory with the inf file. Windows will install the driver for one of the CDC devices. You don't need to repeat the procedure for the others. Just disconnect and reconnect the device and Windows will pick up the rest of the ports.
See below for the inf file archive.
inf.zip

@TadasLes I tested on Windows XP 32 bit myself. Everything's OK.

I bought a code-signing certificate from Comodo. As soon as they complete their validation stuff, I will sign the inf.

XP x32 and W7 x32 - OK
W8 x64 - the system interrupts installation without valid driver certificate. There isn't branch for installing non certified driver.

Hope, with certified driver things will be OK.

@TadasLes Thank you very much for testing. I will get back to you when my certificate is ready.
If you don't mind, what's your use case for the firmware?

@TadasLes I added signed cat file. Now it should work on Win 8 as well. Could you please re-test?
inf.zip

I also testing it on Win 8 x64 myself. Now it works OK.

Hi, tested signed driver on W8 x64. Install successful.
I'm planning add to BluePill level shifters on lines that are not 5V tolerant for safety.
It will be fine tool for debugging systems with UART. No more single USB <-> UART converters.
:1st_place_medal:

@TadasLes Thank you very much for testing. And thank you for describing the use-case, that's interesting. Looks like we're all set, I am closing the issue and merging the pull-request.


fix: Avoid closing a Toast message when auto-close is false

Description
It was noticed that the toast message component would close after some time even if autoClose option was false
This quick change will fix it
Type of change

Bug fix (non-breaking change which fixes an issue)

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Hey @hetunandu  - could you please also (or maybe even instead?) make this change on https://github.com/appsmithorg/design-system/blob/feat/migrate-toast/packages/design-system/src/Toast/index.tsx ? We're in the process of migrating Toast as we speak and this component should land on release by EOW

Updated on https://github.com/appsmithorg/design-system cc @tanvibhakta


It does not work when using the Nginx stream module for transmission

let's assume my server IP is <IP_ADDRESS>, I had bind my domain mtg.example.com to <IP_ADDRESS>.
I run mtg like this:
/usr/local/bin/mtg run [secert] --bind <IP_ADDRESS>:12345

I set my telegram proxy like this, and it works fine:

but when I use nginx stream module to transmit it to mtg, it's not working
stream {
    log_format basic '$remote_addr [$time_local] '
                 '$protocol $status $bytes_sent $bytes_received '
                 '$session_time';

    access_log /var/logs/nginx/stream_access.log basic buffer=32k;

    # Identify SNI
    map $ssl_preread_server_name $backend_name {
        mtg.example.com mtg;
        .example.com web;
        # default to web
        default web;
    }

    # mtg
    upstream mtg {
        server <IP_ADDRESS>:12345;
    }

    # web
    upstream web {
        server <IP_ADDRESS>:444;
    }

    # listen 443
    server {
        listen 443 reuseport;
        listen [::]:443 reuseport;
        proxy_pass  $backend_name;
        ssl_preread on;
    }
}

I also tried to set ssl certificate in the stream server, not working too
# listen 443
server {
    listen 443 reuseport;
    listen [::]:443 reuseport;
    proxy_pass  $backend_name;
    ssl_preread on;

    #ssl start
    ssl_certificate /path/to/fullchain.pem;
    ssl_certificate_key /path/to/private.pem;
    ssl_session_timeout 5m;
    ssl_protocols TLSv1.1 TLSv1.2 TLSv1.3;
    ssl_ciphers ECDHE-RSA-AES128-GCM-SHA256:ECDHE:ECDH:AES:HIGH:!NULL:!aNULL:!MD5:!ADH:!RC4:!DH:!DHE;
    ssl_prefer_server_ciphers on;
    #ssl end
}

by the way, my ip and domain has substituted to <IP_ADDRESS> and mtg.example.com.
so, can someone or the author of mtg tried to make this work?

To be frank, I've never tried to put mtg behind nginx so do not know what to advise here.

Nginx stream mode works in layer 4 and has nothing to do with the 7th layer application.
You need to check again and make sure your config file correct.
For instance, "$backend_name" need to be changed if the config file mentioned above is exact what you are using in your server.

@9seconds It doesn't matter, thank you all the same.

@wwbfred yeah, it should be work theoretically but not working in practice, I'll try to figure it out.

I put mtg behind nginx and it works well.

@wwbfred can you share you nginx config(after substituting sensitive info)?  I really need this.

@xiebruce,
/etc/nginx/nginx.conf:
...
stream {
    map $ssl_preread_server_name $route {
        example.com mtg;
        default web;
    }

    log_format stream '$remote_addr [$time_local] [$ssl_preread_server_name] [$route] $status $bytes_sent $bytes_received $session_time';
    access_log /var/log/nginx/stream.log stream;

    upstream mtg {
        server <IP_ADDRESS>:3128;
    }
    upstream web {
        server <IP_ADDRESS>:443;
    }
    server {
        listen 111.222.333.444:443;
        proxy_pass $route;
        ssl_preread on;
    }
}
...

@Allineer  That's wizard,  mine is nothing different with yours, but did you use two 443 port?

@xiebruce, yup.

Can you please verify this instruction? https://github.com/freejohn123/mtproto-tls-server


protractor.Button.RIGHT simulates a left click every time

Currently in protractor version 5.4.2 there seems to be no way to simulate a mouse right click. The old version pre chromedriver 74 was to use browser.actions().click(elem, protractor.Button.RIGHT), but that always simulates a left click. No other method I attempted was able to reproduce a right.
Bug report

Node Version: 12.13.0
Protractor Version: 5.4.2
Angular Version: ``
Browser(s): chrome, version 79
Operating System and Version win 10

Hi @Fuun347 ,
can you please include an SSCCE (Short, Self Contained, Correct [compilable] example) http://sscce.org/
This will help to understand/debug the issue

let elem = element(by.css('button'));
await browser.actions().click(elem, protractor.Button.RIGHT);
This should perform a right click on the element "elem", but it performs a left click. This is an issue with the new actions API used in version ^75 of chromedriver.

@Fuun347  , try it like this:
await browser.actions().click(elem, protractor.Button.RIGHT).perform();

@adinapitul yea, excuse me there, forgot to type out the perform() function at the end

@Fuun347  , I use it like this, and it works fine (Chrome 80, MacOS)
await browser.actions().mouseMove(elem).perform();
await browser.actions().click(protractor.Button.RIGHT).perform();
I just tried it now with browser.actions().click(elem, protractor.Button.RIGHT).perform(), and indeed it's misbehaving. In my app it does a right click + a left click :)

I'm pretty sure when experimenting I did indeed try your solution and it did not work. I will try again with your method and confirm if it works for me.

@adinapitul I tried again and it does not work me. It always seems to only use a left click.

#5271
use
capabilities: {
'browserName': 'chrome' ,
'chromeOptions': {
w3c: false
},// or 'safari'
},

@praveendvd yes, that did fix the issue for me. Thank you!

@praveendvd That completely fixed it! Thank you so much!


Question concerning #68

I was wondering why xpublisher and xheartbeat are kept as pointers in xserver_zmq. This change has been added during @jcfr's PR #68. Is this change useful in SlicerJupyter?
Also, we were wondering why you added non-virtual destructors for those classes on this commit: https://github.com/QuantStack/xeus/pull/68/commits/a569af036263d89252febae8138146b808229d8a. Was your plan to create a custom publisher and a custom heartbeat?

wondering why xpublisher and xheartbeat are kept as pointers

Since xpublisher and xheartbeat were not public, handling them as pointer allows us to forward declare them in xserver_zmq while making the server part of the public API.
An other approach would be to follow the pImpl pattern.

why you added non-virtual destructors for those classes

This is either a left over from a previous version of the PR where xpublisher and xheartbeat  were made public [most likely] ..  (or it was needed to forward declare  [less likely])


Add 'build' gulp task

Concat all sources
Wrap them in an IIFE
Uglify the result

Closes #3

Two things:

Please add your name to the copyright section when you change a file
Please, do not remove license header of individual files :)

Anyway, good job :+1:


Fixed error in auth.service.ts

Thanks for your PR :)
This actually needs to be the accessToken instead of the idToken. The getUserInfo method relies on that.
What was the problem the accessToken caused in your case?

Without this change the authenticated method does not detect when the user is already logged in if/when the app is restarted:
https://github.com/auth0-samples/auth0-ionic2-samples/blob/master/01-Login/src/services/auth/auth.service.ts#L49
The getUserInfo method does rely on this.accessToken but getUserInfo is only called in the authenticated callback and that callback sets accessToken:
https://github.com/auth0-samples/auth0-ionic2-samples/blob/master/01-Login/src/services/auth/auth.service.ts#L49

ah I see what you're saying. yes, that makes sense. thanks for clarifying 👍

@saschwarz thanks for explaining the issue


Many thanks for your code, where can I get the traindataset and testdataset?

Many thanks for your code, where can I get the traindataset and testdataset?

The traindataset download from internet.

Could anyone give a link to download the traindataset?


libsigrokdecode: 0.5.0 -> 0.5.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/libsigrokdecode/versions.
These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 0.5.1 with grep in /nix/store/bzy46mv9yqpqap2swy9gc25x0hdbq1ny-libsigrokdecode-0.5.1
directory tree listing: https://gist.github.com/4794e65a1bf494e590272e83866d9e95

cc @bjornfor for review

Applied to master (d8b32a1dcf279da1ade0b3f9a58977e80e1ca879).


Update Istio Project Summary

As per discussion with @CathPag , updating the Project Summary Table for Istio.
Would be good to have review from @craigbox @linsun @ericvn @johnhoward before we merge.
Signed-off-by: Faseela K<EMAIL_ADDRESS>

@kfaseela Looking at https://landscape.cncf.io/summary, I think the Integrations is wrong. The page don’t really support a URL in that field. Should we include as a string a subset of the items from https://istio.io/latest/docs/ops/integrations/?
Also, relative to tags. I'm not sure if we need additional tags for security items like authentication and authorization? We do include mTLS. We certainly should have, and do, items from https://istio.io/latest/docs/tasks/, but I'm not sure hot to determine which subtask items we should includes tags.

@AndreyKozlov1984, can you help with the integration link?

@CathPag  - I see an issue here -

I can make the text as a link, is that the goal? i.e. it will look like

@CathPag what about using a text LINK and making it a hyperlink? I can detect links automatically

Sure. Definitely looks cleaner. Does that work for your @ericvn?

@AndreyKozlov1984 If that works as a link, and if the landscape page allows us to navigate to the integrations page, that would be good enough!

Ok, so all text like  https://istio.io/latest/docs/ops/integrations/ will LITERALLY look like LINK

Ok, so all text like https://istio.io/latest/docs/ops/integrations/ will LITERALLY look like LINK

can't we customize what comes instead of "LINK" ? :(

Like with [] in a text, how it is done in github?
[Integrations](https://istio.io/latest/docs/ops/integrations/) ? I can support that

Like with [] in a text, how it is done in github? [Integrations](https://istio.io/latest/docs/ops/integrations/) ? I can support that

Yes!

@kfaseela @CathPag I added both type of links - github style links and just normal links are supported. I've added this to the integration row.

please check the website, it is deployed - https://landscape.cncf.io/summary

Thanks.


setCustomAutocompletion does not work with keepCurrentCompleters=true

Describe the bug
I tried to add more than one custom auto completion category by calling the method several times. But only the last call is transfered to the client side.
To Reproduce
Add the following code:
AceEditor aceEditor = new AceEditor();
aceEditor.setMode(AceMode.sql);
aceEditor.setCustomAutocompletion(Arrays.asList("User", "Role"), "table", true);
aceEditor.setCustomAutocompletion(Arrays.asList("FullName", "Address", "Right"), "column", true);
aceEditor.setAutoComplete(true);

Expected behavior
I expect that auto complete shows all values, but only values with category "column" are shown.
Screenshots
Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version: 3.3.3

Additional context
Vaadin version 23.1.

@rappenze thanks for the issue
I was able to reproduce it and found why and where it happens. I'll try to build a hotfix asap.

Fixed with v3.4.0 (https://vaadin.com/directory/component/ace/3.4.0)
-- Backport to Vaadin 14 soon


Translates "guide/class-and-style.md" #6

docs: portuguese translation
#6 "guide/class-and-style.md"

Waiting for review.
Closes #6.


New option

Will your mod be compatible with Animania?   Default for that mod is to replace the vanilla animals, so your mod requires vanilla cow.  So you can use the two together.   in Animania, it does have the option to not replace vanilla cows, so you can have both.
Just wondering if you might be thinking about using Animania cows to perform the ceremony to change into buffalo.

I already added that. The ceremony should work with Animania cows if nothing broke.


Add admin password to sentry inventory

The sentry.nos.social SSL cert didn't renew properly, and I tried my hand at fixing it this morning. I think the problem was that I ran the scripts without CLOUDFLARE_API_TOKEN set in my environment (or is it supposed to be in the vault now?).
I set that and tried to rerun the scripts, but ran into this error:
TASK [common : Perform a dist-upgrade.] *****************************************************************************************************************************************************************************
fatal: [sentry.nos.social]: FAILED! => {"msg": "Missing sudo password"}

I realized that I set this droplet up before new-do-droplet existed, so it didn't get quite the same setup that common and other roles expect. I tried to mimic the new-do-droplet role by creating an authorized_keys file for admin with our keys, adding admin to the sudoers file, setting a password for the admin user and adding it to the ansible vault and to inventories/sentry/inventory.yml. However running the playbook still gives me the error, so rather than spinning my wheels further I will wait for @cooldracula to tell me what is wrong.

I looked into the certificate and there were a couple things interfering with the renewal.
1.) The API key was not being accepted by cloudflare.  There was either a rollover of this key since it was added perhaps?  I updated the cloudflare config with the key I use in our other deployments and it renewed successfully.
2.) The renewal was missing a hook that restarts nginx, so the updated cert comes into effect.  This is part of our certbot role, but may have been added after sentry was deployed.  I added the hook to this deployment so it should renew automatically and successfully in the future.
Sentry is a tough one i relation to our ansible scripts.  It is similar to posthog where it has it's own deployment strategy and it is safer to just cede to it.  However, it's additionally hard because the deployment takes so long.  We have a sentry role, but I found it's not practical in real cases due to how long the deployment step takes--it is longer than a standard ssh connection, and so our ansible role (even with a number of async polling steps added) would be flaky and fail, wrongly thinking  the installation on the server had stopped.  I find with sentry it is easier to go to the server and run their steps, though it makes the server more of a snowflake.   I am not certain the best option yet.
For the immediate, the certificate is renewed and should renew successfully in the future.  For the ansible script, the sentry deployment is an edge case without a great experience in ansible, and I wonder if its role/playbook should be deprecated?


Doesn't work with scrbook

This package does not work with \documentclass{scrbook}. The first problem is that \refname is \bibname in scrbook, but only changing that doesn't seem to be sufficient to make it work.

Thanks for the bug report. I have fixed the \refname vs \bibname issue in 11bae77; the example document provided with apxproof now compiles when using the scrbook document class.
Can you let me know if this works for you, or if there are still remaining issues? If so, I would appreciate an example (possibly minimal) where the compilation fails.
You can download the latest development version of the package from https://raw.githubusercontent.com/PierreSenellart/apxproof/develop/apxproof.sty

Closing this issue as the main problem identified, \refname vs \bibname, was fixed in 11bae77, incorporated into version 1.0.6. Do not hesitate to open a new bug if there are other compilation issues.


Question about FID

Does the **num_samples** affects the FID?
I performed the sampling operation on the model lsun_bedroom.pt provided in the paper and then evaluated the generated .npz against the reference file  VIRTUAL_lsun_bedroom256.npz and did not get the scores from the article.

My evaluator scripts as following:

Can you give me some advices?
@longouyang @welinder @JoshuaGross @jietang @esigler

I got the same problem as the following:
Inception Score: 2.4110054969787598
FID: 68.9926382683044
sFID:<PHONE_NUMBER>906518
Precision: 0.61
Recall: 0.546

any updates on this?

I evaluate VIRTUAL_imagenet256.npz against itself but get the following resultes:

IS: 19.22413444519043
FID:<PHONE_NUMBER>7962674
sFID:<PHONE_NUMBER>168225
prec: 0.7860169491525424
recall: 0.0785
Is it correct?

I was having similar problem where the FID was very high, i.e., around 150+ for CIFAR-10 dataset.
When I included the diffusion flags such as DIFFUSION_FLAGS="--diffusion_steps 1000 --noise_schedule cosine" with the image_sample.py script, it helped me to reduce the FID: 16. Since I am not using the same hyperparameters as CIFAR-10 and training it for only 100k iterations FID values are not so bad.
Note: Reason how I figured out the issue was based on the generated samples. Try visualizing the image samples in generated npz file. If the images look weird there can be something wrong with the sampling script, assuming you use same or similar hyperparameters for training script as compared to the authors suggestion. In that case, you can try the above solution. I hope this helps!


Convert positional parameters into named ones

Not entirely sure how this works with Python, but I hate that we now have so many configuration options that must be supplied as positional parameters to the script.
I want to use named parameters, so that we can omit unwanted parameters more easily.
But we need to ensure backwards compatibility.

This was addressed as part of v4.0.0


Table: Should not have inline hard coded column widths

Detailed description

Describe in detail the issue you're having.

I'm using the TableHeaderItem.className property to set my column style including the width property which gets overwritten. The table is automatically attaching an inline width attribute to each column and the only way to overwrite it is by using !important in my class which I wish I don't have to use.
If you need to keep the width attribute then at least do it through a class that can be easily overwritten by the consumer application.

Is this a feature request (new component, new icon), a bug, or a general issue?

Bug

Is this issue related to a specific component?

Table

What version of the Carbon Design System are you using?

2.0.7

What offering/product do you work on? Any pressing ship or release dates we should be aware of?

ADI

Table component uses ngStyle. The width of the column inherits from the header item. You can actually set the width by doing :
new TableHeaderItem({
    data: "...",
    style: { "width": "50px" },
    ...
})

Thanks for the reply @esuau but inline styling is not in line with our development guidelines and we don't see it as a good practice.

You can set TableHeaderItem style as a workaround.

I could but it goes to the same point about maintainability.

So originally this API was put in place due to a requirement to have resizable (draggable) columns ... there's no good way to do drag-to-resize without inline styles.
That said, Carbon (currently) doesn't have a requirement for resizable columns ... I think a preferable default would be no inherit styling (unless specified in TableHeaderItem) and TableHeaderItem should gain a class config option ... that way you don't need to poke the carbon class hierarchy directly.

I agree that if I had resizeable columns then inline width is the way to go. My table is neither resizeable nor has draggable columns though :)
TableHeaderItem already has a class property :)

Ooops, there it is! className ... goes to show how familiar I am with the Table source ... okay, so then this is just a bug 👍

Thanks for fixing it @cal-smith!


Create SqueakStore class

Encapsulates all logic related to:

DB queries for inserting/getting squeaks/offers/payments
Events triggered by new squeaks, unlocked squeaks, new offers.

Class will used by:

SqueakController
NetworkHandler

Use squeak core object in squeak store class

https://github.com/squeaknode/squeaknode/pull/1886


CodeMirror addon json-lint.js not working

index.haml
%script(src="https://codemirror.net/lib/codemirror.js")
<EMAIL_ADDRESS>%script(src="https://codemirror.net/addon/lint/lint.js")
%script(src="https://codemirror.net/addon/lint/json-lint.js")
%link(src="https://codemirror.net/lib/codemirror.css")
%link(src="https://codemirror.net/addon/lint/lint.css")
%script(src="https://codemirror.net/mode/javascript/javascript.js")
%script(src="https://codemirror.net/mode/css/css.js")
%script(src="https://codemirror.net/")
%script(src="https://codemirror.net/")

%testarea#json

index.coffee
editor = CodeMirror.fromTextArea(document.getElementById('json'),
      lineNumbers: true
      mode: "application/json"
      lint: true
      gutters: ["CodeMirror-lint-markers"]
    )

No warnings shown on the page.

How could I make it work? I implemented this referencing the official doc and demo.

https://codemirror.net/doc/manual.html#addons
https://github.com/codemirror/CodeMirror/blob/master/demo/lint.html

I don't know haml, but since the demo works, I'd say you just have to figure out what you're doing different, and it's unlikely that there's an issue in the library causing this.


🐛 BUG:D1_ERROR: Wrong number of parameter bindings for SQL query.

Which Cloudflare product(s) does this pertain to?
D1
What version(s) of the tool(s) are you using?
3.60.3[wrangler]
What version of Node are you using?
No response
What operating system and version are you using?
Windows 11
Describe the Bug
Observed behavior
Run the following code:
const { results } = await env.DB.prepare(
    "SELECT * FROM test WHERE name = '?'",
)
    .bind("Bs Beverages")
    .all();

The following error is output:
D1_ERROR: Wrong number of parameter bindings for SQL query.
Expected behavior
The value is bound to "?" correctly.
Steps to reproduce
Run this: https://developers.cloudflare.com/d1/get-started/
Modify the SQL statement: put "?" in quotes.
My worker code:
export interface Env {
  // If you set another name in wrangler.toml as the value for 'binding',
  // replace "DB" with the variable name you defined.
  DB: D1Database;
}

export default {
  async fetch(request, env): Promise<Response> {
    const { pathname } = new URL(request.url);

    if (pathname === "/api/name") {
      // If you did not use `DB` as your binding name, change it here
      const { results } = await env.DB.prepare(
        "SELECT * FROM test WHERE name = '?'",
      )
        .bind("Bs Beverages")
        .all();
      return Response.json(results);
    }

    return new Response(
      "Call /api/beverages to see everyone who works at Bs Beverages",
    );
  },
} satisfies ExportedHandler<Env>;

Visit "/api/name"
Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
No response

Hi @toritreee,
Apologies for the confusing error message. Removing the single quotes around the ? should resolve the issue. For more details, please refer to the parameter binding documentation.
If you have any further questions, feel free to ask!


Syncing changes in master from release 171

Task name: KubernetesV1
Description: Sync changes to master that were pushed to release 171
Documentation changes required: (Y/N) N
Added unit tests: (Y/N) Y
Attached related issue: (Y/N) N
Checklist:

[ ] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

LGTM


CanaTree100 link broken

First, huge thanks for publishing all the data that you are using. I'm sure it will be very appreciated by lots of people around the world.
The link to the dataset CanaTree100 is broken. I'm getting the error message: Sorry, the link has expired.

Renewed, should work now

thanks


Roles that are no longer in configuration still show up in assume role menu

Roles that I previously listed in my aws-extend-switch-roles configuration still appear in my aws console roles menu. This occurs even when I clear all storage for the aws-console, but does not if I use a new profile. How do I clear data stored by the aws-extend-switch-roles plugin?
Within<EMAIL_ADDRESS>there are no files. Additionally, when the extension is remove and re-installed, the problem persists.

This is a duplicate of https://github.com/tilfin/aws-extend-switch-roles/issues/70. Closing.


Merge changes back to feature branch

replaced with https://github.com/microsoft/azure-maven-plugins/pull/994


Move Conway modules in cardano-ledger-conformance to testlib

          I think it's better to handle this in a separate PR, because it entails moving all `Test.Cardano.Ledger.Conformance.ExecSpecRule.Conway.*` modules to the `testlib`. I think it'd be a good idea to do that anyways, but I'd rather not do it in this PR.

Originally posted by @Soupstraw in https://github.com/IntersectMBO/cardano-ledger/pull/4496#discussion_r1703873231

See my comment on the original PR


data_dir lacks optional build tag

This is related to #171 but this issue impacts all wheels with a build tag.
For example, using the following command to create a wheel for a package named foo with version 0.4.2:
python setup.py bdist_wheel --build-number 1337

will create a wheel named foo-0.4.2-1337-py3-none-any.whl. The build_tag in:
https://github.com/pypa/installer/blob/444e5295f2403ab303dcf14160591074396f1d21/src/installer/utils.py#L66-L68
will be 1337.
Data files in the wheel will have paths such as foo-0.4.2-1337.data/some_folder/some_file.txt. Due to the implementation of data_dir:
https://github.com/pypa/installer/blob/444e5295f2403ab303dcf14160591074396f1d21/src/installer/sources.py#L49-L52
we'll thus have the following variable values here:
https://github.com/pypa/installer/blob/444e5295f2403ab303dcf14160591074396f1d21/src/installer/_core.py#L43
assert data_dir == "foo-0.4.2.data"
assert path == "foo-0.4.2-1337.data/some_folder/some_file.txt"
assert posixpath.commonprefix([data_dir, path]) == "foo-0.4.2"
assert posixpath.commonprefix([data_dir, path]) != data_dir
# -> the scheme is incorrectly determined as root_scheme.

Can the implementation of data_dir be changed to take into account the wheel's build tag?

This sounds like exactly the opposite of https://github.com/pypa/setuptools/issues/3997 and is probably a setuptools/wheel bug for generating a non spec-compliant artifact.

but this issue impacts all wheels with a build tag

Presumably it doesn't include any wheels that possess only modules/packages and don't have data files / headers / scripts. So it would impact the minority of wheels, really.

Presumably it doesn't include any wheels that possess only modules/packages and don't have data files / headers / scripts. So it would impact the minority of wheels, really.

Yes you're right: I meant to say: "impacts all wheels with a build tag and data files". I'll edit the issue description for future readers.


Override getConnection in PooledConnectionHandler so it returns the c…

Credit goes to @mccullen as this improves the API.


Pipe in ProfitWell Metrics API

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/32
Originally created by @wwright on 2018-07-31 20:23:58

ProfitWell, a company that supplies analysis on subscription based companies, has an API that pulls in certain metrics that we are having trouble getting at this time. We need to be able to understand retention, along with other metrics, in a more granular view. The ProfitWell API will allow us to get this information. Due to the complications and the experience gap of Zuora, using ProfitWells API will allow us to get the information almost immediately after ingested into our dw.
I can provide the API key when ready.
API documentation
https://profitwellapiv2.docs.apiary.io/#reference/authentication
https://profitwellapiv2.docs.apiary.io/#reference/metrics-api/monthly-metrics
cc: @pmachle, @tayloramurphy

View 11 previous comments from the original issue on GitLab


envsetup.sh is for linux

yes envsetup.sh is for Linux as that is what I use. Do you need something for Windows, or why did you open this issue?

c_u_stetup.txt
Change .txt extension as bat and con.csv should renamed as conn.csv

Thanks, that looks good!
all the category codes are three letters, so we use con instead of conn.
Can you open up a pull request with this change and perhaps add some documentation to README for windows?


Change grikkitog resistance from "all 10" to "physical 10"

The book (as seen on AoN) just says the resistance is "10 (except adamantine)", this is an editing mistake.  The data entry team decided that this means "all 10" in the past, but I believe it's much more likely to be "physical 10".
That is because every other monster with resistance that is pierced by adamantine always has it be a physical resistance.  See filter:  https://2e.aonprd.com/Search.aspx?q=type%3Acreature "except adamantine" -clockwork

Without a Paizo clarification we really shouldn’t editorialize. You’re probably right, but given that it just says “10” all is more appropriate.
I’ll leave it to the data entry leads to make a call (unless they disagree, in which case my tie break is leave all and ask Paizo to errata it)


Potential bug in preprocess_augmentations.py

Dear Organizer,
I am facing errors in running the following code in preprocess_augmentations.py.
for j, i in tqdm(enumerate(valid_files), desc=f"Resuming preprocessing. Validate {len(valid_files)} files"):
            test_file = os.path.join(self.destination, f"{i}_{self.samples_per_file - 1:03d}.npz")
            # Load and process data
            data = np.load(test_file)
            try:
                image = torch.from_numpy(data["input"])
                label = torch.from_numpy(data["label"])
                valid_files.append(test_file)
            except Exception:
                valid_files.pop(j)

        print(f"Found {len(valid_files)} valid files!")
        return valid_files

The code appends the test_file back to the list of valid_files, and when we this newly appended elemnt is enumerated, the new test_file formed from this element is an invalid path when self.destination is not an empty string, leading to error. And in addition, even if there is no error, I believe the enumeration might go on forever because there are constantly new elements appending.
And so I would like to ask the organizer for clarifications on how to address this issue. Thank you very much!
Yours sincerely,
Lennon

Hi Lennon,
thanks for the question. The valid_files.append(test_file) is a cp error and just needs to be removed. Beside that the code is just for validation the saved files since i had some broken files when using exessive amounts of cpus. You could easily ignore the section.
Best Jakob

Dear Jakob,
Thank you very much!
Your sincerely,
Lennon


Document Alternative ACS Base feature

Document Alternative ACS Base feature. Include caveat about Silverstripe CMS 5 and forced "no trailing slash" which caught-out the test (since cherry-pick from CMS 4 branch; also fixed).
https://docs.silverstripe.org/en/5/changelogs/5.0.0/#trailing-slash
Ran DOCTOC as directed by comments.
Silverstripe CMS rebranded a few years ago - reflected this in the docs where the old style wasn't code related (capital middle S)

If this is merged before other work then it will leave the 3 branch with no failing tests.


Add dynatrace integration

Adds dependency on libbuildpack-dynatrace
Adds dynatrace hook
Adds integration tests
Adds test fixures

This PR has been untouched for 2 months. Based on recent commits:
@arjun024 @cf-buildpacks-eng @ForestEckhardt Could you help out here?


Deprecate fromServer options & remove ability to specify model class on relationships  176874011

This PR is the first step in moving us away from using fromServer options it also removes the ability to specify model classes on relationships.
The only actual change that will need to be made in portal and mobile is that fetchOne method signature now takes options (ie {queryParams: {include: 'todos'}}) instead of the queryParams directly like it was (ie. {include: 'todos'})
The corresponding PRs are
https://github.com/artemis-ag/portal/pull/4719
https://github.com/artemis-ag/artemis-mobile/pull/345

@jtorreggiani do you have any thoughts / concerns?


pciutils: fix build on darwin

Makefile assumes CC=gcc, so we should use our own CC. To avoid affecting currently working derivations, this is only done when !stdenv.cc.isGNU.
Darwin also requires IOKit framework.
I'm adding preConfigure via optionalAttrs to avoid affecting any non-darwin derivations. Let me know if there's a clearer way to achieve this.
Motivation for this change
ZHF 20.09 https://github.com/NixOS/nixpkgs/issues/97479
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[x] other Linux distributions (Ubuntu x86_64)

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Let me know if there's a clearer way to achieve this.

Adding null attribute doesn't cause a rebuild, so using that with if is probably a bit more readable, but I don't mind either.

WFM macos 10.14


Add-Secret's Secret parameter should not be Object, but SecureString

When adding a secret as below without the Secret parameter, the secret will be visible.
>Add-Secret -Name test
cmdlet Add-Secret at command pipeline position 1
Supply values for the following parameters:
Secret: Sup3rS3cr3t

As you can see if the type is not SecureString, a pasted Secret will be printed which is not ideal for a secret management module.
>Get-Help Add-Secret -Parameter Secret | Select -ExpandProperty Type
name
----
Object

>Get-Command Add-Secret | Select Source, Version
Source                                                                      Version
------                                                                        ------
Microsoft.PowerShell.SecretsManagement                  0.2.0

Seems like this was already fixed in the master branch, just wasn't published?
https://github.com/PowerShell/SecretManagement/blob/master/src/code/SecretManagement.cs#L1366

@CJHarmath It's an object parameter because a secret can be multiple types: (securestring, byte[], PSCredential). It's not just strings.
In RC2 there is a SecureStringSecret parameter for this purpose to avoid the logging/output issue, it's just not surfaced well because the help isn't updated correctly.

@JustinGrote

Also there is no Add-Secret

Well, here is how I stumbled upon this

google microsoft secrets management
land in here https://devblogs.microsoft.com/powershell/secrets-management-development-release/
link to this package https://www.powershellgallery.com/packages/Microsoft.PowerShell.SecretsManagement/0.1.0-alpha1
install via Install-Module -Name Microsoft.PowerShell.SecretsManagement -RequiredVersion 0.1.0-alpha1 -AllowPrerelease

Which still has Add-Secret.

It's an object parameter because a secret can be multiple types

It's weird for me to say this on the main PowerShell org, but this approach is not the PowerShell way.
There is a difference between supporting multiple parameters with different types and trying to shoehorn an object into a single parameter so that it can support multiple types, then trying to workaround it's security implications by introducing another parameter.
All you need to do is to configure the default parameterset to be the one with SecureString and there is no password printing.

for credential it should be Set-Secret -Credential $cred - please don't reinvent this, it's a done deal for years
for SecureString Set-Secret -Secret $secret
for plaintext Set-Secret -Secret $secret -AsPlainText -Force which should do the same as ConvertTo-SecureString does for protecting privacy
for bytes Set-Secret -Bytes $bytes

https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.security/convertto-securestring?view=powershell-7.1#parameters
Thanks

@CJHarmath I am not part of the powershell team FYI, just providing context :)
There has been a discussion around Add-Secret in the past and you'll have to convince @PaulHigin otherwise why Add-Secret is important over Set-Secret. I agree with your assertion in that regard, there should be more cmdlets that implement the verbs correctly.
As far as parametersets, unfortunately supporting pipelining is what is requiring the Object input usually, you see this often with other cmdlets.
@PaulHigin can provide input but at this point it's past the RFC stage and into the release candidate production-supported stage and no longer alpha, so design changes are likely not going to happen at this point in terms of the parameters because they would constitute breaking changes,, but Add-Secret could be done without being a breaking change.

There has been a discussion around Add-Secret in the past and you'll have to convince @PaulHigin otherwise why Add-Secret is important over Set-Secret.

Actually, I don't mind Set-Secret. That's pretty much a standard.
Not having Get-Secret -List bothers me more and printing out password bothers me even more especially when it comes from a big tech company like Microsoft which has already solved this problem a while ago.

As far as parametersets, unfortunately supporting pipelining is what is requiring the Object input usually, you see this often with other cmdlets.

You can surely have multiple types piped in which then gets mapped to the right parameter set based on the object type.
Here is a dummy example:
function Test-ParamSets {
    [CmdletBinding(DefaultParameterSetName = 'SecureString')]
    param(
        [Parameter(Mandatory, Position=0)]
        [string]
        $Name,

        [Parameter(Mandatory = $true, ParameterSetName = 'SecureString',Position=1, ValueFromPipeline)]
        [SecureString]
        $SecureString,

        [Parameter(Mandatory = $true, ParameterSetName = 'PSCredential', Position=1, ValueFromPipeline)]
        [PSCredential]
        $Credential

    )
    $PSBoundParameters.Keys | where { $_ -ne "Name"} | % { $PSBoundParameters[$_].GetType().Name }
}

Then testing it
> $cred = Get-Credential foo

Windows PowerShell credential request
Enter your credentials.
Password for user foo: ******
> $cred | Test-ParamSets -Name foo

PSCredential
> $secret = Read-Host -AsSecureString -Prompt "type your secret here"

type your secret here: ************
> $secret | Test-ParamSets -Name secret

SecureString
>

So as you can see I am OK to pipe in a PSCredential or a SecureString and it correctly gets mapped to the right parameter.

Not having Get-Secret -List bothers me more and printing out password bothers me even more especially when it comes from a big tech company like Microsoft which has already solved this problem a while ago.

This exists as Get-SecretInfo. This was done on purpose to prevent people from dumping passwords accidentally. You have to explicitly call the secret you want with Get-Secret -Name <secret> but you can also pipe the secretinformation objects to Get-Secret for the same effect, this was considered enough of a barrier to indicate you really know what you are doing when you do this.
As far as the parameter types, I think there's some legacy stuff around that especially since it is a C# cmdlet. I'm not sure. @PaulHigin what do you think? What was the justification for using Object and then stricting-down the type later? Since it wouldn't effect the API to the extension modules, typed parameterSets could probably be added as a non-breaking change post 1.0

Ah, what I meant by Get-Secret -List is to just list out the secret names, not dumping anything sensitive - that would be really bad.
Looking at Get-SecretInfo seem to have the list at least, but it does not seem to be as useful in terms of searching for a secret based on some metadata for example.
I've wrapped HashiCorp's Vault before with PowerShell and some of it's features would have been nice to be factored in here too, such as metadata and list of key value pairs instead of just a dumb secret value.
$ vault kv put secret/hello foo=world excited=yes
$ vault kv get secret/hello

====== Metadata ======
Key              Value
---              -----
created_time     2020-09-02T21:41:17.568155Z
deletion_time    n/a
destroyed        false
version          2

===== Data =====
Key        Value
---        -----
excited    yes
foo        world

https://learn.hashicorp.com/tutorials/vault/getting-started-first-secret?in=vault/getting-started#writing-a-secret
Wonder how would this look like with a Vault extension?

One more thing.
At times you might need to store a user name and password but in other times might want to just get it back as a PSCredential.
So having a Get-Secret -Name foo -AsCredential would be nice in those cases when you just want to grab a credential don't want to deal with having to convert it. Little things matter.
When you store key value pairs like in Vault, these things are as easy as having a username and a password key.

Thanks for all the feedback @CJHarmath in our latest release we do include support for SecretMetadata (https://devblogs.microsoft.com/powershell/secretmanagement-and-secretstore-release-candidate-2/), key-value pairs can be saved using either a PSCredential or hashtable....we do have an -AsPlainText switch for Get-Secret and could consider adding -AsCredential in future releases

@CJHarmath the Get-SecretInfo -Filter  parameter is meant to be implemented however the extension author wants, so you can search on all kinds of metadata however you implement that syntax (you could do OData, KQL, whatever you want). For instance the Chromium extension has its own rudimentary filter syntax.
https://github.com/justingrote/secretmanagement.chromium#get-secretinfo-filter-usage

@CJHarmath  Your original example uses a very old alpha version of SecretManagement.  Be sure you are using the latest version in your testing.
https://www.powershellgallery.com/packages/Microsoft.PowerShell.SecretManagement/0.9.1

I am marking this as answered and closing the issue.


Updated ENSOInfluence notebook

Two updates to the ENSOInfluence notebook:

Updated to use dea_waterbodies.get_time_series instead of loading and parsing waterbody time series itself
Added seasonal checks and deseasonalising

Checklist (replace [ ] with [x] to check off)

[x] Notebook created using the DEA-notebooks template
[x] Remove any unused Python packages from Load packages
[x] Remove any unused/empty code cells
[x] Remove any guidance cells (e.g. General advice)
[x] Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended).
[x] Include relevant tags in the final notebook cell (refer to the DEA Tags Index, and re-use tags if possible)
[x] Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated
[ ] Test notebook on both the NCI and DEA Sandbox (flag if not working as part of PR and ask for help to solve if needed)
[ ] If applicable, update the Notebook currently compatible with the NCI|DEA Sandbox environment only line below the notebook title to reflect the environments the notebook is compatible with

Thank you!


Add client version of (graphql) blackboard to simplify client update

fun blackboard needs a separate list of new funs so that client knows what to do
ClientBlackboardAccessor

blackboard

graphql:
bba.get_new_mutations_and_queries()
bba.updateWithClientData(data)

internally clears out created mutations and queries
—-
functions:
bba.get_new_functions()
bba.update_with_new_client_functions()
internally resets the functions list to the new list

.

generate() should accept CBB to avoid exposing BB to client

rename main_service to main_generator
(up minor version)

done!


[1pt] Implement mission leaving - Actions

Follow the same logic as with the "Join mission" - but you need to set the reserved key to false.
Dispatch these actions upon click on the corresponding buttons.

This task is complete. Great work buddy


output.publicPath: "auto"

Feature to document
output.publicPath: "auto"
Only works for web and web worker targets.
Determines the publicPath automatically from <script> tag.
Author(s)
@sokra
Additional information
Should be listed on:

https://webpack.js.org/configuration/output/#outputpublicpath
https://webpack.js.org/guides/public-path/

[ ] I am willing to work on this issue and submit a pull request.

WIP


Exception on Refresh

Do you have any idea, @kcyarn, where in the code I might debug this? I'd be interested in making a PR.
Atom Exception Report
Steps leading up to the error:

Load a long Markdown document (ApiBlueprint).
Use the command palette to run Markdown Document: Toggle. No sidebar appears.
Use the command palette to run Markdown Document: Refreh.  This exception occurs...

Atom Version: 1.13.0
Electron Version: 1.3.13
System: Mac OS X 10.12.2
Thrown From: markdown-document package, v0.2.1
Stack Trace
Uncaught TypeError: this.refreshClick is not a function
At /Users/brad/.atom/packages/markdown-document/lib/markdown-document-view.coffee:324

TypeError: this.refreshClick is not a function
    at MarkdownDocumentView.module.exports.MarkdownDocumentView.refreshOutline (/Users/brad/.atom/packages/markdown-document/lib/markdown-document-view.coffee:324:6)
    at Object.refresh (/Users/brad/.atom/packages/markdown-document/lib/markdown-document.coffee:47:27)
    at atom-workspace.subscriptions.add.atom.commands.add.markdown-document:refresh (/Users/brad/.atom/packages/markdown-document/lib/markdown-document.coffee:30:93)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:259:29)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:59
    at CommandPaletteView.module.exports.CommandPaletteView.confirmed (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/command-palette/lib/command-palette-view.js:210:32)
    at CommandPaletteView.module.exports.SelectListView.confirmSelection (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-space-pen-views/lib/select-list-view.js:338:21)
    at space-pen-div.atom.commands.add.core:confirm (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-space-pen-views/lib/select-list-view.js:109:19)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:259:29)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:59
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:587:16)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:382:22)
    at WindowEventHandler.module.exports.WindowEventHandler.handleDocumentKeyEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:106:36)
    at HTMLDocument.<anonymous> (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:3:59)

Commands
     -9:46.3.0 find-and-replace:focus-next (input.hidden-input)
  8x -9:45.3.0 core:backspace (input.hidden-input)
     -9:37.7.0 core:save (input.hidden-input)
     -0:40.3.0 markdown-document:toggle (input.hidden-input)
     -0:33.2.0 command-palette:toggle (input.hidden-input)
  3x -0:27.8.0 core:move-down (input.hidden-input)
     -0:23.3.0 core:confirm (input.hidden-input)
     -0:23.3.0 markdown-document:toggle (input.hidden-input)
     -0:13.7.0 editor:consolidate-selections (input.hidden-input)
     -0:13.7.0 core:cancel (input.hidden-input)
     -0:08.7.0 command-palette:toggle (input.hidden-input)
  4x -0:02.6.0 core:move-down (input.hidden-input)
     -0:00.1.0 core:confirm (input.hidden-input)
     -0:00.1.0 markdown-document:refresh (input.hidden-input)

Config
{
  "core": {
    "disabledPackages": [
      "go-find-references",
      "go-signature-statusbar",
      "godef",
      "markdown-table-formatter"
    ],
    "telemetryConsent": "limited",
    "themes": [
      "one-dark-ui",
      "solarized-light-syntax"
    ]
  }
}

Installed Packages
# User
atom-beautify, v0.29.16 (active)
git-plus, v7.1.0 (active)
go-debug, v1.0.3 (active)
go-find-references, v1.2.3 (inactive)
go-outline, v0.6.2 (inactive)
go-plus, v5.1.0 (active)
go-rename, v0.3.0 (active)
go-signature-statusbar, v1.2.0 (inactive)
godef, v0.2.5 (inactive)
hyperclick, v0.0.39 (active)
linter, v1.11.21 (active)
linter-golint, v0.3.0 (active)
linter-golinter, v1.2.1 (active)
markdown-document, v0.2.1 (active)
markdown-table-formatter, v2.8.4 (inactive)
markdown-toc, v0.4.2 (inactive)
markdown-writer, v2.6.3 (active)
project-plus, v0.9.0 (active)
atom-dark-syntax, v0.28.0 (inactive)
atom-dark-ui, v0.53.0 (inactive)
atom-light-syntax, v0.29.0 (inactive)
atom-light-ui, v0.46.0 (inactive)
base16-tomorrow-dark-theme, v1.4.0 (inactive)
base16-tomorrow-light-theme, v1.4.0 (inactive)
one-dark-ui, v1.8.2 (active)
one-light-ui, v1.8.2 (inactive)
one-dark-syntax, v1.6.0 (inactive)
one-light-syntax, v1.6.0 (inactive)
solarized-dark-syntax, v1.1.1 (inactive)
solarized-light-syntax, v1.1.1 (active)
about, v1.7.2 (active)
archive-view, v0.62.0 (active)
autocomplete-atom-api, v0.10.0 (active)
autocomplete-css, v0.14.1 (active)
autocomplete-html, v0.7.2 (active)
autocomplete-plus, v2.33.1 (active)
autocomplete-snippets, v1.11.0 (active)
autoflow, v0.29.0 (inactive)
autosave, v0.23.2 (active)
background-tips, v0.26.1 (active)
bookmarks, v0.43.2 (active)
bracket-matcher, v0.85.1 (active)
command-palette, v0.39.1 (active)
deprecation-cop, v0.55.1 (active)
dev-live-reload, v0.47.0 (active)
encoding-selector, v0.22.0 (active)
exception-reporting, v0.40.0 (active)
find-and-replace, v0.204.5 (active)
fuzzy-finder, v1.4.0 (active)
git-diff, v1.2.0 (active)
go-to-line, v0.31.2 (inactive)
grammar-selector, v0.48.2 (active)
image-view, v0.60.0 (active)
incompatible-packages, v0.26.1 (active)
keybinding-resolver, v0.35.0 (active)
line-ending-selector, v0.5.1 (active)
link, v0.31.2 (inactive)
markdown-preview, v0.159.1 (active)
metrics, v1.1.2 (active)
notifications, v0.65.1 (active)
open-on-github, v1.2.1 (inactive)
package-generator, v1.0.2 (inactive)
settings-view, v0.244.0 (active)
snippets, v1.0.4 (active)
spell-check, v0.68.5 (active)
status-bar, v1.6.0 (active)
styleguide, v0.48.0 (active)
symbols-view, v0.113.1 (inactive)
tabs, v0.103.1 (active)
timecop, v0.33.2 (active)
tree-view, v0.211.1 (active)
update-package-dependencies, v0.10.0 (active)
welcome, v0.35.1 (active)
whitespace, v0.35.0 (active)
wrap-guide, v0.39.0 (active)
language-c, v0.54.0 (active)
language-clojure, v0.22.1 (active)
language-coffee-script, v0.48.1 (active)
language-csharp, v0.13.0 (active)
language-css, v0.40.1 (active)
language-gfm, v0.88.0 (active)
language-git, v0.15.0 (active)
language-go, v0.43.0 (active)
language-html, v0.47.1 (active)
language-hyperlink, v0.16.1 (active)
language-java, v0.24.0 (active)
language-javascript, v0.122.0 (active)
language-json, v0.18.3 (active)
language-less, v0.29.6 (active)
language-make, v0.22.2 (active)
language-mustache, v0.13.0 (active)
language-objective-c, v0.15.1 (active)
language-perl, v0.37.0 (active)
language-php, v0.37.3 (active)
language-property-list, v0.8.0 (active)
language-python, v0.45.1 (active)
language-ruby, v0.70.2 (active)
language-ruby-on-rails, v0.25.1 (active)
language-sass, v0.57.0 (active)
language-shellscript, v0.23.0 (active)
language-source, v0.9.0 (active)
language-sql, v0.25.0 (active)
language-text, v0.7.1 (active)
language-todo, v0.29.1 (active)
language-toml, v0.18.1 (active)
language-xml, v0.34.12 (active)
language-yaml, v0.27.1 (active)

# Dev
No dev packages

You want the master branch, file lib/markdown-document-view.coffee,
which contains everything. I didn't break it into modules and really
don't plan to at this time. I don't normally break things out unless I
exceed 500 lines. If not having everything grouped within the same
file makes you feel more comfortable, feel free to change it.
Markdown-document currently uses a panel to display the outline. In
the past, this did not interfere with the spellcheck. As of Atom
1.12.7, it does. I don't know if this is a conflict with atom itself
or the spellcheck package. Without checking the change logs for Atom
1.13, I don't know if this problem got worse, but I assume it did.
The outline is Angular 1 because that was the quickest way to
implement the auto refresh that didn't conflict with atom. I
restructured the code for 0.2.1, but didn't rewrite it. I just moved
things around until it played nicely with atom's latest version of
coffee script. I don't think I tested the refreshclick function. I
never use it and actually forgot it existed until I read this this
morning so uh...yeah...oops. Off the top of my head, I think I made it
part of the angular component for some bizarre reason. If I did, then
@refreshClick() is looking for a function that is now contained within
the angular 1 controller.
I currently run Atom 1.12.7. I typically run several updates behind
the latest version because every time I update Atom multiple crucial
to my workflow packages break. I'm skipping the 1.13 update. I just
can't justify the time needed to change my terminal package again.
Sorry. I know that's not helpful for troubleshooting.
Sent from My Phone.
I'll check the code tonight on a larger screen and verify this is what I did.
On Thu, Jan 19, 2017 at 12:09 PM, Brad Olson<EMAIL_ADDRESS>wrote:

Do you have any idea, @kcyarn https://github.com/kcyarn, where in the
code I might debug this? I'd be interested in making a PR.
Atom Exception Report Steps leading up to the error:

Load a long Markdown document (ApiBlueprint).
Use the command palette to run Markdown Document: Toggle. No
sidebar appears.
Use the command palette to run Markdown Document: Refreh. This
exception occurs...

Atom Version: 1.13.0
Electron Version: 1.3.13
System: Mac OS X 10.12.2
Thrown From: markdown-document
https://github.com/kcyarn/markdown-document package, v0.2.1
Stack Trace
Uncaught TypeError: this.refreshClick is not a function
At /Users/brad/.atom/packages/markdown-document/lib/markdown-document-view.coffee:324
TypeError: this.refreshClick is not a function
at MarkdownDocumentView.module.exports.MarkdownDocumentView.refreshOutline (/Users/brad/.atom/packages/markdown-document/lib/markdown-document-view.coffee:324:6)
at Object.refresh (/Users/brad/.atom/packages/markdown-document/lib/markdown-document.coffee:47:27)
at atom-workspace.subscriptions.add.atom.commands.add.markdown-document:refresh (/Users/brad/.atom/packages/markdown-document/lib/markdown-document.coffee:30:93)
at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:259:29)
at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:59
at CommandPaletteView.module.exports.CommandPaletteView.confirmed (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/command-palette/lib/command-palette-view.js:210:32)
at CommandPaletteView.module.exports.SelectListView.confirmSelection (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-space-pen-views/lib/select-list-view.js:338:21)
at space-pen-div.atom.commands.add.core:confirm (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-space-pen-views/lib/select-list-view.js:109:19)
at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:259:29)
at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:59
at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:587:16)
at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:382:22)
at WindowEventHandler.module.exports.WindowEventHandler.handleDocumentKeyEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:106:36)
at HTMLDocument.
Reply to this email directly, view it on GitHub
https://github.com/kcyarn/markdown-document/issues/15, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAVT9N20tMSThgx4A3dog4NjgvFT-E_cks5rT5jMgaJpZM4LoXMz
.

Thanks @kcyarn. I like the idea of going somewhere Scrivener-like with this. (I like Scrivener, but miss the editing power of Atom or Sublime when I use it. Seems to me there was a little unnecessary finickiness about Markdown too.)

I have sketches, the beginnings of an angular 2 electron/cordova app
because I desperately need the ability to edit the same project on my
tablet and phone as on my PC, using a similar interface, and with the
option of at least reading my notes without having to open a separate file
or scroll up/down in the current file. I almost hate myself for thinking I
should just go with medium.js (html barebones wysiwyg that plays nicely
with android chrome) and either convert it to markdown to save into
individual files or just save those files as html. I'm leaning toward
markdown because of diff. Oh, and outside of compiling (everything I do
ends up as latex to pdf for print or epub/mobi) I need version control
(git) with the provider of my choice. I haven't pushed this anywhere yet
and I only get to work on it in thirty minute spurts, but I completely get
what you're saying.
My biggest issue with scrivener is and always has been its rtf files and
how it compiles those files into epubs that don't validate, mobis that
freeze up, and html that looks almost as bad as what I'd get out of word.
The quasi-markdown approach they suggest for markdown use is almost as bad.
I actually hate writing in markdown, but I like using the asterisks as
shortcuts.
I originally saw myself doing all this through an atom package with a
separate cordova interface using the same files, but not anymore. At least
with electron, I can fix the version number and should that ever fail
swapping over to nwjs isn't a deal breaker.
On Thu, Jan 19, 2017 at 2:31 PM, Brad Olson<EMAIL_ADDRESS>wrote:

Thanks @kcyarn https://github.com/kcyarn. I like the idea of going
somewhere Scrivener-like with this. (I like Scrivener, but miss the editing
power of Atom or Sublime when I use it. Seems to me there was a little
unnecessary finickiness about Markdown too.)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kcyarn/markdown-document/issues/15#issuecomment-273874391,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVT9JsHNz-HGW5bkdOAlHg3NZnXQoe6ks5rT7oSgaJpZM4LoXMz
.

I have the same problem. as the original poster.

After reviewing the alternatives, I've deprecated this in favor of document-outline, which seems to accomplish the same thing. Sorry for the wait.


Retraining From Existing Weights (Model)

There seems to be a problem with starting retraining from an existing locally saved model. When I start training by calling model.user_release(), after creating an empty model (as in the docs), the training progresses fine. However, let's say, if I stop after one epoch, save my model using my_model.save('mymodel.h5') and then begin training again from this point by calling mymodel_new = deepforest.deepforest('mymodel.h5') the initial loss is too much (more than 1) and not where I left previously. After complete cycles of training, the mean precision comes zero and loss doesn't reduce. I have tried it with the provided model by calling  mymodel = deepforest.deepforest('NEON.h5') and the results remain same. I have debugged a lot and couldn't exactly pick, what parameter should I set like a checkpoint or callback.
Is there a right method in DeepForest package to do it or should I just explore standard ways to do it in keras?
Thanks.

Thanks for this message, I’ll have a look this afternoon. My initial
thought is that this is a different between keras save model and Keras save
weights. When you save just the weights you don’t see the state of the
optimizer. This is definitely upstream of this package, but I’m happy to
provide a guess. To summarize the question is how best to pause and restart
a DeepForest model.  I’ll look into it. Just out of curiosity, because this
package is brand new, can you give me a sense of your workflow, any overall
suggestions, how you found this, maybe drop a sample picture. We are still
gathering information about how best to engage with users
On Fri, Apr 10, 2020 at 7:38 AM Asim D. Bakhshi<EMAIL_ADDRESS>wrote:

There seems to be a problem with starting retraining from an existing
locally saved model. When I start training by calling model.user_release(),
after creating an empty model (as in the docs), the training progresses
fine. However, let's say, if I stop after one epoch, save my model using
my_model.save('mymodel.h5') and then begin training again from this point
by calling mymodel_new = deepforest.deepforest('mymodel.h5') the initial
loss is too much (more than 1) and not where I left previously. After
complete cycles of training, the mean precision comes zero and loss doesn't
reduce. I have tried it with the provided model by calling mymodel =
deepforest.deepforest('NEON.h5') and the results remain same. I have
debugged a lot and couldn't exactly pick, what parameter should I set like
a checkpoint or callback.
Is there a right method in DeepForest package to do it or should I just
explore standard ways to do it in keras?
Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/weecology/DeepForest/issues/102, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJHBLFG6GPRGOSOTPT4DBDRL4VNXANCNFSM4MFPUKTA
.

--
Ben Weinstein, Ph.D.
Postdoctoral Fellow
University of Florida
http://benweinstein.weebly.com/

Thanks for this message, I’ll have a look this afternoon. My initial thought is that this is a different between keras save model and Keras save weights. When you save just the weights you don’t see the state of the optimizer. This is definitely upstream of this package, but I’m happy to provide a guess. To summarize the question is how best to pause and restart a DeepForest model. I’ll look into it. Just out of curiosity, because this package is brand new, can you give me a sense of your workflow, any overall suggestions, how you found this, maybe drop a sample picture. We are still gathering information about how best to engage with users
On Fri, Apr 10, 2020 at 7:38 AM Asim D. Bakhshi @.***> wrote: There seems to be a problem with starting retraining from an existing locally saved model. When I start training by calling model.user_release(), after creating an empty model (as in the docs), the training progresses fine. However, let's say, if I stop after one epoch, save my model using my_model.save('mymodel.h5') and then begin training again from this point by calling mymodel_new = deepforest.deepforest('mymodel.h5') the initial loss is too much (more than 1) and not where I left previously. After complete cycles of training, the mean precision comes zero and loss doesn't reduce. I have tried it with the provided model by calling mymodel = deepforest.deepforest('NEON.h5') and the results remain same. I have debugged a lot and couldn't exactly pick, what parameter should I set like a checkpoint or callback. Is there a right method in DeepForest package to do it or should I just explore standard ways to do it in keras? Thanks. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#102>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJHBLFG6GPRGOSOTPT4DBDRL4VNXANCNFSM4MFPUKTA .
-- Ben Weinstein, Ph.D. Postdoctoral Fellow University of Florida http://benweinstein.weebly.com/

Thank you, Ben. The model.save() method saves both the architecture, that is sequential or functional flow, optimizer choice etc, and the weights into a single HDF5 format file. This is the method that is demonstrated in the DeepForest docs. The model.save_weights() of Keras just saves the weights and not everything else required to reload the model. The Keras model.load() reloads the model but cannot start retraining from the same state. For this purpose there is keras.callbacks.Callback. So my question boils down to this:
I assume 'deepforest.deepforest('thismodel.h5') reloads a saved model (architecture and weights) and we can use it to make predictions. Can we use callback methods using the 'deepforest()' library or is there a standard tested way to do that?
As for the other question of feedback, I think DeepForest is a very useful package for quick application of RetinaNet on detecting tree canopies. I have GIS imagery from which I want to do that. DeepForest is a great starting point to straightaway do that rather than playing with RetinaNet ab initio. However, since it's a project that have presumably evolved from your ecological research, it's use-case generalization is kind of questionable. Therefore, it is excellent for remote sensing or other peripheral applied research but requires tweaking if one wants to work in signal processing or machine learning problems related to ecology. For instance, if I plan to improve RetinaNet for specific purposes of crop analysis, DeepForest doesn't quickly provide a useful starting point.
Secondly, again from generalization perspective, there are choices which were presumably taken from the standpoint of a particular problem at hand. For instance, I debugged through the utilities.py' and changed inttypecast of annotations tofloat' since, in my case, exported JSON or xml were in floating point. But when I changed the typecast, there were errors so I changed utilities.py again and rounded all my floats and typecasted back to int. Not a great way to do it, but I was just doing it to get through my problem at hand. I think a lot of such improvements can be done which would generalize this excellent package.
I think with some improvements, for specific applications of crop or forest analysis, DeepForest can be a good transfer learning point of RetinaNet. Dense environments with focal loss are quite varying and when I go about detecting certain kind of trees and design an optimized RetinaNet (using DeepForest), I should be able to transfer learn from other optimized networks made using DeepForest. Perhaps ambitious but good for ecological and remote sensing researchers.
Best regards

To summarize the question is how best to pause and restart a DeepForest model. I’ll look into it. Just out of curiosity, because this package is brand new, can you give me a sense of your workflow, any overall suggestions, how you found this, maybe drop a sample picture.>

I was wondering why I thought at first place about pausing and retraining from the existing saved model and these lines in section 3.4 of DeepForest docs probably nudged me there.

We envision that for the majority of scientific applications atleast some finetuning of the prebuilt model will be worthwhile. When starting from the prebuilt model for training, we have found that 5-10 epochs is sufficient. We have never seen a retraining task that improved after 10 epochs, but it is possible if there are very large datasets with very diverse classes.>

What exactly does finetuning of the prebuilt model means? I thought it means loading the saved model on my local disk (may it be NEON.h5 or the model that I have retrained on my data) and then training it further with more data. I now realize that it probably means transfer learning from weights of NEON.h5 after creating an empty model with deepforest.deepforest() and then calling deepforest.use_release(). Alternatively, it may mean transfer learning from RetinaNet weights. Not sure which of the two these lines in the doc means. Please clarify this. Thanks.

In continuation to my last comment, I guess I have pretty much clarified my concern to myself. I was perhaps confused and haven't explored all the options. Apologies, if I have wasted your time or cluttered the space. However, not deleting my last comment since it might help someone.
I think the key is the model.config["weights"] parameter which can be set to point the empty model created by deepforest.deepforest() to desired weights. I haven't tested it but I'll do it very soon with a toy file with number of trees that I can count exactly with my eyes so I can match ground truth with predicted ground truth by hand. I hope it will clear a lot.
Thanks for your time.
Asim D. Bakhshi

Glad its cleared up. Thanks for being an early user. Open issues for EVERYTHING that feels like it could be improved. I already know how it all works, so i'm looking for feedback.


[wasm-api] Adopt consistent naming conventions & package structures

This issue is intended as a point of reference for the upcoming breaking naming changes in various wasm-api-* packages, which are all still in ALPHA, regardless of version number...
Naming conventions for custom WASM API modules
The names of each custom WASM API module's TypeScript class should be a PascalCase'd version of the name of its parent package. The name of the interface for declared WASM exports should be the same name, suffixed with Exports, e.g.:

Package name
Class name
Interface name

@thi.ng/wasm-api-canvas
WasmCanvas
WasmCanvasExports

@thi.ng/wasm-api-dom
WasmDom
WasmDomExports

@thi.ng/wasm-api-schedule
WasmSchedule
WasmScheduleExports

C & Zig import paths
Any package exposing headers or libraries for C or Zig should place relevant files into following package structure:

Language
Relative path

C
/include/wasmapi_xxx.h*

Zig
/zig/lib.zig

* The xxx suffix would be related to the actual package name, e.g. wasmapi_dom.h
Zig import names
For consistency, the short name for Zig imports should be the unscoped name of the parent NPM package, instead of the currently somewhat arbitrarily chosen short name. E.g.:
const wasm = @import("wasm-api");
const bindgen = @import("wasm-api-bindgen");
const dom = @import("wasm-api-dom");
const schedule = @import("wasm-api-schedule");

Conclusion
Taken together, these changes will make it much easier for users to determine the correct names and import paths, even without IDE auto-completion...

Done

Related addition (2024) about the new WasmModuleSpec-based API module declarations:
Each of the support packages are now also providing the following module specs:

Package name
Module spec

@thi.ng/wasm-api-canvas
WasmCanvasModule

@thi.ng/wasm-api-dom
WasmDomModule

@thi.ng/wasm-api-schedule
WasmScheduleModule

@thi.ng/wasm-api-webgl
WasmWebGLModule

See Custom API modules for more info about how these module specs are used...


Fix scrolling on keyboard/controller input

This PR targets the 4.x branches, has this issue also in 4.0 stable.
This resolves an issue with the popup menu's weird scrolling behavior when using the keyboard/controller, as mentioned in #80224 . It fixes the problem and also includes some code refactoring.
_Production edit: Fixes https://github.com/godotengine/godot/issues/80224, fixes https://github.com/godotengine/godot/issues/42230

I was also working on a PR to fix this bug and I chose an alternate way to deal with scrolling position :
Instead of scrolling when reaching the bottom or the top of the scrollcontainer, it's scrolling if we reach the middle of the scrollcontainer. Here is a gif. I feel this is smoother and allow to see better what is above and under the selected item.
What do you think about it ?

Good work @jmb462, I thought of the same but for the intuitive usability, I kept the way I did it.

This could solve also #42230

@YuriSizov have a look.

Note that it does not fix #42230. PopupMenu needs something like #63168

Thanks!

Cherry-picked for 4.1.3.


[REGRESSION] get-stdin can no longer be required

My scripts now throw:
let getStdin = require('get-stdin');

node:internal/modules/cjs/loader:1126
      throw new ERR_REQUIRE_ESM(filename, parentPath, packageJsonPath);
      ^

Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: /usr/local/lib/node_modules/get-stdin/index.js
require() of ES modules is not supported.
require() of /usr/local/lib/node_modules/get-stdin/index.js from /Users/evar/scripts/javascript/readtime.js is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename index.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from /usr/local/lib/node_modules/get-stdin/package.json.

    at new NodeError (node:internal/errors:363:5)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1126:13)
    at Module.load (node:internal/modules/cjs/loader:989:32)
    at Function.Module._load (node:internal/modules/cjs/loader:829:14)
    at Module.require (node:internal/modules/cjs/loader:1013:19)
    at require (node:internal/modules/cjs/helpers:93:18)
    at Object.<anonymous> (/Users/evar/scripts/javascript/readtime.js:4:16)
    at Module._compile (node:internal/modules/cjs/loader:1109:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1138:10)
    at Module.load (node:internal/modules/cjs/loader:989:32) {
  code: 'ERR_REQUIRE_ESM'
}

Please read the release notes.

@sindresorhus I read them, but they do not contain a migration path for CLI scripts that use global packages?
I have tried
let getStdin = await import('get-stdin/index.js');

let getStdin = await import('get-stdin');

Or even changing the file extension to .mjs and using the normal import, but they all fail:
node:internal/process/promises:227
        triggerUncaughtException(err, true /* fromPromise */);
        ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'get-stdin' imported from /Users/evar/scripts/javascript/readtime.js
    at new NodeError (node:internal/errors:363:5)
    at packageResolve (node:internal/modules/esm/resolve:698:9)
    at moduleResolve (node:internal/modules/esm/resolve:739:18)
    at Loader.defaultResolve [as _resolve] (node:internal/modules/esm/resolve:853:11)
    at Loader.resolve (node:internal/modules/esm/loader:89:40)
    at Loader.getModuleJob (node:internal/modules/esm/loader:242:28)
    at Loader.import (node:internal/modules/esm/loader:177:28)
    at importModuleDynamically (node:internal/modules/cjs/loader:1049:29)
    at importModuleDynamicallyWrapper (node:internal/vm/module:437:21)
    at importModuleDynamically (node:vm:382:43) {
  code: 'ERR_MODULE_NOT_FOUND'
}

@NightMachinary I’d suggest:

Opening an issue in https://github.com/nodejs/help/
Provide full steps to reproduce for the await import('get-stdin'); variant#
Tagging me in that issue

unless by “global packages” you’re referring to NODE_PATH, which is just generally unsupported with ES modules and not specific to this package at all: https://nodejs.org/api/esm.html#esm_no_node_path

Just move to ESM. There's really no reason not do it for scripts.

As a foundation library, it should ensure compatibility as much as possible, rather than using more advanced technologies.


Automatic PR from RestAPI-PR1183

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/1183

Codecov Report

Merging #1148 into master will decrease coverage by 0.06%.
The diff coverage is 34.37%.

@@            Coverage Diff             @@
##           master    #1148      +/-   ##
==========================================
- Coverage   55.14%   55.07%   -0.07%
==========================================
  Files        2168     2175       +7
  Lines       60461    60642     +181
==========================================
+ Hits        33340    33398      +58
- Misses      27121    27244     +123

Impacted Files
Coverage Δ

...onitor/models/location_threshold_rule_condition.py
55.55% <ø> (ø)
:arrow_up:

...itor/operations/tenant_activity_logs_operations.py
33.33% <ø> (ø)
:arrow_up:

azure-monitor/azure/monitor/monitor_client.py
100% <ø> (ø)
:arrow_up:

...tor/azure/monitor/operations/metrics_operations.py
93.93% <ø> (ø)
:arrow_up:

...onitor/operations/metric_definitions_operations.py
90.9% <ø> (ø)
:arrow_up:

...ure/monitor/operations/activity_logs_operations.py
94.28% <ø> (ø)
:arrow_up:

.../monitor/models/activity_log_alert_action_group.py
71.42% <ø> (ø)
:arrow_up:

...or/azure/mgmt/monitor/monitor_management_client.py
100% <100%> (ø)
:arrow_up:

...mgmt-monitor/azure/mgmt/monitor/models/__init__.py
100% <100%> (ø)
:arrow_up:

...-monitor/azure/mgmt/monitor/operations/__init__.py
100% <100%> (ø)
:arrow_up:

... and 19 more

Continue to review full report at Codecov.

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


[PROPOSAL] Support flink session mode

Purpose:
The documentation states that only application mode is supported. Is it possible to support session mode as well?

Unfortunately there's no way to register JobListener with constructor parameter in this way.


Fid Score does not match the paper

Hey when I try to run the code the fid scores calculated does not match the fid score in the paper. I am getting :

fid_w
fid_no_w

zero
26.518
25.57

ring
26.007
25.57

rand
25.676
25.57

ImageNet ring
66.342
Cell

while the paper states this:

Hi, thanks for letting me know! I think one issue I found is that I copied the wrong config for imagenet, and it should use --w_radius 3 --w_channel 1.
Can you try it again with: python run_tree_ring_watermark_imagenet_fid.py --run_name imgnet_fid_run --gt_data imagenet --model_id 256x256_diffusion --w_radius 3 --w_channel 1 --w_pattern ring --start 0 --end 10000 --with_tracking

for : python run_tree_ring_watermark_imagenet_fid.py --run_name imgnet_fid_run --gt_data imagenet --model_id 256x256_diffusion --w_radius 3 --w_channel 1 --w_pattern ring --start 0 --end 10000 --with_tracking
it's coming 29.855

Thank you. will be waiting for the good news.

Hi @unforgettablexD , I have ran python run_tree_ring_watermark_imagenet_fid.py --run_name imgnet_fid --gt_data imagenet --model_id 256x256_diffusion --w_radius 2 --w_channel 2 --w_pattern ring --start 0 --end 10000 --with_tracking --run_no_w, where I used channel 2 for watermarking. I got 19.40 for non-watermarked case and 18.11 for watermarked case.
I think there are two things you want to try to get even lower FID scores (I learned from how other people do FID in diffusion papers):

One thing I remember I did is to add more images to the ground truth set. In the paper, I think I used the whole imagenet as the ground truth set, and the experiment I did above used 50000 images. I think having more generated images also helps.
You may also want to resize/crop the ground truth images to 256x256, since we are generating 256x265 images.

Hope this helps, and let me know if you have further questions!


[FEATURE] Support application/vnd.api+json media type

Schemathesis is not properly working when "application/vnd.api+json" media type is defined at openapi.yaml endpoint.
I tried registering the serializer inside my test code but still didn't work.
This is my code:

@register("application/vnd.api+json")
class VndApiSerializer:
    def as_requests(self, context: SerializerContext, value: Any) -> Dict[str, Any]:
        return _to_json(value)

    def as_werkzeug(self, context: SerializerContext, value: Any) -> Dict[str, Any]:
        return _to_json(value)

And the returned error:

E SerializationNotPossible: Schemathesis can't serialize data to any of the defined media types: application/vnd.api+json'
E You can register your own serializer with schemathesis.serializers.register and Schemathesis will be able to make API calls with this media type.
E See https://schemathesis.readthedocs.io/en/stable/how.html#payload-serialization for more information.
/usr/local/lib/python3.10/dist-packages/schemathesis/specs/openapi/_hypothesis.py:172: SerializationNotPossible

Hi! Thank you for the report
The error message has a single quote char in the end:
E SerializationNotPossible: Schemathesis can't serialize data to any of the defined media types: application/vnd.api+json'

The error message template does not have it, so I assume it comes from the schema. Is it possible that there is a typo in the schema, e.g. a trailing single quote?
However, I think Schemathesis should work with this media type out of the box anyway:
In [1]: import schemathesis

In [2]: schemathesis.utils.is_json_media_type("application/vnd.api+json")
Out[2]: True

Oh, you are right. Thanks a lot for your help.


fix to handle text-only version

code has to be #ifdef'd out, since the struct definition
is different and xo_color_map_{fg,bg} do not exist
making compilation impossible.

Apologies; I should have added a build for text-only; I didn't build it after adding colors.
Thanks for the patch.
Thanks,
Phil

Please take a look at this patch:
https://gist.github.com/philshafer/65ab990bac1047dd437f576647edefe6
I've tried to keep the code as consistent as possible, adding a xo_text_only function to allow the optimizer to do the work, and ifdefing out only the color_map access lines.  Running strings on the resultant library shows that "underline" and "magenta" don't appear in the text-only version, and the appropriate functions are gone as well.  Ifdefing out more code was too much of a ripple, since other functions (xo_color_find, etc) would have to be ifdef'd out as well.
Thanks,
Phil


Support: plan for cmake build system

Dear espressif,
we are about to start evaluating esp32 for a product.
We see that you have a features/cmake branch on this git repository that support cmake (and we like cmake)

what are your plan regarding cmake? did you plan to switch from the Makefile based system to cmake or only to use it on the windows build?
idf.py : this is needed? can we call cmake directly (more useful in automated build)?
is flash encryption and secure boot supported also with cmake?
can you please generate the documentation on readthedocs also for the cmake branch?
Thanks
Nicola Lunghi

Hi @nicola-lunghi ,
Very glad to hear you're interested in the CMake support! The CMake-based build system is currently in a preview release. There is an announcemnt on our forum with more details (it's not linked from Github, unfortunately):
https://esp32.com/viewtopic.php?f=13&t=5559&p=24109
I think the forum post addresses the questions you have here, but I'll repeat answers below just in case:

We plan to merge the CMake-based build system into our master branch after the upcoming IDF v3.1 release candidate. At this point, CMake will become the default build system. However we will continue to also support the older GNU Make based system for IDF v3.x and possibly longer, depending on how the migration goes.
idf.py is not needed, it's a wrapper to automate some non-build-related functions (like flashing) for command line users. However CMake can be called directly. You can find some more details about this here.
Flash encryption and Secure Boot are currently not supported (the implementation is there in parts, but the full workflow for enabling these features is not complete). Support will be added ASAP.
You can find docs for the CMake branch here: https://docs.espressif.com/projects/esp-idf/en/feature-cmake/ (the Getting Started guides and Build System API Guide both deal with the CMake build system, exclusively.)

Nice!
Thank you for your response.

No problems. If you use the CMake branch and encounter any problems, please raise issues on github and we'll take a look ASAP. Or if you have any more questions I'm happy to answer them, here or on the forum.
The forum post will be updated once some of the missing support (secure boot, flash encryption, etc) is available in CMake.


For issue #92

Checks for dependencies

tesseract
imagemagick

Better message when dependencies not found
Solves #92

Thanks. I've moved your checks around a bit. Function is the same, but there are some benefits worth learning:

fewer if-else functions.
fewer calls to find_executable()
more logical to have checks at the top. Easier to read later because instructions are followed from top to bottom.

If CI is OK, this is done.


Add configuration for Devola Intelligent Heater

Adding support for Devola Intelligent Heater - https://www.devola.co.uk/products/devola-designer-2kw-smart-glass-panel-heater-with-timer-black-dvpw2000b
This also supports the 1kw model and presumably the 1.5kw model as well

This config is identical to the Andersson GSH config, only with added bugs.