Update JoinClauseMixin to support more ANSI SQL joins

🦶Initial step to support https://github.com/cashapp/sqldelight/issues/5086
Add Ansi Sql join operators RIGHT and FULL e.g Now supported in https://www.sqlite.org/releaselog/3_39_0.html
LATERAL modifier not included, as works in two different ways (table/subquery and join) and should be implemented as a separate feature
These are added as forward declarations in the SqlLexer.flex that are needed for dialects to override
🏰  Keep core grammar with LEFT join
This allows the existing dialects to work with LEFT join and enables dialects to add RIGHT, FULL where supported, e.g PostgreSql and latest Sqlite dialect
e.g Postgresql.bnf would override the core join_operator and adds right_join_operator
📸 Tested snapshot build with SqlDelight branch as this appears to work
join_operator ::= ( COMMA
                  | [ NATURAL ] [ ( {left_join_operator} | {right_join_operator} | {full_join_operator} ) [ OUTER ] | INNER | CROSS ] JOIN ) {
  extends = "com.alecstrong.sql.psi.core.psi.impl.SqlJoinOperatorImpl"
  implements = "com.alecstrong.sql.psi.core.psi.SqlJoinOperator"
  override = true
}

Keep core grammar with LEFT join
This allows the existing dialects to work with LEFT join and enables dialects to add RIGHT, FULL where supported, e.g PostgreSql and latest Sqlite dialect

This was on purpose? Then we should revert #618, I didn't know some dialects does not support ANSI RIGHT or FULL...

Keep core grammar with LEFT join
This allows the existing dialects to work with LEFT join and enables dialects to add RIGHT, FULL where supported, e.g PostgreSql and latest Sqlite dialect

This was on purpose? Then we should revert #618, I didn't know some dialects do not support ANSI RIGHT or FULL...

Yes - see https://www.sqlite.org/releaselog/3_39_0.html

SQLite Release 3.39.0 On 2022-06-25
Add (long overdue) support for RIGHT and FULL OUTER JOIN.

This was a way that the core grammar can be overridden easier by supporting dialects that have all joins


MB-3576: Adding components to deploy to app-stg env

Description
This adds config files and CircleCI components to deploy the MilMove application into the GovCloud stg environment. This isn't quite complete; right now it only pushes to this environment with this branch, and the component that checks the SHA of the current deploy against the SHA being deployed is disabled because there is nothing deployed yet, and as a result this step would fail for the first deploy.

Messages

:book:
:link: MB-3576

  Generated by :no_entry_sign: dangerJS against ff6e778d42c60f29bc012c1f2a6254bd44396d41


Domain name change.

Our school name and its related domain name have changed.
Our website is https://www.eafc-uccle.be/.
We organize 2 IT-related curricula which have description pages here :

https://www.eafc-uccle.be/cours/bachelier-en-informatique-de-gestion/
https://www.eafc-uccle.be/cours/bachelier-en-informatique-et-systemes/

Best regards,
François ROLAND
IT teacher and GitHub campus advisor

@froland Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Thanks a lot for keeping the repository up-to-date!


Commit button not showing up for remote repos

I keep having a moment of cognitive dissonance when the Commit button doesn't appear in some of my repos. It seems to boil down to being in a remote repo.
Left side is a local git folder, right side is a remote repo. Even with changed files, the button doesn't appear:

@digitarald, could we set up some time tomorrow so that I can take a look? Thanks!

This has been addressed as currently the "Commit" action button was added to remote repositories.


Loss not on correct device when training with move_metrics_to_cpu=True

🐛 Bug
move_metrics_to_cpu=True seems to move loss to CPU which results in an error when training with native mixed precision.
This is related to the original issue reported in https://github.com/MIC-DKFZ/nnDetection/issues/25 and did not occur with other lightning versions (older version and/or move_metrics_to_cpu=False work fine).
Error when training with mixed precision and move_metrics_to_cpu=True:
/usr/local/lib/python3.7/dist-packages/torch/cuda/amp/grad_scaler.py in scale(self, outputs)
    159         # Short-circuit for the common case.
    160         if isinstance(outputs, torch.Tensor):
--> 161             assert outputs.is_cuda or outputs.device.type == 'xla'
    162             if self._scale is None:
    163                 self._lazy_init_scale_growth_tracker(outputs.device)

To Reproduce
Can be reproduced with the boring model in colab by passing the following flags to the trainer:
        precision=16,  # native mixed precision
        move_metrics_to_cpu=True,
        gpus=[0],  # use GPU

Expected behavior
No error :)

Environment

PyTorch Lightning Version (e.g., 1.3.0):
PyTorch Version (e.g., 1.8)
Python version:
OS (e.g., Linux):
CUDA/cuDNN version:
GPU models and configuration:
How you installed PyTorch (conda, pip, source):
If compiling from source, the output of torch.__config__.show():
Any other relevant information:

Additional context

Hey @mibaumgartner,
Would it be possible for you to share a reproducible script ?
Best,
T.C

Hi @tchaton,
here is the Boring Model Colab Notebook (latest lightning pypi release 1.4.5):
https://colab.research.google.com/drive/1pvHtF7Zor2OOLjE6NFjxxHTAassqdidb?usp=sharing
Best,
Michael

Thanks @mibaumgartner,
I can reproduce the bug locally.
Best,
T.C


Rule proposal: An AssemblyInformationVersion attribute should be in the project with the correct value.

The Application.UserAppDataPath or Application.UserAppDataRegistry properties reference paths and registry keys by building a string from the AssemblyCompany, AssemblyProduct, and AssemblyInformationalVersion attributes. If this attribute is not present, then the value in AssemblyVersion attribute is used instead. If the wildcard is used in the AssemblyVersion attribute, the compiler will change the build and revision number on each build causing information stored on the disk and registry to be stored in a different directory or registry key.
The AssemblyInformationalVersion attribute is used to provide a consistent version number for this version of the product. It should be changed along with the AssemblyVersion attribute number when the major or minor version is changed.
So the rule should check to make sure that the attribute is present, and that it has the same value as the Major and Minor version of the AssemblyVersion attribute.

:-1: Similar to the discussion in #1640

Right now I'm :-1: for the same reason as #1640.

:arrow_right: See #1640 for my recommendation for proposing this rule as a code analysis warning rather than a StyleCop analyzer.


Clean Some Data in Course Generator

Summary
When running the course generator script for SP23, I noticed that we were running into errors using the data because a course was missing a value for titleLong, and we just put a null in the JSON. This is a very quick fix (and does not fix the issues in #688), but it should allow us to run the script and update data for SP23.
Test Plan
Run the script on SP23 semester and verify there are no nulls and no NBSPs in the JSON. Also verify that the script does not include or exclude extra information.

@benjamin-shen Now we have empty strings for fields that didn't exist before

@noschiff hmm are they coming in as empty strings?


New Theme?

Support for a dark Arduino Pro IDE theme. See https://github.com/gabrielcaussi/arduino-pro-ide

I just sent you an invitation to join the Dracula org. Can you transfer the repository now? Thanks @gabrielcaussi!

Repository transferred to the dracula repository. Thanks @zenorocha!

It's live, thanks Gabriel! https://draculatheme.com/arduino-pro-ide 🎉 🎉 🎉


Adding --iree-opt-import-parameters= and cleaning up export.

The new option allows for importing all or some parameters from a parameter file into the compiler. One or more parameter files to one or more scopes can be specified, just as with the --parameters=[scope=]file.* flag in the runtime tool. If a scope is provided only parameters with that scope will be imported. If a parameter (optionally within a scope) is named explicitly it will be imported. If a maximum size is specified all parameters <= that size will be imported.
The export pass has been reworked to be the inverse of the import such that it's possible to round-trip through export and import. The import pass doesn't currently support all of the types we can export as SerializableAttrInterface doesn't have a deserialize method (yet) and we need special handling of packed types (i1, i4, etc). For now warnings are emitted and the globals are left as parameter references to still allow partial imports.
Flag renames:

--iree-opt-parameter-archive-export-file= + --iree-opt-parameter-archive-export-scope= -> --iree-opt-export-parameters=[scope=]file
--iree-opt-minimum-parameter-export-size= -> --iree-opt-export-parameter-minimum-size=
--iree-opt-splat-parameter-archive-export-file= -> --iree-opt-splat-parameters=

This exposed some real nastiness in our global opt pipeline around constant hoisting and inlining that the export pass was previously working around by running as part of the fixed point iteration in just the right place. I'm out of brain today but will see if I can think of something robust next week - if not, I'll make a flag that forces hoisting at a deterministic point for the export pass.

I can help next week. Out of brain for today. But thank you for working on it.

Nice, so you fixed what was blocking this? Is the new import flag now ready to be documented on https://iree.dev/guides/parameters/ (together with the export flag)?


Mechvibes slowing down my computer

hey, so i installed mechvibes in my computer and when i open it it just slows down everything really slow, makes it imposible to move the mouse or to type, besides no sound coming out. I have a decent computer, not very powerfull but runs games like valorant or league and WoW so i wonder what's the problem. Ive unistalled and installed it again but still same problem.
If anyone can help me i would appreciate it.
Thx

I'm going to close this issue due to inactivity but if you're still facing this issue, feel free to reach out on our discord.


Two issues with translations and condition construction

I found two issues:

Translations: I'm using Manjaro with Polish locale and command pacmd list-sink-inputs actually returns on my system muted: tak or muted: nie ("tak" for "yes" and "nie" for "no") and as far as I can see you have hardcoded "yes" and "no" in pulsesink_mute method :) Currently I have no universal solution yet but I believe that we should read values for "yes" and "no" from system properties.
For me condition construction is_ad = self.current_song != self.dbus.get_song_artist() + u" \u2013 " + self.dbus.get_song_title() doesn't work. I had to change it to

dbus_info = self.dbus.get_song_artist() + u" \u2013 " + self.dbus.get_song_title()
is_ad = self.current_song != dbus_info

in order to get it to work. I don't have experience in coding in Python so I cannot explain it but this is what I found under Python debugger.

Thanks, i'll have a look at the translation issue soon. I don't know what the pulse devs were thinking when they used strings to represent the mute state. Maybe there's a way to get a boolean instead.
What you are saying about the equality check would mean that your python version compiles to different bytecode which would be very strange. Are you sure that's the exact statement that failed?
Anyway, thanks for the issue report, i'll have a bit more time the coming weeks to fix some blockify issues.

Sorry for the long wait.
Regarding translations problem I think we can use the following approach:
import locale
import gettext

current_locale, encoding = locale.getdefaultlocale() # current_locale = 'pl_PL' in my case
pulseaudio_domain = 'pulseaudio'
localedir = gettext.find(pulseaudio_domain, languages=[current_locale])
localedir = localedir[:localedir.find('locale/')]+'locale' # not the best code
locale = gettext.translation(pulseaudio_domain, localedir=localedir, languages=[current_locale])
locale.install()
muted_true = _('yes')
muted_false = _('no')
print muted_true # tak
print muted_false # nie

The problem with equality check is strange. I'm quite sure that was an issue under debugger but now I cannot reproduce it so let's assume that was just my mistake.

Hey,
thanks, that could certainly work. I have to admit though, i'm not a big fan of adding options, there are too many as it is.
What is the output of  LANG=en_US.UTF-8 pacmd list-sink-inputs on your system?

Nevermind that last question about the LANG env variable. I simply added an option to blockify.ini called pacmd_muted_value. Set it to tak and let me know if it works.
Regards

Have you tried using LC_ALL? We ran into the same localization issues on Spotify-AdKiller and setting this env variable worked for us.

@Feltzer
Thanks for the hint. I read up on LC_ALL and in this instance it should be preferrable.
@thof
Could you try LC_ALL, too? Ideally, i'd like to know which, if any, of these commands produce english values for the muted key in the list-sink-inputs output.
LANG=en_US pacmd list-sink-inputs
LC_ALL=en_US pacmd list-sink-inputs
LC_ALL=C pacmd list-sink-inputs

Unfortunately none of these commands work, i.e. each of them returns values in Polish.

I see. I will readd the option then. Check again in 15 minutes.

It seems that since Pulseaudio 7.0 only two values (yes and no) are fixed for "muted" property regardless of locale. Unfortunately this change breaks blockify on my system. Therefore I think that you should disable method install_locale.

Are you sure this is what's breaking blockify for you?
Can you confirm that on your system with a previous pulseaudio version the muted property was localized and now it's not?
If that's the case, it'd be a drag because pulseaudio 7.0 won't be available on all distributions for a while. If i were to fix your issue by assuming muted is always "yes" or "no", I'd break blockify for those with localized systems until they can upgrade to pulseaudio 7.0.
A quick fix for now would be to change
self.pulse_unmuted_value = unmuted_value in blockify.py to self.pulse_unmuted_value = "no".
I'd still like some confirmation on this issue. As soon as I have that, i'm sure there's a way to fix it without breaking backward compatibility. Worst case scenario, I'll have to check the pulseaudio version.
Regards

I just downgraded pulseaudio and libpulse to 6.0 and I can confirm what I wrote in my previous post.
I think that checking PA version is not such a bad idea. Sample code that would handle this issue (initialize_pulse_unmuted_value):
pa_ver = subprocess.check_output("pulseaudio --version | awk '{print $2}'", shell=True)
if int(pa_ver[0]) < 7:
                self.install_locale()
                # Translate 'no' to the system locale.
                unmuted_value = _(unmuted_value)

Thanks for checking and thanks for the code snippet.
Just a hint: You can sign up for Hacktoberfest, submit the pull request and be 1/5 for your cheesy T-Shirt ;).
Otherwise, I'll implement this tomorrow. I'm not a big fan of checking for versions but at this point, blockify is a collection of hacks so we might as well add one.
Regards

