Implement image spec

Solves #10. Another PR with additional tests will follow.

Codecov Report

Merging #25 (8cf7e46) into main (1b42768) will decrease coverage by 0.84%.
The diff coverage is 16.56%.

@@            Coverage Diff             @@
##             main      #25      +/-   ##
==========================================
- Coverage   20.31%   19.46%   -0.85%
==========================================
  Files           7       11       +4
  Lines         699      863     +164
  Branches      371      463      +92
==========================================
+ Hits          142      168      +26
- Misses        274      354      +80
- Partials      283      341      +58

@saschagrunert PTAL
I included your suggestions and also formatted the comments to have a uniform width of 80 characters.
Does intellisense work with the generated getters for  you? I tried enabling the required feature in the settings of rust-analyzer and even created a second project and added this crate as a dependency but it does not get picked up. Cargo expand confirms that everything has been generated as expected and I can use them and it compiles fine, but it does not show up in intellisense.

Thanks! Rustfmt allows 100chars but having 80 does not hurt. I’m using vim with YouCompleteMe and the rust analyzer. This works pretty well from my experience, unfortunately I don’t know how it behaves with intellisense.


["Wow-факти"/Interesting facts block] Verify that when the user clicks on the 'Трохи ще' button, a modal window opens.

Priority:
High
Description:
This test case verifies that the user can view and read more about interesting facts.
Preconditions

The 'StreetCode page' is opened.
At least 4 facts have been added, one with a text containing a minimum of 270 characters, including spaces.

Steps to reproduce

Go to the "Wow-факти" block.
Click on the 'Трохи ще' button on one of the facts in the list.

Expected result
A modal window is opened according to the mockup, and all text is available for reading.
User story:
#26
Test
#67

I'm not sure if it's appropriate to add a screenshot from the site, it's not a defect, it's a test case.


im.ene.toro.exoplayer2.ExoVideoView is not visible

What I should add to dependencies to see im.ene.toro.exoplayer2.ExoVideoView class like in toro-sample?

It is impossible to first use your library, because wiki and read.me are inconsistent. Also it's sample is inconsistent with wiki, and read.me both.

@Nominalista Yes that's right. As you can see I push a lot last weekend, which means that Ver 2 is on the go. I have no time to update Wiki right now so please pardon. Right now looking at toro-sapmle and follow it is the best way. Please just ask question here I will response as much as possible. Ver 2 will be out soon.

@Nominalista About the first question, I didn't push the whole thing to neither maven nor jcenter yet. Right now forking the repo and use it as sub module or embed code is the only way. I don't want to mess up current SNAPSHOT since I made a big change on package name as well as project structure.

Thank you for response, I will use it as sub module.

@Nominalista I have pushed my latest code base to sonatype SNAPSHOT repo, you can include it as follow:
// Only Toro core implementation
compile "im.ene.toro2:toro:2.1.0-SNAPSHOT"
// Toro extension that contains Components based on ExoPlayer v2
compile "im.ene.toro2:extension-exoplayer2:2.1.0-SNAPSHOT"
// Extended of Toro, based on Toro Extension for ExoPlayer v2, with some new fun features
compile "im.ene.toro2:toro-extended:2.1.0-SNAPSHOT"

// The following dependencies are obsolete, but I put it together for your interest
// Simple extension for Legacy MediaPlayer API
compile "im.ene.toro2:extension-mediaplayer:2.1.0-SNAPSHOT"
// Fined tuned extension for ExoPlayer v1 (not too bad, but v2 is preferred and recommended)
compile "im.ene.toro2:extension-exoplayer:2.1.0-SNAPSHOT"

Note 1: version name is 2.1.0-SNAPSHOT but not 2.0.0.
Note 2: SNAPSHOT requires sonatype repository which you can find at my latest README.
Note 3: that I drop the 2.0.0 due to some internal changes which may make current users Unhappy, and I move directly to 2.1.0 instead. Some final implementation is done, I'm preparing the final release soon. Hope this helps you.

Excellent, can we expect updated wiki and read.me in near future?

@Nominalista Of course. I need to update it for next release.


4 sockets qwen execute question

Hello, my machine is shown in the above picture, and I want to run the benchmark test of Qwen-14B-Chat with the best performance. What   command should I execute? XFT has been installed in the form of pip.

Comment code between https://github.com/intel/xFasterTransformer/blob/main/benchmark/run_benchmark.sh#L147 and https://github.com/intel/xFasterTransformer/blob/main/benchmark/run_benchmark.sh#L214, and add following code
export OMP_NUM_THREADS=32
run_cmd="mpirun \
    -n 1 bash run.sh 0 0 ${OMP_NUM_THREADS} 0 : \
    -n 1 bash run.sh 1 1 ${OMP_NUM_THREADS} 1 : \
    -n 1 bash run.sh 2 2 ${OMP_NUM_THREADS} 2 : \
    -n 1 bash run.sh 3 3 ${OMP_NUM_THREADS} 3"

The CPU usage should be 50% since it isn't recommended to use hyperthreads. If you can observe that CPU usage is at maximum for CPUs 0-127, then 50% usage is as expected.
You want to run benchmark on 4 numa nodes so 4 instances will be launched by mpirun.

@Duyi-Wang hello，What should I do if I directly want to mpirun benchmark. py? I'm trying OMP_NUM_THREAD=64 mpirun --allow-run-as-root -n 1 numactl -N 0 -C 0-63  python3 benchmark.py: -n 1 numactl -N 1 -C 64-127 python3 benchmark.py
But it keeps reporting errors

use --all in numactl.
It's not recommended to run a rank across sockets like -C 0-63.
And -N and -C is conflicted. -N 1 means -C 32-63,160-191

