Managing dynamic node weights

The cost() callback lets you choose how to prioritize node weight, but is there a way to actually change those node weights dynamically?
If I were to do this:
let graph = [];

for (let x = 0; x < 100; x++) {
  for (let y = 0; y < 100; y++) {
    graph[x] = graph[x] || [];
    graph[x][y] = { weight: 1 };
  }
}

let graphOptions = {
  order: "xy",
  torus: false,
  diagonals: true,
  cutting: false,
  heuristic: "euclidian",
  cost: (a, b) => {
    return b.weight;
  },
  thread: false
};

let nodeGraph = Lowlight.Astar.Configuration(graph, graphOptions);

How would I change the weight of a random node? My intuition tells me that I should be able to do something like this:
let firstNode = nodeGraph.graphs[0].node(1);
firstNode.weight = 1;

But it doesn't seem as if the weight is accessible there.
I have noticed that if I create a node directly that I can then access the weight as I would expect, however. (new Lowlight.Astar.Node(1, { x: 1, y: 0, weight: 0 }))
How would you go about solving this?

I was able to solve my issue by editing Graph.fromArray to accept an object from the cell contents:
Original:
let node = graph.add(nodes ?  nodes.get(id(x, y)) : new Node(id(x, y)) )

New:
let node = graph.add(nodes ?  nodes.get(id(x, y)) : new Node(id(x, y), array[x][y]) )

Made the wrong assumption about how the graph initialization worked.


Update client3:akka-http-backend, ... to 3.3.16

Updates

com.softwaremill.sttp.client3:akka-http-backend
com.softwaremill.sttp.client3:core
com.softwaremill.sttp.client3:play-json

from 3.3.11 to 3.3.16.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.client3" } ]

labels: library-update, semver-patch

Superseded by #129.


release v2.0.2 on conda

@ocefpaf just a placeholder so we don't forget.  I'd be interested in learning how to do the conda releases to take the burden off you.

@Bobfrat take a look in the commits in https://github.com/conda-forge/cc-plugin-ncei-feedstock/pull/10
Usually all that is needed is to bump the version and the checksum. That one was too complex b/c I updated the dependencies and re-rendered (this last one can be done automatically).

Ok thanks @ocefpaf


Error: Timeout awaiting 'request' for 30000ms

Hi there! This happens when I execute the following command, how can I deal with it?

I'm guessing it's a network issue

Yup, happens a lot in scraping. Slow websites, slow proxies... The automatic retries usually deal with it. You can also increase the timeouts.

Oh, actually, this comes from apify-CLI. We're pulling some data from GitHub directly, so I guess your system could not connect to GitHub at that moment.


docs(www): add vite docs

Added Vite docs that use a Vite community-approved template with React, Typescript, Tailwind, Eslint, Postcss, Prefixer.
I added a vite.mdx with a similar structure to nextjs.mdx and also updated the tabs on the index page.

Hi @NikharPandya, I really appreciate your work but a PR for the vite docs has already been opened by @samuelkurdz (#714). He just needs to apply the new layout to it and it's done. Thanks anyway.

ohh my-bad i should have looked before. No worries,  Great work @samuelkurdz.
and @dan5py do we need guide for astro ?

do we need guide for astro ?

@NikharPandya  Yeah we do.

do we need guide for astro ?

@NikharPandya Yeah we do.

let me know once #714  is merged will merge mine.

@dan5py I've restructure the installation docs a bit.
@NikharPandya Yes would be awesome to have docs for Astro.

@dan5py I've restructure the installation docs a bit.

It's awesome. Just one thing the link for the manual redirects to the remix installation, the href on the index.mdx is wrong. Apart from that perfect 💯.


setup ctx of vm in associate

vm.ctx should not be nil.
Signed-off-by: Gao feng<EMAIL_ADDRESS>

it seems good in review, merge it


Implement '$setUnion' in the aggregation

I have a mongo operation that includes the $setUnion pipeline operation in an aggregation operation. I received this error:
NotImplementedError: Although '$setUnion' is a valid set operator for the aggregation pipeline, it is currently not implemented in Mongomock.

Can u implement this?

@YuvalPruss Please take a look at #510 and approve it before I merge it.

@pcorpet Seems OK 👍


Collections of answers

To be discussed next office hours.
Named collections of answers, so you can easily link to a specific updatable set of answers, without infiniscroll.
New table in coda with e.g.
https://aisafety.info/?collection=media mapping to https://aisafety.info/?state=8159_6470_2222_7619_881W_

or maybe by tag is the way to go rather than adding a named thing? Pro: Fewer parts. Con: No ability to order results.
?tag=, updating to ?tag=8159_6470_2222_7619_881W_ when something changes and retaining the non infiniscroll?


plots: make output structure consistent between plot types

Follow up to #322.
The difference in behavior between log_image and log_sklearn_plot bothers me:
├── plots
│   ├── images
│   │   ├── 0
│   │   │   └── img.png
│   │   └── 1
│   │       └── img.png
│   └── sklearn
│       └── confusion_matrix.json

Now that they both get saved under plots, it seems more odd that they have such different behavior and output structure. images create a new folder for each step, while sklearn throws an exception when used with multiple steps. #271 will only make this more complicated.
For 1.0, dvclive should be more consistently opinionated. Since the focus has shifted a bit from serverless live tracking to more of a convenience for dvc logging, I would vote to overwrite plots at each step since this is how tools like dvc plots diff, vs code, and studio expect them (this was my mistake pushing for the nested structure originally). That would mean the output would look like:
├── plots
│   ├── images
│   │   └── img.png # flatten and overwrite at each step.
│   └── sklearn
│       └── confusion_matrix.json # enable overwriting at each step.

I think it makes sense as default behavior, but doesn't really fix the use case of wanting to view the image updates across epochs (https://github.com/iterative/vscode-dvc/issues/1640).
Nested structure makes sense for that use case and anyhow if users want that structure they can manually generate it (and we can document it) as follows:
live.log_image(img, f"{live.get_step()}/img.png")

live.log_image(f"{live.get_step()}/img.png", img)

What about live.log_image(f"img/{live.get_step()}.png", img)? That gives me all steps in a single dir, which is easier for me to browse and easier for us to handle in VS Code, Studio, etc.


Multiple streams per table

Is your feature request related to a problem? Please describe.
It's currently not possible to stream the same table multiple times, because the topic is the same.
Describe the solution you'd like
We can either centralize the stream and return the existing stream on a new method call. Or (much simpler) use a more random topic, because the topic can be anything, right? So instead of using<schema>:<table> use <schema>:<table>:<incrementing_id>.
What do you think?

Yup, love the idea of taking care of this on the client side. It seems to be where people get stuck on.
I'd like to properly return the in memory cache when the user subscribers to the same table at some point. Should we implement the one where we randomly chance the topic for every stream for now for a quick fix, and revisit this issue later on?
Related https://github.com/supabase-community/supabase-flutter/issues/94

Depends on the complexity of the implementation. Do we want to reuse channels in realtime-dart or reuse streams in supabase-dart or both?

In my opinion, I think it's better to not mess with the realtime channels for now to leave the low level control up to the user. Let's try to take care of this in the stream class of supabase-dart!

Makes sense. May we continue here https://github.com/supabase-community/supabase-dart/pull/142 before that. Makes merging later easier.

Although I said the following here

In my opinion, I think it's better to not mess with the realtime channels for now to leave the low level control up to the user. Let's try to take care of this in the stream class of supabase-dart!

Implementing it would require quite a bit of rework, wouldn't it?
supabase.from('table').stream(['id']);
supabase.from('table').stream(['id']).eq('column': 'something');

We could update the code to include the filters in the topic here, but it feels like if we are doing that, why not just implement the idea proposed here by @Vinzent03 to add a increment id in the topoic. This would solve the above issue as well as allow users to listen to the same stream multiple times. Sorry for changing my opinion, but let's implement this for now for a quick fix!
schema:table:incrementing_id

supabase_flutter 1.2.2
Stream<List> _messagesStream = client
.from(Constants.userActionsDbName)
.stream(primaryKey: ['id'])
.order('id', ascending: false)
.eq('sent_to_uid', CurrentUser().uid)
.map((maps) => maps.map((map) {
return UserActions.fromMap(map: map);
}).toList());
Stream<List> _messagesStream2 = client
.from(Constants.userActionsDbName)
.stream(primaryKey: ['id'])
.order('id', ascending: false)
.eq('sent_from_uid', CurrentUser().uid)
.map((maps) => maps.map((map) {
return UserActions.fromMap(map: map);
}).toList());
the (eq) is different.
With two streams on the same table - real time changes stop. What am I doing wrong?

@sunilguptasg
Your code looks fine, and I cannot seem to be able to reproduce your issue.
Could you double check that realtime updates are coming through when listening to only one stream?

Yes, one stream works.
I have created a duplicate table and listen that that for the second stream, for now, I know this is not the best way...


Adding Divyanshu Agrawal as a reviewer

/assign @divyansh42
for lgtm
/label docs-approved
/label px-approved
/label qe-approved

/assign @divyansh42 @apoorvajagtap @mbharatk
for lgtm

/lgtm

/label docs-approved
/label px-approved
/label qe-approved


Another Win attempt - please ignore

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

To summarize the status:

I am using conda's mingw, which has autotools and all the goodies too.
Package is built but it is missing a couple of dependencies (zlib and gcc stdlib of all things).
No MPI.
A couple of trivial patches are needed for openmp (one already in). The reason is that under w64 a pointer is not a unsigned long (but a unsigned long-long, if one really has to be explicit).
Building (needed for debugging deps) is very time consuming.
Overall it seems to be a waste of energy (both human and electricitly for CI...): who seriously needs this when WSL is available?

@GiovanniBussi So here's the status of this attempt.

The package builds (had to disable some tests)
Plumed driver seems mostly ok (130/216 basic regtests pass, some are ~eps, some may indicate deeper problems). E.g. time, despite being a float, seems to wrap around between 10e6 and 15e6 (basic/rt76)
No mpi, openmp, blas, etc.
No python
Other features (e.g. patch, scripts...) rely on unixy assumptions which  may be hard to port without changes
Some general dirtiness due to the fact that m2 system translates paths in a way that conda does not  translate

Honestly not sure if this is worth merging. At least I don't think it breaks anything.

@tonigi this is super useful! I was wondering:

where can I see the results of the failing tests (without buying a Windows computer ;-) ). Is there a CI showing them? even if we don't merge this, it's always useful to improve portability (e.g., by the two small patches that I included already).
do you know where I could do tests in a pure CI framework? I would be happy to make an effort to have the scripts working, if there is a way. Maybe there is just some smart way to manipulate paths from bash
alternatively, do you think I could tests these executable in some way on a MacOS (intel) laptop? Maybe with wine?

