Error: Invariant failed, when react-router v5 export as webpack libraryTarget commonjs

to implement ssr.   at server side, i wan't use webpack. so i export react router as  library and import it at server side。
Version
v5.0.1
Test Case
Steps to reproduce
// App.js export as webpack libraryTarget commonjs

const React = require('react');
const { StaticRouter,  Route } = require('react-router')
const ReactDOMServer = require("react-dom/server");

module.exports = function App() {
    return (
        <div>
            <Route path="/" component={() => 'hello456'}/>
            <Route path="/123" component={() => 'hello123'}/>
        </div>
    )
}

//webpack config
  ...
  output: {
        library: 'xxx',
        libraryTarget: 'commonjs',
    },

ssr.js

const React = require('react');
const r = React;
const ReactDOMServer = require("react-dom/server");
const xx =  require('react-router');

const StaticRouter = xx.StaticRouter;

const App =  require('../dist/static/js/main.615930d8.js').xxx;
console.log(App.toString(), 111);

//function App(){
    //return r.createElement("div",null,r.createElement(route,{path:"/",component:function(){return"hello456"}}),r.createElement(route,{path:"/123",component:function(){return"hello123"}}))}

const context = {};
const html = ReactDOMServer.renderToString(
    React.createElement(StaticRouter, {
        location: '/',
        context,
    }, React.createElement(App))
);

console.log(html);

run  ssr.js
/Users/lijun/git/ty-h5/node_modules/react-dom/cjs/react-dom-server.node.development.js:3166
            throw err;
            ^

Error: Invariant failed
    at S (/Users/lijun/git/ty-h5/dist/static/js/main.615930d8.js:30:21892)
    at Object.children (/Users/lijun/git/ty-h5/dist/static/js/main.615930d8.js:30:29101)
    at ReactDOMServerRenderer.render (/Users/lijun/git/ty-h5/node_modules/react-dom/cjs/react-dom-server.node.development.js:3412:55)
    at ReactDOMServerRenderer.read (/Users/lijun/git/ty-h5/node_modules/react-dom/cjs/react-dom-server.node.development.js:3161:29)
    at Object.renderToString (/Users/lijun/git/ty-h5/node_modules/react-dom/cjs/react-dom-server.node.development.js:3646:27)
    at Object.<anonymous> (/Users/lijun/git/ty-h5/src/test.js:15:29)
    at Module._compile (internal/modules/cjs/loader.js:689:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
    at Module.load (internal/modules/cjs/loader.js:599:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:538:12)

Expected Behavior
console  hello456
Actual Behavior
v4.3.1 work fine, but v5.0.1 not working

This doesn't have anything to do with React Router.

This doesn't have anything to do with React Router.

two version have diff result, you say nothing with React Router.

@8427003 You have a repository ready for reproduction?
@timdorr I agree with 8427003. Given the info we have here, this smells like RR is at fault. It might not be a bug, but I think you close this prematurely.

This is the problem:
const xx =  require('react-router');
const StaticRouter = xx.StaticRouter;
const App =  require('../dist/static/js/main.615930d8.js').xxx;

You're pulling in react-router directly, but pulling in your App from what appears to be a built asset. That's likely giving you two different instances of React Router's Context. This is a common issue, but not a bug with the library.
You need to point to the exact same copy of react-router. I'm not sure how best to do that in your setup, but that's what you need to do.


xDscWebService: DefaultAppPool crash. AcceptSelfSignedCertificates should default to $false

Trying to use the xDscWebService resource, the DefaultAppPool crashes on a 64bit win2012 server with the following message:
The Module DLL 'C:\Windows\SysWOW64\inetsrv\IISSelfSignedCertModule.dll' could not be loaded due to a configuration problem. The current configuration only supports loading images built for a AMD64 processor architecture. The data field contains the error number. To learn more about this issue, including how to troubleshooting this kind of processor architecture mismatch error, see http://go.microsoft.com/fwlink/?LinkId=29349.
Application pool 'DefaultAppPool' is being automatically disabled due to a series of failures in the process(es) serving that application pool.
Checking on the administration.config file, this line was added:

This configuration is wrong because it must contain the precondition "bitness32".
Even after removing this configuration, iis was still trying to load that dll, so any advice on where else this is configured would be helpful. For the moment the only way I found to temporary fix it is to enable the 32bit compatibility on the app pool.
I also don't understand why this was installed while I was not using ssl on the pull server.
This is the DSC configuration:
xDscWebService PSDSCPullServer
{
Ensure                  = "Present"
EndpointName            = "PSDSCPullServer"
Port                    = 8082
PhysicalPath            = "$env:SystemDrive\inetpub\PSDSCPullServer"
CertificateThumbPrint   = "AllowUnencryptedTraffic"
ModulePath              = "$env:PROGRAMFILES\WindowsPowerShell\DscService\Modules"
ConfigurationPath       = "$env:PROGRAMFILES\WindowsPowerShell\DscService\Configuration"
State                   = "Started"
DependsOn               = "[WindowsFeature]DSCServiceFeature"
}
I'm using PS 5.0.10586.117
Thanks
Luigi

It turned out that the AcceptSelfSignedCertificates default value is set to true.
I think that it should be set to false by default, for at least 3 reasons:

it's not documented (yet)
it lowers security standards
it doesn't work properly (it installs x86 binaries on x64 systems)

@gigi81
Yes, I saw it too. I am working on updating the resource to enable self signing only when the resource instance sets this property. Thanks for reporting this issue.

@gigi81
IISSelfSignedCertModule.dll is used in the pull scenarios where the nodes (pull clients) are provisioned with registrationKey. This module is not used for setting up self signed certs on the pull server nor in SSL scenario. I understand that the confusion came from the dll name. Let us leave this issue on to-do list . Thanks for reporting it.

I'm getting this same issue.
I'm not sure that changing the title is right on this issue. the symptom observed is that the App Pool stops/crashes, so by changing the title you've potentially made this harder to search for.
I could do with some help working out how to change the AcceptSelfSignedCertificates value too.

This is set in the config for the web server where you generate the original config for the pull server.
xDscWebService PSDSCPullServer {
<snip>
        AcceptSelfSignedCertificates = $false
}

Here is a easy temp fix.
# To resolve issue with "The Module DLL 'C:\windows\SysWOW64\inetsrv\IISSelfSignedCertModule.dll' could not be loaded due to a configuration problem."
# See https://github.com/PowerShell/xPSDesiredStateConfiguration/issues/104
# https://powershell.org/forums/topic/defaultapppool-crashes-after-pullserver-installation/
xWebAppPool DefaultAppPool
{
    Name                  = 'DefaultAppPool'
    State                 = 'Started'
    enable32BitAppOnWin64 = $True
}

I am experiencing the same issue whereby other web application pools get shut down due to the following error "The Module DLL 'C:\Windows\SysWOW64\inetsrv\IISSelfSignedCertModule.dll' could not be loaded due to a configuration problem. The current configuration only supports loading images built for a x86 processor architecture."
Strange thing is there is no loaded IIS module for that assembly.  I tried removing the assembly and that didn't work.  Setting enable32BitAppOnWin64 on the web applications causes other problems to other custom applications that worked fine before installing DSC 5.1.   This is a serious issue because even removing DSC doesn't resolve the issue so my custom applications are now no longer accessible.  This has been an issue for several months in 5.0 and now 5.1 is released and we still have this lingering.  What to do now?

I recommend we set AcceptSelfSignedCertificates to $false by default or even remove the option.  I suspect you'll find that self-signed certificates work without this (at least on 5.0 and newer servers.)

I encountered this issue.
By default, AcceptSelfSignedCertificates = $false, to set AcceptSelfSignedCertificates = $true, I deleted the Website of the Pull server in IIS. And run the Pull Server configuration with AcceptSelfSignedCertificates = $true.
In addition to setting AcceptSelfSignedCertificates = $true, actually this is no 'C:\Windows\SysWOW64\inetsrv\IISSelfSignedCertModule.dll', so search the dll in C root, find one in other directory, then I copied it to directory 'C:\Windows\SysWOW64\inetsrv, then the issue gone

@pngfei Why did you want to enable this?


Added functionality to clone and load Prezto through Zgen, the function ...

Hello, I wrote a function to integrate Prezto framework. It is written along the lines of the oh-my-zsh function. Completion was adjusted accordingly. Also note that the $ZSH variable is now used to point to either oh-my-zsh or Prezto directory using internal function -zgen-append.

Nice!
Can someone verify if it's working properly? I'm unable to test it right now.

Testers are welcome. Please let me know of any issues.

Running zgen prezto worked well for me, but when I tried zgen prezto modules/git, it tried to clone https://github.com/modules/git.git/ instead of just sourcing the modules/git module.
I could make it work with zgen load sorin-ionescu/prezto modules/git, but I guess that the intended behaviour was to make zgen prezto modules/some_plugin work just like zgen oh-my-zsh plugins/some_plugin.

@N4M3Z: any progress on this issue?

Hi all, could you paste here a specific example so I can reproduce it?

BTW, I've managed to use prezto using origin version of zgen by following:
.zshrc#L3-6:
ln -sfT ~/.zgen/sorin-ionescu/prezto-master ~/.zprezto

source /usr/share/zsh/scripts/zgen/zgen.zsh

.zshrc#L11:
zgen load sorin-ionescu/prezto

How do we proceed about this merge? Should I just move the zgen-prezto function to the new version of zgen?

You may try #40, another approach to enable Zheng for prezto

#40 was already merged, shouldn't this one be closed?

Yes :)


Support backspace with "delete" keycode alias