Should be fixed on both master and spotify-beta


Patients all marked un-urgent

Flagged by CM Kari. Investigating. I think the rake task just nuked everyone but I'm investigating.

Modified by id isn't populating. That's pretty annoying!

Looks like this was case manager behavior,  but:
a) it looks like the rake task isn't working and
b) modified_by_id in audit trail isn't populating.
Task here: spin both of these into separate issues.

AuditTrail made in #639
Cleanup job issue represented in #609


Propose “Reinstate Mixed-Optional Equality Operators”

Rendered Markdown

I think that this should be incorporated into @rudkx's proposal, since we wouldn't want to accept it without doing this.

I think some folks, @rudkx included, are skeptical about whether this is worthwhile, so I'm not sure it should really be in the same proposal. But, @rudkx, if you agree then feel free to incorporate any of this in your proposal and we can abandon this one.

@jtbandes @lattner No, at first I thought this was something we could address separately due to the very small number of changes required in most projects, but I think the dictionary example has me convinced that we might as well make this completely painless.
I'll go ahead and add you as an author if you don't mind. Your feedback has been very useful.

Yes, the dictionary example is what I'm concerned about.  It is better (IMO) to preserve existing behavior for that.  No need to list me as a coauthor.

Actually I mean @jtbandes, but yes your feedback on the proposal itself has been very helpful as well. :)

Sounds good to me. I'm glad all these are coming together for Swift 3 :)


NLog + log4net added NetStandard support

Updated to NLog ver. 4.5.0 RTM
Updated to log4net ver. 2.0.8 RTM
Added support for Flush()
Resolves #155 + Resolves #68 + Resolves #115

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

Earliest will be end of next week (Apr 13th) maybe slipping to early the week of the 16th.
This release is dependent on dotnet and dotnet-server. Those are in progress now and should be finished early-mid next week. (Apr 9-11).

Microsoft.ApplicationInsights.NLogTarget ver. 2.6.0-beta3 has been released:
https://www.nuget.org/packages/Microsoft.ApplicationInsights.NLogTarget/2.6.0-beta3


[Bug]: Paint editor shifting when bottom bar hidden

Package Version
4.0.2
Flutter Version
3.22.2
Platforms
iOS
How to reproduce?
In the latest 4.x version, there seems to be some placement/sizing issue with the Paint Editor screen. The others seem fine.
I am hiding the bottomBar on the main editor, text editor and paint editor, like this:
bottomBar: (editor, rebuildStream) =>
    ReactiveCustomWidget(
  stream: rebuildStream,
  builder: (context) => const SizedBox(height: 0),
),

This seems to work fine. However, I noticed that only on paint editor, there is no SafeArea so the editor expands down to the edge of the screen (where the other editors do not). To fix this, I added my own SafeArea just to the bottomBar on paint editor:
paintEditor: CustomWidgetsPaintEditor(
  bottomBar: (editor, rebuildStream) =>
      ReactiveCustomWidget(
    stream: rebuildStream,
    builder: (context) => const SafeArea(
      child: SizedBox(height: 0),
      bottom: true,
    ),
  ),
),

That fixes the paint editor from being a different dimension to the other editors, however, when starting the paint editor, all of the layers seem to offset themselves upward a bit now:
https://github.com/hm21/pro_image_editor/assets/869261/eb1ef9bd-7f91-46d8-a850-5cb1d4fa22c7
Logs (optional)
No response
Example code (optional)
No response
Device Model (optional)
No response