E.g. time, despite being a float, seems to wrap around between 10e6 and 15e6 (basic/rt76)

For instance, this is interesting. Currently, in driver, step is a long int. And it should be, since cmd("setLongStep") accepts a long int. But, on Windows, long int is 32 bits only.
I think we should add cmd("setStepLongLong") which would enable passing a long long int, and use a long long int in driver.

Hi all,
I can try to arrange a windows-specific CI with appveyor, so it does not waste CI time here. Appveyor also has a nice option to connect interactively.  It's a okish alternative to have a local Windows VM (max 1h). Note that building the packages is very slow, but installing it from conda and then running regtests is tolerable.
The msys2 win/unix path duality is related to translations here... quite messy to handle. On the other hand, m2-* packages exist for all the unix tools needed.

By the way, package here   https://anaconda.org/tonigi/plumed/files
conda install -c tonigi plumed

@tonigi it would be great if this can then be done automatically at every plumed commit! Is there a way? I mean: could we directly build on Windows and run its tests without explicitly using conda?
Otherwise, a possible strategy it to just try to fix apparent issues on master branch and remember to test conda releases in the future.

I suppose so... a couple of questions though:

do you want to use the main plumed2 repo, or a separate one?
do you want to run the build only, or also the regtests?
what if the test fails? ignore, i suppose?

In an ideal world:

in the main plumed repository, at every commit, starting with branch 2.8
build + tests
now, only fails if build fails. After I have fixed all tests (I can do it
if we have a working CI, thought it might take some time), fails if test
fails

I would be happy both with appveyor or with GitHub actions + minGW + wine
(if feasible), or even with conda (if the only solution, since I guess it
is much slower)
This would be the most robust solution if then we want to maintain the
Windows build!
Anyway, any other solution would be fine if it’s simpler. I guess that once
you make a working repository I can make the effort to integrate it in be
official one.
Thanks a lot again!!
Giovanni
Il giorno lun 29 ago 2022 alle 17:10 Toni G @.***> ha
scritto:

I suppose so... a couple of questions though:

do you want to use the main plumed2 repo, or a separate one?
do you want to run the build only, or also the regtests?
what if the test fails? ignore, i suppose?

—
Reply to this email directly, view it on GitHub
https://github.com/conda-forge/plumed-feedstock/pull/31#issuecomment-1230451550,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBNCXVOEDGCNX372FA5WSTV3THGHANCNFSM57JOYEVQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Sent from Gmail mobile

This may be part of the solution?
It installs a build package and runs regtests . Results can be retrieved as an "artifact" or played with interactively.
For building at every commit, one can use something   similar (uncomment the build statements)  after making a recipe which gets from plumed2's master. The recipe should be kept in sync with the one in the feedstock.  (I hope not to be confusing)


Getting "fatal" result

Hi, thanks for your useful action for *.pot files generation!
I tried to use your action yesterday to generate a pot file for a plugin. I applied the same settings as before for the previous plugin. But the file after generation is not written to my repository. Is this something on your side or should I change the launch of the action in my yml file?
There are the logs for the previous using (2022-02-07) and yesterday (22-04-17)
log 2022-04-17.txt
log 2022-02-07.txt
Best Regards!

I'm also getting the same error on a clean repo after following the instructions in README.md.

fatal: unsafe repository ('/github/workspace' is owned by someone else)

Hi @nikitasinelnikov @odil-io,
Thanks for reporting the issue, it seems @odil-io already figured out the issue and fix for the same. I will review the PR soon.
Best Regards!!

Fixed in v1.0.4
@odil-io Thanks a lot for the quick fix this.

Hi @odil-io and @iamdharmesh
Thank you guys for the quick fix.
But I have another issue:
https://www.screencast.com/t/uZJnY3VAN5Fc
I don't know why it's the wrong URL because it's the proper repo's URL https://github.com/ultimatemember/recaptcha
YML file is here: https://github.com/ultimatemember/recaptcha/blob/master/.github/workflows/translations.yml
Secure token exists: https://www.screencast.com/t/b0gjAzPLxtRO
Let me know if you have thoughts or ideas on how to fix that,
Best Regards!

Hi @nikitasinelnikov,
This is a strange issue. Can you please make sure that the TOKEN created by you has enough access to push code to the repo?
Thanks.

Let me know if you have thoughts or ideas on how to fix that,

For as far I can tell, it seems that your token does not have enough rights as everything else is correct.
Also, GITHUB_TOKEN is a standard variable built for this purpose. I see you've made you own token and I'm nearly sure that is where your problem lies in this case.
In short:
In translations.yml#L20 change:
GITHUB_TOKEN: ${{ secrets.GH_POT }}

To
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

And that should fix your problem. If not checkout the GitHub Docs linked above.


4.0 | More consistent type handling of ruleset-set custom properties

Current Situation
Follow up on/Inspired by #704.
End-users can set the values for public properties on sniffs via the XML ruleset.
PHPCS receives these values as strings and includes special handling for a few situations:

Value in XML ruleset
Value set for the sniff property

'' (empty string)
null (null)

'null'
null (null)

'false'
false (boolean)

'true'
true (boolean)

As things are, this special handling:

Is not being applied when 'null', 'true' or 'false' are not received in lowercase.
I.e. a property value of 'False' will remain the string 'False'.
Is not applied to the values of array elements being set on an array property.

It is also not applied to the value of array keys.
Proposal
Proposal - part 1
I'd like to propose to apply the special handling of property values more consistently.

Value in XML ruleset
Value set for the sniff property in PHPCS 3.x
Value set for the sniff property as of PHPCS 4.0

'' (empty string)
null (null)
null (null)

'null'
null (null)
null (null)

'false'
false (boolean)
false (boolean)

'true'
true (boolean)
true (boolean)

'NULL' (and all other case variations)
'NULL' (string)
null (null)

'FALSE' (and all other case variations)
'FALSE' (string)
false (boolean)

'TRUE' (and all other case variations)
'TRUE' (string)
true (boolean)

Proposal - part 2
I'd also like to propose to apply the same special handling to array element values as of PHPCS 4.0.
What will not change ?
I do NOT intend to change the handling of array keys. These will remain strings at all times.
Changing that could have far bigger breaking impact due to how PHP casts all array keys to integers/strings and duplicate keys are not allowed.
I also do NOT intend to add special handling for integer or float property values at this time.
Impact and timeline
Sniffs which explicitly expect string values for all array elements in a public property may break due to this change if the sniff only takes string values for array elements into account.
Sniffs may also contain special handling to work around the above listed short-comings of the property handling. Especially for array values, where PHPCS currently has no special handling, I can imagine that this sniff-specific special handling could break with this change.
So, with the above in mind, I'm proposing to make this change in the 4.0 release and not before.
Opinions ?
Am I overlooking anything ? Are there more things which could break with this change ? Or do you think this will more likely fix some obscure bugs for end-users ?
Please leave a comment with your thoughts.
/cc @asispts @dingo-d @fredden @GaryJones @greg0ire @kukulich @michalbundyra @Ocramius @sirbrillig @stronk7 @weierophinney @wimg

What's the audience for part 1? People who mistakenly type False, or people who expect it to work? I think it wouldn't cross my mind to do that, so I'm a bit skeptical about part 1.
For part 2 however, I think it is a good idea.
By the way, is there currently a syntax that allows escaping that special handling? If there is, then it would be possible to warn users about this, and recommend them to migrate to the escaping syntax ahead of time.

What's the audience for part 1? People who mistakenly type False, or people who expect it to work? I think it wouldn't cross my mind to do that, so I'm a bit skeptical about part 1.

Both. While it may be uncommon, there are people who are used to using uppercase null/true/false everywhere (there's even a sniff to enforce it, though I'm not sure how much it is used nowadays, what with PSR2/12 recommending lowercase).
I can't remember having seen bug reports about this, but why wait for one ? More than anything, this is about the "principle of least surprise", why would one work while the other doesn't ?
If given the choice between documenting that only lowercase values work or fixing this, I think fixing this is the better option and will lead to fewer support requests.

For part 2 however, I think it is a good idea.

:eyes:

By the way, is there currently a syntax that allows escaping that special handling? If there is, then it would be possible to warn users about this, and recommend them to migrate to the escaping syntax ahead of time.

There is no escaping syntax available and never has been AFAICS.

While it may be uncommon, there are people who are used to using uppercase null/true/false everywhere

Ok, I didn't know about that, but just because I haven't encountered them does not mean they don't exist I suppose. In that case, it seems fair to fix rather than document indeed 👍


Fix responsive layout on test run info page (scroll issue)

See scroll bars on tablet/mobile view on the page

fixed


chore(main): release 1.1.2

:robot: I have created a release beep boop
1.1.2 (2024-08-06)
Bug Fixes