Currently when using the delete alias in a on directive with keyup `delete" does not support using backspace. This seems quite counterintuitive. Additionally, on Mac keyboards the key normally identified as "backspace" on PC keyboards is called "delete." I was fairly surprised when using the delete modifier didn't detect the key.
I came upon this when developing a search / tag field that turned a search query into tokens when hitting enter or space. I wanted a way to delete tokens without using the mouse. The solution I came up with was detecting the delete key and removing the last token if the search field was empty. I spent 10 minutes debugging code until I realized that I had to press fn+delete (equivalent of DEL / forward delete on a Mac laptop-sized keyboard) for it to actually remove the token.
You can see a reduced test case / demo of this here: http://codepen.io/thecrypticace/pen/YwGKmJ
This PR makes a change to allow aliases to map to multiple key codes. And, using that, it adds support for delete to react to both backspace and DEL / forward delete.

Thanks for the detailed explanation and test coverage :+1:

Glad to help.


Add "refresh preset" button

Is your feature request related to a problem? Please describe.
I'm not exactly sure what happened, if it was some change done to maspsx or to the Legacy of Kain: Soul Reaver preset, but when accessing older 100% matching scratches that use $gp values, suddenly a lui instruction appears that bogs the matching percent down. Like in the following example scratch: https://decomp.me/scratch/S2xt6
The only way to fix this currently is to select another preset and then switch back to the Legacy of Kain: Soul Reaver one for the extra instruction to go away, but doing this repetitively like I am currently doing feels like a chore.
Describe the solution you'd like
Some sort of "refresh" button next to the preset dropdown wouldn't be bad.
Describe alternatives you've considered
Being able to reselect the "Legacy of Kain: Soul Reaver" preset on the dropdown list and have it automatically update to the current configuration wouldn't be bad, either.

The reason for the scratch score changing is due to maspsx being updated to better differentiate between the different ASPSX versions.
The scratch you linked is passing -Wa,--aspsx-version=2.79 but the updated preset passes -Wa,--aspsx-version=2.81.
We do not retroactively update scratch compiler flags when a preset is updated - it would end up overwriting any custom compiler flags a user might have changed (and therefore result in a scratch potentially no longer matching).
Selecting a different preset and selecting the correct preset again isn't a hugely different workflow to selecting the same preset again (2 extra clicks) so I'm going to close out this ticket as WONTFIX.


Refactoring for XXE Mitigation.

Reference Issue

Ref. #870 and #873.
What does this implement/fix? Explain your changes.

https://github.com/svg-net/SVG/pull/873#discussion_r687870585
Any other comments?

@kimsey0 - can you please have a look? You are already immersed in this...


[util] Splice for prod ROM

Parameterize the splice script, and add support for splicing ROM production images.
Signed-off-by: Miguel Osorio<EMAIL_ADDRESS>

I am looking at how to align the FPGA build steps in CI with the getting started instructions. I will update this PR with additional documentation updates.


fix google pagespeed bugs

increase home page page speed and another pages

increase home page page speed and another pages

home page speed up.
https://github.com/koii-network/home/tree/feature/speedup


Fix notification re-appears

Closes #1914.
Fixes the condition that triggers the "Awaiting confirmation" notification.

Travis automatic deployment:
https://pr1925--safereact.review.gnosisdev.com/rinkeby/app

Travis automatic deployment:
https://pr1925--safereact.review.gnosisdev.com/rinkeby/app

Looks good, The notification pops up once and then doesn't show again until there is a new tx that needs confirmation and you are logged with a user that can sign it.

Travis automatic deployment:
https://pr1925--safereact.review.gnosisdev.com/rinkeby/app


ZADD thrashes on redis >= 3.0.2 with NX/CH/INCR/XX

Issue
I had some issues with ZADD not actually adding values to the set if NX was set on redis > 3.0.2.  When XX, NX, CH, or INCR was used they were interpreted to be the score and were then intermingled with extra pairs of arguments throughout the function.
Workaround
Just stripping out zadd's internals does get it working:
diff --git a/src/replay/src/lib/txredisapi.py b/src/replay/src/lib/txredisapi.py
index cf233d6..3557589 100644
--- a/src/replay/src/lib/txredisapi.py
+++ b/src/replay/src/lib/txredisapi.py
@@ -1118,11 +1118,22 @@ class BaseRedisProtocol(LineReceiver, policies.TimeoutMixin):
         return self.execute_command("SSCAN", key, *args)

     # Commands operating on sorted zsets (sorted sets)
+    def zadd(self, key, *args):
-    def zadd(self, key, score, member, *args):
         """
         Add the specified member to the Sorted Set value at key
         or update the score if it already exist
         """
-        if args:
-            # Args should be pairs (have even number of elements)
-            if len(args) % 2:
-                return defer.fail(InvalidData(
-                    "Invalid number of arguments to ZADD"))
-            else:
-                l = [score, member]
-                l.extend(args)
-                args = l
-        else:
-            args = [score, member]
         return self.execute_command("ZADD", key, *args)

     def zrem(self, key, *args):

Solution
By the previous behavior I'm guessing you want to keep the error checking in there - would you like it re-added and PR'd with a check for NX/CH/XX/INCR?

Good find, thanks!
Parsing "NX", "CH", ... from *args would be obscure and non-obvious API to my taste.
I'd prefer boolean kwargs: add("key", 2, "two", 3, "three", nx=True). Unfortunately, we can't have keyword args after *args in Python 2, but it can be simulated with **kwargs like this:
def zadd(self, key, score, member, *args, **kwargs):
    nx = kwargs.get('nx', False)
    ...

👍


evolution-data-server: Fix crash

In journalctl, several hundred times:
Aug 28 08:33:44 nixos org.gnome.Shell.CalendarServer[913]: (.gnome-shell-calendar-server-wrapped:3103): ShellCalendarServer-CRITICAL **: create_client_for_source: assertion 'client == NULL' failed
Aug 28 08:33:42 nixos org.gnome.Shell.CalendarServer[913]: (.gnome-shell-calendar-server-wrapped:3103): ShellCalendarServer-WARNING **: The calendar backend for 'Tasks' has crashed.
Aug 28 08:33:42 nixos kernel: traps: .evolution-cale[3150] trap int3 ip:7f8e21e655b0 sp:7ffd0f1c8fa0 error:0
Aug 28 08:33:42 nixos org.gnome.evolution.dataserver.Calendar7[913]: (.evolution-calendar-factory-subprocess-wrapped:3150): GLib-GIO-ERROR **: Settings schema 'org.gnome.system.proxy' is not installed

Basically, gnome-shell starts .evolution-calendar-factory-subprocess-wrapped, it crashes due to lack of gsettings, gnome-shell sees it crashed, restarts it, etc.

Haven't tested but looks ok thanks.

I've tested it, there's still some issue with gnome-keyring which prevents evolution from remembering passwords, but at least it doesn't crash.


Tutorial: Change theme toggle from div to button

What kind of changes does this PR include?

New or updated content

Description

Closes #2146 using the suggested changes

Thanks for grabbing this issue, @Jutanium! Will ask @delucis to confirm this change, and reminder that a change here means a change to https://github.com/withastro/blog-tutorial-demo ... AND in the content-collections branch.  😅  I made an issue to track this: https://github.com/withastro/blog-tutorial-demo/issues/4


Fix docker build on Jenkins

https://fabric8-ci.fusesource.com/view/wildfly-camel/job/wildfly-camel-docker/

Needs docker-maven-plugin 0.13.2

done


ProtoVersion 17 being banned

Hey guys, I was testing the api, and it works like charm, but for some cases, the number is banned right after reading the qr code.
What i investigated is the following:
for newly created whatsapp accounts, the received protoversion in CONN is 17 and when you read the qr code you are banned maybe because we are talking in an old protobuff version.
I tried with a previous number that i had and the protoversion in this case is 15 and the api works like a charm.
do you guys have any ideia about this? have you ever faced something similar?

i have same issue how i can solve it ?

What is your exact definition of being banned? You cannot use WhatsApp Web in a normal browser any more?

I can still use WhatsApp with the package and with the default web client without getting banned. What @SchulteMK asked is still not answered so I'm going to close this issue as well.

Maybe they save the current version when you create a new account and if you ever login with a lower version you are banned?


ci(size-test): run size test with newlib and with newlib-nano

Codecov Report
Patch and project coverage have no change.

Comparison is base (3c81784) 98.69% compared to head (126079a) 98.69%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #36   +/-   ##
=======================================
  Coverage   98.69%   98.69%
=======================================
  Files          45       45
  Lines        4597     4597
=======================================
  Hits         4537     4537
  Misses         60       60

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


Avoid O(n) copies when bridgable NSData objects are available

Data created with .mappedIfSafe or .alwaysMapped would inappropriately cause O(n) copies of that NSData when read.

@swift-ci please smoke test

It might be a bit abusive to put a few gb file in the tests. I guess I could write out a small file and read it back with the mapped version and on Darwin ask the malloc usable size and expect that to be 0

That sounds reasonable to me!

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge


create basic store interface

I will keep working on this branch. This merge will make it easier to show what we've done so far to the professor tonight, but the branch will still be used

alguem muda esse quality gate ai sacanagem esse negocio @icrcode


In the case that you just want it logged and not handle it anymore.

But I mean, why that should be tided with the logger? In the default behavior you don't swallow, which is also the current behavior. If stoping the exception in the call chain is desired I would expect a different handler than the logger.

Alright, I made the ErrorHandler not spill the guts to the client by default instead.

Thanks @jhass , ErrorHandler is the right place. 👍 from my side here.

Anything else needed here?


The service "security.authentication.manager" has a dependency on a non-existent service "security.user.provider.concrete.into_people_admin".

Hi,
I'm trying to use RollerworksMultiUserBundle on my symfony application. However I already had an existing bundle with entities and a fosuser class. So I had to create one new bundle, and apply to one existing bundle. I'm pretty new with symfony and fosuserbundle etc and I'm having a hard time implementing everything. At the moment I have following error:

Here is my security.yml

It seems you haven't configured the mapping of your IntoPeopleAdminBundle.
http://symfony.com/doc/current/cookbook/doctrine/multiple_entity_managers.html
doctrine:
   ...

    orm:
        default_entity_manager: default
        entity_managers:
            default:
                connection: default
                mappings:
                    FOSUserBundle:  ...
                    IntoPeopleAdminBundle: ~ # should be automatically configured with this

As this is a problem with your Doctrine ORM configuration and not directly related to the RollerworksMultiUserBundle you can get support for Symfony and Doctrine at https://stackoverflow.com/ or the #Symfony IRC channel https://symfony.com/irc


continuous data acquistion

For our project, after locking, we would like to acquire continuous data for a longer time to look at narrow frequency features. Currently, we can only store 16384 samples, which is not sufficient for cause. Is there a way to get more continuous data by continuously capturing the 16384 samples or by some other means? I tried to use the new DMA option in red Pitya (for example link) along with pyrpl, but they don't seem to work together. If I start the pyRPl and ssh to Red Pitaya to start the DMA acquisition, it doesn't work. But without starting Pyrpl, I can SSH and acquire a longer stream of RF input channels. Let me know if there are options I can explore. Thanks.

Have you looked at the sampler module ?

Ok I guess I read your question to fast. The sampler module can be used to acquire data for a long time with a low and not well define sampling frequency.
Regarding your issue, I guess the two ssh connection are clashing. I took a look at your example, are you doing it the same way ? If so an option could be to not open a second connection but to send the SCPI command using the Pyrpl.rp.ssh connection. Not sure if this would work though.


MMM-NetworkConnection - unable to find server

I am running MM on a rpi 4b.  Here is the error message that I am experiencing:
[18.08.2022 06:00.12.338] [ERROR] Whoops! There was an uncaught exception... [18.08.2022 06:00.12.341] [ERROR] Error: No server found, verify your proxy/network     at gotServers (/home/<user>/MagicMirror/modules/MMM-NetworkConnection/node_modules/speedtest-net/index.js:621:25)     at /home/<user>/MagicMirror/modules/MMM-NetworkConnection/node_modules/speedtest-net/index.js:107:16     at /home/<user>/MagicMirror/modules/MMM-NetworkConnection/node_modules/speedtest-net/index.js:301:7     at Parser.<anonymous> (/home/<user>/MagicMirror/modules/MMM-NetworkConnection/node_modules/xml2js/lib/parser.js:303:18)     at Parser.emit (node:events:527:28)     at Parser.exports.Parser.Parser.parseString (/home/<user>/MagicMirror/modules/MMM-NetworkConnection/node_modules/xml2js/lib/parser.js:313:16)     at Parser.parseString (/home/<user>/MagicMirror/modules/MMM-NetworkConnection/node_modules/xml2js/lib/parser.js:5:59)     at exports.parseString (/home/<user>/MagicMirror/modules/MMM-NetworkConnection/node_modules/xml2js/lib/parser.js:354:19)     at /home/<user>/MagicMirror/modules/MMM-NetworkConnection/node_modules/speedtest-net/index.js:299:5     at /home/<user>/MagicMirror/modules/MMM-NetworkConnection/node_modules/speedtest-net/index.js:107:16 [18.08.2022 06:00.12.342] [ERROR] MagicMirror will not quit, but it might be a good idea to check why this happened. Maybe no internet connection? [18.08.2022 06:00.12.343] [ERROR] If you think this really is an issue, please open an issue on GitHub: https://github.com/MichMich/MagicMirror/issues
DNS resolution is working.  I received this error message several times.

ok, I was able to recreate this. it is a bug in the module  when  the mm config
address:'<IP_ADDRESS>'
is set...
not a MagicMirror problem
apparently the speed-test.net library needs a real IP address
it works ok if address:'localhost' is set

but sometimes it works ok....

Where is this value defined?

at the top of config.js

@droidus @MichMich should be closed, no update for more than a month and unrelated to core mm.


The VMs created by New-ContainerHost.ps1 are missing the ContainerImage provider

@mebersol - Running New-ContainerHost.ps1 installs the ContainerImage provider on the VM host, but not in the created VM. I would expect it to automatically be added to the container host VM so that I could pull down the other base images. I think this should simplify setup quite a bit. Most users today want the Windows Server Core image, and they're having to find the deployment page, rather than the quickstart in order to get it.
View from the container host VM:
PS C:\Users\Administrator> Get-PackageProvider -ListAvailable

Name                     Version          DynamicOptions
----                     -------          --------------
msi                      <IP_ADDRESS>          AdditionalArguments
msu                      <IP_ADDRESS>
PowerShellGet            <IP_ADDRESS>          PackageManagementProvider, Type, Scope, InstallUpdate, NoPathUpdate, Filte...
Programs                 <IP_ADDRESS>          IncludeWindowsInstaller, IncludeSystemComponent

PS C:\Users\Administrator> install-containerimage -Name WindowsServerCore
install-containerimage : The term 'install-containerimage' is not recognized as the name of a cmdlet, function, script
file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct
and try again.
At line:1 char:1
+ install-containerimage -Name WindowsServerCore
+ ~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (install-containerimage:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

Here's the manual steps needed in the VM to get the expected Windows Server Core image:
Install-PackageProvider ContainerImage -Force
Install-ContainerImage -Name WindowsServerCore
Restart-Service  docker
docker tag windowsservercore:10.0.14300.1000 windowsservercore:latest

I think the steps should just be:
Install-ContainerImage -Name WindowsServerCore

Fixing the container provider gets us one step closer

Closing this issue since this installer is no longer in use. Please use the DockerMsftProvider instead.


Add conda recipe

conda-skeleton won't work without setup.py, see https://github.com/conda/conda-build/issues/4313


Update TemporalTableSqlBuilder.cs

Check for a null table name (ex. view)

I created a fork and this pull request for issue #30

Thank you. Please know that I applaud the work you are doing!


"Gârda Seacă" is listed in the drop-down menu

Description:
"Gârda Seacă" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Gârda Seacă" in the search bar.

Expected results
The location is listed in the drop-down menu as "Gârda Seacă, Alba (Gârda de Sus)".


Registry does not respect configured strategy

As I found out the registry class always uses RoundRobinStrategy for ActionCatalog and EventCatalog and my tests showed that the config option registry: {strategy: Moleculer.Strategies.Random} has no effect.
ActionCatalog and EventCatalog are always initialized with RoundRobinStrategy: https://github.com/ice-services/moleculer/blob/master/src/registry/registry.js#L42-L43
And is the correct config name 'strategy' (see service-broker.js:51) or 'StrategyFactory' (see registry.js:38)?

Sorry, I will check it.

Thanks

The correct name in broker options is StrategyFactory, I will fix it.

I rather like the name “strategy” better for an option than having to type out “StrategyFactory”  — which really doesn’t add much clarity IMHO.   So which will you fix?  :)

On Oct 11, 2017, at 4:21 PM, Icebob<EMAIL_ADDRESS>wrote:
The correct name in broker options is StrategyFactory, I will fix it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/ice-services/moleculer/issues/121#issuecomment-335935887, or mute the thread https://github.com/notifications/unsubscribe-auth/AB8M7XUtaSSLgCvTbr_6r482yThGms4aks5srSMugaJpZM4P2D8k.

Yes, I'm thinking on same thing. The strategy is better in options, but StrategyFactory is better in Registry source code :)
So strategy stays, and I fix in the Registry code.

Sounds perfect!

On Oct 11, 2017, at 4:25 PM, Icebob<EMAIL_ADDRESS>wrote:
Yes, I'm thinking on same thing. The strategy is better in options, but StrategyFactory is better in Registry source code :)
So strategy stays, and I fix in the Registry code.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/ice-services/moleculer/issues/121#issuecomment-335937092, or mute the thread https://github.com/notifications/unsubscribe-auth/AB8M7Uh5shvpvCc7bVLrJw2hxjWkajR5ks5srSRSgaJpZM4P2D8k.

Fixed. Tomorrow I will release it.

That's great! Thank you!

Fix is released in v0.11.5


Support SAM2

Add endpoints for SAM2.

Image predictor
Video predictor

https://ai.meta.com/research/publications/sam-2-segment-anything-in-images-and-videos/
https://github.com/facebookresearch/segment-anything-2

Please excuse my ignorance, could you please provide a more detailed description of how to implement this new model?
Thanks so much!

Hi @kess-one, it's done now. https://github.com/ksugar/samapi/issues/21

v0.5.0 now supports SAM2. https://github.com/ksugar/samapi/issues/21


Health Gathering for Doom

Adapt DoomEnviroment and yaml config files to solve health_gathering scenario.

Addressed in this branch https://github.com/moabitcoin/cherry-pytorch/tree/doom-scenario-health

Adressed https://github.com/moabitcoin/cherry-pytorch/pull/25

addressed here https://github.com/moabitcoin/cherry-pytorch/pull/30


ci: raise timeout for golangci

to avoid it locally:
ERRO Running error: context loading failed: failed to load packages: failed to load packages: failed to load with go/packages: context deadline exceeded
ERRO Timeout exceeded: try increasing it by passing --timeout option

I don't see any changes in this comment except for the comment on the existing timeout value. It doesn't match what the PR description says, which also mentions local execution.

I fixed it according you review

Next time please squash the commits


SW-2026 Update remaining weight after withdrawal

In some cases, the remaining quantity on weight-based accessions was not being
updated after withdrawals were added or viability tests were edited. Add test
cases for the specific examples we've encountered, and change the code to fix
them.

Current dependencies on/for this PR:

main

PR #732   👈

This comment was auto-generated by Graphite.


Functions isPathExists(), clickPath() from JTreeFixture stop working with update to 0.11.14 (PathNotFoundException)

Issue appear after update to Remote-Robot 0.11.14, version 0.11.13 and earlier are working without issues
I have some tests working with the Project Explorer (left panel in IntelliJ, there are project files, folder, whole structure)
No issue when creating JTreeFixture fixture, no issue when expanding the tree using this fixture -> means fixture + some functions are without issue
Functions isPathExists() and clickPath() -> PathNotFoundException even when path only one item - the root folder - in my case pe_java_project

We've changed the way how JTreeFixture reads the node values
So now in your case the root node of the tree will contain the path to the project
Try something like tree.clickPath(tree.getValueAtRow(0), ...) or create a subclass of JTreeFixture and replace the underneath fixture in contructor
runJs("""
        ctx.put("fixture", new JTreeFixture(robot, component));
        ctx.put("reader", new BasicJTreeCellReader())
""")

@shishovv Thank you very much, I tried to use the first approach - I have replaced the first item in the path by the tree.getValueAtRow(0) and it worked. This issue is resolved, I close it.


[Question] Provided pre-built binaries releases

Hello @lkeegan,
I see that the releases provided on README contain only libraries and not binaries.
Just curious why binaries are not packaged. Have you seen any issue?
In one of our package (NEURON/CoreNEURON/NMODL - which you know!), we need to ship full compiling environment (for NMODL) and hence was wondering about shipping pre-built LLVM binaries (with binary installers or even for python wheels).
cc: @ferdonline @iomaganaris

Hi @pramodk!
The only reason the binaries are not packaged is that for https://github.com/lkeegan/spatial-model-editor/ all we need are the libraries, and the binaries are quite big.
I haven't seen any issues with the binaries (but also haven't tried shipping them).

Thanks Liam for clarification!  I will fork this and will try enabling binaries. Will see how it goes!


Razor syntax highlighting does not recognize @code or @functions

monaco-editor version: 0.2.0
Browser: Tested on Chromium and Firefox
OS: Ubuntu 19.10
When the language is set to razor, @code or @functions is not recognized and shows up (along with all the C# code inside of it) in the default text color. This can be reproduced by going to the Monaco Editor homepage, setting the language to razor, and entering this code:
@{
    @* This C# code will have syntax highlighting *@
    var test = "testing";
    if (true)
    {
        Console.WriteLine("testing");
    }
}
@code {
    @* This C# code will not have syntax highlighting *@
    var test = "testing";
    if (true)
    {
        Console.WriteLine("testing");
    }
}
@functions {
    @* This C# code will also not have syntax highlighting *@
    var test = "testing";
    if (true)
    {
        Console.WriteLine("testing");
    }
}

I too am interested in making this work. The built-in VS Code razor syntax seems to be here: https://github.com/OmniSharp/omnisharp-vscode/blob/master/src/razor/syntaxes/aspnetcorerazor.tmLanguage.json
How can one translate this to the Monarch syntax?

plz

Could we at least update razor.contribution.ts to declare the '.razor' extension in addition to '.cshtml'?
This could help to enable syntax highlighting on Azure DevOps Repos for Blazor's .razor files. This doesn't fix the issue about the @code and @function syntax but it should at least make the majority of the file display with highlighting instead of plain text.


Module readmes - Test and update parameter usage section in readme in all modules

Description
During a workshop on CARML, teams used the parameter usage sections when creating solutions. Some of these were incorrect, causing the teams to need to check the code for actual parameter usage, thus slowing them down. These parameter usage examples needs to be tested and validated and updated where needed to ensure they are correct.

Linked to #1096

I also would suggest to close this issue as a duplicate of #1096. Moving back to needs triage for this purpose.


create POC

create folder for Python project
download the python .gitignore template
touch README.md
execute git init and commit the initial version

oops, I accidentally pushed to the wrong branch, causing this issue to be resolved directly on main.
This is a one-time occurrence.
fix by: 54d99c6


Can't find "piper" binary on WSL

I was following the tutorial for windows by ssamjh, and it doesn't include info on how to use the model.
So I checked the piper homepage and it says to run the "piper" binary, but when I try to use
echo 'Welcome to the world of speech synthesis!' | \
  ./piper --model ~/piper/modelv3/model.onnx --output_file welcome.wav

I get this error
-bash: ./piper: No such file or directory
I tried navigating to different folders and I can't seem to find what directory it's in.

Okay, so I figured it out. The instructions in the tutorial for Windows by ssamjh don't include a section for installing the main piper release. So, to fix this you will need to follow these instructions.
Open WSL and run:
dpkg --print-architecture

This will tell you WSL's architecture. Then you will need to go to releases on piper and download the respective release. Once you download it, you will need to navigate to the file in WSL. Run this command, making sure to replace 'piper_release' with the correct architecture-specific release filename:
tar -xzvf piper_release.tar.gz.tar.gz

Once the files are done extracting, you can copy them to the main piper folder, and you should now be able to use your voice model properly.


Added support for TPM to OVMF

Motivation for this change
Windows 11 requires TPM2 support. This patch adds TPM2 support to OVMF.
Related to #141224
Things done

Added tpmSupport flag to OVMF
Added the corresponding build flags for TPM support

Based on this gentoo patch:
https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=7ece9892529e7690a1c763a4ab7f33046bc069ae

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages 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/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

nixpkgs-review didn't trigger any rebuilds, but rebuilding it manually worked fine. Tested the result OVMF_CODE.fd file on my Windows 10 VM, and while I didn't tested the TPM part itself, it is still reporting that it is Secure Boot enabled (I did a OVMF.override { secureBoot = true; tpmSupport = true; }), so it seems fine.
BTW, right now there is no way to use this package unless you manually create an overlay, but this will generate a local build. I think it would be nice to expose this to all-packages so we could have a hydra cache of this artifact. To avoid breaking changes, I would propose:
  OVMF = callPackage ../applications/virtualization/OVMF { };
  OVMF-CSM = OVMF.override { csmSupport = true; };
  OVMF-secureBoot = OVMF.override { secureBoot = true; };
  OVMF-secureBootTpm = OVMF.override { secureBoot = true; tpmSupport = true; };

WDYT @witchof0x20 ?
Anyway, tested and approved.

Sounds good to me!


CU 5/12

Thanks for the submission! The current output reflected your submission is:
https://hopkinsidd.github.io/rsv-forecast-hub_website/
(The PR was merged into main with the file which was converted origin_date with as.Date)


support multiple yaml config files

This PR is trying to resolve #201 by merging multiple yaml files.
Basic rules are:

replace for scalar
append for array
merge for map

For example, I hava config1.yaml:
http:
  - name: name1
    url: https://github.com/megaease/easeprobe/1
notify:
  log:
    - name: log
      file: /var/log/easeprobe.log
settings:
  probe:
    interval: 1m

config2.yaml:
http:
  - name: name2
    url: https://github.com/megaease/easeprobe/2
settings:
  probe:
    interval: 2m
    timeout: 20s

and the merged  result is equal to
http:
  - name: name1
    url: https://github.com/megaease/easeprobe/1
  - name: name2
    url: https://github.com/megaease/easeprobe/2
notify:
  log:
    - name: log file
      file: /var/log/easeprobe.log
settings:
  probe:
    interval: 2m
    timeout: 20s

Codecov Report
Base: 99.64% // Head: 98.85% // Decreases project coverage by -0.79% :warning:

Coverage data is based on head (cac176a) compared to base (668a312).
Patch coverage: 56.07% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #218      +/-   ##
==========================================
- Coverage   99.64%   98.85%   -0.80%
==========================================
  Files          71       69       -2
  Lines        5061     5143      +82
==========================================
+ Hits         5043     5084      +41
- Misses         12       44      +32
- Partials        6       15       +9

Impacted Files
Coverage Δ

conf/merge.go
55.33% <55.33%> (ø)

conf/conf.go
96.31% <75.00%> (-0.37%)
:arrow_down:

notify/log/log_windows.go

daemon/daemon_unix.go

daemon/daemon_windows.go

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

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

I am not sure this merge function would work well.
because the array is nested in the notify section. and for settings, we should merge the settings instead of overwriting the whole settings
I think we can do this better.
how about just using the yq? here is an example use it as a library: https://github.com/kamolhasan/yq-for-strings

The settings are merged already, the example maybe a little misleading, so I modified it and will add more tests later.
The merge code is simple now, I will research the yq to see if there is a better solution.

At least, I think we must pass the following  mandatory test cases:
1. Section Merge Case
into
http:
  - name: "test 1"
    url: "http://localhost:8080"
    method: "GET"
tcp:
  - name: "test 2"
    host: "localhost:8080"

from
notify:
  slack:
    - name: "slack"
      webhook: "https://hooks.slack.com/services/xxxxxx"

result
http:
  - name: "test 1"
    url: "http://localhost:8080"
    method: "GET"
tcp:
  - name: "test 2"
    host: "localhost:8080"
notify:
  slack:
    - name: "slack"
      webhook: "https://hooks.slack.com/services/xxxxxx"

2. Same Probe Merge Case
into
http:
    - name: "test 1"
        url: "http://localhost:8080"
        method: "GET"

from
http:
    - name: "test 2"
        url: "http://localhost:8181"
        method: "GET"

result
http:
  - name: "test 1"
    url: "http://localhost:8080"
    method: "GET"
  - name: "test 2"
    url: "http://localhost:8181"
    method: "GET"

3. Notify Merge Case
into
notify:
  slack:
    - name: slack
      webhook: "https://hooks.slack.com/services/xxxxxx"

from
notify:
  discord:
    - name: discord
      webhook: "https://discord.com/api/webhooks/xxxxxx"

result
notify:
  slack:
    - name: slack
      webhook: "https://hooks.slack.com/services/xxxxxx"
  discord:
    - name: discord
      webhook: "https://discord.com/api/webhooks/xxxxxx"

4. Notify Array Merge Case
into
notify:
  slack:
    - name: slack1
      webhook: "https://hooks.slack.com/services/xxxxxx"

from
notify:
  slack:
    - name: slack1
      webhook: "https://hooks.slack.com/services/xxxxxx"

result
notify:
  slack:
    - name: slack1
      webhook: "https://hooks.slack.com/services/xxxxxx"
    - name: slack1
      webhook: "https://hooks.slack.com/services/xxxxxx"

5. Settings Merge Case
into
settings:
  name: easeprobe
  sla:
    schedule: "daily"
    time: "00:00"
  notify:
    retry:
      times: 5
      interval: 10

from
settings:
  name: easeprobe_from
  probe:
    timeout: 10s
    interval: 30s
   sla:
    schedule: "weekly"

result
settings:
  name: easeprobe_from
  probe:
    timeout: 10s
    interval: 30s
  notify:
    retry:
      times: 5
      interval: 10
  sla:
    schedule: "weekly"


DCGM_FI_PROF_NVLINK_L0_TX_BYTES can't be collected on A800x8 machine

DCGM_FI_PROF_NVLINK_L0_TX_BYTES can't be collected, while DCGM_FI_PROF_NVLINK_TX_BYTES can be collected.
How to collect DCGM_FI_PROF_NVLINK_L0_TX_BYTES?
nvidia-smi:
$ nvidia-smi
Wed Oct 18 17:59:14 2023
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 535.104.05             Driver Version: 535.104.05   CUDA Version: 12.2     |
|-----------------------------------------+----------------------+----------------------+
| GPU  Name                 Persistence-M | Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |         Memory-Usage | GPU-Util  Compute M. |
|                                         |                      |               MIG M. |
|=========================================+======================+======================|
|   0  NVIDIA A800-SXM4-80GB          Off | 00000000:xxxxxx Off |                    0 |
| N/A   67C    P0             433W / 400W |  77272MiB / 81920MiB |     99%      Default |
|                                         |                      |             Disabled |
+-----------------------------------------+----------------------+----------------------+
... ...
+-----------------------------------------+----------------------+----------------------+
|   7  NVIDIA A800-SXM4-80GB          Off | 00000000:xxxxxx Off |                    0 |
| N/A   69C    P0             441W / 400W |  77556MiB / 81920MiB |    100%      Default |
|                                         |                      |             Disabled |
+-----------------------------------------+----------------------+----------------------+

dcgmi modules:
$ dcgmi modules -l
+-----------+--------------------+--------------------------------------------------+
| List Modules                                                                      |
| Status: Success                                                                   |
+===========+====================+==================================================+
| Module ID | Name               | State                                            |
+-----------+--------------------+--------------------------------------------------+
| 0         | Core               | Loaded                                           |
| 1         | NvSwitch           | Loaded                                           |
| 2         | VGPU               | Not loaded                                       |
| 3         | Introspection      | Not loaded                                       |
| 4         | Health             | Not loaded                                       |
| 5         | Policy             | Not loaded                                       |
| 6         | Config             | Not loaded                                       |
| 7         | Diag               | Not loaded                                       |
| 8         | Profiling          | Loaded                                           |
+-----------+--------------------+--------------------------------------------------+

part of /etc/dcgm-exporter/default-counters.csv:
CGM_FI_PROF_NVLINK_L0_TX_BYTES,     counter, the counter
DCGM_FI_PROF_NVLINK_TX_BYTES,            gauge, The total number of bytes of active NvLink tx (transmit) data including both header and payload.

dcgm-exporter run in docker:
docker run -d --restart=always --gpus all -p 9400:9400 --cap-add SYS_ADMIN nvcr.io/nvidia/k8s/dcgm-exporter:3.2.5-3.1.8-ubuntu20.04 -c 1000

nv-hostengine log:
host_part.log

It appears it is misspelled with "CGM_FI_PROF_NVLINK_L0_TX_BYTES". Also, its numeric value "1040" does not show up in the log, but "1011" for DCGM_FI_PROF_NVLINK_TX_BYTES does show up in the log.

It appears it is misspelled with "CGM_FI_PROF_NVLINK_L0_TX_BYTES". Also, its numeric value "1040" does not show up in the log, but "1011" for DCGM_FI_PROF_NVLINK_TX_BYTES does show up in the log.

Hello, that misspelled field is my mistake when write this issue.  In default-counters.csv it is actually DCGM_FI_PROF_NVLINK_L0_TX_BYTES. And when I manually run dcgm-exporter, the log shows it skippped this field:
$ sudo docker exec 0c3490fc6481 /usr/bin/dcgm-exporter -c 1000
time="2023-10-19T01:44:45Z" level=info msg="Starting dcgm-exporter"
time="2023-10-19T01:44:45Z" level=info msg="DCGM successfully initialized!"
time="2023-10-19T01:44:45Z" level=info msg="Collecting DCP Metrics"
time="2023-10-19T01:44:45Z" level=info msg="No configmap data specified, falling back to metric file /etc/dcgm-exporter/default-counters.csv"
time="2023-10-19T01:44:45Z" level=warning msg="Skipping line 22 ('DCGM_FI_PROF_NVLINK_L0_TX_BYTES'): metric not enabled"
time="2023-10-19T01:44:45Z" level=info msg="Initializing system entities of type: GPU"

And I tested dcgm-exporter on host, it also can't collect DCGM_FI_PROF_NVLINK_L0_TX_BYTES:
Start nv-hostengine and dcgm-exporter:
$ sudo nv-hostengine -f ./host_4.log --log-level debug
Started host engine version 3.2.6 using port number: 5555

$ sudo ./dcgm-exporter/cmd/dcgm-exporter/dcgm-exporter -f ./default-counters.csv -r localhost:5555 -c 2000
INFO[0000] Starting dcgm-exporter
INFO[0000] Attemping to connect to remote hostengine at localhost:5555
INFO[0000] DCGM successfully initialized!
INFO[0000] Collecting DCP Metrics
INFO[0000] No configmap data specified, falling back to metric file ./default-counters.csv
WARN[0000] Skipping line 22 ('DCGM_FI_PROF_NVLINK_L0_TX_BYTES'): metric not enabled
INFO[0000] Initializing system entities of type: GPU
INFO[0000] Initializing system entities of type: NvSwitch
INFO[0000] Initializing system entities of type: NvLink
INFO[0000] Pipeline starting
INFO[0000] Starting webserver

In the other terminal, check metrics:
$ curl localhost:9400/metrics | grep -i DCGM_FI_PROF_NVLINK_L0_TX_BYTES
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 39440    0 39440    0     0  37.6M      0 --:--:-- --:--:-- --:--:-- 37.6M

$ curl localhost:9400/metrics | grep -i DCGM_FI_PROF_NVLINK_TX_BYTES
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 39442    0 39442    0     0  18.8M      0 --:--:-- --:--:-- --:--:-- 37.6M
# HELP DCGM_FI_PROF_NVLINK_TX_BYTES The total number of bytes of active NvLink tx (transmit) data including both header and payload. B/s
# TYPE DCGM_FI_PROF_NVLINK_TX_BYTES gauge
DCGM_FI_PROF_NVLINK_TX_BYTES{gpu="0",UUID="GPU-xxxxxx",device="nvidia0",modelName="NVIDIA A800-SXM4-80GB",Hostname="xxxxxx",DCGM_FI_DRIVER_VERSION="535.104.05"} 238
... ...
DCGM_FI_PROF_NVLINK_TX_BYTES{gpu="7",UUID="GPU-xxxxxx",device="nvidia7",modelName="NVIDIA A800-SXM4-80GB",Hostname="xxxxxx",DCGM_FI_DRIVER_VERSION="535.104.05"} 238

Here are log file and *.csv file:
host_4.log
default-counters.csv

After investigating, this field is only available on H100 and newer.

After investigating, this field is only available on H100 and newer.

Hello, I tried command nvidia-smi nvlink --id 0 --getthroughput d :
$ nvidia-smi nvlink --id 0 --getthroughput d
GPU 0: NVIDIA A800-SXM4-80GB (UUID: GPU-xxxxxxx)
         Link 0: Data Tx:<PHONE_NUMBER>1 KiB
         Link 0: Data Rx:<PHONE_NUMBER>0 KiB
         Link 1: Data Tx:<PHONE_NUMBER>9 KiB
         Link 1: Data Rx:<PHONE_NUMBER>1 KiB
         Link 2: Data Tx:<PHONE_NUMBER>4 KiB
         Link 2: Data Rx:<PHONE_NUMBER>3 KiB
         Link 3: Data Tx:<PHONE_NUMBER>5 KiB
         Link 3: Data Rx:<PHONE_NUMBER>4 KiB
         Link 4: Data Tx:<PHONE_NUMBER>7 KiB
         Link 4: Data Rx:<PHONE_NUMBER>6 KiB
         Link 5: Data Tx:<PHONE_NUMBER>4 KiB
         Link 5: Data Rx:<PHONE_NUMBER>5 KiB
         Link 6: Data Tx:<PHONE_NUMBER>6 KiB
         Link 6: Data Rx:<PHONE_NUMBER>8 KiB
         Link 7: Data Tx:<PHONE_NUMBER>1 KiB
         Link 7: Data Rx:<PHONE_NUMBER>3 KiB

We can see it succeed to get Tx/Rx data counters for every link of a GPU. As far as I know, nvidia-smi is based on NVML, and DCGM is based on NVML too.
Is it a limitation of dcgm-exporter to not collect throughput of every link for GPUs older than H100? or it's a limitation of DCGM libaray?

In the case of the nvidia-smi nvlink command, the data is coming from a different metric that corresponds to DCGM_FI_DEV_NVLINK_BANDWIDTH_L0 (and NVML_FI_DEV_NVLINK_THROUGHPUT_DATA_RX) . See https://github.com/NVIDIA/DCGM/blob/master/dcgmlib/src/DcgmCacheManager.cpp#L10843
The DCGM_FI_PROF_NVLINK_L0_TX_BYTES field is a GPM field and corresponds to NVML_GPM_METRIC_NVLINK_L0_TX_PER_SEC, which is only available on H100 and newer.

I‘ve tested nvml NVML_FI_DEV_NVLINK_THROUGHPUT_DATA_RX, it works. And I maybe should to expect something like DCGM_FI_DEV_NVLINK_BANDWIDTH_Lx_RX/TX to be supported in DCGM.
Thank you!


refactor: optmized requests

DRAFT PR - Aligning backend with Edu to reduce rpc requests.

We forgot to add something very important @RafaDSan :
When the query with readContract(viem) at getTokenData, and it reverts in case the contract isn't ERC20 (no decimals function), it tries to call tokenURI to check if it is ERC721 but the tokenId we are testing is the tokenId of index 1, and sometimes the contract might not have the tokenId  of index 1, resulting in a revert and thus setting the token.type as UNDEFINED.  But this is not the expected behaviour we are looking for.
The revert message will be revert ERC721NonexistentToken(tokenId); for new versions (^8.20) or "ERC721Metadata: URI query for nonexistent token" for most versions.
We must:

[ ] Get the revert error
[ ] Check if it contains ERC721NonexistentToken in the string using regex
[ ] Check if it contains "ERC721Metadata: URI query for nonexistent token"
[ ] Set the token.type equals UNDEFINED as proposed by the types.ts file only if we can't find the strings above in the error message. Otherwise, even if the contract reversed we must set the token.type as ERC721


Reshade.json is broken

Looks like a bad/incomplete merge. Causes scoop search to fail:

oops.

Fixed in commit kiedtl@c020e4c6600597d71fbaba649d2bdfb0ea5b1473


useTranslation hook return object is not memoized

The return object of the useTranslation hook is not memoized, which means it'll be a "new" object on every render.
This is especially an issue when the t function is included in the dependency array of useMemo, useCallback etc.
It can easily be reproduced like this:
import useTranslation from 'next-translate/useTranslation'
import { useMemo, useState } from 'react'
export const UseTranslationRerender: React.FC<{}> = () => {
  const { t } = useTranslation('myroofs')
  const [someValue, setSomeValue] = useState(false)
  useMemo(() => {
    console.log('useMemo')
  }, [t])
  return <button onClick={() => setSomeValue((prev) => !prev)}>Change State</button>
}

By the looks of it the object can be memoized based on the namespace.

https://github.com/vinissimus/next-translate/pull/743

You can try now the 1.3.6-canary.1 with this change

@aralroca looks good to me.

@aralroca Memoizing the t-function seems reasonable. I tried the canary 1 version referenced, but it seems it does not precisely provide what is necessary. I think the ctx-object should not be part of the dependable array; maybe ctx.lang can be allowed, but the entire ctx-object does not work.
I removed the ctx-object locally because page exit transitions still were not working with canary 1.
Page exit transitions due to a route change mean that a component animates out; the past translations have to stay loaded even when Next loads the new page.
Page transitions are "global"; we can not put all page-translations mapping in a common namespace. The page exit transitions are not working because the context has already changed during the route change; mounted components need to keep their translation until the component is unmounted.
export default function useTranslation(defaultNS?: string): I18n {
  const ctx = useContext(I18nContext)
  return useMemo(
    () => ({
      ...ctx,
      t: wrapTWithDefaultNs(ctx.t, defaultNS),
    }),
    [ctx, defaultNS] => remove ctx here !!!
  )
}

Thx!

Thanks @gurkerl83, feel free to PR!

@aralroca I have provided a PR for the problem of not losing translations within page exit transitions. See the commit message for more details. The PR baseline is the canary branch.
https://github.com/vinissimus/next-translate/pull/829
Thx!

Thanks for your contribution @gurkerl83, I prerelease your change to 1.3.6-canary.3.
I hope it works correctly now 😊. If so, I will soon make the release.

@aralroca Tested the canary release, works so far!


Fix broken sonarscan and deploy

This is due to dependabot not having access to the various cloudflare and sonar repo secrets.

Fixed


Print requested V-Sync mode when Print Fps is enabled (3.x)

3.x version of https://github.com/godotengine/godot/pull/56106.
Note that the OS may still override the actual V-Sync mode, but this should provide first steps for troubleshooting already.
This can be used to diagnose why the printed FPS is locked to the monitor refresh rate.

Thanks!


SyntaxError: Unexpected token 'export'

Describe the bug
It seems there is a problem with exporting different versions inside of uuid. My project is using uuidv4 which is using uuid.
To Reproduce
Steps to reproduce the behavior:

Install 'uuidv4'
Run project (commonJs style)
See error

Expected behavior
Expected to run without problem.
Runtime

OS: Win10
Runtime: Node.js
Runtime Version: 14.3.0

Additional context
You can see the error here:
...\node_modules\uuidv4\node_modules\uuid\dist\esm-browser\index.js:1
export { default as v1 } from './v1.js';
^^^^^^

SyntaxError: Unexpected token 'export'

This point is not clear. Is it node project or some bundler, which bundler?

Run project (commonJs style)

It is node project... I guess it happens because of my node written in commonJs and uuid is es6!

Nope. UUID is distributed as commonjs by default.
<EMAIL_ADDRESS>How do you import uuid? And provide please minimal reproduction.

But my console shows an error
node_modules\uuidv4\node_modules\uuid\dist\esm-browser\index.js:1
    export { default as v1 } from './v1.js';
    ^^^^^^

    SyntaxError: Unexpected token 'export'
        at compileFunction (<anonymous>)
        at wrapSafe (internal/modules/cjs/loader.js:1116:16)
        at Module._compile (internal/modules/cjs/loader.js:1164:27)

which pointed to this file:
export { default as v1 } from './v1.js';
export { default as v3 } from './v3.js';
export { default as v4 } from './v4.js';
export { default as v5 } from './v5.js';

What am i missing here?

Show how do you import uuid, give us a reproduction.

This is how i import:
const { uuid } = require('uuidv4');

This package?
https://www.npmjs.com/package/uuidv4

The wrong one. Please read documentation.

I see. This package is the dependency of your one. Could you switch to using uuid package directly?

Closing since this seems to be related to a reverse dependency of this package. Please open a bug report in the appropriate issue tracker if the problem persists: https://github.com/thenativeweb/uuidv4/

Hey folks, I think this still might be an issue, getting exactly the same problem, importing uuid directly.
Made a repo here to show the problem.
Hope it helps

@riccardoNovaglia your node version has incomplete support for the package.exports feature. It's also not actively supported by Node.js anymore, see https://nodejs.org/en/about/releases/
Please upgrade to Node.js v14 if you want to use ESM or downgrade to v12 if you want to use CommonJS.

@ctavan absolutely right. Can confirm it works with both v12 and v14. Thanks for the help!

update your node. i had the same problem using node 13.3.0 and uuid 8.3.0.
then i upgraded to node 14.12.0 and it worked

const { v4 : uuidv4 } = require('uuid');
export { default as v1 } from './v1.js';
^^^^^^
SyntaxError: Unexpected token 'export'
How can I solve this?

@RatonBiswas Why do you mix commonjs and es modules? uuid supports both. And what is your environment? We cannot help without additional information.

update your node. i had the same problem using node 13.3.0 and uuid 8.3.0.
then i upgraded to node 14.12.0 and it worked

yes, you are right.

if it is still not running after updated node, try update  uuid npm install uuid

I still have this issue

@guillefix please see: https://github.com/uuidjs/uuid/issues/488#issuecomment-664585063


Get privacytests working on node v19

Something seems to be broken on node v19, while it runs on v17.

I just ran the code node 19.7.0, so this seems to be a false alarm.


Hand on default layer - what to do?

Hey,
this isn't exactly an issue, but I wasn't sure where else to ask so here goes nothing :)
At some point in my current project my left and right hand objects ended up on the default layer, so now i'm getting the warning "Hand is on default layer. This puts unnecessary strain on hover checks as it is always true for hand colliders (which are then ignored)." (Line 822 in Hand.cs)
In the original prefab the Hand is on no layer, but as far as i know there is no easy way to "unlayer" an object?
Would the proper fix be to put the Hand on a custom layer and simply untag all boxes in the collision matrix?
Thanks in advance, have a nice week! :)
LaKawa

This should help. https://stackoverflow.com/questions/49862799/unity-blank-layer-mask
Add a layer select layer 3 it should be blank.


why the limit is 2-64 hosts？

I didn't see any reason why this tool has this limit. Threads number ? or timeout window?

Hi that is the testing limit. I recall on early tests we saw issue son RDMA tests when over 64 hosts so we limit it to 64 for all.
But again if you run i ton more than 64, please share.


CRUD integration tests remove application

In section 2.3.3, it might be worth documenting that the integration tests for the Vert.x CRUD booster completely remove the application when the tests are complete.  If you navigate to the application overview page after the tests have been run, you get the "Add to Project" button (i.e. an empty project).

@inoxx03 do you want to take this, or would you like me to grab it?

I think all integration tests do that. Actually all integration tests also deploy the app, before the test. They are just completely self-contained. (The only exception is the SSO mission, but let's not go there.)

resolved by PR #619


Fix scrolling by pixels

https://github.com/glideapps/glide-data-grid/assets/512055/2d6c4004-0060-4e68-aab1-491f41549189

@jassmith for merge?


[1.16.5] Game crashes directly after world loading - Java 9?

1.16.5
Forge <IP_ADDRESS>
DynamicSurroundings-1.16.4-<IP_ADDRESS>.jar
It seems to me this mod isn't compatible with Java 8 Runtime anymore.
crash-2021-01-29_09.14.23-client.txt

Same issue for me.

Same issue for me.

Yep - its a dupe.

Yep - its a dupe.


form_ajax_refs are not respected by association_proxy fields unless association proxy has exactly the same name as referenced model field

When I add a field to form_columns that references an association proxy and also add it to form_ajax_refs, the field is not handled as an ajax field by the front end. Looks like a bug in contrib.sqla.form._convert_relation.

Can you please provide a minimal working example that displays the behavior, so it can be investigated?


Shift code

@jstac there are some .py files which are not used at all. What to do with those .py files?

@AnjuJoon This PR is actually quite confusing for me.  I can't keep track of what's happening.
For example, here I see you are dealing with the aiyagari lecture, but there's also another PR related to the same lecture.
What I would like is one PR per lecture.  That PR only deals with fixing one lecture, and it makes all the steps for that lecture together.

Regarding the py files that aren't used, please delete them.

@AnjuJoon This PR is actually quite confusing for me. I can't keep track of what's happening.
For example, here I see you are dealing with the aiyagari lecture, but there's also another PR related to the same lecture.
What I would like is one PR per lecture. That PR only deals with fixing one lecture, and it makes all the steps for that lecture together.

@jstac Please merge this PR and I am making a separate PR for each lecture after this PR. The other PR related to aiyagari lecture, can we just leave that alone for some time? I will close that PR once this will be merged or maybe I could just resolve conflicts in that PR if in case the merge conflicts will not be serious.

Regarding the py files that aren't used, please delete them.

Sure!

@AnjuJoon I don't really like merging PRs that I find confusing but in this case I'll trust your judgement and merge.  Afterwards I'll delete the branch.


Read only teacher class page for school admins

Still WIP

Fixed a typo for success while I was at it, should that be its own change?
Added readOnly "prop" for the template

Closing - it was merged to school-administrator-dashboard


Enhancement: Attract layer with scrolling text.

Hey Team,
We should look at adding a scrolling layer to the attract mode on SG.  You know, for a "Special Thanks" kind of section.

PanningLayer (scrolls any type of layer) exists, as does the MarkupLayer (use to generate long layers of text), but not every layer type is exposed through the yaml interface.  It is worth some discussion to figure out which types of layers we want to expose through that YAML interface.
To do this in code, here is an example:
    credits_frame = gen.frame_for_markup("""
#Special thanks to:#
{multimorphic-logo-color}
[Gerry Stellenberg]
[Adam Preble]
[Josh (Rosh)]
[Scott Danesi]
""")
    thanksto = dmd.PanningLayer(width=self.game.dmd.width, height=self.game.dmd.height, frame=credits_frame, origin=(0,-140), translate=(0,4), bounce=False)
    self.layer = thanksto

Thanks Michael, I think there are some really cool features that we could potentially expose to the YAML interface.  Let me know if you would like to get a discussion going.

Closed.  This and more were added in the most recent commit. here
Docs:
http://pinballprogramming.com/display-layer-markup-language/


HV-1001 @NotEmpty and @NotBlank definition is not consistent with other constraints

https://hibernate.atlassian.net/browse/HV-1001

Updated @NotEmpty and @NotBlank annotations to not use @NotNull and added a property canBeNull to allow null values to be valid.

Hi @marko-bekhta, first off, many thanks for this contribution and your effort.
I'm having some general doubts about the usefulness of HV-1001, though (it's only now that I see it, otherwise I think we'd have it closed already). I see these constraints as some specialization of @NotNull, hence the general recommendation to consider null as valid value does not really apply to these two constraints. They are meant as shortcuts really. If one want to consider null valid, one always can use @Size(min=1) (admittedly, it's not that easy for @NotBlank).
All in all I'd rather have us reject this issue and leave things as are. That said, your contribution is appreciated very much and I can dig out some other issues to work on if you like (though I think @gsmet gave you some already?). WDYT?

Hi @gunnarmorling, it's up to you to decide what to do with the issue and this pull request right :)
Even if it will be rejected, for me it's ok, as I've learned a thing or two while working on it :)
And yes, @gsmet provided me with the list of items (filtered with 'Suitable for new contributors'), so I'll look through those and try find something more to work on. Or if you have any other issues or some issue in particular for me to work on I'd be glad to try.

Ok, then I'd say let's close this issue and the PR. Glad to hear that you are up for other stuff nevertheless, that's great!
In terms of issues to work on, @gsmet surely has compiled a nice list. I've also tagged some issues with the "hackergarten" label which are easy starters: https://hibernate.atlassian.net/issues/?jql=project %3D HV AND labels %3D hackergarten
Feel free to tackle any issue you like. I'd only recommend you leave a short comment on the issue when you get started, so others are aware of you working on it. Looking forward to your further PRs!


Deprecate support for old gems

Before contributing, please read our contributing guidelines and code of conduct.
Overview
Removed older gems from being tested
The following is the updated version support for the changed gems in this PR:
activemerchant: 1.62.0+
delayed_job: 3+
excon: 0.64.0+
grape: 0.17.0+
httprb: 2.2.2+
json: 1.8.6+
padrino: 0.14.0+
bunny: 2.8.1+
redis: 3.3.5+
resque: 1.27.4
sequel: 5.17.0+
sidekiq: 4.2.10+
sinatra: 1.4.8+
typhoeus: 1.3.0+

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.6 out of 7 committers have signed the CLA.:white_check_mark: angelatan2:white_check_mark: tannalynn:white_check_mark: warp:white_check_mark: mwlang:white_check_mark: viraptor:white_check_mark: amhuntsman:x: nbolserYou have signed the CLA already but the status is still pending? Let us recheck it.

This PR is replaced by #639 due to overwhelming git conflicts.


Users of zombie Vue extension cannot get migrated to the new version

Type: Bug
This issue was supposed to be fixed in https://github.com/microsoft/vscode/issues/211702, but we have recently received complaints from some users regarding this.
I counted the first 50 issues with Vue extension installed in https://github.com/microsoft/vscode/issues, and 25 users are using 1.8.27 ~ 2.0.10 (2.0.10 was released 4 months ago ), only 19 users are using the latest version (2.0.22 ~ 2.0.28).
Numbers:

2.0.10: 13 (Issue 225210, Issue 224993, Issue 223464, Issue 222157, Issue 221691, Issue 221503, Issue 221463, Issue 221461, Issue 221229, Issue 220972, Issue 220630, Issue 220106, Issue 220062)
2.0.8: 6 (Issue 224271, Issue 224171, Issue 223290, Issue 221794, Issue 219837, Issue 219707)
2.0.7: 2 (Issue 224527, Issue 224131)
2.0.6: 2 (Issue 224896, Issue 223596)
1.8.27: 2 (Issue 222893, Issue 221266)
2.0.22~2.0.28: 19

Since some versions in 2.0.0~2.0.10 may have serious problems, this is probably the reason why the Vue extension has continued to receive negative reviews on the Marketplace and VSCode has continued to receive bug reports about the Vue extension for several months.
I found a Windows computer with Vue.volar 2.0.7 installed and was able to reproduce the issue of not being able to update the version. The following is the content of .vscode/extensions/extensions.json. (Formatted for easier reading)
[
	{
		"identifier": {
			"id": "vue.volar",
			"uuid": "a5223b43-8621-4351-a14e-3d560f85f277"
		},
		"version": "2.0.7",
		"location": {
			"$mid": 1,
			"fsPath": "c:\\Users\\87156\\.vscode\\extensions\\vue.volar-2.0.7",
			"_sep": 1,
			"external": "file:///c%3A/Users/87156/.vscode/extensions/vue.volar-2.0.7",
			"path": "/c:/Users/87156/.vscode/extensions/vue.volar-2.0.7",
			"scheme": "file"
		},
		"relativeLocation": "vue.volar-2.0.7",
		"metadata": {
			"id": "a5223b43-8621-4351-a14e-3d560f85f277",
			"publisherId": "49983e2f-38ad-4441-beea-d678b53d0549",
			"publisherDisplayName": "Vue",
			"targetPlatform": "undefined",
			"isApplicationScoped": false,
			"updated": true,
			"isPreReleaseVersion": false,
			"installedTimestamp":<PHONE_NUMBER>789,
			"preRelease": false
		}
	}
]

And I can reproduce the problem of not being able to update by overwriting the above metadata to the .vscode/extensions/extensions.json file of another macbook.
Steps to Reproduce

If Vue.volar is installed, uninstall it and then close VSCode and reopen it.
Download Vue.volar 2.0.7 from https://open-vsx.org/extension/Vue/volar/2.0.7 and install it to VSCode, VSCode can normally prompt that Vue extension needs to be updated.
Open ~/.vscode/extensions/extensions.json and modify the metadata of the last Vue.volar extension to:

{
	"identifier": {
		"id": "vue.volar",
		"uuid": "a5223b43-8621-4351-a14e-3d560f85f277"
	},
	"version": "2.0.7",
	"location": {
		"$mid": 1,
		"fsPath": "c:\\Users\\87156\\.vscode\\extensions\\vue.volar-2.0.7",
		"_sep": 1,
		"external": "file:///c%3A/Users/87156/.vscode/extensions/vue.volar-2.0.7",
		"path": "/c:/Users/87156/.vscode/extensions/vue.volar-2.0.7",
		"scheme": "file"
	},
	"relativeLocation": "vue.volar-2.0.7",
	"metadata": {
		"id": "a5223b43-8621-4351-a14e-3d560f85f277",
		"publisherId": "49983e2f-38ad-4441-beea-d678b53d0549",
		"publisherDisplayName": "Vue",
		"targetPlatform": "undefined",
		"isApplicationScoped": false,
		"updated": true,
		"isPreReleaseVersion": false,
		"installedTimestamp":<PHONE_NUMBER>789,
		"preRelease": false
	}
}

Reload VSCode will find that the Vue extension cannot detect newer versions.

VS Code version: Code 1.92.1 (eaa41d57266683296de7d118f574d0c2652e1fc4, 2024-08-07T20:16:39.455Z)
OS version: Darwin arm64 23.1.0
Modes:

cc @isidorn

I just checked our telemetry for the most popular combinations of VS Code version and Vue extension version and here's what I see. This proves that vue version 2.0.8 and 2.0.10 are indeed not getting properly updated.
The root cause of this is the fact the vue extension, unpublished from the Marketplace and published again the same name (thus exposing a bug in the Marketplace) as explained here https://github.com/vuejs/language-tools/issues/4227
We still want to help transition your users, and I will go ahead and move this issue to the VS Code repo and Sandeep will investigate next week once he is back from vacation. Thank you 🙏


Street orientations towards church steeples

Great map. I love it.
In Europe a lot of roads are oriented towards the steeples of churches that are seen from far across the country. (how far can a human see a 50m steeple in free horizon?).
Is it possible to show on a map the streets converging toward the nearest churches?

This would be interesting. Instead of grouping streets by cardinal direction they could be grouped by the angle to the church/cathedral, according to a polar coordinate system instead of a orthogonal one.
The problem is probably finding the church. Lots of towns have had tons of smaller churches that is not the center of the street network. There can also be newer ones that are more prominent today than the one in the original location. Training an AI to identify the church? Nah. Perhaps it could simply be based on the center of the view, and you manually center it on the church yourself? This would also allow for centering on other places, e.g. a start hub (or whatever they are called) in Paris or Capital Hill, Canberra, Australia.


Issue a new Release with Improvements

Thanks @najuzilu -- when you're ready it would be great to get a new release. There are some great improvements that have been merged.

Actually, would love to have one more feature before the new release.  In addition to exercise, could we please have solution?
The styling can be the same as proof.
Even more awsome would be {proof:solution:exercise_label} or some other syntax --- whatever works --- that produces
"Solution to Exercise X"
as a header and then gives the same styling as proof. :grimacing:

We'll issue a new release as soon as we merge the new solution directive. I'm very close to being done -- just finalizing tests and documentation.

v0.0.3 is out and it includes a changelog 🎉


Create a script/Concourse pipeline/other that will automatically update the Fastly origin configuration to use the backup bucket and purge the cache of the full Fastly distribution

oops


Support podman instead of docker

Hello,
Is it possible that kong-pongo support podman instead of docker ?
Best Regards

there seem to be others using podman; https://github.com/Kong/kong-pongo/pull/404
It's not likely that we'll explicitly support it in the short term, but please let us know what issues you're explicitly running into.


Fix json_matchOptionalArray tokenIdx validation

Adds missing tokenIdx test in json_matchOptionalArray. Plus, changed the test from >= to !(<), which makes them easier to comprehend (in all parser_impl functions).
Related to recently merged pull request:  https://github.com/LedgerHQ/app-flow/pull/10

Included in LedgerHQ/master. To be merged from there.


v2.0.7 contains .node_modules.ember-try folder

see<EMAIL_ADDRESS>

this was fixed in #89 but there was no release yet with this fix.
I would also appreciate 🙏 a new release with this fix. It slows down our CI build.


Prepare some interface screenshots for Eleni for marking

She mentioned this in class, it is not on the Sprint 2 Rubric but could be useful to have ready as a stretch goal

Sent


Cached and returned not updated collection

After updating entry and saving data, file contains updated data. But find functions (whole collection or by id) return outdated data.
After adding new entry. Finding by id returns ot, but getting whole collection returns data without it.

Probably, it's problem with implementation not library.


[Bug]: text not visible

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
'Add your property details' text is not visible in dark mode
Expected behavior
change color to white
Add ScreenShots

On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[ ] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Suvadip-sana
Assign me.
I would like to fix this.


Raphael.st example not working, possible bug with .forEach() method

The example at http://raphaeljs.com/reference.html#Raphael.st does not work.
It looks like the "this" ref within the forEach function is not a raphael object, which might explain why  this.red() does not run

this instead, does work,
Raphael.el.red = function () {
this.attr({fill: "#f00"});
};
Raphael.st.red = function () {
this.forEach(function (rObj) {
rObj.red();
});
};
// then use it
paper.set(paper.circle(100, 100, 20), paper.circle(110, 100, 20)).red();

this instead, does work,
Raphael.el.red = function () {
this.attr({fill: "#f00"});
};
Raphael.st.red = function () {
this.forEach(function (rObj) {
rObj.red();
});
};
// then use it
paper.set(paper.circle(100, 100, 20), paper.circle(110, 100, 20)).red();

Yep, my bad. I’ll fix the doc and will commit it with next release.


Fixed errors in notebooks

This references to #909

Changed the type of the cell in knowledge.ipynb
Fixed umbrella_prior not defined error in probability.ipynb
Minor changes in learning_apps.iypnb.

@MrDupin and @norvig, Can you take a look at this?

Nice catches! Looks good.


Markdown preview in browser window

Check for existing issues

[X] Completed

Describe the feature
I would like to have CTRL+SHIFT+P option to view created Markdown. Upon execution, ZED should parse/compile the Markdown source and display in browser of choice window.
If applicable, add mockups / screenshots to help present your vision of the feature

extra feature could allow for parsing/compiling of markdown to given platform supporter formatting such as: Vanilla Markdown, GitHub Markdown, BB Markdown etc.

This is the one feature holding me back from using Zed. I work a lot with generating technical documentation and I would love to collaborate with my technical writers and education engineers when working on content together.


Alternative project

Does anyone know any alternative projects like this because this project is outdated and it seems that the developer has lost interest in it.

I've also been following this project for months. When it was migrated to .NET Core 2.0 I thought this project was going to be very dynamic. It seemed the author was very enthusiastic and committed, but it's been months since we last heard from him. It's unfortunate because being a newbie with these patterns I was really looking forward to version 2.0 so I could learn how to properly implement ES and CQRS with .NET Core in a real-life application.  I have searched for other projects similar to this and they are either outdated or not complete. Eduardo, where are you?

@nicotravassos @jxvalenz
I don't know if he gave up the project (I do think he'd not), but seen hes facebook I can tell you that he is really busy recently. He have hes own school of programers, He was at MVP Conference this month (and probably was helping on the organization of that event in the previosly weeks). Yesterday he posted that he was named Microsoft Regional Director, what makes him even more busy.


Why white space in text field

looks not good :(

It something that is not managed by .xpi (theme file) but needs to be managed by css which isn't a part of this theme.

Without doing something akin to the "Dark Reader" browser extension, this is not possible. There are other Thunderbird addons that provide a dark mode for the content view.


Build errors due to missing files in 4.0.0-X releases

Hi, I tried to use one of the 4.0.0-RCx releases (before go v1.8, since I am not yet there) with glide. Trying version v4.0.0-rc9
- package: gopkg.in/src-d/go-git.v4
  version: v4.0.0-rc9

results in errors due to missing dirs/files:

plumbing/filemode
plumbing/cache
plumbing/format/diff

Looking at  https://github.com/src-d/go-git/tree/v4.0.0-rc9 seems to confirm this. What am I doing wrong - or are the releases incompletely tagged?
@matthiaskubik: fyi

@ruediger-maass is it possible that, somehow, you have a mixed checkout of v4.0.0-rc9 and v4.0.0-rc10 on your vendor directory? Both v4.0.0-rc9 and v4.0.0-rc10 are full releases, no missing files.
All packages you meantion are new from v4.0.0-rc10 and did not exist in v4.0.0-rc10.

Closing issue, since this seems an external problem related to how vendor/imports were setup in your project.
@ruediger-maass Please, reopen if that is not the case.

@smola - thanks your response! Have you checked your own link https://github.com/src-d/go-git/tree/v4.0.0-rc9? These are the files I get when I clone rc9, for example, and this file set is not complete and causes build errors, as mentioned above. Btw, cloning releases from your earlier gopkg.in/src-d/go-git.v3 works without problems, using the same environment (just for info, I cannot respond during the next 10 days due to vacation). Thanks for your help!

@smola : FYI, your comment about the dir's helped me to find the problem I ran into (see rc8 and rc9 below) - here's a list of the rc's - especially rc8 and rc9, explain the problem I ran into:
rc10: builds (with go v1.8) ok for me - you switched back to gopkg.in/src-d/go-git.v4 import path
rc9: you switched to srcd.works/go-git.v4 import path - I still used gopkg.in/src-d/go-git.v4
     in the code and with glide - this broke my neck and caused the symptoms I reported above,
     changing glide and code imports to srcd.works/go-git.v4 also did not work for me (couldn't set
     the version for this repo path)
rc8: same as rc9
- older rc's use gopkg.in/src-d/go-git.v4 (I know it's history, but I wanted to know :-)
rc7: I get [ERROR]	Error scanning srcd.works/go-billy.v1/os: open /home/vagrant/.glide/cache/src/https-srcd.works-go-billy.v1/os
rc6: same as rc7
rc5: ok, no build errors
rc4: ok, no build errors
...ok, stop now...


Updated folder structure in the readme

Is your feature request related to a problem? Please describe.
There is a file and folder structure in the readme. There have been some changes after that version of structure hence needs an updation

@kaiwalyakoparkar I want to work on this can you guide me ...

okk
On Wed, Jun 29, 2022 at 9:20 PM Kaiwalya Koparkar @.***>
wrote:

Closed #303
https://github.com/kaiwalyakoparkar/Full-Stack-Development-Learning-Path/issues/303
as completed via #304
https://github.com/kaiwalyakoparkar/Full-Stack-Development-Learning-Path/pull/304
.
—
Reply to this email directly, view it on GitHub
https://github.com/kaiwalyakoparkar/Full-Stack-Development-Learning-Path/issues/303#event-6903684961,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW2UZ24A2J54RV7RHOZPNH3VRRWFNANCNFSM5Z6ISMKQ
.
You are receiving this because you were mentioned.Message ID:
<kaiwalyakoparkar/Full-Stack-Development-Learning-Path/issue/303/issue_event/6903684961
@github.com>


Improve the footer design

Please assign me this issue

Please assign me this issue

I would like to work on this issue under LGMSOC21, please assign me.

I would like to work on this issue under LGMSOC21, please assign me.

How would you do so. Do gives us updates regarding that.

I want to contribute on this issue under LGMSOC.

I would like to work on this issue under LGMSOC21, please assign me to contribute
.

Duplicate #668


Expose antonmedv/expr support as a template function.

See https://github.com/antonmedv/expr .
As soon as I saw this project, I thought of using it to bring eval to hugo templates. This will allow a more expressive syntax for a lot of data manipulation tasks that are often found in hugo templates, as well as give an embedded DSL capability to hugo beyond what's offered by the template functions today.
Semantics could be something like:

   {{ $tweets := getJSON "https://x.com/..." }}

   {{ $shorties := evalexpr "all(Tweets, .Size <= 240)"  (dict "Tweets" $tweets) }}
   {{ range $shorties }}

{{ .content }}

   {{ end }}

I wonder if a feature like this belongs in hugo-extended instead of hugo? WDYT, community?

was this closed as completed? or closed as declined? sorry; been meaning to follow up with more meaningful examples but have been busy. happy to reopen a new issue when i can express the request better.

Declined to lack of follow-up.


Create PlansController

This controller was created because an endpoint is neccessary to return all recurring contribution plans.

Coverage increased (+0.02%) to 81.679% when pulling 0192ee0720dc5e382dec5fddfe17ea0c20c16390 on plans-controller into 52fb6feff7ba9c531bc9745d47ec2529da7f908c on recurring-contributions-feature.

Coverage increased (+0.02%) to 81.679% when pulling 0192ee0720dc5e382dec5fddfe17ea0c20c16390 on plans-controller into 52fb6feff7ba9c531bc9745d47ec2529da7f908c on recurring-contributions-feature.

Coverage increased (+0.02%) to 81.679% when pulling ed77c4e839c5837650c2f0ecc37760adaef470e5 on plans-controller into 52fb6feff7ba9c531bc9745d47ec2529da7f908c on recurring-contributions-feature.


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-06-24)
⚠ BREAKING CHANGES

use pronounceable name (#26)

Features

use pronounceable name (#26) (8bcfd77)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.0
:sunflower:


readme cleanup, removed -kiln and -ropsten cli args

📝 Summary

Cleaned up the readme.
Removed kiln and ropsten cli args

✅ I have run these commands

[x] make lint
[x] make test-race
[x] go mod tidy

Codecov Report

Merging #354 (a42aa16) into main (af2741f) will increase coverage by 0.44%.
The diff coverage is n/a.

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

@@            Coverage Diff             @@
##             main     #354      +/-   ##
==========================================
+ Coverage   81.44%   81.88%   +0.44%
==========================================
  Files           5        5
  Lines         679      679
==========================================
+ Hits          553      556       +3
+ Misses         96       94       -2
+ Partials       30       29       -1

Flag
Coverage Δ

unittests
81.88% <ø> (+0.44%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

server/service.go
80.00% <0.00%> (+0.74%)
:arrow_up:

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


Use Ohai to detect Linux architecture (i386/x86_64)

Description
Uses Ohai to detect architecture flavor (i386/x86_64).
Issues Resolved
Fixes #203
Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

(bug fix/no new functionality)

Works for me. I'll update the PR.
On Tue, Mar 21, 2017 at 11:41 AM, Seth Chisamore<EMAIL_ADDRESS>wrote:

@schisamo requested changes on this pull request.
In recipes/_omnibus_toolchain.rb
https://github.com/chef-cookbooks/omnibus/pull/204#discussion_r107191099
:

@@ -20,7 +20,7 @@
chef_ingredient node['omnibus']['toolchain_name'] do
version node['omnibus']['toolchain_version']
channel node['omnibus']['toolchain_channel'].to_sym

architecture(windows_arch_i386? ? 'i386' : 'x86_64')

architecture(windows_arch_i386? || node['kernel']['machine'] == 'i686' ? 'i386' : 'x86_64')

Let's just use the i386? helper from chef-sugar
https://github.com/sethvargo/chef-sugar/blob/master/lib/chef/sugar/architecture.rb
:
architecture(windows_arch_i386? || i386? ? 'i386' : 'x86_64')
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chef-cookbooks/omnibus/pull/204#pullrequestreview-28148149,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABsTv6tLMyUHV1v_96gLWzPXcf_2ai2qks5rn-_DgaJpZM4MjQ15
.

@tradel Closing this for now as your changes will be part of #206.


Prepare celery for task workers

closes #16
closes #8

broker queue / backend result choice
https://stackoverflow.com/questions/45863053/multiple-docker-containers-and-celery/45908901#45908901


[Coverity CID :174410]Memory - corruptions in /ext/hal/ti/simplelink/source/ti/drivers/net/wifi/source/wlan.c

Static code scan issues seen in File: /ext/hal/ti/simplelink/source/ti/drivers/net/wifi/source/wlan.c
Category: Memory - corruptions
Function: sl_WlanProvisioning
Component: Networking
CID: 174410
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

Duplicate of #3956


Change "year" selector on application to not use US-specific terms

At least in Canada, we generally don't use these terms to describe what grade we're in. I think it might be better just to have a more general grade selection, maybe something like:

Year/Grade:

8 or earlier
9
10
11
12

I include 8 or earlier because we're starting to get some interest from middle schools. Even if we're targeting high schools, we might as well capture these younger students early on in the process, so we can help guide them to other resources, if we feel they aren't a good fit for starting a club yet.

Agreed, and while we're at it, we may as well add options for teachers (and maybe parents).

On Feb 25, 2016, at 1:37 PM, Zach Bruggeman<EMAIL_ADDRESS>wrote:
At least in Canada, we generally don't use these terms to describe what grade we're in. I think it might be better just to have a more general grade selection, maybe something like:
Year/Grade:
8 or earlier
9
10
11
12
I include 8 or earlier because we're starting to get some interest from middle schools. Even if we're targeting high schools, we might as well capture these younger students early on in the process, so we can help guide them to other resources, if we feel they aren't a good fit for starting a club yet.
—
Reply to this email directly or view it on GitHub.


Case-specific Sorting of Strings [Competitive Programming Problem]

Is your feature request related to a problem? Please describe.
Given a string S consisting of uppercase and lowercase characters. The task is to sort uppercase and lowercase letters separately such that if the ith place in the original string had an Uppercase character then it should not have a lowercase character after being sorted and vice versa.
Describe the Test-Case
Example 1:
Input:
N = 12
S = defRTSersUXI
Output: deeIRSfrsTUX
Explanation: A sorted form of a given string with the same case of character as that in the original string is "deeIRSfrsTUX"
Describe alternatives you've considered
This code is originally mine.
Additional context
@skully-coder .Please assign this issue to me.
[x] I would like to contribute in Java

@skully-coder . Please assign this issue to me. I will contribute in Java

@skully-coder . Please review the merge request.


Project can throw when item elements are split

Exception based implementation of https://github.com/Microsoft/msbuild/issues/736

Should I also change the documentation of each exploding method to say it can now throw InvalidOperationException?

Are there any tests around the weird behavior of not expanding when there was only 1 match? Would be curious to know if anything fails if that were changed. Or if we should add one?

@AndyGerlicher
Looking at the operations that actually use the splitting method, if the element did not require splitting, then the operations apply on the actual xml of that item. So the item number condition seems more of a perf optimization: if there is only one item, don't bother splitting it out because the calling operations will operate directly on the xml,
Also, none of the item operations are glob aware, except Rename. Rename item actually checks if the new name is covered by the glob that produced the item. If it is covered, then it only updates the object and does nothing to the xml.
I will add a comment to the condition. :)

@jviau does this look OK?

@cdmihai - so this will not throw if you have <Compile Include="**\*.cs" />, only 1 .cs file covered by it  in the current evaluation and you try to perform an operation on that single item?

@jviau That's correct. Apparently the splitting code does not take globbing into account. So if it is only a glob that produced one item, then the item operations will silently operate on the glob (rename, remove, change type, etc). I'll look into adding globbing characters in splitting method's blacklist.

@cdmihai that would be preferable, we don't want to ever modify a glob from an evaluated ProjectItem, regardless of it is the only item or not produced by the glob.

@jviau My bad, I misread the code. Globs are also considered when splitting, as expected :)
I'll write more tests to make sure exceptions are thrown for them too

I made the following changes:

splitting is not conditioned on the number of items anymore
added tests to assert that the methods throw when globbing is present with either one or multiple items

Looks great to me.

@dotnet-bot test Windows_NT Build for CoreCLR please


duration constraint enabled and tests

As discussed, I enabled the duration constraint macro and used CMT6 and CMT7 to test duration constrained instances. I also used a random instance to test in test.c.

Hello @chkwon, thanks again for the great contribution, and my apologies for the delay. The pull request looks great, I will integrate it right now. In the future, I noticed that sometimes our conventions regarding the display of solution values are not consistent:

Usually, we compute solution values (and check constraints) with full precision when distances are not rounded. Only when displaying the final solution values, it is common in papers to report two significant digits. Perhaps we could make sure that a similar convention is satisfied through the code, as well as in the tests scripts. This could help avoid possible issues when validating the results, e.g., 909.675 possibly not recognized as 909.68 in asserts, etc...

Thanks. As you said, 909.675 seems like the value with full precision. But as a solver/library, I think it should return the value with full precision, not a rounded value. It must be the user's responsibility to round appropriately.
But I share your concern about testing. As you see below, I used 909.675 as a benchmark value.
https://github.com/vidalt/HGS-CVRP/blob/2dcd92763e0075e88027197be3d8c51cbd94d778/CMakeLists.txt#L45-L49

Sounds good, I agree that the solver should return full precision results.
What is the accuracy/tolerance of the value check in the test?

I used whatever the value was written in the output .sol file.  Currently, in both displaying the cost on screen and writing it on a file, it uses << operator to the stream. It has the precision of 6 (all significant digits). As a result for CMT7 it is 909.675.
https://github.com/vidalt/HGS-CVRP/blob/ed0f3507bc9fa303a84d5d10997ec677eca65a6e/Program/Individual.cpp#L68
https://github.com/vidalt/HGS-CVRP/blob/ed0f3507bc9fa303a84d5d10997ec677eca65a6e/Program/Individual.cpp#L81
The actual value of myCostSol.penalizedCost is 909.675053.


Discord issues

Here I will put all Discord issues. However, if an issue is too big I will maybe open a special issue.

[Theme] Fix theme padding (from aero15)

[Bug] Fix issue with back and forward buttons. Steps to reproduce are pinned in #bugs. (from Kaktus)

[Grammar] Add missing ")" in General settings page (from Francois)

[Bug] Fix Adblock easylist (from Jus de Sielo)

[Bug] Fix tab title on home page for some external pages (from gareth)