Split "Enemy Config" and "Bonus Stats"

Suggest having the "Bonus Stats" apply to the inidividual Loadout rather than the entire Team.
The "Enemy Config" can apply to the entire Team, and propergate down to all Loadouts within the Team since this is a "world" setting, rather than a "situational" setting.
That way you can account for the following situation:
You have an Ayaka freeze team that you use for both Overworld/domains and Abyss.
So you want to create 2 loadouts for that same team; "Overworld" and "Abyss",
The "Abyss" loadout you want to have the crit buffs in the Bonus Stats (CR+8 CD+15).
But with the current implentation these buffs are saved to the Team, not the Loadout.
So you'll have to add and remove them every time you edit your Overworld and Abyss loadouts.

Relavent to #1525

Bonus stats is applied to the individual loadout, and the enemy is applied to the entire team. This is how it is implemented in Team loadout right now.

I suggest you test this. Because this is not how it works in the current Beta. If you change the bonus stats in one Loadout, it changes it in all Loadouts with that Team.

Hmm, then it might be a bug


urlencode searches to fix slash in search causing 404

Fix for https://github.com/flathub/website/issues/378

A . still seems to trigger this problem unfortunatly.
Btw, it would be helpful, if you could work on a different branch in your repo and not main :)


Left nav item text shifts slightly when a test switches from "in progress" to "pass/fail" status

Describe the bug
The text alignment of the Test names in the left nav bar shifts slightly when the icon changes from a number to a pass/fail indicator:

It appears that either the numerical circles or the pass/fail icons are not quite centered with one another. In the mock there is an equal 8px of top and bottom padding, with a 24px by 24px size.
To Reproduce
Steps to reproduce the behavior:

Start an Assessment on any page
Click on any Test with manual pass/fail requirements (eg, "Errors")
Mark all the Requirements as "Pass"
From any requirement, toggle between setting it as "Pass" and resetting its status
As you toggle, observe that the text alignment of the "Errors" item in the left nav bar changes slightly

Expected behavior
This toggling should not shift the text in the nav bar
Extension (please complete the following information)

OS: Windows
Version 2.2.0
Microsoft Edge Version <IP_ADDRESS> (Official build) dev (64-bit)
Target Page irrelevant

Are you willing to submit a PR?
yes
Did you search for similar existing issues?
yes

@dbjorge per conversation with Mark, won't fix.


Contributors

Summary
Let's use this issue to add contributors information with @allcontributors.
What needs to be done?
Write a comment with the contribution information with the following nomenclature:
@all-contributors please add @<username> for <contributions>
We suggest typing up to 5 contribution types (see the available Emoji Key ✨ here).
Feel free to list those contribution types you find more relevant to Scivision 🐨
Further info of the all-contributors bot here.
Who can help?
Anyone

@all-contributors please add @ots22 for code, ideas, doc, infra

@all-contributors please add @miquelmassot for code, ideas, doc, plugin

@all-contributors please add @evangeline-corcoran for code, ideas, doc, infra

Add @evangeline-corcoran as a contributor for code, ideas, doc, infra

@all-contributors please add @evangeline-corcoran for code, ideas, doc, infra

https://github.com/all-contributors please add @mooniean for code, ideas, doc, design, example

@all-contributors please add @mooniean for code, ideas, doc, design, example

@all-contributors please add @acocac for code, ideas, design, example

@all-contributors please add @scotthosking for fundingFinding, ideas, research

https://github.com/all-contributors please add @edwardchalstrey1  for code, ideas, doc, infra

@all-contributors please add @edwardchalstrey1  for code, ideas, doc, infra

@all-contributors please add @quantumjot for code, ideas, doc, infra, research, fundingFinding

https://github.com/all-contributors please add @AidaMehonic for eventOrganizing, doc, ideas

@all-contributors please add @AidaMehonic for eventOrganizing, doc, ideas

https://github.com/all-contributors please add @lupinthief for ideas

@all-contributors please add @lupinthief for ideas

@all-contributors please add @RobBlackwell for ideas

@all-contributors please add @ for <data_provider>

@all-contributors please add @JamesAliScott for data_provider

@all-contributors please add @JamesAliScott for ideas

@all-contributors please add @JamesAliScott for data

@all-contributors please add @martinsjrogers for data example code and ideas

@all-contributors please add @kasra-hosseini for code, ideas, doc, infra

https://github.com/all-contributors please add @nbarlowATI for ideas, eventOrganizing

@all-contributors please add @nbarlowATI for ideas, eventOrganizing, examples
@all-contributors please add @pwochner for ideas, eventOrganizing, examples

@all-contributors please add @jmp1985 for ideas data plugin

@all-contributors please add @shmh40 for Examples, Talks

@all-contributors please add @Erioldoesdesign for ideas, design

@allcontributors please add @vimode for ideas, design, code and a11y

@allcontributors please add @IFenton for code, ideas, doc, infra

@allcontributors please add @louisavz for ideas, promotion

@allcontributors please add @louisavz for ideas, promotion

@allcontributors please add @louisavz for ideas, promotion, blog

@all-contributors please add @aldenc for ideas, doc, design, eventOrganizing, promotion, projectManagement


NoClassDefFoundError: org/apache/spark/Logging

I am trying to run sample code with $pyspark --packages databricks:tensorframes:0.2.3-s_2.10
I am getting this error (below attached) with spark-1.6, spark-2.0 and custom build on spark with branch-2.0.
I tried to find org.apache.spark.logging jar, which I am unable to find anywhere.
I am using Mac OS ElCapitain.
Please help.
Py4JJavaError                             Traceback (most recent call last)
 in ()