remove demo columns and allow additional properties (#20) (ebc39ea)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.1.2
:sunflower:


Re: Minecraft plugin - v5 and jetty https2 SSL -> failed to find class org/conscrypt/CryptoUpcalls

@tipsy @zugazagoitia @Playacem
This is just an update about the problem I had wich turn to be my mistake....
Finnaly found what the problem was -__-
There's no obligation for me to add libraries entry in my plugin.yml so that the server install those libs.
I've deleted everything in the server/libraries dir to be sure (or are they cached ?)
e.g.:
libraries:
  - "io.javalin:javalin:5.1.3"
  - "org.slf4j:slf4j-simple:1.7.36"
  - "com.fasterxml.jackson.core:jackson-databind:2.13.2"

I think my mistake was in my build file i've changed:
set BUILDER="C:\projects\Whitelist-Je-Node"
call mvn clean install package -f %BUILDER%

to:
@echo off
cls
...

call mvn clean
call mvn install
call mvn package

Everything works after this only change + the SSLPlugin provider by @zugazagoitia
Looks like -f or -U where not params that I should have used for my build
Although while researching I've used a java decompiler but in both case the Conscrypt class was present and at the sam level:

Another question:
Since i'm here I could do my own research on keystrore,jks but do you guys have  any recommandation should I let it in the jar ressources ? should I put it in the plugin data folder aside of the user config.yml for the plugin and load it from there ? Should I use .pem instead ?
Here example of code:
    public PluginEntrypoint(@NotNull JavaPluginLoader loader, @NotNull PluginDescriptionFile description,
            @NotNull File dataFolder, @NotNull File file) {
        super(loader, description, dataFolder, file);
    }

    @Override
    public void onEnable() {

        ITransaction transaction = Sentry.startTransaction("onEnable", "configurePlugin");
        try {
            FileConfiguration bukkitConfig = getConfig();
            instance = this;

            // Temporarily switch the plugin classloader to load Javalin.
            ClassLoader classLoader = Thread.currentThread().getContextClassLoader();
            Thread.currentThread().setContextClassLoader(this.getClassLoader());
            // BungeeCord:
            // Thread.currentThread().setContextClassLoader(this.getClass().getClassLoader());

            if (app == null) {
                // Create a Javalin instance.
                app = javalinServer();
            }

            app.start(bukkitConfig.getInt("port", 4567));

            // Restore default loader.
            Thread.currentThread().setContextClassLoader(classLoader);

            logger.info("Javalin server is running...");

            configManager = new ConfigManager();
            sentryService = new SentryService(this);
            transaction = sentryService.createTx("onEnable", "configurePlugin");

            daoManager = new DaoManager(configManager, this);
            discordManager = new DiscordManager(this);
            guildManager = new GuildManager(discordManager.getGuild(), this);
            bukkitManager = new BukkitManager(this);

            updateAllPlayers();

            Logger.getLogger("WhiteList-Je").info(this.getfiglet());
            guildManager.getAdminChannel()
                    .sendMessage("**Le plugin `" + this.getName() + "` est loader**\n\n" + getPluginInfos(false))
                    .queue();

        } catch (Exception e) {
            try {
                transaction.setThrowable(e);
                transaction.setStatus(SpanStatus.INTERNAL_ERROR);
                SentryService.captureEx(e);
                guildManager.getAdminChannel().sendMessage("**`OUPS`, Le plugin `" + this.getName() + "`" +
                        " a rencontré des `problèmes` à l'initialisation**\n" +
                        "**Regarder les fichers de `logger` !!!!**\n\n" +
                        getPluginInfos(false)).queue();

            } catch (Exception err) {
                transaction.setThrowable(err);
                transaction.setStatus(SpanStatus.INTERNAL_ERROR);
                SentryService.captureEx(err);
            }
        }

        transaction.setStatus(SpanStatus.OK);
        transaction.finish();
    }

        private Javalin javalinServer() {
        try {
            // Tell bStats what plugin this is
            new Metrics(this, 9492);

            // Initialize config file + set defaults
            saveDefaultConfig();
            FileConfiguration bukkitConfig = getConfig();

            // Initialize any economy integration (if one is installed)
            EconomyWrapper.getInstance().setupEconomy();

            this.authEnabled = bukkitConfig.getBoolean("useKeyAuth", true);

            // Warn about default auth key
            if (this.authEnabled) {
                if (bukkitConfig.getString("key", "change_me").equals("change_me")) {
                    logger.warning("AUTH KEY IS SET TO DEFAULT \"change_me\"");
                    logger.warning("CHANGE THE key IN THE config.yml FILE");
                    logger.warning("FAILURE TO CHANGE THE KEY MAY RESULT IN SERVER COMPROMISE");
                }
            }

            maxConsoleBufferSize = bukkitConfig.getInt("websocketConsoleBuffer");
            Bukkit.getScheduler().runTaskTimer(this, new Lag(), 100, 1);

            Javalin app = Javalin.create(config -> {

                config.jsonMapper(new GsonJsonMapper());
                config.showJavalinBanner = false;

                // unpack the list of strings into varargs
                List<String> corsOrigins = bukkitConfig.getStringList("corsOrigins");
                String[] originArray = new String[corsOrigins.size()];
                for (int i = 0; i < originArray.length; i++) {
                    logger.info(String.format("Enabling CORS for %s", corsOrigins.get(i)));
                    originArray[i] = corsOrigins.get(i);
                }

                // Create an accessManager to verify the path is a swagger call, or has the
                // correct authentication
                config.accessManager((handler, ctx, permittedRoles) -> {
                    String path = ctx.req().getPathInfo();
                    String[] noAuthPaths = new String[] { "/docs", "/swagger-docs", "/swagger-ui" };
                    List<String> noAuthPathsList = Arrays.asList(noAuthPaths);

                    // If the request is for an excluded path, or the user has auth turned off, just
                    // serve the req
                    if (noAuthPathsList.contains(path) || !this.authEnabled) {
                        handler.handle(ctx);
                        return;
                    }

                    final String KEY_HEADER = "key";
                    final String KEY_COOKIE = "x-wje-node-key";

                    // Auth is turned on, make sure there is a header called "key"
                    String authKey = bukkitConfig.getString("key", "change_me");
                    if (ctx.header(KEY_HEADER) != null
                            && ctx.header(KEY_HEADER).equals(authKey)) {
                        handler.handle(ctx);
                        return;
                    }

                    // If the request is still not handled, check for a cookie (websockets use
                    // cookies for auth)
                    if (ctx.cookie(KEY_COOKIE) != null
                            && ctx.cookie(KEY_COOKIE).equals(authKey)) {
                        handler.handle(ctx);
                        return;
                    }

                    // fall through, failsafe
                    ctx.status(401).result("Unauthorized key, reference the key existing in config.yml");
                });

                config.plugins.register(sslPlugin());
                config.plugins.register(new OpenApiPlugin(getOpenApiConf()));
                config.plugins.register(new SwaggerPlugin(getSwagerConf()));
            });

            if (bukkitConfig.getBoolean("debug")) {
                app.before(ctx -> logger.info(ctx.req().getPathInfo()));
            }

            apiRoutes();
            getServer().getPluginManager().registerEvents(new WebhookEventListener(this), this);
            return app;

        } catch (Exception e) {
            e.printStackTrace();
            SentryService.captureEx(e);
            return null;
        }
    }

I believe you could just run mvn clean install and that would be enough, there is no need to run the command multiple times, maven executes each goal sequentially. (And also package is a goal of Install)
Regarding the PEM/Keystore issue, it is up to you. I, and probably everyone else, personally hate JKS, since all my keys are PEM encoded. One of the purposes of the SSL plugin is easy PEM key loading, but don't bother to convert keys if you already have a Keystore since they do the same Job.
If you're going to distribute your code it would be good practice to keep the key only in the server, as well as any other private configuration. In other cases it wouldn't really matter, that's also up to you.

Since this is just a question, I'll close it. Feel free to continue the discussion though.


Add legacy config to clear active Spark 3.1.0 session in tests

Fixes #967.
Spark 3.1.0 recently disallowed clearing the session by default which causes unit tests to fail.  Until #969 is addressed, this is a workaround that uses the legacy config to allow the session to be cleared as it was before.

build


Add command to toggle chatbox

I love the feature where the Minimap closes when the Chatbox opens. What would be even more neat would be to have a way to open and close the chatbox without using your mouse, for example with "ALT+Z" Also allow users to change the command to any key command they prefer to use.

This can be achieved with the Toggle Chat plugin.


python 3.5 issues

There is already #10, I couldn't re-open, so I filed a new ticket instead.
When running on python 3.5 I get errors
(v) harm@sensor1:~/uwsgi-sloth$ uwsgi-sloth start -c josene.conf
[2019-02-28 10:53:29,072] uwsgi_sloth INFO: Start from last savepoint, last_log_datetime: 2019-02-28 10:48:29.071966
[2019-02-28 10:53:35,884] uwsgi_sloth INFO: Rendering HTML file /home/harm/uwsgi-sloth/html/html/latest_5mins.html...
[2019-02-28 10:53:35,916] uwsgi_sloth INFO: Rendering HTML file /home/harm/uwsgi-sloth/html/html/day_2019-02-28.html...
Traceback (most recent call last):
  File "/home/harm/uwsgi-sloth/v/bin/uwsgi-sloth", line 32, in <module>
    main()
  File "/home/harm/uwsgi-sloth/v/bin/uwsgi-sloth", line 25, in main
    args.func(args)
  File "/home/harm/uwsgi-sloth/v/lib/python3.5/site-packages/uwsgi_sloth/commands/start.py", line 118, in start
    day_requests_data.save()
  File "/home/harm/uwsgi-sloth/v/lib/python3.5/site-packages/uwsgi_sloth/models.py", line 48, in save
    pickle.dump(self.data, fp)
TypeError: write() argument must be str, not bytes

When changing models.py::RequestsData  to open() as  'rb' and 'wb' this crash disappears, but then I get
[2019-02-28 10:56:56,919] uwsgi_sloth INFO: Start from last savepoint, last_log_datetime: 2019-02-28 10:51:56.919301
[2019-02-28 10:57:03,777] uwsgi_sloth INFO: Rendering HTML file /home/harm/uwsgi-sloth/html/html/latest_5mins.html...
[2019-02-28 10:57:03,812] uwsgi_sloth INFO: Rendering HTML file /home/harm/uwsgi-sloth/html/html/day_2019-02-28.html...
Traceback (most recent call last):
  File "/home/harm/uwsgi-sloth/v/bin/uwsgi-sloth", line 32, in <module>
    main()
  File "/home/harm/uwsgi-sloth/v/bin/uwsgi-sloth", line 25, in main
    args.func(args)
  File "/home/harm/uwsgi-sloth/v/lib/python3.5/site-packages/uwsgi_sloth/commands/start.py", line 111, in start
    for date in analyzer.data.keys():
RuntimeError: dictionary changed size during iteration

last crash might not be a python3.5 issue, but first crash is I think.
version
uwsgi-sloth==3.0.1

Hi, @harmv sorry for the inconvenience, I just release a new version 3.0.2 to fix the problem you reported. Please give it a try.
This project was not very active for a long time and definitely needs more unit tests, if you encounter any other problems, please let me know.
Thank you.


localhost:2222/metrics prepopulate with some keptn_ metrics

During initial installation, users can port forward to localhost:2222/metrics and see the Prometheus metrics page.
However, this page only shows the "default" go_ metrics (before a KeptnApp deployment has occurred.
This causes user confusion: Am I looking at the correct thing? Is Keptn working properly?
My suggestion would be to add a new metric with the prefix keptn_ which signals that KLT is indeed installed correctly and waiting.
Docs can always make this clear that new and additional metrics will flow in after the first deployment.
TLDR: Add a new keptn_active=1 (or equivalent) metric which appears OOTB before any application deployments.
DoD

[ ] have a new metric called keptn_lifecycle_active in the lifecycle-operator that is set to 1 as soon as the operator starts
[ ] have a new metric called keptn_metric_active in the metrics-operator that is set to 1 as soon as the operator starts
[ ] make a kuttl test with a job that checks the localhost:2222/metrics endpoint has the new metrics,  you can take an example  from this test

AC

[ ] integration tests are green
[ ] the example exercise shows the metrics on localhost:2222/metrics

thanks @geoffrey1330 I will assign you !


update rgb paint color calculations

Math.Round() was causing the rgb values to be rounded to 0 or 1

Hey, thank you for the contribution @gavinanthony!
What was the issue exactly? Could you maybe post an example of the RGB code and the inaccurate output?
I added the Math.round because the Figma numbers return crazy long decimals. And this other plugin that does a similar thing, so I was like "cool seems right?" 😁
So I'm definitely not married to it, I just want to know the issue it causes to understand it better. Definitely my fault for not having tests in place that would just verify this automatically, sorry about that 😅

Just tested this PR and noticed it fixed colors on import. Thanks for the contribution!
I'll note your name in the README for now, but I'll try to install the Github bot for contributors later to automate the process 👍

@all-contributors please add @gavinanthony for code, bug


[VL] Limit the number of compile jobs in builddeps-veloxbe.sh

Description
It is better to limit the number of compile jobs in builddeps-veloxbe.sh
https://github.com/oap-project/gluten/blob/ed843672bc1223aa2dbe44281d51db5906740882/dev/builddeps-veloxbe.sh#L159
Just like compile.sh
https://github.com/oap-project/gluten/blob/ed843672bc1223aa2dbe44281d51db5906740882/cpp/compile.sh#L146
Because compile broke caused by many jobs when exec ./dev/builddeps-veloxbe.sh --build_tests=ON --build_benchmarks=ON --enable_s3=ON --enable_hdfs=ON.

similar with #4123 so close this


Add skip and query parameters to MaverickApi methods

MaverickApi.createQuery and MaverickApi.createSubscription need to support smartQuery skip and query parameters.

Aargh - variables parameter, not query.  2 hours of life I'll not get back..


add dynamic types for sizes

Hi, I've added typings for all sizes inside a project. It should be useful coz lots of components have different sizes, and it is always a bummer when u need to check docs. I was inspired by this issue (https://github.com/nuxtlabs/ui/issues/442).

Will look into this after #588 is merged!

Yea, that would be great, it is something like Proof Of Concept for now, but I can expand it into the rest of the all props

ok, no problem I just hope to Nuxt/UI achive better typings :)


add golang open sourch books

What does this PR do?
Add resource(s)
For resources
Add open-source Golang Bangla book.
it's an open-source Golang Bangla book.
How do we know it's really free?
Because it's open-source
For book lists, is it a book? For course lists, is it a course? etc.
Yes
Checklist:

[X] Read our contributing guidelines
[X] Search for duplicates.
[X] Include author(s) and platform where appropriate.
[X] Put lists in alphabetical order, correct spacing.
[X] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Duplicate of #6710


Issue nesting second dropdown menu for dropdown menu item "themes"

This is what currently happens when trying to access "themes" dropdown menu. Code has been copy pasted from parent dropdown menu and changed where necessary.

Ended up changing planned layout, and instead of a nested dropdown menu, I added a nested list and used "Themes" as the list header.  It fixed my issue and I believe the UX gets better this way too, since there will be less clicks to get to the desired content.


Replace P Monad with transformer stack

There's really only one purpose for this, which is to embed a "Context" for the parsing into the return type, which indicates which subcommand the parser failed inside, so a better help text can be printed.

Stack looks right. I am a bit meh on stacks that deep in scala, because there are some walls where you will find it won't work any more (for example I don't know if you will be able to implement MonadTrans and use liftM generically in this situation). But if it works for now I am fine with it.