@Duyi-Wang OK，Thanks.When I run OMP_NUM_THREAD=64 mpirun --allow-run-as-root -n 1 numactl --all python3 benchmark.py : -n 1 numactl --all python3 benchmark.py` a new error occurred, what is the reason for it?

Did you install mpirun from oneCCL? Please use oneCCL and source its environment.
And use numactl --all -C 0-31 to bind cores.
The value of OMP_NUM_THREADS should be set to 32, ensuring it does not exceed the number of physical cores per node.
You have 4 NUMA nodes, so 4 ranks are required. We introduce multi-rank to avoid running across NUMAs, which can significantly impact performance.

You want to run benchmark on 4 numa nodes so 4 instances will be launched by mpirun.

@Duyi-Wang 我很疑惑啊，因为我通过命令运行自然得到的OMP_NUM_THREADS=64，为什么你让我设置32啊？

Because the script did not account for the scenario of 4 sockets and 4 NUMA nodes.

Because the script did not account for the scenario of 4 sockets and 4 NUMA nodes.

@Duyi-Wang ok,thanks.my command is right to get best performance?
OMP_NUM_THREAD=32 mpirun -n 1 numactl --all -C 0-31 python3 benchmark.py : -n 1 numactl --all -C 32-63 python3 benchmark.py: -n 1 numactl --all -C 64-95 python3 benchmark.py: -n 1 numactl --all -C 96-127 python3 benchmark.py
However, executing the above command got stuck.

Please ensure the path of benchmark.py is /root/enmotech/xFasterTransformer-main/benchmark/benchmark.py in your environment.
It's right, and it's recommended to preload libiomp5.so which you can get from https://github.com/oneapi-src/oneDNN/releases/download/v0.21/mklml_lnx_2<IP_ADDRESS>90502.tgz

@Duyi-Wang hello, I have tried in both cases, I believe that the underlying code being executed should be consistent, but why is the difference in results so significant? Buy the way，The parameters are all consistent.

No idea. Could you show me the modifications you made to the scripts?


Problem with "emtpy" APIResponseException

From time to time i am getting some weird APIResponseExceptions when sending Mails.
In the exception no APIMessage and no APIDescription is set and the APICode is 0.
I can't replicate this errors, but i am seeing them from time to time in my logs.
It gets even weirder: sometimes some of these mails get send and i can find them in the Sparkpost Backend, sometimes they dont show up.
Anyone with the same problems and some hints?

I'm having the same issue, were you able to track this down?

@jsonnull no, this is happing in production and i cant replicate the error localy yet..do you?

It's just the opposite, I'm getting the empty exception in my dev environment but the same code on our production server is working fine.

I have installed dependencies locally more recently than on the server, I'm wondering if there's some upstream conflict. That's my first guess when code works one day and doesn't work the next.

@jsonnull, @heikokrebs are you still experiencing this intermittently?

I'm going to go ahead and close this - feel free to re-open if you continue to see issues.


-D option not working ?

Just installed, and Im excited to use this project ! Exactly what I need for something. But when I use the -D option I get an error:

Not a blocker for me I can pull from the default download directory but figured you might want to know.
Thanks for this project, starred it !

Interesting catch, thanks for reporting!
I must admit I dont use -D myself, and have tested this approach only for the integer flags. Maybe I'm overlooking something though, as it seems the /tmp arg is not consumed in this line, and instead gets passed to URLS a bit further down.
https://github.com/ddelange/yt/blob/0.2.2/yt/yt#L53
Will need to investigate further!

Ok cool Ill take look

Maybe it gets consumed (destination is set as expected), but then the OPTIND shift allows it to get consumed a second time in filling URLS, or something similar.
Another possibility is removing the double quotes from the line I linked above, just a hunch...

I tried w/ and w/o quotes

Released 0.3.0, once the tap PR auto-merges, please run brew upgrade yt :)

Maybe not fully fixed ?
Im semi-intermediate with bash so Im happy to take a look tomorrow, no rush not a blocker.
Heres the project im working on:
https://github.com/the-real-adammork/yt-extension
https://github.com/the-real-adammork/yt-server
started tonight, made it so I can speed up my music digging workflow. Idea is to have the chrome extension send requests to the express server on localhost to download whatever audio is in the current tab (soundcloud/youtube).
Right now it activates when you click the extension icon, I think I'll add keyboard shortcut as well.
Im glad I came across your project seems like it improves on youtube-dl in all the ways Id like and the take from clipboard by default is really smart, that was actually part of my old script.
Thanks 🦾

Thanks for checking!! Fixed in 0.3.1.
Your project sounds really cool! The extension would definitely save clicks and make the flow more natural!
If you encounter any bugs in the future, please let me know. And also feel free to tag me on your PRs or so if you need a second pair of eyes :)


Not making path using NPathOverlay

I am using NPathOverlay to make a route path between source and destination but it is making a polyline between lines to making route. Can you tell me how to make path route between points

route data is not provided in naver mobile dynamic map sdk.
please check route api service(https://api.ncloud-docs.com/docs/en/ai-naver-mapsdirections).

if you have more question, please re-open this issue and comment here.
thank you.

Thanks for helping @note11g


scroll-while-dragging

fix for Scroll while dragging #121

Hey there, this seems like a much leaner solution: https://github.com/luckylooke/dragular/commit/0871a581613555ad94da54b8daed631a254903a7, he uses the wheel event to propagate the scroll amount using e.deltaY. While we're on it, consider updating el.scrollLeft as well, using e.deltaX. Mind taking a look?
+@luckylooke

sure, I'll try to update it for scrollLeft as well. I'm trying not to use any other library like jquery, angular, etc

Oh no I just meant looking at that changeset to see how he handles it with wheel events, as it seems much cleaner (and lots less of code, too!)

Oh got it. Yeah sure I will check that

Cool! :)

Hey @zeeamber do you have a chance to look at fixing the code?

Hi @bevacqua, I tried to implement  using wheel event but sadly no luck. can you give me some time to optimize it other way around if find some. I think i should be able to reduce some LOC

Hi @bevacqua, both horizontal and vertical scrolls are working now. I tested it for chrome, firefox, & safari.

What about mobile?

@bevacqua, I just tested it with mobile. its working fine.

The demo isn't working for me on Chrome
We should try and implement this using wheel events (like here), seems much less involved too

@bevacqua  I'm still on to implement it using wheel events. I know using wheel event is the best solution here. hope I succeed soon

Oh cool

I have made different solution since touch-devices don't have wheels, see [demo] (http://luckylooke.github.io/dragular/#/docs/exampleScrollingDrag) and changes in code. I have still left wheel listener for devices with wheel:
// wheel event listener
function scrollContainer(e) {
      if (_targetContainer) {
        var before = _targetContainer.scrollTop;
        _targetContainer.scrollTop += e.deltaY;
        // block scroll of the document when container can be scrolled
        if (before !== _targetContainer.scrollTop) {
          e.stopPropagation();
          e.preventDefault();
        }
      }
    }

Do these solutions work when the user is dragging something (but not moving it, just sitting) over the edge of a container? Kind of like how you can scroll a Finder window while dragging something just by hovering over the bottom of the window?

@bevacqua yes, there is interval used, but its not part of library, you need just simple event listeners to make it work.

I just grabbed the code and run it, i didnt change anything on the dragula.js or in the configurations, but i am getting a weird behaviour, i just clicked and the page scrolled up:

For me, this feature of scroll the page needs to work just when the item in on the borders, like if some piece of the div touched any side of the page.
o/

Hi @luanfonceca. thanks for raising the bug. did some changes in scroll criteria. should work properly now.

Agree that functionality should trigger just near edges, not during scroll

Also, no browser sniffing. Figure out another condition

Hi @zeeamber, i just tested that behaviour and it is ok, but i found another one:
When i drag the item to outside of his container, the scroll doesn't work anymore:

That's what I would expect

Ok, then everything is OK for me
o/
Em sáb, 22 de ago de 2015 7:55 AM, Nicolas Bevacqua <
<EMAIL_ADDRESS>escreveu:

That's what I would expect
—
Reply to this email directly or view it on GitHub
https://github.com/bevacqua/dragula/pull/129#issuecomment-133676952.

@zeeamber could you get rid of browser sniffing and merge with master? I can take it from there

yeah sure I will give it a try.
@bevacqua

merged with master @bevacqua

Doesn't look that way

Hi @zeeamber, sorry for ask this, but did you how to update your code with the origin one, I mean, pull the the code from this repo, merge it with your own fork and fixing the conflicts, all that?!
o/

Hey @luanfonceca , yeah little confusion working on git repo, I'm leafing through the documentation though :)

I pulled the changes from orig master, updated it and merged with my master branch.
can you check it now if its okay? @luanfonceca @bevacqua

Hi folks, i think i dont have so much knowledge about what code was in the master, but it seems everything is fine for me.

Aren't the conflicts resolved? Is there something else stopping this PR from being merged?

Conflicts are resolved. I think @bevacqua can take it from here

Looking forward this feature.

Can't merge this yet

All checks have passed here. can u merge it now ? @bevacqua @btecu

Yeah, please squash the commits. Thanks

I think the code should also go in dragula.js not just dist/dragula.js.

Agreed

@bevacqua about what @luanfonceca told when dragging outside of container. I don't agree: I expect to continue scrolling. I say more: even when we drag some item outside of window (window is resized and we move cursor, dragging item, until we have the cursor outside of the window) it should continue scrolling... and not only scroll when we have the cursor inside the container.
And also, we should take in consideration the drag distance to set the scroll speed: take in consideration the starting dragging point and the current point we are dragging and set the scroll speed according to that...
Example > do like trello is doing: https://trello.com/b/Ragttcss/drakula (make a trello board like this one and see how trello does it) ;)
You see that they don't limit the scroll by just the container where we drag

+1 drag speed

i took the code and run it, it only works in chrome for me. It doesn't work on Mozilla and IE9 browser. I din't check IE9+ browsers though.

Closing in favor of modules such as dom-autoscroller

@zeeamber I have a PR that address scrolling while dragging. Let me know if this works for you.


Web Storage is not available

When testing the origin for Web Storage across our test apps, it looked like it isn’t currently available in the Android test app.
My understanding is that this is opt-in for Android web views but I couldn’t find any occurrence of setDatabaseEnabled with a quick search. Please correct me if I’m wrong, obviously. If it is indeed enabled, then there’s another bug preventing it from being used somewhere.
Of course the next issue is managing the origin on a publication basis. A lot of existing Reading Apps on Android don’t do this, so you can access data from other publications and remove them.
See Dan’s doc for further details if needed.

Database Storage is deprecated, so this is issue is superseded by #266 about Local Storage.


Reapp doesn't play nice with nodist :(

I dont think this is a big deal. I simply got rid of nodist and went back to bare bones node without node version management.
Until the dust settles on what node version management is the one for windows - it really doesnt matter.

I am using nvm...
Will this work for you?


Hide prefilled props on SmartForm when fields restraint is specified

When a fields restraint and prefilled props are specified on SmartForm, it will add the fields to the values to the form but hide them if they are not inside the fields array, that way, prefilled props will be submitted and you won't need to specify hidden on the schema.

Can you clarify what the current behavior is, and what the desired behavior should be?

How it is now, you have to specify hidden:true on the schema in order for it to not show up in the SmartForm fields and it still submit.
But what I'm thinking is, when defining a SmartForm you could do this <SmartForm fields={["name"]} prefilledProps={{gender: "male"}}/> and have the gender submit in the form while being hidden without it having gender be hidden in the schema of the field.  That way you could show/hide it on a form by form basis.

Yup, however then, the name prefilledProps might not be completely intuitive.

prefilledValues would be better I think? I'm not even sure why I used the word props in the first place. I think this would require a little tweaking of the way prefilledProps/values are used currently, but it should not be that hard to add.
@eric-burel can you see any downsides to doing this?

Seems a reasonable behaviour to me, I'd expect the name props to be always respected in prority. I agree with the prefilledValues name too.
What I don't get yet is why we get this behaviour currently:

PrefilledProps are merged into the initial document. That's the only place they are used directly. https://github.com/VulcanJS/Vulcan/blob/devel/packages/vulcan-forms/lib/components/Form.jsx#L94
Then, the fields are computed based on the document and user rights. https://github.com/VulcanJS/Vulcan/blob/devel/packages/vulcan-forms/lib/modules/schema_utils.js

I can't find where the fields props is used though to limit the inputs to display.
Expected behaviour is simply to decorrelate fields and prefilledProps props: it should display only the provided fields (respecting user's authorizations), and it should always add the prefilledProps to the document, even when they don't show up in the form.

I can write a pr for this :).  I noticed that there's no way to specify disabled on an input.  I was also thinking that the property hidden should maybe be moved to input properties?

Also
inputProperties: {
      addonBefore: <Icon>attach_money</Icon>,
      defaultValue: () => 2424,
      disabled: true
    }

doesn't seem to submit
and
  startDate: {
    type: Date,
    label: "Start Date",
    optional: true,
    canCreate: ['members'],
    canRead: ['guests'],
    autoValue: function() {
      return moment(d).add(1, "M").toDate();
    }
  },

doesn't seem to add the value to submit, do you plan on supporting autoValue, I know you want to get away from simple schema eventually but it's so dang helpful :).

The autoValue runs when I take off the canCreate and canRead from the schema, but when I add it to the mutation fragment that is returned from the SmartForm, it says
Error: GraphQL error: Cannot query field "startDate" on type "Event".
GraphQL error: Cannot query field "endDate" on type "Event".
    at new ApolloError (modules.js?hash=c5fad458235c229ee5485493ea45c7b1c0c0e5f7:85720)
    at modules.js?hash=c5fad458235c229ee5485493ea45c7b1c0c0e5f7:86634
    at meteor.js?hash=33066830ab46d87e2b249d2780805545e40ce9ba:1223

however if I take it off the fragment, everything runs just fine!

Those seem like separate issue but

defaultValue is a property of the field, not the inputProperties property.
instead of autoValue we have onCreate.

Sorry for deviating a little bit!  (Default value as input properties)[https://github.com/VulcanJS/vulcan-docs/blob/master/source/schema-properties.md#inputproperties].  What I was getting at here is that hidden should probably be apart of inputProperties as it describes the way the input is rendered.  Perhaps, when I'm working on this I can shuffle some things to inputProperties and write deprecation notices to solidify the SmartForm api more.

What I was getting at here is that hidden should probably be apart of inputProperties as it describes the way the input is rendered.

I think that could be misleading as it might imply that it'll be rendered as a hidden input, whereas it's actually not rendered at all.
Also inputProperties (if I'm not mistaken) is specifically for properties that you want to pass to the input, and of course if the input is not rendered it doesn't make sense to pass it properties.
By the way, Autoform had a form sub-property which would take properties targeting the field when displayed in a form but I found that confusing as I could never remember which properties were part of the form sub-property, and which properties were part of the general field schema. Which is why I think it's easier if everything is at the top level.
But maybe the name hidden itself is confusing then?


Hidden menu icons show in viewer.html as browser window is made narrower & then hide again as made even narrower

Two of 3 hidden toolbar Buttons display as the browser window narrows, and then they disappear again as the browser window keeps narrowing.
Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version:  Happens in all browsers I tried:
Chrome Version 91.0.4472.77 (Official Build) (64-bit)
FireFox Version 88.0.1 (64-bit)
Edge Version 91.0.864.37 (Official build) (64-bit)

Operating system and its version:  Win 10 21H1 OS Build 19043.1023

PDF.js version:  2.8.335 Legacy, using viewer.html

Is a browser extension: No

Steps to reproduce the problem:

Open a pdf in viewer.html where presentationMode, openFile and the bookmark toolbar buttons are hidden via "display: none" in viewer.css :
.toolbarButton.presentationMode,
.toolbarButton.openFile,
.toolbarButton.bookmark {
display: none;
}

When the browser is "wider" (normal viewing kind of mode on a monitor in landscape mode), the buttons cannot be seen.

Grab either side of the browser window and drag it to make the browser narrower.   As the browser narrows, two of the 3 hidden toolbar buttons display.

As the browser narrows further, the two "appearing" buttons  disappear and do not display again.

What is the expected behavior? (add screenshot)
The expected behavior is that buttons styled not to display won't display.
What went wrong? (add screenshot)
Three screenshots are provided below:
-- wider (normal landscape viewing width),
-- narrowing (where buttons appear)
-- even narrower (where buttons disappear again).
I set a fixed width black background behind the browser window that is narrowing, and each screenshot has this same black background width, so you can get a feel for how much narrower the window has to become for the hidden toolbar buttons to show.
I underlined in red the buttons on the toolbar on the right hand side that start properly displayed/hidden, hen gain two hidden buttons, and then end properly displayed/hidden again.
Screenshot 1: Wider mode (the hidden buttons do not show).

Screenshot 2:  Narrowing: 2 of 3 hidden buttons show as browser window narrows.

Screenshot 3. Narrows more: the 2 "appearing" buttons hide again as the browser window narrows further.

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):
https://kkcomcon.com/libraries/pdfjs/web/viewerkk.html?file=../../../../doc/KPQA.pdf#page=1

 .toolbarButton.presentationMode,
 .toolbarButton.openFile,
 .toolbarButton.bookmark {
   display: none;
 }

[...]
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):
https://kkcomcon.com/libraries/pdfjs/web/viewerkk.html?file=../../../../doc/KPQA.pdf#page=1

Using that link, I cannot re-produce the issue described in this issue.
However, in general, it's possible that the display: none; line could get overridden by something with a higher specificity and the one guaranteed way of fixing such problems (while generally not recommended) is to append !important to that line; see also https://developer.mozilla.org/en-US/docs/Web/CSS/Specificity
Closing as answered, since this looks like a more general CSS question rather than a PDF.js bug.


Strip channel when calling semver.satisfies to allow custom channels

The semver packages doesn't seem to have any problems dealing with custom channels when calling gt:
> semver.gt('v0.14.1-canary', 'v0.14.0-canary')
true

But satisfies doesn't seem to support them:
> semver.satisfies('v0.14.1-canary', '>=v0.14.0-canary')
false

As a result, nuts always returns HTTP 204 for URLs like:
curl "http://nuts-canary.itch.ovh/update/darwin_x64/0.14.0-canary"

This patch strips the channel before calling semver.satisfies, which fixes the issue for us.

Sorry for the delay.
I'm not sure it's a "good" idea. I understand the problem (I'm having it with semver and gitbook-cli/gitbook).
But here are the expectations: for our application:
// We want to ship the next beta (or normal)  to user having the beta
satisfies('>=5.0.0-beta.0', '5.0.0-beta.1') == true
satisfies('>=5.0.0-beta.0', '5.0.1') == true

// We don't want to ship the beta to user using the normal version
satisfies('>=4.9.0', '5.0.0-beta.1') == false

I think that your PR will break this behaviour, since:
satisfies('>=4.9.0', '5.0.0-beta.1') -> satisfies('>=4.9.0', '5.0.0') -> true

I agree that it might not be a good idea for everyone.
Personally, I've set up 'unstable' release in a completely different github repo, with a different nuts instance, everything different — I wanted it to be invisible to most users & have the deploy process not touch the main repo at all.
Both approaches have drawbacks, tbh. Do you think there's a way to support both with an option without making the code too complex/costly to maintain? It is a very small patch but as a maintainer I'd completely understand if you'd rather not support it in your version.

+1 also running two servers and two repositories for QA / production. It feels clunky but it works for now

Some more clarity on the matter
https://github.com/npm/node-semver#prerelease-tags

Hi, I merged the project to a new repo to start maintain it, I would be glad if you can put your pull request here : https://github.com/loprima-l/nuts-2


using custom sonoff firmware without soldering

Hi, just came across your sonoff custom firmware just moments ago. Nice work. I have a question though, most custom firmware project i've seen so far for sonoff requires some minor soldering on the sonoff board but looking at your read me doc for this project there isn't any mention of soldering. does it mean with your custom firmware no soldering is required?

You just need a FTDI and some wires with male connectors to program the Sonoff. For simplicity I soldered a pin header, but it's not mandatory. You have to press the Sonoff's button before/while you're connecting it to the computer (with the FTDI, NOT to mains power).
Schematic: https://4.bp.blogspot.com/-o2aOFpB7rC4/Vs29BGlDrWI/AAAAAAAADtw/aqf_o8tulck/s1600/Sonoff_FTDI.jpg
Video about the Sonoff, with explanations: https://www.youtube.com/watch?v=_389pQPLyr8

Ok, thanks


feat(theme): use<EMAIL_ADDRESS>

Related to #2

removal of custom setup for dark partial highlight since it's integrated directly into the theme

what mean the PWA100on the image docs/web_modules/sfeir-school-theme/docs/docs/images/first-slide-conf.png   ?

@ThibDujardin docs of the theme (good practice)


Fix non-actionable code bugs: Logout

These issues are from Telemetry records, and most of them are caused by obvious code bugs. The detail error messages and the corresponding command and CLI version info can be viewed in the CLI error dashboard.
As the info of these issues are limited and may be more difficult to identify and solve, we opened a Teams channel for command groups owners to sync up the common error types and fixing methods. Feel free to share your experience and ask for help (if any) there.
A brief summary of the code bug errors of AKS here:

FailureType
EventCount

TypeError
6

AttributeError
2

ImportError
1

ModuleNotFoundError
1

Non-actionable error messages are helpless to end users when Azure CLI error out. One of Fe OKR is to improve Azure CLI command pass rate for better user experience. We've identified non-actionable errors of top used command groups. Pls help to fix non-actionable errors. More detail in email. Feel free to ping us at teams,  or ping @houk-ms @yungezz  if any issue.

@jiasli  for awareness.


Error message

Not sure if anyone else is seeing this error message upon log in. Just sent a separate note to IAB for investigation.

Thanks. I will look into it.
Thanks
Shailley Singh | IAB Technology Lab | T:<PHONE_NUMBER>tel:212.378.4290 | M:<PHONE_NUMBER> | Skype: talk2shails
On Apr 28, 2016, at 9:19 AM, ChristianCarrilloABOUT <notifications<EMAIL_ADDRESS>wrote:
Not sure if anyone else is seeing this error message upon log in. Just sent a separate note to IAB for investigation.
[unknown]https://cloud.githubusercontent.com/assets/18723673/14892941/4717cc22-0d3b-11e6-9f69-3bbd1a36c38e.png

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHubhttps://github.com/InteractiveAdvertisingBureau/HTML5-AdValidator/issues/12


supply example for updating customData via REST for vmss

[Enter feedback here]
The docs state that customData for vmss can be updated via REST. I cannot find a good example. Can you supply an example or link?
"you can update VMSS custom data via REST API"

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

ID: 282b39fe-e8fe-f698-36c2-321f088f6554
Version Independent ID: 2a7c130c-291b-40ff-cae5-43255bcb3bc3
Content: Custom data and Azure Virtual Machines - Azure Virtual Machines
Content Source: articles/virtual-machines/custom-data.md
Service: virtual-machines
GitHub Login: @mimckitt
Microsoft Alias: mimckitt

@JeffGiroux Thank you for bringing this to our attention. I will have an engineer from the compute team review this and respond with the available documentation.

Thanks, ideally an example for update as well as create would be great as I cannot find either.
update page - https://docs.microsoft.com/en-us/rest/api/compute/virtualmachinescalesets/update
Update page does not list any examples.
create-or-update page - https://docs.microsoft.com/en-us/rest/api/compute/virtualmachinescalesets/createorupdate
Create page has lots of examples, but do not contain examples with customData in use. Suggest adding an example for customData to this list.
https://docs.microsoft.com/en-us/rest/api/compute/virtualmachinescalesets/createorupdate#examples

@JeffGiroux
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@mimckitt
Can you please check and add your comments on this doc update request as applicable.

@JeffGiroux good catch on this! I added a link to the REST API information. Once the PR merges the changes will go live after a few hours.
The REST information is here: https://docs.microsoft.com/en-us/rest/api/compute/virtualmachinescalesets/update
You will find in the update sections that there is a field for custom data that provides information.

Will this specifically include an example with the entire POST body?

I just added a link to the REST API reference where we have information on custom data.
We don't generally put POST samples in the docs as there are a load of examples that could be added and all the info is kept in the reference docs.

OK, I see it now. That doesn't really help me though. The initial ask of the ticket was for an example. I can see the link and I can read the docs, but not having a customData example payload (at least one little example) makes it tough to understand how exactly to structure the payload.
Again, lots of example here - https://docs.microsoft.com/en-us/rest/api/compute/virtualmachinescalesets/createorupdate#examples
Would be nice to have one of those examples updated with "using customData". I understand the potential example-explosion by having examples for every single item. I feel though customData is a really good and important example to show.
Thx,

For instance, based on this example...how do I incorporate customData.
https://docs.microsoft.com/en-us/rest/api/compute/virtualmachinescalesets/createorupdate#create-a-scale-set-with-a-marketplace-image-plan.
customData appears to fall under this...
https://docs.microsoft.com/en-us/rest/api/compute/virtualmachinescalesets/createorupdate#virtualmachinescalesetosprofile
So...I would put the customData reference here?
      "osProfile": {
        "computerNamePrefix": "{vmss-name}",
        "adminUsername": "{your-username}",
        "adminPassword": "{your-password}",
        "customData": base64encode("onboard.sh")
      },

Is that it?

#reassign: @danielsollondon
@danielsollondon can you help on this one? I know you wrote the majority of this doc.

We are currently cleaning up old issues and closing out items that are greater than 90 days old. If an issue is still present, please open a new feedback item on the document so we can prioritize correctly.
#please-close

weird that this hasn't been mentioned: instead of going through the hassle of registering your client on the rest api, you can use az rest
az rest -m patch -b '{
  "properties": {
    "virtualMachineProfile": {
      "osProfile": {
        "customData": "IyEvYmuL3NoCgpl.....Y2hvICJoZWxsbPi9bXAvd=="
      }
    }
  }
}' -u 'https://management.azure.com/subscriptions/{subscriptionId}/resourceGroups/{group-name}/providers/Microsoft.Compute/virtualMachineScaleSets/{vmss-name}?api-version=2022-08-01'

You just need to:

encode the whole customData file contents into Base64
replace {group-name} and {vmss-name} with the right values. az restwill replace {subscriptionId} automatically


REL-1434 v24.1.7: [Docs] Generate release notes for Cloud

Fixes REL-1434
edited notes

@kathancox, fyi I based most of my release notes on PR 19119.


Debug build is failing code analysis

build.cmd Debug

it's failing because file is declared but not used. I'm not sure what the loop is for

also string[] settingFile


x265: 1.5 -> 1.6

Removed unused x265-hg package
Fix missing optionals
Fix high bit-depth to allow for all 64bit platforms

Great, thank you!


Update LAB_01L01_Prepare_for_deployment_of_AVD_AADDS.md

Updated with Microsoft Entra
Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

Updated with Microsoft Entra


Rotation cleanup: Remove Assembly.rotatePins, make Block.rotatePins private

What is the change?
🚨 This change does not handle the rotation of all parameters that are tracked on pins and need to be rotated. That will come next after this cleanup because that will be a lot harder to manage and test and handle correctly 🚨
As described in #1533, assembly rotation needed some love.
Block.rotatePins didn't do as much as people maybe would like (e.g., rotate parameters) so it is now private. Block.rotate should be preferred and will do (hopefully) everything people expect it to do
Assembly.rotatePins suffered from the same problem, and now is removed. Assembly.rotate should be preferred.
The justCompute flag in Block._rotatePins caught my eye and I found it was used when determining the new location of a pin with max burnup post-rotation. This means that in order to figure out where one pin would go, we would compute new locations for every pin in the block. And then maybe not use that as we rotate the assembly in all the possible orientations. This felt weird so I the "pin location post-rotate" functionality from that method and made it a new utility in armi.utils.hexagon. Now, when we are determining the optimal location of a pin during rotation, we just look at the proposed location of that pin.
Block.rotate did a lot (and it should!) but the method was very long. So I broke it up into a few private methods that are a bit more contained.
Why is the change being made?
A large body of work is in progress to handle the parameter rotation. That work will be made easier after this cleanup.
Other things to (maybe) do

Remove justCompute switch in the now private Block._rotatePins? I can't find any usage of it now.
Move the Block.setOrientation calls from Block._rotatePins to Block.rotate

Checklist

[x] This PR has only one purpose or idea.
[x] Tests have been added/updated to verify any new/changed code.

[x] The code style follows good practices.
[x] The commit message(s) follow good practices.

[ ] The release notes have been updated if necessary.
[ ] The documentation is still up-to-date in the doc folder.
[x] The dependencies are still up-to-date in pyproject.toml.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Maybe this exists already, but it would be good to know if something has already been rotated relative to its as-designed (e.g., by construction) and by how much. Maybe this is a good use of flags or some type of flag on the composite?

Maybe this exists already, but it would be good to know if something has already been rotated relative to its as-designed layout (e.g., by construction) and by how much. Maybe this is a good use of flags or some type of flag on the composite?

We have Block.p.orientation and Assembly.p.orientation so assuming a value of zero indicates the "as designed" orientation, that may suffice. If I'm missing something, can you make a ticket so I can track that as part of the upcoming assembly rotation work?

@john-science I'm not sure where to add this to the release notes. We have a 0.4.1 section that's TBD, but this is a breaking change and feels like it deserves a larger release number bump. That said, there are other removals in the current 0.4.1 section

@john-science or @albeanth this is ready to go and has some interesting connections to something @mgjarrett put up - #1859

I have one of the only internal downstream uses of the public rotatePins method with justCompute=True (at least I know of). The justCompute=True is really just about doing hex lattice logic / math and doesn't need to be bound to a specific instance of a block. I would be fine with removing the justCompute argument entirely. I can refactor the downstream application to use the new armi.utils.hexagon.getIndexOfRotatedCell

@drewj-tp Does this PR close the ticket #1533 ?

Yes, I just didn't add the special language for github to link the two. They should be linked now

This PR breaks my FFTF model, FYI. I was using assembly.rotatePins() (whether that was bugged or not, I don't know).

This PR breaks my FFTF model, FYI. I was using assembly.rotatePins() (whether that was bugged or not, I don't know).

@drewj-tp - FYI


Bump to thor's master

What was the end-user problem that led to this PR?
The problem is that we are currently cherry-picking some thor changes that are currently living on its master branch, but have not been released yet. The changes are about using require_relativeand not relying on the LOAD_PATH. I cherry-picked them, because they were needed to fix some issues with bundler installed as a default gem.
Anyways, it's always better to depend on released versions of our dependencies.
What was your diagnosis of the problem?
My diagnosis was...
What is your fix for the problem, implemented in this PR?
My fix is to do the same thing that I did for Molinilloin https://github.com/bundler/bundler/pull/7210, but for thor. I don't intend to merge it right now, but I want to make sure that we are ready to support a new version of thor if they released right now.
So, I can block this PR until a release is out.

Hi again!
I would like to reconsider this. Unfortunately, we don't control the release schedule of thor, and vendoring our own copy gives us liberty of which patches to include or not. I think the current master of thor is more stable than the latest released version because it includes a bunch of fixes that we can actually use. I happen to need https://github.com/erikhuda/thor/commit/8c57f1dee1d6cb9d0071c793eae05278ece4d8c3 to get https://github.com/rubygems/rubygems/pull/2981 green, for example.
So, I propose to include thor's master in the next version of bundler.

I fixed a few more issues in thor regarding Windows compatibility and I included them here, so things should be good now.
I'm merging this to unblock the next release!
@bundlerbot merge


Package request: evince on macos

Project description
homebrew provides an evince installation method on macos: evince formulae. Is this possible for nix?

homepage URL: https://formulae.brew.sh/formula/evince
source URL: https://github.com/Homebrew/homebrew-core/blob/1d647932d0c46f64891d6494a6191791d8e51326/Formula/evince.rb
license: gpl2+
platforms: darwin

@bobby285271 @tormeh
Are you interested in this?

Not sure if I'm the right person to tag. Don't think I've ever been involved in anything related

Seems to build fine at least, just needs to have meta.platforms expanded to unix.
https://github.com/NixOS/nixpkgs/blob/b72aa95f7f096382bff3aea5f8fde645bca07422/pkgs/desktops/gnome/core/evince/default.nix#L140

thanks very much :)


DL: Replace use_gpu flag with gpus_per_host integer

JIRA: MADLIB-1308
Previously, gpus_per_host were hard coded to 4. This commit removes this
hard coding and takes in this value from the user.
We also tried to use the tensorflow function list_local_devices to get
the count of gpus per host. This did give us the count but would hang
forever on some segments. So we decided to not use this function.
We now cache the CUDA_VISIBLE_DEVICES env variable (which is set to
-1 for master) and then reset it at the end of fit function.
Also fixed a bug  in get device name function for postgres

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/863/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/864/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/869/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/870/

Closing this PR for now, will re open it later


Enhance the UI of contact us form

Better a better UI to the web page, making it more interactive and adds to better user experience

for now we don't want any improvements in contact us form, and contact pages also.
So i am closing this issue,
Please Understand.


Regent: Provide interface for getting raw pointers to fields

Getting raw pointers to fields in Regent is error prone. Its easy to make mistakes that cause stack corruption and memory leaks while not having any tools that can help you find problems. Due to this, Regent should provide users with an easy way to get raw pointers to fields instead of having users write code to do so themselves.

How about the following interface (naming subject to change)?
var {base, strides} = __raw_pointer(my_region.field, rect)

Other options include:

Split out the base/stride into separate calls so you don't have to deal with a struct return value.
Return the address of a specific point (e.g., I could do that for &r[i].f though that syntax would be a bit inconsistent with other behaviors in the language right now).


Image Extraction Scalability Improvements

Issue #49 , if available:
Description of changes:

The name lengths are exceeding 64 chars for iam roles, due to poorly defined manifests, the name:
addf-ros-image-demo-ford-analysis-ford-pipeline-dag-ap-southeast-1
will now be:
addf-ros-image-demo-dags-ford-dag-ap-southeast-1
by changing the group name to "dags" and the module name to "ford"

Refactor Image Extraction to use EBS Volumes for task storage and EC2 capacity providers for ECS, as AWS Batch does not support increasing ephemeralStorage for fargate tasks, and with large files the tasks run out of disk space.

Image upload task uses ThreadPools for uploading files in parallel, increasing uploads by 10x or more.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The validation is failing due to not having a check if the launch_template_name exists
Other than that, LGTM

The validation is failing due to not having a check if the launch_template_name exists
Other than that, LGTM

Hey @dgraeber  - all good from my side as well. I tested the batch module and pipeline with and without the ebs volumes and think this ready to merge.

Hey @kevinsoucy Pls look into the above comments, other than that it looks good to me

Thanks @dgraeber , @srinivasreddych , could one of you please merge this?


4.1.1

4.1.1

Closing for now. Please contribute code changes to develop branch. The maintainers will manage reviewing and releasing.


canvas_test.py: TypeError

micah@micah-VirtualBox:~/PyAgg-master$ python canvas_tester.py
Traceback (most recent call last):
  File "canvas_tester.py", line 195, in <module>
    test_text()
  File "canvas_tester.py", line 36, in test_text
    canvas.draw_axis("x", 0, 140, 0, 25, ticklabeloptions={"textsize":30, "rotate":40, "anchor":"sw","fillcolor":"white"})
  File "/home/micah/PyAgg-master/pyagg/canvas.py", line 1178, in draw_axis
    for x in xpos:
TypeError: 'int' object is not iterable

This happens on Linux and Windows using Python 2.7.

This looks like an issue of not having updated the test file following changes to the API. The test files as of right now are not very structured and not very frequently run. The entire library in general is still quite fluid as I add more features and fix bugs, although I am definitively getting closer to a version that I might want to solidify and add more proper documentation and testing to. I do appreciate the extra pair of eyes, and I don't have much experience with unittests, so any additions of those would be especially appreciated.
But for now I updated the test file and added docstring to the draw_axis() method.
https://github.com/karimbahgat/PyAgg/commit/52cf86a97873e0e55a60f97fd28ace0d6f4d03d4


Update GetDeviceList.js to have hyphen in API URL

References user story #82
API endpoint name was changed from "device_list" to "device-list"

was erroneously committed strati to feature/s2, but it's there now


Update basic-stateful-set.md

fix the wrong translation

I have signed a CLA

I have signed a CLA


Using own Feature - constant for "delete with returning"

#1527 introduced DELETE with RETURNING and used Feature.insertReturningExpressionList
within the validation net.sf.jsqlparser.util.validation.validator.DeleteValidator - the Feature.insertReturningExpressionList is part of the validation group FeaturesAllowed.INSERT and therefore only allowed for INSERT statements.
The bugfix fixes the problem with a new feature constant Feature.deleteReturningExpressionList which is part of the validation group FeaturesAllowed.DELETE.

Thank you for helping me out! I honestly have no idea how this validation framework works and tried brute force.

Are the correlations clearer for you now, when you look at the changes?

I've added a "Further development" section to the wiki of my fork - could you please merge it to the original wiki of JSQLParser @wumpz ?
https://github.com/gitmotte/JSqlParser/wiki/Examples-of-SQL-Validation/_compare/f2b297d92baa0729a2c49f8b6e54e670b222b38a

merged it :) manually


Firestore DocumentSnapshot needs a getReference method.

My personal use-case is I want to display all the contents of a Collection, and have the user select one from the list and save the reference to another object. Presently, CollectionReference.snapshots returns a QuerySnapshot that returns a list of DocumentSnapshots without any way to identify one from the other.
This feature is necessary to make Firestore a drop-in replacement for RTDB.
getReference Android / iOS docs.

Closed in favor of #12471


vaadin-spring-mvp: Could you provide a test with mocked views ?

Hello,
could you please provide a test to show how to use vaadin-spring-mvp in a testing scenario ?
Thanks in advance for your help

Answered by @nfrankel no response from @thirionjl
Closed.


High memory consumption at F.affine_grid()

Hi, thanks for you implementation. I've tried this model on my pc with GTX 1050 Ti. Unfortunately, this model consumed huge amount of memory at this block and soon my GPU memory ran out:
            td_feature = part_enc_res.feature
            if td_feature is not None:
                td_feature = td_feature.repeat(n_obj_caps, 1, 1)

            td_templates = self.template_generator(feature=td_feature).templates
            td_pose = res.vote.view(-1, *res.vote.shape[2:])
            td_presence = res.vote_presence.view(-1, *res.vote_presence.shape[2:]) \
                          * part_enc_res.presence.repeat(n_obj_caps, 1)
            res.top_down_per_caps_rec = self.part_decoder(
                templates=td_templates,
                pose=td_pose,
                presence=td_presence)
            del td_feature, td_templates, td_pose, td_presence

This happened after this line in part_decoder.py:
            affine_grids = F.affine_grid(affine_matrices, target_size)

It seems like some variables aren't released properly. Even if I set the batch_size value to 4, GPU memory will still run out. Just a matter of time.
However, the offical tensorflow implementation at https://github.com/akosiorek/stacked_capsule_autoencoders and other pytorch implementation at https://github.com/phanideepgampa/stacked-capsule-networks can run well (while that pytorch implementation has some problems). The offcial one also gains faster loss decreasing speed. I wonder if there is any solution to this problem?

Hi, thanks for reporting the problem.
I've noticed memory problem as well and recently added manual garbage collection after each batch (commit 8211d641). Did you run your experiment before or after that commit?
I have spotted a bug in tiling tensors and an unnecessary tiling of features. I fixed them, hope it solves the problem.
If not, can you try training the model without that block since it is not included in loss calculation?
I have run the model on Colab for only 10 epochs with batch size of 64, but did not train it further.
I have seen the other PyTorch implementation but haven't trained it since it didn't seem good to me. I mostly kept my implementation same with official TensorFlow implementation except Set Transformer which I made it similar to the official implementation of Set Transformer paper. There might be a mistake causing the difference you mentioned.
Unfortunately, I currently don't have time and resources for investigating these problems in detail immediately, but I will probably be able check them before July.

Thanks for reply. I'm using the version with del and gc.collect(). But in my experiment, it has no effect on this problem. Also I've run the model with this block removed for 10 epochs, and it finished successfully. Maybe I will train it for more epochs in the future. If there is any other problem, I will open a new issue to let you know.

Hi again, I have made some changes.

Added a flag argument, reconstruct_alternatives,  to the SCAE class for optional reconstructions. Without computing them, I was able to train for batch size of 512 with 10 GB of GPU memory.
Changed the Set Transformer implementation to be same with original implementation.
Made RMSProp as default optimizer with same parameters. Also, added Adam and RAdam optimizer as options.

Hope these improve the convergence behavior.


Limit the use of templates only for a template entities

An entity that is not a template should not be able to use a template entity as an actant / action / prop value / prop type / meta prop / relation / valency
If a template is searched in Suggester, only the "create entity from template" action will be available (no link template action possible)

@jancimertel
If any entity has isTemplate: true, then it cannot use other entities with isTemplate: true in props (both value and type), references, tags, actions (including action.props value and type), actants (including action.props value and type) and territory.


Colorspace

When generating an image, the pipeline uses the default device colorspace:
https://github.com/apple/ml-stable-diffusion/blob/bea04420b5958935a975d3b7aeb071bbbaa9a097/swift/StableDiffusion/pipeline/CGImage%2BvImage.swift#L61
But when 3rd parties save and open the image, sRGB is used (I’m not sure if this is on purpose or if something is just defaulting to sRGB). It’s noticeably different and sRGB seems to be the correct colorspace.
Should line 61 be changed to use device-independent sRGB?

Yes. Please free to create a PR.


Add negative XDP test for https://github.com/vbpf/ebpf-verifier/issues/560

Add a negative test once PREVAIL fix for https://github.com/vbpf/ebpf-verifier/issues/560 is merged in ebpf-for-windows repo.

PR #3273 merged in the fix for this, so this should be doable now.


[New game]: Space Invaders

🎮 Game Request
Game logic and basic description
Space Invaders is a classic game in which we have to shoot down all the alien space shuttles preventing them from reaching our base.
Point down the features
Game points
Space Shuttle - Our space shuttle have a gun using which we will shoot down enemies.
Movement - We can move left and right and aim at the aliens.
Aliens - Auto generated aliens move closer to the base, and our task is to prevent them from reaching our base.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Please assign this to me

Hey @zanzo2003 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hello @zanzo2003, Time's up!⏰
Sorry for closing your issue!
But it's more than a week since we haven't received anything from your side 😢 .
Come up with new ideas, create a new issue and make sure you finish it within a week! 🔥
All the best! 🚀
Happy Hacking! 💗


Change Pg connection to be lazy

This changes the connection pool to postgres to open connections lazily.
Motivation
This might improve application startup a bit - both in speed and in reducing the likelihood of crashes on start up.
Solution
Use the connect_lazy function instead of connect, and remove now-unnecessary async/await calls.

I think we do the same in redis too, connect on start?
But anyhow, I think there's value in actually checking connection at start. Maybe we should change it to be more intentional, but if we don't do it we don't actually ever know it can connect to PG. This can mean that a new service can become healthy when it's in fact not healthy.
The health check won't catch it, as we don't actually check we can connect to PG in the health check ping (which is what should be used) because it would mean that PG goes down would fail health checks for the server which will in turn bring the whole service down when the issue is just PG.

Btw, we already have --wait-for to reduce the likelihood of crashes on startup because it verifies it can connect to the TCP socket (not actually negotiate with PG though).


Address issue found in JavaScript-Files/begin_round.js [Opened by bitHound]

bitHound discovered 1 ComplexFunction in JavaScript-Files/begin_round.js. For details go to bitHound.

I think we should shorten this function! Any ideas @Goredbirds?

Also I finished adding all the players!

I think you misunderstood something. Seven players should not be playing at
the same time.
On Wed, Mar 22, 2017 at 4:19 PM, Steven Copeland<EMAIL_ADDRESS>wrote:

bitHound discovered 1 ComplexFunction in JavaScript-Files/begin_round.js
https://github.com/StevenCopeland/Stores/blob/c178c7ef2571ec31c3806e378535c1e0a84606fa/JavaScript-Files/begin_round.js.
For details go to bitHound
https://www.bithound.io/github/StevenCopeland/Stores/blob/c178c7ef2571ec31c3806e378535c1e0a84606fa/JavaScript-Files/begin_round.js
.
I think we should shorten this function! Any ideas @Goredbirds
https://github.com/Goredbirds?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/StevenCopeland/Stores/issues/41, or mute the thread
https://github.com/notifications/unsubscribe-auth/AYAvcetLP1klntkFEw3HUwzjdApz3p1Hks5roYJngaJpZM4MlxBt
.

Oh........ Great......... Perfect........
Well, then you want me to fix that right? I'll do that a little bit later because I probably need to get off now. I'll fix that.

On saturday at codecademy, I'll make the 1-4 player version. Once I'm done with that you can look at the code.

Why don't you just fork this and do it here! Also, I will not be on Github all day on Saturday because we have the ICCFS.

this issue is now useless


I can't see my device in the traccar server!

Dears ,,
Greetings ,,
As the subject states, I can't see my device on the Traccar server.
And here is all of my settings :
1- I installed the windows version of Traccar server and open it with localhost:8082.
2- I donwloaded the Traccar client from google play and put my computer IP address the as the server address.
3- I put the port as 5005, after I managed to make port forwarding to allow my computer to be accessible through the internet.
4- The connection is succeed and location updated without any send failed in the device.
5- and here is my server log :
2014-09-14 08:56:30  WARN: Referential integrity constraint violation: "FK65C08C6ADB0C3B8A: PUBLIC.POSITIONS FOREIGN KEY(DEVICE_ID) REFERENCES PUBLIC.DEVICES(ID) (5)"; SQL statement:
    INSERT INTO positions (device_id, time, valid, latitude, longitude, altitude, speed, course, power, other)
    VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?);
 [23506-180] - org.h2.jdbc.JdbcSQLException (DbException.java:345)

2014-09-14 08:56:31 DEBUG: [5005 <- <IP_ADDRESS>] - HEX: 244750524d432c3035353632332e3038312c412c323434322e373938342c4e2c30343634312e383334362c452c302e30302c302e30302c3134303931342c2c2a30380d0a
2014-09-14 08:56:31 DEBUG: [5005 -> <IP_ADDRESS>] - HEX: 4f4b310d0a
2014-09-14 08:56:31  INFO: device: 5, time: Sun Sep 14 08:56:23 AST 2014, lat: 24.713306666666668, lon: 46.69724333333333
2014-09-14 08:56:31  WARN: Referential integrity constraint violation: "FK65C08C6ADB0C3B8A: PUBLIC.POSITIONS FOREIGN KEY(DEVICE_ID) REFERENCES PUBLIC.DEVICES(ID) (5)"; SQL statement:
    INSERT INTO positions (device_id, time, valid, latitude, longitude, altitude, speed, course, power, other)
    VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?);
 [23506-180] - org.h2.jdbc.JdbcSQLException (DbException.java:345)

2014-09-14 08:56:32 DEBUG: [5005 <- <IP_ADDRESS>] - HEX: 244750524d432c3035353632332e3038312c412c323434322e373938342c4e2c30343634312e383334362c452c302e30302c302e30302c3134303931342c2c2a30380d0a
2014-09-14 08:56:32 DEBUG: [5005 -> <IP_ADDRESS>] - HEX: 4f4b310d0a
2014-09-14 08:56:32  INFO: device: 5, time: Sun Sep 14 08:56:23 AST 2014, lat: 24.713306666666668, lon: 46.69724333333333
2014-09-14 08:56:32  WARN: Referential integrity constraint violation: "FK65C08C6ADB0C3B8A: PUBLIC.POSITIONS FOREIGN KEY(DEVICE_ID) REFERENCES PUBLIC.DEVICES(ID) (5)"; SQL statement:
    INSERT INTO positions (device_id, time, valid, latitude, longitude, altitude, speed, course, power, other)
    VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?);
 [23506-180] - org.h2.jdbc.JdbcSQLException (DbException.java:345)

Kindly help me to solve the problem.
Regards.

The restart fixed the same issue for me.

Hello dears,
Why I'm engaged to this conversation?
On Sunday, October 25, 2015, Sergio Rubio<EMAIL_ADDRESS>wrote:

The restart fixed the same issue for me.
—
Reply to this email directly or view it on GitHub
https://github.com/tananaev/traccar/issues/872#issuecomment-150946822.

@ammarimad you can click unsubscribe (right side of this page) if you're not interested in receiving notifications from thread anymore.


Migration necessary before importing addresses

Awesome package.
I think it should say in the README file that the user must run
$ python manage.py migrate

before running
$ python manage.py import_ice_addresses

Also there is a new version of Stadfangaskra available here: http://www.opingogn.is/dataset/stadsetning-fasteigna/resource/bd9e812b-2e8a-4705-86a8-cd235658cc46

Nice.
I see that
$ python manage.py migrate

is now in the README. I also see that you have run
$ python manage.py makemigrations

and added the migrations to the project so the migration stuff from Django 1.7 is almost all there. You probably just have to remove the south migrations directory.
Perhaps the import_ice_addresses command should be added to a RunPython migration, that would conform to the Django 1.7 style.
Once the stadfangaskra has been updated you can close this issue :8ball:


Fixed a typo in the README.md

Problem: There was a typo in the README.md (technolgoies)
Solution: I fixed the typo (technologies)
Signed-off-by: Dino Omanovic<EMAIL_ADDRESS>

Closing this as it was fixed by https://github.com/k3s-io/k3s/pull/3048.


Failing spec for trait with same name as factory

After upgrading from 2.6.4 → 4.8.0 we started getting a "Self-referencing association" error.  The problem is that the trait in question had the same name as a factory, as this spec illustrates.

Oops, that commit should have said closes #1221 instead of #1017.


Make a repos.html page

Make a repos.html page
List links to projects' GitHub Pages if they have them
Maybe something like this:
* **[RepositoryName](https://link-to-pages)** ([repo](https://link-to-repo))
  _Reference, if this is forked or in another org_
  Show the description of the repository here

RepositoryName (repo)
Reference, if this is forked or in another org
Show the description of the repository here

Use the default layout, which will put a h1 title at the top
We'll link to this page from the default head

This will help:
https://help.github.com/articles/repository-metadata-on-github-pages/

Did it: https://solvaholic.github.io/repos.html


web: Update type-system.md

The correct syntax for a link in markdown is [text](link url).
The delimiter were swapped.
Classic error.

Thank you @mav3ri3k !


Objective-C interface support

I'm trying to try out SwiftCoAP with Objective-C. Are there any examples of how to send requests / act as a server bridged from Objective-C.

Hey daveismith. Unfortunately, I cannot provide an example in Objective-C at the moment. Do you have problems using the library in an Objective-C environment?

I couldn't actually figure out how to instantiate an SCMessage and set the
code, etc.
David
On Thu, Oct 13, 2016 at 3:54 AM, Wojtek Kordylewski <
<EMAIL_ADDRESS>wrote:

Hey daveismith. Unfortunately, I cannot provide an example in Objective-C
at the moment. Do you have problems using the library in an Objective-C
environment?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stuffrabbit/SwiftCoAP/issues/6#issuecomment-253441087,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABp3wz61ohNg6SoiC2qrKUk21_tD3M-iks5qzeO-gaJpZM4KUHx8
.

Do you have any updates on this? Maybe you could try the iCoAP library for now.

Unfortunately for the IoT standard I'm working on I need to have both a client and server running. I'll figure out swift eventually. I haven't had opportunity to loop back to this as I've been too busy with other parts of the project.


Data Lake Store Filesystem named "GroupClient"

Why is the client of Data Lake Store File system call "GroupClient" instead of something Azure Data Lake Store filesystem related?
https://github.com/Azure/azure-sdk-for-go/blob/b558f7a651d50f9b2256b1f45abfef9e92efdfae/datalake-store/filesystem/filesystemgroup.go#L37

Howdy @arsenvlad, it's a good question and naming has certainly been something we spend a lot of time wringing our hands about. In this case, the best answer I have for you is that there is the potential for their to be operations that don't pertain to FileSystemGroups, but still pertain to DataLake Store Files. For that reason, we try to name clients as locally to the functionality they represent as possible.


[Design] Removing ModelMetadata.Model

Removing ModelMetadata.Model and replacing with a new abstraction ModelExplorer.
This means that any functionality related to the actual Model value has to leave ModelMetadata. ModelMetdata is now dedicated to static functionality around the declared type of a model.
ModelExplorer is a new abstraction for consumers that want to walk a runtime model and consume model metadata at the same time. ModelExplorer is based on the runtime type of the model. Of course, the model may be null, so we have the standard fallback-to-declared-type functionality.
Summary of the impact of this change:
Anything that handles display needs to work on ModelExplorer now. This includes all the HtmlHelper protected methods, expression processing, display templates and VDD.
Validation makes use of ModelExplorer as it's often concerned with property traversal.
Model Binding does not make use of ModelExplorer as it needs to set data. The model binding context now has a Model property to pass the container when binding subproperties. Note that some model binding code invokes validators, and so needs to use ModelExplorer for this purpose.

:watch: (looking good.  like the removal of all the modelAccessor: null and () => null arguments.)

@dougbu updated

:shipit: once build.cmd succeeds


[Bug] can not input Chinese characters when creating indexes

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to 'Accounts'
Click on 'Indexes'
Create new Index, input Chinese characters in the form and click create button
See error input validation error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Versions (please complete the following information):

Release Version installed [e.g. v1.0.3]

Additional context
Add any other context about the problem here.

Fixed this issue in PR


fix: correct typo + capitalization in title

Changed "Getting starting fine-tuning Mistral 7B" to "Getting Started with Fine-Tuning Mistral 7B". Corrected "starting" to "started" to fix the verb form, added "with" for grammatical correctness, and capitalized the title for proper title case formatting.

Thanks!


Make version 4.9 possible

would be very nice when we can use that module for oxid-ce version 4.9, too.
The problem is, that the oxModule class has changed since version 4.7.

I merged a pull request that updates the structure to Oxid 4.9 or higher.

Thanks a lot.


Cannot untick buff

How buff supposed to be unticked? I tried so much.

Right click on the specific buff in the popup menu that appears for each category


Exchange update

Added Melotic and removed ANXPRO and Vault of Satoshi

@TheWildHorse Please merge :)

@TheWildHorse Also removed Crypto-Trade. Please merge :)

@TheWildHorse Also added PS Coin to exchanges. Please merge :)

@FuzzyBearBTC Can you merge this pull, please.

Sorreh, mergeing now.


hello ，how to get data to train model?

execution order:

data_preparation.py
feature_engineering.py
train.py

then
predict.py (I am still working on this part)


safari: mouse over error does not render text

make typo
mouse over error
only a gray box with no text is shown.

I have no plugins that should be causing this, and it works perfectly with chrome.

Ey! https://github.com/Microsoft/monaco-editor/issues/408 seems to be releated. A workaround is provided, so I'll check out if it doesn't break anything and try fix it

@RabsRincon thank your for the quick response.
The monaco-issue seems related, as I get the exact same symptoms as described in https://github.com/Microsoft/monaco-editor/issues/408
My safari version is 10.1.1 (12603.2.1)
Running on MacOS 10.12.5 Beta


portalicious: payment page breadcrumbs AB#31322

AB#31322
Describe your changes

Made breadcrumbs generic
Created payment page header
Changed a bit ifs and elses in the layout component

Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have added tests wherever relevant
[x] I have made sure that all automated checks pass before requesting a review
[x] I do not need any deviation from our PR guidelines

Closing this in favour of #6145


Feature allow integration for media upload for other CMS

Context
Some CMS may not provide a MediaModal like WordPress does, this PR aims to provide some flexibility to creators who want to add some props via CMS-Story-Editor component like wp-story-editor. These attributes may be data-bs-* which will help integrators provide their own modal while integrating with other CMS.
Summary
Relevant Technical Choices
To-do
User-facing changes
Testing Instructions

Pass some attributes via MediaUpload component from wp-story-editor File
Create an object with attributes.
Pass them via the render function.
Check the elements in developer tools.
Search for the button which opens the WordPress MediaModal.
You should now see the attribute which you had passed.

[x] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:
Reviews
Does this PR have a security-related impact?

Does this PR change what data or activity we track or use?

Does this PR have a legal-related impact?

Checklist

[ ] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This code is covered by automated tests (unit, integration, and/or e2e) to verify it works as intended (docs)
[x] I have added documentation where necessary
[ ] I have added a matching Type: XYZ label to the PR

Fixes #9927

Havent included MediaUpload used in wp-story-editor and stories-block because of the following reasons:

wp-story-editor is decoupled and any other CMS will have CMS-story-editor then it would be the integrators job to integrate these attributes into their decoupled CMS-story-editor.
Not every CMS will have the same stories-block for embedding stories in the article.

Closing as https://github.com/google/web-stories-wp/issues/9927#issuecomment-990852676


GT06 decodeStatus function

Hi!
I've been working on some of the status values such as ignition and power. The ignition seems to be working fine, I do get different values, which tells me if it is on or off. But when it comes to the power information, I always get full battery, no matter what the real status of the battery is. I've tested unplugging the battery from the car and leaving it off for hours, and no sign of change.
So, I've starting digging up the code. More specifically, the decodeStatus function from the Gt06ProtocolDecoder class. There, on line 162 (master), I've noticed that the param for the KEY_POWER is "buf.readUnsignedByte()", which is the same as KEY_GSM and KEY_STATUS, the latter being the flags variable which has the same value at end.
My question is: is this correct? Shouldn't it be passing a value like on line 161 BitUtil.check(flags, 1), changing the param "flags" in each case with different values?
What can you tell me about all that? I'm sorry if I misinterpreted something here, I'm just trying to understand if it's something wrong with the code or with my devices.
Thanks!

I don't quite understand the question. Ignition is a flag (1 bit). Power is a numerical value that is stored as full byte. Everything looks correct to me.

Let me try explaining this another way.
For example, I've decoded the hex code:
78781f1210051310340dc702233e7204b760d40438ee02d4170083002add002210970d0a
This is code I got from my Traccar log, and it was send by one of my devices.
The Status info starts at the byte 32 and ends at the byte 34, as follows:
221097
22 is the device information content, and we need to convert to binary, as follows:
00100010
Breaking it down, from right to left:
0 // 0 not fortified / 1 fortified
1 // 0 low ACC / 1 high ACC
0 // 0 not charged / 1 charged ??????????
0 // 000 normal / 001 vibration alarm / 011 low-power alarm / 100 sos
0
1
0 // 0 gps has not located / 1 gps has located
0 // 0 petrol/electricity on / 1 petrol/electricity off

As you can se the charging flag bit is set to 0, as it should, since I've detached the battery from the car. But this information is being ignored on traccar, as it is always is set to full on the database:
"power":6
What am I missing here? Is this really the bit traccar checks?
Can you explain me what's is going on this method, and how do you get different values from the same method buf.readUnsignedByte()?
    private void decodeStatus(Position position, ChannelBuffer buf) {

        position.set(Event.KEY_ALARM, true);

        int flags = buf.readUnsignedByte();

        position.set(Event.KEY_IGNITION, BitUtil.check(flags, 1));
        position.set(Event.KEY_STATUS, flags);
        position.set(Event.KEY_POWER, buf.readUnsignedByte());
        position.set(Event.KEY_GSM, buf.readUnsignedByte());
    }

Hope this helps!
Thanks!

Second byte in status block is voltage. That's what you get in the "power" field. It has nothing to do with device information bitmask.


ISPN-5687 Server thread pools

https://issues.jboss.org/browse/ISPN-5687
Inline the thread pool configuration into the infinispan subsystem since WildFly 9 has deprecated the threads subsytem.
I apologize for the large number of changes, but these are necessary to minimize alignment with the equivalent subsystem in WF.
Also bump the WildFly used for integration tests to 10.0.0.Beta1

None of the failures are related to this, yay !

merged, tks @tristantarrant !


refactor: don't save channel in toolchain installation

Removing leftover code from a previous iteration of fuelup where we saved channels to the toolchain directory as a method of comparing nightly versions.

CI problems seem to have cleared up now


[FIX] canvas size bug

_currentSize only init once, so that will not update when we call mutate().
Issue related: #41

@PatrickStar125 Thank you for the contribution. Looks good!


EntityCast - Boolean - Use Filter Var

Description
We have bool cast just cast the type to (bool) this changes it to use filter_var instead so values like true , false, yes, no can be casted properly to boolean.
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

I would prefer to leave it as it is. PHP booleans are confusing (as this thread demonstrates) so I think sticking to the docs will help developers know what to expect: https://www.php.net/manual/en/language.types.boolean.php#language.types.boolean.casting
If any other cast of (bool) 'no' would be true then why should our Entities return false?

I would prefer to leave it as it is. PHP booleans are confusing (as this thread demonstrates) so I think sticking to the docs will help developers know what to expect: https://www.php.net/manual/en/language.types.boolean.php#language.types.boolean.casting
If any other cast of (bool) 'no' would be true then why should our Entities return false?

The issue happens with nosql databases like ElasticSearch or Mongo where db will return string false instead of bool value

See also https://www.php.net/manual/en/function.boolval.php

What about adding a new cast 'strict-bool' which will do the casting on both setter and getter with filter var ?

The issue happens with nosql databases like ElasticSearch or Mongo where db will return string false instead of bool value

Hmm that is unfortunate. Where is that coming from? Native PHP drivers or extensions?
Would we want to "fix" this for all Results at the driver level? E.g. $result = $query->get()->getResultArray() would still have 'false' which could cause unexpected issues mixing Entities and raw results.

The 'false' can come form ElasticSearch which accepts both 'false' and false in json and it index the value properly as bool but reading it back it can return it either as string or as bool

I really don't think this is an Entity issue but a problem we are inheriting from PHP generally and these database drivers specifically. I would like to see a little research on how other frameworks/libraries handle this, but my instinct is still the same: we either should not "fix" this (let the developers handle it) or we should handle it universally before Entity ever comes into play.

Closing, as this seems to be needing to be seen from a different angle. Feel free to open a new PR with a new implementation.


[WIP] Add fix & tests

I've found some issues with JSONField and want to discuss expected behaviour

Should None value be saved as JSON null or as SQL NULL ?
String foo will be encoded like "foo" and it will be valid JSON value, and will be decoded correctly as well. Is it expected behavior on such input values?

If #818 will be merged, then all tests will pass. I've changed some behavior of JSONField.
Also I want to add more tests (e.g. on update())

I committed a few of your changes to fix the JSONField tests.
Would like to get other people testing this so we can be sure it does not break anything.
After that lets merge the other changes and discuss fixes/improvement that changes the behavior of JSONField.

Sounds cool! I'll do rebase and adapt these changes

Sorry for having to rebase.

Hello! Any news? :)

As friendly reminder :) any feedback? Are these changes OK?

Sorry for the (far too) late reply.
It's been busy, don't we all wish that we could make this our day job ? :-)
Does this PR still changes the behavior of the JSONField (in a backwards incompatible way) ?
And does fix things that are still broken ?

@Stranger6667 @trbs any update on this ?

I've tested and django 1.9.X with django-extensions 1.5.x is also broken.
Python 2.7.10 (default, Sep 23 2015, 04:34:21)
[GCC 4.2.1 Compatible Apple LLVM 7.0.0 (clang-700.0.72)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
(InteractiveConsole)
>>> xa = TestJson.objects.create(json={'a': 1})
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "/Users/wsantos/.virtualenvs/dpool/lib/python2.7/site-packages/django/db/models/manager.py", line 122, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/Users/wsantos/.virtualenvs/dpool/lib/python2.7/site-packages/django/db/models/query.py", line 401, in create
    obj.save(force_insert=True, using=self.db)
  File "/Users/wsantos/.virtualenvs/dpool/lib/python2.7/site-packages/django/db/models/base.py", line 708, in save
    force_update=force_update, update_fields=update_fields)
  File "/Users/wsantos/.virtualenvs/dpool/lib/python2.7/site-packages/django/db/models/base.py", line 736, in save_base
    updated = self._save_table(raw, cls, force_insert, force_update, using, update_fields)
  File "/Users/wsantos/.virtualenvs/dpool/lib/python2.7/site-packages/django/db/models/base.py", line 820, in _save_table
    result = self._do_insert(cls._base_manager, using, fields, update_pk, raw)
  File "/Users/wsantos/.virtualenvs/dpool/lib/python2.7/site-packages/django/db/models/base.py", line 859, in _do_insert
    using=using, raw=raw)
  File "/Users/wsantos/.virtualenvs/dpool/lib/python2.7/site-packages/django/db/models/manager.py", line 122, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/Users/wsantos/.virtualenvs/dpool/lib/python2.7/site-packages/django/db/models/query.py", line 1039, in _insert
    return query.get_compiler(using=using).execute_sql(return_id)
  File "/Users/wsantos/.virtualenvs/dpool/lib/python2.7/site-packages/django/db/models/sql/compiler.py", line 1060, in execute_sql
    cursor.execute(sql, params)
  File "/Users/wsantos/.virtualenvs/dpool/lib/python2.7/site-packages/django/db/backends/utils.py", line 83, in execute
    sql = self.db.ops.last_executed_query(self.cursor, sql, params)
  File "/Users/wsantos/.virtualenvs/dpool/lib/python2.7/site-packages/django/db/backends/sqlite3/operations.py", line 140, in last_executed_query
    params = self._quote_params_for_last_executed_query(params)
  File "/Users/wsantos/.virtualenvs/dpool/lib/python2.7/site-packages/django/db/backends/sqlite3/operations.py", line 129, in _quote_params_for_last_executed_query
    return cursor.execute(sql, params).fetchone()
InterfaceError: Error binding parameter 0 - probably unsupported type.
>>> import django
>>> import django_extensions
>>> django.VERSION
(1, 9, 7, 'final', 0)
>>> django_extensions.VERSION
(1, 5, 9)
>>>

Is this still a problem ?


Configuration: specifiy font size

First of all, looks quite nice what you got here! I like what I've seen after playing with it a bit
Anyway, I noticed that the font on my setup appears a notch too large and I'd like to use a smaller font size. I haven't seen a hint in the example config, so is this already possible through the config?
If not and you're interested in getting help, I'd like to give it a shoot and try implementing this feature.

Oops, thanks for noticing, forgot to update the readme file. The feature was already implemented in https://github.com/l4l/yofi/commit/1a34eb29b8219f39787f56e6aedf5b7f627ff179 (yet no release for that fix) so all you need is to add font_size = 14 to config file.

Added to sample config at readme: https://github.com/l4l/yofi/commit/1dc20840a2543ece1f93073c1f5ee23060e28aa2

Nice, thank you!


[Bug?]: HMR briefly updates before reverting in docker container with windows host.

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
When I make a change to my source code, the web page updates briefly to reflect my changes before reverting.
Expected behavior 🤔
When I make a change to my source code, I expect the web page to update and be persistent.
Steps to reproduce 🕹
I've created a minimal setup to reproduce this bug.
https://github.com/NathanSMB/devcontainer-bug
Steps:

Have a windows machine with docker setup and the dev containers extension in VS Code
Clone the repo above and open it from the container using the extension
Got to localhost:3000 in your browser
Make a change to the source code and observe the browser.

Note: There is a gif in the above repo that shows the bug in action.
Context 🔦
I'm trying to setup a devcontainer environment that allows for quick onboarding. I want it to be as simple as building and connecting to the containerized environment and then changes happen automatically. I feel like I'm this 🤏 close to getting it working.
Your environment 🌎
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.60
MSRDC version: 1.2.5105
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.22631.3296

Docker Desktop Version: 4.28.0 (139021)
Docker Engine Version: 25.0.3
Docker Compose Version: Compose: v2.24.6-desktop.1

Docker desktop is using the WSL2 based engine.

Note: I have confirmed this does not happen from my m2 Macbook so it seems to be Windows specific.

Adding usePolling to all routers should fix this.
The page reload (ssr) returns stale data. Looks like the ssr router never gets the fs event but the client build is updated correctly. When navigating between routes it shows the changes.
import { defineConfig } from "@solidjs/start/config";

export default defineConfig({
    vite ({ router }) {
        if (router === "client") {
            return {
                server: {
                    hmr: {
                        port: 4000,
                    },
                    watch: {
                        usePolling: true,
                    }
                },
            };
        }
        return {
            server: {
                watch: {
                    usePolling: true,
                }
            }
        };
    }
});

Thank you! I tried that and it worked.