8     # The TensorFlow placeholder that corresponds to column 'x'.
9     # The shape of the placeholder is automatically inferred from the DataFrame.
---> 10     x = tfs.block(df, "x")
11     # The output that adds 3 to x
12     z = tf.add(x, 3, name='z')
/private/var/folders/_f/fzxfs9dj33j0381wvy5bdsdm0000gn/T/spark-1831a94f-7cdd-4691-a32a-4cbc3d9bc3a5/userFiles-66cd46aa-619d-41cf-806d-3dc6e509e13d/databricks_tensorframes-0.2.3-s_2.10.jar/tensorframes/core.py in block(df, col_name, tf_name)
313     :return: a TensorFlow placeholder.
314     """
--> 315     return _auto_placeholder(df, col_name, tf_name, block = True)
316
317 def row(df, col_name, tf_name = None):
/private/var/folders/_f/fzxfs9dj33j0381wvy5bdsdm0000gn/T/spark-1831a94f-7cdd-4691-a32a-4cbc3d9bc3a5/userFiles-66cd46aa-619d-41cf-806d-3dc6e509e13d/databricks_tensorframes-0.2.3-s_2.10.jar/tensorframes/core.py in _auto_placeholder(df, col_name, tf_name, block)
331
332 def _auto_placeholder(df, col_name, tf_name, block):
--> 333     info = _java_api().extra_schema_info(df._jdf)
334     col_shape = [x.shape() for x in info if x.fieldName() == col_name]
335     if len(col_shape) == 0:
/private/var/folders/_f/fzxfs9dj33j0381wvy5bdsdm0000gn/T/spark-1831a94f-7cdd-4691-a32a-4cbc3d9bc3a5/userFiles-66cd46aa-619d-41cf-806d-3dc6e509e13d/databricks_tensorframes-0.2.3-s_2.10.jar/tensorframes/core.py in _java_api()
28     # You cannot simply call the creation of the the class on the _jvm due to classloader issues
29     # with Py4J.
---> 30     return _jvm.Thread.currentThread().getContextClassLoader().loadClass(javaClassName)
31         .newInstance()
32
/usr/local/spark/python/lib/py4j-0.10.1-src.zip/py4j/java_gateway.py in call(self, *args)
931         answer = self.gateway_client.send_command(command)
932         return_value = get_return_value(
--> 933             answer, self.gateway_client, self.target_id, self.name)
934
935         for temp_arg in temp_args:
/usr/local/spark/python/pyspark/sql/utils.pyc in deco(_a, *_kw)
61     def deco(_a, *_kw):
62         try:
---> 63             return f(_a, *_kw)
64         except py4j.protocol.Py4JJavaError as e:
65             s = e.java_exception.toString()
/usr/local/spark/python/lib/py4j-0.10.1-src.zip/py4j/protocol.py in get_return_value(answer, gateway_client, target_id, name)
310                 raise Py4JJavaError(
311                     "An error occurred while calling {0}{1}{2}.\n".
--> 312                     format(target_id, ".", name), value)
313             else:
314                 raise Py4JError(
Py4JJavaError: An error occurred while calling o41.loadClass.
: java.lang.NoClassDefFoundError: org/apache/spark/Logging
at java.lang.ClassLoader.defineClass1(Native Method)
at java.lang.ClassLoader.defineClass(ClassLoader.java:800)
at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:142)
at java.net.URLClassLoader.defineClass(URLClassLoader.java:449)
at java.net.URLClassLoader.access$100(URLClassLoader.java:71)
at java.net.URLClassLoader$1.run(URLClassLoader.java:361)
at java.net.URLClassLoader$1.run(URLClassLoader.java:355)
at java.security.AccessController.doPrivileged(Native Method)
at java.net.URLClassLoader.findClass(URLClassLoader.java:354)
at java.lang.ClassLoader.loadClass(ClassLoader.java:425)
at java.lang.ClassLoader.loadClass(ClassLoader.java:358)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at py4j.reflection.MethodInvoker.invoke(MethodInvoker.java:237)
at py4j.reflection.ReflectionEngine.invoke(ReflectionEngine.java:357)
at py4j.Gateway.invoke(Gateway.java:280)
at py4j.commands.AbstractCommand.invokeMethod(AbstractCommand.java:128)
at py4j.commands.CallCommand.execute(CallCommand.java:79)
at py4j.GatewayConnection.run(GatewayConnection.java:211)
at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.ClassNotFoundException: org.apache.spark.Logging
at java.net.URLClassLoader$1.run(URLClassLoader.java:366)
at java.net.URLClassLoader$1.run(URLClassLoader.java:355)
at java.security.AccessController.doPrivileged(Native Method)
at java.net.URLClassLoader.findClass(URLClassLoader.java:354)
at java.lang.ClassLoader.loadClass(ClassLoader.java:425)
at java.lang.ClassLoader.loadClass(ClassLoader.java:358)
... 22 more

I have the same issue and using Mac OS El Capitan as well.

Is someone have the solution for this issue ?

To make it work on Mac, clone this repository, and then change the file build.sbt by replacing the line
    "tensorflow-0.8.0-1.2-macosx-x86_64.jar" // This is not the main target, excluding
with
tensorflow-0.8.0-1.2-linux-x86_64.jar
It bundles the tensorflow native libraries for mac (instead of linux).
Then follow the developer's instruction in README, with the update from this PR https://github.com/databricks/tensorframes/pull/61/files.
That should do it.

Any update on this issue?
It seems that this exception is due to the fact that org/apache/spark/Logging class has been removed from Spark since 1.5.2. Is there any reason why tensorframe is taking a dependency on this library?
Include the stackoverflow link that explains the issue:
http://stackoverflow.com/questions/40287289/java-lang-noclassdeffounderror-org-apache-spark-logging

@yanji84 this issue should be corrected with the latest published release (0.2.5-rc1). Can you give it a try? Note that the support for Spark 1.6 has been removed (there has been too many changes in the developer API to be able to maintain compatibility between the two releases).

I just published a new version (0.2.5-rc2) that should work on some macos configurations. It seems there is no clear pattern, it is an issue upstream with javacpp-tensorflow.

Ok. I am using Spark 2.1. I built tensorframes from source (latest version as of today) on my mac an the assembly jar seems to contain the lazylogging class.
jar tf tensorframes-assembly-0.2.8.jar |grep log
com/typesafe/scalalogging/
com/typesafe/scalalogging/slf4j/
com/typesafe/scalalogging/BaseLogger.class
com/typesafe/scalalogging/Logger.class
com/typesafe/scalalogging/Logging.class
com/typesafe/scalalogging/slf4j/LazyLogging$class.class
com/typesafe/scalalogging/slf4j/LazyLogging.class
com/typesafe/scalalogging/slf4j/Logger$.class
com/typesafe/scalalogging/slf4j/Logger.class
Not sure why I am still getting this error -
Py4JJavaError                             Traceback (most recent call last)
 in ()
4     # The TensorFlow placeholder that corresponds to column 'x'.
5     # The shape of the placeholder is automatically inferred from the DataFrame.
----> 6     x = tfs.block(df, "x")
7     # The output that adds 3 to x
8     z = tf.add(x, 3, name='z')
........
.............
..................
Py4JJavaError: An error occurred while calling o46.loadClass.
: java.lang.NoClassDefFoundError: com/typesafe/scalalogging/slf4j/LazyLogging
at java.lang.ClassLoader.defineClass1(Native Method)
at java.lang.ClassLoader.defineClass(ClassLoader.java:763)
at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:142)
at java.net.URLClassLoader.defineClass(URLClassLoader.java:467)
at java.net.URLClassLoader.access$100(URLClassLoader.java:73)


alias and proxy conflict

Describe the bug
alias config:
alias: [{ find: '/@', replacement: path.resolve(__dirname, './src') }],

proxy config:
server: {
    proxy: {
        '/api': {
            target: env.VITE_APP_BASE_API,
            changeOrigin: true,
            rewrite: (path: string) => {
                return path.replace('/api', '');
            },
        },
    },
},

if import local file like this
import x from '/@/api/somepath';

console will throw error: GET http://localhost:3000/api/somepath.ts 404
Reproduction
System Info

vite version: 2.0.0-beta.30
Operating System: macos
Node version: 12.x
Package manager (npm/yarn/pnpm) and version: yarn

Logs (Optional if provided reproduction)

because path.replace('/api', ''); will replace /api in /@/api/somepath,  use path.replace(/^\/api/, '') that only match the beginning of the string.

because path.replace('/api', ''); will replace /api in /@/api/somepath,  use path.replace(/^\/api/, '') that only match the beginning of the string.

I also tried, the result is the same.

I also tried, the result is the same.

It works. Can you provide a link to a repo that can reproduce the problem?

It works. Can you provide a link to a repo that can reproduce the problem?

after compared with @vitejs/app,I removed root,base in vite.config.ts ,then problem fixed,thanks!

after compared with @vitejs/app,I removed root,base in vite.config.ts ,then problem fixed,thanks!


When displaying the empty space, windows terminal sometimes displays it as if it was selected with the mouse

Windows Terminal version (or Windows build number)
10.0.19042.0
Other Software
git bash
Steps to reproduce
It seems to happen more frequently when I call programs that generate lots of lines of text but it can also happen on a simple ls.
Expected Behavior
The terminal doesn't show empty space as if it was selected (or white ?)
Actual Behavior
The terminal sometimes shows empty space as if it was selected. Sometimes every new white space / space after a line return ? becomes white after the bug triggers and you need to shutdown the terminal and start another to get rid of the display bug.

Do you happen to have any more specific repro steps? "It happens sometimes" doesn't really give us a lot to work with.

What types of programs specifically?
Are you resizing the Terminal at all?

"it can also happen on a simple ls" would be a great place to start, if you can get it to happen consistently.
This does look vaguely like #3848 (and the threads that are dupe'd to that one), but it's almost impossible to say for sure.

The program doesn't seem to matter.
I had it happen in vim, with make, with ls or any program really.
I am not resizing the terminal.
I will do more tests. It happens frequently on my machine.
One intuition I want to verify is if the white lines start to appear when the output text reaches the last line of the terminal.

Couldn't get the bug to happen yesterday. I'll retry this weekend.

Couldn't get the bug to happen yesterday. I'll retry this weekend.

Hi @grandemk I am also seeing this issue on the windows terminal. Were you able to solve this? I have to clear the screen each time this happens.


PowerRename - Apply Button Option

Description of the new feature / enhancement
It would be nice if there were a setting that specified the default option on the "Apply" button. Either that or allow it to remember what option I used last.
Scenario when this would be used?
Almost all of the time for me.
Generally speaking, I'll operate on a bunch of files, click "Apply and Close" and that's it.
Supporting information
None at all. This is just something I'd like that would make a useful tool even better

/dup https://github.com/microsoft/PowerToys/issues/14852


Test PHP 7 on Travis

It would be nice to know whether this library also supports PHP 7.

Thanks!


[FEAT]: Allow for user to modify context window size after supplying a suitable default.

What would you like to see?
I followed another thread that looked as if it were closed already, requesting to allow to set a context size per workspace.  I am currently using the Windows version of AnythingLLM (version 1.6.7) and notice that when I create new workspaces, set to different models, I cannot set a context Window size at the workspace level, only in the main settings.  This is contrary to what I had seen in earlier videos showing AnythingLLM where it looked as if the context window size could be set for the workspace.
I surmise AnythingLLM looks at model properties for convenience, as Ollama provides model information that can identify the context window size. Still, I've run into a problem, specifically with DeepseekCoder v2, where the context size in the Ollama model file does not match what it supports.  I ran across some documentation that mentioned this was the case for the smaller models trained with fewer parameters.
If the context window is inferred from model parameters, I would like to see a context window size still provided for the workspace, whose settings change when a different provider/model is selected, populating it with the parameters that it can infer, but still allow the user to provide an override value for the context window to overcome models that identify themselves as supporting a larger context size, but in actually, do not.

Context window has never been set on the workspace level, it is a model property so it would not make sense to tie it directly to a workspace.
With Ollama specifically, the context window can be configured from the  LLM selection.

However, this issue reads very odd to me. Is the request here to add a context window size override based on the model selection since you can select a model per workspace - which could indeed result in the model running on a workspace having a different context from the selected default LLM?
Just trying to better understand the ask

Hi @timothycarambat, I think I'm trying to convey the idea of allowing MaxTokens configurable per unique provider and model.  So for example, if I'm using the Ollama provider, and Ollama has three models downloaded and available for it), then the MaxTokens is configurable per model.  If I'm not mistaken (and of course, this is specifically in the case of Ollama models), metadata in the model file provides information about the context size.
I can see why setting it for a workspace is impractical as the same model can be used in different workspaces.  However, I have my MaxTokens set to 128k, but I am using some models that only have an 8k context window.  My concern is that setting the MaxTokens that large is problematic when I have two different workspaces set up with Ollama as the provider but using two other models.
In some cases, I might want to use a lower MaxTokens for certain models, but then use the largest possible for those that support it.  Hopefully, that made a little more sense.


Implement CannonJS sleep API

See https://github.com/donmccurdy/aframe-physics-system/pull/14

Implemented by @InfiniteLee in #125 for the AmmoJS driver. I don't intend to include a CannonJS version, which ran into trouble last time I tried.


add dwarven rock cake to menu swapper

can we add the dwarven rock cake to the menu swapper? so guzzle can be made the left click/first option?
thanks for reading

Use shift click customization https://github.com/runelite/runelite/wiki/Menu-Entry-Swapper#shift-click-configuration

Duplicate of #1860


Mini-Voting-system

Create a readme.md which describes your project

Added it

Congrats on making your first PR successfully 🥳

Pls add hactoberfest tag in it...
On Mon, Oct 10, 2022, 2:55 PM Arijit Ghosh @.***> wrote:

Congrats on making your first PR successfully 🥳
—
Reply to this email directly, view it on GitHub
https://github.com/Rancho2002/C-projects/pull/2#issuecomment-1273030876,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2GMBZCTQOKBCSLWYKWUV73WCPOJ3ANCNFSM6AAAAAARBEO63U
.
You are receiving this because you were assigned.Message ID:
@.***>

Congrats on making your first PR successfully 🥳

Thank you 😊


Mirror hapi reply behavior for hydrate callback

I think it would be useful if the signature of the hydrate callback worked similar to the hapi reply callback. I believe it would be more consistent in the hapi eco-system, especially if you are upgraded a standard reply handler to be a tacky hydrate function.
That is to say, rather than callback(err, res) if you tested the first arg to see if it is an error object, rather than just treating it as an error.
I think there are a few other smarts in the hapi reply, promises, and so forth, and closest parity to those would be great (maybe there's even an internal function that can be tapped into?).
This is all personal preference -- very useful plugin regardless. Thanks!

I did think about this, but the hydrate function technically isn't replying back to the requestor, it is only hydrating the cache. For that reason, I used the Node standard err, data callback style to keep it simple and consistent the rest of Node.
Glad you find the plugin useful.

Cool, thanks for explaining. Makes sense.

On Jun 12, 2015, at 12:43 PM, Adam Bretz<EMAIL_ADDRESS>wrote:
I did think about this, but the hydrate function technically isn't replying back to the requestor, it is only hydrating the cache. For that reason, I used the Node standard err, data callback style to keep it simple and consistent the rest of Node.
Glad you find the plugin useful.
—
Reply to this email directly or view it on GitHub https://github.com/continuationlabs/tacky/issues/16#issuecomment-111552494.


[Website] Update wiki listing for 3.2.0, and correct header for Ukrainian language

[ ] Update to 3.2.0
[ ] Correct local name for Ukrainian Українська
[ ] Move Chinese to the end based on ISO code of zh

Listings updated through 3.3.0! https://getindie.wiki/listings/
Thanks for the language/ordering fixes.


feat: vm.confirmContinue cheat code

Component
Forge
Describe the feature you would like
// Prints the provided string to the console, prompts the user to enter Y/N to
// continue execution, and reverts if N is entered, (infoString) => ()
vm.confirmContinue(string calldata info) external;

This cheatcode prints the full string entered and asks you to continue with a Y/N prompt, and reverts if you enter N. The use case here is to confirm deploy parameters in a script in a table or similar before executing the script.
The vm.toString() cheatcodes along with string.concat() make it simple to put the string together. It should support things like \n, or some other common syntax used for aligning lines such as rust's format! macro.
The benefit to making this a cheatcode instead of putting it in forge-std is to avoid requiring ffi for it.
Additional context
No response

Unclear how this should work with forge test since all of them run in parallel and we'd have to block the execution... Makes it super hard to test deploy scritps that use the cheatcode, no?

That's a good point. And now that I think about it, I don't think doing this with ffi would since ffi doesn't open/expose a shell for you.
What if we scope the cheatcode to only work when invoked with forge script? I think that is the best solution, since you don't want/need this in tests, and it's only useful for scripts anyway (though from an implementation perspective I'm not sure how feasible that is).

It's feasible-ish, but still some edge cases with forge script as well, e.g. when using the debugger it should also not work etc. Don't have a general stance on whether the cheatcode is good/useful so will leave that up for discussion, just trying to map out edge cases :cowboy_hat_face:

True good call. Personally this is something we always add to our deploy scripts to add assurances around the deploy parameters, and just the other week we deployed something via forge script with the wrong params partly because we did not have something like this.
But happy to hear other opinions / alternative ideas as well and will let others chime in 🙂

@mds1 Can we not achieve the same result by having the user confirm before broadcast instead? Possibly also using decoded transaction data?

One downside is that if you have e.g. a bug in the config for tx 2, you already sent tx 1 by the time you notice.
But in general I think it's a good idea. Maybe an upfront summary of all decoded txs, like a partial trace, where only the top-level calls of broadcasted txs are shown (or two levels deep for create2 to show the actual contract creation)

@mds1 We already have all of the transactions before we broadcast them, so my idea was that we would display them all decoded so you can inspect them if you want before broadcasting any of them?

Cool, yea I like that idea. So I imagine the UX being something like:

A new flag to enable/disable the "confirm continue" terminal prompt
If enabled, show the decoded txs, perhaps in a view that looks similar to the tx traces? (I'm mostly indifferent as to the actual format here)

For normal txs we can decode like usual.
For create we'd want to decode contract name and constructor args.
For create2 txs, we'll need to show one level deeper (since top-level calldata is a tx to the create2 deployer, which then makes the create2 call).

Does that line up with what you're thinking?

Yep!

Hey, wondering if there's been any progress on this issue. If no one is actively working/planning to work on this, I could take a crack at it, though this would be my first issue in Foundry so a code pointer or two would be much appreciated.

Feel free to take a stab at it :)

@Dendrimer I unassigned for you now, but if you do still plan to work on this let me know and I can reassign you

Hello, just adding my two 2ct here.
We're building lot's of scripts and utilize logs assertions and even some ffi to give users a clear picture of what they are going to submit before actually doing it.
While this works fine with ppl using ledger (as the flow is stuck till they confirm) we noticed especially ppl using pk mess things up, as there's no confirmation needed. Even if they see sth is wrong it's usually to late (ofc, there's dry run etc, but that's not the point here).
Therefore i'm wondering if instead of having a vm.confirmContinue cheatcode it would make more sense to just have a --requireConfirm on the script command which would force you to enter before every tx broadcast?
I assume ppl either always or never want this, so using a cheatcode seems a bit overkill to me.

Yea the title is outdated, the current UX that was agreed on is a flag as described in this comment: https://github.com/foundry-rs/foundry/issues/2399#issuecomment-1212485018

@mds1 should we close this considering that we now have vm.prompt* cheats?


Dashboard: Import or upload an Open API file

As a user I would like to be able to upload my Open API file so that it is easier, faster and there is less chance of making a mistake by copying and pasting text.
Upload file by drag and drop or selecting.
UX Design:
https://www.figma.com/file/wecNPLnwPgaQiZaXDWolSa/Kusk-Gateway-Dashboard?node-id=1894%3A37635

blocked by https://github.com/kubeshop/kusk-gateway/issues/678

https://github.com/kubeshop/kusk-gateway-dashboard/pull/225


[SPARK-32072][CORE][TESTS] Fix table formatting with benchmark results

What changes were proposed in this pull request?
Set column width w/ benchmark names to maximum of either

40 (before this PR) or
The length of benchmark name or
Maximum length of cases names

Why are the changes needed?
To improve readability of benchmark results. For example, MakeDateTimeBenchmark.
Before:
make_timestamp():                         Best Time(ms)   Avg Time(ms)   Stdev(ms)    Rate(M/s)   Per Row(ns)   Relative
------------------------------------------------------------------------------------------------------------------------
prepare make_timestamp()                           3636           3673          38          0.3        3635.7       1.0X
make_timestamp(2019, 1, 2, 3, 4, 50.123456)             94             99           4         10.7          93.8      38.8X
make_timestamp(2019, 1, 2, 3, 4, 60.000000)             68             80          13         14.6          68.3      53.2X
make_timestamp(2019, 12, 31, 23, 59, 60.00)             65             79          19         15.3          65.3      55.7X
make_timestamp(*, *, *, 3, 4, 50.123456)            271            280          14          3.7         270.7      13.4X

After:
make_timestamp():                            Best Time(ms)   Avg Time(ms)   Stdev(ms)    Rate(M/s)   Per Row(ns)   Relative
---------------------------------------------------------------------------------------------------------------------------
prepare make_timestamp()                              3694           3745          82          0.3        3694.0       1.0X
make_timestamp(2019, 1, 2, 3, 4, 50.123456)             82             90           9         12.2          82.3      44.9X
make_timestamp(2019, 1, 2, 3, 4, 60.000000)             72             77           5         13.9          71.9      51.4X
make_timestamp(2019, 12, 31, 23, 59, 60.00)             67             71           5         15.0          66.8      55.3X
make_timestamp(*, *, *, 3, 4, 50.123456)               273            289          14          3.7         273.2      13.5X

Does this PR introduce any user-facing change?
No
How was this patch tested?
By re-generating benchmark results for MakeDateTimeBenchmark:
$ SPARK_GENERATE_BENCHMARK_FILES=1 build/sbt "sql/test:runMain org.apache.spark.sql.execution.benchmark.MakeDateTimeBenchmark"

in the environment:

Item
Description

Region
us-west-2 (Oregon)

Instance
r3.xlarge

AMI
ubuntu/images/hvm-ssd/ubuntu-bionic-18.04-amd64-server-20190722.1 (ami-06f2f779464715dc5)

Java
OpenJDK 64-Bit Server VM 1.8.0_252 and OpenJDK 64-Bit Server VM 11.0.7+10

Test build #124415 has started for PR 28906 at commit f0598b5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/29034/
Test PASSed.

Test build #124415 has finished for PR 28906 at commit f0598b5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/124415/
Test PASSed.

@srowen @dongjoon-hyun Please, review this PR.

Test build #124428 has started for PR 28906 at commit e1a242e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/29047/
Test PASSed.

Test build #124428 has finished for PR 28906 at commit e1a242e.

This patch fails build dependency tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/124428/
Test FAILed.

jenkins, retest this, please

Test build #124429 has started for PR 28906 at commit e1a242e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/29048/
Test PASSed.

Test build #124429 has finished for PR 28906 at commit e1a242e.

This patch fails build dependency tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/124429/
Test FAILed.

jenkins, retest this, please

Test build #124432 has started for PR 28906 at commit e1a242e.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/29051/
Test FAILed.

Test build #124432 has finished for PR 28906 at commit e1a242e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/124432/
Test PASSed.

thanks, merging to master!


[do not merge yet] add missing volume reference for ssh key

Addresses https://github.ibm.com/IBMPrivateCloud/roadmap/issues/56077#issuecomment-50876533

/assign ashank07


Allow specifying custom node_modules path

I have a custom fork of the monaco editor in my own npm scope which resides at a different node_modules path after installing and I would like to use this plugin without forking this plugin too :)

You can use monacoEditorPath for that.
https://github.com/microsoft/monaco-editor/blob/4a80f8e85250b80d44d2249076ba5afba6721db5/webpack-plugin/src/index.ts#L154


refactor evaluation_metrics_plot methods

Currently, evaluation_metrics_plot takes either a Dict of plot_data, or the ingredients necessary to build this dict. In the first case, it returns only a figure; in the second case, it returns the dictionary along with a figure.
This PR refactors thing so that evaluation_metrics builds the Dict (without doing any plotting), and evaluation_metrics_plot plots it. The old method which both built the dict and plotted it is deprecated in this PR.
This should be entirely non-breaking, but allow downstream users to access the plotting dict without needing to plot, and streamline things to be a bit more coherent (in my opinion).

Percy diff is the opacity of the legend border, which I think is an upstream fix, so I approved it.


Hi @jonmann20, what's your use case of shutting off router? Not sure if this can help, to make a link not managed by router, you can use the router-ignore attribute, e.g. XXX.

Hello, thanks for the quick response.  I am writing an abstract login component that changes the URL, but I need to give control of routing back to the application once login is successful.
In this case ignoring links won't work as I need a different clientside router in place.

I've tried .unsubscribe(), but I still get path not found errors.

The unsubscribe() method should do exactly what you describe here. If it does not, it sounds like a bug. Could you please create a small example showing the issue?
[You could use an online IDE like StackBlitz or Glitch and start by forking an example Vaadin Router project]

So I'm not sure this is actually an issue.  In my original code I was also using removeRoutes() and a service worker (via es-dev-server) was still serving the old files making it look like the router was still active.
Here is a glitch to show what is happening.  With the removeRoutes() preceding unsubscribe() an error is thrown.
[Vaadin.Router] Page not found (/demo.html)

https://glaze-water-bread.glitch.me/demo.html
https://glitch.com/~glaze-water-bread   (home-view.js#L22)

I will close this ticket for now, we can reopen it later on if the problem is confirmed from the router.


CORS problem in signup page and validation in form

There is a cors problem in the backend by which the signup page is not working. I can fix this issue in the backend and also put validations in both the front and backend for proper functioning. You can assign this issue to me under GSSOC'23

@tushar-glitch , go ahead

@tushar-glitch any updates ?


chore: Provide a release retry script

Apologies for the slowness here. This PR syncs the release scripts with my changes upstream that implemented a "retry release" action, which I finally got a chance to test end-to-end with a real release failure on another repo.
Usage: If a release fails partway through, run the retry action, giving it the pull request number of the release pull request, and it will rerun through the release jobs and attempt to complete anything that didn't succeed the first time. (If the retry also fails, it can be rerun.)
Closes #543

@fbogsany You were asking for this capability. Okay to add it?

@fbogsany You were asking for this capability. Okay to add it?


Array initialization

Hey Mike,
I noticed that the headerArr is not being initialized and it's giving an error here: https://github.com/marklogic/mlphp/blob/develop/api/MarkLogic/MLPHP/RESTClient.php#L253-L257

Fixed by https://github.com/marklogic/mlphp/pull/21


The link in the tutorial subsection "Using the QtCreator IDE" has a page not found error.

Hello,
When I tried the link "QtCreator Usage" in the tutorial. I got an error "404 file not found" error. Can some one fix this problem?
Thanks,
Rong

In the meantime you can see the MD document for it.
https://github.com/Open-TEE/Open-Tee.github.io/blob/master/_posts/2015-10-10-qtcreator.md

Qt version 4.8
Ubuntu 16.04 TLS

The Ubuntu I am using without Qt installed. By following the guide I had following errors:
void DBusMenuExporterPrivate::addAction(QAction*, int): Already tracking action "Follow Symbol Under Cursor" under id 173
The cause of the problem is Qt SDK is missing. Even the Qt SDK gets installed afterward, the path of some *.prl files are installed in the Qt folder. But the QtCreator is searching in different folder. The way I fix the problem is to remove the QtCreator and install the Qt SDK. After that re-install QtCreator again.
Removing the QtCreator by running the command
./MaintenanceTool
from the Qt install folder.
After QtCreator is uninstalled. The Qt SDK need to be installed by using the following command.
sudo apt-get install qt-sdk
After the SDK is installed. I followed the guide to re-install the QtCreator. Then everything is fine.


Commands alternative in sparsey 0.7

Hi 👋 I'm interested  in the sparsey 0.7 strategy for the Commands removal :)
Questions
Sumamry of my understandings:

[x] Inserting resources from a system: use Resources
[x] Creating an Entity: use Entities::create
[ ] Inserting components to an Entity: ?

Entities usage example?
EDIT: World::maintain in the sync point of atomic Entity creation, and using the Entity returned by Entities::create before World::maintain might confuse sparsey. Adding note about that on Entities::create would be desirable.
Here is a misleading Entities use example:
use sparsey::prelude::*;

#[derive(Clone, Copy, Debug)]
struct Position(i32, i32);

fn main() {
    let mut world = World::default();
    world.register::<Position>();

    let x = world.borrow_entities();
    let entity = x.create();

    world.insert(entity, (Position(1, 1),));

    // panic (len = 0)
    assert_eq!(world.borrow::<Position>().len(), 1);
}

Alternative to Commands::insert_components?
When we spawn Entity in a system, we often want to add components to the entity:
fn spawn_game_ui(entities: Entities) {
    let ui = entities.create();
    // insert components to the UI entity
}

We would even want to create entity relationships:
fn spawn_game_ui(entities: Entities) {
    let ui = entities.create();
    insert_delayed(ui, Node { .. });

    let hud = entities.create();
    insert_delayed(hud, Node { .. });

    add_child_node_delayed(hud, ui);
}

Is there any strategy for them without Commands as of today?
Note on other crates

shipyard does not have Layout and ViewMut implements AddComponent without requiring the whole &mut World.
bevy is very similary to sparsey 0.6 if I understand correctly.

Hi! The Commands system parameter was removed because each issued command created a separate heap allocation and I did not want people to rely on such an inefficient feature. My preferred alternative is to create a "spawner" resource that holds the entities created with Entities::create and the components that should be added to them, and system that takes the components from the "spawner" resource and adds them to the World.
use sparsey::prelude::*;

#[derive(Clone, Copy, Debug)]
struct Bullet;

#[derive(Clone, Copy, Debug)]
struct Position(f32, f32);

#[derive(Default)]
struct BulletSpawner {
    bullets: Vec<(Entity, (Bullet, Position))>,
}

fn spawn_bullets(entities: Entities, mut bullet_spawner: ResMut<BulletSpawner>) {
    for _ in 0..3 {
        bullet_spawner.bullets.push((entities.create(), (Bullet, Position(0.0, 0.0))));
    }
}

fn maintain_spawns(world: &mut World, resources: &mut Resources) {
    let mut bullet_spawner = resources.borrow_mut::<BulletSpawner>().unwrap();

    for (entity, components) in bullet_spawner.bullets.drain(..) {
        world.insert(entity, components);
    }
}

fn print_bullets(bullets: Comp<Bullet>, positions: Comp<Position>) {
    (&positions).include(&bullets).for_each_with_entity(|(entity, position)| {
        println!("{:?} - {:?}", entity, position);
    });
    println!();
}

fn main() {
    let mut schedule = Schedule::builder()
        .add_system(spawn_bullets)
        .add_barrier_fn(maintain_spawns)
        .add_system(print_bullets)
        .build();

    let mut world = World::default();
    schedule.set_up(&mut world);

    let mut resources = Resources::default();
    resources.insert(BulletSpawner::default());

    for _ in 0..3 {
        schedule.run(&mut world, &mut resources);
    }
}

World::maintain in the sync point of atomic Entity creation, and using the Entity returned by Entities::create before World::maintain might confuse sparsey. Adding note about that on Entities::create would be desirable.

I agree. The documentation should state that the changes are only applied after a call to World::maintain.

Oooh spawner sounds like a simple, straight-forward solution. I'm pretty sure you're developing something secret with sparsey ;) Thanks!


Fix broken tests

I forget that the old webcomponents lib was referenced in the tests. Commit 5944a21 renamed the file and the index.html was not updated.

Once merged it should fix the test page: http://newhaven.io/js/tests/


Feature/animation: add animation

Description
Please provide a brief summary of the changes. If it is related to an issue, please bring the issue number, like fixes #${number}.
Type of change
Please select the relevant option.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Codecov Report

Merging #143 (11a9e1a) into master (9626155) will decrease coverage by 0.56%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #143      +/-   ##
==========================================
- Coverage   90.78%   90.22%   -0.57%
==========================================
  Files         155      146       -9
  Lines        7377     6494     -883
  Branches     2041     1829     -212
==========================================
- Hits         6697     5859     -838
+ Misses        559      514      -45
  Partials      121      121

Impacted Files
Coverage Δ

packages/tuya-panel-utils/src/time.ts

packages/tuya-panel-utils/src/json.ts

packages/tuya-panel-utils/src/temperature.ts

packages/tuya-panel-utils/src/core.ts

packages/tuya-panel-utils/src/number.ts

packages/tuya-panel-utils/src/index.ts

packages/tuya-panel-utils/src/ratio.ts

packages/tuya-panel-utils/src/color.ts

packages/tuya-panel-utils/src/string.ts

Continue to review full report at Codecov.

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


Enable better authentication mechanism for CouchDB

Currently, Peer uses the username and password authentication for the CouchDB authentication.
We should be looking at improving the authentication mechanism.
CouchDB supports the following types of authentication.

Basic
Cookie
Proxy
JWT
https://docs.couchdb.org/en/3.2.0/api/server/authn.html
We should be looking for enhancing the authentication by having other mechanism apart from Basic.

Hello team, I am working on this issue. Requesting you to assign the same to me.


ReDoS attack with inline code blocks

Our users found another ReDoS, this time with inline code blocks.
Here's a repro:
const d = '`                                                                                ';
const start = Date.now();
SimpleMarkdown.defaultInlineParse(d);
console.log(`${Date.now() - start}ms`);
> 3936ms

With nested parsing intermixed with other markdown syntax this can trigger very long parses (upwards of 30seconds for us) since this regex gets triggered again and again.

@night thanks for reporting! I’ll do my best to fix this asap.

@night this should be fixed in version 0.5.2 which i just published. Please let me know if you find a case that fix isn't working for.
I also did a quick pass over the other regexes, and found + fixed a redos in del (strikethrough, which I believe y'all use), and in headings & code fences (which I believe y'all don't use). So hopefully that staves off these issues for a while at least.
Hope this wasn't causing problems for too long! Please send my best to the folks at discord!

@ariabuckles Thanks for the quick fix. I will update our fork, test this out, and throw this up on our canary release to see what happens. If you have some time we'd also love to get https://github.com/Khan/simple-markdown/pull/66 in to get off the branch on our fork (but no rush, really).

Oops, I lost track of that one; thanks for the bump. I’ll get back to that in the next few days; sorry @FuegoFro!


pprof issue with httprouter

I am using httprouter for my server.

@Vikash082 This happened to me and it was because conflicting read timeouts - you can check a longer description of it here and see it that helps?

@Coccodrillo I changed the write timeout to 31, as suggested in the link you posted. But, it says profile is empty
(pprof) tree
profile is empty
(pprof)

Slightly off topic: I believe you can avoid hardcoding all those handlers by using this: router.Handler(http.MethodGet, "/debug/pprof/:item", http.DefaultServeMux) (where :item serves as a dummy placeholder variable which pprof will ignore).


Fix linting to use AirBnB instead of Prettier

The difference between prettier and airbnb is causing me pain, but chainging it will touch every file and be a huge headache.

done in #29


Deprecated implementations

This pulls out guards for safariwatir, celerity & deprecated versions of IE.

:+1:

Merged - 7cfe0f605f2bfcc84964c817f2383c86874183ae


Invalid HTTP_HOST header

I used raven package in Django before and it worked well so far.
Recently I replaced it with latest sentry-python and getting weird error notifications continuously.
Invalid HTTP_HOST header: 'server_ip'. You may need to add 'server_ip' to ALLOWED_HOSTS.

I added server domain, <IP_ADDRESS> and localhost in allowed hosts.
I think there is no direct request to server IP instead of the domain.
Any help to fix the issue is appreciated.

Where are you getting that error? In your project or when submitting events from the SDK to Sentry?

Sentry is integrated in my django project and the same event from the project pushed to sentry.
Sometimes it also produces events with example.com and <IP_ADDRESS> instead of server_ip.

Could you link me the sentry issue that shows this error?

https://sentry.io/share/issue/8d076660ee2d4009b734d0d594ab0288/

Yeah this came up a few times before. I think we should just start ignoring that logger, for now you can do:
from sentry.integrations.logging import ignore_logger
ignore_logger("django.security.DisallowedHost")

Thanks.
But I cannot ignore all errors related to DisallowedHost.
I have to track some of those.
Not sure why there are example.com and other strange server ips.

Because those errors happen. Somebody is sending this as Host header. I am not sure by which criteria you want to filter.

Is it possible to ignore errors with certain host headers in project?
I don't want such invalid errors to fill my sentry event count.

Yes, you can use the before_send option to look at the event that is about to be sent and return None to throw the event away.

Ok, let me try. Thanks.

Another option is to use the "message filters" option in the project settings.

I'm having the same Issue.
@untitaker if i choose the "message filters" it will count at the sentry messages per month?

@camilonova no, you will not be billed for those events

@camilonova It's better to resolve it on web server such as nginx or apache instead of ignoring such issues.

I got this nginx configuration:
server {
    listen                      *:80 default_server deferred;

    server_name                 _ "";

    return                      444;
}

It is supposed to catch them but seems is not working. This issue started when migrated from raven to sentry-sdk.
@discover59 how did you do?

I do not remember including any "example.com" request in the code or in a script (even tried to find one)
Is there a possibility this is related to being on a shared cloud service such as AWS' ECS?
Could you provide an explanation of why this happens? Are these caused due to django nature, sentry bug?
Another example of error message
Invalid HTTP_HOST header: '<IP_ADDRESS>'
On /latest/dynamic/instance-identity/document
I am pushing a new nginx image to my repo to check @camilonova 's tip

wasted some hours on investigating what the hell is going on with these, i also get weird example.com errors
I do not remember including any "example.com" request in the code or in a script (even tried to find one)
Is there a possibility this is related to being on a shared cloud service such as AWS' ECS?
Could you provide an explanation of why this happens? Are these caused due to django nature, sentry bug?
Another example of error message
Invalid HTTP_HOST header: '<IP_ADDRESS>'
On /latest/dynamic/instance-identity/document

"AWS Security Scanner", sounds like AWS employs a bot that is used to figure out if you're accepting random host headers. You're not, because you're using Django.
I wasn't aware of those types of scanners. The initial thought of reporting those types of messages to Sentry was that you'd catch broken configuration that way (requests from your own services with invalid host headers). I guess as long as it's easy enough to filter those events using either before_send or server-side message filters it's alright though.
Let me know your opinions.

One thing we could do is to add AWS Security Scanners to our list of web crawlers that we offer a filter for: https://docs.sentry.io/accounts/quotas/#inbound-data-filters

We decided not to swallow those errors but rather filter out false positive errors like AWS Security Scanner. see also https://github.com/getsentry/relay/pull/577

@untitaker im not the original issue creator, but thanks for this :)

Updated nginx configuration and it fixed this issue.
In addition to snippet in https://github.com/getsentry/sentry-python/issues/641#issuecomment-609054574, I had to add the configuration for 443 as well.
server {
    listen                      80 default_server;
    listen                      [::]:80 default_server;
    listen                      443 ssl default_server;
    ssl_certificate /etc/letsencrypt/live/example.com/fullchain.pem; # managed by Certbot
    ssl_certificate_key /etc/letsencrypt/live/example.com/privkey.pem; # managed by Certbot
    return                      444;
}

from sentry_sdk.integrations.logging import ignore_logger

ignore_logger("django.security.DisallowedHost")

Yeah this came up a few times before. I think we should just start ignoring that logger, for now you can do:
from sentry.integrations.logging import ignore_logger
ignore_logger("django.security.DisallowedHost")

All the above gives me the impression this is actually, 'literally', to do with Logging. Not message-sending.
I'll dig a bit deeper.


Entity.set(or get, unsure)ExtraData() doesn't appear to work

My mod relies on setExtraData and getExtraData to filter the spawns it has made to natural spawns (preventing regular mobs spawning) and there is a function in the mod that kills all the currently alive mod zombies (ignoring any regular zombies that may have spawned in the tick before being removed), however it does not kill any zombies. This means that it cannot either get the extradata or set it. This worked back in 0.15 so it's a new issue, if you could fix it that'd be awesome :)
Edit: the extradata tags are identical, so that's not the issue either

I could never get this to reproduce: can I see a script?


Handle Interactive Comics

Some comics aren't just images, they might be interactive pages or GIFs or some other stuff that we need to handle accordingly. First we need to investigate what these comics are and how to handle them. We should also use this area to keep track of any we come across.

#1525 - Emojic :8ball:

http://xkcd.com/1608/
only, this one's image URL is invalid, so it's also breaking the cacher.

Also #1663 - Garden

Click and Drag

Hmm?

Just adding to the list of non-working interactive xkcds.

@Gaelan What number is that?

@mamaral 1110

Current implementation will do for now.


Bump Laravel Nova

In order to Bump Nova, we had to upgrade Laravel to version 7 too.
Outcome:
Tests still pass
No more yellow

@devingray Thank you!!!


[Cosmos] Refactor endpoint discovery and failover

This PR addresses several issues in the current endpoint discovery and multi-region failover process. There are two known bugs:

Preferred locations are not respected
Background refresh of endpoints does not occur

There may be other issues or edge cases not yet encountered. Since 1 is a significant bug, I am largely re-implementing the endpoint discovery code.  This is a high-level spec of the behavior:

The SDK encounters an explicit failover error on a request. These are explicit backend status (403) and sub status (3, 1008) codes that a healthy cosmos service will return when a region has been dropped or configured as read-only. The Cosmos SDK will not failover under other error conditions such as 503, ENOTFOUND, etc
The SDK will immediately refresh its internal list of available endpoints by reloading the database account from the global endpoint.
The SDK will mark the endpoint that returned the error in step 1 as unavailable
The SDK will retry the request against the next available endpoint, giving preference to preferred locations.
If all endpoints are unavailable fall back to the user-supplied endpoint

This PR does not reimplement background refresh of endpoints. It may be added in a future PR.

I can confirm this is fixed.

@dvester Great! Glad to hear it


fixing flickering when placeholder is replaced with dropped element

#771

Coverage increased (+0.4%) to 86.782% when pulling d0ea209d85f3857324f30fcb43d911d9b0aad060 on tkrakowiak:master into 425bed949c4f66cd9c69b022782400556c352358 on lukasoppermann:master.

Great thank you!


Sparsify (prune) models for performance

Orders of magnitude performance increases have been observed: https://github.com/mlcommons/inference_results_v3.0/tree/main/open/NeuralMagic

https://github.com/neuralmagic/deepsparse


[Snyk] Security upgrade rollup from 0.66.6 to 2.79.2

Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

sdk/edgegateway/arm-edgegateway/package.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Cross-site Scripting (XSS) SNYK-JS-ROLLUP-8073097
  144

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting (XSS)

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: security/snyk check is complete. No issues have been found. (View Details)
:white_check_mark: license/snyk check is complete. No issues have been found. (View Details)


Error: {'photo': [ErrorDetail(string='The submitted data was not a file. Check the encoding type on the form.', code='invalid')]}

There's an issue where the server can't recognize the submitted image file from the client as an image file.
I hope this issue will be solved:)
Related branch: https://github.com/AISVisioner/cameraDetector/tree/seiya.reestablish-env

When using base base64 in face_client:
Another error occurred: 'utf-8' codec can't decode byte 0x85 in position 715: invalid start byte
The background:
base64 encodes the binary image file into its formats.
The django server then tries to decode the base64 to save as ImageFile, but django internally can't decode the base64, which is suited for decoding normally encoded files, not base64

I tried using requests with binary image files, not base64 encoding, as following:
files = {       'photo': open(f"./images/{guest_name}.jpg", 'rb')   }
This is how the image file sent to the server was decoded successfully, converted to Inmemoryuploaded file, a file type django automatically converts to when requested.

But the error above Error: {'photo': [ErrorDetail(string='The submitted data was not a file. Check the encoding type on the form.', code='invalid')]} occurred again when using this way.
So, I tried using initiating the ImageFile using the binary InMemoryUploadedFile as a parameter as follows:
ImageFile(open(request.FILES['photo']), 'rb')
This is how I successfully made it possible to send image file to the server and respond properly.
There's now no error in this process.

Now after deleting any instance in database, two errors below occurs,
error - matching query does not exist
expected str, bytes or os.PathLike object, not InMemoryUploadedFile

First error occurs after deleting any instance, where the corresponding field remains in corresponding serializer objects. The query is created according to the serializer objects, where information of the deleted instance is contained.
That's why matching query does not exist.
We have to find a way to stop the serializer from containing the field of a deleted instance.

I still couldn't find the reason why exactly the second error occurs.
But I can solve the error, by changing  the ImageFile to ContentFile, and reversing back the object.
This is a tricky problem.
But you don't have to care about the error by now, when we don't expect an administrator to delete any object.

I solved all of the issues above:

Error: {'photo': [ErrorDetail(string='The submitted data was not a file. Check the encoding type on the form.', code='invalid')]}
--- we have to use serializer.data instead of serializer.validated_data as a parameter of Response object in lookup/api/views.py.

error - matching query does not exist
expected str, bytes or os.PathLike object, not InMemoryUploadedFile

--- changed the location of queryset field of LookupViewSet class
from
class LookupViewSet(viewsets.ModelViewSet):     queryset = Visitor.objects.all().order_by("-created_at")
to
def list(self, request):     queryset = Visitor.objects.all().order_by("-created_at")
def create(self, request):      queryset = Visitor.objects.all().order_by("-created_at")
with which we can update the queryset variable everytime there's a get or post request from a client.
(originally the queryset field was used only as a field of LookupViewset, but now as a variable, not a field of a class)


Move to MUnit and whale-tail

All tests reworked in terms of Munit.
All test (file)names normalised to "suite" from "spec".
Replaced test-containers with whale-tail to allow cross building for JS and native.

After this merges I'll work on the JS and Native cross-builds. Thanks, these changes are a big help towards that!


Add world blacklist/whitelist

This feature is requested by FireFlies. So we need to add a new section to the configuration.
world-blacklist:
  fallback-world: world
  use-as-whitelist: false
  worlds:
    - ANameOfAWorld

Configuration added in 40d0b7244bd3d38229dc31d8f64f3b3785a7630c
Feature added in 962ac7cb75df9ec502e3ea5b84f7c2a15b39aad8


Define IAM Permission Boundary for Project

Description of changes

Adds a new advanced project configuration option to specify a permission boundary that will be applied to all IAM roles in the project. This is broken into 3 components:

add a permission boundary state manager to amplify-cli-core
add a prompt to amplify configure project that writes the boundary to the state manager
adds a permission boundary template modifier to the pre-push cloudformation transformer that reads the value from the state manager and applies it to cfn templates

Issue #, if available

https://github.com/aws-amplify/amplify-cli/issues/4618
Description of how you validated changes
Manually validated as well as unit tested and e2e tested
Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[x] Relevant documentation is changed or added (and PR referenced)
https://github.com/aws-amplify/docs/pull/3157

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #7144 (f5fa7e2) into master (8775152) will decrease coverage by 4.21%.
The diff coverage is 45.12%.

@@            Coverage Diff             @@
##           master    #7144      +/-   ##
==========================================
- Coverage   56.43%   52.21%   -4.22%
==========================================
  Files         445      499      +54
  Lines       21859    25499    +3640
  Branches     4373     4999     +626
==========================================
+ Hits        12336    13315     +979
- Misses       8745    11228    +2483
- Partials      778      956     +178

Impacted Files
Coverage Δ

.../awscloudformation/utils/auth-defaults-appliers.ts
89.65% <0.00%> (ø)

...wscloudformation/utils/verification-bucket-name.ts
42.85% <0.00%> (ø)

packages/amplify-cli-core/src/deploymentState.ts
100.00% <ø> (ø)

...amplify-cli-core/src/feature-flags/featureFlags.ts
83.10% <ø> (ø)

...-cli/src/extensions/amplify-helpers/git-manager.ts
85.18% <ø> (ø)

.../amplify-frontend-ios/lib/amplify-config-helper.js
1.01% <ø> (ø)

...provider-awscloudformation/src/aws-utils/aws-s3.ts
9.32% <0.00%> (-0.77%)
:arrow_down:

...ion/src/iterative-deployment/deployment-manager.ts
7.05% <0.00%> (ø)

...y-provider-awscloudformation/src/resourceParams.js
15.15% <ø> (ø)

packages/amplify-util-mock/src/func/index.ts
94.11% <0.00%> (ø)

... and 114 more

Continue to review full report at Codecov.

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


Update to bevy and leafwing input manager 0.13

Wanted to use this crate for my new project, so decided to help update it. But with the recent removal of methods like ActionLike::variants(), I am not sure how to properly change the charges system. (which is where most errors come from right now)

Awesome, thanks! Preparing a release now.


General report

Proposal
A work in progress.
Checklist

[x] I have made this pull request to the master branch
[x] I have run npm run lint and there are no errors

I think everything works ok for general reports and it looks like the suggested format.
I am not sure if in the navigation dropdown General report should go first or last. Also, the table isn't looking very nice when there are comments on it but that's how the client wants it :shrug:

I think we can add a "Button" like "Reset" before that will hide the feedback column and show it back (Using names like "Hide feedback" and "Show feedback").

Added max-width for that feedback message to take less space and look a bit better. To introduce buttons to hide a whole column we can add later (I don't have a good idea of how it should look).


Chap 1 - explain base concepts

https://www.thoughtspot.com/data-trends/ai/what-is-transformer-architecture-chatgpt

https://openai.com/index/chatgpt/

RLHF
https://openai.com/index/learning-from-human-preferences/

https://bea.stollnitz.com/blog/gpt-transformer/

https://arxiv.org/abs/1706.03762

https://en.m.wikipedia.org/wiki/Transformer_(deep_learning_architecture)

More on:

No-code/Low-code GPTs builder: https://help.openai.com/en/articles/8770868-gpt-builder
GPT store: https://openai.com/index/introducing-the-gpt-store/


Add schema specification files, start of mil2525c_b2 dev + other enhancements

PR Changes:

Add current schema and domain specification files to this repo

We originally weren't sure about moving/copying the domains also but
the GDB GP creation tool assumes that they will be in the same root folder (not sure why) so that made that decision easier -and-
it makes sense to keep the entire specification together with the GDB it specifies and to more closely track changes (since these frequent changes in the other repo have been an issue)

Add toolbox for Editor Tracking and metadata export
Add in-progress mil2525c_b2 military feature classes
Moved "legacy-stylx-utilities" folder under stylx-utilities folder

Schema and domain specifications pulled out of the transient (and sometimes inadvertently deleted) branches in JMSML Repo:

https://github.com/Esri/joint-military-symbology-xml/tree/Esri-implementation-no-subtypes
https://github.com/Esri/joint-military-symbology-xml/tree/Esri-implementation-mil2525c

And moved to the GDB folder of the GDB they specify.
Sorry for the larger PR, but most of this stuff goes together and the schema files (95% of the PR) were already reviewed in the other repo.

@csmoore reviewed commits, everything looks good. Thanks for adding the .tbx for editor tracking and metadata support as well.


nUpdate doesn't wait to install updates until app has closed

Hi,
I want to use nUpdate to update my executable app on launch. Currently have set my instance of the UpdateManager to have a "CloseHostApplication" of "true". However when I run the InstallPackage routine, nUpdate will attempt to overwrite my executable before it's had a chance to close causing it to open a dialog saying the update couldn't be completed.
Is there a way to delay the update from starting for a few seconds to allow time for the app to close?

Hi,
you do not need to set this property to true as it is the default value.
What kind of application are you running? Windows Forms, Console or WPF?
By default, nUpdate will call Application.Exit on your program once the installation process is started. May it be that you have some code in your program that prevents the closing of each form from happening?
Of course, it would be good to know your application type as WPF and console need a small workaround to get this working atm. For that the TerminateApplication method is marked as virtual. You may want to override it and kill your application the way that it needs to be done.
Best,
Dominic

I'm programming in WinForms currently.
The process I can see on screen once the InstallPackage method is called is that an error box pops up saying the executable is locked just as my application closes. This does all occur within the space of a second or so. I don't have any code overriding the shutdown of the app.

Do you have any code in your application that keeps it running even if all forms are closed (e. g. NotifyIcon or any else running services)?
You may want to try creating a class that inherits from UpdateManager and override the TerminateApplication method to kill your process without any resting active resources.
Then use this UpdateManager subclass instead.
Best,
Dominic

Hi @ProgTrade, took a while to get back to you as I'm working on many things at once.
I was able to overcome this issue by shifting the updating code into my Program.cs before my first form can be initialized. It would have been desirable to have included this in my form so I can notify the user that the update is starting but that isn't a big deal as my use case will require all updates to be mandatory anyway and most only delay startup by a few more seconds by my testing.
Thanks for your help!
Hope this assists anyone with a similar issue


consider shorter seasons

I propose to shorten seasons to 1 hour (3600 ticks per season). 120 seasons for the whole trip.

allows to observe full implications of changes in bots faster
makes differences in starting locations more fair
should be enough time to determine winning bots (except for some potential fuckups)
less wasted time in case one bot starts absolutely dominating the map

1 hours seems good


Thumbnails not working in Windows 11.

Describe the bug
Thumbnails (for step files at least) don't work on Windows 11.
(I did not check other file types.)
To Reproduce
Steps to reproduce the behavior:

Installed f3d through convention .exe file from https://f3d.app/doc/user/INSTALLATION.html
Checked that thumb-nailing option is enabled in installation wizard.
F3d works and opens step file

System Information:
Edition	Windows 11 Business
Version	23H2
Installed on	11/22/2023
OS build	22631.2861
Experience	Windows Feature Experience Pack 1000.22681.1000.0
Processor	Intel(R) Core(TM) i9-14900KF   3.20 GHz
[Display]
DirectX version:	12.0
GPU processor:		NVIDIA GeForce RTX 4070
F3D Information
F3D-2.2.1-Windows-x86_64-raytracing

Hi @odyskat
I have no access to a Windows 11 computer at the moment so I cant test this right away.
However I see a lot of posts online about general issues with thumbnails on Windows 11. Can you confirm thumbnaisl are working in general for you ?

Hi @mwestphal and thanks for the quick reply.
I checked for a stl file and
I think f3d did not provide a thumbnail (I see a picture like the one below even when I uninstalled it).
I suppose windows provides the image and not f3d.
Hope it helps.

@Meakk tested with Windows 11 and it seems to work great. can you try reinstalling ?

@mwestphal I did it multiple times, same result.
I will try it at home with another computer for comparison.

I tried in a Windows 10 machine at home with no luck for the thumbnails.
Step files were exported from 2 different cad programs to make sure that the problem is not file related.
Note: I use Greek as system locale, not s long ago a Lenovo tool refused to work if the system locale was not English us.
It was hard to find and probably not connected to the thumbnailing issue.

Maybe it's related to one drive? Can you try on a local disk?

Ok, I tried many cases :
Number 1: win10 folder not connected to OneDrive --> works!

Number 2: Win 11 folder not connected to OneDrive --> works!

Number 3: win11 folder in OneDrive fist time copying of the file works, when closed and reopened it's gone.

Comment : I noticed that when the file is stored in SharePoint (for anyone who hasn't used this, it's like Microsoft's business version of OneDrive) and also saved locally on the computer, thumbnails seem to work fine.

I think OneDrive might remove thumbnails to save space (they are saved in thumbs.db file)
I came across this page: https://answers.microsoft.com/en-us/windows/forum/all/video-thumbnails-not-generating-in-onedrive-folder/bf54402a-2306-444a-a986-4642430f6b95
They are mentioning an option Files On-Demand: Save space and download files as you use them to disable. Can you try that?

I have the same install, and no thumbnails what so ever are showing for step files, but whenI copy the complete folder to the C drive: this happens:

and then the thumbnails appear. one by one.
So, when you have a onedrive, it downloads the thumbnails from onedrive (just do the 'see online' option, and you will see which files have thumbnails, and which not) same for PDF, and others, if not local but onedrive (even if 'always keep on this device' is checked), no thumbnail which MS supports)
You have to sync differently with onedrive.

Regarding the error in the window, it's a new issue.
@mwestphal see https://github.com/f3d-app/f3d/blob/2f95ee30cc1d36f5f522558656926d13ed952dd9/winshellext/F3DThumbnailProvider.cxx#L168

@jeedee23 I think the files without thumbnails are probably the bigger files, can you confirm?

fix incoming, but I think @jeedee23  is using a old nightly.
https://github.com/f3d-app/f3d/pull/1151

Yes, but the new latest, doesn’t give any thumbnails
Van: Mathieu Westphal @.>
Verzonden: Tuesday, 9 January 2024 22:58
Aan: f3d-app/f3d @.>
CC: Johan Degraeve @.>; Mention @.>
Onderwerp: Re: [f3d-app/f3d] Thumbnails not working in Windows 11 with OneDrive (Issue #1129)
fix incoming, but I think @jeedee23 https://github.com/jeedee23  is using a old nightly.
#1151 https://github.com/f3d-app/f3d/pull/1151
—
Reply to this email directly, view it on GitHub https://github.com/f3d-app/f3d/issues/1129#issuecomment-1883863088 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AJJMV5L2TBPMRKOCQQMYQ43YNW4PPAVCNFSM6AAAAABBEDXVMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBTHA3DGMBYHA .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AJJMV5MEW2QV776ALGMHPPLYNW4PPA5CNFSM6AAAAABBEDXVMWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTQJF4DA.gif Message ID: @.*** @.***> >

Hi Michael, yes, bigger
Van: Michael MIGLIORE @.>
Verzonden: Tuesday, 9 January 2024 22:58
Aan: f3d-app/f3d @.>
CC: Johan Degraeve @.>; Mention @.>
Onderwerp: Re: [f3d-app/f3d] Thumbnails not working in Windows 11 with OneDrive (Issue #1129)
@jeedee23 https://github.com/jeedee23  I think the files without thumbnails are probably the bigger files, can you confirm?
—
Reply to this email directly, view it on GitHub https://github.com/f3d-app/f3d/issues/1129#issuecomment-1883863035 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AJJMV5KTZ7G7TVDZ2S6YWY3YNW4PFAVCNFSM6AAAAABBEDXVMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBTHA3DGMBTGU .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AJJMV5LVBYFTD24UGQIMDFLYNW4PFA5CNFSM6AAAAABBEDXVMWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTQJF37W.gif Message ID: @.*** @.***> >

@jeedee23 If you look at your installation directory, you will find a share\f3d\configs\thumbnail.d\10_global.json config file.
This config file sets the max size limit for generating thumbnails. By default, it's 100MB: "max-size":100.
You can increase this value directly here, but it's recommended to create your own "user" config file to avoid an overwrite during a f3d update, please refer to https://f3d.app/doc/user/CONFIGURATION_FILE.html

Yes, but the new latest, doesn’t give any thumbnails

Is it expected @mwestphal?

||   Yes, but the new latest, doesn’t give any thumbnails
| Is it expected @mwestphal?
Definitely not, but we will need actual version numbers from @jeedee23 to move forward. Testing the last nightly on windows would also be a start.

I think OneDrive might remove thumbnails to save space (they are saved in thumbs.db file) I came across this page: https://answers.microsoft.com/en-us/windows/forum/all/video-thumbnails-not-generating-in-onedrive-folder/bf54402a-2306-444a-a986-4642430f6b95 They are mentioning an option Files On-Demand: Save space and download files as you use them to disable. Can you try that?

I don't have enough space in my work comp.
But I tried with one folder, same issue.
I updated to Windows 11, version 23H2. Now, new files downloaded from SharePoint locally do not want to display thumbnails, but files I had downloaded still have thumbnails.

|| Yes, but the new latest, doesn’t give any thumbnails
| Is it expected @mwestphal?
Definitely not, but we will need actual version numbers from @jeedee23 to move forward. Testing the last nightly on windows would also be a start.

F3D-2.2.1-95-g2f95ee3-Windows-x86_64 is the version I use. The only one giving thumbnails for step files, but not for all of them (even if size < 100MB) and it shows the pop-up

I think OneDrive might remove thumbnails to save space (they are saved in thumbs.db file) I came across this page: https://answers.microsoft.com/en-us/windows/forum/all/video-thumbnails-not-generating-in-onedrive-folder/bf54402a-2306-444a-a986-4642430f6b95 They are mentioning an option Files On-Demand: Save space and download files as you use them to disable. Can you try that?

This is an old thread and this option is not present anymore.

I just tested the last nightly and thumbnails are working fine (outside of onedrive which I do not use).
@Meakk did you actually try with onedrive ?

Yes it is a onedrive issue, other apps have the same problem. The files are not stored the usual way, I think there is need for a MS api to be able to do this

@Meakk did you actually try with onedrive ?

No I don't have a onedrive account

You have to sync differently with onedrive.

Ok, looks like we do need to do something else to support onedrive file. Which we will probably not do for now I'm afraid.
Related to: #797

You have to sync differently with onedrive.

Ok, looks like we do need to do something else to support onedrive file. Which we will probably not do for now I'm afraid.
Related to: #797

Yes, that is what I thought, because it is almost impossible.

labelling as wont fix.


TrackSwitchOffMode not showing last frame

Describe the bug
It suggests in the documentation that the last frame will be shown when a track is switched off: https://www.twilio.com/docs/video/tutorials/using-bandwidth-profile-api#track-switch-offs
I have seen that any time a track is switched off it just shows the users' display name instead.

Thanks for the issue @dannnnthemannnn!

the last frame will be shown when a track is switched off

This is correct. When a video track is switched off, the <video> element will continue to display the last frame that was received.
This means that a participant's video will appear to be frozen when the track is switched off. In this app, however, we did not want the participant's video to appear frozen, so we display a black background (and avatar icon) over the <video> element when the track is switched off.
This behavior can be disabled by removing the following lines of code from ParticipantInfo.tsx: https://github.com/twilio/twilio-video-app-react/blob/master/src/components/ParticipantInfo/ParticipantInfo.tsx#L201-L205


Web app hangs when debugging with Rider and using RunOaktonCommands

I've added Marten and Marten.CommandLine v5.11.0 to a .NET6.0 web application.
It runs and debugs fine, using JetBrains Rider 2022.2.3, when the startup is:
public static void Main(string[] args) =>
     CreateHostBuilder(args).Build().Run();

When I switch to:
public static Task<int> Main(string[] args)
{
    return CreateHostBuilder(args).RunOaktonCommands(args);
}

then running is still fine, but when debugging the Swagger web-page it is trying to open isn't displayed.
The browser just displays:

Unable to connect
Firefox can’t establish a connection to the server at localhost:5000.

In Rider, it behaves as if it is hitting an invisible break-point.  Clicking the "Resume Program" button to continue debugging just breaks again immediately at the same place:

@stevemossquill Not sure what to tell you here, but it's going to take a lot more information than that to solve this. Do you have any custom IHostedService registrations? What are you trying to do at start up time? Anything overwriting the port number?
And I very seriously doubt this is directly related to Marten itself

Ok, I'm suitably embarrassed.. this was just down to my Rider debug settings for exceptions:
The RunOaktonCommands extension leads to RegisterCommandsFromExtensionAssemblies which scans all assemblies for the OaktonCommandAssemblyAttribute.
This automatically included scanning all the Microsoft.Code.Analysis.* files that appear in my bin directory.
The "problem" was this tries to load the .pdb file for each DLL, and when not found threw an exception.
In release mode, this exception is just swallowed (correctly), but my exception settings were set to break on any CLR exception.
Once I stopped breaking on CLR exceptions, the app started up fine.


add CI for publishing on the fortran-lang twitter account upon merging a certain PR

Hello everyone!
As discussed in the fortran-lang discourse, this PR implements a CI that:

Generates tweets by commenting on a PR
Published these tweets when the PR is closed

Please have a look, and feel free to test it by submitting a PR in this repo I created: p-costa/piu-piu-sandbox.
If you are happy with it, to get it to work, one needs to create a twitter app, following the steps similar to those indicated here: github.com/gr2m/twitter-together/blob/master/docs/01-create-twitter-app.md and add the corresponding keys and tokens to this repo.
I should thank the author of find-comment for very promptly considering issues #24 and #26.
The tweets are published using send-tweet-action, as per @LKedward suggestion.
It works as follows. The CI will look for a specific keyword ([tweet]) published by the GitHub actions bot, and, if it exists in the PR comment thread, it will be deployed for publication in the fortran-lang twitter account upon merging.

Amazing, great work @p-costa! I'll have a look through this later today.

Great work. I tested it a bit.
Here are my questions:
https://github.com/p-costa/piu-piu-sandbox/pull/1#issuecomment-703616971

Thanks for creating the PR and testing it @certik . I should have mentioned here as well that:
#tweet text will:

remove a previous tweet, if it exists.
add a new tweet to be published by the CI once the PR is merged.

#untweet will remove the tweet.

Thanks, @p-costa! I have 2 questions:

Considering that #tweet text will remove a previous tweet if it exists, that means that there can be only one Tweet per PR, correct?
What happens if text exceeds 280 characters? Do we get a thread of tweets or is the tweet truncated (or something else)?

Hi @milancurcic ,

Correct. I assumed that one will not be tweeting too much from here, so 1 tweet per merged PR sounded reasonable.
The CI always prints a preview of the tweet showing the character count. So if it is exceeded one will see it and can fix it before merging.

I converted this to draft so that I can polish the implementation tomorrow and make some final tests. Feel free to review it in any case, as it should not change much from my side.

Just want to comment here that it's ready. But no rush from my side -- I understand that this being a small convenience shouldn't be a high priority thing.

+1 with me also
On Fri, Oct 9, 2020, at 3:08 AM, Laurence Kedward wrote:

@LKedward approved this pull request.
Great stuff! I don't see any issues with the workflow which has already
been tested and demonstrated. Clearly some work is still required to
link to the twitter account.
It looks like anyone can create and modify a tweet, but only
maintainers can merge and publish the tweet so I think this workflow
works well. Thanks @p-costa https://github.com/p-costa 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fortran-lang/fortran-lang.org/pull/147#pullrequestreview-505495397, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAFAWCPBVYHG75UIKRBAQLSJ3HJHANCNFSM4SESIBIA.

It looks like the homework for me is to set up the app and keys:

Let me know if you run into problems. One thing that was not written in the instructions and I had to do was making sure that the app as read and write permissions:

Okay, this is now done. I set the secrets in the Org and allowed them to be shared with this repo. This way we can easily reuse them in other repos when we want.
Should this now be merged?


CMake FetchContent support?

So that I can use FetchContent to avoid pkg managers.
Like use FetchContent and the library will automatically be setup

You don't add support for FetchContent, it pulls from git. You're still in charge of making sure you have the dependencies that D++ requires. A quick google search would tell you that, see link.
If you want to use D++ via FetchContent, do something like
include(FetchContent)

FetchContent_Declare(dpp
  GIT_REPOSITORY https://github.com/brainboxdotcc/DPP.git
  GIT_TAG v10.0.30
)

FetchContent_MakeAvailable(dpp)

Jaskowicz it is very insensitive of you to say words such as "you" and "it", they don't believe in pronouns!, you're going to make them cry!
...Oh shoot I used pronouns too
Oh no I did it again
Help

Huh

You don't add support for FetchContent, it pulls from git. You're still in charge of making sure you have the dependencies that D++ requires. A quick google search would tell you that, see link.
If you want to use D++ via FetchContent, do something like
include(FetchContent)

FetchContent_Declare(dpp
  GIT_REPOSITORY https://github.com/brainboxdotcc/DPP.git
  GIT_TAG v10.0.30
)

FetchContent_MakeAvailable(dpp)

You can then link to dpp like you would normally.

Ye but I want to statically link to dpp
Does setting
BUILD_SHARED_LIBS to OFF In CMake help?

We do on Linux, and yes that will do it


Android|Java -c compile error

After running mono Embeddinator-4000.exe "path to some xamarin forms based dll" -gen=Java -p=Android -o="some folder" -c
I receive following exceptions:
Failed to read .. dll with debugging symbols. Retrying to load it without it.
warning: unknown enum constant Scope.LIBRARY_GROUP
reason: class file for android.support.annotation.RestrictTo$Scope not found
src\mono\android\support\v7\widget\ShareActionProvider_OnShareTargetSelectedListenerImplementor.java:28: error: cannot find symbol
public boolean onShareTargetSelected (android.support.v7.widget.ShareActionProvider p0, android.content.Intent p1)
src\mono\android\support\v4\view\ViewPager_OnPageChangeListenerImplementor.java:8: error: package android.support.v4.view.ViewPager does not exist
android.support.v4.view.ViewPager.OnPageChangeListener

Hi @ysmoradi,
A couple questions to help figure this out:

Are you using NuGet or building Embeddinator from source?
Can you share your packages.config or your project?

One thing to check is that you aren't using anything API 26+ (such as support libraries v26.x). The current release on NuGet does not support those.

I've downloaded Embeddinator's nuget package, then I ran command in "embeddinator-4<IP_ADDRESS>/tools" folder.
My projects is a xamarin android library which contains "xamarin forms content page" which has simple controls such as stack layout, entry, button and label. In my xamarin.android app, I'm converting that content page to fragment. It's working fine.
I'd like to use my fragment in Java-Android app.
https://github.com/ysmoradi/XamComponent

I know such a simple codes relies on multiple assemblies with deep technical concepts. But could you please tell me how much you think it will increases the apk size of my java app? Thanks in advance.

Yeah, you have API 26 stuff in here, which is why you have the failure. I think we are due for a new release on NuGet, but I don't know when it will be yet.
You can either:

use TargetFrameworkVersion 7.1 and make sure all the support libraries are 25.x
add a submodule for Embeddinator in your repo, and build it from source. This will get you API 26 support that we have on master.

It will increase the size of your app by:

(number of Android abis) x (size of mono ~3.4MB) - you can turn off some of these in gradle, if you only need x86 / armv7, for example
size of all .NET assemblies

Thanks so much. :) Does it link unused IL from assemblies?

Right now it will do the same thing as Link SDK assemblies only, just like a Xamarin.Android app.

@ysmoradi We should be releasing a new version on Nuget soon, in case you don't want to go through the trouble of compiling master yourself.

Thanks so much. I'm waiting for the next release :)


fix: adds accessibility label for ios to match android

Describe your PR
While writing up a recipe for adding Maestro to an Ignite app, I noticed that we added an accessibility label to the android Podcast Card but not for iOS. This is helpful for running e2e tests.

:tada: This PR is included in version 8.5.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


buttons not responding to emulated touch events in chrome devtools

I'm not sure why this is, pointer touch events seem to be working ok but not for touch events on buttons.

Check the pointer is placed correctly in https://samme.github.io/phaser-examples-mirror/debug/debug input.html?

@samme this is for CE. the buttons do work properly on my mobile phone but not in chrome devtools. chrome dev tools allows you to enter an emulated mobile device for testing and normally html5 buttons work in that mode. Are there any testing environments you suggest i use for mobile?

https://samme.github.io/phaser-examples-mirror/buttons/action on click.html is working for me in Chrome's mobile simulator.

@samme am I using the correct thing? this isn't working

That's the same thing I'm using. Is yours Chrome 65? Did you check https://samme.github.io/phaser-examples-mirror/debug/debug input.html?

@samme that does work though it is buggy for different devices, like when i choose the ipad for example, the dot is offset from where i clicked. The dots offset is sometimes non existent, sometimes small, and like on the ipad large.

I think I saw that problem before. Make sure your Chrome is updated to the latest.

awesome that did the trick. I went to the "more" button on the top right of google chrome, the three dots. went down to the 'help' option in the popup menu. chose 'about google chrome' in the submenu. There I saw that my chrome auto updates and that I had the latest update installed, it said i just needed to relaunch chrome. guess i haven't closed chrome in a while
Thanks @samme ! this is really handy to have working obviously


Updated the list of Natives for v2

Let me know if I broke something.
Closes #879

I was not able to set this PR as a Draft, so please wait until I update the rest of them.

Might be a good time to move Natives to the Source and then add access for both v2 and v3 branch to work from one list. The game only needs one set of Natives, it seems crazy to be maintaining 2.

I agree. If you guys find a way to store the natives for both versions I can send a PR to update them.

v2 needs a separate list for backwards compatibility. It still has _0x123456789 entries in the enum that are used in the wild. In v3 we got rid of those in favor of casting. We cannot remove any existing entries in v2 without breaking all scripts that use them. Only add new ones. v3 is still frer from that burden because not officially released, so no backwards compatibility to maintain.

If the Csharp port of V2 is viable we should be able to use the V2 Native Hash list for both V2 and V3.

@Tanjitsu The port works for the most part but has one issue that breaks it currently: https://github.com/crosire/scripthookvdotnet/issues/869#issuecomment-520260653

So, little problem. On SHVDN I have:
_0x2B7E9A4EAAA93C89 = 0x2B7E9A4EAAA93C89,
_0x1CCD9A37359072CF = 0x1CCD9A37359072CF,
_SET_NOTIFICATION_MESSAGE = 0x1CCD9A37359072CF,
_0xC6F580E4C94926AC = 0xC6F580E4C94926AC,
_0x1E6611149DB3DB6B = 0x1E6611149DB3DB6B,

But on Unknown's Native Database I have:
static int _SET_NOTIFICATION_MESSAGE(const char* picName1, const char* picName2, BOOL flash, int iconType, BOOL p4, const char* sender, const char* subject) { return invoke<int>(0x2B7E9A4EAAA93C89, picName1, picName2, flash, iconType, p4, sender, subject); } // 0x2B7E9A4EAAA93C89 0xED130FA1 b323
static int _SET_NOTIFICATION_MESSAGE_2(const char* picName1, const char* picName2, BOOL flash, int iconType, const char* sender, const char* subject) { return invoke<int>(0x1CCD9A37359072CF, picName1, picName2, flash, iconType, sender, subject); } // 0x1CCD9A37359072CF 0xE7E3C98B b323
static int _SET_NOTIFICATION_MESSAGE_3(const char* picName1, const char* picName2, BOOL flash, int iconType, const char* sender, const char* subject) { return invoke<int>(0xC6F580E4C94926AC, picName1, picName2, flash, iconType, sender, subject); } // 0xC6F580E4C94926AC b323
static int _SET_NOTIFICATION_MESSAGE_4(const char* picName1, const char* picName2, BOOL flash, int iconType, const char* sender, const char* subject, float duration) { return invoke<int>(0x1E6611149DB3DB6B, picName1, picName2, flash, iconType, sender, subject, duration); } // 0x1E6611149DB3DB6B 0x0EB382B7 b323

What should be done on this case? How should I name them?

CC @crosire @Tanjitsu

I'd keep them as they were in v2 and fix them in v3 later.

I'm closing this because the merge of the C# version of v2 broke the list of natives and I'm not going to check every single one of them manually again.

Please also note that we actually have an automated process to generate this enum: Run the "NativeGen" target in Visual Studio and it will update the v3 list to the latest version from nativedb.

A couple of problems with that:

It uses NativeDB, that has been outdated for approx. 2 years (one of the reasons why I'm using the native list by @UnknownModder )
The code generated is for v3, not v2

Copy/Pasting the values on v2 is not possible without breaking backwards compatibility

I can update the NativeGen tool to poll from https://github.com/UnknownModder/gta5-nativedb-data/raw/master/natives.json instead of nativedb. But I think it makes sense to keep this process automated as much as possible, since it's a pretty huge list to maintain manually otherwise. Could also change the tool to support some options to generate the appropiate lists for v2 or v3.


Question: Why is the platform limited to amd64 ?

Hey everybody,
I was recently starting using your nice module to inspect my powershell script performance and figured out the module is limited to amd64 processors.
# Processor architecture (None, X86, Amd64) required by this module
ProcessorArchitecture = 'Amd64'

However, I saved the module locally Save-Module psprofiler -Path ~/modules/ and simply comment out the line above and Measure-Script $PROFILE also worked on arm64.
I tried to figure out which line of code limits the module to amd64 but wasn't able to find anything.
I'm quite new to powershell and wouldn't say I'm a good programmer, could you help me why this limitation is required?
Platform information
PS /> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.2.1
PSEdition                      Core
GitCommitId                    7.2.1
OS                             Darwin 21.2.0 Darwin Kernel Version 21.2.0: Sun Nov 28 20:29:10 PST 2021; root:xnu-8019.61.5~1/RELEASE_ARM64_T8101
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

@clowa Excellent question! Target platform is specified as such because I've never tested it on anything other than x86_64.
To my best knowledge none of the APIs used are platform-sensitive, so it's entirely likely that it'll "just work" out of the box on ARM :-)

@IISResetMe, I really appreciate your quick reply!
I thought a bit about the reason for ProcessorArchitecture and if it is really intended for script modules. Since powershell core supports different processors/OS, any native powershell script built on powershell Core or dotnet core will run on all supported platforms, so I think it's more for binary modules compiled on a specific architecture. What do you think about my theory? I think it would be pretty cool to be able to use the module away from the classic Intel PCs. 🤔

Binary modules are definitely the primary reason for the directive in the first place, but there are cases where it's relevant to script modules too:

Module has a dependency on an assembly of a certain bitness
Commands in module are designed to make platform-specific changes (eg. a function to disable SysWOW64 redirection in a 32-bit process on a 64-bit, only makes sense under x86 process)

@IISResetMe oh wow, I haven't thought about things so specific to processor architecture. Thanks for the explaination! Since my initinal question is already answered I' going to close this issue. Thanks again!


Avoid invalid escape sequences.

They drive my OCD wild.

OCD?

OCD - I know the code already was working, but I see invalid escape sequences in a string and my left eye starts twitching.

I still don't know why that is an invalid escape sequence…
It just is a backslash. It's only an escape sequence if followed by a suited character... But meh…

@bwoebi No, it's invalid if followed by non-suited character. Otherwise, new characters like \u will be a BC break.