One note though, I think we really need to start adding a much more comprehensive test suite before the internals get any more churn.


ci: add default timeouts to tests

TestExpectaions may be wrong now that we we may Fail rather expect Mocha Timeout.
So we may need to handle that in the test runner.

We would need a solution for when we set the timeout inside the test themself,
Maybe we can extend the original this.timeout to do this as well.


Argument grouping bug

Grouping error
I am currently trying this on Windows 11 and noticed, that  '  is not working as it shut, grouping spaced commands into a singular argument.
Example:
cargo watch -x 'build --release'

Results in:
error: Found argument '--release'' which wasn't expected, or isn't valid in this context

USAGE:
    cargo watch [FLAGS] [OPTIONS]

For more information try --help

Am I using the tool wrong, or is this assumption correct, that this is a bug?
Kind regards Jakob

You are not technically using the tool wrong, and this is not a bug! This is, most likely, shell quoting issues. See e.g. https://superuser.com/questions/324278/how-to-make-windows-command-prompt-treat-single-quote-as-though-it-is-a-double-q


Disable "declared but its value is never read" and other linter rules

Is your feature request related to a problem? Please describe.
Super annoying when I delete some code and then the linter pipes up with this non-sense.
Describe the solution you'd like
yarn start shouldn't complain about non-fatal errors/warnings. yarn build can. It's best to fix these things before going to production but there's no need to interrupt my development.
Describe alternatives you've considered
Don't know of a workaround.
Additional context

I'm using the TypeScript template. Not sure if this happens in the JS version.

Nevermind, found the option. Just add {typeCheck: false} to the plugin. My apologies.


Revert "[progressbar] Add progressbar from conda/conda-recipes"

Reverts omnia-md/conda-recipes#396

I assume there's probably another well-maintained progressbar popular library, since it's used in many apps.

Thanks! And agree.


Linux Keystroke Limiter Support

Keystroke limiter is not working on Linux (With Async)
This issue does not exist when not using async
System: Manjaro
Game Version: 2.7.0 & 2.8.0

I think this is the same issue as #34. Can you try making your sync and async KeyLimiter settings the same?

I'm sure that my async and sync setting are the same, You know, when I change to async, the display of the keylimiter settings will turn to Key name and The keycode will be display in parentheses，but now, the display isn't change. The display is still the same as the sync display
later I will upload a video.Forgive my bad English.

Uploading 2024-08-03 14-24-06.mp4…

https://github.com/user-attachments/assets/2aa2f824-4e26-4e1f-9457-1cc839b6f3fd

Hmm, can you upload your Player.log? I think Linux might load async differently from Windows, which is why it's not working properly.

Player.log

and My game will crush when I use adofai tweaks and "together" mod

Log.txt
this is umm log


Support for idempotent operations

Would it be possible to add option-based support to make LXD operations idempotent?
For example, lxc start throws an error if a container is already started, and lxc delete fails
if the target container is already deleted. As a user I don't generally care whether a container was
actually started / deleted; I'm simply trying to get the container into a certain state. The non-idempotency
of the operations means I have to write a lot of repeated if { } else { } style code to handle each case.
It's painful having to wrap each command to make it idempotent, and dropping error-codes etc. requires more boilerplate and worsens error-handling.

+1

I think I responded to a similar question before. The conclusion was that I'd be happy to review a branch which either introduces a different return code in such case (0 = success, 1 = failure, 2 = already in requested state) or a flag to turn the client into such mode (either a global command line argument or entry in config.yml).
You may find implementing this a bit challenging though as most of those errors right now are returned by the API directly and just displayed by the lxc command line tool. That is, the client doesn't necessarily know how to differentiate between an error and something already being in the expected state.
Changing the API behavior itself wouldn't be acceptable, so you'd need to add extra logic on the client side to handle this, making sure not to make the normal case any slower in the process (only query actual state if in idempotent mode).

Indeed, we don't want to do string matching on things like that but could make better use of some HTTP codes to indicate that something is already in the desired state.


React Native Version Bugs

Since introducing app templates, the react-native version inside package.json is blank.
We also lost the ability to pass through --react-native-version.

Fixed in #718


End-to-end test: Additional container config and enable more test to run in compat mode

Change the API to make/load container from passing in the registry to a config object that contains the registry.
Add runtimeOptions to configure the containe runtime to the config object.
Move testFluidDataObject configuration to the container config instead of the local test object provider config.
Enable compat test for sharedInterval and fluidObjecthandle.
blobs test is a new feature, so compat mode is not enabled, but converted to be easily enabled when N-2 is 0.28.

Change the API to make/load container from passing in the registry to a config object that contains the registry.
Add runtimeOptions to configure the containe runtime to the config object.
Move testFluidDataObject configuration to the container config instead of the local test object provider config.
Enable compat test for sharedInterval and fluidObjecthandle.
blobs test is a new feature, so compat mode is not enabled, but converted to be easily enabled when N-2 is 0.28.


Last PR before the v0.5.3 release

Updated CHANGELOG.rst
Updated the version number in __init__.py and setup.py
Updated README.rst

Codecov Report

Merging #479 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #479   +/-   ##
=======================================
  Coverage   98.74%   98.74%
=======================================
  Files           5        5
  Lines         239      239
=======================================
  Hits          236      236
  Misses          3        3


There's a blank space in the menus in the top in macOS

Issue Report Checklist

[ ] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[ ] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
In the current master, when navigating through the menus a blank entry is shown

Maybe this is being caused by the changes in PR #14619 . Probably the space is the initial 'empty' action to enable the addition of the menu to the menuBar on Mac since the menus are populated dynamically in the aboutToShow signal

Maybe this is being caused by the changes in PR #14619 . Probably the space is the initial 'empty' action to enable the addition of the menu to the menuBar on Mac since the menus are populated dynamically in the aboutToShow signal

Seems like this happens only for non-migrated menus (currently only Views and Help are migrated).
@ccordoba12 should I add handling for non-migrated and migrated menus to prevent this (basically some kind of flag when creating the menu to indicate if the initial action is needed or not) or just wait to have all the menus migrated?

Seems like this happens only for non-migrated menus (currently only Views and Help are migrated).
@ccordoba12 should I add handling for non-migrated and migrated menus to prevent this (basically some kind of flag when creating the menu to indicate if the initial action is needed or not) or just wait to have all the menus migrated?

should I add handling for non-migrated and migrated menus to prevent this (basically some kind of flag when creating the menu to indicate if the initial action is needed or not)

Yes, please do that so we can have a uniform look and feel for menus until all of them are migrated.

should I add handling for non-migrated and migrated menus to prevent this (basically some kind of flag when creating the menu to indicate if the initial action is needed or not)

Yes, please do that so we can have a uniform look and feel for menus until all of them are migrated.


Safe dispose, nested tidy, and tidyExclude