Thanks for reporting this issue.
I didn't make any tests now but what I saw is that in the code which you posted you did not add the key to the bottombar widget. This is required that the editor can know which size the bottombar have. Can you try it like below and give me feedback if it works? If not, will I search the issue in detail.
mainEditor: CustomWidgetsMainEditor(
  bottomBar: (editor, rebuildStream, key) => ReactiveCustomWidget(
    stream: rebuildStream,
    builder: (context) => SizedBox(key: key, height: 0),
  ),

Main editor and text editor seem fine actually. But I did try adding the key to CustomWidgetsMainEditor bottomBar but that didn't seem to do anything. There is no key passed into CustomWidgetsPaintEditor bottomBar builder, only editor and stream.
I'm guessing the slight shift is due to there not being any SafeArea on the paint editor, whereas the main editor and text editor seem to have a SafeArea so the sizing is slightly different.

Actually adding the key to main editor seems to make it worse. When moving text around it also resizes incorrectly. But when leaving key out it works fine for text.

Can you post for me the full code how you now use the editor like ProImageEditor.network(...) that will help a lot to reproduce the issue.

Here's the quickest repro:
https://github.com/adamkoch/pro_image_editor/commit/339e4c0c28d3865caf01e6fdcdd3c10eb5507556
(using the "Custom AppBar and BottomBar" from the sample)

Ah, I see, the image will move because it will always be centered in the body. This is helpful, for example, when you open the Filter Editor, where the bottom bar size is larger than in the main editor so that it takes the full possible space before it scales down.
However, if you don't want the image to move, you can create a “fake” bottombar like the one below:
paintEditor: CustomWidgetsPaintEditor(
  bottomBar: (paintEditor, rebuildStream) => ReactiveCustomWidget(
    stream: rebuildStream,
    builder: (_) =>
        const SizedBox(height: kBottomNavigationBarHeight),
  ),
),

Keep in mind, depending on how you use the editor and set the SystemUiOverlayStyle you maybe also need to wrap it inside a SafeArea.

Anyway, this does not resolve the problem you show in the first video because there is the problem that the layers get the wrong y-position. I have released version 4.0.3 which should resolve this issue. Please let me know if you still have the issue after you updating

4.0.3 fixed it up! Thank you once again 🙇


Add x64Shape and arm64Shape flags to control default machine shapes p…

…er architecture, with appropriate defaults.
Set the correct archicture machine type on every VM.
Add a default value for the zone flag which will work with the default machine types

/test all

/lgtm

/hold

/unhold This broke CIT builds in a way that wasn't caught by gobuild somehow, fixed.

/lgtm

/unhold


Derivative computation is shifted

I have tried to compute cpu usage from scratch. I use derivative's function, but the results are shifted in comparison of telegraf's computation.
Tick script
var guest = stream
    .from().measurement('cpu_time_guest')
    .where(lambda: "cpu" == 'cpu-total')
    .groupBy('host')
    .derivative('value')
      .unit(10s)
      .nonNegative()

var guest_nice = stream
    .from().measurement('cpu_time_guest_nice')
    .where(lambda: "cpu" == 'cpu-total')
    .groupBy('host')
    .derivative('value')
      .unit(10s)
      .nonNegative()

var idle = stream
    .from().measurement('cpu_time_idle')
    .where(lambda: "cpu" == 'cpu-total')
    .groupBy('host')
    .derivative('value')
      .unit(10s)
      .nonNegative()

var iowait = stream
    .from().measurement('cpu_time_iowait')
    .where(lambda: "cpu" == 'cpu-total')
    .groupBy('host')
    .derivative('value')
      .unit(10s)
      .nonNegative()

var irq = stream
    .from().measurement('cpu_time_irq')
    .where(lambda: "cpu" == 'cpu-total')
    .groupBy('host')
    .derivative('value')
      .unit(10s)
      .nonNegative()

var nice = stream
    .from().measurement('cpu_time_nice')
    .where(lambda: "cpu" == 'cpu-total')
    .groupBy('host')
    .derivative('value')
      .unit(10s)
      .nonNegative()

var softirq = stream
    .from().measurement('cpu_time_softirq')
    .where(lambda: "cpu" == 'cpu-total')
    .groupBy('host')
    .derivative('value')
      .unit(10s)
      .nonNegative()

var steal = stream
    .from().measurement('cpu_time_steal')
    .where(lambda: "cpu" == 'cpu-total')
    .groupBy('host')
    .derivative('value')
      .unit(10s)
      .nonNegative()

var system = stream
    .from().measurement('cpu_time_system')
    .where(lambda: "cpu" == 'cpu-total')
    .groupBy('host')
    .derivative('value')
      .unit(10s)
      .nonNegative()

var user = stream
    .from().measurement('cpu_time_user')
    .where(lambda: "cpu" == 'cpu-total')
    .groupBy('host')
    .derivative('value')
      .unit(10s)
      .nonNegative()

var percent = guest
    .join(guest_nice, idle, iowait, irq, nice, softirq, steal, system, user)
        .as('guest', 'guest_nice', 'idle', 'iowait', 'irq', 'nice', 'softirq', 'steal', 'system', 'user')
    .eval(lambda: float(100) * ("user.value" / ("guest.value" + "guest_nice.value" + "idle.value" + "iowait.value" + "irq.value" + "nice.value" + "softirq.value" + "steal.value" + "system.value" + "user.value")))
        .as('value')
        .keep('host', 'value')
    .influxDBOut()
        .database('telegraf')
        .measurement('_temp')

Graph results:

By how much is it shifted? From what I can see on the graph it looks to be shifted by about 10s, is that right? This could be because of how the derivative is calculated left hand or right handed. For example, given the following data:
cpu_time_guest value=1000<PHONE_NUMBER>
cpu_time_guest value=1002<PHONE_NUMBER>
cpu_time_guest value=1004<PHONE_NUMBER>
cpu_time_guest value=1006<PHONE_NUMBER>
cpu_time_guest value=1008<PHONE_NUMBER>

The derivative could be calculated one of two ways:
Using first timestamp
dcpu_time_guest value=2<PHONE_NUMBER>
dcpu_time_guest value=2<PHONE_NUMBER>
dcpu_time_guest value=2<PHONE_NUMBER>
dcpu_time_guest value=2<PHONE_NUMBER>

Using last timestamp
dcpu_time_guest value=2<PHONE_NUMBER>
dcpu_time_guest value=2<PHONE_NUMBER>
dcpu_time_guest value=2<PHONE_NUMBER>
dcpu_time_guest value=2<PHONE_NUMBER>

Kapacitor uses the first timestamp. What is your use case? Does the time shift from using the first timestamp affect your overall result? It makes sense that it should probably be the same as Telegraf so that they line up but I would like to understand your use case a bit better.
Thanks for the detailed report.

I wanted to understand tick script and manipulate stream. To do that, I choose to start by a reproduction of cpu usage metrics. I was wondering why time was shifted from one tick. Thanks for the explanation. Is there way to choose derivative "right handed" ?
My original goal was to calculate procstats cpu metrics from telegraf in percentage. But i'm a little stuck for now.

@melck  There is no option to make the derivative right handed. I choose left handed because it makes it easier to reason about windows later on since the start of the window is the start of the data set instead of dropping the first timestamp. Its not a big deal and could be changed to right handed or maybe turned into an option if we found a strong use case.
Something that you do bring up though is it would be nice to explicitly shift values in time, since that has a much wider usefulness and would also solve your issue.
Something like this would be trivial to implement:
stream.from()...
   .timeShift(10s)

or
stream.from()...
   .timeShift(-10s)

@nathanielc I understand and i also prefer shift values in time. It will may be usefull for other cases.

Shift has been implemented


How to get alive ads on iOS

I build my project to iPhone, without "AdRequest.Builder.AddTestDevice()", but I can still not get alive ad, the Xcode have a message on console: To get test ads on this device, call: request.testDevices

This issue is being closed due to its age. For bugs in the plugin open a new issue or reach out on our support group at https://groups.google.com/forum/#!forum/google-admob-ads-sdk.

Ram


PartialRealizationConfig class and usage in requests and dataservice

Create new PartialRealizationConfig class to represent partial realization config, based on details that need to be user-supplied
Update AbstractNgenRequest to utilize PartialRealizationConfig
Update dataservice.ServiceManager class to be able to both

recognize and support situations when a realization config isn't available for a request job but can be derived because of an included PartialRealizationConfig in the request
derived realization configs and containing datasets from a job request's PartialRealizationConfig when needed

This closes #263, #291, and #292.

@aaraney, I might need your help a bit with finishing the tests.
I'm attempting to use NgenRealization objects from ngen-config as a part of the unit test setup.  I initialize those from realization config files essentially copied from the ngen repo.  Or at least I try to.  I have beat my head into a wall of validation errors for most of today.  For the first of the two files, they look like what I've copied below.
I eventually did a quick swap of the realization config from the AGU town hall example setup, and it did not fail like this.  But even with looking at the errors and comparing the two configs side by side, I can't tell what is causing these issues.
Some of the error are just unclear (the global -> formulations -> 0 -> params -> __root__ items).  Others don't make sense, assuming I am interpreting them correctly (unexpected value; permitted: 'bmi_python' (type=value_error.const; given=bmi_multi; permitted=['bmi_python'])).
Error
Traceback (most recent call last):
  File "/Users/rbartel/Developer/noaa/dmod/python/lib/communication/dmod/test/test_partial_realization_config.py", line 66, in setUp
    self._base_real_configs: List[NgenRealization] = self.read_example_realization_config_sources()
  File "/Users/rbartel/Developer/noaa/dmod/python/lib/communication/dmod/test/test_partial_realization_config.py", line 61, in read_example_realization_config_sources
    return [NgenRealization(**json.load(cfg_path.open())) for cfg_path in config_paths]
  File "/Users/rbartel/Developer/noaa/dmod/python/lib/communication/dmod/test/test_partial_realization_config.py", line 61, in <listcomp>
    return [NgenRealization(**json.load(cfg_path.open())) for cfg_path in config_paths]
  File "pydantic/main.py", line 342, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 48 validation errors for NgenRealization
global -> formulations -> 0 -> params -> __root__
  expected str, bytes or os.PathLike object, not NoneType (type=type_error)
global -> formulations -> 0 -> params -> __root__
  expected str, bytes or os.PathLike object, not NoneType (type=type_error)
global -> formulations -> 0 -> params -> __root__
  expected str, bytes or os.PathLike object, not NoneType (type=type_error)
global -> formulations -> 0 -> params -> __root__
  expected str, bytes or os.PathLike object, not NoneType (type=type_error)
global -> formulations -> 0 -> params -> name
  unexpected value; permitted: 'bmi_python' (type=value_error.const; given=bmi_multi; permitted=['bmi_python'])
global -> formulations -> 0 -> params -> main_output_variable
  unexpected value; permitted: 'land_surface_water__runoff_depth' (type=value_error.const; given=Q_OUT; permitted=('land_surface_water__runoff_depth',))
global -> formulations -> 0 -> params -> __root__
  expected str, bytes or os.PathLike object, not NoneType (type=type_error)
global -> formulations -> 0 -> params -> modules -> 0 -> params -> name
  unexpected value; permitted: 'bmi_c' (type=value_error.const; given=bmi_c++; permitted=['bmi_c'])
global -> formulations -> 0 -> params -> modules -> 0 -> params -> model_type_name
  unexpected value; permitted: 'TOPMODEL' (type=value_error.const; given=bmi_c++_sloth; permitted=['TOPMODEL'])
global -> formulations -> 0 -> params -> modules -> 0 -> params -> name
  unexpected value; permitted: 'bmi_c' (type=value_error.const; given=bmi_c++; permitted=['bmi_c'])
global -> formulations -> 0 -> params -> modules -> 0 -> params -> model_type_name
  unexpected value; permitted: 'CFE' (type=value_error.const; given=bmi_c++_sloth; permitted=['CFE'])
global -> formulations -> 0 -> params -> modules -> 0 -> params -> name
  unexpected value; permitted: 'bmi_c' (type=value_error.const; given=bmi_c++; permitted=['bmi_c'])
global -> formulations -> 0 -> params -> modules -> 0 -> params -> main_output_variable
  unexpected value; permitted: 'water_potential_evaporation_flux' (type=value_error.const; given=z; permitted=('water_potential_evaporation_flux',))
global -> formulations -> 0 -> params -> modules -> 0 -> params -> name
  unexpected value; permitted: 'bmi_fortran' (type=value_error.const; given=bmi_c++; permitted=['bmi_fortran'])
global -> formulations -> 0 -> params -> modules -> 0 -> params -> model_type_name
  unexpected value; permitted: 'NoahOWP' (type=value_error.const; given=bmi_c++_sloth; permitted=['NoahOWP'])
global -> formulations -> 0 -> params -> modules -> 0 -> params -> name
  unexpected value; permitted: 'bmi_python' (type=value_error.const; given=bmi_c++; permitted=['bmi_python'])
global -> formulations -> 0 -> params -> modules -> 0 -> params -> main_output_variable
  unexpected value; permitted: 'land_surface_water__runoff_depth' (type=value_error.const; given=z; permitted=('land_surface_water__runoff_depth',))
global -> formulations -> 0 -> params -> modules -> 0 -> params -> model_type_name
  unexpected value; permitted: 'SLOTH' (type=value_error.const; given=bmi_c++_sloth; permitted=['SLOTH'])
global -> formulations -> 0 -> params -> modules -> 0 -> params -> name
  unexpected value; permitted: 'bmi_multi' (type=value_error.const; given=bmi_c++; permitted=['bmi_multi'])
global -> formulations -> 0 -> params -> modules -> 0 -> params -> init_config
  unexpected value; permitted: '' (type=value_error.const; given=/dev/null; permitted=[''])
global -> formulations -> 0 -> params -> modules -> 0 -> params -> model_params
  unexpected value; permitted: None (type=value_error.const; given={'sloth_ice_fraction_schaake(1,double,m,node)': '0.0', 'sloth_ice_fraction_xinan(1,double,1,node)': '0.0', 'sloth_smp(1,double,1,node)': '0.0'}; permitted=[None])
global -> formulations -> 0 -> params -> modules -> 0 -> params -> modules
  field required (type=value_error.missing)
global -> formulations -> 0 -> params -> modules -> 1 -> params -> name
  unexpected value; permitted: 'bmi_c' (type=value_error.const; given=bmi_fortran; permitted=['bmi_c'])
global -> formulations -> 0 -> params -> modules -> 1 -> params -> model_type_name
  unexpected value; permitted: 'TOPMODEL' (type=value_error.const; given=bmi_fortran_noahowp; permitted=['TOPMODEL'])
global -> formulations -> 0 -> params -> modules -> 1 -> params -> name
  unexpected value; permitted: 'bmi_c' (type=value_error.const; given=bmi_fortran; permitted=['bmi_c'])
global -> formulations -> 0 -> params -> modules -> 1 -> params -> model_type_name
  unexpected value; permitted: 'CFE' (type=value_error.const; given=bmi_fortran_noahowp; permitted=['CFE'])
global -> formulations -> 0 -> params -> modules -> 1 -> params -> name
  unexpected value; permitted: 'bmi_c' (type=value_error.const; given=bmi_fortran; permitted=['bmi_c'])
global -> formulations -> 0 -> params -> modules -> 1 -> params -> main_output_variable
  unexpected value; permitted: 'water_potential_evaporation_flux' (type=value_error.const; given=QINSUR; permitted=('water_potential_evaporation_flux',))
global -> formulations -> 0 -> params -> modules -> 1 -> params -> model_type_name
  unexpected value; permitted: 'NoahOWP' (type=value_error.const; given=bmi_fortran_noahowp; permitted=['NoahOWP'])
global -> formulations -> 0 -> params -> modules -> 1 -> params -> name
  unexpected value; permitted: 'bmi_python' (type=value_error.const; given=bmi_fortran; permitted=['bmi_python'])
global -> formulations -> 0 -> params -> modules -> 1 -> params -> main_output_variable
  unexpected value; permitted: 'land_surface_water__runoff_depth' (type=value_error.const; given=QINSUR; permitted=('land_surface_water__runoff_depth',))
global -> formulations -> 0 -> params -> modules -> 1 -> params -> name
  unexpected value; permitted: 'bmi_c++' (type=value_error.const; given=bmi_fortran; permitted=['bmi_c++'])
global -> formulations -> 0 -> params -> modules -> 1 -> params -> model_type_name
  unexpected value; permitted: 'SLOTH' (type=value_error.const; given=bmi_fortran_noahowp; permitted=['SLOTH'])
global -> formulations -> 0 -> params -> modules -> 1 -> params -> name
  unexpected value; permitted: 'bmi_multi' (type=value_error.const; given=bmi_fortran; permitted=['bmi_multi'])
global -> formulations -> 0 -> params -> modules -> 1 -> params -> init_config
  unexpected value; permitted: '' (type=value_error.const; given=./data/bmi/fortran/noah-owp-modular-init-{{id}}.namelist.input; permitted=[''])
global -> formulations -> 0 -> params -> modules -> 1 -> params -> modules
  field required (type=value_error.missing)
global -> formulations -> 0 -> params -> modules -> 2 -> params -> model_type_name
  unexpected value; permitted: 'TOPMODEL' (type=value_error.const; given=bmi_c_cfe; permitted=['TOPMODEL'])
global -> formulations -> 0 -> params -> modules -> 2 -> params -> model_type_name
  unexpected value; permitted: 'CFE' (type=value_error.const; given=bmi_c_cfe; permitted=['CFE'])
global -> formulations -> 0 -> params -> modules -> 2 -> params -> main_output_variable
  unexpected value; permitted: 'water_potential_evaporation_flux' (type=value_error.const; given=Q_OUT; permitted=('water_potential_evaporation_flux',))
global -> formulations -> 0 -> params -> modules -> 2 -> params -> name
  unexpected value; permitted: 'bmi_fortran' (type=value_error.const; given=bmi_c; permitted=['bmi_fortran'])
global -> formulations -> 0 -> params -> modules -> 2 -> params -> model_type_name
  unexpected value; permitted: 'NoahOWP' (type=value_error.const; given=bmi_c_cfe; permitted=['NoahOWP'])
global -> formulations -> 0 -> params -> modules -> 2 -> params -> name
  unexpected value; permitted: 'bmi_python' (type=value_error.const; given=bmi_c; permitted=['bmi_python'])
global -> formulations -> 0 -> params -> modules -> 2 -> params -> main_output_variable
  unexpected value; permitted: 'land_surface_water__runoff_depth' (type=value_error.const; given=Q_OUT; permitted=('land_surface_water__runoff_depth',))
global -> formulations -> 0 -> params -> modules -> 2 -> params -> name
  unexpected value; permitted: 'bmi_c++' (type=value_error.const; given=bmi_c; permitted=['bmi_c++'])
global -> formulations -> 0 -> params -> modules -> 2 -> params -> model_type_name
  unexpected value; permitted: 'SLOTH' (type=value_error.const; given=bmi_c_cfe; permitted=['SLOTH'])
global -> formulations -> 0 -> params -> modules -> 2 -> params -> name
  unexpected value; permitted: 'bmi_multi' (type=value_error.const; given=bmi_c; permitted=['bmi_multi'])
global -> formulations -> 0 -> params -> modules -> 2 -> params -> init_config
  unexpected value; permitted: '' (type=value_error.const; given=./data/bmi/c/cfe/{{id}}_bmi_config.ini; permitted=[''])
global -> formulations -> 0 -> params -> modules -> 2 -> params -> modules
  field required (type=value_error.missing)

@robertbartel, sorry that pydantic assaulted you like this. Im taking a look now!

This is the source issue in ngen-cal that documents the problem: https://github.com/NOAA-OWP/ngen-cal/issues/32.

Ok, I will have a look at those. If I can, I'll try to do a review on #34 so it can be merged in.

FYI, @aaraney, I tried to install the package from the branch in ngen-cal#34 to see if I could get the tests passing.  It looked promising at first, down to just a problem with SLOTH failing because that model type wasn't added yet.  But switching to a copy of that branch that I rebased, I get a new set of errors.  I think you talk about this problem some over in the PR.
Traceback (most recent call last):
  File "/Users/rbartel/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/231.8109.175/IntelliJ IDEA.app.plugins/python/helpers/pycharm/_jb_unittest_runner.py", line 38, in <module>
    sys.exit(main(argv=args, module=None, testRunner=unittestpy.TeamcityTestRunner,
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/unittest/main.py", line 100, in __init__
    self.parseArgs(argv)
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/unittest/main.py", line 147, in parseArgs
    self.createTests()
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/unittest/main.py", line 158, in createTests
    self.test = self.testLoader.loadTestsFromNames(self.testNames,
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/unittest/loader.py", line 220, in loadTestsFromNames
    suites = [self.loadTestsFromName(name, module) for name in names]
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/unittest/loader.py", line 220, in <listcomp>
    suites = [self.loadTestsFromName(name, module) for name in names]
  File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/unittest/loader.py", line 154, in loadTestsFromName
    module = __import__(module_name)
  File "/Users/rbartel/Developer/noaa/dmod/python/lib/communication/dmod/test/test_partial_realization_config.py", line 4, in <module>
    from ..communication.maas_request.ngen.partial_realization_config import PartialRealizationConfig
  File "/Users/rbartel/Developer/noaa/dmod/python/lib/communication/dmod/communication/__init__.py", line 2, in <module>
    from .client import DataServiceClient, InternalServiceClient, ModelExecRequestClient, ExternalRequestClient, \
  File "/Users/rbartel/Developer/noaa/dmod/python/lib/communication/dmod/communication/client.py", line 15, in <module>
    from .maas_request import ExternalRequest, ExternalRequestResponse, ModelExecRequest, ModelExecRequestResponse, NWMRequest, \
  File "/Users/rbartel/Developer/noaa/dmod/python/lib/communication/dmod/communication/maas_request/__init__.py", line 15, in <module>
    from .ngen import AbstractNgenRequest, NGENRequest, NGENRequestResponse, NgenCalibrationRequest, \
  File "/Users/rbartel/Developer/noaa/dmod/python/lib/communication/dmod/communication/maas_request/ngen/__init__.py", line 1, in <module>
    from .abstract_nextgen_request import AbstractNgenRequest
  File "/Users/rbartel/Developer/noaa/dmod/python/lib/communication/dmod/communication/maas_request/ngen/abstract_nextgen_request.py", line 1, in <module>
    from .partial_realization_config import PartialRealizationConfig
  File "/Users/rbartel/Developer/noaa/dmod/python/lib/communication/dmod/communication/maas_request/ngen/partial_realization_config.py", line 2, in <module>
    from ngen.config.formulation import Formulation
  File "/Users/rbartel/Developer/noaa/dmod/venv/lib/python3.8/site-packages/ngen/config/formulation.py", line 17, in <module>
    Formulation.update_forward_refs()
  File "pydantic/main.py", line 816, in pydantic.main.BaseModel.update_forward_refs
  File "pydantic/typing.py", line 553, in pydantic.typing.update_model_forward_refs
    '''Return a list containing n references to x.'''
  File "pydantic/typing.py", line 524, in pydantic.typing.update_field_forward_refs
    def __eq__(self, other):
  File "pydantic/fields.py", line 552, in pydantic.fields.ModelField.prepare
  File "pydantic/fields.py", line 666, in pydantic.fields.ModelField._type_analysis
  File "pydantic/fields.py", line 780, in pydantic.fields.ModelField.prepare_discriminated_union_sub_fields
  File "pydantic/utils.py", line 794, in pydantic.utils.get_discriminator_alias_and_values
pydantic.errors.ConfigError: Field 'model_name' of model 'PET' needs to be a `Literal`

Process finished with exit code 1

FYI, I was able to locally work around the issue and make sure tests pass.  For now, the tests in the committed revision are skipped.  I've opened #317 to make sure they get put back in once the ngen-config issues are sorted out.

Thanks for pinging me to review this! Ill set aside some time midday to review this.


Deploy: Force async on large deployments

- Summary
Fixes: https://github.com/netlify/cli/issues/745

- Test plan
Test manually with large scale sites that fail with timeout

- Description for the changelog
Set deployment syncFileLimit: 1 to force an async deployment

- A picture of a cute animal (not mandatory but encouraged)
🐠

Reasoning is explained here: https://netlify.slack.com/archives/C9D3B8RGF/p1593767386073000


Cannot figure out how to open up a page in a new tab

Component
Version

Operating system
Linux/Windows/Mac

Node.js
v9.8.0

Chrome/Chromium/...
Chromium 64.0.3282.167 Built on Ubuntu , running on Ubuntu 17.10

chrome-remote-interface
<EMAIL_ADDRESS>

Is Chrome running in a container? YES - kubernetes/docker
Code: https://gist.github.com/nddipiazza/9d1e6f4ad7ca46a100b5ca0a6acba1c4
Is there a way I can alter this code to open a new tab instead of taking over the current tab?
Right now when it calls Page.navigate it will take over the tab it's using originally.
I want to do something like when you call:
chrome-remote-interface new 'http://example.com'
This will open a new tab, then return the ID of the tab. Which is perfect.
I'd like to alter my little program above to take a list of URLs instead of just one. But I can't figure out how.

See this wiki and the README.


cmake - Cross-compile for Windows fails

Under Macros.cmake:
string(REGEX REPLACE "(.*)/bin/.*\\.exe" "\\1" STANDARD_LIBS_PATH "${CMAKE_CXX_COMPILER}")

This won't work when cross-compiling because the executable doesn't end with .exe.  So, when searching for libs, the path ends up being /usr/bin/i486-mingw32-g++/lib.  To fix it, it must be changed to:
set(STANDARD_LIBS_PATH "${CMAKE_INSTALL_PREFIX}")

For my system.

Actually, scratch that.  In my case, STANDARD_LIBS_PATH should be:
${CMAKE_INSTALL_PREFIX}/libs.  The above will just set it to /usr/lib, which won't contain MinGW's libraries.

In addition to this, I've also discovered that the *.o a few lines below that one should be *.obj.  Maybe it could be .o, so that it catches everything?

Sorry for the closing/reopening.  I keep pressing the wrong button. :/

It also seems that I still can't get this to build because it's surrounding *.o in quotation marks, which prevents it from expanding.

Obviously this part of the code is Windows-specific and cannot be adapted to Linux. You should rather compile SFML dynamic libraries, instead of the static ones.

Yes, but the dynamic libraries require me to include the DLLs with the program; the static ones don't.  Wouldn't it be possible to add a flag like SFML_CROSS_WINDOWS so that we can build the static libraries from Linux?

Yes, but the dynamic libraries require me to include the DLLs with the program

And?

And, with static libraries, I can optimise the overall size to be smaller than an EXE with included DLLs.

I don't see the issue with DLLs. We use shared libs on Linux and we like them.  The resulting size should be roughly the same. The cross-chain I put into AUR works very nicely for cross-compiling SFML DLLs for Windows and I never had the need to compile static ones.

On top of that, you lose exception support if you truly go all the way static (-static-libgcc -static-libstdc++).

If you go all the way static, you don't lose exceptions (why would you? Everything is in a single binary, there are no boundaries on which throwing could fail). If you go partially static, you may or may not lose exceptions, depending on what patches were applied to the cross-compiler.

True dat.


Why didn't I find "from_i32"?

I use the tonic, when I def an enum, but I didn't find the function from_i32

I would recommend documenting the types you should be able to see what is available. The from_i32 exists on the generated enum types.

Resolved, the Clion does not display the function


Morty upgrade on 1.X

@resin-jenkins retest this please

@resin-jenkins retest this please

@resin-jenkins retest this please

This needs bump of meta-resin too.

Superseded by https://github.com/resin-os/resin-qemu/pull/69


Update App.js

Illegal constructor
at :1:1

Illegal constructor
at :1:1

Illegal constructor
at :1:1


[Tensor] Support multiple data types in copyData

Previously, copyData only supported copying data of the same data type.
Copying data of different data types is needed with the increased demand for flexibility of mixed-precision.
Changes proposed in this PR:

copyData supports copying data of different types with the use of NEON
Remove the flate function
utilize copyData in dequantize

Self-evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2359. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

LGTM


Headers customization

I have met several difficulties. Maybe you could make some changes.

If I make cross-site request, then Upgrade-Insecure-Requests is not suitable. So, I need to remove it by myself after headers generation.
If I do POST instead of GET, I need to add content-type and content-length headers. And I need them in the right order. But your ordering file does not include them, so I need to do ordering by myself.
The same is true for Origin header.
Also, for cross-site requests I need to change the cors headers and it is now convenient now too. I can't pass them as additional headers: it does not works reliable due to case sensitivity etc. So,  I need to modify cors headers after generation. Probably, this is not the best way too.

Thank you.

Hello and thank you for submitting this issue!
Other than the second point (which I have already prepared a PR for, see #190), your requests seem to be centered on your use case. Can you please provide a better description of what you are trying to achieve with this library (possibly share some snippets of your code), so it gives us a better idea of what your priorities are? Thanks!

Hello, and thank you for the answer.
I am making a cross-site POST-request. I don't think that code snippets are relevant for the explanation. I could write headers if it needed for clarity.
I think we have three different things here.

Headers list (what headers should be included and excluded from this particular request).
Headers values (for cross-site requests, we need to modify the provided cors headers).
Headers ordering.
PR 190 should solve the first problem. The second is not a big problem.

But the ordering seems complicated. I ran your solver script and also made different requests in different browsers manually and saw headers in Wireshark. It looks like in Chrome request headers are in different orders for different requests. I don't know which parameters determine the header order. For now, I just give the exact order for needed request and sort headers in this order. But this is an ad hoc way and not convenient.
So, it seems that for POST requests, your ordering solution does not work properly (at least for Chrome).


Tables should have an order to make it easier for the user

I noticed that the tables are not in any order in the select data screen.
I think I should have been more explicit about this here as it is important for the tables to have an order in the customise tables step, to help the user learn the contracting process and associated data.
The feedback I had from people is that they really appreciated having a logical order to the tables to help them understand the data.
The order should be

Parties
Planning
Tenders
Awards
Contracts
Documents
Milestones
Amendments

The video attached shows that there is no ordering in the select tables step or the customise table step.
https://user-images.githubusercontent.com/11212467/116535031-3119a300-a8db-11eb-95da-2ae4ebc84654.mp4


Requiring constant "reconfigure" processes

Been working ok but as of 2 days ago it keeps coming up as Reconfigure Required. It makes me login with my kwikset account and get a code.
Using HAOS 2023.11.3

Me too.

Thank you so much @explosivo22 for looking at this for us!!! I appreciate your help!

tracking on #69 now. closing this one


Update documentation to use the new syntax

Hello,
Right now in your doc, all examples uses the old github syntax that no longer exists since it went into GA.
Is that possible to update your examples with the newer syntax ? I'd love to use your HTTPie wrapper;

Yea, thanks for leaving this deadend out to waste a 1/2 hour of my time trying to figure out to get it work with the new syntax. Gave up.


More clear sentiment analysis example?

I had been trying to figure out how to implement sentiment analysis by following sequence to one example but found it rather hard to understand.
Just wondering is there any more detailed examples using real world datasets like imdb or twitter to showcase the true power of rnn? Thanks.

@coodoo It is on my To Do list to include the twitter dataset. Just to be clear, you mean the one referred to on this page right : http://help.sentiment140.com/for-students/ ?

yes you may start from this link
http://cs.stanford.edu/people/alecmgo/trainingandtestdata.zip
though it is not true data since
it was labeled by emotions
but code development it is ok
however
for model tuning to be  get useful model
it is faked data
by the way for me better to have only python based code
since torch gives up to python
do you know python code example for rnn/last/gru
except
https://github.com/fchollet/keras/blob/master/examples/imdb_lstm.py
http://deeplearning.net/tutorial/lstm.html
https://github.com/mesnilgr/is13
https://github.com/umass-semeval/semeval16
http://k8si.github.io/2016/01/28/lstm-networks-for-sentiment-analysis-on-tweets.html
you will find more data here
semeval16-master\data\
On Sun, Mar 6, 2016 at 12:22 PM, Nicholas Léonard<EMAIL_ADDRESS>wrote:

@coodoo https://github.com/coodoo It is on my To Do list to include the
twitter dataset. Just to be clear, you mean the one referred to on this
page right : http://help.sentiment140.com/for-students/ ?
—
Reply to this email directly or view it on GitHub
https://github.com/Element-Research/rnn/issues/148#issuecomment-192938598
.

@nicholas-leonard Yes, that's the original source, later on there's a couple of variations based on that, including examples from keras as mentioned above.
I was basically following kaggle's sunny side up example which provides both IMDB and twitter datasets, might be a useful reference.
But really any example that walks the reader from end to end with detailed instructions would be highly appreciated!

Jeremy
I did not find some RNN based theano source code in this project
only search gave me
sunny-side-up-master_mar7\src\examples\tufs_cnn.py
'''
Model below is based on paper by Xiang Zhang "Character-Level Convolutional
Networks for Text Classification" (http://arxiv.org/abs/1509.01626) paper
was
formerly known as "Text Understanding from Scratch" (
http://arxiv.org/pdf/1502.01710v4.pdf)
THEANO_FLAGS=mode=FAST_RUN,device=gpu1,floatX=float32 python tufs_cnn.py
'''
On Mon, Mar 7, 2016 at 7:34 AM, Jeremy Lu<EMAIL_ADDRESS>wrote:

@nicholas-leonard https://github.com/nicholas-leonard Yes, that's the
original source, later on there's a couple of variations based on that,
including examples from keras as mentioned above.
I was basically following kaggle's sunny side up
https://github.com/Lab41/sunny-side-up example which provides both IMDB
and twitter datasets, might be a useful reference.
But really any example that walks the reader from end to end with detailed
instructions would be highly appreciated!
—
Reply to this email directly or view it on GitHub
https://github.com/Element-Research/rnn/issues/148#issuecomment-193232436
.

@Sandy4321 Yes, I mentioned sunny side up project only because they provided tweet dataset, not because they are using RNN for sentiment analysis, I was planning to implement one with RNN but to no avail hence asking help here :)

Did you checked recommend semeval 16 from git,
She uses word2vec and lastm lasagna
On Mar 10, 2016 18:43, "Jeremy Lu"<EMAIL_ADDRESS>wrote:

@Sandy4321 https://github.com/Sandy4321 Yes, I mentioned sunny side up
project only because they provided tweet dataset, not because they are
using RNN for sentiment analysis, I was planning to implement one with RNN
but to no avail hence asking help here :)
—
Reply to this email directly or view it on GitHub
https://github.com/Element-Research/rnn/issues/148#issuecomment-195102820
.

@Sandy4321 Thanks for pointing out semeval, their char-based LSTM for sentiment analysis example looks very interesting, will have a thorough study today.
It would be prefect if there's a RNN implementation so that it could be run on OpenCL (e.g. mackbook pro) instead of CUDA, maybe @nicholas-leonard could help? :)

try this
https://github.com/youssefahres/EntitySentiment
https://github.com/Vict0rSch/deep_learning/tree/master/keras/recurrent
https://github.com/itdxer/neupy/tree/release/v0.2.0
https://github.com/stevenxxiu/senti/tree/master/senti
https://github.com/dandxy89/DeepLearning_MachineLearning/tree/master/EmbeddingKeras
On Thu, Mar 10, 2016 at 10:20 PM, Jeremy Lu<EMAIL_ADDRESS>wrote:

@Sandy4321 https://github.com/Sandy4321 Thanks for pointing out
semeval, their char-based LSTM for sentiment analysis example looks very
interesting, will have a thorough study today.
It would be prefect if there's a RNN implementation so that it could be
run on OpenCL (e.g. mackbook pro) instead of CUDA, maybe @nicholas-leonard
https://github.com/nicholas-leonard could help? :)
—
Reply to this email directly or view it on GitHub
https://github.com/Element-Research/rnn/issues/148#issuecomment-195160995
.

and this
https://github.com/fchollet/keras/issues/1327
https://github.com/fchollet/keras/issues/631
https://github.com/fchollet/keras/issues/853
On Thu, Mar 10, 2016 at 10:20 PM, Jeremy Lu<EMAIL_ADDRESS>wrote:

@Sandy4321 https://github.com/Sandy4321 Thanks for pointing out
semeval, their char-based LSTM for sentiment analysis example looks very
interesting, will have a thorough study today.
It would be prefect if there's a RNN implementation so that it could be
run on OpenCL (e.g. mackbook pro) instead of CUDA, maybe @nicholas-leonard
https://github.com/nicholas-leonard could help? :)
—
Reply to this email directly or view it on GitHub
https://github.com/Element-Research/rnn/issues/148#issuecomment-195160995
.

Thanks for the links.
I went through semval16 example the other day and found it's code quite hard to understand (not sure it's due to the nature of lasagne or author's coding style). I also looked at various keras examples on LSTM but none of them are char-based.
Lastly, as a total newbie in deep learning and NLP, I'm really looking forward to examples built with torch so that it supports cltorch and clnn which can be run on macbook pro with OpenCL graphic card to make the computation time acceptable.
IMHO example like that could really showcase the power of rnn lib.

OpenCL graphic card
is it external card?
On Tue, Mar 15, 2016 at 3:47 AM, Jeremy Lu<EMAIL_ADDRESS>wrote:

Thanks for the links.
I went through semval16 example the other day and found it's code quite
hard to understand (not sure it's due to the nature of lasagne or
author's coding style). I also looked at various keras examples
https://github.com/fchollet/keras/tree/d229c4784505346e6a039cf2bdd5cbca3ce9e648/examples
on LSTM but none of them are char-based.
Lastly, as a total newbie in deep learning and NLP, I'm really looking
forward to examples built with torch so that it supports cltorch and clnn
which can be run on macbook pro with OpenCL graphic card to make the
computation time acceptable.
IMHO example like that could really showcase the power of rnn lib.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/Element-Research/rnn/issues/148#issuecomment-196705561

@coodoo @Sandy4321 We (me, @sagarwaghmare69, @ywelement) just completed wrapping the Twitter Sentiment140 dataset into a wrapper. @sagarwaghmare69 will be working on a example using the dataset.

@coodoo @Sandy4321 https://github.com/Element-Research/rnn/blob/master/examples/twitter_sentiment_rnn.lua


A policy_scope with additional context

I have a Workspace model that has many projects, and many users through memberships. The project model has a private flag. Admins can see all projects in a workspace. Members can only see public projects.
class Workspace
  has_many :memberships
  has_many :users, through: :memberships
  has_many :projects
end

In the ProjectsController index action could look like this, but I'd rather not apply this pattern.
class ProjectsController < ApplicationController
  def index
    @projects = if @workspace.admins.include?(current_user)
      @workspace.projects
    else
      @workspace.projects.public
    end
  end

The above code doesn't map nicely to the PolicyScope pattern that pundit provides. The PolicyScope has no concept of the workspace, here.
class ProjectsController < ApplicationController
  def index
    @projects = policy_scope(Project.where(workspace: @workspace))
  end
end

Here's the issue. The Scope class in ProjectPolicy has no idea about the @workspace. So we can determine the user's level of authorization. Since his Membership is the model that stores its auth level.
class ProjectPolicy < ApplicationPolicy
  class Scope
    def resolve
      if workspace.admins.include?(user) # <--- There's no workspace in this context!
        scope.all
      else
        scope.where(private: false)
      end
    end
  end
end

Sorry if this has been asked before; I scoured the issues and couldn't find anything I could use. How do the pundits go about this situation?

One way I can think of is to use the WorkspacePolicy and return a scope. This means skipping the ProjectPolicy.
class ProjectsController < ApplicationController
  def index
    @projects = WorkspacePolicy.new(current_user, @workspace).projects_scope
  end
end

Then in the WorkspacePolicy.
class WorkspacePolicy < ApplicationPolicy
  alias_method :workspace, :record

  def projects_scope
    if workspace.admins.include?(user)
      workspace.projects.all
    else
      workspace.projects.public
    end
  end
end

Any thoughts? A better way?

Personally I probably would have used a join in the ProjectPolicy. Something like:
class ProjectPolicy < ApplicationPolicy
  class Scope
    def resolve
      outer_joins(:workspace).merge(Workspace.outer_joins(:admins)).where({ users: { id: user.id }).or(private: false).uniq
    end
  end
end

Since Rails still doesn't have outer_joins, you can use this: https://github.com/elabs/ar_outer_joins
It's probably wrong somehow, I haven't tested the code at all, but you get the idea. This accurately reflects what you actually have to do to figure out which projects you are allowed to load.

@jnicklas thanks for chiming in. This is a good suggestion. I'll try it out. In the end, I opted to change the user context so that the authorization takes place on the Membership record. So instead of passing in the user, I'm passing in the Membership in the Workspace. This avoids the need to lookup the workspace in the ProjectPolicy scope.
class UserContext
  def initialize(membership)
    # ...
  end
end

class ProjectPolicy < ApplicationPolicy
  class Scope
    def resolve
      if user.admin?
        scope.all
      else
        scope.where(private: false)
      end
    end
  end
end

The membership record became my pundit user. But this is becoming limitted as my business logic grows. I'm thinking this was a bad idea.
Thanks again!


Create Connect wallet

May want to look this over and make changes based on front end

Thanks for sharing Jon
As it's a react component, I think It should be in the frontend repository

yes youre correct
On Thu, Apr 21, 2022 at 1:46 PM Hexdee @.***> wrote:

Thanks for sharing Jon
As it's a react component, I think It should be in the frontend repository
—
Reply to this email directly, view it on GitHub
https://github.com/Tescrow/Backend/pull/2#issuecomment-1105516163, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWQWXVDLQ3WFWA6GWMRKCFTVGGH7TANCNFSM5T757OWA
.
You are receiving this because you authored the thread.Message ID:
@.***>

i canceled the request
i suppose the frontend guys will want to look at it to fix with their
graphics and such
On Thu, Apr 21, 2022 at 1:47 PM Jonathan Mangin <
@.***> wrote:

yes youre correct
On Thu, Apr 21, 2022 at 1:46 PM Hexdee @.***> wrote:

Thanks for sharing Jon
As it's a react component, I think It should be in the frontend repository
—
Reply to this email directly, view it on GitHub
https://github.com/Tescrow/Backend/pull/2#issuecomment-1105516163, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWQWXVDLQ3WFWA6GWMRKCFTVGGH7TANCNFSM5T757OWA
.
You are receiving this because you authored the thread.Message ID:
@.***>


HNC: Update cert-controller

Cer-controller updated its WH support from v1beta1 to v1 (https://github.com/open-policy-agent/cert-controller/pull/34). HNC has already updated to v1 for WHs (https://github.com/kubernetes-sigs/multi-tenancy/pull/1434) but our cert-controller still supports v1beta1:
https://github.com/kubernetes-sigs/multi-tenancy/blob/a6253034e7d197089b6d3003c459c5f394355dd7/incubator/hnc/vendor/github.com/open-policy-agent/cert-controller/pkg/rotator/rotator.go#L72-L74
It's currently working because the builtin type is still serving both v1beta1 and v1.
However after K8s v1.22, v1beta1 WH api will be unserved. We need to upgrade our cert-controller so HNC won't break.

/assign @adrianludwin


Review

Thank you for your adjustments. I have made a few more. In addition to many details, the linter works again.
Since @echang15 hasn't done anything to the module for years, I'm afraid he won't merge my PR. That's why I created this PR for your repo.

@brucetony Did you saw this PR? :slightly_smiling_face:

I must have missed it, thanks for the update!


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_fd094b78-e000-4922-89f4-9c5aa9cae692/pull/1 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target_fd094b78-e000-4922-89f4-9c5aa9cae692.git

Docs Build status updates of commit 0de80ab:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-01-07-08-53-46 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-01-07-08-53-46

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


migrate to github boringssl  mirror

Why?

Why?

I guess it's because Google services has been filtered out (blocked) in some areas but Github has less such problem…… ;-)

I have a question for curiosity's sake: What about users which already have the submodule checked out? Will they have to run git submodule sync manually in order to notice the URL change? If yes, and if they don't do it, do we care if they remain on the old URL?

If yes, and if they don't do it, do we care if they remain on the old URL?

As long as the old URL still works fine, even if they remain it, there seems no harm

This PR has the label 'hold: cla required' and is stale: it has not been updated in 253 days. Note that this PR may be automatically closed in the future if no CLA is provided.  For CLA help see https://www.openssl.org/policies/cla.html

This PR has the label 'hold: cla required' and is stale: it has not been updated in 284 days. Note that this PR may be automatically closed in the future if no CLA is provided.  For CLA help see https://www.openssl.org/policies/cla.html

I guess @daleydeng isn't going to confirm that this change is trivial from a copyright perspective, so we will have to close it unmerged.

This PR has the label 'hold: cla required' and is stale: it has not been updated in 30 days. Note that this PR may be automatically closed in the future if no CLA is provided.  For CLA help see https://www.openssl.org/policies/cla.html

Closing due to inactive contributor


Invalid OAuth2 challenge-response authentication implementation

Kong OAuth2 plugin doesn't follow standards specified in

rfc2616 – HTTP/1.1
rfc2617 – HTTP Authentication and
rfc6750 – The OAuth 2.0 Authorization Framework: Bearer Token Usage.

The current OAuth2 plugin responses are forcing clients to implement hacks, and/or incorrect implementations for handling authentication challenges. This is especially problematic when using a generic client implementations that handle OAuth2 authentication challenges for services behind kong gateway.
Response to a request if it lacks any authentication information
Currently kong response is 403 Forbidden

The server understood the request, but is refusing to fulfill it.
Authorization will not help and the request SHOULD NOT be repeated.
rfc2616#section-10.4.4

This response is misleading. It doesn't even contradict the 403 by having any other why of telling the client that the endpoint requires user authentication e.g. WWW-Authenticate. If the client was unaware that authentication is necessary it may interpret the response as an application error and think that the request was otherwise legal.
Correct response would be 401 Unauthorized with WWW-Authenticate: Bearer realm="example" header

The request requires user authentication. The response MUST include a
WWW-Authenticate header field ... The client MAY repeat the
request with a suitable Authorization header field
rfc2616#section-10.4.2

Response to a request with Authorization header containing the access token that is expired, revoked, malformed, or invalid for other reasons
Kong's response to expired token the is 400 Bad Request:

The request could not be understood by the server due to malformed
syntax. The client SHOULD NOT repeat the request without
modifications.
rfc2616#section-10.4.1

and in case of an invalid token it is 403 Forbidden. Both responses do not follow the standards.
rfc2617#section-1.2 gives a very clear description on how the server should respond in this case.

If the origin server does not wish to accept the credentials sent
with a request, it SHOULD return a 401 (Unauthorized) response. The
response MUST include a WWW-Authenticate header field containing at
least one (possibly new) challenge applicable to the requested
resource.

In addition rfc6750#section-3.1 clearifies the error codes and specifies appropriate HTTP status codes.
Correct response would again be in both cases 401 Unauthorized with WWW-Authenticate: Bearer realm="example", error="invalid_token", error_description="The access token expired/The access token invalid/etc"
PR
I'm willing work on a PR to fix the issues mentioned above to make the OAuth2 plugin follow the standards. I'm aware that changing the implementation without maintaining backward compatible could be an issue, but trying to maintain BC can be very painful and unnecessarily bloat the code.
Please let me know what you think and then lets take it from there.

Hi @CyExy, thanks for the precious feedback. I can start working on this, but if you would like to push a PR that would certainly help.

trying to maintain BC can be very painful and unnecessarily bloat the code.

I agree.


not launching

File "/usr/local/bin/evillimiter", line 33, in
sys.exit(load_entry_point('evillimiter==1.5.0', 'console_scripts', 'evillimiter')())
File "/usr/local/lib/python3.8/dist-packages/evillimiter-1.5.0-py3.8.egg/evillimiter/evillimiter.py", line 167, in run
args = process_arguments(args)
File "/usr/local/lib/python3.8/dist-packages/evillimiter-1.5.0-py3.8.egg/evillimiter/evillimiter.py", line 70, in process_arguments
interface = netutils.get_default_interface()
File "/usr/local/lib/python3.8/dist-packages/evillimiter-1.5.0-py3.8.egg/evillimiter/networking/utils.py", line 13, in get_default_interface
gateways = netifaces.gateways()
AttributeError: module 'netifaces' has no attribute 'gateways'

have same issue, please help

have same issue, please help

Same issue please help

Same issue please help


New members of expression

This PR adds discord members of expression which returns a list of members in a certain object

I'm actually not sure why my commits from the other PR was commited in this branch


IMU captor position

Please provide the distance from scanning plane to internal sensor for the new scanner model (C3HD). This is needed to apply rotation quaternion to the CAD model. Finding this position ourselves is too difficult given inaccuracies in #2 .

is this for HD or HD3 model, if HD, then the internal CAD is provided here: https://github.com/clariusdev/cad/blob/main/hd.zip

Thanks for your answer.
We have the HD. I checked the CAD file and it does not contain any electronical component. There is also an imu.step file but I don't think it's for the C3HD.

Let us know where is the IMU sensor in our probe.

you can actually use the imu.step file, ignore the model of the scanhead, all of our devices have the same inside

Is it at the center of the red rectangle?

It would be much more reliable for every users if you indicated the position of the captor for all the HD and HD3 probes as you did for your v1.

check out the opposite side, the small solid rectangle.

Thanks for your reply. According to our measurement the captor is at -116mm from the imaging plane on the z-axis and we will assume it is centered on the x- and y-axis.


[$1000] Distance Request preview doesn't load

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

This issue is only present on the last main
This issue requires an immediate action. If you're posting a proposal, make sure you can open a PR within 6 hours of proposal approval
Action Performed:

Go to a workspace chat
Click "+" next to the text composer and click "Request money"
Click "Distance request" tab
Click "Start" and type any US address. Click any address suggestion.
Click "Finish" and and type any US address. Click any address suggestion. Make sure that the address is not the same address from Step 4.
Click "Next" and on the next page, click "Save"
Check that the distance request preview shown on the bottom of the workspace chat is infinitely loading.

This is a bug reported in this PR
Expected Result:
The preview should be loaded with correct data
Actual Result:
The preview doesn't load
Workaround:
N/A
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number:
Reproducible in staging?:
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
https://github.com/Expensify/App/assets/98560306/6c2021f4-9d11-4d7f-9591-dd259d600f11
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~010faa8f0f2edce9aa
Upwork Job ID:<PHONE_NUMBER>018233856
Last Price Increase: 2023-09-01

This is two buggy :)

@hayata-suenaga Have you removed the distance request from the latest main?

@shubham1206agra
No we didn't

@hayata-suenaga Can you link a video, just to confirm what I'm experience is same or not.

@b4s36t4 I updated the OP to include the video

Working on Safari

I found the problem @hayata-suenaga

tryResolveUrlFromApiRoot giving /receipts/w_5486f01c48729e148e9ba0354138110639a523bd.png for input of https://www.expensify.com/receipts/w_5486f01c48729e148e9ba0354138110639a523bd.png

But it should remain https://www.expensify.com/receipts/w_5486f01c48729e148e9ba0354138110639a523bd.png

@shubham1206agra No that's not the issue.

See this

Wait

Proposal
Please re-state the problem that we are trying to solve in this issue.
Distance request previews respond with 404s
What is the root cause of that problem?
The problem is that image URLs are always converted to the current root:
https://github.com/Expensify/App/blob/2fd54c1e47eea713b2c81631caccfb6816916d95/src/components/ReportActionItem/ReportActionItemImage.js#L41
But distance request previews are only visible through the original URL.
What changes do you think we should make in order to solve the problem?

I suggest we add a new prop, perhaps called keepOriginalSource that we pass down from the MoneyRequestPreview.
I think the correct way to pass this would be keepOriginalSource={props.transaction.category === 'Car'}, but I could be wrong.
If this is true in ReportActionItemImage, we do not replace the URL. The code would look something like:
let thumbnailSource = thumbnail;
if (!keepOriginalSource) {
    thumbnailSource = tryResolveUrlFromApiRoot(thumbnail);
}

What alternative solutions did you explore? (Optional)

Proposal ready for review @hayata-suenaga

This is what the result looks like:
https://github.com/Expensify/App/assets/138504087/3100c01b-d2bb-47ae-ae9e-4012963ca019

Proposal
Please re-state the problem that we are trying to solve in this issue.

Receipts image when fetched returns a 404 error.

What is the root cause of that problem?

Issue with the proxy server as we might not have access to /receipts endpoint on expensify server.

What changes do you think we should make in order to solve the problem?

Like show here for /chat-attachments we might need to handle /receipts endpoint as well.

https://github.com/Expensify/App/blob/2fd54c1e47eea713b2c81631caccfb6816916d95/web/proxy.js#L27-L38

Also I think we need to add a new entry in webpack.dev.js for /receipts endpoint just like we have for /chat-attachments endpoint.

https://github.com/Expensify/App/blob/2fd54c1e47eea713b2c81631caccfb6816916d95/config/webpack/webpack.dev.js#L24-L26
What alternative solutions did you explore? (Optional)

Quick fix could be like this:

https://github.com/Expensify/App/blob/2fd54c1e47eea713b2c81631caccfb6816916d95/src/components/ReportActionItem/ReportActionItemImage.js#L41
- const thumbnailSource = tryResolveUrlFromApiRoot(thumbnail);
+ let thumbnailSource = tryResolveUrlFromApiRoot(thumbnail)
+ if (thumbnailSource.startsWith('/receipts')) {
+     thumbnailSource = thumbnailSource.replace('/receipts', CONST.EXPENSIFY_URL + '/receipts');
}

My confusion creating problems

Proposal
Please re-state the problem that we are trying to solve in this issue.
Distance Request preview doesn't load
What is the root cause of that problem?
There are two main problems.

We may not have access to the /receipts endpoint on the Expensify server because we are currently accessing it on localhost.
We are encountering a lot of console errors due to incorrect usage of React.memo on the Image component.

What changes do you think we should make in order to solve the problem?
We need to change some code in Image component.
https://github.com/Expensify/App/blob/e460459ce0393edfcbda8ddbc5197a84b38bbac8/src/components/Image/index.js#L55
return prevProps.source=== nextProps.source || prevProps.source.uri === nextProps.source.uri;

https://github.com/Expensify/App/blob/e460459ce0393edfcbda8ddbc5197a84b38bbac8/src/components/Image/index.js#L22
if (propsSource.uri.startsWith('/receipts'))
  imageUri = propsSource.uri.replace('/receipts', `${CONST.EXPENSIFY_URL}/receipts`);

return {uri: `${imageUri}?encryptedAuthToken=${encodeURIComponent(authToken)}`};

What alternative solutions did you explore? (Optional)
n/a
Result

distance request preview works fine on staging and production.

@jeet-dhandha's proposal looks promising let's implement it

I will create a PR in short while 👍 .

@b4s36t4 how do you think about tons of console errors from your comment? Is this not a bug?

Sorry, @b4s36t4 the url was incorrect. could you please take a look again please? 🙏

@napster125 , let sync on slack not on GH and that too different issue. Ping on slack. Mahesh Vagicherla is my slack name.

I will add screenshots in a while. 👍

Done with PR. Please review as soon as possible.

@strepanier03 for clarification, we didn't use the help of @sobitneupane in selecting the proposal.
For PR review, we're using the help of @allroundexperts
So I unassigned @sobitneupane and @allroundexperts as C+ for this issue

I added this to the project as low priority because it only occurs in the dev environment for contributors testing against staging, due to the proxy.

@JmillsExpensify - Payment has been requested by @allroundexperts and the payment summary can be found in this comment.

@JmillsExpensify - I'm unsure if this has been paid but a friendly bump just in case. Thank you!

I see a comment from Jason in Slack that this doesn't need to stay open so I'm going to close it. Jason has the request.

$1,500 payment for @allroundexperts approved based on BZ summary.


Init Husky

Run npm install and npm run prepare to install the hooks. Boom!

I'm not entirely sure if it's worth adding node dependency just to set a pre-commit hook. We already have a Python dependency so maybe we could use something in Python instead. Or simply do it in a Makefile


use_cookie causes console warnings due to .get()

This warning appears on every page because I am using cookies via use_cookie() (thank you by the way, it works wonderfully).
I believe this issue is because somewhere there is a cookie.get() instead of the no desired cookie().
:1, you access a signal or memo (defined at /.cargo/registry/src/index.crates.io-6f17d22bba15001f/leptos-use-0.10.10/src/use_cookie.rs:180:32) outside a reactive tracking context. This might mean your app is not responding to changes in signal values in the way you expect.

Here’s how to fix it:

1. If this is inside a `view!` macro, make sure you are passing a function, not a value.
  ❌ NO
{x.get() * 2}

  ✅ YES
{move || x.get() * 2}

2. If it’s in the body of a component, try wrapping this access in a closure:
  ❌ NO  let y = x.get() * 2
  ✅ YES let y = move || x.get() * 2.

3. If you’re *trying* to access the value without tracking, use `.get_untracked()` or `.with_untracked()` instead.

Thanks for the kind words. Could you post the code where you use it?

this is the only location where I use the cookie:
pub fn get_signed_in_signals() -> (Signal<Option<bool>>, WriteSignal<Option<bool>>) {
    let (cookie, set_cookie) = use_cookie_with_options::<bool, FromToStringCodec>(
        "signed_in",
        UseCookieOptions::default()
            .max_age(86400_000) // one day
            .same_site(SameSite::Strict),
    );

    if cookie().is_none() {
        set_cookie.set(Some(false));
    }

    (cookie, set_cookie)
}

The warning should be gone if you change the line
    if cookie().is_none() {

to
    if cookie.get_untracked().is_none() {

Or make sure that you use your function always in a reactive context which I'm pretty sure you're not doing right now.

Ah, okay! So I am using the setters and getters in a reactive context but since the instantiation of them is not reactive, the warning is shown. Warnings gone :)
Odd that the warning didn't reference the file where it came from! Sorry about that.

No worries! I'm glad you were able to find it.


Flatbuffer inference support via ttrt

Need to be able to support runtime of flatbuffers which includes items like generating random data

merged: 5d7fa3ab5ca3d9b7f1b9daf4ebbf807713d2fa47


Added OPP support for processing switch matrices.  This includes adding

OPPMatrixCard and support for READ_MATRIX_INP command.  Small amounts of code cleanup so processing is similar in all cases.  Insure msgs contain enough bytes for crc8 before verifying crc8 is correct.  Removed use of CFG_SOL_USE_SWITCH since using SET_SOL_INP_CMD.  Added config2.yaml for
some additional unit tests.

Thanks for adding matrix switches and cleaning up the rest of the code!


Refractor File Class

File Class has to be refactored or deleted.  It provides no or very little use

It provides use for avatar changes and when the channel objects will have a send_message() method, it will be used to send files


Q Trigger create flow - help text is missing

note %%
even if these are parameters, the sentence itself is not clear

This needs to be fixed in the templates repo. Naren and Matthew should work on this.

@mattchenderson Could you move this to the templates repo


[20200229] - Local Environment in Window

#28  Please include the following information
I get these error for
docker-compose down || true;
Removing fastpages_converter_1 ... done
Removing fastpages_fastpages_1 ... done
Removing fastpages_watcher_1   ... done
Removing fastpages_jekyll_1    ... done
Removing fastpages_jupyter_1   ... done
Removing network fastpages_default
docker-compose up
Creating network "fastpages_default" with the default driver
Creating fastpages_jekyll_1    ... done
Creating fastpages_watcher_1   ... done
Creating fastpages_jupyter_1   ... done
Creating fastpages_converter_1 ... done
Creating fastpages_fastpages_1 ... done
Attaching to fastpages_converter_1, fastpages_jupyter_1, fastpages_watcher_1, fastpages_fastpages_1, fastpages_jekyll_1
converter_1  | standard_init_linux.go:211: exec user process caused "no such file or directory"
fastpages_1  | standard_init_linux.go:211: exec user process caused "no such file or directory"

Then I find out that multiple links are link to /fastpages/...
Which caused the problem as this does not match my directory.
My current solution is to

Steps to reproduce the problem
Window 10
make server
A link to your repository where error is occuring
The Two dockers cannot start successfully
standard_init_linux.go:211: exec user process caused "no such file or directory

I does some more digging, which I found notebook post are not generated probably. (As localhost:4000 show only markdown post but not notebooks).
I can reproduce this error message by manually edit notebook to force it regenerate notebook post.
jekyll_1     |     Server address: http://<IP_ADDRESS>:4000/
jekyll_1     |   Server running... press ctrl-c to stop.
watcher_1    | /bin/sh: 1: /fastpages/action_entrypoint.sh: not found
watcher_1    | /bin/sh: 1: /fastpages/action_entrypoint.sh: not found
I then try to login to watcher_1 docker to check if this file exist, it does.

Then I did some more digging by manually execute the script

It seems to related to some special token with issue editing in Windows. Therefore, I change to save from "CRLF" to "LF" but got no luck yet. And it complains about something called directory2post.py which I am not sure where is this file (can't find in fastpages nor nbdev)
https://stackoverflow.com/questions/51888625/docker-r-command-not-found-on-windows

Maybe I am not cleaning the token correctly, I clean the token in my own directory, I am not sure if the files in Docker are pulled from the directory or somewhere else.

It seems that I have to clean up in watcher_1 docker directly with commands
sed -i 's/\r//g' /fastpages/action_entrypoint.sh
sed -i 's/\r//g' /fastpages/word2post.sh

I can successfully to generate notebook post now!

But I can't create a fix PR from my side as I don't know where does the file in /fastpages/*.sh come from. I tried clean up on my local repository and re-run make server but does not work, it only works when I clean up in docker manually.

I think we need to focus on reproducing this error
converter_1  | standard_init_linux.go:211: exec user process caused "no such file or directory"
fastpages_1  | standard_init_linux.go:211: exec user process caused "no such file or directory"

If you clone this repo fresh and run make server  Do you get this error?

I wonder if I can add a test in Windows to the CI.  I think I can

For the record I am running this in Window 10 with docker, I tried WSL with no luck.
Here is how I reproduce
git clone https://github.com/fastai/fastpages.git
cd fastpages
make build
make server

Thanks @noklam this is very helpful.  We will keep this issue open until we figure this out.  Thanks for bringing this to our attention

@noklam I'll go ahead and close this issue since I merged your PR.  Please feel free to reopen this issue if the problem persists.
Thank you

Thanks @hamelsmu

I was down with a cold most of the week ;( Thank you @noklam for reporting and fixing it!
On Sat, Feb 29, 2020 at 9:50 AM, noklam <<EMAIL_ADDRESS>> wrote:

Thanks @ hamelsmu ( https://github.com/hamelsmu )
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (
https://github.com/fastai/fastpages/issues/144?email_source=notifications&email_token=AAAKU4OQI56DKNVNLLJO4GDRFFFGPA5CNFSM4K6VKNQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENMA65Y#issuecomment-592973687
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AAAKU4JI5F6WJSDPC6AKXTTRFFFGPANCNFSM4K6VKNQA
).

@xnutsive thx! 😃

Hello, I still have an issue with this one. I tried to build a local environment based on my repo (https://github.com/arisbw/p/) but didn't succeed. And also I tried to make a local server based on the fresh repo of fastpages (ref: https://github.com/fastai/fastpages/issues/144#issuecomment-592964219) but didn't find any success either.
Instead of using https://<IP_ADDRESS>:4000 as the preview to my page, the server address was directed to http://<IP_ADDRESS>:4000/
I'm using WSL2 with docker installed.
Here is the screenshot when I tried to run make server based on the original repo of fastpages:

@arisbw what's wrong though?  I don't see anything that is broken?  You can go to that URL and see fastpages

Sorry that I didn't make it clear. I did try to access though to the server address: http://<IP_ADDRESS>:4000/ but I didn't see the site.
Here is the screenshot when I attempted to access the address:

Not sure what’s wrong it’s perhaps your local networking or Docker settings
On Fri, Oct 30, 2020 at 7:31 AM Aris Budi Wibowo<EMAIL_ADDRESS>wrote:

Sorry that I didn't make it clear. I did try to access though to the
server address: http://<IP_ADDRESS>:4000/ but I didn't see the site.
Here is the screenshot when I attempted to access the address:
[image: image]
https://user-images.githubusercontent.com/4155427/97717156-0f3f8f00-1af7-11eb-80e4-5f70c809bb57.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fastai/fastpages/issues/144#issuecomment-719586526,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALKJEU7VXJESBVPS4QM643SNLE3DANCNFSM4K6VKNQA
.

Hello, I'm quite a noob at networking and I had a similar problem as @arisbw. From what I understand the server runs locally so it automatically finds the address(localhost).
I fixed the problem by addressing another issue. Docker couldn't find the correct base url so I commented line 37 in the _config.yml

baseurl: "/Blogging" # the subpath of your site, e.g. "/blog".

This fixed both issues for me.

I hope this helps.


set default DB_URI to correct value

In the import_csv PR (https://github.com/apache/incubator-superset/pull/3643) I accidentally changed the defult SQLALCHEMY_DATABASE_URI. This corrects that mistake.
@graceguo-supercat @mistercrunch

Already opened a PR two hours ago :) https://github.com/apache/incubator-superset/pull/3955


RFC 6648: SHOULD NOT prefix their parameter names with "X-"

Angular has built-in support for sending XSRF token back in a header. It uses header X-XSRF-TOKEN. RFC 6648 recommends against using X- prefix for non-standard HTTP headers. Would be nice of angular to switch to header XSRF-Token, recommend its use in the documentation and link to RFC 6648.

I am not sure about changing this. RFC 6648 is not a standard nor a proposed standard (as oppose to the http/1.1 standard that defines all headers starting with X- as extensions). On top, this can be configured using $httpProvider.defaults.xsrfHeaderName. If there was a standard that defines this, then it would be a clear cut that this needs to change, but this is not the case

The points in this RFC look quite convincing to me.

Hi, sorry if I was not clear. The question is "is the chance convincing enough to break the app for people that check the use X-XSRF-TOKEN now?". This is a hard question as there is no standard that would make any header name better than the other. Some libraries default to CSRF-TOKEN (check that there is a C and not an X) and some to other header names.
If a standard or a proposed standard was in place, then this is easier to decide, but there is no such a standard. Sorry, I still feel that this change is not worth breaking existing applications. Anyhow, would not oppose if other members decide that this is right and the way to go

I understand your concerns, but it's definitely not the first breaking change Angular introduces. Also it's not subtle: all the POSTs will stop working and it's hard to miss that during the upgrade. The fix on the server side is also minimal: just a string change.
Would you mind implementing that at least in Angular 2.0 which already promises to break almost everything?

Hi, Angular 2 is a different story, but they are tracking the issues at https://github.com/angular/angular

Thanks for the link @lgalfaso!

At this point in the AngularJS lifecycle, this would be a quite an annoying BC with little value, so we should not do it.


Disable source mapping

This PR disables source mapping for production.

@darrt2008 Do you like this? Check the site.

Visit the preview URL for this PR (updated for commit d27cfd4):
https://to-do-46--pr27-disable-source-mappi-ubib4bnf.web.app
(expires Sat, 16 Apr 2022 04:54:42 GMT)
🔥 via Firebase Hosting GitHub Action 🌎

Don't check this thing, check the main site.


Update README.md

fixes #6

Lgtm, there is one test failing though.

@majorz @nazrhom can you please look into the rust-test check failing here?

@curcuz This is the formatting for Rust code that gives warnings. With Rust before a commit is done cargo fmt needs to be run that will automatically format the code. At the other end the CI will whether it is formatted well and will complain if not so. This is probably caused by difference in versions as not long ago the formatting rules changed often before they were stabilized. I will check now how we can fix this properly.

The formatting warnings are gone, but the package does not build because of a dependency update. Will have to fix this one as well.

hey @majorz any chance you have news on this? or who else can I bother?

hey @majorz any chance you have news on this? or who else can I bother?

ping @majorz

ping @majorz


Make flyer for Tonomy ID + HR

[ ] Make a flyer for Tonomy ID + HR;
[ ] Handover to Mike Chris, Cai and spandana, so they know where to find it.

https://www.canva.com/design/DAFchRs4Onc/iGv65focKGdY9yS0Bfasog/edit?utm_content=DAFchRs4Onc&utm_campaign=designshare&utm_medium=link2&utm_source=sharebutton

It's way too busy.
This is only a simple issue of hierarchy. Hierarchy tells you where to look.
The illustrations are as loud as the photoshopped image.  Either find a nonphotoshopped image of people to make it more blended or put the feature illustrations on another page.
The paragraph could be easier to read. The light weight letters for the white background dont work.
Page 2: Remove the illustration in the blue band in the middle of the page as well as the black ribbon thingy. The flyers are suppose to be clean.
The screens of Tonomy dont fit on the mobile frame making the flyer cheap looking.
Exchange Customisable for your needs to just customisable. Customisable alludes to it being fit to one's needs. Also the other 2 statements are 1 word so this should be as well.
The description text below the Scalable, Privacy and Customisable is too thin, so its hard to read. Increase the weight. The green with the black lettering- How do we do this? Change to How is this done?
The bottom link should be in capital letters and the for more information text should be put together with the link and centered.

Thank you the feedback has been added to an new file for design feedback; https://docs.google.com/spreadsheets/d/1ITomObhhhHkPMFDDZFw5ZGNfDlpKiB7rtOA9QQwQNB0/edit?usp=sharing

@kamitor, nice job, very organized.


Added table of contents to the readme file

Title and Issue number
Title: Added table of contents to the readme file
Issue No.: #184
Close #184
Screenshots/Video (mandatory)

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] I have created a helpful and easy-to-understand README.md if it's a new page/tech stack.
[X] I have gone through the contributing.md file before contributing.

Additional context (Mandatory):
Are you contributing under any Open-source programme?

[X] I'm a GSSOC-EXT contributor
[X] I'm a HACKTOBERFEST contributor

Please review and merge my code under gssoc-ext and hackoberfest.

Hyy @T-Fathima thanks for your conrtributions bitbox


[BUG] Extension was rejected by Google due to the Google Analytic code.

What happened?
Violation: The Manifest V3 product contains remote code.
.plasmo/cache/remote-code/e3b1320b18219fdb.js:
(function(a){function b(){m=F.documentElement.getAttribute("data-tag-assistant-present");Tt(m)&&
(l=h.Uj)}if(!E["__TAGGY_INSTALLED"]){var c=!1;if(F.referrer){var d=qh(F.referrer);c="cct.google"===kh(d,"host")}
if(!c){var e=sj("googTaggyReferrer");c=e.length&&e[0].length}c&&
(E["__TAGGY_INSTALLED"]=!0,Xb("https://cct.google/taggy/agent.js"))}if(Je)a();
else{var f=function(u){var v="GTM",w="GTM";De?(v="OGT",w="GTAG"):Je&&(w=v="OPT");
var x=E["google.tagmanager.debugui2.queue"];x||(x=[],

https://cct.google/taggy/agent.js

Version
Latest
What OS are you seeing the problem on?
No response
What browsers are you seeing the problem on?
No response
Relevant log output
No response
(OPTIONAL) Contribution

[ ] I would like to fix this BUG via a PR

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

I run into this same issue the other day, and had to switch to using GA Measurement (ref: https://github.com/PlasmoHQ/examples/tree/main/with-google-analytics-measurement)
I don't know if Plasmo will be able to fix this, since the remote code is actually loaded by the JS from GA. But regardlessly, I think some comments/documents should be added about this, including:

https://docs.plasmo.com/quickstarts/with-google-analytics
https://github.com/PlasmoHQ/examples/tree/main/with-google-analytics