https://github.com/facebookresearch/shumai/issues/143

Let me know if I can do anything to help get this merged


Fix DANDI Upload URL

This PR fixes an error when staging=False in the dandi upload utility. The existing code resulted in a URL that is just the dandiset_id rather than the full URL, resulting in an error.
PR Summary:

[x] Fix DANDI upload URL
[x] Test the fix locally
[x] Update CHANGELOG
[x] Update version.py

Version 0.5.4 is tagged and released on GitHub, published to PyPI, and docs are deployed.


Feature request: interpret ^D as quit

It seems kanban expects ^C for quitting.
Usually on Linux, ^D (EOF) is regarded as a signal to stop reading from standard input. When giving kanban a ^D at the “Choose wisely” prompt, kanban dies with an EOFError.
I suggest treating ^C and ^D as synonyms.
(And perhaps allow q for quitting, too.)

So https://github.com/Zaloog/kanban-python/blob/2e652435ce618529b1201cec75ae98c1a8351e2d/src/kanban_python/app.py#L113-L114
This should just be a change to:
except (KeyboardInterrupt, EOFError):
    utils.console.print(utils.get_motivational_quote())

To make it quitable with ^D right?
q is not that simple, it needs more dependencies and some of those are platform specific.
Also I cant have it work on the overall app level, but just on specific inputs, cause it shouldnt abort the process when you enter a new task with q in its Title/Description/Tag or create a new board.
I think 2 options with ^C and ^D are enough for now.

Perfect! :+1:


Expose addresses on npm

Summary

Split addresses pkg in two, one for public address other for tooling
Publish public addresses

Can we expose a getAddress util in the addresses package? It's a much better dev experience since it's typed.

@lucas-janon updated and updated all the other scripts to use from one pkg or the other


Transition to official TT API

Now that TT has released their official API, it's a bit easier to implement the changes and no longer requires a reverse-engineering setup.
All of this project remains relevant and useful towards a Python implementation of the official API.
There's already another person out there making a Python implementation, I'm not sure if he's aware of this project but I reached out to him to see if we can collaborate since there's no need to start from scratch.

Do you have a link to the official API I cannot seem to find it on their webpage

Yes! It's a bit buried. First you have to enable it on the main page, log in then go to My Profile > Developer.
From there you can log in at developer.tastytade.com

Closing as the transition is underway!


Support for VMPORT entries in the VM features xml domain.

Is your feature request related to a problem? Please describe:
Before I moved to kubvirt, I used the vmport disable function directly to the xml of virsch.
Like the kvm hidden function, it is very important as a function that can hide virtual machines, but unfortunately, kubirt does not support the vmport disable function.
If vmport is not disabled as well as simply kvm hidden, vmware virtual devices remain in the device manager, and a specific application may notice virtualization.
Describe the solution you'd like:
As shown below, I would like to add the vmport disable function like the kvm hidden function.
spec:
      domain:
        machine:
          # This value indicates QEMU machine type.
          type: q35
            #pc-q35
        ### disable BIOS ###
        firmware:
          bootloader:
            efi: {}
        features:
          acpi: {}
          kvm:
            hidden: true
          ### require support ###
          vmport:
            hidden: true

Describe alternatives you've considered:
This is the xml that was originally used by the virsh.
  <features>
    <acpi/>
    <apic/>
    <kvm>
      <hidden state='on'/>
    </kvm>
    <vmport `state='off'/>

Additional context:
If I have time, I want to try to do some PR myself, should I modify the Go?

Hi @LunarInterface, #10986 and #10985 are identical.
I suggest to close this one as duplicate.

Hi @LunarInterface, #10986 and #10985 are identical. I suggest to close this one as duplicate.
I accidentally duplicated it. Sorry for that.
Close this Issue.


Use --filter=blob:none to clone CPython faster

Current dependencies on/for this PR:

main

PR #1   👈

This comment was auto-generated by Graphite.


The option to turn off progress bar when you double tap

Describe your suggested feature
When you are watching a video and you double tap to go back or forward with your prefered set seconds the progress bar will pop up from the bottom of the screen it lasts about 3-4 seconds and it will show you how far you are on the video before it drops back down. This is very distracting can you add the option to turn this off the same way you can turn off Horizontal seek gesture. There should be a option when you double tap that the progress bar does not appear.
Other details
In the older version v<IP_ADDRESS> when you double tap nothing appears, but on the new version it does and it's distracting. Also if possible to turn off the animations and effects when you go back or forward to make the player plain and simple this way it lags and it's very slow. Because off all the animation and effects can you add a quick instant option that does not have animation and effects.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I will fill out all of the requested information in this form.

bottom of the screen it lasts about 3-4 seconds
bruh it goes away like the next second you stop pressing it.

but it's super laggy and gives you a delay effect sometimes you can hear audio for a few seconds when you exit

duplicate of #1279


Update docs with "how to merge" instructions

Update the docs with more instructions on how to safely merge, incorporating the new feedback we've gotten.

updated readme.


Need a function to add or substitute two dates

Zentinext - Ballerina Hackathon

This can be done via following functions in ballerina.lang.time package.
native function subtractDuration(Time timeData, int years, int months, int days, int hours, int minutes, int seconds, int milliSeconds)(Time);

native function addDuration(Time timeData, int years, int months, int days, int hours, int minutes, int seconds, int milliSeconds)(Time);

Refer the example Ballerina by Example - Date Time for more details.


Add fitted components

https://github.com/user-attachments/assets/9dfb5729-d980-4429-b7b5-724471b457d3
https://github.com/user-attachments/assets/cd8c34b8-f5e8-49fc-af45-945ce73ab2b6
https://github.com/user-attachments/assets/9bdfe060-9b06-4283-a9cf-945b741ce783

The BlogPost should probably use markdown to render the body

For Pet, let's try capturing the metadata similar to how we did with Media Registry:

Latest screenshots
https://github.com/user-attachments/assets/76e4d4b5-cea4-469d-9fda-96418a5a31c9
https://github.com/user-attachments/assets/3400b183-9523-48a4-bdc1-5e0191c062e6
https://github.com/user-attachments/assets/31ca6d82-c2e7-44b1-9d8e-6b3cf074af49

The Pet name value pairs should never have the horizontal scroll, IMO. The task of authoring a fitted component is to show as much important info as you can fit and leave out that which doesn't make the cut. Using a horizontal scroll defeats that intention.
In the Person fitted format, when we do the delegated render to the Pet, we should probably make that @format="fitted" as well since we have a limited space we are working within.


Multiclient actions

Is it possible to set up multi client action (different credentials) for one looker instance ?

Using action_state it is possible but it isn't ideal at this time. This is an area we are looking to improve

@phillipperalez Thanks for response. It would be useful.


[CLOSED] Bump gds-sso and `govuk_content_models'

 Issue by benlovell
Friday 18 December 2015 at 14:04 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/618

Trello: https://trello.com/c/ED5D8oNy
Bumps these dependencies to take advantage of linting for GDS-SSO
User and project-specific User models.

benlovell included the following code: https://github.com/alphagov/specialist-publisher/pull/618/commits

 Comment by benlovell
Monday 04 January 2016 at 14:29 GMT

This was naive. After speaking to @tommyp it turns out it would be best just to park this for now and wait for his magical rework to land.


Initial implementation of SPDX22 Detector

This PR introduces detector for SPDX 2.2 files and SpdxComponent type. Initial implementation parses JSON SPDX file and extracts minimal set of information about the SPDX document necessary to link SPDX file in other SPDX files. It implements IDefaultOffComponentDetector interface.
This detector could eventually be expanded to retrieve information about what SPDX documents describes and what are dependencies for a product that document describes.
Detector has been tested inside Microsoft teams in external referencing feature for SBOMs and has been live in SBOM generation process for about a month now.

I see that VerificationTests are failing on windows. Is there something that I missed and I need to address?

I don't think so. We've seen some flakiness regarding the verification tests on Windows recently. I know GitHub actions recently changed windows-latest from Windows Server 2019 to Windows Server 2022, and I suspect that might be part of the reason.
Let me try re-running and see if it passes.


Store user preferences locally  #14

Service classes for shared prefernces with the functions to get the current user data and set the user data with displayName,age,email,isAdult fields to persist the data onto the disk. Please do lemme know if any changes are needed further.

Please do lemme know if any changes are required

Please do lemme know if any changes are required

Will check it by tomorrow morning and let you know, if no more changes required I'll merge the code 😃

Sure 👍


What is the api to get list of markets(exchanges)?

https://github.com/ExodusMovement/cryptocompare#exchangelist I think.


Password Cheacker

Add this into a Folder then again try to pull the request...

How to create the folder?
On Sat, Oct 8, 2022, 18:06 Anmol Manocha @.***> wrote:

Add this into a Folder then again try to pull the request...
—
Reply to this email directly, view it on GitHub
https://github.com/Amdbase1/DSA-CP/pull/5#issuecomment-1272309061, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AVJXF2NPURDQ425PTBLDEKLWCFTETANCNFSM6AAAAAARAHRSQA
.
You are receiving this because you authored the thread.Message ID:
@.***>

If it is just random program, upload your file in Random Programs folder.


Support CLI list with attributes

Description

Support CLI list with attributes
Improves the CLI attributes parameter to support nesting/chaining.

Issues Resolved
Fixes #346
Check List

[x] New functionality includes testing.

[x] All tests pass ($ rake test).

[x] New functionality has been documented in the README if applicable.

[x] New functionality has been thoroughly documented in the examples (please include helpful comments).

[x] New endpoints supported are updated in the endpoints-support.md file.
[x] Changes are documented in the CHANGELOG.

Bah! Ruby 2.2 doesn't support Hash#dig 😞 Fix coming shortly


Document attachments should all be Saved after updating a document

After successfully running the UpdateDocument action, Chill should change any unsaved attachments in the document to the Saved variant. Otherwise, Chill may redundantly upload the attachment if the document is updated again.

This is a breaking change because the Client::update_document method has a &Document parameter that must be changed to &mut Document.


"Error: Conversation ended" when switching to another component from webchat/bot

We are migrating our bot from v3 to v4 and encountered an issue where if I move from bot component to any other component in our angular app, below uncaught error is thrown by bot framework:
ERROR Error: Uncaught (in promise): Error: conversation ended
Error: conversation ended
    at Object.<anonymous> (webchat-es5.js:67)
    at n (webchat-es5.js:1)
    at Object.<anonymous> (webchat-es5.js:67)
    at n (webchat-es5.js:1)
    at Object.<anonymous> (webchat-es5.js:29)
    at n (webchat-es5.js:1)
    at Object.<anonymous> (webchat-es5.js:67)
    at n (webchat-es5.js:1)
    at Object.<anonymous> (webchat-es5.js:67)
    at n (webchat-es5.js:1)

We are currently using "https://cdn.botframework.com/botframework-webchat/latest/webchat-es5.js". This issue seems to have been fixed in master branch of webchat. I am not sure what PR has fixed it but would like to know when the master is expected to be deployed to latest production. Is there any place where I can track it? Thanks.

Hi @r-meh, glad the fix is available to you on /master/ bits. Unfortunately we don't have a feed for announcing our releases yet. We have issue #1399 for tracking ideas on how to communicate releases with our customers. Please feel free to add ideas or your +1 over there. :)
Our 4.3 release should be publicly available early March. Please check back then!


Datepicker locks in when value is Invalid and does not open.

When a user accidentally clicks on one of the empty spaces (previous month's days on the current month) in the date picker,  it outputs "invalid date" as the value and will become disabled. Clicking on the input will not open the date picker and the input will not change when anything is typed.
Up till now, I was using a workaround where if the output is "Invalid Date" I pass in an empty string. This only works for independent date pickers. When 2 date pickers are tied together with min and max values. It is possible that both of them can get locked.
Here is a demo codesandbox.
Here is what it will look like when the bug happens.

Please help me fix this
Thanks for looking at my issue.

Thank you for fixing this bug. I'm very excited to try it out.


Web proxy setup

Change ci-initial.yml to ansible/all-initial.yml, and apply to all, so it can be used for setting up any new server.
Change idr-services.yml to expand root and log volumes

Barring
ERROR! the file_name '/home/travis/build/openmicroscopy/infrastructure/ansible/ci-initial.yml' does not exist, or is not readable

this looks good. (Tested a -C run locally with ansible 2.1 which handled LVM much better.)

The problem now is
$ ansible-playbook -i ansible/example-hosts --syntax-check ansible/*.yml
ERROR! vars file vars/os-create-{{ os_cloud_provider | default('uod') }}.yml was not found

Is there a fix for that?

Not without refactoring the playbooks, it looks like you can't have a conditional (when:) in vars_files

Then exclude this playbook from travis?

I've just found this https://github.com/ansible/ansible/issues/10010#issuecomment-70647392
Not quite the same behaviour (since it'll also default to -uod if os_cloud_provider is invalid instead of only when it's empty)

Pushed, see the commit message for caveats

No possibility of a noop.yml? If not, let's go with this, and we can see how often it bites us.

I could do that, but it means os_cloud_provider becomes mandatory whereas previously it defaults to uod. Reckon it's worth it?

Hmmm.....don't know. Let's hold off and see.


CreateImageWizard: don't obscure activation key

Activation key was previously obscured by stars, instead just show it
(both in the input and in the review step) as it isn't secret.

Closed in favor of #458


Force a map render when the view is not updated

See https://github.com/openlayers/ol3/pull/5764#issuecomment-241665841

I think this change makes sense, forcing a render in a very specific case.
@ahocevar any objections from your side given your previous comments?

I cannot come up with a better solution for now, so please merge.


Fix deprecation warnings emitted by import qiskit.primitives

Summary
The primitives still need to support the legacy
qiskit.opflow.PauliSumOp while that object is deprecated but not removed.  However, we do not want the import of qiskit.primitives to need to import opflow to do that, since that will cause the deprecation warnings emitted by the import of opflow to be hidden (they'll be blamed on Qiskit library code, so hidden by default).
All type-hint usage we can hide behind the TYPE_CHECKING static analysis gate.  For cases where we were actively runtime type-checking an object, we can gate the isinstance check behind a check that qiskit.opflow is already imported; the object cannot be a PauliSumOp if the module isn't initialised.

Details and comments
Fix #10245

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

12 of 17 (70.59%)  changed or added relevant lines in 4 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.02%) to 85.958%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/primitives/backend_estimator.py
2
3
66.67%

qiskit/primitives/base/base_estimator.py
2
3
66.67%

qiskit/primitives/estimator.py
2
3
66.67%

qiskit/primitives/utils.py
6
8
75.0%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/expr.rs
1
93.76%

crates/qasm2/src/lex.rs
2
91.65%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
71348

Relevant Lines:
83003

💛  - Coveralls


Partition Strategy for hive sink

Allow for HiveSink users to specify a strategy to determine how partition names are created.

Well you can kind of already do this with the Frame map function...the following code snippet uses a JDBCSource and adds a new column called title depending on the source column name.  The Hive table defines title as the partition key.
    // Write to a HiveSink from a JDBCSource
    val query = "SELECT NAME, AGE, SALARY, CREATION_TIME FROM PERSON"
    implicit val hadoopFileSystem = FileSystem.get(new Configuration())
    implicit val hiveMetaStoreClient = new HiveMetaStoreClient(new HiveConf())
    JdbcSource(() => dataSource.getConnection, query)
      .withFetchSize(10)
      .toFrame
      .withLowerCaseSchema
      // Transformation - add title to row
      .map { row => if (row.get("name").toString == "Alice") row.add("title", "Mrs") else row.add("title", "Mr") }
      .to(HiveSink("eel_test", "person").withIOThreads(1).withInheritPermission(true))

This can now be done with the withPartitionPathStrategy option on hive sink, and even the withFilenameStrategy to decide how the file names are created.


Implementation for Left/Right margins in TextFormat

Main idea is to replace all the hardcoded offsets of 2 (and their sums for both sides of 4) to actual margins or margins sum. I've updated everything I believed is required to update. I would appreciate your feedback on this :)
Example I've tested on:
package;

import openfl.text.TextFormatAlign;
import openfl.text.TextFormat;
import openfl.text.TextField;
import openfl.display.Sprite;

class Main extends Sprite {
    public function new() {
        super();

        var format:TextFormat = new TextFormat('Arial', 40, 0xFF0000);
        format.leftMargin = 10;
        format.rightMargin = 10;
        format.align = TextFormatAlign.JUSTIFY;
        format.underline = true;

        var field:TextField = new TextField();
        field.defaultTextFormat = format;
        field.backgroundColor = 0x00FF00;
        field.background = true;
        field.multiline = true;
        field.wordWrap = true;
        field.width = 450;
        field.height = 100;
        field.text = 'A b c d e f g h i j k l m n o p r s t u w x y z a b c d e f';

        this.addChild(field);
    }
}

It's HTML5 on the left | Flash on the right, on every attached screenshot. Underscore was just left for decoration and it sometimes hints where is the issue just looking at its length :)
margins 10 10 width 450 aligned left

margins 250 250 width 450 aligned center

margins 10 10 width 450 aligned justify

margins 250 250 width 450 aligned left

margins 100 25 width 450 aligned center

@lmodlinski long delay :) but I will get refreshed on the various text PRs and get this merged in one way or another


fixup bicycle hackernews tests

If approved, this PR will:

fix(proto): add duration_ms and bike_model to bicycle_rental model
Add two new attributes duration_ms and bike_model that has been added to the public dataset bicycle rental. Integration tests now pass again.

fix: remove no longer existing public data hacker_news.stories
The stories table in the public hacker news dataset have been removed. Removing references to it.

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

Project coverage is 60.03%. Comparing base (3cdd61d) to head (64b5b3b).
Report is 69 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

@@            Coverage Diff             @@
##           master     #378      +/-   ##
==========================================
+ Coverage   58.94%   60.03%   +1.09%
==========================================
  Files           7        7
  Lines        1023     1061      +38
==========================================
+ Hits          603      637      +34
- Misses        353      354       +1
- Partials       67       70       +3

see 2 files with indirect coverage changes


Update your client software to continue using Let's Encrypt

Hey Guys,
I'm using this docker-container to cert my web-services.
But now I get an email from letsencrypt which says I should use a cert client which use ACMEv2.
Can I change it or will you update this client?
T
https://community.letsencrypt.org/t/end-of-life-plan-for-acmev1

Saw https://github.com/nginx-proxy/docker-letsencrypt-nginx-proxy-companion/issues/319


Configurable Classes (Round 2)

See #17 and #20
This resolves the invariant exception noted in the latter.

Thanks again @Jador :100:
I tried and yup the exception is gone :+1:
But i still couldn't make custom classnames work am i doing it wrong? The following has no effect for me:
<DateRange
   classNames={{ calendar : 'MyCalendar' }}

you did it right, Calendar was trying to get classNames from this >.< Now for sure it should be good

Thanks @Jador


Prettier ignore swagger doc

AB#31624
Describe your changes

Checklist before requesting a review

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

@RubenGeo I included a similar change in #6192 - I would keep those changes because it also includes a version of the json that is not prettified. If you agree then you can review that one and close this PR


guides: add Twitter and GitHub links to nav bar of guides page

Top right we now have a really good "Help" link. We should add the Twitter and GitHub links to the nav bar too.

Thoughts, @marcosnils ?

This has been done.


[Fix #7779] Fix a false positive for Style/MultilineMethodCallIndentation

Fixes #7779.
This PR fixes a false positive for Style/MultilineMethodCallIndentation when using Ruby 2.7's numbered parameter.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Thanks!


creating modules

simple execution code and also image to test py360convert code.

closing for now; i do think that we need better examples in the README, but we have unit tests now via pytest.


Add 2.6 prebuilt models

Fixes #2565
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

You can find all models in this file https://github.com/opensearch-project/ml-commons/blob/2.x/ml-algorithms/src/test/resources/org/opensearch/ml/engine/algorithms/text_embedding/pre_trained_models_list.json


Update cryptography dependency to 1.5.3

Nb. Service doesn't use dependency listed in requirements.txt - dependencies are held in the base docker image onsdigital/flask-cryto
Docker rebuild will update to new dependency as the base image is pulled in.
Closes #33

LGTM 👍


[SU-100] Increase font weight of table headers

Requested by UX for new data tab design (enable with configOverridesStore.set({ isDataTabRedesignEnabled: true })). I made the change in the base table component so that all tables would be consistent.
Before

After

This is affecting other tables, in somewhat unexpected ways. It could be a good thing in the end, but I'm not sure. I think changing this universally would require auditing every place where this is used... which could be tricky given some places have very careful width settings to prevent ellipses or line-wrapping. Maybe target this more specifically at the new data table?

Oops... forgot to attach an example screenshot.

I'd claim that, rather than introducing an issue in the workflow inputs/outputs table, it made an existing issue more visible. Those tables currently have a mix of 3 different font weights for column heading: 500 for task name and type, 400 for variable, and 700 for attribute.

I think changing this universally would require auditing every place where this is used

I went through every file found by git grep HeaderCell and checked that if any column header in a table used HeaderCell, then every column in the table also used it.

WorkflowView was really just an example of a case that I found that was impacted outside the scope of the data table redesign that this PR is targeting. I was thinking that a visual inspection of other tables might be needed.
One that I know I was careful about setting column widths for is JobHistory. However, that didn't see to get the extra font weight with this change. I'm not sure why, but if it had, it definitely would have affected the layout and probably required more tweaking of the column widths.
All that said, I did try to find as many of these as I could, and WorkflowView was the only one I found that was affected. I probably didn't test all of them, but probably enough of them, and we can always fix others later.

Going down the rabbit hole, JobHistory doesn't use the HeaderCell component. It uses a different shared component for column headers: HeaderRenderer (I don't know why there are multiple components for column headers).
https://github.com/DataBiosphere/terra-ui/blob/661e8f817013cedb1b83685cd8b9564855035941/src/components/common.js#L573-L575
HeaderRenderer uses a font weight of 600. So this PR will bring HeaderCell and HeaderRenderer in lline. JobHistory overrides that font weight though:
https://github.com/DataBiosphere/terra-ui/blob/96d986bb25a73a1541cf18b57744b5a4e8ce8681/src/pages/workspaces/workspace/JobHistory.js#L165-L174


confusion matrix got the background FP: 100% TP0%

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (master) or latest version (3.x).

Task
I have modified the scripts/configs, or I'm working on my own tasks/models/datasets.
Branch
master branch https://github.com/open-mmlab/mmdetection
Environment
TorchVision: 0.9.1+cu101
OpenCV: 4.6.0
MMCV: 1.7.0
MMCV Compiler: GCC 7.3
MMCV CUDA Compiler: 10.2
MMDetection: 2.26.0+e4bf27c
Reproduces the problem - code sample
i run the python tools/analysis_tools/confusion_matrix.py
Reproduces the problem - command or script
python tools/analysis_tools/confusion_matrix.py \
${CONFIG}  \
${DETECTION_RESULTS} \
${SAVE_DIR} \
--show
Reproduces the problem - error message
no erroe message
Additional information

I only have one class in my dataset

@MARUIHAN123 I have the same problem. Please tell me how to fix it If you know. Thanks!


Fix broken packages

JAVA_HOME is not set correctly on Linux...

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@sodre : here is the conda_build_config.yaml file:
c_compiler:
  - toolchain_c  # [linux or osx]
  - gcc  # [linux]
  - clang  # [osx]
  - vs2008  # [win]
  - vs2015  # [win]

For OSX and Linux, everything is fine as conda uploads one builds for each C compiler. But it's not the case for Windows, or if it's the case they have the same name. Linux and OSX packages have a custom name.
Do you know why?

Sorry , I don't


Map pin quantities

Description
Create a new parameter category pinQuantities for parameters that are defined on a pin-by-pin basis within a block (or other spatial basis at a finer scale than the assembly). Add this parameter category to the list of values to be mapped between assemblies by the UniformMeshConverter.

Checklist

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

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

[x] The release notes (location doc/release/0.X.rst) are up-to-date with any bug fixes or new features.
[x] The documentation is still up-to-date in the doc folder.
[x] The dependencies are still up-to-date in setup.py.

@jakehader @john-science this update provides the functionality to map the pinQuantities parameters from the uniform assembly to a non-uniform assembly when using the nonUniformAssemFlags setting.
Related PRs include #879 #893 #919 .

Hi @mgjarrett, given this update to add a new category is there anything else in the uniform mesher that should change to treat this or does this just enable some behavior for other possible plugins to manage?

Quick Note: 2 of the 3 reactor projects that are currently based on ARMI do not have "pins" or "assemblies", so this should not be in the parameter list by default. It should be turned on by something.

@jakehader Looks good to me.
Thoughts?


It is suggested that the future development direction is still based on full memory, and supports K33 and K34.

come on

Does anyone care?


Clustintime: a toolbox for spatio-temporal clustering of fMRI data

Title
Clustintime: a toolbox for spatio-temporal clustering of fMRI data
Leaders
Cris Tobías (@ctobias)
Collaborators
No response
Brainhack Global 2023 Event
Brainhack Donostia
Project Description

What are you doing, for whom, and why?
This project is a toolbox that allows researchers in neuroimage to apply clustering methods to fMRI data on the spatiotemporal domain. Conventional methods of clustering in fMRI allow to see spatial patterns but cannot describe the temporal dynamics of functional activity.

What makes your project special and exciting?
Providing a tool for researchers interested in analyzing brain patterns in uncontrolled fmri experiments or clinical settings is a necessity that has not been covered yet and that could be done through clustintime.

How to get started?
Clustintime has been under development for a few years and has been already tried on data from individual epileptic patients, the skeletton of the project is already done, now there is need to transform it into a proper toolbox to release the first version of the software.

Where to find key resources?
The project already has its repository on GitHub (https://github.com/Cristina-Tobias/clustintime) and a public fMRI data will be used for testing.

Link to project repository/sources
https://github.com/Cristina-Tobias/clustintime
Goals for Brainhack Global
Create integration testing suite/framework
Create CI/CD pipeline
Populate CI/CD pipeline with checks
Release version 0.1.0
Create documentation
Improve user experience of adding input
Structure output messages
Release version 0.2.0
Introduce new features

group analysis
consensus clustering

Release version 1.0.0
Good first issues
Issue 1: Create integration testing suite/framework

Find usable data
Execute each algorithm in a reproducible way (fix seed)
Integrate into CI/CD pipeline

Issue 2: Create CI/CD pipeline

Find free tier provider
e.g., github actions, circleci

Issue 3: Populate CI/CD pipeline with checks

Pylint, flake8, black, isort, ...

Communication channels
https://mattermost.bcbl.eu/spin-lab/channels/clustintime_bhd
Skills

Git: intermediate
Python: advanced (only for those that want to develop new methods)
Markdown: intermediate
Bash: begginer

Onboarding documentation
No response
What will participants learn?

Experience on collaborating with git
Experience on how to design and build an open-source library
Different methods of fMRI visualization
Unsupervised machine learning methods for fMRI

Data to use
No response
Number of collaborators
more
Credit to collaborators
Project contributors will be acknowledged as contributors in GitHub, credit will also be given in future publications (if any) to those who make a major contribution to the toolbox.
Image
Leave this text if you don't have an image yet.
Type
coding_methods, documentation, method_development, pipeline_development
Development status
1_basic structure
Topic
data_visualisation, machine_learning
Tools
other
Programming language
Python
Modalities
fMRI
Git skills
2_branches_PRs
Anything else?
No response
Things to do after the project is submitted and ready to review.

[ ] Add a comment below the main post of your issue saying: Hi @brainhackorg/project-monitors my project is ready!
[ ] Twitter-sized summary of your project pitch.

good with me


Remove duplicate entry of build 19028 on Flight Hub

Subj.

@bleblancMSFT - PR is ready for final review.
Thank you.
cc: @kenwith


File name replaced wrong

@heavymery In my tests, had two files with similar names. The first was jquery-migrate.js and jquery.js.
As I understand it, the first jquery-migrate.js file had its name changed, but according to my RequireJS configuration file, was to be jquery.js only.
Example:
Before
- app
	- bower_components
		- jquery
			- jquery-migrate.js
			- jquery.js
	- scripts
		- bootstrap.js
		- main.js

app/scripts/bootstrap.js
'use strict';
require.config({
    baseUrl: '/scripts',
    paths: {
        'main': 'main',
        'jquery': '../bower_components/jquery/jquery'
    }
});
require([
    'jquery',
    'main'
], function(jquery, main) {
    $('body').html('Hello!');
});

After
- app
	- bower_components
		- jquery
			- 996f42cf.jquery-migrate.js // Wrong
			- jquery.js // Right
	- scripts
		- bootstrap.js
		- eeedbd79.main.js

app/scripts/bootstrap.js
'use strict';
require.config({
    baseUrl: '/scripts',
    paths: {
        'eeedbd79.main': 'eeedbd79.main',
        'jquery': '../bower_components/jquery/jquery'
    }
});
require([
    'jquery',
    'eeedbd79.main'
], function(jquery, main) {
    $('body').html('Hello!');
});

Beyond what I mentioned above, an error appears in the terminal:
Running "requireRev: dist" (requireRev) task
Warning: Can not read property 'source' of undefined Use --force to continue.
Aborted due to warnings.

@jefferson-william Let me show requireRev configuration on Gruntfile.js.

@heavymery Sorry, but I wiggled the file and do not know but as it was.
You could do some testing there according to my example?
I'm trying to use this package the days and none of the ways I tried to work perfectly.
It's a shame I did not find out exactly how was the Gruntfile.js file of this example. But if you can help me just by looking at the above information I thank you. Otherwise I will understand.

@jefferson-william Sorry for late response. I was able to reproduce this bug. Maybe, your requireRev configuration was like below. right?
    requireRev: {
      dist: {
        expand: true,
        cwd: 'app',
        src: [
          'scripts/**/*.js',
          'bower_components/**/*.js'
        ]
      },

This issue is bug. But purpose of requireRev is revisioning requirejs dependency files. So, configuration should be like below.
    requireRev: {
      dist: {
        expand: true,
        cwd: 'app',
        src: [
          'scripts/**/*.js'
        ]
      },

@jefferson-william And If your project has no need to deploy multiple requireJS dependency files, I suggest to use grunt-contrib-requirejs.
This package will help you to optimize multiple requireJS dependency files to one js file like below.
Before
// hello.js
define(function(){return "hello";});

// world.js
define(function(){return "world";});

// project.js
require(['hello', 'world'], function(hello, world) {
  console.log(hello,world);
});

After
// project-optimized.js
define("hello",[],function(){return"hello"}),
define("world",[],function(){return"world"}),
require(["hello","world"],function(e,t){console.log(e,t)}),define("project",function(){});


docsPath issue with / character

in my code this is my docsPath def docsPath = "project-docs-api" when i try to use def docsPath = "project/docs/api" swagger cannot found the json
what i'm doing wrong?

https://github.com/swagger-spray/swagger-spray is a maintained fork of spray-swagger and there is some basic code to fix up the docsPath for you.


Question about validate function

Hi @raymin0223,
Thank you for making your research code open source.
I have a question about the validate function in main.py file. I think I do not fully understand the working of the validate function.
I do fully understand the first half of this function, but after the calculation of specificity (sp), sensitivity (se) and ICBHI score (sc), I might be missing something. I think these metrics are calculated for every batch of the validation set. But then the validation metrics of the final batch are printed to the console. I would expect to see metrics printed in the console that are averaged out across all validation batches? Would this reasoning be correct?
The validate function currently looks like this:
def validate(val_loader, model, classifier, criterion, args, best_acc, best_model=None):
    save_bool = False
    model.eval()
    classifier.eval()

    batch_time = AverageMeter()
    losses = AverageMeter()
    top1 = AverageMeter()
    hits, counts = [0.0] * args.n_cls, [0.0] * args.n_cls

    with torch.no_grad():
        end = time.time()
        for idx, (images, labels, metadata) in enumerate(val_loader):
            images = images.cuda(non_blocking=True)
            labels = labels.cuda(non_blocking=True)
            bsz = labels.shape[0]

            with torch.cuda.amp.autocast():
                features = model(images)
                output = classifier(features)
                loss = criterion[0](output, labels)

            losses.update(loss.item(), bsz)
            [acc1], _ = accuracy(output, labels, topk=(1,))
            top1.update(acc1[0], bsz)

            _, preds = torch.max(output, 1)
            for idx in range(preds.shape[0]):
                counts[labels[idx].item()] += 1.0
                if not args.two_cls_eval:
                    if preds[idx].item() == labels[idx].item():
                        hits[labels[idx].item()] += 1.0
                else:  # only when args.n_cls == 4
                    if labels[idx].item() == 0 and preds[idx].item() == labels[idx].item():
                        hits[labels[idx].item()] += 1.0
                    elif labels[idx].item() != 0 and preds[idx].item() > 0:  # abnormal
                        hits[labels[idx].item()] += 1.0

            sp, se, sc = get_score(hits, counts)

            batch_time.update(time.time() - end)
            end = time.time()

            if (idx + 1) % args.print_freq == 0:
                print('Test: [{0}/{1}]\t'
                      'Time {batch_time.val:.3f} ({batch_time.avg:.3f})\t'
                      'Loss {loss.val:.4f} ({loss.avg:.4f})\t'
                      'Acc@1 {top1.val:.3f} ({top1.avg:.3f})'.format(
                       idx + 1, len(val_loader), batch_time=batch_time,
                       loss=losses, top1=top1))

    if sc > best_acc[-1] and se > 5:
        save_bool = True
        best_acc = [sp, se, sc]
        best_model = [deepcopy(model.state_dict()), deepcopy(classifier.state_dict())]

    print(' * S_p: {:.2f}, S_e: {:.2f}, Score: {:.2f} (Best S_p: {:.2f}, S_e: {:.2f}, Score: {:.2f})'.format(sp, se, sc, best_acc[0], best_acc[1], best_acc[-1]))
    print(' * Acc@1 {top1.avg:.2f}'.format(top1=top1))

    return best_acc, best_model, save_bool

But in my reasoning, it would have to look more or less like this:
def validate(val_loader, model, classifier, criterion, args, best_acc, best_model=None):
    save_bool = False
    model.eval()
    classifier.eval()

    batch_time = AverageMeter()
    losses = AverageMeter()
    top1 = AverageMeter()
    hits, counts = [0.0] * args.n_cls, [0.0] * args.n_cls

    # Create AverageMeter for all metrics, such that these can be averaged out across all validation batches
    specificities = AverageMeter()
    sensitivities = AverageMeter()
    scores = AverageMeter()

    with torch.no_grad():
        end = time.time()
        for idx, (images, labels, metadata) in enumerate(val_loader):
            images = images.cuda(non_blocking=True)
            labels = labels.cuda(non_blocking=True)
            bsz = labels.shape[0]

            with torch.cuda.amp.autocast():
                features = model(images)
                output = classifier(features)
                loss = criterion[0](output, labels)

            losses.update(loss.item(), bsz)
            [acc1], _ = accuracy(output, labels, topk=(1,))
            top1.update(acc1[0], bsz)

            _, preds = torch.max(output, 1)
            for idx in range(preds.shape[0]):
                counts[labels[idx].item()] += 1.0
                if not args.two_cls_eval:
                    if preds[idx].item() == labels[idx].item():
                        hits[labels[idx].item()] += 1.0
                else:  # only when args.n_cls == 4
                    if labels[idx].item() == 0 and preds[idx].item() == labels[idx].item():
                        hits[labels[idx].item()] += 1.0
                    elif labels[idx].item() != 0 and preds[idx].item() > 0:  # abnormal
                        hits[labels[idx].item()] += 1.0

            sp, se, sc = get_score(hits, counts)

            # Add metrics of this validation batch to the AverageMeter of each metric
            specificities.update(sp)
            sensitivities.update(se)
            scores.update(sc)

            batch_time.update(time.time() - end)
            end = time.time()

            if (idx + 1) % args.print_freq == 0:
                print('Test: [{0}/{1}]\t'
                      'Time {batch_time.val:.3f} ({batch_time.avg:.3f})\t'
                      'Loss {loss.val:.4f} ({loss.avg:.4f})\t'
                      'Acc@1 {top1.val:.3f} ({top1.avg:.3f})'.format(
                       idx + 1, len(val_loader), batch_time=batch_time,
                       loss=losses, top1=top1))

    # Save model if the average ICBHI score is better than the current best one
    if scores.avg > best_acc[-1] and sensitivities.avg > 5:
        save_bool = True
        best_acc = [specificities.avg, sensitivities.avg, scores.avg]
        best_model = [deepcopy(model.state_dict()), deepcopy(classifier.state_dict())]

    print(' * S_p: {:.2f}, S_e: {:.2f}, Score: {:.2f} (Best S_p: {:.2f}, S_e: {:.2f}, Score: {:.2f})'.format(specificities.avg, sensitivities.avg, scores.avg, best_acc[0], best_acc[1], best_acc[-1]))
    print(' * Acc@1 {top1.avg:.2f}'.format(top1=top1))

    return best_acc, best_model, save_bool

Could you please help me figure out where my reasoning might be wrong?
Thank you very much.

Hi @michiel-j,
So sorry for the late reply -- idk why I missed this issues and notification emails..
Thanks for reporting some suspicious codes. And to be honest, I should have implemented much clear haha..
Your intuition is perfect, and I'll explain why I implemented it in that way, and what it could be wrong.
Evaluation of all samples in validation datasets will be stored hits and counts at the end of the for loop for val_loader.
Thus, at the end of the loop, the last sp, se, sc will be the final metrics for validation datasets.
I'm assuming it is correct implementation to get the final evaluation metrics, intermediate metrics for three metrics (sp, se, sc) are kinda confusing though.
And about your suggested codes, not sure, but it could induce different metrics of sp, se, and sc. Since AverageMeter will continuously average the values after the updates at each step.

Hi @raymin0223
Thank you for your answer.
No need to apologise for the late reply.
I understand the function working now. Your validate function behaves correctly indeed.
I will close this issue now.


Codebase should be integrated with Prettier and Eslint

Additional Information
I am working on it .

Hi @aryamanpuri, please refrain from creating such issues. It is against the spirit of open source if you don't even describe the issue and simply write "I'm working on it".
A good issue should have -

The problem you're facing
Steps to replicate problem (if applicable)
What is expected
Any idea you have about solving the issue
A checklist of points to check this issue's resolution against

Hope this helps you in future issues.

@xprilion Sorry for that
Problem Faced
Codebase should be aligned which helps the user and the developers to read the code easily.
To Reproduce
It can be seen in the project codebase.
Actual behavior
Nothing happens.
Expected behavior
Codebase integrated with prettier and eslint will help the code to ensure consistent rules are being followed for indentation, spacing, semicolons, single quotes vs double quotes etc

@xprilion Are you mentoring this project for GSOC 2020?

Hey @aryamanpuri, thanks for addressing my suggestions.
No, I'm unfortunately not associated with GSoC anyway this year. I was simply looking through the new organizations this year and found your issue as the first one on this repo and chimed in hope to help you in creating better issues in the future! :slightly_smiling_face:


Python version doesn't sort triggers properly when multiple [*] are used

With this rivescript:
+ [*] what [*] fram is [*]
- A1

+ [*] what [*] flow [*] getting [*] reviewed [*] fram is answer type [*]
- A2

+ hey
@ what flow getting reviewed fram is answer type

Using the rivescript.com "try it" and typing in "hey" gives you "A2", the proper answer.  Using python shell.py will give you "A1".  Checking the sorting logic shows it sorting 3 stars before 6 stars.  Not what I want it to do.  I need it to match the longer trigger.

Fixed in #144.


Move string parsing to findTemplate method

I've omitted tests for the other functions, cause as far as I checked, there are no places where they're used with none-string objects. Fixes https://github.com/Sylius/SyliusThemeBundle/pull/19#discussion_r260784462

The base of this pull-request was changed, you need fetch and reset your local branch
if you want to add new commits to this pull request. Reset before you pull, else commits
may become messed-up.
Unless you added new commits (to this branch) locally that you did not push yet,
execute git fetch origin && git reset "theme-filesystem-loader-hotfix" to update your local branch.
Feel free to ask for assistance when you get stuck :+1:

Thank you, Mateusz! :1st_place_medal:


TELEWORK: Supervisor Notification Email

User Story
As a supervisor, I want more comprehensive email when an employee submits a new telework form for my review so that I don't get confused about the telework approval process.
Tasks

[x] Add who submitted the form (with proper formatting) in the notification email

[x] Have the number of requested telework days in the body of the notification email

[x] Explain why the draft telework agreement is included in the email as a PDF

[x] Have explicit next-steps for when the employee is asking for three or more days (need attachment with ADM approval, expectation of what they'll need with them to approve the form)

[x] Suggested wording for if they have to send a request to the ADM (how to send request, procedure for supervisors -- could just be pointing them to the 'for more info' document)

Acceptance Criteria

[x] All tasks are included in the body of the email sent to the supervisor in all three environments

Confirmation on how the process really works before we draft directions.

Successfully tested in PROD
Ready to be closed

Tested in Prod.


Deleting the sensors test cases which are covered in the inventory

Deleting the Test cases in test_sensors.robot which are covered in the inventory.

https://gerrit.openbmc-project.xyz/#/c/5948/


Know issues and fix deadlines

Know issues: data product publisher has identified the need for corrections, communicated them to users, and described when they will be corrected (fix dl).

Added to spec